1berry/ruby
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Make test_nested_timeouts less flaky

Browse Source 
This test randomly fails due to the bug reported in [Bug #20314], where
the two timeouts are too close so that they can expire at the same time.

As a workaround, this change increases the time difference between
timeouts. This will reduce the probability of simultaneous expirations
and lower flakiness.
This commit is contained in:
Yusuke Endoh 2024-05-28 12:04:34 +09:00
parent 7c6e4bc7ca
commit 2114d0af1e
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
test/test_timeout.rb
View File

@ -66,7 +66,7 @@ class TestTimeout < Test::Unit::TestCase
a = nil a = nil
assert_raise(Timeout::Error) do assert_raise(Timeout::Error) do
Timeout.timeout(0.1) { Timeout.timeout(0.1) {
Timeout.timeout(1) { Timeout.timeout(30) {
nil while true nil while true
} }
a = 1 a = 1
@ -84,7 +84,7 @@ class TestTimeout < Test::Unit::TestCase
def test_nested_timeout_error_identity def test_nested_timeout_error_identity
begin begin
Timeout.timeout(0.1, MyNewErrorOuter) { Timeout.timeout(0.1, MyNewErrorOuter) {
Timeout.timeout(1, MyNewErrorInner) { Timeout.timeout(30, MyNewErrorInner) {
nil while true nil while true
} }
} }