From 823e3d51d0d49ae80f28b68096461bdb24575390 Mon Sep 17 00:00:00 2001
From: Daniel Shahaf <d.s@daniel.shahaf.name>
Date: Sat, 7 Nov 2015 17:45:55 -0500
Subject: [PATCH] Test: Make 'make test' exit non-zero if tests failed

---
 test/problems | 3 +--
 test/run_all  | 7 ++-----
 2 files changed, 3 insertions(+), 7 deletions(-)

diff --git a/test/problems b/test/problems
index 35f4c8bbf..677a9761f 100755
--- a/test/problems
+++ b/test/problems
@@ -129,5 +129,4 @@ if __name__ == "__main__":
         print_category(expected)
 
     # If we encoutered any failures, return non-zero code
-    if sum(errors.values()):
-        sys.exit(1)
+    sys.exit(1 if error_int or unexpected_int else 0)
diff --git a/test/run_all b/test/run_all
index 3c5914c71..038b414ef 100755
--- a/test/run_all
+++ b/test/run_all
@@ -198,11 +198,8 @@ def main():
     runner = TestRunner()
     runner.start()
 
-    # If we're producing summary report, propagate the return code
-    if not cmd_args.verbose:
-        return runner.show_report()
-    else:
-        return 0
+    # Propagate the return code
+    return runner.show_report()
 
 
 if __name__ == "__main__":