rust-lang/rust
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Include message on tests that should panic

Browse Source
This commit is contained in:
Kenny Goodin
2019-09-24 14:20:52 -04:00
parent ddf43867a9
commit 821ff46878
2 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
src/libtest/lib.rs
View File

@@ -1546,6 +1546,7 @@ fn calc_result(desc: &TestDesc, task_result: Result<(), Box<dyn Any + Send>>) ->
} }
} }
} }
(&ShouldPanic::Yes, Ok(())) => TrFailedMsg("test did not panic as expected".to_string()),
_ if desc.allow_fail => TrAllowedFail, _ if desc.allow_fail => TrAllowedFail,
_ => TrFailed, _ => TrFailed,
} }

4
src/libtest/tests.rs
View File

@@ -2,7 +2,7 @@ use super::*;
use crate::test::{ use crate::test::{
filter_tests, parse_opts, run_test, DynTestFn, DynTestName, MetricMap, RunIgnored, filter_tests, parse_opts, run_test, DynTestFn, DynTestName, MetricMap, RunIgnored,
ShouldPanic, StaticTestName, TestDesc, TestDescAndFn, TestOpts, TrFailed, TrFailedMsg, ShouldPanic, StaticTestName, TestDesc, TestDescAndFn, TestOpts, TrFailedMsg,
TrIgnored, TrOk, TrIgnored, TrOk,
}; };
use std::sync::mpsc::channel; use std::sync::mpsc::channel;
@@ -167,7 +167,7 @@ fn test_should_panic_but_succeeds() {
let (tx, rx) = channel(); let (tx, rx) = channel();
run_test(&TestOpts::new(), false, desc, tx, Concurrent::No); run_test(&TestOpts::new(), false, desc, tx, Concurrent::No);
let (_, res, _, _) = rx.recv().unwrap(); let (_, res, _, _) = rx.recv().unwrap();
assert!(res == TrFailed); assert!(res == TrFailedMsg("test did not panic as expected".to_string()));
} }
fn report_time_test_template(report_time: bool) -> Option<TestExecTime> { fn report_time_test_template(report_time: bool) -> Option<TestExecTime> {