diff options
| author | Michael Goulet <michael@errs.io> | 2024-06-24 15:51:02 -0400 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-06-24 15:51:02 -0400 |
| commit | 46e43984d18db0febeb1e9b8e80feebc6839edba (patch) | |
| tree | dbe9b25af9eb777f9b7587fd5349c679146f89b9 /src | |
| parent | ed460d2eaa38846f3e555a722566b93daecf93e5 (diff) | |
| parent | 8ffb5f936a17190ad9aae442fe52c709521c5b29 (diff) | |
| download | rust-46e43984d18db0febeb1e9b8e80feebc6839edba.tar.gz rust-46e43984d18db0febeb1e9b8e80feebc6839edba.zip | |
Rollup merge of #126413 - matthiaskrgr:crshmsg, r=oli-obk
compiletest: make the crash test error message abit more informative r? ```@oli-obk```
Diffstat (limited to 'src')
| -rw-r--r-- | src/tools/compiletest/src/runtest.rs | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/src/tools/compiletest/src/runtest.rs b/src/tools/compiletest/src/runtest.rs index 72b57d91c23..1a4101e4de8 100644 --- a/src/tools/compiletest/src/runtest.rs +++ b/src/tools/compiletest/src/runtest.rs @@ -374,11 +374,12 @@ 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( - "test no longer crashes/triggers ICE! Please give it a mearningful name, \ + self.fatal(&format!( + "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 description \ + ensures that the corresponding ticket is auto-closed upon merge." + )); } } |
