Skip to content

Commit fda1967

Browse files
committed
Actually fail the process when compiletest has a test failure
We previously failed by coincidence because of memory leaks.
1 parent 69d6e0d commit fda1967

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

src/test/compiletest/compiletest.rs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -110,8 +110,9 @@ fn run_tests(config: &config) {
110110
let opts = test_opts(config);
111111
let cx = {config: config, procsrv: procsrv::mk()};
112112
let tests = make_tests(cx);
113-
test::run_tests_console_(opts, tests.tests, tests.to_task);
113+
let res = test::run_tests_console_(opts, tests.tests, tests.to_task);
114114
procsrv::close(cx.procsrv);
115+
if !res { fail "Some tests failed"; }
115116
}
116117

117118
fn test_opts(config: &config) -> test::test_opts {

0 commit comments

Comments
 (0)