]> granicus.if.org Git - python/commitdiff
bpo-33723: Fix test_time.test_thread_time() (GH-10724)
authorVictor Stinner <vstinner@redhat.com>
Mon, 26 Nov 2018 20:19:29 +0000 (21:19 +0100)
committerGitHub <noreply@github.com>
Mon, 26 Nov 2018 20:19:29 +0000 (21:19 +0100)
Tolerate up to 30 ms, instead of 15 min, in other threads.

Lib/test/test_time.py

index 16b48a92f327dcb4411842c80fb1d8c87b12dfaf..381cc823014e26f651890653da7bea63511e00a9 100644 (file)
@@ -496,7 +496,8 @@ class TimeTestCase(unittest.TestCase):
         self.assertLess(stop - start, 0.020)
 
         # bpo-33723: A busy loop of 100 ms should increase process_time()
-        # by at least 15 ms
+        # by at least 15 ms. Tolerate 15 ms because of the bad resolution of
+        # the clock on Windows (around 15.6 ms).
         min_time = 0.015
         busy_time = 0.100
 
@@ -534,8 +535,11 @@ class TimeTestCase(unittest.TestCase):
         self.assertLess(stop - start, 0.020)
 
         # bpo-33723: A busy loop of 100 ms should increase thread_time()
-        # by at least 15 ms
+        # by at least 15 ms, but less than 30 ms in other threads.
+        # Tolerate 15 and 30 ms because of the bad resolution
+        # of the clock on Windows (around 15.6 ms).
         min_time = 0.015
+        max_time = 0.030
         busy_time = 0.100
 
         # thread_time() should include CPU time spent in current thread...
@@ -550,7 +554,7 @@ class TimeTestCase(unittest.TestCase):
         t.start()
         t.join()
         stop = time.thread_time()
-        self.assertLess(stop - start, min_time)
+        self.assertLess(stop - start, max_time)
 
         info = time.get_clock_info('thread_time')
         self.assertTrue(info.monotonic)