[Python-checkins] r84913 - python/branches/py3k/Lib/test/test_subprocess.py
antoine.pitrou
python-checkins at python.org
Mon Sep 20 03:33:21 CEST 2010
Author: antoine.pitrou
Date: Mon Sep 20 03:33:21 2010
New Revision: 84913
Log:
Try a more robust implementation of _kill_process
Modified:
python/branches/py3k/Lib/test/test_subprocess.py
Modified: python/branches/py3k/Lib/test/test_subprocess.py
==============================================================================
--- python/branches/py3k/Lib/test/test_subprocess.py (original)
+++ python/branches/py3k/Lib/test/test_subprocess.py Mon Sep 20 03:33:21 2010
@@ -826,26 +826,20 @@
def _kill_process(self, method, *args):
# Do not inherit file handles from the parent.
# It should fix failures on some platforms.
- p = subprocess.Popen([sys.executable, "-c", "input()"], close_fds=True,
- stdin=subprocess.PIPE, stderr=subprocess.PIPE)
-
- # Let the process initialize (Issue #3137)
- time.sleep(0.4)
- # The process should not terminate prematurely
- self.assertIsNone(p.poll())
- # Retry if the process do not receive the signal.
- count, maxcount = 0, 10
- while count < maxcount and p.poll() is None:
- getattr(p, method)(*args)
- time.sleep(0.1)
- count += 1
-
- if count == maxcount:
- self.skipTest("apparently failed to send the signal")
- self.assertIsNotNone(p.poll(), "the subprocess did not terminate")
- if count > 1:
- print("p.{}{} succeeded after "
- "{} attempts".format(method, args, count), file=sys.stderr)
+ p = subprocess.Popen([sys.executable, "-c", """if 1:
+ import sys, time
+ sys.stdout.write('x\\n')
+ sys.stdout.flush()
+ time.sleep(30)
+ """],
+ close_fds=True,
+ stdin=subprocess.PIPE,
+ stdout=subprocess.PIPE,
+ stderr=subprocess.PIPE)
+ # Wait for the interpreter to be completely initialized before
+ # sending any signal.
+ p.stdout.read(1)
+ getattr(p, method)(*args)
return p
def test_send_signal(self):
More information about the Python-checkins
mailing list