about summary refs log tree commit diff
path: root/src/libtest
diff options
context:
space:
mode:
authorAlex Crichton <alex@alexcrichton.com>2015-08-17 12:58:19 -0700
committerAlex Crichton <alex@alexcrichton.com>2015-08-17 12:58:19 -0700
commitbfc45834c110f7f021daffefcc2290c9133d6139 (patch)
treef118b4e34011fd02304017a269f122603623ddd4 /src/libtest
parenta49d9bab1e79d44321e6a8e1a8cb81a4ffa6bad9 (diff)
downloadrust-bfc45834c110f7f021daffefcc2290c9133d6139.tar.gz
rust-bfc45834c110f7f021daffefcc2290c9133d6139.zip
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.
Diffstat (limited to 'src/libtest')
-rw-r--r--src/libtest/lib.rs2
1 files changed, 1 insertions, 1 deletions
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<TestDescAndFn> ) {
         };
     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),
     }
 }