cpython-lru_cache-weakref: Lib/test/test

annotate Lib/test/test_subprocess.py @ 70049:b00a64a5cb93 3.2

merge - 7a3f3ad83676 Fixes Issue #12044.

author	Gregory P. Smith <greg@krypto.org>
date	Wed, 11 May 2011 22:18:23 -0700
parents	a7363288c8d4
children	4f248dd34dd9 361f87c8f36a

rev	line source
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	1 import unittest
47201 9c930a32cebb #2621 rename test.test_support to test.support Benjamin Peterson <benjamin@python.org> parents: 46710 diff changeset	2 from test import support
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	3 import subprocess
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	4 import sys
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	5 import signal
68526 ad2bd8d338b0 Add unittests demonstrating issue #11432. Gregory P. Smith <greg@krypto.org> parents: 68492 diff changeset	6 import io
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	7 import os
59209 1f158238fa76 Merged revisions 78524 via svnmerge from Gregory P. Smith <greg@mad-scientist.com> parents: 55626 diff changeset	8 import errno
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	9 import tempfile
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	10 import time
33739 76355363101b New helper remove_stderr_debug_decorations(). This test passes in a Tim Peters <tim.peters@gmail.com> parents: 33738 diff changeset	11 import re
58931 4c0be37b46b0 Merged revisions 78136 via svnmerge from Ezio Melotti <ezio.melotti@gmail.com> parents: 58415 diff changeset	12 import sysconfig
66676 b37da60c9072 issue7213 + issue2320: Cause a DeprecationWarning if the close_fds argument is Gregory P. Smith <greg@mad-scientist.com> parents: 66622 diff changeset	13 import warnings
66868 de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	14 import select
68491 141bbcb19c06 Fix the @test_NNNN_tmp file terds being left in whatever your $PWD was Gregory P. Smith <greg@krypto.org> parents: 68432 diff changeset	15 import shutil
59723 00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	16 try:
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	17 import gc
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	18 except ImportError:
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	19 gc = None
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	20
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	21 mswindows = (sys.platform == "win32")
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	22
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	23 #
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	24 # Depends on the following external programs: Python
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	25 #
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	26
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	27 if mswindows:
33732 d9404db29bc6 Wrap long lines. Tim Peters <tim.peters@gmail.com> parents: 33731 diff changeset	28 SETBINARY = ('import msvcrt; msvcrt.setmode(sys.stdout.fileno(), '
d9404db29bc6 Wrap long lines. Tim Peters <tim.peters@gmail.com> parents: 33731 diff changeset	29 'os.O_BINARY);')
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	30 else:
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	31 SETBINARY = ''
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	32
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	33
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	34 try:
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	35 mkstemp = tempfile.mkstemp
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	36 except AttributeError:
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	37 # tempfile.mkstemp is not available
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	38 def mkstemp():
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	39 """Replacement for mkstemp, calling mktemp."""
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	40 fname = tempfile.mktemp()
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	41 return os.open(fname, os.O_RDWR\|os.O_CREAT), fname
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	42
33739 76355363101b New helper remove_stderr_debug_decorations(). This test passes in a Tim Peters <tim.peters@gmail.com> parents: 33738 diff changeset	43
63930 8499ae5d9719 #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63843 diff changeset	44 class BaseTestCase(unittest.TestCase):
39442 654c380cf8b9 Merged revisions 46753-51188 via svnmerge from Thomas Wouters <thomas@python.org> parents: 38453 diff changeset	45 def setUp(self):
654c380cf8b9 Merged revisions 46753-51188 via svnmerge from Thomas Wouters <thomas@python.org> parents: 38453 diff changeset	46 # Try to minimize the number of children we have so this test
654c380cf8b9 Merged revisions 46753-51188 via svnmerge from Thomas Wouters <thomas@python.org> parents: 38453 diff changeset	47 # doesn't crash on some buildbots (Alphas in particular).
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	48 support.reap_children()
39442 654c380cf8b9 Merged revisions 46753-51188 via svnmerge from Thomas Wouters <thomas@python.org> parents: 38453 diff changeset	49
654c380cf8b9 Merged revisions 46753-51188 via svnmerge from Thomas Wouters <thomas@python.org> parents: 38453 diff changeset	50 def tearDown(self):
59340 04955e7eb597 Merged revisions 78661-78662 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59216 diff changeset	51 for inst in subprocess._active:
04955e7eb597 Merged revisions 78661-78662 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59216 diff changeset	52 inst.wait()
04955e7eb597 Merged revisions 78661-78662 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59216 diff changeset	53 subprocess._cleanup()
04955e7eb597 Merged revisions 78661-78662 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59216 diff changeset	54 self.assertFalse(subprocess._active, "subprocess._active not empty")
39442 654c380cf8b9 Merged revisions 46753-51188 via svnmerge from Thomas Wouters <thomas@python.org> parents: 38453 diff changeset	55
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	56 def assertStderrEqual(self, stderr, expected, msg=None):
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	57 # In a debug build, stuff like "[6580 refs]" is printed to stderr at
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	58 # shutdown time. That frustrates tests trying to check stderr produced
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	59 # from a spawned Python process.
63727 5931112eabe7 Factor out stripping of interpreter debug output in test.support.strip_python_stderr() Antoine Pitrou <solipsis@pitrou.net> parents: 63141 diff changeset	60 actual = support.strip_python_stderr(stderr)
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	61 self.assertEqual(actual, expected, msg)
33731 53c468d0955b Whitespace normalization. Tim Peters <tim.peters@gmail.com> parents: 33723 diff changeset	62
53c468d0955b Whitespace normalization. Tim Peters <tim.peters@gmail.com> parents: 33723 diff changeset	63
63930 8499ae5d9719 #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63843 diff changeset	64 class ProcessTestCase(BaseTestCase):
60057 bb2ca1b63321 Merged revisions 79344,79346,79350 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59723 diff changeset	65
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	66 def test_call_seq(self):
33733 39a13e39d24e Experience with Zope2's tests showed it's a Bad Idea to make unittest Tim Peters <tim.peters@gmail.com> parents: 33732 diff changeset	67 # call() function with sequence argument
33732 d9404db29bc6 Wrap long lines. Tim Peters <tim.peters@gmail.com> parents: 33731 diff changeset	68 rc = subprocess.call([sys.executable, "-c",
d9404db29bc6 Wrap long lines. Tim Peters <tim.peters@gmail.com> parents: 33731 diff changeset	69 "import sys; sys.exit(47)"])
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	70 self.assertEqual(rc, 47)
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	71
34302 998fb1e1c8e9 New subprocess utility function: check_call. Closes #1071764. Peter Astrand <astrand@lysator.liu.se> parents: 34052 diff changeset	72 def test_check_call_zero(self):
998fb1e1c8e9 New subprocess utility function: check_call. Closes #1071764. Peter Astrand <astrand@lysator.liu.se> parents: 34052 diff changeset	73 # check_call() function with zero return code
998fb1e1c8e9 New subprocess utility function: check_call. Closes #1071764. Peter Astrand <astrand@lysator.liu.se> parents: 34052 diff changeset	74 rc = subprocess.check_call([sys.executable, "-c",
998fb1e1c8e9 New subprocess utility function: check_call. Closes #1071764. Peter Astrand <astrand@lysator.liu.se> parents: 34052 diff changeset	75 "import sys; sys.exit(0)"])
998fb1e1c8e9 New subprocess utility function: check_call. Closes #1071764. Peter Astrand <astrand@lysator.liu.se> parents: 34052 diff changeset	76 self.assertEqual(rc, 0)
998fb1e1c8e9 New subprocess utility function: check_call. Closes #1071764. Peter Astrand <astrand@lysator.liu.se> parents: 34052 diff changeset	77
998fb1e1c8e9 New subprocess utility function: check_call. Closes #1071764. Peter Astrand <astrand@lysator.liu.se> parents: 34052 diff changeset	78 def test_check_call_nonzero(self):
998fb1e1c8e9 New subprocess utility function: check_call. Closes #1071764. Peter Astrand <astrand@lysator.liu.se> parents: 34052 diff changeset	79 # check_call() function with non-zero return code
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	80 with self.assertRaises(subprocess.CalledProcessError) as c:
34302 998fb1e1c8e9 New subprocess utility function: check_call. Closes #1071764. Peter Astrand <astrand@lysator.liu.se> parents: 34052 diff changeset	81 subprocess.check_call([sys.executable, "-c",
998fb1e1c8e9 New subprocess utility function: check_call. Closes #1071764. Peter Astrand <astrand@lysator.liu.se> parents: 34052 diff changeset	82 "import sys; sys.exit(47)"])
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	83 self.assertEqual(c.exception.returncode, 47)
34302 998fb1e1c8e9 New subprocess utility function: check_call. Closes #1071764. Peter Astrand <astrand@lysator.liu.se> parents: 34052 diff changeset	84
50074 024a9897c651 Merged revisions 67511,67536-67537,67543 via svnmerge from Georg Brandl <georg@python.org> parents: 49200 diff changeset	85 def test_check_output(self):
024a9897c651 Merged revisions 67511,67536-67537,67543 via svnmerge from Georg Brandl <georg@python.org> parents: 49200 diff changeset	86 # check_output() function with zero return code
024a9897c651 Merged revisions 67511,67536-67537,67543 via svnmerge from Georg Brandl <georg@python.org> parents: 49200 diff changeset	87 output = subprocess.check_output(
024a9897c651 Merged revisions 67511,67536-67537,67543 via svnmerge from Georg Brandl <georg@python.org> parents: 49200 diff changeset	88 [sys.executable, "-c", "print('BDFL')"])
58415 f6a626f34840 use assert[Not]In where appropriate Benjamin Peterson <benjamin@python.org> parents: 55626 diff changeset	89 self.assertIn(b'BDFL', output)
50074 024a9897c651 Merged revisions 67511,67536-67537,67543 via svnmerge from Georg Brandl <georg@python.org> parents: 49200 diff changeset	90
024a9897c651 Merged revisions 67511,67536-67537,67543 via svnmerge from Georg Brandl <georg@python.org> parents: 49200 diff changeset	91 def test_check_output_nonzero(self):
024a9897c651 Merged revisions 67511,67536-67537,67543 via svnmerge from Georg Brandl <georg@python.org> parents: 49200 diff changeset	92 # check_call() function with non-zero return code
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	93 with self.assertRaises(subprocess.CalledProcessError) as c:
50074 024a9897c651 Merged revisions 67511,67536-67537,67543 via svnmerge from Georg Brandl <georg@python.org> parents: 49200 diff changeset	94 subprocess.check_output(
024a9897c651 Merged revisions 67511,67536-67537,67543 via svnmerge from Georg Brandl <georg@python.org> parents: 49200 diff changeset	95 [sys.executable, "-c", "import sys; sys.exit(5)"])
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	96 self.assertEqual(c.exception.returncode, 5)
50074 024a9897c651 Merged revisions 67511,67536-67537,67543 via svnmerge from Georg Brandl <georg@python.org> parents: 49200 diff changeset	97
024a9897c651 Merged revisions 67511,67536-67537,67543 via svnmerge from Georg Brandl <georg@python.org> parents: 49200 diff changeset	98 def test_check_output_stderr(self):
024a9897c651 Merged revisions 67511,67536-67537,67543 via svnmerge from Georg Brandl <georg@python.org> parents: 49200 diff changeset	99 # check_output() function stderr redirected to stdout
024a9897c651 Merged revisions 67511,67536-67537,67543 via svnmerge from Georg Brandl <georg@python.org> parents: 49200 diff changeset	100 output = subprocess.check_output(
024a9897c651 Merged revisions 67511,67536-67537,67543 via svnmerge from Georg Brandl <georg@python.org> parents: 49200 diff changeset	101 [sys.executable, "-c", "import sys; sys.stderr.write('BDFL')"],
024a9897c651 Merged revisions 67511,67536-67537,67543 via svnmerge from Georg Brandl <georg@python.org> parents: 49200 diff changeset	102 stderr=subprocess.STDOUT)
58415 f6a626f34840 use assert[Not]In where appropriate Benjamin Peterson <benjamin@python.org> parents: 55626 diff changeset	103 self.assertIn(b'BDFL', output)
50074 024a9897c651 Merged revisions 67511,67536-67537,67543 via svnmerge from Georg Brandl <georg@python.org> parents: 49200 diff changeset	104
024a9897c651 Merged revisions 67511,67536-67537,67543 via svnmerge from Georg Brandl <georg@python.org> parents: 49200 diff changeset	105 def test_check_output_stdout_arg(self):
024a9897c651 Merged revisions 67511,67536-67537,67543 via svnmerge from Georg Brandl <georg@python.org> parents: 49200 diff changeset	106 # check_output() function stderr redirected to stdout
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	107 with self.assertRaises(ValueError) as c:
50074 024a9897c651 Merged revisions 67511,67536-67537,67543 via svnmerge from Georg Brandl <georg@python.org> parents: 49200 diff changeset	108 output = subprocess.check_output(
024a9897c651 Merged revisions 67511,67536-67537,67543 via svnmerge from Georg Brandl <georg@python.org> parents: 49200 diff changeset	109 [sys.executable, "-c", "print('will not be run')"],
024a9897c651 Merged revisions 67511,67536-67537,67543 via svnmerge from Georg Brandl <georg@python.org> parents: 49200 diff changeset	110 stdout=sys.stdout)
024a9897c651 Merged revisions 67511,67536-67537,67543 via svnmerge from Georg Brandl <georg@python.org> parents: 49200 diff changeset	111 self.fail("Expected ValueError when stdout arg supplied.")
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	112 self.assertIn('stdout', c.exception.args[0])
50074 024a9897c651 Merged revisions 67511,67536-67537,67543 via svnmerge from Georg Brandl <georg@python.org> parents: 49200 diff changeset	113
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	114 def test_call_kwargs(self):
33733 39a13e39d24e Experience with Zope2's tests showed it's a Bad Idea to make unittest Tim Peters <tim.peters@gmail.com> parents: 33732 diff changeset	115 # call() function with keyword args
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	116 newenv = os.environ.copy()
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	117 newenv["FRUIT"] = "banana"
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	118 rc = subprocess.call([sys.executable, "-c",
43777 0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	119 'import sys, os;'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	120 'sys.exit(os.getenv("FRUIT")=="banana")'],
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	121 env=newenv)
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	122 self.assertEqual(rc, 1)
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	123
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	124 def test_stdin_none(self):
33733 39a13e39d24e Experience with Zope2's tests showed it's a Bad Idea to make unittest Tim Peters <tim.peters@gmail.com> parents: 33732 diff changeset	125 # .stdin is None when not redirected
40788 e9a6234b26af * Remove PRINT_ITEM(_TO), PRINT_NEWLINE(_TO) opcodes. Georg Brandl <georg@python.org> parents: 40643 diff changeset	126 p = subprocess.Popen([sys.executable, "-c", 'print("banana")'],
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	127 stdout=subprocess.PIPE, stderr=subprocess.PIPE)
65955 6e1ae323fde6 Add cleanups to stdout/stderr pipes to remove ResourceWarnings. Brian Curtin <brian.curtin@gmail.com> parents: 65799 diff changeset	128 self.addCleanup(p.stdout.close)
6e1ae323fde6 Add cleanups to stdout/stderr pipes to remove ResourceWarnings. Brian Curtin <brian.curtin@gmail.com> parents: 65799 diff changeset	129 self.addCleanup(p.stderr.close)
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	130 p.wait()
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	131 self.assertEqual(p.stdin, None)
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	132
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	133 def test_stdout_none(self):
33733 39a13e39d24e Experience with Zope2's tests showed it's a Bad Idea to make unittest Tim Peters <tim.peters@gmail.com> parents: 33732 diff changeset	134 # .stdout is None when not redirected
33738 a4f747e72138 Kill several problems at once: test_poll() failed sometimes for me. Tim Peters <tim.peters@gmail.com> parents: 33737 diff changeset	135 p = subprocess.Popen([sys.executable, "-c",
40788 e9a6234b26af * Remove PRINT_ITEM(_TO), PRINT_NEWLINE(_TO) opcodes. Georg Brandl <georg@python.org> parents: 40643 diff changeset	136 'print(" this bit of output is from a '
33737 e4f7d3b0c5d4 test_stdout_none(): Don't print "banana" to the screen in the middle Tim Peters <tim.peters@gmail.com> parents: 33736 diff changeset	137 'test of stdout in a different '
40788 e9a6234b26af * Remove PRINT_ITEM(_TO), PRINT_NEWLINE(_TO) opcodes. Georg Brandl <georg@python.org> parents: 40643 diff changeset	138 'process ...")'],
33737 e4f7d3b0c5d4 test_stdout_none(): Don't print "banana" to the screen in the middle Tim Peters <tim.peters@gmail.com> parents: 33736 diff changeset	139 stdin=subprocess.PIPE, stderr=subprocess.PIPE)
65955 6e1ae323fde6 Add cleanups to stdout/stderr pipes to remove ResourceWarnings. Brian Curtin <brian.curtin@gmail.com> parents: 65799 diff changeset	140 self.addCleanup(p.stdin.close)
6e1ae323fde6 Add cleanups to stdout/stderr pipes to remove ResourceWarnings. Brian Curtin <brian.curtin@gmail.com> parents: 65799 diff changeset	141 self.addCleanup(p.stderr.close)
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	142 p.wait()
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	143 self.assertEqual(p.stdout, None)
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	144
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	145 def test_stderr_none(self):
33733 39a13e39d24e Experience with Zope2's tests showed it's a Bad Idea to make unittest Tim Peters <tim.peters@gmail.com> parents: 33732 diff changeset	146 # .stderr is None when not redirected
40788 e9a6234b26af * Remove PRINT_ITEM(_TO), PRINT_NEWLINE(_TO) opcodes. Georg Brandl <georg@python.org> parents: 40643 diff changeset	147 p = subprocess.Popen([sys.executable, "-c", 'print("banana")'],
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	148 stdin=subprocess.PIPE, stdout=subprocess.PIPE)
65955 6e1ae323fde6 Add cleanups to stdout/stderr pipes to remove ResourceWarnings. Brian Curtin <brian.curtin@gmail.com> parents: 65799 diff changeset	149 self.addCleanup(p.stdout.close)
6e1ae323fde6 Add cleanups to stdout/stderr pipes to remove ResourceWarnings. Brian Curtin <brian.curtin@gmail.com> parents: 65799 diff changeset	150 self.addCleanup(p.stdin.close)
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	151 p.wait()
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	152 self.assertEqual(p.stderr, None)
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	153
58931 4c0be37b46b0 Merged revisions 78136 via svnmerge from Ezio Melotti <ezio.melotti@gmail.com> parents: 58415 diff changeset	154 def test_executable_with_cwd(self):
59507 a111024a0bcb Merged revisions 78833 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59504 diff changeset	155 python_dir = os.path.dirname(os.path.realpath(sys.executable))
58931 4c0be37b46b0 Merged revisions 78136 via svnmerge from Ezio Melotti <ezio.melotti@gmail.com> parents: 58415 diff changeset	156 p = subprocess.Popen(["somethingyoudonthave", "-c",
4c0be37b46b0 Merged revisions 78136 via svnmerge from Ezio Melotti <ezio.melotti@gmail.com> parents: 58415 diff changeset	157 "import sys; sys.exit(47)"],
4c0be37b46b0 Merged revisions 78136 via svnmerge from Ezio Melotti <ezio.melotti@gmail.com> parents: 58415 diff changeset	158 executable=sys.executable, cwd=python_dir)
4c0be37b46b0 Merged revisions 78136 via svnmerge from Ezio Melotti <ezio.melotti@gmail.com> parents: 58415 diff changeset	159 p.wait()
4c0be37b46b0 Merged revisions 78136 via svnmerge from Ezio Melotti <ezio.melotti@gmail.com> parents: 58415 diff changeset	160 self.assertEqual(p.returncode, 47)
4c0be37b46b0 Merged revisions 78136 via svnmerge from Ezio Melotti <ezio.melotti@gmail.com> parents: 58415 diff changeset	161
4c0be37b46b0 Merged revisions 78136 via svnmerge from Ezio Melotti <ezio.melotti@gmail.com> parents: 58415 diff changeset	162 @unittest.skipIf(sysconfig.is_python_build(),
4c0be37b46b0 Merged revisions 78136 via svnmerge from Ezio Melotti <ezio.melotti@gmail.com> parents: 58415 diff changeset	163 "need an installed Python. See #7774")
4c0be37b46b0 Merged revisions 78136 via svnmerge from Ezio Melotti <ezio.melotti@gmail.com> parents: 58415 diff changeset	164 def test_executable_without_cwd(self):
4c0be37b46b0 Merged revisions 78136 via svnmerge from Ezio Melotti <ezio.melotti@gmail.com> parents: 58415 diff changeset	165 # For a normal installation, it should work without 'cwd'
4c0be37b46b0 Merged revisions 78136 via svnmerge from Ezio Melotti <ezio.melotti@gmail.com> parents: 58415 diff changeset	166 # argument. For test runs in the build directory, see #7774.
4c0be37b46b0 Merged revisions 78136 via svnmerge from Ezio Melotti <ezio.melotti@gmail.com> parents: 58415 diff changeset	167 p = subprocess.Popen(["somethingyoudonthave", "-c",
4c0be37b46b0 Merged revisions 78136 via svnmerge from Ezio Melotti <ezio.melotti@gmail.com> parents: 58415 diff changeset	168 "import sys; sys.exit(47)"],
33732 d9404db29bc6 Wrap long lines. Tim Peters <tim.peters@gmail.com> parents: 33731 diff changeset	169 executable=sys.executable)
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	170 p.wait()
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	171 self.assertEqual(p.returncode, 47)
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	172
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	173 def test_stdin_pipe(self):
33733 39a13e39d24e Experience with Zope2's tests showed it's a Bad Idea to make unittest Tim Peters <tim.peters@gmail.com> parents: 33732 diff changeset	174 # stdin redirection
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	175 p = subprocess.Popen([sys.executable, "-c",
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	176 'import sys; sys.exit(sys.stdin.read() == "pear")'],
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	177 stdin=subprocess.PIPE)
42927 f3b0ac4e684f More str/bytes goodness. Guido van Rossum <guido@python.org> parents: 42863 diff changeset	178 p.stdin.write(b"pear")
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	179 p.stdin.close()
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	180 p.wait()
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	181 self.assertEqual(p.returncode, 1)
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	182
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	183 def test_stdin_filedes(self):
33733 39a13e39d24e Experience with Zope2's tests showed it's a Bad Idea to make unittest Tim Peters <tim.peters@gmail.com> parents: 33732 diff changeset	184 # stdin is set to open file descriptor
33731 53c468d0955b Whitespace normalization. Tim Peters <tim.peters@gmail.com> parents: 33723 diff changeset	185 tf = tempfile.TemporaryFile()
65799 0ee86fd902f1 close some more files Benjamin Peterson <benjamin@python.org> parents: 65415 diff changeset	186 self.addCleanup(tf.close)
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	187 d = tf.fileno()
49200 bdb660dca74d Issue #3782: os.write() must not accept unicode strings Antoine Pitrou <solipsis@pitrou.net> parents: 48395 diff changeset	188 os.write(d, b"pear")
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	189 os.lseek(d, 0, 0)
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	190 p = subprocess.Popen([sys.executable, "-c",
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	191 'import sys; sys.exit(sys.stdin.read() == "pear")'],
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	192 stdin=d)
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	193 p.wait()
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	194 self.assertEqual(p.returncode, 1)
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	195
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	196 def test_stdin_fileobj(self):
33733 39a13e39d24e Experience with Zope2's tests showed it's a Bad Idea to make unittest Tim Peters <tim.peters@gmail.com> parents: 33732 diff changeset	197 # stdin is set to open file object
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	198 tf = tempfile.TemporaryFile()
65799 0ee86fd902f1 close some more files Benjamin Peterson <benjamin@python.org> parents: 65415 diff changeset	199 self.addCleanup(tf.close)
42927 f3b0ac4e684f More str/bytes goodness. Guido van Rossum <guido@python.org> parents: 42863 diff changeset	200 tf.write(b"pear")
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	201 tf.seek(0)
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	202 p = subprocess.Popen([sys.executable, "-c",
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	203 'import sys; sys.exit(sys.stdin.read() == "pear")'],
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	204 stdin=tf)
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	205 p.wait()
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	206 self.assertEqual(p.returncode, 1)
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	207
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	208 def test_stdout_pipe(self):
33733 39a13e39d24e Experience with Zope2's tests showed it's a Bad Idea to make unittest Tim Peters <tim.peters@gmail.com> parents: 33732 diff changeset	209 # stdout redirection
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	210 p = subprocess.Popen([sys.executable, "-c",
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	211 'import sys; sys.stdout.write("orange")'],
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	212 stdout=subprocess.PIPE)
65955 6e1ae323fde6 Add cleanups to stdout/stderr pipes to remove ResourceWarnings. Brian Curtin <brian.curtin@gmail.com> parents: 65799 diff changeset	213 self.addCleanup(p.stdout.close)
41845 9ce8fa0a0899 Make test_subprocess work. Fix universal newlines in io.py. Guido van Rossum <guido@python.org> parents: 41732 diff changeset	214 self.assertEqual(p.stdout.read(), b"orange")
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	215
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	216 def test_stdout_filedes(self):
33733 39a13e39d24e Experience with Zope2's tests showed it's a Bad Idea to make unittest Tim Peters <tim.peters@gmail.com> parents: 33732 diff changeset	217 # stdout is set to open file descriptor
33731 53c468d0955b Whitespace normalization. Tim Peters <tim.peters@gmail.com> parents: 33723 diff changeset	218 tf = tempfile.TemporaryFile()
65799 0ee86fd902f1 close some more files Benjamin Peterson <benjamin@python.org> parents: 65415 diff changeset	219 self.addCleanup(tf.close)
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	220 d = tf.fileno()
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	221 p = subprocess.Popen([sys.executable, "-c",
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	222 'import sys; sys.stdout.write("orange")'],
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	223 stdout=d)
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	224 p.wait()
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	225 os.lseek(d, 0, 0)
41732 bbb099d4ec80 Make test_subprocess pass. The subprocess module now reads and writes Guido van Rossum <guido@python.org> parents: 40788 diff changeset	226 self.assertEqual(os.read(d, 1024), b"orange")
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	227
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	228 def test_stdout_fileobj(self):
33733 39a13e39d24e Experience with Zope2's tests showed it's a Bad Idea to make unittest Tim Peters <tim.peters@gmail.com> parents: 33732 diff changeset	229 # stdout is set to open file object
33731 53c468d0955b Whitespace normalization. Tim Peters <tim.peters@gmail.com> parents: 33723 diff changeset	230 tf = tempfile.TemporaryFile()
65799 0ee86fd902f1 close some more files Benjamin Peterson <benjamin@python.org> parents: 65415 diff changeset	231 self.addCleanup(tf.close)
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	232 p = subprocess.Popen([sys.executable, "-c",
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	233 'import sys; sys.stdout.write("orange")'],
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	234 stdout=tf)
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	235 p.wait()
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	236 tf.seek(0)
41845 9ce8fa0a0899 Make test_subprocess work. Fix universal newlines in io.py. Guido van Rossum <guido@python.org> parents: 41732 diff changeset	237 self.assertEqual(tf.read(), b"orange")
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	238
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	239 def test_stderr_pipe(self):
33733 39a13e39d24e Experience with Zope2's tests showed it's a Bad Idea to make unittest Tim Peters <tim.peters@gmail.com> parents: 33732 diff changeset	240 # stderr redirection
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	241 p = subprocess.Popen([sys.executable, "-c",
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	242 'import sys; sys.stderr.write("strawberry")'],
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	243 stderr=subprocess.PIPE)
65955 6e1ae323fde6 Add cleanups to stdout/stderr pipes to remove ResourceWarnings. Brian Curtin <brian.curtin@gmail.com> parents: 65799 diff changeset	244 self.addCleanup(p.stderr.close)
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	245 self.assertStderrEqual(p.stderr.read(), b"strawberry")
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	246
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	247 def test_stderr_filedes(self):
33733 39a13e39d24e Experience with Zope2's tests showed it's a Bad Idea to make unittest Tim Peters <tim.peters@gmail.com> parents: 33732 diff changeset	248 # stderr is set to open file descriptor
33731 53c468d0955b Whitespace normalization. Tim Peters <tim.peters@gmail.com> parents: 33723 diff changeset	249 tf = tempfile.TemporaryFile()
65799 0ee86fd902f1 close some more files Benjamin Peterson <benjamin@python.org> parents: 65415 diff changeset	250 self.addCleanup(tf.close)
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	251 d = tf.fileno()
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	252 p = subprocess.Popen([sys.executable, "-c",
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	253 'import sys; sys.stderr.write("strawberry")'],
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	254 stderr=d)
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	255 p.wait()
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	256 os.lseek(d, 0, 0)
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	257 self.assertStderrEqual(os.read(d, 1024), b"strawberry")
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	258
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	259 def test_stderr_fileobj(self):
33733 39a13e39d24e Experience with Zope2's tests showed it's a Bad Idea to make unittest Tim Peters <tim.peters@gmail.com> parents: 33732 diff changeset	260 # stderr is set to open file object
33731 53c468d0955b Whitespace normalization. Tim Peters <tim.peters@gmail.com> parents: 33723 diff changeset	261 tf = tempfile.TemporaryFile()
65799 0ee86fd902f1 close some more files Benjamin Peterson <benjamin@python.org> parents: 65415 diff changeset	262 self.addCleanup(tf.close)
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	263 p = subprocess.Popen([sys.executable, "-c",
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	264 'import sys; sys.stderr.write("strawberry")'],
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	265 stderr=tf)
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	266 p.wait()
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	267 tf.seek(0)
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	268 self.assertStderrEqual(tf.read(), b"strawberry")
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	269
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	270 def test_stdout_stderr_pipe(self):
33733 39a13e39d24e Experience with Zope2's tests showed it's a Bad Idea to make unittest Tim Peters <tim.peters@gmail.com> parents: 33732 diff changeset	271 # capture stdout and stderr to the same pipe
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	272 p = subprocess.Popen([sys.executable, "-c",
43777 0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	273 'import sys;'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	274 'sys.stdout.write("apple");'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	275 'sys.stdout.flush();'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	276 'sys.stderr.write("orange")'],
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	277 stdout=subprocess.PIPE,
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	278 stderr=subprocess.STDOUT)
65955 6e1ae323fde6 Add cleanups to stdout/stderr pipes to remove ResourceWarnings. Brian Curtin <brian.curtin@gmail.com> parents: 65799 diff changeset	279 self.addCleanup(p.stdout.close)
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	280 self.assertStderrEqual(p.stdout.read(), b"appleorange")
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	281
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	282 def test_stdout_stderr_file(self):
33733 39a13e39d24e Experience with Zope2's tests showed it's a Bad Idea to make unittest Tim Peters <tim.peters@gmail.com> parents: 33732 diff changeset	283 # capture stdout and stderr to the same open file
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	284 tf = tempfile.TemporaryFile()
65799 0ee86fd902f1 close some more files Benjamin Peterson <benjamin@python.org> parents: 65415 diff changeset	285 self.addCleanup(tf.close)
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	286 p = subprocess.Popen([sys.executable, "-c",
43777 0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	287 'import sys;'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	288 'sys.stdout.write("apple");'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	289 'sys.stdout.flush();'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	290 'sys.stderr.write("orange")'],
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	291 stdout=tf,
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	292 stderr=tf)
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	293 p.wait()
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	294 tf.seek(0)
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	295 self.assertStderrEqual(tf.read(), b"appleorange")
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	296
40450 cc992d75d5b3 Four months of trunk changes (including a few releases...) Thomas Wouters <thomas@python.org> parents: 39442 diff changeset	297 def test_stdout_filedes_of_stdout(self):
cc992d75d5b3 Four months of trunk changes (including a few releases...) Thomas Wouters <thomas@python.org> parents: 39442 diff changeset	298 # stdout is set to 1 (#1531862).
49200 bdb660dca74d Issue #3782: os.write() must not accept unicode strings Antoine Pitrou <solipsis@pitrou.net> parents: 48395 diff changeset	299 cmd = r"import sys, os; sys.exit(os.write(sys.stdout.fileno(), b'.\n'))"
40450 cc992d75d5b3 Four months of trunk changes (including a few releases...) Thomas Wouters <thomas@python.org> parents: 39442 diff changeset	300 rc = subprocess.call([sys.executable, "-c", cmd], stdout=1)
66340 2beb25369289 Merged revisions 86596 via svnmerge from Ezio Melotti <ezio.melotti@gmail.com> parents: 64853 diff changeset	301 self.assertEqual(rc, 2)
40450 cc992d75d5b3 Four months of trunk changes (including a few releases...) Thomas Wouters <thomas@python.org> parents: 39442 diff changeset	302
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	303 def test_cwd(self):
44373 2750d531fdc0 Merged revisions 59565-59594 via svnmerge from Christian Heimes <christian@cheimes.de> parents: 43777 diff changeset	304 tmpdir = tempfile.gettempdir()
33980 4005f79d281a Use os.chdir/os.getcwd instead of os.path.realpath, to support Tru64 Peter Astrand <astrand@lysator.liu.se> parents: 33814 diff changeset	305 # We cannot use os.path.realpath to canonicalize the path,
4005f79d281a Use os.chdir/os.getcwd instead of os.path.realpath, to support Tru64 Peter Astrand <astrand@lysator.liu.se> parents: 33814 diff changeset	306 # since it doesn't expand Tru64 {memb} strings. See bug 1063571.
4005f79d281a Use os.chdir/os.getcwd instead of os.path.realpath, to support Tru64 Peter Astrand <astrand@lysator.liu.se> parents: 33814 diff changeset	307 cwd = os.getcwd()
4005f79d281a Use os.chdir/os.getcwd instead of os.path.realpath, to support Tru64 Peter Astrand <astrand@lysator.liu.se> parents: 33814 diff changeset	308 os.chdir(tmpdir)
4005f79d281a Use os.chdir/os.getcwd instead of os.path.realpath, to support Tru64 Peter Astrand <astrand@lysator.liu.se> parents: 33814 diff changeset	309 tmpdir = os.getcwd()
4005f79d281a Use os.chdir/os.getcwd instead of os.path.realpath, to support Tru64 Peter Astrand <astrand@lysator.liu.se> parents: 33814 diff changeset	310 os.chdir(cwd)
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	311 p = subprocess.Popen([sys.executable, "-c",
43777 0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	312 'import sys,os;'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	313 'sys.stdout.write(os.getcwd())'],
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	314 stdout=subprocess.PIPE,
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	315 cwd=tmpdir)
65955 6e1ae323fde6 Add cleanups to stdout/stderr pipes to remove ResourceWarnings. Brian Curtin <brian.curtin@gmail.com> parents: 65799 diff changeset	316 self.addCleanup(p.stdout.close)
33742 179e42ea2e9c normalize case when comparing directory names (problem reported by Fredrik Lundh <fredrik@pythonware.com> parents: 33739 diff changeset	317 normcase = os.path.normcase
42927 f3b0ac4e684f More str/bytes goodness. Guido van Rossum <guido@python.org> parents: 42863 diff changeset	318 self.assertEqual(normcase(p.stdout.read().decode("utf-8")),
f3b0ac4e684f More str/bytes goodness. Guido van Rossum <guido@python.org> parents: 42863 diff changeset	319 normcase(tmpdir))
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	320
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	321 def test_env(self):
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	322 newenv = os.environ.copy()
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	323 newenv["FRUIT"] = "orange"
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	324 p = subprocess.Popen([sys.executable, "-c",
43777 0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	325 'import sys,os;'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	326 'sys.stdout.write(os.getenv("FRUIT"))'],
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	327 stdout=subprocess.PIPE,
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	328 env=newenv)
65955 6e1ae323fde6 Add cleanups to stdout/stderr pipes to remove ResourceWarnings. Brian Curtin <brian.curtin@gmail.com> parents: 65799 diff changeset	329 self.addCleanup(p.stdout.close)
41845 9ce8fa0a0899 Make test_subprocess work. Fix universal newlines in io.py. Guido van Rossum <guido@python.org> parents: 41732 diff changeset	330 self.assertEqual(p.stdout.read(), b"orange")
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	331
34671 db7a8310b19a Added three more testcases: Using communicate with only one of Peter Astrand <astrand@lysator.liu.se> parents: 34302 diff changeset	332 def test_communicate_stdin(self):
db7a8310b19a Added three more testcases: Using communicate with only one of Peter Astrand <astrand@lysator.liu.se> parents: 34302 diff changeset	333 p = subprocess.Popen([sys.executable, "-c",
43777 0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	334 'import sys;'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	335 'sys.exit(sys.stdin.read() == "pear")'],
34671 db7a8310b19a Added three more testcases: Using communicate with only one of Peter Astrand <astrand@lysator.liu.se> parents: 34302 diff changeset	336 stdin=subprocess.PIPE)
42927 f3b0ac4e684f More str/bytes goodness. Guido van Rossum <guido@python.org> parents: 42863 diff changeset	337 p.communicate(b"pear")
34671 db7a8310b19a Added three more testcases: Using communicate with only one of Peter Astrand <astrand@lysator.liu.se> parents: 34302 diff changeset	338 self.assertEqual(p.returncode, 1)
db7a8310b19a Added three more testcases: Using communicate with only one of Peter Astrand <astrand@lysator.liu.se> parents: 34302 diff changeset	339
db7a8310b19a Added three more testcases: Using communicate with only one of Peter Astrand <astrand@lysator.liu.se> parents: 34302 diff changeset	340 def test_communicate_stdout(self):
db7a8310b19a Added three more testcases: Using communicate with only one of Peter Astrand <astrand@lysator.liu.se> parents: 34302 diff changeset	341 p = subprocess.Popen([sys.executable, "-c",
db7a8310b19a Added three more testcases: Using communicate with only one of Peter Astrand <astrand@lysator.liu.se> parents: 34302 diff changeset	342 'import sys; sys.stdout.write("pineapple")'],
db7a8310b19a Added three more testcases: Using communicate with only one of Peter Astrand <astrand@lysator.liu.se> parents: 34302 diff changeset	343 stdout=subprocess.PIPE)
db7a8310b19a Added three more testcases: Using communicate with only one of Peter Astrand <astrand@lysator.liu.se> parents: 34302 diff changeset	344 (stdout, stderr) = p.communicate()
41845 9ce8fa0a0899 Make test_subprocess work. Fix universal newlines in io.py. Guido van Rossum <guido@python.org> parents: 41732 diff changeset	345 self.assertEqual(stdout, b"pineapple")
34671 db7a8310b19a Added three more testcases: Using communicate with only one of Peter Astrand <astrand@lysator.liu.se> parents: 34302 diff changeset	346 self.assertEqual(stderr, None)
db7a8310b19a Added three more testcases: Using communicate with only one of Peter Astrand <astrand@lysator.liu.se> parents: 34302 diff changeset	347
db7a8310b19a Added three more testcases: Using communicate with only one of Peter Astrand <astrand@lysator.liu.se> parents: 34302 diff changeset	348 def test_communicate_stderr(self):
db7a8310b19a Added three more testcases: Using communicate with only one of Peter Astrand <astrand@lysator.liu.se> parents: 34302 diff changeset	349 p = subprocess.Popen([sys.executable, "-c",
db7a8310b19a Added three more testcases: Using communicate with only one of Peter Astrand <astrand@lysator.liu.se> parents: 34302 diff changeset	350 'import sys; sys.stderr.write("pineapple")'],
db7a8310b19a Added three more testcases: Using communicate with only one of Peter Astrand <astrand@lysator.liu.se> parents: 34302 diff changeset	351 stderr=subprocess.PIPE)
db7a8310b19a Added three more testcases: Using communicate with only one of Peter Astrand <astrand@lysator.liu.se> parents: 34302 diff changeset	352 (stdout, stderr) = p.communicate()
db7a8310b19a Added three more testcases: Using communicate with only one of Peter Astrand <astrand@lysator.liu.se> parents: 34302 diff changeset	353 self.assertEqual(stdout, None)
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	354 self.assertStderrEqual(stderr, b"pineapple")
34671 db7a8310b19a Added three more testcases: Using communicate with only one of Peter Astrand <astrand@lysator.liu.se> parents: 34302 diff changeset	355
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	356 def test_communicate(self):
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	357 p = subprocess.Popen([sys.executable, "-c",
43777 0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	358 'import sys,os;'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	359 'sys.stderr.write("pineapple");'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	360 'sys.stdout.write(sys.stdin.read())'],
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	361 stdin=subprocess.PIPE,
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	362 stdout=subprocess.PIPE,
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	363 stderr=subprocess.PIPE)
65977 fe8962fc0e11 Fix a number of ResourceWarnings on Windows due to open pipes. Brian Curtin <brian.curtin@gmail.com> parents: 65955 diff changeset	364 self.addCleanup(p.stdout.close)
fe8962fc0e11 Fix a number of ResourceWarnings on Windows due to open pipes. Brian Curtin <brian.curtin@gmail.com> parents: 65955 diff changeset	365 self.addCleanup(p.stderr.close)
fe8962fc0e11 Fix a number of ResourceWarnings on Windows due to open pipes. Brian Curtin <brian.curtin@gmail.com> parents: 65955 diff changeset	366 self.addCleanup(p.stdin.close)
48395 bb65f9b76c1f #2683: Popen.communicate() argument must be bytes. Georg Brandl <georg@python.org> parents: 47557 diff changeset	367 (stdout, stderr) = p.communicate(b"banana")
41732 bbb099d4ec80 Make test_subprocess pass. The subprocess module now reads and writes Guido van Rossum <guido@python.org> parents: 40788 diff changeset	368 self.assertEqual(stdout, b"banana")
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	369 self.assertStderrEqual(stderr, b"pineapple")
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	370
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	371 # Test for the fd leak reported in http://bugs.python.org/issue2791.
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	372 def test_communicate_pipe_fd_leak(self):
67130 38d66980187e Issue #10763: subprocess.communicate() closes stdout and stderr if both are Victor Stinner <victor.stinner@haypocalc.com> parents: 66896 diff changeset	373 for stdin_pipe in (False, True):
38d66980187e Issue #10763: subprocess.communicate() closes stdout and stderr if both are Victor Stinner <victor.stinner@haypocalc.com> parents: 66896 diff changeset	374 for stdout_pipe in (False, True):
38d66980187e Issue #10763: subprocess.communicate() closes stdout and stderr if both are Victor Stinner <victor.stinner@haypocalc.com> parents: 66896 diff changeset	375 for stderr_pipe in (False, True):
38d66980187e Issue #10763: subprocess.communicate() closes stdout and stderr if both are Victor Stinner <victor.stinner@haypocalc.com> parents: 66896 diff changeset	376 options = {}
38d66980187e Issue #10763: subprocess.communicate() closes stdout and stderr if both are Victor Stinner <victor.stinner@haypocalc.com> parents: 66896 diff changeset	377 if stdin_pipe:
38d66980187e Issue #10763: subprocess.communicate() closes stdout and stderr if both are Victor Stinner <victor.stinner@haypocalc.com> parents: 66896 diff changeset	378 options['stdin'] = subprocess.PIPE
38d66980187e Issue #10763: subprocess.communicate() closes stdout and stderr if both are Victor Stinner <victor.stinner@haypocalc.com> parents: 66896 diff changeset	379 if stdout_pipe:
38d66980187e Issue #10763: subprocess.communicate() closes stdout and stderr if both are Victor Stinner <victor.stinner@haypocalc.com> parents: 66896 diff changeset	380 options['stdout'] = subprocess.PIPE
38d66980187e Issue #10763: subprocess.communicate() closes stdout and stderr if both are Victor Stinner <victor.stinner@haypocalc.com> parents: 66896 diff changeset	381 if stderr_pipe:
38d66980187e Issue #10763: subprocess.communicate() closes stdout and stderr if both are Victor Stinner <victor.stinner@haypocalc.com> parents: 66896 diff changeset	382 options['stderr'] = subprocess.PIPE
38d66980187e Issue #10763: subprocess.communicate() closes stdout and stderr if both are Victor Stinner <victor.stinner@haypocalc.com> parents: 66896 diff changeset	383 if not options:
38d66980187e Issue #10763: subprocess.communicate() closes stdout and stderr if both are Victor Stinner <victor.stinner@haypocalc.com> parents: 66896 diff changeset	384 continue
38d66980187e Issue #10763: subprocess.communicate() closes stdout and stderr if both are Victor Stinner <victor.stinner@haypocalc.com> parents: 66896 diff changeset	385 p = subprocess.Popen((sys.executable, "-c", "pass"), **options)
38d66980187e Issue #10763: subprocess.communicate() closes stdout and stderr if both are Victor Stinner <victor.stinner@haypocalc.com> parents: 66896 diff changeset	386 p.communicate()
38d66980187e Issue #10763: subprocess.communicate() closes stdout and stderr if both are Victor Stinner <victor.stinner@haypocalc.com> parents: 66896 diff changeset	387 if p.stdin is not None:
38d66980187e Issue #10763: subprocess.communicate() closes stdout and stderr if both are Victor Stinner <victor.stinner@haypocalc.com> parents: 66896 diff changeset	388 self.assertTrue(p.stdin.closed)
38d66980187e Issue #10763: subprocess.communicate() closes stdout and stderr if both are Victor Stinner <victor.stinner@haypocalc.com> parents: 66896 diff changeset	389 if p.stdout is not None:
38d66980187e Issue #10763: subprocess.communicate() closes stdout and stderr if both are Victor Stinner <victor.stinner@haypocalc.com> parents: 66896 diff changeset	390 self.assertTrue(p.stdout.closed)
38d66980187e Issue #10763: subprocess.communicate() closes stdout and stderr if both are Victor Stinner <victor.stinner@haypocalc.com> parents: 66896 diff changeset	391 if p.stderr is not None:
38d66980187e Issue #10763: subprocess.communicate() closes stdout and stderr if both are Victor Stinner <victor.stinner@haypocalc.com> parents: 66896 diff changeset	392 self.assertTrue(p.stderr.closed)
47557 98827a21e352 Merged revisions 63724,63726,63732,63744,63754-63755,63757-63758,63760,63775,63781-63782,63787,63805-63808,63818-63819,63823-63824 via svnmerge from Georg Brandl <georg@python.org> parents: 47331 diff changeset	393
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	394 def test_communicate_returns(self):
33733 39a13e39d24e Experience with Zope2's tests showed it's a Bad Idea to make unittest Tim Peters <tim.peters@gmail.com> parents: 33732 diff changeset	395 # communicate() should return None if no redirection is active
33732 d9404db29bc6 Wrap long lines. Tim Peters <tim.peters@gmail.com> parents: 33731 diff changeset	396 p = subprocess.Popen([sys.executable, "-c",
d9404db29bc6 Wrap long lines. Tim Peters <tim.peters@gmail.com> parents: 33731 diff changeset	397 "import sys; sys.exit(47)"])
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	398 (stdout, stderr) = p.communicate()
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	399 self.assertEqual(stdout, None)
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	400 self.assertEqual(stderr, None)
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	401
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	402 def test_communicate_pipe_buf(self):
33733 39a13e39d24e Experience with Zope2's tests showed it's a Bad Idea to make unittest Tim Peters <tim.peters@gmail.com> parents: 33732 diff changeset	403 # communicate() with writes larger than pipe_buf
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	404 # This test will probably deadlock rather than fail, if
33731 53c468d0955b Whitespace normalization. Tim Peters <tim.peters@gmail.com> parents: 33723 diff changeset	405 # communicate() does not work properly.
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	406 x, y = os.pipe()
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	407 if mswindows:
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	408 pipe_buf = 512
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	409 else:
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	410 pipe_buf = os.fpathconf(x, "PC_PIPE_BUF")
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	411 os.close(x)
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	412 os.close(y)
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	413 p = subprocess.Popen([sys.executable, "-c",
43777 0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	414 'import sys,os;'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	415 'sys.stdout.write(sys.stdin.read(47));'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	416 'sys.stderr.write("xyz"*%d);'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	417 'sys.stdout.write(sys.stdin.read())' % pipe_buf],
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	418 stdin=subprocess.PIPE,
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	419 stdout=subprocess.PIPE,
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	420 stderr=subprocess.PIPE)
65977 fe8962fc0e11 Fix a number of ResourceWarnings on Windows due to open pipes. Brian Curtin <brian.curtin@gmail.com> parents: 65955 diff changeset	421 self.addCleanup(p.stdout.close)
fe8962fc0e11 Fix a number of ResourceWarnings on Windows due to open pipes. Brian Curtin <brian.curtin@gmail.com> parents: 65955 diff changeset	422 self.addCleanup(p.stderr.close)
fe8962fc0e11 Fix a number of ResourceWarnings on Windows due to open pipes. Brian Curtin <brian.curtin@gmail.com> parents: 65955 diff changeset	423 self.addCleanup(p.stdin.close)
41732 bbb099d4ec80 Make test_subprocess pass. The subprocess module now reads and writes Guido van Rossum <guido@python.org> parents: 40788 diff changeset	424 string_to_write = b"abc"*pipe_buf
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	425 (stdout, stderr) = p.communicate(string_to_write)
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	426 self.assertEqual(stdout, string_to_write)
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	427
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	428 def test_writes_before_communicate(self):
33733 39a13e39d24e Experience with Zope2's tests showed it's a Bad Idea to make unittest Tim Peters <tim.peters@gmail.com> parents: 33732 diff changeset	429 # stdin.write before communicate()
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	430 p = subprocess.Popen([sys.executable, "-c",
43777 0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	431 'import sys,os;'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	432 'sys.stdout.write(sys.stdin.read())'],
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	433 stdin=subprocess.PIPE,
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	434 stdout=subprocess.PIPE,
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	435 stderr=subprocess.PIPE)
65977 fe8962fc0e11 Fix a number of ResourceWarnings on Windows due to open pipes. Brian Curtin <brian.curtin@gmail.com> parents: 65955 diff changeset	436 self.addCleanup(p.stdout.close)
fe8962fc0e11 Fix a number of ResourceWarnings on Windows due to open pipes. Brian Curtin <brian.curtin@gmail.com> parents: 65955 diff changeset	437 self.addCleanup(p.stderr.close)
fe8962fc0e11 Fix a number of ResourceWarnings on Windows due to open pipes. Brian Curtin <brian.curtin@gmail.com> parents: 65955 diff changeset	438 self.addCleanup(p.stdin.close)
42927 f3b0ac4e684f More str/bytes goodness. Guido van Rossum <guido@python.org> parents: 42863 diff changeset	439 p.stdin.write(b"banana")
f3b0ac4e684f More str/bytes goodness. Guido van Rossum <guido@python.org> parents: 42863 diff changeset	440 (stdout, stderr) = p.communicate(b"split")
41732 bbb099d4ec80 Make test_subprocess pass. The subprocess module now reads and writes Guido van Rossum <guido@python.org> parents: 40788 diff changeset	441 self.assertEqual(stdout, b"bananasplit")
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	442 self.assertStderrEqual(stderr, b"")
33731 53c468d0955b Whitespace normalization. Tim Peters <tim.peters@gmail.com> parents: 33723 diff changeset	443
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	444 def test_universal_newlines(self):
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	445 p = subprocess.Popen([sys.executable, "-c",
43777 0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	446 'import sys,os;' + SETBINARY +
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	447 'sys.stdout.write("line1\\n");'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	448 'sys.stdout.flush();'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	449 'sys.stdout.write("line2\\n");'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	450 'sys.stdout.flush();'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	451 'sys.stdout.write("line3\\r\\n");'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	452 'sys.stdout.flush();'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	453 'sys.stdout.write("line4\\r");'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	454 'sys.stdout.flush();'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	455 'sys.stdout.write("\\nline5");'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	456 'sys.stdout.flush();'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	457 'sys.stdout.write("\\nline6");'],
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	458 stdout=subprocess.PIPE,
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	459 universal_newlines=1)
65955 6e1ae323fde6 Add cleanups to stdout/stderr pipes to remove ResourceWarnings. Brian Curtin <brian.curtin@gmail.com> parents: 65799 diff changeset	460 self.addCleanup(p.stdout.close)
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	461 stdout = p.stdout.read()
41732 bbb099d4ec80 Make test_subprocess pass. The subprocess module now reads and writes Guido van Rossum <guido@python.org> parents: 40788 diff changeset	462 self.assertEqual(stdout, "line1\nline2\nline3\nline4\nline5\nline6")
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	463
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	464 def test_universal_newlines_communicate(self):
33733 39a13e39d24e Experience with Zope2's tests showed it's a Bad Idea to make unittest Tim Peters <tim.peters@gmail.com> parents: 33732 diff changeset	465 # universal newlines through communicate()
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	466 p = subprocess.Popen([sys.executable, "-c",
43777 0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	467 'import sys,os;' + SETBINARY +
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	468 'sys.stdout.write("line1\\n");'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	469 'sys.stdout.flush();'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	470 'sys.stdout.write("line2\\n");'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	471 'sys.stdout.flush();'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	472 'sys.stdout.write("line3\\r\\n");'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	473 'sys.stdout.flush();'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	474 'sys.stdout.write("line4\\r");'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	475 'sys.stdout.flush();'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	476 'sys.stdout.write("\\nline5");'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	477 'sys.stdout.flush();'
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	478 'sys.stdout.write("\\nline6");'],
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	479 stdout=subprocess.PIPE, stderr=subprocess.PIPE,
0d462d789b18 Merging the py3k-pep3137 branch back into the py3k branch. Guido van Rossum <guido@python.org> parents: 43684 diff changeset	480 universal_newlines=1)
65977 fe8962fc0e11 Fix a number of ResourceWarnings on Windows due to open pipes. Brian Curtin <brian.curtin@gmail.com> parents: 65955 diff changeset	481 self.addCleanup(p.stdout.close)
fe8962fc0e11 Fix a number of ResourceWarnings on Windows due to open pipes. Brian Curtin <brian.curtin@gmail.com> parents: 65955 diff changeset	482 self.addCleanup(p.stderr.close)
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	483 (stdout, stderr) = p.communicate()
41845 9ce8fa0a0899 Make test_subprocess work. Fix universal newlines in io.py. Guido van Rossum <guido@python.org> parents: 41732 diff changeset	484 self.assertEqual(stdout, "line1\nline2\nline3\nline4\nline5\nline6")
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	485
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	486 def test_no_leaking(self):
33733 39a13e39d24e Experience with Zope2's tests showed it's a Bad Idea to make unittest Tim Peters <tim.peters@gmail.com> parents: 33732 diff changeset	487 # Make sure we leak no resources
64814 77425f2d7d76 Merged revisions 84883 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 64810 diff changeset	488 if not mswindows:
34673 00e47eb136b2 Only run extensive subprocess tests if -usubprocess to regrtest is specified. Fixes #1124637 Peter Astrand <astrand@lysator.liu.se> parents: 34671 diff changeset	489 max_handles = 1026 # too much for most UNIX systems
00e47eb136b2 Only run extensive subprocess tests if -usubprocess to regrtest is specified. Fixes #1124637 Peter Astrand <astrand@lysator.liu.se> parents: 34671 diff changeset	490 else:
64814 77425f2d7d76 Merged revisions 84883 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 64810 diff changeset	491 max_handles = 2050 # too much for (at least some) Windows setups
77425f2d7d76 Merged revisions 84883 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 64810 diff changeset	492 handles = []
68491 141bbcb19c06 Fix the @test_NNNN_tmp file terds being left in whatever your $PWD was Gregory P. Smith <greg@krypto.org> parents: 68432 diff changeset	493 tmpdir = tempfile.mkdtemp()
64814 77425f2d7d76 Merged revisions 84883 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 64810 diff changeset	494 try:
77425f2d7d76 Merged revisions 84883 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 64810 diff changeset	495 for i in range(max_handles):
77425f2d7d76 Merged revisions 84883 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 64810 diff changeset	496 try:
68491 141bbcb19c06 Fix the @test_NNNN_tmp file terds being left in whatever your $PWD was Gregory P. Smith <greg@krypto.org> parents: 68432 diff changeset	497 tmpfile = os.path.join(tmpdir, support.TESTFN)
141bbcb19c06 Fix the @test_NNNN_tmp file terds being left in whatever your $PWD was Gregory P. Smith <greg@krypto.org> parents: 68432 diff changeset	498 handles.append(os.open(tmpfile, os.O_WRONLY\|os.O_CREAT))
64814 77425f2d7d76 Merged revisions 84883 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 64810 diff changeset	499 except OSError as e:
77425f2d7d76 Merged revisions 84883 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 64810 diff changeset	500 if e.errno != errno.EMFILE:
77425f2d7d76 Merged revisions 84883 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 64810 diff changeset	501 raise
77425f2d7d76 Merged revisions 84883 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 64810 diff changeset	502 break
77425f2d7d76 Merged revisions 84883 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 64810 diff changeset	503 else:
77425f2d7d76 Merged revisions 84883 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 64810 diff changeset	504 self.skipTest("failed to reach the file descriptor limit "
77425f2d7d76 Merged revisions 84883 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 64810 diff changeset	505 "(tried %d)" % max_handles)
77425f2d7d76 Merged revisions 84883 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 64810 diff changeset	506 # Close a couple of them (should be enough for a subprocess)
77425f2d7d76 Merged revisions 84883 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 64810 diff changeset	507 for i in range(10):
77425f2d7d76 Merged revisions 84883 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 64810 diff changeset	508 os.close(handles.pop())
77425f2d7d76 Merged revisions 84883 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 64810 diff changeset	509 # Loop creating some subprocesses. If one of them leaks some fds,
77425f2d7d76 Merged revisions 84883 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 64810 diff changeset	510 # the next loop iteration will fail by reaching the max fd limit.
77425f2d7d76 Merged revisions 84883 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 64810 diff changeset	511 for i in range(15):
77425f2d7d76 Merged revisions 84883 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 64810 diff changeset	512 p = subprocess.Popen([sys.executable, "-c",
77425f2d7d76 Merged revisions 84883 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 64810 diff changeset	513 "import sys;"
77425f2d7d76 Merged revisions 84883 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 64810 diff changeset	514 "sys.stdout.write(sys.stdin.read())"],
77425f2d7d76 Merged revisions 84883 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 64810 diff changeset	515 stdin=subprocess.PIPE,
77425f2d7d76 Merged revisions 84883 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 64810 diff changeset	516 stdout=subprocess.PIPE,
77425f2d7d76 Merged revisions 84883 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 64810 diff changeset	517 stderr=subprocess.PIPE)
77425f2d7d76 Merged revisions 84883 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 64810 diff changeset	518 data = p.communicate(b"lime")[0]
77425f2d7d76 Merged revisions 84883 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 64810 diff changeset	519 self.assertEqual(data, b"lime")
77425f2d7d76 Merged revisions 84883 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 64810 diff changeset	520 finally:
77425f2d7d76 Merged revisions 84883 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 64810 diff changeset	521 for h in handles:
77425f2d7d76 Merged revisions 84883 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 64810 diff changeset	522 os.close(h)
68491 141bbcb19c06 Fix the @test_NNNN_tmp file terds being left in whatever your $PWD was Gregory P. Smith <greg@krypto.org> parents: 68432 diff changeset	523 shutil.rmtree(tmpdir)
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	524
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	525 def test_list2cmdline(self):
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	526 self.assertEqual(subprocess.list2cmdline(['a b c', 'd', 'e']),
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	527 '"a b c" d e')
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	528 self.assertEqual(subprocess.list2cmdline(['ab"c', '\\', 'd']),
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	529 'ab\\"c \\ d')
44810 fa6cee14dcba Merged revisions 60094-60123 via svnmerge from Christian Heimes <christian@cheimes.de> parents: 44373 diff changeset	530 self.assertEqual(subprocess.list2cmdline(['ab"c', ' \\', 'd']),
fa6cee14dcba Merged revisions 60094-60123 via svnmerge from Christian Heimes <christian@cheimes.de> parents: 44373 diff changeset	531 'ab\\"c " \\\\" d')
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	532 self.assertEqual(subprocess.list2cmdline(['a\\\\\\b', 'de fg', 'h']),
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	533 'a\\\\\\b "de fg" h')
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	534 self.assertEqual(subprocess.list2cmdline(['a\\"b', 'c', 'd']),
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	535 'a\\\\\\"b c d')
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	536 self.assertEqual(subprocess.list2cmdline(['a\\\\b c', 'd', 'e']),
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	537 '"a\\\\b c" d e')
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	538 self.assertEqual(subprocess.list2cmdline(['a\\\\b\\ c', 'd', 'e']),
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	539 '"a\\\\b\\ c" d e')
40643 47f9efdc0855 Merged revisions 53304-53433,53435-53450 via svnmerge from Thomas Wouters <thomas@python.org> parents: 40587 diff changeset	540 self.assertEqual(subprocess.list2cmdline(['ab', '']),
47f9efdc0855 Merged revisions 53304-53433,53435-53450 via svnmerge from Thomas Wouters <thomas@python.org> parents: 40587 diff changeset	541 'ab ""')
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	542
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	543
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	544 def test_poll(self):
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	545 p = subprocess.Popen([sys.executable,
33738 a4f747e72138 Kill several problems at once: test_poll() failed sometimes for me. Tim Peters <tim.peters@gmail.com> parents: 33737 diff changeset	546 "-c", "import time; time.sleep(1)"])
a4f747e72138 Kill several problems at once: test_poll() failed sometimes for me. Tim Peters <tim.peters@gmail.com> parents: 33737 diff changeset	547 count = 0
a4f747e72138 Kill several problems at once: test_poll() failed sometimes for me. Tim Peters <tim.peters@gmail.com> parents: 33737 diff changeset	548 while p.poll() is None:
a4f747e72138 Kill several problems at once: test_poll() failed sometimes for me. Tim Peters <tim.peters@gmail.com> parents: 33737 diff changeset	549 time.sleep(0.1)
a4f747e72138 Kill several problems at once: test_poll() failed sometimes for me. Tim Peters <tim.peters@gmail.com> parents: 33737 diff changeset	550 count += 1
a4f747e72138 Kill several problems at once: test_poll() failed sometimes for me. Tim Peters <tim.peters@gmail.com> parents: 33737 diff changeset	551 # We expect that the poll loop probably went around about 10 times,
a4f747e72138 Kill several problems at once: test_poll() failed sometimes for me. Tim Peters <tim.peters@gmail.com> parents: 33737 diff changeset	552 # but, based on system scheduling we can't control, it's possible
a4f747e72138 Kill several problems at once: test_poll() failed sometimes for me. Tim Peters <tim.peters@gmail.com> parents: 33737 diff changeset	553 # poll() never returned None. It "should be" very rare that it
a4f747e72138 Kill several problems at once: test_poll() failed sometimes for me. Tim Peters <tim.peters@gmail.com> parents: 33737 diff changeset	554 # didn't go around at least twice.
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	555 self.assertGreaterEqual(count, 2)
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	556 # Subsequent invocations should just return the returncode
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	557 self.assertEqual(p.poll(), 0)
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	558
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	559
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	560 def test_wait(self):
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	561 p = subprocess.Popen([sys.executable,
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	562 "-c", "import time; time.sleep(2)"])
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	563 self.assertEqual(p.wait(), 0)
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	564 # Subsequent invocations should just return the returncode
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	565 self.assertEqual(p.wait(), 0)
33731 53c468d0955b Whitespace normalization. Tim Peters <tim.peters@gmail.com> parents: 33723 diff changeset	566
34052 79d2ae0a62ef Raise TypeError if bufsize argument is not an integer. Patch 1071755, slightly modified. Peter Astrand <astrand@lysator.liu.se> parents: 33998 diff changeset	567
79d2ae0a62ef Raise TypeError if bufsize argument is not an integer. Patch 1071755, slightly modified. Peter Astrand <astrand@lysator.liu.se> parents: 33998 diff changeset	568 def test_invalid_bufsize(self):
79d2ae0a62ef Raise TypeError if bufsize argument is not an integer. Patch 1071755, slightly modified. Peter Astrand <astrand@lysator.liu.se> parents: 33998 diff changeset	569 # an invalid type of the bufsize argument should raise
79d2ae0a62ef Raise TypeError if bufsize argument is not an integer. Patch 1071755, slightly modified. Peter Astrand <astrand@lysator.liu.se> parents: 33998 diff changeset	570 # TypeError.
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	571 with self.assertRaises(TypeError):
34052 79d2ae0a62ef Raise TypeError if bufsize argument is not an integer. Patch 1071755, slightly modified. Peter Astrand <astrand@lysator.liu.se> parents: 33998 diff changeset	572 subprocess.Popen([sys.executable, "-c", "pass"], "orange")
79d2ae0a62ef Raise TypeError if bufsize argument is not an integer. Patch 1071755, slightly modified. Peter Astrand <astrand@lysator.liu.se> parents: 33998 diff changeset	573
41984 e420039368b7 The bufsize argument to Popen() should accept None meaning the default (0). Guido van Rossum <guido@python.org> parents: 41969 diff changeset	574 def test_bufsize_is_none(self):
e420039368b7 The bufsize argument to Popen() should accept None meaning the default (0). Guido van Rossum <guido@python.org> parents: 41969 diff changeset	575 # bufsize=None should be the same as bufsize=0.
e420039368b7 The bufsize argument to Popen() should accept None meaning the default (0). Guido van Rossum <guido@python.org> parents: 41969 diff changeset	576 p = subprocess.Popen([sys.executable, "-c", "pass"], None)
e420039368b7 The bufsize argument to Popen() should accept None meaning the default (0). Guido van Rossum <guido@python.org> parents: 41969 diff changeset	577 self.assertEqual(p.wait(), 0)
e420039368b7 The bufsize argument to Popen() should accept None meaning the default (0). Guido van Rossum <guido@python.org> parents: 41969 diff changeset	578 # Again with keyword arg
e420039368b7 The bufsize argument to Popen() should accept None meaning the default (0). Guido van Rossum <guido@python.org> parents: 41969 diff changeset	579 p = subprocess.Popen([sys.executable, "-c", "pass"], bufsize=None)
e420039368b7 The bufsize argument to Popen() should accept None meaning the default (0). Guido van Rossum <guido@python.org> parents: 41969 diff changeset	580 self.assertEqual(p.wait(), 0)
e420039368b7 The bufsize argument to Popen() should accept None meaning the default (0). Guido van Rossum <guido@python.org> parents: 41969 diff changeset	581
51721 33d5d7daa8c7 Merged revisions 69576,69579-69580,69589,69619-69620,69633,69703-69704,69728-69730 via svnmerge from Benjamin Peterson <benjamin@python.org> parents: 50074 diff changeset	582 def test_leaking_fds_on_error(self):
33d5d7daa8c7 Merged revisions 69576,69579-69580,69589,69619-69620,69633,69703-69704,69728-69730 via svnmerge from Benjamin Peterson <benjamin@python.org> parents: 50074 diff changeset	583 # see bug #5179: Popen leaks file descriptors to PIPEs if
33d5d7daa8c7 Merged revisions 69576,69579-69580,69589,69619-69620,69633,69703-69704,69728-69730 via svnmerge from Benjamin Peterson <benjamin@python.org> parents: 50074 diff changeset	584 # the child fails to execute; this will eventually exhaust
33d5d7daa8c7 Merged revisions 69576,69579-69580,69589,69619-69620,69633,69703-69704,69728-69730 via svnmerge from Benjamin Peterson <benjamin@python.org> parents: 50074 diff changeset	585 # the maximum number of open fds. 1024 seems a very common
33d5d7daa8c7 Merged revisions 69576,69579-69580,69589,69619-69620,69633,69703-69704,69728-69730 via svnmerge from Benjamin Peterson <benjamin@python.org> parents: 50074 diff changeset	586 # value for that limit, but Windows has 2048, so we loop
33d5d7daa8c7 Merged revisions 69576,69579-69580,69589,69619-69620,69633,69703-69704,69728-69730 via svnmerge from Benjamin Peterson <benjamin@python.org> parents: 50074 diff changeset	587 # 1024 times (each call leaked two fds).
33d5d7daa8c7 Merged revisions 69576,69579-69580,69589,69619-69620,69633,69703-69704,69728-69730 via svnmerge from Benjamin Peterson <benjamin@python.org> parents: 50074 diff changeset	588 for i in range(1024):
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	589 # Windows raises IOError. Others raise OSError.
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	590 with self.assertRaises(EnvironmentError) as c:
51721 33d5d7daa8c7 Merged revisions 69576,69579-69580,69589,69619-69620,69633,69703-69704,69728-69730 via svnmerge from Benjamin Peterson <benjamin@python.org> parents: 50074 diff changeset	591 subprocess.Popen(['nonexisting_i_hope'],
33d5d7daa8c7 Merged revisions 69576,69579-69580,69589,69619-69620,69633,69703-69704,69728-69730 via svnmerge from Benjamin Peterson <benjamin@python.org> parents: 50074 diff changeset	592 stdout=subprocess.PIPE,
33d5d7daa8c7 Merged revisions 69576,69579-69580,69589,69619-69620,69633,69703-69704,69728-69730 via svnmerge from Benjamin Peterson <benjamin@python.org> parents: 50074 diff changeset	593 stderr=subprocess.PIPE)
68433 67f4ef6094ed Merge fix for #11490 from 3.1. R David Murray <rdmurray@bitdance.com> parents: 68257 68432 diff changeset	594 # ignore errors that indicate the command was not found
68437 f65aad1226b7 Fix whitespace R David Murray <rdmurray@bitdance.com> parents: 68433 diff changeset	595 if c.exception.errno not in (errno.ENOENT, errno.EACCES):
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	596 raise c.exception
51721 33d5d7daa8c7 Merged revisions 69576,69579-69580,69589,69619-69620,69633,69703-69704,69728-69730 via svnmerge from Benjamin Peterson <benjamin@python.org> parents: 50074 diff changeset	597
61798 73416b1bbcd5 Issue #8780: Only backport the new test, the fix is not needed Victor Stinner <victor.stinner@haypocalc.com> parents: 61014 diff changeset	598 def test_issue8780(self):
73416b1bbcd5 Issue #8780: Only backport the new test, the fix is not needed Victor Stinner <victor.stinner@haypocalc.com> parents: 61014 diff changeset	599 # Ensure that stdout is inherited from the parent
73416b1bbcd5 Issue #8780: Only backport the new test, the fix is not needed Victor Stinner <victor.stinner@haypocalc.com> parents: 61014 diff changeset	600 # if stdout=PIPE is not used
73416b1bbcd5 Issue #8780: Only backport the new test, the fix is not needed Victor Stinner <victor.stinner@haypocalc.com> parents: 61014 diff changeset	601 code = ';'.join((
73416b1bbcd5 Issue #8780: Only backport the new test, the fix is not needed Victor Stinner <victor.stinner@haypocalc.com> parents: 61014 diff changeset	602 'import subprocess, sys',
73416b1bbcd5 Issue #8780: Only backport the new test, the fix is not needed Victor Stinner <victor.stinner@haypocalc.com> parents: 61014 diff changeset	603 'retcode = subprocess.call('
73416b1bbcd5 Issue #8780: Only backport the new test, the fix is not needed Victor Stinner <victor.stinner@haypocalc.com> parents: 61014 diff changeset	604 "[sys.executable, '-c', 'print(\"Hello World!\")'])",
73416b1bbcd5 Issue #8780: Only backport the new test, the fix is not needed Victor Stinner <victor.stinner@haypocalc.com> parents: 61014 diff changeset	605 'assert retcode == 0'))
73416b1bbcd5 Issue #8780: Only backport the new test, the fix is not needed Victor Stinner <victor.stinner@haypocalc.com> parents: 61014 diff changeset	606 output = subprocess.check_output([sys.executable, '-c', code])
66340 2beb25369289 Merged revisions 86596 via svnmerge from Ezio Melotti <ezio.melotti@gmail.com> parents: 64853 diff changeset	607 self.assertTrue(output.startswith(b'Hello World!'), ascii(output))
61798 73416b1bbcd5 Issue #8780: Only backport the new test, the fix is not needed Victor Stinner <victor.stinner@haypocalc.com> parents: 61014 diff changeset	608
63774 bb2608f73eb7 Issue #3210: Ensure stdio handles are closed if CreateProcess fails Tim Golden <mail@timgolden.me.uk> parents: 63142 diff changeset	609 def test_handles_closed_on_exception(self):
bb2608f73eb7 Issue #3210: Ensure stdio handles are closed if CreateProcess fails Tim Golden <mail@timgolden.me.uk> parents: 63142 diff changeset	610 # If CreateProcess exits with an error, ensure the
bb2608f73eb7 Issue #3210: Ensure stdio handles are closed if CreateProcess fails Tim Golden <mail@timgolden.me.uk> parents: 63142 diff changeset	611 # duplicate output handles are released
63772 ad292fce79f9 Issue #3210: Ensure stdio handles are closed if CreateProcess fails Tim Golden <mail@timgolden.me.uk> parents: 63727 diff changeset	612 ifhandle, ifname = mkstemp()
ad292fce79f9 Issue #3210: Ensure stdio handles are closed if CreateProcess fails Tim Golden <mail@timgolden.me.uk> parents: 63727 diff changeset	613 ofhandle, ofname = mkstemp()
ad292fce79f9 Issue #3210: Ensure stdio handles are closed if CreateProcess fails Tim Golden <mail@timgolden.me.uk> parents: 63727 diff changeset	614 efhandle, efname = mkstemp()
63774 bb2608f73eb7 Issue #3210: Ensure stdio handles are closed if CreateProcess fails Tim Golden <mail@timgolden.me.uk> parents: 63142 diff changeset	615 try:
bb2608f73eb7 Issue #3210: Ensure stdio handles are closed if CreateProcess fails Tim Golden <mail@timgolden.me.uk> parents: 63142 diff changeset	616 subprocess.Popen (["*"], stdin=ifhandle, stdout=ofhandle,
bb2608f73eb7 Issue #3210: Ensure stdio handles are closed if CreateProcess fails Tim Golden <mail@timgolden.me.uk> parents: 63142 diff changeset	617 stderr=efhandle)
bb2608f73eb7 Issue #3210: Ensure stdio handles are closed if CreateProcess fails Tim Golden <mail@timgolden.me.uk> parents: 63142 diff changeset	618 except OSError:
bb2608f73eb7 Issue #3210: Ensure stdio handles are closed if CreateProcess fails Tim Golden <mail@timgolden.me.uk> parents: 63142 diff changeset	619 os.close(ifhandle)
bb2608f73eb7 Issue #3210: Ensure stdio handles are closed if CreateProcess fails Tim Golden <mail@timgolden.me.uk> parents: 63142 diff changeset	620 os.remove(ifname)
bb2608f73eb7 Issue #3210: Ensure stdio handles are closed if CreateProcess fails Tim Golden <mail@timgolden.me.uk> parents: 63142 diff changeset	621 os.close(ofhandle)
bb2608f73eb7 Issue #3210: Ensure stdio handles are closed if CreateProcess fails Tim Golden <mail@timgolden.me.uk> parents: 63142 diff changeset	622 os.remove(ofname)
bb2608f73eb7 Issue #3210: Ensure stdio handles are closed if CreateProcess fails Tim Golden <mail@timgolden.me.uk> parents: 63142 diff changeset	623 os.close(efhandle)
bb2608f73eb7 Issue #3210: Ensure stdio handles are closed if CreateProcess fails Tim Golden <mail@timgolden.me.uk> parents: 63142 diff changeset	624 os.remove(efname)
bb2608f73eb7 Issue #3210: Ensure stdio handles are closed if CreateProcess fails Tim Golden <mail@timgolden.me.uk> parents: 63142 diff changeset	625 self.assertFalse(os.path.exists(ifname))
bb2608f73eb7 Issue #3210: Ensure stdio handles are closed if CreateProcess fails Tim Golden <mail@timgolden.me.uk> parents: 63142 diff changeset	626 self.assertFalse(os.path.exists(ofname))
bb2608f73eb7 Issue #3210: Ensure stdio handles are closed if CreateProcess fails Tim Golden <mail@timgolden.me.uk> parents: 63142 diff changeset	627 self.assertFalse(os.path.exists(efname))
bb2608f73eb7 Issue #3210: Ensure stdio handles are closed if CreateProcess fails Tim Golden <mail@timgolden.me.uk> parents: 63142 diff changeset	628
69154 158495d49f58 Issue #10963: Ensure that subprocess.communicate() never raises EPIPE. Ross Lagerwall <rosslagerwall@gmail.com> parents: 68980 diff changeset	629 def test_communicate_epipe(self):
158495d49f58 Issue #10963: Ensure that subprocess.communicate() never raises EPIPE. Ross Lagerwall <rosslagerwall@gmail.com> parents: 68980 diff changeset	630 # Issue 10963: communicate() should hide EPIPE
158495d49f58 Issue #10963: Ensure that subprocess.communicate() never raises EPIPE. Ross Lagerwall <rosslagerwall@gmail.com> parents: 68980 diff changeset	631 p = subprocess.Popen([sys.executable, "-c", 'pass'],
158495d49f58 Issue #10963: Ensure that subprocess.communicate() never raises EPIPE. Ross Lagerwall <rosslagerwall@gmail.com> parents: 68980 diff changeset	632 stdin=subprocess.PIPE,
158495d49f58 Issue #10963: Ensure that subprocess.communicate() never raises EPIPE. Ross Lagerwall <rosslagerwall@gmail.com> parents: 68980 diff changeset	633 stdout=subprocess.PIPE,
158495d49f58 Issue #10963: Ensure that subprocess.communicate() never raises EPIPE. Ross Lagerwall <rosslagerwall@gmail.com> parents: 68980 diff changeset	634 stderr=subprocess.PIPE)
158495d49f58 Issue #10963: Ensure that subprocess.communicate() never raises EPIPE. Ross Lagerwall <rosslagerwall@gmail.com> parents: 68980 diff changeset	635 self.addCleanup(p.stdout.close)
158495d49f58 Issue #10963: Ensure that subprocess.communicate() never raises EPIPE. Ross Lagerwall <rosslagerwall@gmail.com> parents: 68980 diff changeset	636 self.addCleanup(p.stderr.close)
158495d49f58 Issue #10963: Ensure that subprocess.communicate() never raises EPIPE. Ross Lagerwall <rosslagerwall@gmail.com> parents: 68980 diff changeset	637 self.addCleanup(p.stdin.close)
158495d49f58 Issue #10963: Ensure that subprocess.communicate() never raises EPIPE. Ross Lagerwall <rosslagerwall@gmail.com> parents: 68980 diff changeset	638 p.communicate(b"x" * 2**20)
158495d49f58 Issue #10963: Ensure that subprocess.communicate() never raises EPIPE. Ross Lagerwall <rosslagerwall@gmail.com> parents: 68980 diff changeset	639
158495d49f58 Issue #10963: Ensure that subprocess.communicate() never raises EPIPE. Ross Lagerwall <rosslagerwall@gmail.com> parents: 68980 diff changeset	640 def test_communicate_epipe_only_stdin(self):
158495d49f58 Issue #10963: Ensure that subprocess.communicate() never raises EPIPE. Ross Lagerwall <rosslagerwall@gmail.com> parents: 68980 diff changeset	641 # Issue 10963: communicate() should hide EPIPE
158495d49f58 Issue #10963: Ensure that subprocess.communicate() never raises EPIPE. Ross Lagerwall <rosslagerwall@gmail.com> parents: 68980 diff changeset	642 p = subprocess.Popen([sys.executable, "-c", 'pass'],
158495d49f58 Issue #10963: Ensure that subprocess.communicate() never raises EPIPE. Ross Lagerwall <rosslagerwall@gmail.com> parents: 68980 diff changeset	643 stdin=subprocess.PIPE)
158495d49f58 Issue #10963: Ensure that subprocess.communicate() never raises EPIPE. Ross Lagerwall <rosslagerwall@gmail.com> parents: 68980 diff changeset	644 self.addCleanup(p.stdin.close)
158495d49f58 Issue #10963: Ensure that subprocess.communicate() never raises EPIPE. Ross Lagerwall <rosslagerwall@gmail.com> parents: 68980 diff changeset	645 time.sleep(2)
158495d49f58 Issue #10963: Ensure that subprocess.communicate() never raises EPIPE. Ross Lagerwall <rosslagerwall@gmail.com> parents: 68980 diff changeset	646 p.communicate(b"x" * 2**20)
158495d49f58 Issue #10963: Ensure that subprocess.communicate() never raises EPIPE. Ross Lagerwall <rosslagerwall@gmail.com> parents: 68980 diff changeset	647
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	648
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	649 # context manager
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	650 class _SuppressCoreFiles(object):
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	651 """Try to prevent core files from being created."""
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	652 old_limit = None
33731 53c468d0955b Whitespace normalization. Tim Peters <tim.peters@gmail.com> parents: 33723 diff changeset	653
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	654 def __enter__(self):
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	655 """Try to save previous ulimit, then set it to (0, 0)."""
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	656 try:
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	657 import resource
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	658 self.old_limit = resource.getrlimit(resource.RLIMIT_CORE)
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	659 resource.setrlimit(resource.RLIMIT_CORE, (0, 0))
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	660 except (ImportError, ValueError, resource.error):
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	661 pass
51721 33d5d7daa8c7 Merged revisions 69576,69579-69580,69589,69619-69620,69633,69703-69704,69728-69730 via svnmerge from Benjamin Peterson <benjamin@python.org> parents: 50074 diff changeset	662
63141 2ad8e8ce46c6 Workaround for issue 4047: in some configurations of Ronald Oussoren <ronaldoussoren@mac.com> parents: 63094 diff changeset	663 if sys.platform == 'darwin':
2ad8e8ce46c6 Workaround for issue 4047: in some configurations of Ronald Oussoren <ronaldoussoren@mac.com> parents: 63094 diff changeset	664 # Check if the 'Crash Reporter' on OSX was configured
2ad8e8ce46c6 Workaround for issue 4047: in some configurations of Ronald Oussoren <ronaldoussoren@mac.com> parents: 63094 diff changeset	665 # in 'Developer' mode and warn that it will get triggered
2ad8e8ce46c6 Workaround for issue 4047: in some configurations of Ronald Oussoren <ronaldoussoren@mac.com> parents: 63094 diff changeset	666 # when it is.
2ad8e8ce46c6 Workaround for issue 4047: in some configurations of Ronald Oussoren <ronaldoussoren@mac.com> parents: 63094 diff changeset	667 #
2ad8e8ce46c6 Workaround for issue 4047: in some configurations of Ronald Oussoren <ronaldoussoren@mac.com> parents: 63094 diff changeset	668 # This assumes that this context manager is used in tests
2ad8e8ce46c6 Workaround for issue 4047: in some configurations of Ronald Oussoren <ronaldoussoren@mac.com> parents: 63094 diff changeset	669 # that might trigger the next manager.
2ad8e8ce46c6 Workaround for issue 4047: in some configurations of Ronald Oussoren <ronaldoussoren@mac.com> parents: 63094 diff changeset	670 value = subprocess.Popen(['/usr/bin/defaults', 'read',
63142 5a698a152f30 Merged revisions 83067 via svnmerge from Ronald Oussoren <ronaldoussoren@mac.com> parents: 63095 diff changeset	671 'com.apple.CrashReporter', 'DialogType'],
5a698a152f30 Merged revisions 83067 via svnmerge from Ronald Oussoren <ronaldoussoren@mac.com> parents: 63095 diff changeset	672 stdout=subprocess.PIPE).communicate()[0]
63141 2ad8e8ce46c6 Workaround for issue 4047: in some configurations of Ronald Oussoren <ronaldoussoren@mac.com> parents: 63094 diff changeset	673 if value.strip() == b'developer':
2ad8e8ce46c6 Workaround for issue 4047: in some configurations of Ronald Oussoren <ronaldoussoren@mac.com> parents: 63094 diff changeset	674 print("this tests triggers the Crash Reporter, "
2ad8e8ce46c6 Workaround for issue 4047: in some configurations of Ronald Oussoren <ronaldoussoren@mac.com> parents: 63094 diff changeset	675 "that is intentional", end='')
2ad8e8ce46c6 Workaround for issue 4047: in some configurations of Ronald Oussoren <ronaldoussoren@mac.com> parents: 63094 diff changeset	676 sys.stdout.flush()
63142 5a698a152f30 Merged revisions 83067 via svnmerge from Ronald Oussoren <ronaldoussoren@mac.com> parents: 63095 diff changeset	677
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	678 def __exit__(self, *args):
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	679 """Return core file behavior to default."""
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	680 if self.old_limit is None:
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	681 return
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	682 try:
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	683 import resource
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	684 resource.setrlimit(resource.RLIMIT_CORE, self.old_limit)
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	685 except (ImportError, ValueError, resource.error):
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	686 pass
63142 5a698a152f30 Merged revisions 83067 via svnmerge from Ronald Oussoren <ronaldoussoren@mac.com> parents: 63095 diff changeset	687
5a698a152f30 Merged revisions 83067 via svnmerge from Ronald Oussoren <ronaldoussoren@mac.com> parents: 63095 diff changeset	688
59340 04955e7eb597 Merged revisions 78661-78662 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59216 diff changeset	689 @unittest.skipIf(mswindows, "POSIX specific tests")
60057 bb2ca1b63321 Merged revisions 79344,79346,79350 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59723 diff changeset	690 class POSIXProcessTestCase(BaseTestCase):
59340 04955e7eb597 Merged revisions 78661-78662 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59216 diff changeset	691
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	692 def test_exceptions(self):
59605 768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	693 nonexistent_dir = "/_this/pa.th/does/not/exist"
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	694 try:
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	695 os.chdir(nonexistent_dir)
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	696 except OSError as e:
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	697 # This avoids hard coding the errno value or the OS perror()
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	698 # string and instead capture the exception that we want to see
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	699 # below for comparison.
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	700 desired_exception = e
66307 c06ed25e69b8 add filename to ENOENT message #4925 Benjamin Peterson <benjamin@python.org> parents: 65977 diff changeset	701 desired_exception.strerror += ': ' + repr(sys.executable)
59605 768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	702 else:
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	703 self.fail("chdir to nonexistant directory %s succeeded." %
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	704 nonexistent_dir)
39442 654c380cf8b9 Merged revisions 46753-51188 via svnmerge from Thomas Wouters <thomas@python.org> parents: 38453 diff changeset	705
59605 768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	706 # Error in the child re-raised in the parent.
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	707 try:
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	708 p = subprocess.Popen([sys.executable, "-c", ""],
59605 768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	709 cwd=nonexistent_dir)
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	710 except OSError as e:
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	711 # Test that the child process chdir failure actually makes
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	712 # it up to the parent process as the correct exception.
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	713 self.assertEqual(desired_exception.errno, e.errno)
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	714 self.assertEqual(desired_exception.strerror, e.strerror)
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	715 else:
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	716 self.fail("Expected OSError: %s" % desired_exception)
39442 654c380cf8b9 Merged revisions 46753-51188 via svnmerge from Thomas Wouters <thomas@python.org> parents: 38453 diff changeset	717
59605 768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	718 def test_restore_signals(self):
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	719 # Code coverage for both values of restore_signals to make sure it
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	720 # at least does not blow up.
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	721 # A test for behavior would be complex. Contributions welcome.
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	722 subprocess.call([sys.executable, "-c", ""], restore_signals=True)
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	723 subprocess.call([sys.executable, "-c", ""], restore_signals=False)
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	724
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	725 def test_start_new_session(self):
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	726 # For code coverage of calling setsid(). We don't care if we get an
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	727 # EPERM error from it depending on the test execution environment, that
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	728 # still indicates that it was called.
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	729 try:
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	730 output = subprocess.check_output(
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	731 [sys.executable, "-c",
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	732 "import os; print(os.getpgid(os.getpid()))"],
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	733 start_new_session=True)
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	734 except OSError as e:
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	735 if e.errno != errno.EPERM:
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	736 raise
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	737 else:
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	738 parent_pgid = os.getpgid(os.getpid())
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	739 child_pgid = int(output)
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	740 self.assertNotEqual(parent_pgid, child_pgid)
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	741
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	742 def test_run_abort(self):
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	743 # returncode handles signal termination
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	744 with _SuppressCoreFiles():
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	745 p = subprocess.Popen([sys.executable, "-c",
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	746 'import os; os.abort()'])
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	747 p.wait()
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	748 self.assertEqual(-p.returncode, signal.SIGABRT)
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	749
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	750 def test_preexec(self):
59605 768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	751 # DISCLAIMER: Setting environment variables is not a good use
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	752 # of a preexec_fn. This is merely a test.
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	753 p = subprocess.Popen([sys.executable, "-c",
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	754 'import sys,os;'
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	755 'sys.stdout.write(os.getenv("FRUIT"))'],
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	756 stdout=subprocess.PIPE,
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	757 preexec_fn=lambda: os.putenv("FRUIT", "apple"))
65955 6e1ae323fde6 Add cleanups to stdout/stderr pipes to remove ResourceWarnings. Brian Curtin <brian.curtin@gmail.com> parents: 65799 diff changeset	758 self.addCleanup(p.stdout.close)
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	759 self.assertEqual(p.stdout.read(), b"apple")
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	760
59605 768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	761 def test_preexec_exception(self):
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	762 def raise_it():
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	763 raise ValueError("What if two swallows carried a coconut?")
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	764 try:
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	765 p = subprocess.Popen([sys.executable, "-c", ""],
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	766 preexec_fn=raise_it)
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	767 except RuntimeError as e:
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	768 self.assertTrue(
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	769 subprocess._posixsubprocess,
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	770 "Expected a ValueError from the preexec_fn")
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	771 except ValueError as e:
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	772 self.assertIn("coconut", e.args[0])
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	773 else:
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	774 self.fail("Exception raised by preexec_fn did not make it "
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	775 "to the parent process.")
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	776
59723 00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	777 @unittest.skipUnless(gc, "Requires a gc module.")
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	778 def test_preexec_gc_module_failure(self):
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	779 # This tests the code that disables garbage collection if the child
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	780 # process will execute any Python.
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	781 def raise_runtime_error():
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	782 raise RuntimeError("this shouldn't escape")
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	783 enabled = gc.isenabled()
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	784 orig_gc_disable = gc.disable
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	785 orig_gc_isenabled = gc.isenabled
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	786 try:
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	787 gc.disable()
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	788 self.assertFalse(gc.isenabled())
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	789 subprocess.call([sys.executable, '-c', ''],
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	790 preexec_fn=lambda: None)
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	791 self.assertFalse(gc.isenabled(),
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	792 "Popen enabled gc when it shouldn't.")
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	793
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	794 gc.enable()
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	795 self.assertTrue(gc.isenabled())
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	796 subprocess.call([sys.executable, '-c', ''],
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	797 preexec_fn=lambda: None)
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	798 self.assertTrue(gc.isenabled(), "Popen left gc disabled.")
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	799
59723 00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	800 gc.disable = raise_runtime_error
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	801 self.assertRaises(RuntimeError, subprocess.Popen,
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	802 [sys.executable, '-c', ''],
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	803 preexec_fn=lambda: None)
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	804
59723 00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	805 del gc.isenabled # force an AttributeError
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	806 self.assertRaises(AttributeError, subprocess.Popen,
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	807 [sys.executable, '-c', ''],
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	808 preexec_fn=lambda: None)
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	809 finally:
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	810 gc.disable = orig_gc_disable
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	811 gc.isenabled = orig_gc_isenabled
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	812 if not enabled:
00daa8b736cc * Fix a refleak when a preexec_fn was supplied (preexec_fn_args_tuple was not Gregory P. Smith <greg@mad-scientist.com> parents: 59605 diff changeset	813 gc.disable()
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	814
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	815 def test_args_string(self):
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	816 # args is a string
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	817 fd, fname = mkstemp()
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	818 # reopen in text mode
65415 340359d80ca0 test_subprocess: use surrogateescape error handler to write shell scripts Victor Stinner <victor.stinner@haypocalc.com> parents: 65302 diff changeset	819 with open(fd, "w", errors="surrogateescape") as fobj:
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	820 fobj.write("#!/bin/sh\n")
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	821 fobj.write("exec '%s' -c 'import sys; sys.exit(47)'\n" %
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	822 sys.executable)
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	823 os.chmod(fname, 0o700)
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	824 p = subprocess.Popen(fname)
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	825 p.wait()
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	826 os.remove(fname)
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	827 self.assertEqual(p.returncode, 47)
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	828
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	829 def test_invalid_args(self):
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	830 # invalid arguments should raise ValueError
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	831 self.assertRaises(ValueError, subprocess.call,
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	832 [sys.executable, "-c",
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	833 "import sys; sys.exit(47)"],
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	834 startupinfo=47)
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	835 self.assertRaises(ValueError, subprocess.call,
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	836 [sys.executable, "-c",
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	837 "import sys; sys.exit(47)"],
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	838 creationflags=47)
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	839
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	840 def test_shell_sequence(self):
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	841 # Run command through the shell (sequence)
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	842 newenv = os.environ.copy()
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	843 newenv["FRUIT"] = "apple"
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	844 p = subprocess.Popen(["echo $FRUIT"], shell=1,
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	845 stdout=subprocess.PIPE,
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	846 env=newenv)
65955 6e1ae323fde6 Add cleanups to stdout/stderr pipes to remove ResourceWarnings. Brian Curtin <brian.curtin@gmail.com> parents: 65799 diff changeset	847 self.addCleanup(p.stdout.close)
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	848 self.assertEqual(p.stdout.read().strip(b" \t\r\n\f"), b"apple")
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	849
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	850 def test_shell_string(self):
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	851 # Run command through the shell (string)
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	852 newenv = os.environ.copy()
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	853 newenv["FRUIT"] = "apple"
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	854 p = subprocess.Popen("echo $FRUIT", shell=1,
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	855 stdout=subprocess.PIPE,
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	856 env=newenv)
65955 6e1ae323fde6 Add cleanups to stdout/stderr pipes to remove ResourceWarnings. Brian Curtin <brian.curtin@gmail.com> parents: 65799 diff changeset	857 self.addCleanup(p.stdout.close)
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	858 self.assertEqual(p.stdout.read().strip(b" \t\r\n\f"), b"apple")
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	859
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	860 def test_call_string(self):
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	861 # call() function with string argument on UNIX
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	862 fd, fname = mkstemp()
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	863 # reopen in text mode
65415 340359d80ca0 test_subprocess: use surrogateescape error handler to write shell scripts Victor Stinner <victor.stinner@haypocalc.com> parents: 65302 diff changeset	864 with open(fd, "w", errors="surrogateescape") as fobj:
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	865 fobj.write("#!/bin/sh\n")
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	866 fobj.write("exec '%s' -c 'import sys; sys.exit(47)'\n" %
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	867 sys.executable)
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	868 os.chmod(fname, 0o700)
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	869 rc = subprocess.call(fname)
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	870 os.remove(fname)
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	871 self.assertEqual(rc, 47)
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	872
63094 aa0b56dd14af Issue #9265: Incorrect name passed as arg[0] when shell=True Stefan Krah <stefan@bytereef.org> parents: 62402 diff changeset	873 def test_specific_shell(self):
aa0b56dd14af Issue #9265: Incorrect name passed as arg[0] when shell=True Stefan Krah <stefan@bytereef.org> parents: 62402 diff changeset	874 # Issue #9265: Incorrect name passed as arg[0].
aa0b56dd14af Issue #9265: Incorrect name passed as arg[0] when shell=True Stefan Krah <stefan@bytereef.org> parents: 62402 diff changeset	875 shells = []
aa0b56dd14af Issue #9265: Incorrect name passed as arg[0] when shell=True Stefan Krah <stefan@bytereef.org> parents: 62402 diff changeset	876 for prefix in ['/bin', '/usr/bin/', '/usr/local/bin']:
aa0b56dd14af Issue #9265: Incorrect name passed as arg[0] when shell=True Stefan Krah <stefan@bytereef.org> parents: 62402 diff changeset	877 for name in ['bash', 'ksh']:
aa0b56dd14af Issue #9265: Incorrect name passed as arg[0] when shell=True Stefan Krah <stefan@bytereef.org> parents: 62402 diff changeset	878 sh = os.path.join(prefix, name)
aa0b56dd14af Issue #9265: Incorrect name passed as arg[0] when shell=True Stefan Krah <stefan@bytereef.org> parents: 62402 diff changeset	879 if os.path.isfile(sh):
aa0b56dd14af Issue #9265: Incorrect name passed as arg[0] when shell=True Stefan Krah <stefan@bytereef.org> parents: 62402 diff changeset	880 shells.append(sh)
aa0b56dd14af Issue #9265: Incorrect name passed as arg[0] when shell=True Stefan Krah <stefan@bytereef.org> parents: 62402 diff changeset	881 if not shells: # Will probably work for any shell but csh.
aa0b56dd14af Issue #9265: Incorrect name passed as arg[0] when shell=True Stefan Krah <stefan@bytereef.org> parents: 62402 diff changeset	882 self.skipTest("bash or ksh required for this test")
aa0b56dd14af Issue #9265: Incorrect name passed as arg[0] when shell=True Stefan Krah <stefan@bytereef.org> parents: 62402 diff changeset	883 sh = '/bin/sh'
aa0b56dd14af Issue #9265: Incorrect name passed as arg[0] when shell=True Stefan Krah <stefan@bytereef.org> parents: 62402 diff changeset	884 if os.path.isfile(sh) and not os.path.islink(sh):
aa0b56dd14af Issue #9265: Incorrect name passed as arg[0] when shell=True Stefan Krah <stefan@bytereef.org> parents: 62402 diff changeset	885 # Test will fail if /bin/sh is a symlink to csh.
aa0b56dd14af Issue #9265: Incorrect name passed as arg[0] when shell=True Stefan Krah <stefan@bytereef.org> parents: 62402 diff changeset	886 shells.append(sh)
aa0b56dd14af Issue #9265: Incorrect name passed as arg[0] when shell=True Stefan Krah <stefan@bytereef.org> parents: 62402 diff changeset	887 for sh in shells:
aa0b56dd14af Issue #9265: Incorrect name passed as arg[0] when shell=True Stefan Krah <stefan@bytereef.org> parents: 62402 diff changeset	888 p = subprocess.Popen("echo $0", executable=sh, shell=True,
aa0b56dd14af Issue #9265: Incorrect name passed as arg[0] when shell=True Stefan Krah <stefan@bytereef.org> parents: 62402 diff changeset	889 stdout=subprocess.PIPE)
65955 6e1ae323fde6 Add cleanups to stdout/stderr pipes to remove ResourceWarnings. Brian Curtin <brian.curtin@gmail.com> parents: 65799 diff changeset	890 self.addCleanup(p.stdout.close)
63094 aa0b56dd14af Issue #9265: Incorrect name passed as arg[0] when shell=True Stefan Krah <stefan@bytereef.org> parents: 62402 diff changeset	891 self.assertEqual(p.stdout.read().strip(), bytes(sh, 'ascii'))
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	892
59471 ac80884ddcc3 Merged revisions 78736,78759,78761,78767,78788-78789 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59398 diff changeset	893 def _kill_process(self, method, *args):
59376 d9e6f318568f Merged revisions 78701 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59353 diff changeset	894 # Do not inherit file handles from the parent.
d9e6f318568f Merged revisions 78701 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59353 diff changeset	895 # It should fix failures on some platforms.
64843 76fd68bba1a8 Try a more robust implementation of _kill_process Antoine Pitrou <solipsis@pitrou.net> parents: 64842 diff changeset	896 p = subprocess.Popen([sys.executable, "-c", """if 1:
76fd68bba1a8 Try a more robust implementation of _kill_process Antoine Pitrou <solipsis@pitrou.net> parents: 64842 diff changeset	897 import sys, time
76fd68bba1a8 Try a more robust implementation of _kill_process Antoine Pitrou <solipsis@pitrou.net> parents: 64842 diff changeset	898 sys.stdout.write('x\\n')
76fd68bba1a8 Try a more robust implementation of _kill_process Antoine Pitrou <solipsis@pitrou.net> parents: 64842 diff changeset	899 sys.stdout.flush()
76fd68bba1a8 Try a more robust implementation of _kill_process Antoine Pitrou <solipsis@pitrou.net> parents: 64842 diff changeset	900 time.sleep(30)
76fd68bba1a8 Try a more robust implementation of _kill_process Antoine Pitrou <solipsis@pitrou.net> parents: 64842 diff changeset	901 """],
76fd68bba1a8 Try a more robust implementation of _kill_process Antoine Pitrou <solipsis@pitrou.net> parents: 64842 diff changeset	902 close_fds=True,
76fd68bba1a8 Try a more robust implementation of _kill_process Antoine Pitrou <solipsis@pitrou.net> parents: 64842 diff changeset	903 stdin=subprocess.PIPE,
76fd68bba1a8 Try a more robust implementation of _kill_process Antoine Pitrou <solipsis@pitrou.net> parents: 64842 diff changeset	904 stdout=subprocess.PIPE,
76fd68bba1a8 Try a more robust implementation of _kill_process Antoine Pitrou <solipsis@pitrou.net> parents: 64842 diff changeset	905 stderr=subprocess.PIPE)
76fd68bba1a8 Try a more robust implementation of _kill_process Antoine Pitrou <solipsis@pitrou.net> parents: 64842 diff changeset	906 # Wait for the interpreter to be completely initialized before
76fd68bba1a8 Try a more robust implementation of _kill_process Antoine Pitrou <solipsis@pitrou.net> parents: 64842 diff changeset	907 # sending any signal.
76fd68bba1a8 Try a more robust implementation of _kill_process Antoine Pitrou <solipsis@pitrou.net> parents: 64842 diff changeset	908 p.stdout.read(1)
76fd68bba1a8 Try a more robust implementation of _kill_process Antoine Pitrou <solipsis@pitrou.net> parents: 64842 diff changeset	909 getattr(p, method)(*args)
59471 ac80884ddcc3 Merged revisions 78736,78759,78761,78767,78788-78789 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59398 diff changeset	910 return p
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	911
59471 ac80884ddcc3 Merged revisions 78736,78759,78761,78767,78788-78789 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59398 diff changeset	912 def test_send_signal(self):
ac80884ddcc3 Merged revisions 78736,78759,78761,78767,78788-78789 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59398 diff changeset	913 p = self._kill_process('send_signal', signal.SIGINT)
60057 bb2ca1b63321 Merged revisions 79344,79346,79350 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59723 diff changeset	914 _, stderr = p.communicate()
bb2ca1b63321 Merged revisions 79344,79346,79350 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59723 diff changeset	915 self.assertIn(b'KeyboardInterrupt', stderr)
59340 04955e7eb597 Merged revisions 78661-78662 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59216 diff changeset	916 self.assertNotEqual(p.wait(), 0)
46505 63965327b3ef Merged revisions 62386-62387,62389-62393,62396,62400-62402,62407,62409-62410,62412-62414,62418-62419 via svnmerge from Christian Heimes <christian@cheimes.de> parents: 44810 diff changeset	917
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	918 def test_kill(self):
59471 ac80884ddcc3 Merged revisions 78736,78759,78761,78767,78788-78789 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59398 diff changeset	919 p = self._kill_process('kill')
60057 bb2ca1b63321 Merged revisions 79344,79346,79350 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59723 diff changeset	920 _, stderr = p.communicate()
bb2ca1b63321 Merged revisions 79344,79346,79350 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59723 diff changeset	921 self.assertStderrEqual(stderr, b'')
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	922 self.assertEqual(p.wait(), -signal.SIGKILL)
46505 63965327b3ef Merged revisions 62386-62387,62389-62393,62396,62400-62402,62407,62409-62410,62412-62414,62418-62419 via svnmerge from Christian Heimes <christian@cheimes.de> parents: 44810 diff changeset	923
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	924 def test_terminate(self):
59471 ac80884ddcc3 Merged revisions 78736,78759,78761,78767,78788-78789 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59398 diff changeset	925 p = self._kill_process('terminate')
60057 bb2ca1b63321 Merged revisions 79344,79346,79350 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59723 diff changeset	926 _, stderr = p.communicate()
bb2ca1b63321 Merged revisions 79344,79346,79350 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59723 diff changeset	927 self.assertStderrEqual(stderr, b'')
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	928 self.assertEqual(p.wait(), -signal.SIGTERM)
63095 ad66eff9800b Merged revisions 82971 via svnmerge from Stefan Krah <stefan@bytereef.org> parents: 62404 diff changeset	929
67310 a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	930 def check_close_std_fds(self, fds):
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	931 # Issue #9905: test that subprocess pipes still work properly with
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	932 # some standard fds closed
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	933 stdin = 0
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	934 newfds = []
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	935 for a in fds:
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	936 b = os.dup(a)
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	937 newfds.append(b)
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	938 if a == 0:
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	939 stdin = b
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	940 try:
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	941 for fd in fds:
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	942 os.close(fd)
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	943 out, err = subprocess.Popen([sys.executable, "-c",
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	944 'import sys;'
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	945 'sys.stdout.write("apple");'
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	946 'sys.stdout.flush();'
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	947 'sys.stderr.write("orange")'],
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	948 stdin=stdin,
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	949 stdout=subprocess.PIPE,
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	950 stderr=subprocess.PIPE).communicate()
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	951 err = support.strip_python_stderr(err)
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	952 self.assertEqual((out, err), (b'apple', b'orange'))
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	953 finally:
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	954 for b, a in zip(newfds, fds):
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	955 os.dup2(b, a)
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	956 for b in newfds:
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	957 os.close(b)
46505 63965327b3ef Merged revisions 62386-62387,62389-62393,62396,62400-62402,62407,62409-62410,62412-62414,62418-62419 via svnmerge from Christian Heimes <christian@cheimes.de> parents: 44810 diff changeset	958
67310 a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	959 def test_close_fd_0(self):
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	960 self.check_close_std_fds([0])
46505 63965327b3ef Merged revisions 62386-62387,62389-62393,62396,62400-62402,62407,62409-62410,62412-62414,62418-62419 via svnmerge from Christian Heimes <christian@cheimes.de> parents: 44810 diff changeset	961
67310 a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	962 def test_close_fd_1(self):
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	963 self.check_close_std_fds([1])
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	964
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	965 def test_close_fd_2(self):
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	966 self.check_close_std_fds([2])
46505 63965327b3ef Merged revisions 62386-62387,62389-62393,62396,62400-62402,62407,62409-62410,62412-62414,62418-62419 via svnmerge from Christian Heimes <christian@cheimes.de> parents: 44810 diff changeset	967
67310 a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	968 def test_close_fds_0_1(self):
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	969 self.check_close_std_fds([0, 1])
46505 63965327b3ef Merged revisions 62386-62387,62389-62393,62396,62400-62402,62407,62409-62410,62412-62414,62418-62419 via svnmerge from Christian Heimes <christian@cheimes.de> parents: 44810 diff changeset	970
67310 a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	971 def test_close_fds_0_2(self):
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	972 self.check_close_std_fds([0, 2])
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	973
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	974 def test_close_fds_1_2(self):
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	975 self.check_close_std_fds([1, 2])
33731 53c468d0955b Whitespace normalization. Tim Peters <tim.peters@gmail.com> parents: 33723 diff changeset	976
67310 a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	977 def test_close_fds_0_1_2(self):
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	978 # Issue #10806: test that subprocess pipes still work properly with
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	979 # all standard fds closed.
a66a6e42538f Issue #10806, issue #9905: Fix subprocess pipes when some of the standard Antoine Pitrou <solipsis@pitrou.net> parents: 67303 diff changeset	980 self.check_close_std_fds([0, 1, 2])
61014 6a90dc967fed test_subprocess: test_undecodable_env() is specific to POSIX system Victor Stinner <victor.stinner@haypocalc.com> parents: 60975 diff changeset	981
67327 a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	982 def test_remapping_std_fds(self):
a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	983 # open up some temporary files
a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	984 temps = [mkstemp() for i in range(3)]
a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	985 try:
a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	986 temp_fds = [fd for fd, fname in temps]
61014 6a90dc967fed test_subprocess: test_undecodable_env() is specific to POSIX system Victor Stinner <victor.stinner@haypocalc.com> parents: 60975 diff changeset	987
67327 a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	988 # unlink the files -- we won't need to reopen them
a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	989 for fd, fname in temps:
a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	990 os.unlink(fname)
61014 6a90dc967fed test_subprocess: test_undecodable_env() is specific to POSIX system Victor Stinner <victor.stinner@haypocalc.com> parents: 60975 diff changeset	991
67327 a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	992 # write some data to what will become stdin, and rewind
a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	993 os.write(temp_fds[1], b"STDIN")
a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	994 os.lseek(temp_fds[1], 0, 0)
66895 b066bd0b64e7 Merged revisions 87233 via svnmerge from Gregory P. Smith <greg@mad-scientist.com> parents: 66340 diff changeset	995
67327 a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	996 # move the standard file descriptors out of the way
a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	997 saved_fds = [os.dup(fd) for fd in range(3)]
67311 e9a51d463579 Merged revisions 87695 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 66897 diff changeset	998 try:
67327 a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	999 # duplicate the file objects over the standard fd's
a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	1000 for fd, temp_fd in enumerate(temp_fds):
a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	1001 os.dup2(temp_fd, fd)
46505 63965327b3ef Merged revisions 62386-62387,62389-62393,62396,62400-62402,62407,62409-62410,62412-62414,62418-62419 via svnmerge from Christian Heimes <christian@cheimes.de> parents: 44810 diff changeset	1002
67327 a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	1003 # now use those files in the "wrong" order, so that subprocess
a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	1004 # has to rearrange them in the child
a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	1005 p = subprocess.Popen([sys.executable, "-c",
a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	1006 'import sys; got = sys.stdin.read();'
a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	1007 'sys.stdout.write("got %s"%got); sys.stderr.write("err")'],
a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	1008 stdin=temp_fds[1],
a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	1009 stdout=temp_fds[2],
a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	1010 stderr=temp_fds[0])
a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	1011 p.wait()
67311 e9a51d463579 Merged revisions 87695 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 66897 diff changeset	1012 finally:
67327 a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	1013 # restore the original fd's underneath sys.stdin, etc.
a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	1014 for std, saved in enumerate(saved_fds):
a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	1015 os.dup2(saved, std)
a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	1016 os.close(saved)
33731 53c468d0955b Whitespace normalization. Tim Peters <tim.peters@gmail.com> parents: 33723 diff changeset	1017
67327 a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	1018 for fd in temp_fds:
a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	1019 os.lseek(fd, 0, 0)
a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	1020
a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	1021 out = os.read(temp_fds[2], 1024)
a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	1022 err = support.strip_python_stderr(os.read(temp_fds[0], 1024))
a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	1023 self.assertEqual(out, b"got STDIN")
a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	1024 self.assertEqual(err, b"err")
a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	1025
a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	1026 finally:
a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	1027 for fd in temp_fds:
a7c499266a9f Add a subprocess test of remapping standard file descriptors (issue #1187). Antoine Pitrou <solipsis@pitrou.net> parents: 67310 diff changeset	1028 os.close(fd)
61014 6a90dc967fed test_subprocess: test_undecodable_env() is specific to POSIX system Victor Stinner <victor.stinner@haypocalc.com> parents: 60975 diff changeset	1029
60902 dc8c8d54504e Issue #8391: os.execvpe() and os.getenv() supports unicode with surrogates and Victor Stinner <victor.stinner@haypocalc.com> parents: 60896 diff changeset	1030 def test_surrogates_error_message(self):
60896 8428eaa2d3ad Issue #8467: Pure Python implementation of subprocess encodes the error message Victor Stinner <victor.stinner@haypocalc.com> parents: 60057 diff changeset	1031 def prepare():
8428eaa2d3ad Issue #8467: Pure Python implementation of subprocess encodes the error message Victor Stinner <victor.stinner@haypocalc.com> parents: 60057 diff changeset	1032 raise ValueError("surrogate:\uDCff")
67311 e9a51d463579 Merged revisions 87695 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 66897 diff changeset	1033
60896 8428eaa2d3ad Issue #8467: Pure Python implementation of subprocess encodes the error message Victor Stinner <victor.stinner@haypocalc.com> parents: 60057 diff changeset	1034 try:
8428eaa2d3ad Issue #8467: Pure Python implementation of subprocess encodes the error message Victor Stinner <victor.stinner@haypocalc.com> parents: 60057 diff changeset	1035 subprocess.call(
8428eaa2d3ad Issue #8467: Pure Python implementation of subprocess encodes the error message Victor Stinner <victor.stinner@haypocalc.com> parents: 60057 diff changeset	1036 [sys.executable, "-c", "pass"],
8428eaa2d3ad Issue #8467: Pure Python implementation of subprocess encodes the error message Victor Stinner <victor.stinner@haypocalc.com> parents: 60057 diff changeset	1037 preexec_fn=prepare)
8428eaa2d3ad Issue #8467: Pure Python implementation of subprocess encodes the error message Victor Stinner <victor.stinner@haypocalc.com> parents: 60057 diff changeset	1038 except ValueError as err:
8428eaa2d3ad Issue #8467: Pure Python implementation of subprocess encodes the error message Victor Stinner <victor.stinner@haypocalc.com> parents: 60057 diff changeset	1039 # Pure Python implementations keeps the message
8428eaa2d3ad Issue #8467: Pure Python implementation of subprocess encodes the error message Victor Stinner <victor.stinner@haypocalc.com> parents: 60057 diff changeset	1040 self.assertIsNone(subprocess._posixsubprocess)
8428eaa2d3ad Issue #8467: Pure Python implementation of subprocess encodes the error message Victor Stinner <victor.stinner@haypocalc.com> parents: 60057 diff changeset	1041 self.assertEqual(str(err), "surrogate:\uDCff")
8428eaa2d3ad Issue #8467: Pure Python implementation of subprocess encodes the error message Victor Stinner <victor.stinner@haypocalc.com> parents: 60057 diff changeset	1042 except RuntimeError as err:
8428eaa2d3ad Issue #8467: Pure Python implementation of subprocess encodes the error message Victor Stinner <victor.stinner@haypocalc.com> parents: 60057 diff changeset	1043 # _posixsubprocess uses a default message
8428eaa2d3ad Issue #8467: Pure Python implementation of subprocess encodes the error message Victor Stinner <victor.stinner@haypocalc.com> parents: 60057 diff changeset	1044 self.assertIsNotNone(subprocess._posixsubprocess)
8428eaa2d3ad Issue #8467: Pure Python implementation of subprocess encodes the error message Victor Stinner <victor.stinner@haypocalc.com> parents: 60057 diff changeset	1045 self.assertEqual(str(err), "Exception occurred in preexec_fn.")
8428eaa2d3ad Issue #8467: Pure Python implementation of subprocess encodes the error message Victor Stinner <victor.stinner@haypocalc.com> parents: 60057 diff changeset	1046 else:
8428eaa2d3ad Issue #8467: Pure Python implementation of subprocess encodes the error message Victor Stinner <victor.stinner@haypocalc.com> parents: 60057 diff changeset	1047 self.fail("Expected ValueError or RuntimeError")
67311 e9a51d463579 Merged revisions 87695 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 66897 diff changeset	1048
60902 dc8c8d54504e Issue #8391: os.execvpe() and os.getenv() supports unicode with surrogates and Victor Stinner <victor.stinner@haypocalc.com> parents: 60896 diff changeset	1049 def test_undecodable_env(self):
dc8c8d54504e Issue #8391: os.execvpe() and os.getenv() supports unicode with surrogates and Victor Stinner <victor.stinner@haypocalc.com> parents: 60896 diff changeset	1050 for key, value in (('test', 'abc\uDCFF'), ('test\uDCFF', '42')):
dc8c8d54504e Issue #8391: os.execvpe() and os.getenv() supports unicode with surrogates and Victor Stinner <victor.stinner@haypocalc.com> parents: 60896 diff changeset	1051 # test str with surrogates
64839 4374b1eaead1 Try to fix test_subprocess on "x86 debian parallel 3.x" buildbot Antoine Pitrou <solipsis@pitrou.net> parents: 64813 diff changeset	1052 script = "import os; print(ascii(os.getenv(%s)))" % repr(key)
60905 e79a6c802934 Fix test_undecodable_env of test_subproces for non-ASCII directory Victor Stinner <victor.stinner@haypocalc.com> parents: 60902 diff changeset	1053 env = os.environ.copy()
e79a6c802934 Fix test_undecodable_env of test_subproces for non-ASCII directory Victor Stinner <victor.stinner@haypocalc.com> parents: 60902 diff changeset	1054 env[key] = value
65302 e2c98c60dd64 test_subprocess doesn't need to C locale to test os.environb Victor Stinner <victor.stinner@haypocalc.com> parents: 65301 diff changeset	1055 # Use C locale to get ascii for the locale encoding to force
e2c98c60dd64 test_subprocess doesn't need to C locale to test os.environb Victor Stinner <victor.stinner@haypocalc.com> parents: 65301 diff changeset	1056 # surrogate-escaping of \xFF in the child process; otherwise it can
e2c98c60dd64 test_subprocess doesn't need to C locale to test os.environb Victor Stinner <victor.stinner@haypocalc.com> parents: 65301 diff changeset	1057 # be decoded as-is if the default locale is latin-1.
65301 2853cad9cbc0 test_subprocess: use C locale to get ascii locale encoding Victor Stinner <victor.stinner@haypocalc.com> parents: 65265 diff changeset	1058 env['LC_ALL'] = 'C'
60902 dc8c8d54504e Issue #8391: os.execvpe() and os.getenv() supports unicode with surrogates and Victor Stinner <victor.stinner@haypocalc.com> parents: 60896 diff changeset	1059 stdout = subprocess.check_output(
dc8c8d54504e Issue #8391: os.execvpe() and os.getenv() supports unicode with surrogates and Victor Stinner <victor.stinner@haypocalc.com> parents: 60896 diff changeset	1060 [sys.executable, "-c", script],
60905 e79a6c802934 Fix test_undecodable_env of test_subproces for non-ASCII directory Victor Stinner <victor.stinner@haypocalc.com> parents: 60902 diff changeset	1061 env=env)
60902 dc8c8d54504e Issue #8391: os.execvpe() and os.getenv() supports unicode with surrogates and Victor Stinner <victor.stinner@haypocalc.com> parents: 60896 diff changeset	1062 stdout = stdout.rstrip(b'\n\r')
66310 9fe76c26ea9f #9424: Replace deprecated assert* methods in the Python test suite. Ezio Melotti <ezio.melotti@gmail.com> parents: 66307 diff changeset	1063 self.assertEqual(stdout.decode('ascii'), ascii(value))
67311 e9a51d463579 Merged revisions 87695 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 66897 diff changeset	1064
60902 dc8c8d54504e Issue #8391: os.execvpe() and os.getenv() supports unicode with surrogates and Victor Stinner <victor.stinner@haypocalc.com> parents: 60896 diff changeset	1065 # test bytes
dc8c8d54504e Issue #8391: os.execvpe() and os.getenv() supports unicode with surrogates and Victor Stinner <victor.stinner@haypocalc.com> parents: 60896 diff changeset	1066 key = key.encode("ascii", "surrogateescape")
dc8c8d54504e Issue #8391: os.execvpe() and os.getenv() supports unicode with surrogates and Victor Stinner <victor.stinner@haypocalc.com> parents: 60896 diff changeset	1067 value = value.encode("ascii", "surrogateescape")
64839 4374b1eaead1 Try to fix test_subprocess on "x86 debian parallel 3.x" buildbot Antoine Pitrou <solipsis@pitrou.net> parents: 64813 diff changeset	1068 script = "import os; print(ascii(os.getenvb(%s)))" % repr(key)
60905 e79a6c802934 Fix test_undecodable_env of test_subproces for non-ASCII directory Victor Stinner <victor.stinner@haypocalc.com> parents: 60902 diff changeset	1069 env = os.environ.copy()
e79a6c802934 Fix test_undecodable_env of test_subproces for non-ASCII directory Victor Stinner <victor.stinner@haypocalc.com> parents: 60902 diff changeset	1070 env[key] = value
60902 dc8c8d54504e Issue #8391: os.execvpe() and os.getenv() supports unicode with surrogates and Victor Stinner <victor.stinner@haypocalc.com> parents: 60896 diff changeset	1071 stdout = subprocess.check_output(
dc8c8d54504e Issue #8391: os.execvpe() and os.getenv() supports unicode with surrogates and Victor Stinner <victor.stinner@haypocalc.com> parents: 60896 diff changeset	1072 [sys.executable, "-c", script],
60905 e79a6c802934 Fix test_undecodable_env of test_subproces for non-ASCII directory Victor Stinner <victor.stinner@haypocalc.com> parents: 60902 diff changeset	1073 env=env)
60902 dc8c8d54504e Issue #8391: os.execvpe() and os.getenv() supports unicode with surrogates and Victor Stinner <victor.stinner@haypocalc.com> parents: 60896 diff changeset	1074 stdout = stdout.rstrip(b'\n\r')
66310 9fe76c26ea9f #9424: Replace deprecated assert* methods in the Python test suite. Ezio Melotti <ezio.melotti@gmail.com> parents: 66307 diff changeset	1075 self.assertEqual(stdout.decode('ascii'), ascii(value))
67311 e9a51d463579 Merged revisions 87695 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 66897 diff changeset	1076
61693 cf74ec3e1f82 Issue #8513: os.get_exec_path() supports b'PATH' key and bytes value. Victor Stinner <victor.stinner@haypocalc.com> parents: 61332 diff changeset	1077 def test_bytes_program(self):
cf74ec3e1f82 Issue #8513: os.get_exec_path() supports b'PATH' key and bytes value. Victor Stinner <victor.stinner@haypocalc.com> parents: 61332 diff changeset	1078 abs_program = os.fsencode(sys.executable)
cf74ec3e1f82 Issue #8513: os.get_exec_path() supports b'PATH' key and bytes value. Victor Stinner <victor.stinner@haypocalc.com> parents: 61332 diff changeset	1079 path, program = os.path.split(sys.executable)
cf74ec3e1f82 Issue #8513: os.get_exec_path() supports b'PATH' key and bytes value. Victor Stinner <victor.stinner@haypocalc.com> parents: 61332 diff changeset	1080 program = os.fsencode(program)
cf74ec3e1f82 Issue #8513: os.get_exec_path() supports b'PATH' key and bytes value. Victor Stinner <victor.stinner@haypocalc.com> parents: 61332 diff changeset	1081
cf74ec3e1f82 Issue #8513: os.get_exec_path() supports b'PATH' key and bytes value. Victor Stinner <victor.stinner@haypocalc.com> parents: 61332 diff changeset	1082 # absolute bytes path
cf74ec3e1f82 Issue #8513: os.get_exec_path() supports b'PATH' key and bytes value. Victor Stinner <victor.stinner@haypocalc.com> parents: 61332 diff changeset	1083 exitcode = subprocess.call([abs_program, "-c", "pass"])
66310 9fe76c26ea9f #9424: Replace deprecated assert* methods in the Python test suite. Ezio Melotti <ezio.melotti@gmail.com> parents: 66307 diff changeset	1084 self.assertEqual(exitcode, 0)
61693 cf74ec3e1f82 Issue #8513: os.get_exec_path() supports b'PATH' key and bytes value. Victor Stinner <victor.stinner@haypocalc.com> parents: 61332 diff changeset	1085
cf74ec3e1f82 Issue #8513: os.get_exec_path() supports b'PATH' key and bytes value. Victor Stinner <victor.stinner@haypocalc.com> parents: 61332 diff changeset	1086 # bytes program, unicode PATH
cf74ec3e1f82 Issue #8513: os.get_exec_path() supports b'PATH' key and bytes value. Victor Stinner <victor.stinner@haypocalc.com> parents: 61332 diff changeset	1087 env = os.environ.copy()
cf74ec3e1f82 Issue #8513: os.get_exec_path() supports b'PATH' key and bytes value. Victor Stinner <victor.stinner@haypocalc.com> parents: 61332 diff changeset	1088 env["PATH"] = path
cf74ec3e1f82 Issue #8513: os.get_exec_path() supports b'PATH' key and bytes value. Victor Stinner <victor.stinner@haypocalc.com> parents: 61332 diff changeset	1089 exitcode = subprocess.call([program, "-c", "pass"], env=env)
66310 9fe76c26ea9f #9424: Replace deprecated assert* methods in the Python test suite. Ezio Melotti <ezio.melotti@gmail.com> parents: 66307 diff changeset	1090 self.assertEqual(exitcode, 0)
67311 e9a51d463579 Merged revisions 87695 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 66897 diff changeset	1091
61693 cf74ec3e1f82 Issue #8513: os.get_exec_path() supports b'PATH' key and bytes value. Victor Stinner <victor.stinner@haypocalc.com> parents: 61332 diff changeset	1092 # bytes program, bytes PATH
cf74ec3e1f82 Issue #8513: os.get_exec_path() supports b'PATH' key and bytes value. Victor Stinner <victor.stinner@haypocalc.com> parents: 61332 diff changeset	1093 envb = os.environb.copy()
cf74ec3e1f82 Issue #8513: os.get_exec_path() supports b'PATH' key and bytes value. Victor Stinner <victor.stinner@haypocalc.com> parents: 61332 diff changeset	1094 envb[b"PATH"] = os.fsencode(path)
cf74ec3e1f82 Issue #8513: os.get_exec_path() supports b'PATH' key and bytes value. Victor Stinner <victor.stinner@haypocalc.com> parents: 61332 diff changeset	1095 exitcode = subprocess.call([program, "-c", "pass"], env=envb)
66310 9fe76c26ea9f #9424: Replace deprecated assert* methods in the Python test suite. Ezio Melotti <ezio.melotti@gmail.com> parents: 66307 diff changeset	1096 self.assertEqual(exitcode, 0)
67311 e9a51d463579 Merged revisions 87695 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 66897 diff changeset	1097
66868 de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1098 def test_pipe_cloexec(self):
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1099 sleeper = support.findfile("input_reader.py", subdir="subprocessdata")
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1100 fd_status = support.findfile("fd_status.py", subdir="subprocessdata")
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1101
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1102 p1 = subprocess.Popen([sys.executable, sleeper],
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1103 stdin=subprocess.PIPE, stdout=subprocess.PIPE,
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1104 stderr=subprocess.PIPE, close_fds=False)
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1105
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1106 self.addCleanup(p1.communicate, b'')
67311 e9a51d463579 Merged revisions 87695 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 66897 diff changeset	1107
66868 de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1108 p2 = subprocess.Popen([sys.executable, fd_status],
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1109 stdout=subprocess.PIPE, close_fds=False)
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1110
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1111 output, error = p2.communicate()
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1112 result_fds = set(map(int, output.split(b',')))
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1113 unwanted_fds = set([p1.stdin.fileno(), p1.stdout.fileno(),
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1114 p1.stderr.fileno()])
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1115
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1116 self.assertFalse(result_fds & unwanted_fds,
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1117 "Expected no fds from %r to be open in child, "
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1118 "found %r" %
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1119 (unwanted_fds, result_fds & unwanted_fds))
67311 e9a51d463579 Merged revisions 87695 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 66897 diff changeset	1120
66868 de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1121 def test_pipe_cloexec_real_tools(self):
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1122 qcat = support.findfile("qcat.py", subdir="subprocessdata")
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1123 qgrep = support.findfile("qgrep.py", subdir="subprocessdata")
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1124
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1125 subdata = b'zxcvbn'
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1126 data = subdata * 4 + b'\n'
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1127
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1128 p1 = subprocess.Popen([sys.executable, qcat],
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1129 stdin=subprocess.PIPE, stdout=subprocess.PIPE,
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1130 close_fds=False)
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1131
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1132 p2 = subprocess.Popen([sys.executable, qgrep, subdata],
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1133 stdin=p1.stdout, stdout=subprocess.PIPE,
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1134 close_fds=False)
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1135
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1136 self.addCleanup(p1.wait)
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1137 self.addCleanup(p2.wait)
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1138 self.addCleanup(p1.terminate)
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1139 self.addCleanup(p2.terminate)
67311 e9a51d463579 Merged revisions 87695 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 66897 diff changeset	1140
66868 de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1141 p1.stdin.write(data)
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1142 p1.stdin.close()
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1143
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1144 readfiles, ignored1, ignored2 = select.select([p2.stdout], [], [], 10)
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1145
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1146 self.assertTrue(readfiles, "The child hung")
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1147 self.assertEqual(p2.stdout.read(), data)
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1148
67303 77483f3e491a test_subprocess: close pipes at the end of test_pipe_cloexec_real_tools() Victor Stinner <victor.stinner@haypocalc.com> parents: 67130 diff changeset	1149 p1.stdout.close()
77483f3e491a test_subprocess: close pipes at the end of test_pipe_cloexec_real_tools() Victor Stinner <victor.stinner@haypocalc.com> parents: 67130 diff changeset	1150 p2.stdout.close()
77483f3e491a test_subprocess: close pipes at the end of test_pipe_cloexec_real_tools() Victor Stinner <victor.stinner@haypocalc.com> parents: 67130 diff changeset	1151
66868 de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1152 def test_close_fds(self):
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1153 fd_status = support.findfile("fd_status.py", subdir="subprocessdata")
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1154
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1155 fds = os.pipe()
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1156 self.addCleanup(os.close, fds[0])
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1157 self.addCleanup(os.close, fds[1])
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1158
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1159 open_fds = set(fds)
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1160
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1161 p = subprocess.Popen([sys.executable, fd_status],
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1162 stdout=subprocess.PIPE, close_fds=False)
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1163 output, ignored = p.communicate()
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1164 remaining_fds = set(map(int, output.split(b',')))
66895 b066bd0b64e7 Merged revisions 87233 via svnmerge from Gregory P. Smith <greg@mad-scientist.com> parents: 66340 diff changeset	1165
66868 de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1166 self.assertEqual(remaining_fds & open_fds, open_fds,
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1167 "Some fds were closed")
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1168
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1169 p = subprocess.Popen([sys.executable, fd_status],
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1170 stdout=subprocess.PIPE, close_fds=True)
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1171 output, ignored = p.communicate()
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1172 remaining_fds = set(map(int, output.split(b',')))
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1173
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1174 self.assertFalse(remaining_fds & open_fds,
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1175 "Some fds were left open")
de05d846795e issue7213: Open the pipes used by subprocesses with the FD_CLOEXEC flag from Gregory P. Smith <greg@mad-scientist.com> parents: 66867 diff changeset	1176 self.assertIn(1, remaining_fds, "Subprocess failed")
67311 e9a51d463579 Merged revisions 87695 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 66897 diff changeset	1177
66889 59b43dc34158 Issue #6559: fix the subprocess.Popen pass_fds implementation. Add a unittest. Gregory P. Smith <greg@mad-scientist.com> parents: 66868 diff changeset	1178 def test_pass_fds(self):
59b43dc34158 Issue #6559: fix the subprocess.Popen pass_fds implementation. Add a unittest. Gregory P. Smith <greg@mad-scientist.com> parents: 66868 diff changeset	1179 fd_status = support.findfile("fd_status.py", subdir="subprocessdata")
67311 e9a51d463579 Merged revisions 87695 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 66897 diff changeset	1180
66889 59b43dc34158 Issue #6559: fix the subprocess.Popen pass_fds implementation. Add a unittest. Gregory P. Smith <greg@mad-scientist.com> parents: 66868 diff changeset	1181 open_fds = set()
67311 e9a51d463579 Merged revisions 87695 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 66897 diff changeset	1182
66889 59b43dc34158 Issue #6559: fix the subprocess.Popen pass_fds implementation. Add a unittest. Gregory P. Smith <greg@mad-scientist.com> parents: 66868 diff changeset	1183 for x in range(5):
59b43dc34158 Issue #6559: fix the subprocess.Popen pass_fds implementation. Add a unittest. Gregory P. Smith <greg@mad-scientist.com> parents: 66868 diff changeset	1184 fds = os.pipe()
59b43dc34158 Issue #6559: fix the subprocess.Popen pass_fds implementation. Add a unittest. Gregory P. Smith <greg@mad-scientist.com> parents: 66868 diff changeset	1185 self.addCleanup(os.close, fds[0])
59b43dc34158 Issue #6559: fix the subprocess.Popen pass_fds implementation. Add a unittest. Gregory P. Smith <greg@mad-scientist.com> parents: 66868 diff changeset	1186 self.addCleanup(os.close, fds[1])
59b43dc34158 Issue #6559: fix the subprocess.Popen pass_fds implementation. Add a unittest. Gregory P. Smith <greg@mad-scientist.com> parents: 66868 diff changeset	1187 open_fds.update(fds)
67311 e9a51d463579 Merged revisions 87695 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 66897 diff changeset	1188
66889 59b43dc34158 Issue #6559: fix the subprocess.Popen pass_fds implementation. Add a unittest. Gregory P. Smith <greg@mad-scientist.com> parents: 66868 diff changeset	1189 for fd in open_fds:
59b43dc34158 Issue #6559: fix the subprocess.Popen pass_fds implementation. Add a unittest. Gregory P. Smith <greg@mad-scientist.com> parents: 66868 diff changeset	1190 p = subprocess.Popen([sys.executable, fd_status],
59b43dc34158 Issue #6559: fix the subprocess.Popen pass_fds implementation. Add a unittest. Gregory P. Smith <greg@mad-scientist.com> parents: 66868 diff changeset	1191 stdout=subprocess.PIPE, close_fds=True,
59b43dc34158 Issue #6559: fix the subprocess.Popen pass_fds implementation. Add a unittest. Gregory P. Smith <greg@mad-scientist.com> parents: 66868 diff changeset	1192 pass_fds=(fd, ))
59b43dc34158 Issue #6559: fix the subprocess.Popen pass_fds implementation. Add a unittest. Gregory P. Smith <greg@mad-scientist.com> parents: 66868 diff changeset	1193 output, ignored = p.communicate()
59b43dc34158 Issue #6559: fix the subprocess.Popen pass_fds implementation. Add a unittest. Gregory P. Smith <greg@mad-scientist.com> parents: 66868 diff changeset	1194
59b43dc34158 Issue #6559: fix the subprocess.Popen pass_fds implementation. Add a unittest. Gregory P. Smith <greg@mad-scientist.com> parents: 66868 diff changeset	1195 remaining_fds = set(map(int, output.split(b',')))
59b43dc34158 Issue #6559: fix the subprocess.Popen pass_fds implementation. Add a unittest. Gregory P. Smith <greg@mad-scientist.com> parents: 66868 diff changeset	1196 to_be_closed = open_fds - {fd}
67311 e9a51d463579 Merged revisions 87695 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 66897 diff changeset	1197
66889 59b43dc34158 Issue #6559: fix the subprocess.Popen pass_fds implementation. Add a unittest. Gregory P. Smith <greg@mad-scientist.com> parents: 66868 diff changeset	1198 self.assertIn(fd, remaining_fds, "fd to be passed not passed")
59b43dc34158 Issue #6559: fix the subprocess.Popen pass_fds implementation. Add a unittest. Gregory P. Smith <greg@mad-scientist.com> parents: 66868 diff changeset	1199 self.assertFalse(remaining_fds & to_be_closed,
59b43dc34158 Issue #6559: fix the subprocess.Popen pass_fds implementation. Add a unittest. Gregory P. Smith <greg@mad-scientist.com> parents: 66868 diff changeset	1200 "fd to be closed passed")
66895 b066bd0b64e7 Merged revisions 87233 via svnmerge from Gregory P. Smith <greg@mad-scientist.com> parents: 66340 diff changeset	1201
66889 59b43dc34158 Issue #6559: fix the subprocess.Popen pass_fds implementation. Add a unittest. Gregory P. Smith <greg@mad-scientist.com> parents: 66868 diff changeset	1202 # pass_fds overrides close_fds with a warning.
59b43dc34158 Issue #6559: fix the subprocess.Popen pass_fds implementation. Add a unittest. Gregory P. Smith <greg@mad-scientist.com> parents: 66868 diff changeset	1203 with self.assertWarns(RuntimeWarning) as context:
59b43dc34158 Issue #6559: fix the subprocess.Popen pass_fds implementation. Add a unittest. Gregory P. Smith <greg@mad-scientist.com> parents: 66868 diff changeset	1204 self.assertFalse(subprocess.call(
59b43dc34158 Issue #6559: fix the subprocess.Popen pass_fds implementation. Add a unittest. Gregory P. Smith <greg@mad-scientist.com> parents: 66868 diff changeset	1205 [sys.executable, "-c", "import sys; sys.exit(0)"],
59b43dc34158 Issue #6559: fix the subprocess.Popen pass_fds implementation. Add a unittest. Gregory P. Smith <greg@mad-scientist.com> parents: 66868 diff changeset	1206 close_fds=False, pass_fds=(fd, )))
59b43dc34158 Issue #6559: fix the subprocess.Popen pass_fds implementation. Add a unittest. Gregory P. Smith <greg@mad-scientist.com> parents: 66868 diff changeset	1207 self.assertIn('overriding close_fds', str(context.warning))
67311 e9a51d463579 Merged revisions 87695 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 66897 diff changeset	1208
68526 ad2bd8d338b0 Add unittests demonstrating issue #11432. Gregory P. Smith <greg@krypto.org> parents: 68492 diff changeset	1209 def test_stdout_stdin_are_single_inout_fd(self):
ad2bd8d338b0 Add unittests demonstrating issue #11432. Gregory P. Smith <greg@krypto.org> parents: 68492 diff changeset	1210 with io.open(os.devnull, "r+") as inout:
ad2bd8d338b0 Add unittests demonstrating issue #11432. Gregory P. Smith <greg@krypto.org> parents: 68492 diff changeset	1211 p = subprocess.Popen([sys.executable, "-c", "import sys; sys.exit(0)"],
ad2bd8d338b0 Add unittests demonstrating issue #11432. Gregory P. Smith <greg@krypto.org> parents: 68492 diff changeset	1212 stdout=inout, stdin=inout)
ad2bd8d338b0 Add unittests demonstrating issue #11432. Gregory P. Smith <greg@krypto.org> parents: 68492 diff changeset	1213 p.wait()
67311 e9a51d463579 Merged revisions 87695 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 66897 diff changeset	1214
68526 ad2bd8d338b0 Add unittests demonstrating issue #11432. Gregory P. Smith <greg@krypto.org> parents: 68492 diff changeset	1215 def test_stdout_stderr_are_single_inout_fd(self):
ad2bd8d338b0 Add unittests demonstrating issue #11432. Gregory P. Smith <greg@krypto.org> parents: 68492 diff changeset	1216 with io.open(os.devnull, "r+") as inout:
ad2bd8d338b0 Add unittests demonstrating issue #11432. Gregory P. Smith <greg@krypto.org> parents: 68492 diff changeset	1217 p = subprocess.Popen([sys.executable, "-c", "import sys; sys.exit(0)"],
ad2bd8d338b0 Add unittests demonstrating issue #11432. Gregory P. Smith <greg@krypto.org> parents: 68492 diff changeset	1218 stdout=inout, stderr=inout)
ad2bd8d338b0 Add unittests demonstrating issue #11432. Gregory P. Smith <greg@krypto.org> parents: 68492 diff changeset	1219 p.wait()
67311 e9a51d463579 Merged revisions 87695 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 66897 diff changeset	1220
68526 ad2bd8d338b0 Add unittests demonstrating issue #11432. Gregory P. Smith <greg@krypto.org> parents: 68492 diff changeset	1221 def test_stderr_stdin_are_single_inout_fd(self):
ad2bd8d338b0 Add unittests demonstrating issue #11432. Gregory P. Smith <greg@krypto.org> parents: 68492 diff changeset	1222 with io.open(os.devnull, "r+") as inout:
ad2bd8d338b0 Add unittests demonstrating issue #11432. Gregory P. Smith <greg@krypto.org> parents: 68492 diff changeset	1223 p = subprocess.Popen([sys.executable, "-c", "import sys; sys.exit(0)"],
ad2bd8d338b0 Add unittests demonstrating issue #11432. Gregory P. Smith <greg@krypto.org> parents: 68492 diff changeset	1224 stderr=inout, stdin=inout)
ad2bd8d338b0 Add unittests demonstrating issue #11432. Gregory P. Smith <greg@krypto.org> parents: 68492 diff changeset	1225 p.wait()
67311 e9a51d463579 Merged revisions 87695 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 66897 diff changeset	1226
66893 767420808a62 Issue #1731717: Fixed the problem where subprocess.wait() could cause an Gregory P. Smith <greg@mad-scientist.com> parents: 66889 diff changeset	1227 def test_wait_when_sigchild_ignored(self):
767420808a62 Issue #1731717: Fixed the problem where subprocess.wait() could cause an Gregory P. Smith <greg@mad-scientist.com> parents: 66889 diff changeset	1228 # NOTE: sigchild_ignore.py may not be an effective test on all OSes.
767420808a62 Issue #1731717: Fixed the problem where subprocess.wait() could cause an Gregory P. Smith <greg@mad-scientist.com> parents: 66889 diff changeset	1229 sigchild_ignore = support.findfile("sigchild_ignore.py",
767420808a62 Issue #1731717: Fixed the problem where subprocess.wait() could cause an Gregory P. Smith <greg@mad-scientist.com> parents: 66889 diff changeset	1230 subdir="subprocessdata")
767420808a62 Issue #1731717: Fixed the problem where subprocess.wait() could cause an Gregory P. Smith <greg@mad-scientist.com> parents: 66889 diff changeset	1231 p = subprocess.Popen([sys.executable, sigchild_ignore],
767420808a62 Issue #1731717: Fixed the problem where subprocess.wait() could cause an Gregory P. Smith <greg@mad-scientist.com> parents: 66889 diff changeset	1232 stdout=subprocess.PIPE, stderr=subprocess.PIPE)
767420808a62 Issue #1731717: Fixed the problem where subprocess.wait() could cause an Gregory P. Smith <greg@mad-scientist.com> parents: 66889 diff changeset	1233 stdout, stderr = p.communicate()
767420808a62 Issue #1731717: Fixed the problem where subprocess.wait() could cause an Gregory P. Smith <greg@mad-scientist.com> parents: 66889 diff changeset	1234 self.assertEqual(0, p.returncode, "sigchild_ignore.py exited"
66896 12381f19e2ad Fix "BytesWarning: str() on a bytes instance" Gregory P. Smith <greg@mad-scientist.com> parents: 66893 diff changeset	1235 " non-zero with this error:\n%s" %
12381f19e2ad Fix "BytesWarning: str() on a bytes instance" Gregory P. Smith <greg@mad-scientist.com> parents: 66893 diff changeset	1236 stderr.decode('utf8'))
67311 e9a51d463579 Merged revisions 87695 via svnmerge from Antoine Pitrou <solipsis@pitrou.net> parents: 66897 diff changeset	1237
68680 7451da272111 Issue #11459: A `bufsize` value of 0 in subprocess.Popen() really creates Antoine Pitrou <solipsis@pitrou.net> parents: 68554 68679 diff changeset	1238 def test_select_unbuffered(self):
7451da272111 Issue #11459: A `bufsize` value of 0 in subprocess.Popen() really creates Antoine Pitrou <solipsis@pitrou.net> parents: 68554 68679 diff changeset	1239 # Issue #11459: bufsize=0 should really set the pipes as
7451da272111 Issue #11459: A `bufsize` value of 0 in subprocess.Popen() really creates Antoine Pitrou <solipsis@pitrou.net> parents: 68554 68679 diff changeset	1240 # unbuffered (and therefore let select() work properly).
7451da272111 Issue #11459: A `bufsize` value of 0 in subprocess.Popen() really creates Antoine Pitrou <solipsis@pitrou.net> parents: 68554 68679 diff changeset	1241 select = support.import_module("select")
7451da272111 Issue #11459: A `bufsize` value of 0 in subprocess.Popen() really creates Antoine Pitrou <solipsis@pitrou.net> parents: 68554 68679 diff changeset	1242 p = subprocess.Popen([sys.executable, "-c",
7451da272111 Issue #11459: A `bufsize` value of 0 in subprocess.Popen() really creates Antoine Pitrou <solipsis@pitrou.net> parents: 68554 68679 diff changeset	1243 'import sys;'
7451da272111 Issue #11459: A `bufsize` value of 0 in subprocess.Popen() really creates Antoine Pitrou <solipsis@pitrou.net> parents: 68554 68679 diff changeset	1244 'sys.stdout.write("apple")'],
7451da272111 Issue #11459: A `bufsize` value of 0 in subprocess.Popen() really creates Antoine Pitrou <solipsis@pitrou.net> parents: 68554 68679 diff changeset	1245 stdout=subprocess.PIPE,
7451da272111 Issue #11459: A `bufsize` value of 0 in subprocess.Popen() really creates Antoine Pitrou <solipsis@pitrou.net> parents: 68554 68679 diff changeset	1246 bufsize=0)
7451da272111 Issue #11459: A `bufsize` value of 0 in subprocess.Popen() really creates Antoine Pitrou <solipsis@pitrou.net> parents: 68554 68679 diff changeset	1247 f = p.stdout
68981 72f6f5a29e7a Merge with 3.1 Ross Lagerwall <rosslagerwall@gmail.com> parents: 68680 68980 diff changeset	1248 self.addCleanup(f.close)
68680 7451da272111 Issue #11459: A `bufsize` value of 0 in subprocess.Popen() really creates Antoine Pitrou <solipsis@pitrou.net> parents: 68554 68679 diff changeset	1249 try:
7451da272111 Issue #11459: A `bufsize` value of 0 in subprocess.Popen() really creates Antoine Pitrou <solipsis@pitrou.net> parents: 68554 68679 diff changeset	1250 self.assertEqual(f.read(4), b"appl")
7451da272111 Issue #11459: A `bufsize` value of 0 in subprocess.Popen() really creates Antoine Pitrou <solipsis@pitrou.net> parents: 68554 68679 diff changeset	1251 self.assertIn(f, select.select([f], [], [], 0.0)[0])
7451da272111 Issue #11459: A `bufsize` value of 0 in subprocess.Popen() really creates Antoine Pitrou <solipsis@pitrou.net> parents: 68554 68679 diff changeset	1252 finally:
7451da272111 Issue #11459: A `bufsize` value of 0 in subprocess.Popen() really creates Antoine Pitrou <solipsis@pitrou.net> parents: 68554 68679 diff changeset	1253 p.wait()
7451da272111 Issue #11459: A `bufsize` value of 0 in subprocess.Popen() really creates Antoine Pitrou <solipsis@pitrou.net> parents: 68554 68679 diff changeset	1254
66895 b066bd0b64e7 Merged revisions 87233 via svnmerge from Gregory P. Smith <greg@mad-scientist.com> parents: 66340 diff changeset	1255
59340 04955e7eb597 Merged revisions 78661-78662 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59216 diff changeset	1256 @unittest.skipUnless(mswindows, "Windows specific tests")
60057 bb2ca1b63321 Merged revisions 79344,79346,79350 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59723 diff changeset	1257 class Win32ProcessTestCase(BaseTestCase):
59340 04955e7eb597 Merged revisions 78661-78662 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59216 diff changeset	1258
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1259 def test_startupinfo(self):
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1260 # startupinfo argument
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1261 # We uses hardcoded constants, because we do not want to
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1262 # depend on win32all.
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1263 STARTF_USESHOWWINDOW = 1
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1264 SW_MAXIMIZE = 3
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1265 startupinfo = subprocess.STARTUPINFO()
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1266 startupinfo.dwFlags = STARTF_USESHOWWINDOW
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1267 startupinfo.wShowWindow = SW_MAXIMIZE
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1268 # Since Python is a console process, it won't be affected
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1269 # by wShowWindow, but the argument should be silently
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1270 # ignored
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1271 subprocess.call([sys.executable, "-c", "import sys; sys.exit(0)"],
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	1272 startupinfo=startupinfo)
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	1273
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1274 def test_creationflags(self):
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1275 # creationflags argument
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1276 CREATE_NEW_CONSOLE = 16
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1277 sys.stderr.write(" a DOS box should flash briefly ...\n")
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1278 subprocess.call(sys.executable +
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1279 ' -c "import time; time.sleep(0.25)"',
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1280 creationflags=CREATE_NEW_CONSOLE)
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	1281
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1282 def test_invalid_args(self):
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1283 # invalid arguments should raise ValueError
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1284 self.assertRaises(ValueError, subprocess.call,
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1285 [sys.executable, "-c",
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1286 "import sys; sys.exit(47)"],
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1287 preexec_fn=lambda: 1)
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1288 self.assertRaises(ValueError, subprocess.call,
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1289 [sys.executable, "-c",
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1290 "import sys; sys.exit(47)"],
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1291 stdout=subprocess.PIPE,
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1292 close_fds=True)
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	1293
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1294 def test_close_fds(self):
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1295 # close file descriptors
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1296 rc = subprocess.call([sys.executable, "-c",
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1297 "import sys; sys.exit(47)"],
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1298 close_fds=True)
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1299 self.assertEqual(rc, 47)
41969 9d02a4adf574 Merged revisions 55631-55794 via svnmerge from Guido van Rossum <guido@python.org> parents: 41845 diff changeset	1300
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1301 def test_shell_sequence(self):
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1302 # Run command through the shell (sequence)
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1303 newenv = os.environ.copy()
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1304 newenv["FRUIT"] = "physalis"
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1305 p = subprocess.Popen(["set"], shell=1,
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1306 stdout=subprocess.PIPE,
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1307 env=newenv)
65977 fe8962fc0e11 Fix a number of ResourceWarnings on Windows due to open pipes. Brian Curtin <brian.curtin@gmail.com> parents: 65955 diff changeset	1308 self.addCleanup(p.stdout.close)
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1309 self.assertIn(b"physalis", p.stdout.read())
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	1310
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1311 def test_shell_string(self):
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1312 # Run command through the shell (string)
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1313 newenv = os.environ.copy()
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1314 newenv["FRUIT"] = "physalis"
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1315 p = subprocess.Popen("set", shell=1,
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1316 stdout=subprocess.PIPE,
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1317 env=newenv)
65977 fe8962fc0e11 Fix a number of ResourceWarnings on Windows due to open pipes. Brian Curtin <brian.curtin@gmail.com> parents: 65955 diff changeset	1318 self.addCleanup(p.stdout.close)
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1319 self.assertIn(b"physalis", p.stdout.read())
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	1320
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1321 def test_call_string(self):
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1322 # call() function with string argument on Windows
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1323 rc = subprocess.call(sys.executable +
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1324 ' -c "import sys; sys.exit(47)"')
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1325 self.assertEqual(rc, 47)
46505 63965327b3ef Merged revisions 62386-62387,62389-62393,62396,62400-62402,62407,62409-62410,62412-62414,62418-62419 via svnmerge from Christian Heimes <christian@cheimes.de> parents: 44810 diff changeset	1326
59471 ac80884ddcc3 Merged revisions 78736,78759,78761,78767,78788-78789 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59398 diff changeset	1327 def _kill_process(self, method, *args):
ac80884ddcc3 Merged revisions 78736,78759,78761,78767,78788-78789 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59398 diff changeset	1328 # Some win32 buildbot raises EOFError if stdin is inherited
64927 f6a0eb730a9b Make _kill_process more robust under Windows too (see issue #8432) Antoine Pitrou <solipsis@pitrou.net> parents: 64843 diff changeset	1329 p = subprocess.Popen([sys.executable, "-c", """if 1:
f6a0eb730a9b Make _kill_process more robust under Windows too (see issue #8432) Antoine Pitrou <solipsis@pitrou.net> parents: 64843 diff changeset	1330 import sys, time
f6a0eb730a9b Make _kill_process more robust under Windows too (see issue #8432) Antoine Pitrou <solipsis@pitrou.net> parents: 64843 diff changeset	1331 sys.stdout.write('x\\n')
f6a0eb730a9b Make _kill_process more robust under Windows too (see issue #8432) Antoine Pitrou <solipsis@pitrou.net> parents: 64843 diff changeset	1332 sys.stdout.flush()
f6a0eb730a9b Make _kill_process more robust under Windows too (see issue #8432) Antoine Pitrou <solipsis@pitrou.net> parents: 64843 diff changeset	1333 time.sleep(30)
f6a0eb730a9b Make _kill_process more robust under Windows too (see issue #8432) Antoine Pitrou <solipsis@pitrou.net> parents: 64843 diff changeset	1334 """],
f6a0eb730a9b Make _kill_process more robust under Windows too (see issue #8432) Antoine Pitrou <solipsis@pitrou.net> parents: 64843 diff changeset	1335 stdin=subprocess.PIPE,
f6a0eb730a9b Make _kill_process more robust under Windows too (see issue #8432) Antoine Pitrou <solipsis@pitrou.net> parents: 64843 diff changeset	1336 stdout=subprocess.PIPE,
f6a0eb730a9b Make _kill_process more robust under Windows too (see issue #8432) Antoine Pitrou <solipsis@pitrou.net> parents: 64843 diff changeset	1337 stderr=subprocess.PIPE)
65977 fe8962fc0e11 Fix a number of ResourceWarnings on Windows due to open pipes. Brian Curtin <brian.curtin@gmail.com> parents: 65955 diff changeset	1338 self.addCleanup(p.stdout.close)
fe8962fc0e11 Fix a number of ResourceWarnings on Windows due to open pipes. Brian Curtin <brian.curtin@gmail.com> parents: 65955 diff changeset	1339 self.addCleanup(p.stderr.close)
fe8962fc0e11 Fix a number of ResourceWarnings on Windows due to open pipes. Brian Curtin <brian.curtin@gmail.com> parents: 65955 diff changeset	1340 self.addCleanup(p.stdin.close)
64927 f6a0eb730a9b Make _kill_process more robust under Windows too (see issue #8432) Antoine Pitrou <solipsis@pitrou.net> parents: 64843 diff changeset	1341 # Wait for the interpreter to be completely initialized before
f6a0eb730a9b Make _kill_process more robust under Windows too (see issue #8432) Antoine Pitrou <solipsis@pitrou.net> parents: 64843 diff changeset	1342 # sending any signal.
f6a0eb730a9b Make _kill_process more robust under Windows too (see issue #8432) Antoine Pitrou <solipsis@pitrou.net> parents: 64843 diff changeset	1343 p.stdout.read(1)
f6a0eb730a9b Make _kill_process more robust under Windows too (see issue #8432) Antoine Pitrou <solipsis@pitrou.net> parents: 64843 diff changeset	1344 getattr(p, method)(*args)
60057 bb2ca1b63321 Merged revisions 79344,79346,79350 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59723 diff changeset	1345 _, stderr = p.communicate()
bb2ca1b63321 Merged revisions 79344,79346,79350 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59723 diff changeset	1346 self.assertStderrEqual(stderr, b'')
64927 f6a0eb730a9b Make _kill_process more robust under Windows too (see issue #8432) Antoine Pitrou <solipsis@pitrou.net> parents: 64843 diff changeset	1347 returncode = p.wait()
59471 ac80884ddcc3 Merged revisions 78736,78759,78761,78767,78788-78789 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59398 diff changeset	1348 self.assertNotEqual(returncode, 0)
46505 63965327b3ef Merged revisions 62386-62387,62389-62393,62396,62400-62402,62407,62409-62410,62412-62414,62418-62419 via svnmerge from Christian Heimes <christian@cheimes.de> parents: 44810 diff changeset	1349
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1350 def test_send_signal(self):
59471 ac80884ddcc3 Merged revisions 78736,78759,78761,78767,78788-78789 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59398 diff changeset	1351 self._kill_process('send_signal', signal.SIGTERM)
46505 63965327b3ef Merged revisions 62386-62387,62389-62393,62396,62400-62402,62407,62409-62410,62412-62414,62418-62419 via svnmerge from Christian Heimes <christian@cheimes.de> parents: 44810 diff changeset	1352
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1353 def test_kill(self):
59471 ac80884ddcc3 Merged revisions 78736,78759,78761,78767,78788-78789 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59398 diff changeset	1354 self._kill_process('kill')
46505 63965327b3ef Merged revisions 62386-62387,62389-62393,62396,62400-62402,62407,62409-62410,62412-62414,62418-62419 via svnmerge from Christian Heimes <christian@cheimes.de> parents: 44810 diff changeset	1355
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1356 def test_terminate(self):
59471 ac80884ddcc3 Merged revisions 78736,78759,78761,78767,78788-78789 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59398 diff changeset	1357 self._kill_process('terminate')
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	1358
63930 8499ae5d9719 #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63843 diff changeset	1359
47331 0525cd5b2582 The commands module has been removed. The getoutput() and getstatusoutput() Brett Cannon <bcannon@gmail.com> parents: 47303 diff changeset	1360 # The module says:
0525cd5b2582 The commands module has been removed. The getoutput() and getstatusoutput() Brett Cannon <bcannon@gmail.com> parents: 47303 diff changeset	1361 # "NB This only works (and is only relevant) for UNIX."
0525cd5b2582 The commands module has been removed. The getoutput() and getstatusoutput() Brett Cannon <bcannon@gmail.com> parents: 47303 diff changeset	1362 #
0525cd5b2582 The commands module has been removed. The getoutput() and getstatusoutput() Brett Cannon <bcannon@gmail.com> parents: 47303 diff changeset	1363 # Actually, getoutput should work on any platform with an os.popen, but
0525cd5b2582 The commands module has been removed. The getoutput() and getstatusoutput() Brett Cannon <bcannon@gmail.com> parents: 47303 diff changeset	1364 # I'll take the comment as given, and skip this suite.
59340 04955e7eb597 Merged revisions 78661-78662 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 59216 diff changeset	1365 @unittest.skipUnless(os.name == 'posix', "only relevant for UNIX")
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1366 class CommandTests(unittest.TestCase):
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1367 def test_getoutput(self):
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1368 self.assertEqual(subprocess.getoutput('echo xyzzy'), 'xyzzy')
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1369 self.assertEqual(subprocess.getstatusoutput('echo xyzzy'),
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1370 (0, 'xyzzy'))
47331 0525cd5b2582 The commands module has been removed. The getoutput() and getstatusoutput() Brett Cannon <bcannon@gmail.com> parents: 47303 diff changeset	1371
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1372 # we use mkdtemp in the next line to create an empty directory
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1373 # under our exclusive control; from that, we can invent a pathname
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1374 # that we _know_ won't exist. This is guaranteed to fail.
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1375 dir = None
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1376 try:
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1377 dir = tempfile.mkdtemp()
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1378 name = os.path.join(dir, "foo")
47331 0525cd5b2582 The commands module has been removed. The getoutput() and getstatusoutput() Brett Cannon <bcannon@gmail.com> parents: 47303 diff changeset	1379
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1380 status, output = subprocess.getstatusoutput('cat ' + name)
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1381 self.assertNotEqual(status, 0)
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1382 finally:
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1383 if dir is not None:
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1384 os.rmdir(dir)
47331 0525cd5b2582 The commands module has been removed. The getoutput() and getstatusoutput() Brett Cannon <bcannon@gmail.com> parents: 47303 diff changeset	1385
55602 680ad8eed33b Merged revisions 73833,73838,73850-73852,73856-73857 via svnmerge from Georg Brandl <georg@python.org> parents: 55593 diff changeset	1386
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1387 @unittest.skipUnless(getattr(subprocess, '_has_poll', False),
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1388 "poll system call not supported")
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1389 class ProcessTestCaseNoPoll(ProcessTestCase):
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1390 def setUp(self):
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1391 subprocess._has_poll = False
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1392 ProcessTestCase.setUp(self)
63930 8499ae5d9719 #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63843 diff changeset	1393
59193 01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1394 def tearDown(self):
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1395 subprocess._has_poll = True
01c58cef82d4 Recorded merge of revisions 78508-78509 via svnmerge from Florent Xicluna <florent.xicluna@gmail.com> parents: 58931 diff changeset	1396 ProcessTestCase.tearDown(self)
63930 8499ae5d9719 #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63843 diff changeset	1397
8499ae5d9719 #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63843 diff changeset	1398
59605 768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	1399 @unittest.skipUnless(getattr(subprocess, '_posixsubprocess', False),
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	1400 "_posixsubprocess extension module not found.")
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	1401 class ProcessTestCasePOSIXPurePython(ProcessTestCase, POSIXProcessTestCase):
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	1402 def setUp(self):
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	1403 subprocess._posixsubprocess = None
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	1404 ProcessTestCase.setUp(self)
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	1405 POSIXProcessTestCase.setUp(self)
55602 680ad8eed33b Merged revisions 73833,73838,73850-73852,73856-73857 via svnmerge from Georg Brandl <georg@python.org> parents: 55593 diff changeset	1406
59605 768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	1407 def tearDown(self):
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	1408 subprocess._posixsubprocess = sys.modules['_posixsubprocess']
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	1409 POSIXProcessTestCase.tearDown(self)
768722b2ae0a * Replaces the internals of the subprocess module from fork through exec on Gregory P. Smith <greg@mad-scientist.com> parents: 59507 diff changeset	1410 ProcessTestCase.tearDown(self)
55602 680ad8eed33b Merged revisions 73833,73838,73850-73852,73856-73857 via svnmerge from Georg Brandl <georg@python.org> parents: 55593 diff changeset	1411
680ad8eed33b Merged revisions 73833,73838,73850-73852,73856-73857 via svnmerge from Georg Brandl <georg@python.org> parents: 55593 diff changeset	1412
59209 1f158238fa76 Merged revisions 78524 via svnmerge from Gregory P. Smith <greg@mad-scientist.com> parents: 55626 diff changeset	1413 class HelperFunctionTests(unittest.TestCase):
59217 d1c62882c73b Merged revisions 78533 via svnmerge from Gregory P. Smith <greg@mad-scientist.com> parents: 59209 diff changeset	1414 @unittest.skipIf(mswindows, "errno and EINTR make no sense on windows")
59209 1f158238fa76 Merged revisions 78524 via svnmerge from Gregory P. Smith <greg@mad-scientist.com> parents: 55626 diff changeset	1415 def test_eintr_retry_call(self):
1f158238fa76 Merged revisions 78524 via svnmerge from Gregory P. Smith <greg@mad-scientist.com> parents: 55626 diff changeset	1416 record_calls = []
1f158238fa76 Merged revisions 78524 via svnmerge from Gregory P. Smith <greg@mad-scientist.com> parents: 55626 diff changeset	1417 def fake_os_func(*args):
1f158238fa76 Merged revisions 78524 via svnmerge from Gregory P. Smith <greg@mad-scientist.com> parents: 55626 diff changeset	1418 record_calls.append(args)
1f158238fa76 Merged revisions 78524 via svnmerge from Gregory P. Smith <greg@mad-scientist.com> parents: 55626 diff changeset	1419 if len(record_calls) == 2:
1f158238fa76 Merged revisions 78524 via svnmerge from Gregory P. Smith <greg@mad-scientist.com> parents: 55626 diff changeset	1420 raise OSError(errno.EINTR, "fake interrupted system call")
1f158238fa76 Merged revisions 78524 via svnmerge from Gregory P. Smith <greg@mad-scientist.com> parents: 55626 diff changeset	1421 return tuple(reversed(args))
1f158238fa76 Merged revisions 78524 via svnmerge from Gregory P. Smith <greg@mad-scientist.com> parents: 55626 diff changeset	1422
1f158238fa76 Merged revisions 78524 via svnmerge from Gregory P. Smith <greg@mad-scientist.com> parents: 55626 diff changeset	1423 self.assertEqual((999, 256),
1f158238fa76 Merged revisions 78524 via svnmerge from Gregory P. Smith <greg@mad-scientist.com> parents: 55626 diff changeset	1424 subprocess._eintr_retry_call(fake_os_func, 256, 999))
1f158238fa76 Merged revisions 78524 via svnmerge from Gregory P. Smith <greg@mad-scientist.com> parents: 55626 diff changeset	1425 self.assertEqual([(256, 999)], record_calls)
1f158238fa76 Merged revisions 78524 via svnmerge from Gregory P. Smith <greg@mad-scientist.com> parents: 55626 diff changeset	1426 # This time there will be an EINTR so it will loop once.
1f158238fa76 Merged revisions 78524 via svnmerge from Gregory P. Smith <greg@mad-scientist.com> parents: 55626 diff changeset	1427 self.assertEqual((666,),
1f158238fa76 Merged revisions 78524 via svnmerge from Gregory P. Smith <greg@mad-scientist.com> parents: 55626 diff changeset	1428 subprocess._eintr_retry_call(fake_os_func, 666))
1f158238fa76 Merged revisions 78524 via svnmerge from Gregory P. Smith <greg@mad-scientist.com> parents: 55626 diff changeset	1429 self.assertEqual([(256, 999), (666,), (666,)], record_calls)
1f158238fa76 Merged revisions 78524 via svnmerge from Gregory P. Smith <greg@mad-scientist.com> parents: 55626 diff changeset	1430
59207 55778855cd25 Merged revisions 78523 via svnmerge from Gregory P. Smith <greg@mad-scientist.com> parents: 59193 diff changeset	1431
63922 56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1432 @unittest.skipUnless(mswindows, "Windows-specific tests")
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1433 class CommandsWithSpaces (BaseTestCase):
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1434
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1435 def setUp(self):
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1436 super().setUp()
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1437 f, fname = mkstemp(".py", "te st")
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1438 self.fname = fname.lower ()
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1439 os.write(f, b"import sys;"
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1440 b"sys.stdout.write('%d %s' % (len(sys.argv), [a.lower () for a in sys.argv]))"
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1441 )
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1442 os.close(f)
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1443
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1444 def tearDown(self):
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1445 os.remove(self.fname)
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1446 super().tearDown()
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1447
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1448 def with_spaces(self, args, *kwargs):
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1449 kwargs['stdout'] = subprocess.PIPE
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1450 p = subprocess.Popen(args, *kwargs)
65977 fe8962fc0e11 Fix a number of ResourceWarnings on Windows due to open pipes. Brian Curtin <brian.curtin@gmail.com> parents: 65955 diff changeset	1451 self.addCleanup(p.stdout.close)
63922 56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1452 self.assertEqual(
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1453 p.stdout.read ().decode("mbcs"),
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1454 "2 [%r, 'ab cd']" % self.fname
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1455 )
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1456
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1457 def test_shell_string_with_spaces(self):
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1458 # call() function with string argument with spaces on Windows
63956 aa9c8e82c435 Fix #9588. Add sys.executable to two shell=True tests. Brian Curtin <brian.curtin@gmail.com> parents: 63922 diff changeset	1459 self.with_spaces('"%s" "%s" "%s"' % (sys.executable, self.fname,
aa9c8e82c435 Fix #9588. Add sys.executable to two shell=True tests. Brian Curtin <brian.curtin@gmail.com> parents: 63922 diff changeset	1460 "ab cd"), shell=1)
63922 56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1461
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1462 def test_shell_sequence_with_spaces(self):
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1463 # call() function with sequence argument with spaces on Windows
63956 aa9c8e82c435 Fix #9588. Add sys.executable to two shell=True tests. Brian Curtin <brian.curtin@gmail.com> parents: 63922 diff changeset	1464 self.with_spaces([sys.executable, self.fname, "ab cd"], shell=1)
63922 56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1465
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1466 def test_noshell_string_with_spaces(self):
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1467 # call() function with string argument with spaces on Windows
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1468 self.with_spaces('"%s" "%s" "%s"' % (sys.executable, self.fname,
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1469 "ab cd"))
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1470
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1471 def test_noshell_sequence_with_spaces(self):
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1472 # call() function with sequence argument with spaces on Windows
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1473 self.with_spaces([sys.executable, self.fname, "ab cd"])
56066a5825ca #2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows Tim Golden <mail@timgolden.me.uk> parents: 63840 diff changeset	1474
66622 e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1475
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1476 class ContextManagerTests(ProcessTestCase):
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1477
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1478 def test_pipe(self):
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1479 with subprocess.Popen([sys.executable, "-c",
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1480 "import sys;"
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1481 "sys.stdout.write('stdout');"
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1482 "sys.stderr.write('stderr');"],
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1483 stdout=subprocess.PIPE,
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1484 stderr=subprocess.PIPE) as proc:
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1485 self.assertEqual(proc.stdout.read(), b"stdout")
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1486 self.assertStderrEqual(proc.stderr.read(), b"stderr")
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1487
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1488 self.assertTrue(proc.stdout.closed)
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1489 self.assertTrue(proc.stderr.closed)
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1490
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1491 def test_returncode(self):
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1492 with subprocess.Popen([sys.executable, "-c",
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1493 "import sys; sys.exit(100)"]) as proc:
70049 b00a64a5cb93 merge - 7a3f3ad83676 Fixes Issue #12044. Gregory P. Smith <greg@krypto.org> parents: 69155 diff changeset	1494 pass
b00a64a5cb93 merge - 7a3f3ad83676 Fixes Issue #12044. Gregory P. Smith <greg@krypto.org> parents: 69155 diff changeset	1495 # __exit__ calls wait(), so the returncode should be set
66622 e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1496 self.assertEqual(proc.returncode, 100)
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1497
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1498 def test_communicate_stdin(self):
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1499 with subprocess.Popen([sys.executable, "-c",
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1500 "import sys;"
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1501 "sys.exit(sys.stdin.read() == 'context')"],
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1502 stdin=subprocess.PIPE) as proc:
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1503 proc.communicate(b"context")
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1504 self.assertEqual(proc.returncode, 1)
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1505
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1506 def test_invalid_args(self):
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1507 with self.assertRaises(EnvironmentError) as c:
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1508 with subprocess.Popen(['nonexisting_i_hope'],
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1509 stdout=subprocess.PIPE,
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1510 stderr=subprocess.PIPE) as proc:
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1511 pass
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1512
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1513 if c.exception.errno != errno.ENOENT: # ignore "no such file"
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1514 raise c.exception
e02021fbfe22 Fix #10554. Added context manager support to Popen objects. Brian Curtin <brian.curtin@gmail.com> parents: 66310 diff changeset	1515
59209 1f158238fa76 Merged revisions 78524 via svnmerge from Gregory P. Smith <greg@mad-scientist.com> parents: 55626 diff changeset	1516
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	1517 def test_main():
68527 8c79e3abdb31 revert the test_main() change from 08daf3ef6509 so that regrtest continues to run this properly. Gregory P. Smith <greg@krypto.org> parents: 68526 diff changeset	1518 unit_tests = (ProcessTestCase,
8c79e3abdb31 revert the test_main() change from 08daf3ef6509 so that regrtest continues to run this properly. Gregory P. Smith <greg@krypto.org> parents: 68526 diff changeset	1519 POSIXProcessTestCase,
8c79e3abdb31 revert the test_main() change from 08daf3ef6509 so that regrtest continues to run this properly. Gregory P. Smith <greg@krypto.org> parents: 68526 diff changeset	1520 Win32ProcessTestCase,
8c79e3abdb31 revert the test_main() change from 08daf3ef6509 so that regrtest continues to run this properly. Gregory P. Smith <greg@krypto.org> parents: 68526 diff changeset	1521 ProcessTestCasePOSIXPurePython,
8c79e3abdb31 revert the test_main() change from 08daf3ef6509 so that regrtest continues to run this properly. Gregory P. Smith <greg@krypto.org> parents: 68526 diff changeset	1522 CommandTests,
8c79e3abdb31 revert the test_main() change from 08daf3ef6509 so that regrtest continues to run this properly. Gregory P. Smith <greg@krypto.org> parents: 68526 diff changeset	1523 ProcessTestCaseNoPoll,
8c79e3abdb31 revert the test_main() change from 08daf3ef6509 so that regrtest continues to run this properly. Gregory P. Smith <greg@krypto.org> parents: 68526 diff changeset	1524 HelperFunctionTests,
8c79e3abdb31 revert the test_main() change from 08daf3ef6509 so that regrtest continues to run this properly. Gregory P. Smith <greg@krypto.org> parents: 68526 diff changeset	1525 CommandsWithSpaces,
8c79e3abdb31 revert the test_main() change from 08daf3ef6509 so that regrtest continues to run this properly. Gregory P. Smith <greg@krypto.org> parents: 68526 diff changeset	1526 ContextManagerTests)
8c79e3abdb31 revert the test_main() change from 08daf3ef6509 so that regrtest continues to run this properly. Gregory P. Smith <greg@krypto.org> parents: 68526 diff changeset	1527
55602 680ad8eed33b Merged revisions 73833,73838,73850-73852,73856-73857 via svnmerge from Georg Brandl <georg@python.org> parents: 55593 diff changeset	1528 support.run_unittest(*unit_tests)
47331 0525cd5b2582 The commands module has been removed. The getoutput() and getstatusoutput() Brett Cannon <bcannon@gmail.com> parents: 47303 diff changeset	1529 support.reap_children()
33723 76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	1530
76641824cf05 Added Peter Astrand's subprocess module. Fredrik Lundh <fredrik@pythonware.com> parents: diff changeset	1531 if __name__ == "__main__":
68526 ad2bd8d338b0 Add unittests demonstrating issue #11432. Gregory P. Smith <greg@krypto.org> parents: 68492 diff changeset	1532 unittest.main()

Mercurial > cpython-lru_cache-weakref

annotate Lib/test/test_subprocess.py @ 70049:b00a64a5cb93 3.2