user0/rust
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Rollup merge of #61375 - varkor:panic-contains-string-lowercase, r=Centril

Browse source 
Make "panic did not include expected string" message consistent

Note messages are typically lowercase.
This commit is contained in:
Mazdak Farrokhzad 2019-06-01 06:50:05 +02:00 • committed by GitHub
commit d0a0ba42d7
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
src/libtest/lib.rs
View file

@ -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))
}
}
}
@ -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"),