]> granicus.if.org Git - python/commitdiff
Merged revisions 78671 via svnmerge from
authorFlorent Xicluna <florent.xicluna@gmail.com>
Fri, 5 Mar 2010 00:52:00 +0000 (00:52 +0000)
committerFlorent Xicluna <florent.xicluna@gmail.com>
Fri, 5 Mar 2010 00:52:00 +0000 (00:52 +0000)
svn+ssh://pythondev@svn.python.org/python/trunk

........
  r78671 | florent.xicluna | 2010-03-05 01:47:40 +0100 (ven, 05 mar 2010) | 3 lines

  Workaround #3137: Retry SIGINT if it is not received the first time.
  test_send_signal should not hang anymore on various Linux distributions.
........

Lib/test/test_subprocess.py

index 3de46c615aedbf6837276c2af625f699393e7ef6..d58108157ec762bf9b6319f1db8763054965d907 100644 (file)
@@ -652,9 +652,20 @@ class POSIXProcessTestCase(unittest.TestCase):
         p = subprocess.Popen([sys.executable, "-c", "input()"])
 
         # Let the process initialize correctly (Issue #3137)
-        time.sleep(.1)
+        time.sleep(0.1)
         self.assertIs(p.poll(), None)
-        p.send_signal(signal.SIGINT)
+        count, maxcount = 0, 3
+        # Retry if the process do not receive the SIGINT signal.
+        while count < maxcount and p.poll() is None:
+            p.send_signal(signal.SIGINT)
+            time.sleep(0.1)
+            count += 1
+        if p.poll() is None:
+            raise support.TestFailed("the subprocess did not receive "
+                                     "the signal SIGINT")
+        elif count > 1:
+            print("p.send_signal(SIGINT) succeeded "
+                  "after {} attempts".format(count), file=sys.stderr)
         self.assertNotEqual(p.wait(), 0)
 
     def test_kill(self):
@@ -839,6 +850,7 @@ def test_main():
                   ProcessTestCaseNoPoll,
                   HelperFunctionTests)
 
+    unit_tests = (POSIXProcessTestCase,)
     support.run_unittest(*unit_tests)
     support.reap_children()