mirror of
https://github.com/python/cpython.git
synced 2025-08-04 08:59:19 +00:00
Fix ProcessTestCasePOSIXPurePython to test the module from import when
_posixsubprocess doesn't exist rather than simply stubbing it out after the fact. This adds coverage for the RuntimeWarning as well as using the pure python _create_pipe instead of using _posixsubprocess.cloexec_pipe unintentionally with the pure python code. Ironically: I don't think any platform should ever actually _use_ the pure Python subprocess code on POSIX platforms anymore. This at least tests it properly in this stable branch. The pure python code for this is likely to be removed in 3.3.
This commit is contained in:
parent
6e5fd04ce2
commit
7439e7b7ce
1 changed files with 16 additions and 8 deletions
|
@ -1399,15 +1399,23 @@ class ProcessTestCaseNoPoll(ProcessTestCase):
|
|||
@unittest.skipUnless(getattr(subprocess, '_posixsubprocess', False),
|
||||
"_posixsubprocess extension module not found.")
|
||||
class ProcessTestCasePOSIXPurePython(ProcessTestCase, POSIXProcessTestCase):
|
||||
def setUp(self):
|
||||
subprocess._posixsubprocess = None
|
||||
ProcessTestCase.setUp(self)
|
||||
POSIXProcessTestCase.setUp(self)
|
||||
@classmethod
|
||||
def setUpClass(cls):
|
||||
global subprocess
|
||||
assert subprocess._posixsubprocess
|
||||
# Reimport subprocess while forcing _posixsubprocess to not exist.
|
||||
with support.check_warnings(('.*_posixsubprocess .* not being used.*',
|
||||
RuntimeWarning)):
|
||||
subprocess = support.import_fresh_module(
|
||||
'subprocess', blocked=['_posixsubprocess'])
|
||||
assert not subprocess._posixsubprocess
|
||||
|
||||
def tearDown(self):
|
||||
subprocess._posixsubprocess = sys.modules['_posixsubprocess']
|
||||
POSIXProcessTestCase.tearDown(self)
|
||||
ProcessTestCase.tearDown(self)
|
||||
@classmethod
|
||||
def tearDownClass(cls):
|
||||
global subprocess
|
||||
# Reimport subprocess as it should be, restoring order to the universe.
|
||||
subprocess = support.import_fresh_module('subprocess')
|
||||
assert subprocess._posixsubprocess
|
||||
|
||||
|
||||
class HelperFunctionTests(unittest.TestCase):
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue