From 272cbd198d88cc2036e8dc2c2aa4db207e93139f Mon Sep 17 00:00:00 2001
From: Bruce Lacey <Bruce@blacey.com>
Date: Wed, 9 Mar 2016 21:47:53 -0800
Subject: [PATCH] Exit with non-zero status when unit tests fail. 1.  Enhanced
 TestApp.py to return the unittest TestResult object to the calling method 2. 
 FreeCADTest.py now checks the TestResult object.  If all tests pass,
 FreeCADTest.py exits with a 0 however if any tests fail, then it exits with
 1.  FreeCADTest.py calls os._exit() with the exit code instead of sys.exit()
 because Base::Interpreter:SystemExitException intercepts sys.exit() calls and
 swallows the exit code so there is no way to propogate the non-zero code from
 the python test cases through the interpreter to the shell in the current
 call stack.

---
 src/App/FreeCADTest.py  | 5 +++--
 src/Mod/Test/TestApp.py | 4 ++--
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/App/FreeCADTest.py b/src/App/FreeCADTest.py
index 31759e10e..ac396a58f 100644
--- a/src/App/FreeCADTest.py
+++ b/src/App/FreeCADTest.py
@@ -32,9 +32,10 @@
 
 Log ("FreeCAD test running...\n\n")
 
-import TestApp;TestApp.TestText("TestApp.All")
+import TestApp, os
 
+testResult = TestApp.TestText("TestApp.All")
 
 Log ("FreeCAD test done\n")
 
-
+os._exit(0 if testResult.wasSuccessful() else 1)
diff --git a/src/Mod/Test/TestApp.py b/src/Mod/Test/TestApp.py
index accb90d84..ca23c56a8 100644
--- a/src/Mod/Test/TestApp.py
+++ b/src/Mod/Test/TestApp.py
@@ -63,7 +63,7 @@ def All():
 def TestText(s):
     s = unittest.defaultTestLoader.loadTestsFromName(s)
     r = unittest.TextTestRunner(stream=sys.stdout, verbosity=2)
-    r.run(s)
+    return r.run(s)
 
 
 def Test(s):
@@ -72,7 +72,7 @@ def Test(s):
 
 def testAll():
     r = unittest.TextTestRunner(stream=sys.stdout, verbosity=2)
-    r.run(All())
+    return r.run(All())
 
 
 def testUnit():