]> granicus.if.org Git - python/commitdiff
Issue #7042: Use a better mechanism for testing timers in test_signal.
authorMark Dickinson <dickinsm@gmail.com>
Sat, 31 Oct 2009 10:36:06 +0000 (10:36 +0000)
committerMark Dickinson <dickinsm@gmail.com>
Sat, 31 Oct 2009 10:36:06 +0000 (10:36 +0000)
Lib/test/test_signal.py

index df0828dc737739aae8776aef7c9a1508b81517fb..c4b8e4eb55cc2a1b03d6b7b847fc50d4af4fd284 100644 (file)
@@ -360,11 +360,14 @@ class ItimerTest(unittest.TestCase):
         signal.signal(signal.SIGVTALRM, self.sig_vtalrm)
         signal.setitimer(self.itimer, 0.3, 0.2)
 
-        for i in xrange(100000000):
+        start_time = time.time()
+        while time.time() - start_time < 5.0:
             # use up some virtual time by doing real work
             _ = pow(12345, 67890, 10000019)
             if signal.getitimer(self.itimer) == (0.0, 0.0):
                 break # sig_vtalrm handler stopped this itimer
+        else:
+            self.fail('timeout waiting for sig_vtalrm signal')
 
         # virtual itimer should be (0.0, 0.0) now
         self.assertEquals(signal.getitimer(self.itimer), (0.0, 0.0))
@@ -376,9 +379,14 @@ class ItimerTest(unittest.TestCase):
         signal.signal(signal.SIGPROF, self.sig_prof)
         signal.setitimer(self.itimer, 0.2, 0.2)
 
-        for i in xrange(100000000):
+        start_time = time.time()
+        while time.time() - start_time < 5.0:
+            # do some work
+            _ = pow(12345, 67890, 10000019)
             if signal.getitimer(self.itimer) == (0.0, 0.0):
                 break # sig_prof handler stopped this itimer
+        else:
+            self.fail('timeout waiting for sig_prof signal')
 
         # profiling itimer should be (0.0, 0.0) now
         self.assertEquals(signal.getitimer(self.itimer), (0.0, 0.0))