summaryrefslogtreecommitdiffstats
path: root/Lib/test/test_subprocess.py
diff options
context:
space:
mode:
authorGregory P. Smith ext:(%20%5BGoogle%20Inc.%5D) <greg@krypto.org>2016-06-05 02:58:38 (GMT)
committerGregory P. Smith ext:(%20%5BGoogle%20Inc.%5D) <greg@krypto.org>2016-06-05 02:58:38 (GMT)
commit7a9ddd1d85453485240dd666d679f7b6729470e0 (patch)
tree43c6c4ef9cadb057eb2b2f5a98f96d2bd13f5343 /Lib/test/test_subprocess.py
parentf39571ed64ebcedf1b27c7453aef8680ef8a2b23 (diff)
parent2daf8e7f76c2b6917468dc50ebae792f34481cf4 (diff)
downloadcpython-7a9ddd1d85453485240dd666d679f7b6729470e0.zip
cpython-7a9ddd1d85453485240dd666d679f7b6729470e0.tar.gz
cpython-7a9ddd1d85453485240dd666d679f7b6729470e0.tar.bz2
merge from 3.5. (moves the issue26372 tests to the proper class)
Diffstat (limited to 'Lib/test/test_subprocess.py')
-rw-r--r--Lib/test/test_subprocess.py92
1 files changed, 46 insertions, 46 deletions
diff --git a/Lib/test/test_subprocess.py b/Lib/test/test_subprocess.py
index 43d2f498..ba91bbc 100644
--- a/Lib/test/test_subprocess.py
+++ b/Lib/test/test_subprocess.py
@@ -1238,52 +1238,6 @@ class ProcessTestCase(BaseTestCase):
fds_after_exception = os.listdir(fd_directory)
self.assertEqual(fds_before_popen, fds_after_exception)
- def test_communicate_BrokenPipeError_stdin_close(self):
- # By not setting stdout or stderr or a timeout we force the fast path
- # that just calls _stdin_write() internally due to our mock.
- proc = subprocess.Popen([sys.executable, '-c', 'pass'])
- with proc, mock.patch.object(proc, 'stdin') as mock_proc_stdin:
- mock_proc_stdin.close.side_effect = BrokenPipeError
- proc.communicate() # Should swallow BrokenPipeError from close.
- mock_proc_stdin.close.assert_called_with()
-
- def test_communicate_BrokenPipeError_stdin_write(self):
- # By not setting stdout or stderr or a timeout we force the fast path
- # that just calls _stdin_write() internally due to our mock.
- proc = subprocess.Popen([sys.executable, '-c', 'pass'])
- with proc, mock.patch.object(proc, 'stdin') as mock_proc_stdin:
- mock_proc_stdin.write.side_effect = BrokenPipeError
- proc.communicate(b'stuff') # Should swallow the BrokenPipeError.
- mock_proc_stdin.write.assert_called_once_with(b'stuff')
- mock_proc_stdin.close.assert_called_once_with()
-
- def test_communicate_BrokenPipeError_stdin_flush(self):
- # Setting stdin and stdout forces the ._communicate() code path.
- # python -h exits faster than python -c pass (but spams stdout).
- proc = subprocess.Popen([sys.executable, '-h'],
- stdin=subprocess.PIPE,
- stdout=subprocess.PIPE)
- with proc, mock.patch.object(proc, 'stdin') as mock_proc_stdin, \
- open(os.devnull, 'wb') as dev_null:
- mock_proc_stdin.flush.side_effect = BrokenPipeError
- # because _communicate registers a selector using proc.stdin...
- mock_proc_stdin.fileno.return_value = dev_null.fileno()
- # _communicate() should swallow BrokenPipeError from flush.
- proc.communicate(b'stuff')
- mock_proc_stdin.flush.assert_called_once_with()
-
- def test_communicate_BrokenPipeError_stdin_close_with_timeout(self):
- # Setting stdin and stdout forces the ._communicate() code path.
- # python -h exits faster than python -c pass (but spams stdout).
- proc = subprocess.Popen([sys.executable, '-h'],
- stdin=subprocess.PIPE,
- stdout=subprocess.PIPE)
- with proc, mock.patch.object(proc, 'stdin') as mock_proc_stdin:
- mock_proc_stdin.close.side_effect = BrokenPipeError
- # _communicate() should swallow BrokenPipeError from close.
- proc.communicate(timeout=999)
- mock_proc_stdin.close.assert_called_once_with()
-
class RunFuncTestCase(BaseTestCase):
def run_python(self, code, **kwargs):
@@ -2470,6 +2424,52 @@ class POSIXProcessTestCase(BaseTestCase):
if not gc_enabled:
gc.disable()
+ def test_communicate_BrokenPipeError_stdin_close(self):
+ # By not setting stdout or stderr or a timeout we force the fast path
+ # that just calls _stdin_write() internally due to our mock.
+ proc = subprocess.Popen([sys.executable, '-c', 'pass'])
+ with proc, mock.patch.object(proc, 'stdin') as mock_proc_stdin:
+ mock_proc_stdin.close.side_effect = BrokenPipeError
+ proc.communicate() # Should swallow BrokenPipeError from close.
+ mock_proc_stdin.close.assert_called_with()
+
+ def test_communicate_BrokenPipeError_stdin_write(self):
+ # By not setting stdout or stderr or a timeout we force the fast path
+ # that just calls _stdin_write() internally due to our mock.
+ proc = subprocess.Popen([sys.executable, '-c', 'pass'])
+ with proc, mock.patch.object(proc, 'stdin') as mock_proc_stdin:
+ mock_proc_stdin.write.side_effect = BrokenPipeError
+ proc.communicate(b'stuff') # Should swallow the BrokenPipeError.
+ mock_proc_stdin.write.assert_called_once_with(b'stuff')
+ mock_proc_stdin.close.assert_called_once_with()
+
+ def test_communicate_BrokenPipeError_stdin_flush(self):
+ # Setting stdin and stdout forces the ._communicate() code path.
+ # python -h exits faster than python -c pass (but spams stdout).
+ proc = subprocess.Popen([sys.executable, '-h'],
+ stdin=subprocess.PIPE,
+ stdout=subprocess.PIPE)
+ with proc, mock.patch.object(proc, 'stdin') as mock_proc_stdin, \
+ open(os.devnull, 'wb') as dev_null:
+ mock_proc_stdin.flush.side_effect = BrokenPipeError
+ # because _communicate registers a selector using proc.stdin...
+ mock_proc_stdin.fileno.return_value = dev_null.fileno()
+ # _communicate() should swallow BrokenPipeError from flush.
+ proc.communicate(b'stuff')
+ mock_proc_stdin.flush.assert_called_once_with()
+
+ def test_communicate_BrokenPipeError_stdin_close_with_timeout(self):
+ # Setting stdin and stdout forces the ._communicate() code path.
+ # python -h exits faster than python -c pass (but spams stdout).
+ proc = subprocess.Popen([sys.executable, '-h'],
+ stdin=subprocess.PIPE,
+ stdout=subprocess.PIPE)
+ with proc, mock.patch.object(proc, 'stdin') as mock_proc_stdin:
+ mock_proc_stdin.close.side_effect = BrokenPipeError
+ # _communicate() should swallow BrokenPipeError from close.
+ proc.communicate(timeout=999)
+ mock_proc_stdin.close.assert_called_once_with()
+
@unittest.skipUnless(mswindows, "Windows specific tests")
class Win32ProcessTestCase(BaseTestCase):