p

Signed-off-by: kevin <[email protected]>

scripts/pipeline_generator/utils.py

@@ -39,8 +39,8 @@ def get_agent_queue(no_gpu: Optional[bool], gpu_type: Optional[str], num_gpus: O
 def get_full_test_command(test_commands: List[str], step_working_dir: str) -> str:
     """Convert test commands into one-line command with the right directory."""
     working_dir = step_working_dir or DEFAULT_WORKING_DIR
-    test_commands_str = "; ".join(test_commands)
-    return f"cd {working_dir}; {test_commands_str}"
+    test_commands_str = ";\n".join(test_commands)
+    return f"cd {working_dir};\n{test_commands_str}"
 
 
 def get_multi_node_test_command(

scripts/tests/pipeline_generator/test_utils.py

@@ -27,9 +27,9 @@ def test_get_agent_queue(no_gpu: bool, gpu_type: str, num_gpus: int, expected_re
 @pytest.mark.parametrize(
     ("test_commands", "step_working_dir", "expected_result"),
     [
-        (["echo 'hello'"], None, "cd /vllm-workspace/tests; echo 'hello'"),
-        (["echo 'hello'"], "/vllm-workspace/tests", "cd /vllm-workspace/tests; echo 'hello'"),
-        (["echo 'hello1'", "echo 'hello2'"], None, "cd /vllm-workspace/tests; echo 'hello1'; echo 'hello2'"),
+        (["echo 'hello'"], None, "cd /vllm-workspace/tests;\necho 'hello'"),
+        (["echo 'hello'"], "/vllm-workspace/tests", "cd /vllm-workspace/tests;\necho 'hello'"),
+        (["echo 'hello1'", "echo 'hello2'"], None, "cd /vllm-workspace/tests;\necho 'hello1';\necho 'hello2'"),
     ],
 )
 def test_get_full_test_command(test_commands: List[str], step_working_dir: str, expected_result: str):