Skip to content

Commit

Permalink
compiletest: make the crash test error message abit more informative
Browse files Browse the repository at this point in the history
  • Loading branch information
matthiaskrgr committed Jun 13, 2024
1 parent 9fdbfe1 commit 4c3c3fd
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions src/tools/compiletest/src/runtest.rs
Original file line number Diff line number Diff line change
Expand Up @@ -375,9 +375,10 @@ impl<'test> TestCx<'test> {
// if a test does not crash, consider it an error
if proc_res.status.success() || matches!(proc_res.status.code(), Some(1 | 0)) {
self.fatal(&format!(
"test no longer crashes/triggers ICE! Please give it a mearningful name, \
"Crashtest no longer crashes/triggers ICE, horray! Please give it a meaningful name, \
add a doc-comment to the start of the test explaining why it exists and \
move it to tests/ui or wherever you see fit."
move it to tests/ui or wherever you see fit. Adding 'Fixes #<issueNr>' to your PR body \
and commit message ensures that the correspondig ticket is auto-closed upon merge."
));
}
}
Expand Down

0 comments on commit 4c3c3fd

Please sign in to comment.