Remove shell option

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

Author: YongGoose

tests/pytest/package_manager/test_android.py

@@ -15,7 +15,7 @@
 @pytest.mark.ubuntu
 def test_ubuntu(input_path, output_path, extra_args):
     command = f"fosslight_dependency -p {input_path} -o {output_path} {extra_args}"
-    result = subprocess.run(command, shell=True, capture_output=True, text=True)
+    result = subprocess.run(command, capture_output=True, text=True)
     assert result.returncode == 0, f"Command failed: {command}\nstdout: {result.stdout}\nstderr: {result.stderr}"
     assert any(os.scandir(output_path)), f"Output file does not exist: {output_path}"
 
@@ -26,6 +26,6 @@ def test_ubuntu(input_path, output_path, extra_args):
 @pytest.mark.windows
 def test_windows(input_path, output_path):
     command = f"{DIST_PATH} -p {input_path} -o {output_path}"
-    result = subprocess.run(command, shell=True, capture_output=True, text=True)
+    result = subprocess.run(command, capture_output=True, text=True)
     assert result.returncode == 0, f"Command failed: {command}\nstdout: {result.stdout}\nstderr: {result.stderr}"
     assert any(os.scandir(output_path)), f"Output file does not exist: {output_path}"

tests/pytest/package_manager/test_cocoapods.py

@@ -13,6 +13,6 @@
 @pytest.mark.ubuntu
 def test_ubuntu(input_path, output_path, extra_args):
     command = f"fosslight_dependency -p {input_path} -o {output_path} {extra_args}"
-    result = subprocess.run(command, shell=True, capture_output=True, text=True)
+    result = subprocess.run(command, capture_output=True, text=True)
     assert result.returncode == 0, f"Command failed: {command}\nstdout: {result.stdout}\nstderr: {result.stderr}"
     assert any(os.scandir(output_path)), f"Output file does not exist: {output_path}"

tests/pytest/package_manager/test_gradle.py

@@ -16,7 +16,7 @@
 @pytest.mark.ubuntu
 def test_ubuntu(input_path, output_path):
     command = f"fosslight_dependency -p {input_path} -o {output_path}"
-    result = subprocess.run(command, shell=True, capture_output=True, text=True)
+    result = subprocess.run(command, capture_output=True, text=True)
     assert result.returncode == 0, f"Command failed: {command}\nstdout: {result.stdout}\nstderr: {result.stderr}"
     assert any(os.scandir(output_path)), f"Output file does not exist: {output_path}"
 
@@ -28,6 +28,6 @@ def test_ubuntu(input_path, output_path):
 @pytest.mark.windows
 def test_windows(input_path, output_path):
     command = f"{DIST_PATH} -p {input_path} -o {output_path} -m gradle"
-    result = subprocess.run(command, shell=True, capture_output=True, text=True)
+    result = subprocess.run(command, capture_output=True, text=True)
     assert result.returncode == 0, f"Command failed: {command}\nstdout: {result.stdout}\nstderr: {result.stderr}"
     assert any(os.scandir(output_path)), f"Output file does not exist: {output_path}"

tests/pytest/package_manager/test_helm.py

@@ -13,6 +13,6 @@
 @pytest.mark.ubuntu
 def test_ubuntu(input_path, output_path, extra_args):
     command = f"fosslight_dependency -p {input_path} -o {output_path} {extra_args}"
-    result = subprocess.run(command, shell=True, capture_output=True, text=True)
+    result = subprocess.run(command, capture_output=True, text=True)
     assert result.returncode == 0, f"Command failed: {command}\nstdout: {result.stdout}\nstderr: {result.stderr}"
     assert any(os.scandir(output_path)), f"Output file does not exist: {output_path}"

tests/pytest/package_manager/test_maven.py

@@ -16,7 +16,7 @@
 @pytest.mark.ubuntu
 def test_ubuntu(input_path, output_path):
     command = f"fosslight_dependency -p {input_path} -o {output_path}"
-    result = subprocess.run(command, shell=True, capture_output=True, text=True)
+    result = subprocess.run(command, capture_output=True, text=True)
     assert result.returncode == 0, f"Command failed: {command}\nstdout: {result.stdout}\nstderr: {result.stderr}"
     assert any(os.scandir(output_path)), f"Output file does not exist: {output_path}"
 
@@ -27,6 +27,6 @@ def test_ubuntu(input_path, output_path):
 @pytest.mark.windows
 def test_windows(input_path, output_path):
     command = f"{DIST_PATH} -p {input_path} -o {output_path} -m maven"
-    result = subprocess.run(command, shell=True, capture_output=True, text=True)
+    result = subprocess.run(command, capture_output=True, text=True)
     assert result.returncode == 0, f"Command failed: {command}\nstdout: {result.stdout}\nstderr: {result.stderr}"
     assert any(os.scandir(output_path)), f"Output file does not exist: {output_path}"

tests/pytest/package_manager/test_npm.py

@@ -14,6 +14,6 @@
 @pytest.mark.ubuntu
 def test_ubuntu(input_path, output_path, extra_args):
     command = f"fosslight_dependency -p {input_path} -o {output_path} {extra_args}"
-    result = subprocess.run(command, shell=True, capture_output=True, text=True)
+    result = subprocess.run(command, capture_output=True, text=True)
     assert result.returncode == 0, f"Command failed: {command}\nstdout: {result.stdout}\nstderr: {result.stderr}"
     assert any(os.scandir(output_path)), f"Output file does not exist: {output_path}"

tests/pytest/package_manager/test_nuget.py

@@ -21,7 +21,7 @@
 @pytest.mark.ubuntu
 def test_ubuntu(input_path, output_path):
     command = f"fosslight_dependency -p {input_path} -o {output_path}"
-    result = subprocess.run(command, shell=True, capture_output=True, text=True)
+    result = subprocess.run(command, capture_output=True, text=True)
     assert result.returncode == 0, f"Command failed: {command}\nstdout: {result.stdout}\nstderr: {result.stderr}"
     assert any(os.scandir(output_path)), f"Output file does not exist: {output_path}"
 
@@ -33,6 +33,6 @@ def test_ubuntu(input_path, output_path):
 @pytest.mark.windows
 def test_windows(input_path, output_path):
     command = f"{DIST_PATH} -p {input_path} -o {output_path}"
-    result = subprocess.run(command, shell=True, capture_output=True, text=True)
+    result = subprocess.run(command, capture_output=True, text=True)
     assert result.returncode == 0, f"Command failed: {command}\nstdout: {result.stdout}\nstderr: {result.stderr}"
     assert any(os.scandir(output_path)), f"Output file does not exist: {output_path}"

tests/pytest/package_manager/test_pub.py

@@ -16,7 +16,7 @@
 @pytest.mark.ubuntu
 def test_ubuntu(input_path, output_path):
     command = f"fosslight_dependency -p {input_path} -o {output_path}"
-    result = subprocess.run(command, shell=True, capture_output=True, text=True)
+    result = subprocess.run(command, capture_output=True, text=True)
     assert result.returncode == 0, f"Command failed: {command}\nstdout: {result.stdout}\nstderr: {result.stderr}"
     assert any(os.scandir(output_path)), f"Output file does not exist: {output_path}"
 
@@ -29,6 +29,6 @@ def test_ubuntu(input_path, output_path):
 @pytest.mark.windows
 def test_windows(input_path, output_path, extra_args):
     command = f"{DIST_PATH} -p {input_path} -o {output_path} {extra_args}"
-    result = subprocess.run(command, shell=True, capture_output=True, text=True)
+    result = subprocess.run(command, capture_output=True, text=True)
     assert result.returncode == 0, f"Command failed: {command}\nstdout: {result.stdout}\nstderr: {result.stderr}"
     assert any(os.scandir(output_path)), f"Output file does not exist: {output_path}"

tests/pytest/package_manager/test_pypi.py

@@ -17,7 +17,7 @@
 @pytest.mark.ubuntu
 def test_ubuntu(input_path, output_path, extra_args):
     command = f"fosslight_dependency -p {input_path} -o {output_path} {extra_args}"
-    result = subprocess.run(command, shell=True, capture_output=True, text=True)
+    result = subprocess.run(command, capture_output=True, text=True)
     assert result.returncode == 0, f"Command failed: {command}\nstdout: {result.stdout}\nstderr: {result.stderr}"
     assert any(os.scandir(output_path)), f"Output file does not exist: {output_path}"
 
@@ -28,6 +28,6 @@ def test_ubuntu(input_path, output_path, extra_args):
 @pytest.mark.windows
 def test_windows(input_path, output_path):
     command = f"{DIST_PATH} -p {input_path} -o {output_path}"
-    result = subprocess.run(command, shell=True, capture_output=True, text=True)
+    result = subprocess.run(command, capture_output=True, text=True)
     assert result.returncode == 0, f"Command failed: {command}\nstdout: {result.stdout}\nstderr: {result.stderr}"
     assert any(os.scandir(output_path)), f"Output file does not exist: {output_path}"