Also capture stderr so that no errors are seen on console

s0undt3ch · s0undt3ch · commit cd523485aed4 · 2024-01-04T11:54:16.000Z
Signed-off-by: Pedro Algarvio &lt;palgarvio@vmware.com&gt;
diff --git a/changelog/33.bugfix.rst b/changelog/33.bugfix.rst
@@ -0,0 +1 @@
+Also capture ``stderr`` so that no errors are seen on console when checking for FIPS support
diff --git a/src/pytestskipmarkers/utils/platform.py b/src/pytestskipmarkers/utils/platform.py
@@ -217,7 +217,7 @@ def is_fips_enabled() -> bool:
             [sysctl_path, "crypto.fips_enabled"],
             check=False,
             shell=False,
-            stdout=subprocess.PIPE,
+            capture_output=True,
             text=True,
         )
         if ret.returncode == 0:

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	+Also capture ``stderr`` so that no errors are seen on console when checking for FIPS support
Original file line number	Diff line number	Diff line change
`@@ -217,7 +217,7 @@ def is_fips_enabled() -> bool:`
`217`	`217`	`[sysctl_path, "crypto.fips_enabled"],`
`218`	`218`	`check=False,`
`219`	`219`	`shell=False,`
`220`		`- stdout=subprocess.PIPE,`
	`220`	`+ capture_output=True,`
`221`	`221`	`text=True,`
`222`	`222`	`)`
`223`	`223`	`if ret.returncode == 0:`