From fda196769a2aad38d84c79a21e1d7d692364c502 Mon Sep 17 00:00:00 2001
From: Brian Anderson <banderson@mozilla.com>
Date: Sun, 11 Sep 2011 21:13:59 -0700
Subject: [PATCH] Actually fail the process when compiletest has a test failure

We previously failed by coincidence because of memory leaks.
---
 src/test/compiletest/compiletest.rs | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/test/compiletest/compiletest.rs b/src/test/compiletest/compiletest.rs
index c931369cfc7..c89b851cdb5 100644
--- a/src/test/compiletest/compiletest.rs
+++ b/src/test/compiletest/compiletest.rs
@@ -110,8 +110,9 @@ fn run_tests(config: &config) {
     let opts = test_opts(config);
     let cx = {config: config, procsrv: procsrv::mk()};
     let tests = make_tests(cx);
-    test::run_tests_console_(opts, tests.tests, tests.to_task);
+    let res = test::run_tests_console_(opts, tests.tests, tests.to_task);
     procsrv::close(cx.procsrv);
+    if !res { fail "Some tests failed"; }
 }
 
 fn test_opts(config: &config) -> test::test_opts {