diff options
| author | Victor Stinner <vstinner@redhat.com> | 2018-07-21 01:31:51 (GMT) |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-07-21 01:31:51 (GMT) |
| commit | e78dace8dcb23c371df19c9add65895adf436995 (patch) | |
| tree | 10de92edabad6e6c0e1af4078db2ee8718a6aeaf /Lib/test/test_time.py | |
| parent | b1147e43daeb3c51a63056b489e8d868404d4e22 (diff) | |
| download | cpython-e78dace8dcb23c371df19c9add65895adf436995.zip cpython-e78dace8dcb23c371df19c9add65895adf436995.tar.gz cpython-e78dace8dcb23c371df19c9add65895adf436995.tar.bz2 | |
bpo-33723: Fix test_time.test_process_time() (GH-8358)
The test failed on my laptop because the busy loop took 15.9 ms
whereas the test expects at least 20 ms. Modify test_process_time()
as test_thread_time() has been modified recently: only require 15 ms
instead of 20 ms.
Diffstat (limited to 'Lib/test/test_time.py')
| -rw-r--r-- | Lib/test/test_time.py | 13 |
1 files changed, 9 insertions, 4 deletions
diff --git a/Lib/test/test_time.py b/Lib/test/test_time.py index 1ebe0cf..cab437d 100644 --- a/Lib/test/test_time.py +++ b/Lib/test/test_time.py @@ -496,18 +496,23 @@ class TimeTestCase(unittest.TestCase): # on Windows self.assertLess(stop - start, 0.020) + # bpo-33723: A busy loop of 100 ms should increase process_time() + # by at least 15 ms + min_time = 0.015 + busy_time = 0.100 + # process_time() should include CPU time spent in any thread start = time.process_time() - busy_wait(0.100) + busy_wait(busy_time) stop = time.process_time() - self.assertGreaterEqual(stop - start, 0.020) # machine busy? + self.assertGreaterEqual(stop - start, min_time) - t = threading.Thread(target=busy_wait, args=(0.100,)) + t = threading.Thread(target=busy_wait, args=(busy_time,)) start = time.process_time() t.start() t.join() stop = time.process_time() - self.assertGreaterEqual(stop - start, 0.020) # machine busy? + self.assertGreaterEqual(stop - start, min_time) info = time.get_clock_info('process_time') self.assertTrue(info.monotonic) |