Skip to content

Commit

Permalink
Auto merge of rust-lang#72089 - Mark-Simulacrum:error-is-really-an-er…
Browse files Browse the repository at this point in the history
…ror, r=pietroalbini

Fail if I/O error occurs during testing

First known case of this is in rust-lang#72053 (comment) but may have happened before then.

r? @pietroalbini
  • Loading branch information
bors committed May 11, 2020
2 parents 3fe4dd2 + a5ba752 commit 99cb9cc
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion src/tools/compiletest/src/main.rs
Original file line number Diff line number Diff line change
Expand Up @@ -347,7 +347,10 @@ pub fn run_tests(config: Config) {
Ok(true) => {}
Ok(false) => panic!("Some tests failed"),
Err(e) => {
println!("I/O failure during tests: {:?}", e);
// We don't know if tests passed or not, but if there was an error
// during testing we don't want to just suceeed (we may not have
// tested something), so fail.
panic!("I/O failure during tests: {:?}", e);
}
}
}
Expand Down

0 comments on commit 99cb9cc

Please sign in to comment.