fix path error

Author: EmmonsCurse

.github/workflows/_unit_test_coverage.yml

@@ -355,7 +355,7 @@ jobs:
             if [ -f "${filename}" ];then
               echo "Failed test cases:"
               cat "${filename}"
-              echo "unittest_logs_url=${UNIT_TEST_LOGS_URL}"
+              echo "unittest_logs_url=${unittest_logs_url}"
             fi
             exit "$TEST_EXIT_CODE"
           fi

scripts/coverage_run.sh

@@ -54,7 +54,7 @@ run_test_with_logging() {
     local log_prefix=$2
     local status
 
-    echo "Running: $test_file"
+    echo "Running pytest file: $test_file"
 
     # Create isolated log directory for this test to avoid race conditions
     # Format: unittest_logs/<test_dir>/<test_file_base>/log
@@ -97,9 +97,36 @@ run_test_with_logging() {
             grep -Rni --color=auto "error" "${isolated_log_dir}" || true
         fi
 
+        # print all server logs
+        server_logs=("${run_path}"/*.log)
+        if [ "${#server_logs[@]}" -gt 0 ]; then
+            for server_log in "${server_logs[@]}"; do
+                # skip failed_tests_file
+                [[ "$(basename "$server_log")" == "$failed_tests_file" ]] && continue
+                if [ -f "${server_log}" ]; then
+                    echo
+                    echo "---------------- ${server_log} (last 100 lines) ----------------"
+                    tail -n 100 "${server_log}" || true
+                    echo "---------------------------------------------------------------"
+                fi
+            done
+        else
+            echo "No *.log files found"
+        fi
+
         echo "======================================================="
     fi
 
+     # if passed, remove the isolated log directory and server logs
+     if [ "$status" -eq 0 ]; then
+         rm -rf "${isolated_log_dir}" || true
+         # Clean up server logs in run_path on pass
+         for f in "${run_path}"/*.log; do
+             [[ "$(basename "$f")" != "${failed_tests_file}" ]] && rm -f "$f" || true
+         done
+     fi
+
+
     # Clean up port-related processes
     if [ -n "$FD_CACHE_QUEUE_PORT" ]; then
         ps -ef | grep "${FD_CACHE_QUEUE_PORT}" | grep -v grep | awk '{print $2}' | xargs -r kill -9 || true
@@ -297,25 +324,25 @@ echo "===================================="
 
 # Exit with error and package logs if there were failures
 if [ "$failed_count" -ne 0 ]; then
-  echo "Failed test cases are listed in $failed_tests_file"
-  cat "$failed_tests_file"
+    echo "Failed test cases are listed in $failed_tests_file"
+    cat "$failed_tests_file"
 
-  # Only package logs when there are failures
-  echo "===================================="
-  echo "Step 5: Packaging logs (only on failure)"
-  echo "===================================="
+    # Only package logs when there are failures
+    echo "===================================="
+    echo "Step 5: Packaging logs (only on failure)"
+    echo "===================================="
 
-  if [ -d "${run_path}/unittest_logs" ]; then
-    tar -czf "${run_path}/unittest_logs.tar.gz" -C "${run_path}" unittest_logs
-    echo "unittest_logs packaged to: ${run_path}/unittest_logs.tar.gz"
-    ls -lh "${run_path}/unittest_logs.tar.gz"
-  else
-    echo "No unittest_logs directory found."
-  fi
+    if [ -d "${run_path}/unittest_logs" ]; then
+        tar -czf "${run_path}/unittest_logs.tar.gz" -C "${run_path}" unittest_logs
+        echo "Logs packaged to: ${run_path}/unittest_logs.tar.gz"
+        ls -lh "${run_path}/unittest_logs.tar.gz"
+    else
+        echo "No unittest_logs directory found."
+    fi
 
-  echo "===================================="
+    echo "===================================="
 
-  exit 8
+    exit 8
 fi
 
 echo "All tests passed!"

tests/e2e/test_EB_Lite_serving.py

@@ -1389,7 +1389,8 @@ def test_streaming_chat_finish_reason(openai_client):
 
 def test_profile_reset_block_num():
     """测试profile reset_block_num功能，与baseline diff不能超过5%"""
-    log_file = "./log/config.log"
+    log_dir = os.getenv("FD_LOG_DIR", "log")
+    log_file = os.path.join(log_dir, "config.log")
     baseline = 31446
 
     if not os.path.exists(log_file):

tests/e2e/test_EB_VL_Lite_serving.py

@@ -734,7 +734,8 @@ def test_chat_with_response_max_tokens(openai_client):
 
 def test_profile_reset_block_num():
     """测试profile reset_block_num功能，与baseline diff不能超过5%"""
-    log_file = "./log/config.log"
+    log_dir = os.getenv("FD_LOG_DIR", "log")
+    log_file = os.path.join(log_dir, "config.log")
     baseline = 40000
 
     if not os.path.exists(log_file):

tests/e2e/test_Qwen2-7B-Instruct_serving.py

@@ -612,7 +612,8 @@ def test_streaming(openai_client, capsys):
 
 def test_profile_reset_block_num():
     """测试profile reset_block_num功能，与baseline diff不能超过5%"""
-    log_file = "./log/config.log"
+    log_dir = os.getenv("FD_LOG_DIR", "log")
+    log_file = os.path.join(log_dir, "config.log")
     baseline = 32562
 
     if not os.path.exists(log_file):

tests/e2e/test_Qwen2_5_VL_serving.py

@@ -430,7 +430,8 @@ def test_streaming_chat_with_return_token_ids(openai_client, capsys):
 
 def test_profile_reset_block_num():
     """测试profile reset_block_num功能，与baseline diff不能超过15%"""
-    log_file = "./log/config.log"
+    log_dir = os.getenv("FD_LOG_DIR", "log")
+    log_file = os.path.join(log_dir, "config.log")
     baseline = 30000
 
     if not os.path.exists(log_file):

tests/entrypoints/openai/test_multi_api_server.py

@@ -99,7 +99,8 @@ def test_start_servers_success(self, mock_is_port_available, mock_popen):
         # Verify environment variables are set correctly
         first_call_kwargs = mock_popen.call_args_list[0][1]
         self.assertIn("env", first_call_kwargs)
-        self.assertEqual(first_call_kwargs["env"]["FD_LOG_DIR"], "log/log_0")
+        log_dir = os.getenv("FD_LOG_DIR", "log")
+        self.assertEqual(first_call_kwargs["env"]["FD_LOG_DIR"], os.path.join(log_dir, "log_0"))
 
     @patch("fastdeploy.entrypoints.openai.multi_api_server.is_port_available")
     def test_check_param_success(self, mock_is_port_available):