Skip to content

Commit

Permalink
Make "panic did not include expected string" message consistent
Browse files Browse the repository at this point in the history
Note messages are typically lowercase.
  • Loading branch information
varkor committed May 30, 2019
1 parent c28084a commit c3febc0
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 c3febc0

Please sign in to comment.