Auto merge of #1611 - RalfJung:timing-tests, r=RalfJung

bors · bors · commit e4840ef4c98e · 2020-10-30T18:22:11.000Z
increase timeout to avoid spurious test failures

just saw this fail on macOS in PR CI, so add some extra safety margin
diff --git a/tests/run-pass/concurrency/sync.rs b/tests/run-pass/concurrency/sync.rs
@@ -109,7 +109,7 @@ fn check_conditional_variables_timed_wait_notimeout() {
         cvar.notify_one();
     });
 
-    let (_guard, timeout) = cvar.wait_timeout(guard, Duration::from_millis(200)).unwrap();
+    let (_guard, timeout) = cvar.wait_timeout(guard, Duration::from_millis(500)).unwrap();
     assert!(!timeout.timed_out());
     handle.join().unwrap();
 }

Original file line number	Diff line number	Diff line change
`@@ -109,7 +109,7 @@ fn check_conditional_variables_timed_wait_notimeout() {`
`109`	`109`	`cvar.notify_one();`
`110`	`110`	`});`
`111`	`111`
`112`		`- let (_guard, timeout) = cvar.wait_timeout(guard, Duration::from_millis(200)).unwrap();`
	`112`	`+ let (_guard, timeout) = cvar.wait_timeout(guard, Duration::from_millis(500)).unwrap();`
`113`	`113`	`assert!(!timeout.timed_out());`
`114`	`114`	`handle.join().unwrap();`
`115`	`115`	`}`