Skip to content

Commit 99cb9cc

Browse files
committed
Auto merge of #72089 - Mark-Simulacrum:error-is-really-an-error, r=pietroalbini
Fail if I/O error occurs during testing First known case of this is in #72053 (comment) but may have happened before then. r? @pietroalbini
2 parents 3fe4dd2 + a5ba752 commit 99cb9cc

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

src/tools/compiletest/src/main.rs

+4-1
Original file line numberDiff line numberDiff line change
@@ -347,7 +347,10 @@ pub fn run_tests(config: Config) {
347347
Ok(true) => {}
348348
Ok(false) => panic!("Some tests failed"),
349349
Err(e) => {
350-
println!("I/O failure during tests: {:?}", e);
350+
// We don't know if tests passed or not, but if there was an error
351+
// during testing we don't want to just suceeed (we may not have
352+
// tested something), so fail.
353+
panic!("I/O failure during tests: {:?}", e);
351354
}
352355
}
353356
}

0 commit comments

Comments
 (0)