From 5395d2f07d3d50aef07c70c26a95193c66cb7d92 Mon Sep 17 00:00:00 2001 From: Andrew Svetlov Date: Wed, 15 Aug 2012 22:46:43 +0300 Subject: [PATCH] Add yet another test for subprocess.Popen.communicate --- Lib/test/test_subprocess.py | 28 ++++++++++++++++++++++++++++ 1 file changed, 28 insertions(+) diff --git a/Lib/test/test_subprocess.py b/Lib/test/test_subprocess.py index e591d2fd56c..72079d07d3d 100644 --- a/Lib/test/test_subprocess.py +++ b/Lib/test/test_subprocess.py @@ -645,6 +645,34 @@ class ProcessTestCase(BaseTestCase): p.communicate() self.assertEqual(p.returncode, 0) + def test_universal_newlines_communicate_stdin_stdout_stderr(self): + # universal newlines through communicate(), with only stdin + p = subprocess.Popen([sys.executable, "-c", + 'import sys,os;' + SETBINARY + '''\nif True: + s = sys.stdin.readline() + sys.stdout.write(s) + sys.stdout.write("line2\\r") + sys.stderr.write("eline2\\n") + s = sys.stdin.read() + sys.stdout.write(s+"line4\\n") + sys.stdout.write(s+"line5\\r\\n") + sys.stderr.write("eline6\\n") + sys.stderr.write("eline7\\r") + sys.stderr.write("eline8\\r\\n") + '''], + stdin=subprocess.PIPE, + stderr=subprocess.PIPE, + stdout=subprocess.PIPE, + universal_newlines=1) + self.addCleanup(p.stdout.close) + self.addCleanup(p.stderr.close) + (stdout, stderr) = p.communicate("line1\nline3\n") + self.assertEqual(p.returncode, 0) + self.assertEqual("line1\nline2\nline3\nline4\nline3\nline5\n", stdout) + # Python debug build push something like "[42442 refs]\n" + # to stderr at exit of subprocess. + self.assertTrue(stderr.startswith("eline2\neline6\neline7\neline8\n")) + def test_no_leaking(self): # Make sure we leak no resources if not mswindows: