dont use a subshell in tests cases if you intend to signal it...

locustio · Nov 18, 2023 · d16ae25 · d16ae25
1 parent aa170aa
commit d16ae25
Showing 1 changed file with 10 additions and 2 deletions.
diff --git a/locust/test/test_main.py b/locust/test/test_main.py
@@ -1837,8 +1837,16 @@ def test_processes_ctrl_c(self):
 
         with mock_locustfile() as mocked:
             proc = subprocess.Popen(
-                f"locust -f {mocked.file_path} --processes 4 --headless -L DEBUG",
-                shell=True,
+                [
+                    "locust",
+                    "-f",
+                    mocked.file_path,
+                    "--processes",
+                    "4",
+                    "--headless",
+                    "-L",
+                    "DEBUG",
+                ],
                 stdout=PIPE,
                 stderr=PIPE,
                 text=True,