Properly gate tests requiring subprocesses

ambv · ambv · commit 0448edd52516 · 2025-07-19T10:51:52.000+02:00
diff --git a/Lib/test/test_pyrepl/test_pyrepl.py b/Lib/test/test_pyrepl/test_pyrepl.py
@@ -12,7 +12,7 @@
 from unittest import TestCase, skipUnless, skipIf
 from unittest.mock import patch
 from test.support import force_not_colorized, make_clean_env, Py_DEBUG
-from test.support import SHORT_TIMEOUT, STDLIB_DIR
+from test.support import has_subprocess_support, SHORT_TIMEOUT, STDLIB_DIR
 from test.support.import_helper import import_module
 from test.support.os_helper import EnvironmentVarGuard, unlink
 
@@ -38,6 +38,10 @@
 
 
 class ReplTestCase(TestCase):
+    def setUp(self):
+        if not has_subprocess_support:
+            raise unittest.SkipTest("test module requires subprocess")
+
     def run_repl(
         self,
         repl_input: str | list[str],
@@ -1371,6 +1375,7 @@ def setUp(self):
         # Cleanup from PYTHON* variables to isolate from local
         # user settings, see #121359.  Such variables should be
         # added later in test methods to patched os.environ.
+        super().setUp()
         patcher = patch('os.environ', new=make_clean_env())
         self.addCleanup(patcher.stop)
         patcher.start()
diff --git a/Lib/test/test_pyrepl/test_terminfo.py b/Lib/test/test_pyrepl/test_terminfo.py
@@ -5,7 +5,7 @@
 import subprocess
 import sys
 import unittest
-from test.support import requires
+from test.support import requires, has_subprocess_support
 from textwrap import dedent
 
 # Only run these tests if curses is available
@@ -38,6 +38,10 @@ def setUp(self):
             raise unittest.SkipTest(
                 "`curses` capability provided to regrtest but `_curses` not importable"
             )
+
+        if not has_subprocess_support:
+            raise unittest.SkipTest("test module requires subprocess")
+
         self.original_term = os.environ.get("TERM", None)
 
     def tearDown(self):
