diff --git a/subprocess2.py b/subprocess2.py index b22d3e1ab5..3cc79aeb89 100644 --- a/subprocess2.py +++ b/subprocess2.py @@ -259,7 +259,10 @@ def capture(args, **kwargs): - Discards returncode. - Discards stderr. By default sets stderr=STDOUT. + - Blocks stdin by default since no output will be visible. """ + if kwargs.get('stdin') is None: + kwargs['stdin'] = VOID if kwargs.get('stdout') is None: kwargs['stdout'] = PIPE if kwargs.get('stderr') is None: @@ -275,7 +278,10 @@ def check_output(args, **kwargs): - Discards stderr. By default sets stderr=STDOUT. - Throws if return code is not 0. - Works even prior to python 2.7. + - Blocks stdin by default since no output will be visible. """ + if kwargs.get('stdin') is None: + kwargs['stdin'] = VOID if kwargs.get('stdout') is None: kwargs['stdout'] = PIPE if kwargs.get('stderr') is None: diff --git a/tests/subprocess2_test.py b/tests/subprocess2_test.py index 448a82d343..552a93c49e 100755 --- a/tests/subprocess2_test.py +++ b/tests/subprocess2_test.py @@ -125,6 +125,7 @@ class Subprocess2Test(unittest.TestCase): expected = { 'args': ['foo'], 'a':True, + 'stdin': subprocess2.VOID, 'stdout': subprocess2.PIPE, 'stderr': subprocess2.STDOUT, }