Commit f7f1bb7f authored by Peter Astrand's avatar Peter Astrand

Only run extensive subprocess tests if -usubprocess to regrtest is specified. Fixes #1124637

parent 23109f00
...@@ -93,6 +93,8 @@ resources to test. Currently only the following are defined: ...@@ -93,6 +93,8 @@ resources to test. Currently only the following are defined:
in the standard library and test suite. This takes in the standard library and test suite. This takes
a long time. a long time.
subprocess Run all tests for the subprocess module.
To enable all resources except one, use '-uall,-<resource>'. For To enable all resources except one, use '-uall,-<resource>'. For
example, to run all the tests except for the bsddb tests, give the example, to run all the tests except for the bsddb tests, give the
option '-uall,-bsddb'. option '-uall,-bsddb'.
...@@ -136,7 +138,7 @@ if sys.platform == 'darwin': ...@@ -136,7 +138,7 @@ if sys.platform == 'darwin':
from test import test_support from test import test_support
RESOURCE_NAMES = ('audio', 'curses', 'largefile', 'network', 'bsddb', RESOURCE_NAMES = ('audio', 'curses', 'largefile', 'network', 'bsddb',
'decimal', 'compiler') 'decimal', 'compiler', 'subprocess')
def usage(code, msg=''): def usage(code, msg=''):
......
...@@ -382,9 +382,10 @@ class ProcessTestCase(unittest.TestCase): ...@@ -382,9 +382,10 @@ class ProcessTestCase(unittest.TestCase):
def test_no_leaking(self): def test_no_leaking(self):
# Make sure we leak no resources # Make sure we leak no resources
max_handles = 1026 # too much for most UNIX systems if test_support.is_resource_enabled("subprocess") and not mswindows:
if mswindows: max_handles = 1026 # too much for most UNIX systems
max_handles = 65 # a full test is too slow on Windows else:
max_handles = 65
for i in range(max_handles): for i in range(max_handles):
p = subprocess.Popen([sys.executable, "-c", p = subprocess.Popen([sys.executable, "-c",
"import sys;sys.stdout.write(sys.stdin.read())"], "import sys;sys.stdout.write(sys.stdin.read())"],
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment