Skip to content

Commit

Permalink
Rollup merge of #111826 - ehuss:bootstrap-test-render-message, r=club…
Browse files Browse the repository at this point in the history
…by789

Render test messages from bootstrap

Bootstrap was not rendering messages from the test harness when a test failed. This can include messages like "test did not panic as expected". This fixes it by making sure those messages are printed on failure.

Fixes #111825
  • Loading branch information
matthiaskrgr committed May 22, 2023
2 parents b0415db + 59dff76 commit e2362d7
Showing 1 changed file with 7 additions and 2 deletions.
9 changes: 7 additions & 2 deletions src/bootstrap/render_tests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -168,9 +168,14 @@ impl<'a> Renderer<'a> {
if !self.failures.is_empty() {
println!("\nfailures:\n");
for failure in &self.failures {
if let Some(stdout) = &failure.stdout {
if failure.stdout.is_some() || failure.message.is_some() {
println!("---- {} stdout ----", failure.name);
println!("{stdout}");
if let Some(stdout) = &failure.stdout {
println!("{stdout}");
}
if let Some(message) = &failure.message {
println!("note: {message}");
}
}
}

Expand Down

0 comments on commit e2362d7

Please sign in to comment.