From bfc45834c110f7f021daffefcc2290c9133d6139 Mon Sep 17 00:00:00 2001 From: Alex Crichton Date: Mon, 17 Aug 2015 12:58:19 -0700 Subject: [PATCH] test: Don't panic if some tests failed This commit removes the call to `panic!("Some tests failed")` at the end of all tests run when running with libtest. The panic is replaced with `std::process::exit` to have a nonzero error code, but this change both: 1. Makes the test runner no longer print out the extraneous panic message at the end of a failing test run that some tests failed. (this is already summarized in the output of the test run). 2. When running tests with `RUST_BACKTRACE` set it removes an extraneous backtrace from the output (only failing tests will have their backtraces in the output. --- src/libtest/lib.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/libtest/lib.rs b/src/libtest/lib.rs index 47c50f70a39..4de7e7c5866 100644 --- a/src/libtest/lib.rs +++ b/src/libtest/lib.rs @@ -250,7 +250,7 @@ pub fn test_main(args: &[String], tests: Vec ) { }; match run_tests_console(&opts, tests) { Ok(true) => {} - Ok(false) => panic!("Some tests failed"), + Ok(false) => std::process::exit(101), Err(e) => panic!("io error when running tests: {:?}", e), } }