From 91a640176af5b164b62aa867beaf0b02fd2d93da Mon Sep 17 00:00:00 2001 From: Chris Denton Date: Wed, 6 Jul 2022 18:47:44 +0100 Subject: [PATCH] Fix ui-fulldep test --- ...s.rs => issue-81357-unsound-file-methods.rs} | 17 +++++++++++------ 1 file changed, 11 insertions(+), 6 deletions(-) rename src/test/ui-fulldeps/{issues-81357-unsound-file-methods.rs => issue-81357-unsound-file-methods.rs} (83%) diff --git a/src/test/ui-fulldeps/issues-81357-unsound-file-methods.rs b/src/test/ui-fulldeps/issue-81357-unsound-file-methods.rs similarity index 83% rename from src/test/ui-fulldeps/issues-81357-unsound-file-methods.rs rename to src/test/ui-fulldeps/issue-81357-unsound-file-methods.rs index 94b5798a0e7d2..fdf1150f8d25a 100644 --- a/src/test/ui-fulldeps/issues-81357-unsound-file-methods.rs +++ b/src/test/ui-fulldeps/issue-81357-unsound-file-methods.rs @@ -55,10 +55,9 @@ fn main() { let b = buf[0]; // capture buf[0] thread::sleep(Duration::from_millis(200)); - // In this test, success is indicated by failing. - if buf[0] == b { - panic!("Success!"); - } + // Check the buffer hasn't been written to after read. + dbg!(buf[0], b); + assert_eq!(buf[0], b); } } }) @@ -71,6 +70,12 @@ fn main() { let _ = server.write(b"x"); thread::sleep(Duration::from_millis(100)); let _ = server.write(b"y"); - let _ = t1.join(); - let _ = t2.join(); + + // This is run fail because we need to test for the `abort`. + // That failing to run is the success case. + if t1.join().is_err() || t2.join().is_err() { + return; + } else { + panic!("success"); + } }