diff options
-rw-r--r-- | test/lib/test/unit/assertions.rb | 16 | ||||
-rw-r--r-- | test/ruby/test_io.rb | 2 |
2 files changed, 16 insertions, 2 deletions
diff --git a/test/lib/test/unit/assertions.rb b/test/lib/test/unit/assertions.rb index fcd9795402..34dbe18738 100644 --- a/test/lib/test/unit/assertions.rb +++ b/test/lib/test/unit/assertions.rb @@ -715,10 +715,24 @@ eom skip end - def assert_cpu_usage_low(msg = nil, pct: 0.015) + def assert_cpu_usage_low(msg = nil, pct: 0.01) require 'benchmark' + tms = Benchmark.measure(msg || '') { yield } max = pct * tms.real + if tms.real < 0.1 # TIME_QUANTUM_USEC in thread_pthread.c + warn "test #{msg || 'assert_cpu_usage_low'} too short to be accurate" + end + + # kernel resolution can limit the minimum time we can measure + # [ruby-core:81540] + min_hz = windows? ? 67 : 100 + min_measurable = 1.0 / min_hz + min_measurable *= 1.10 # add a little (10%) to account for misc. overheads + if max < min_measurable + max = min_measurable + end + assert_operator tms.total, :<=, max, msg end diff --git a/test/ruby/test_io.rb b/test/ruby/test_io.rb index 5c98a59be0..2cd60a4fca 100644 --- a/test/ruby/test_io.rb +++ b/test/ruby/test_io.rb @@ -536,7 +536,7 @@ class TestIO < Test::Unit::TestCase msg = 'r58534 [ruby-core:80969] [Backport #13533]' IO.pipe do |r,w| r.nonblock = true - assert_cpu_usage_low(msg, pct: 0.11) do + assert_cpu_usage_low(msg) do th = Thread.new { IO.copy_stream(r, IO::NULL) } sleep 0.1 w.close |