test_warning_with_lower_user_count_than_fixed_count: dont use shell=True

locustio · Aug 1, 2024 · 5dbf27f · 5dbf27f
1 parent 6cfefdf
commit 5dbf27f
Showing 1 changed file with 10 additions and 13 deletions.
diff --git a/locust/test/test_main.py b/locust/test/test_main.py
@@ -1047,21 +1047,18 @@ def t(self):
         )
         with mock_locustfile(content=LOCUSTFILE_CONTENT) as mocked:
             proc = subprocess.Popen(
-                " ".join(
-                    [
-                        "locust",
-                        "-f",
-                        mocked.file_path,
-                        "--headless",
-                        "--run-time",
-                        "1s",
-                        "-u",
-                        "3",
-                    ]
-                ),
+                [
+                    "locust",
+                    "-f",
+                    mocked.file_path,
+                    "--headless",
+                    "--run-time",
+                    "1s",
+                    "-u",
+                    "3",
+                ],
                 stderr=STDOUT,
                 stdout=PIPE,
-                shell=True,
                 text=True,
             )