Skip to content

Commit

Permalink
Rollup merge of rust-lang#61375 - varkor:panic-contains-string-lowerc…
Browse files Browse the repository at this point in the history
…ase, r=Centril

Make "panic did not include expected string" message consistent

Note messages are typically lowercase.
  • Loading branch information
Centril authored Jun 1, 2019
2 parents 7d3a0dd + c3febc0 commit d0a0ba4
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/libtest/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1536,7 +1536,7 @@ fn calc_result(desc: &TestDesc, task_result: Result<(), Box<dyn Any + Send>>) ->
if desc.allow_fail {
TrAllowedFail
} else {
TrFailedMsg(format!("Panic did not include expected string '{}'", msg))
TrFailedMsg(format!("panic did not include expected string '{}'", msg))
}
}
}
Expand Down Expand Up @@ -1890,7 +1890,7 @@ mod tests {
panic!("an error message");
}
let expected = "foobar";
let failed_msg = "Panic did not include expected string";
let failed_msg = "panic did not include expected string";
let desc = TestDescAndFn {
desc: TestDesc {
name: StaticTestName("whatever"),
Expand Down

0 comments on commit d0a0ba4

Please sign in to comment.