Raise an exception instead of exiting with a non-zero return code in the unit tests (#1983)

Jürgen Knödlseder
1 parent 9502ac30
Showing 4 changed files with 14 additions and 6 deletions
test/test_ctbin.py
test/test_examples.py.in
test/test_python_cscripts.py
test/test_python_ctools.py
@@ -87,7 +87,8 @@ class Test(test):
         # Load counts cube and check content.
         evt = gammalib.GCTAEventCube('cntmap_cmd1.fits')
-        self._check_cube(evt, 5542)
+        #JKself._check_cube(evt, 5542)
+        self._check_cube(evt, 5543)
         # Setup ctbin command
         cmd = ctbin+' inobs="events_that_do_not_exist.fits"'+ \
@@ -487,7 +487,8 @@ class Test(gammalib.GPythonTestSuite):
         args = '-p example_show_pointings.eps %s/obs_unbinned.xml' % self._datadir
         # Execute script
-        self._execute_python('show_pointings', args=args)
+        #JKself._execute_python('show_pointings', args=args)
+        self._execute_python('show_pointing', args=args)
         # Return
         return
@@ -533,4 +534,6 @@ if __name__ == &#39;__main__&#39;:
         rc = 1
     # Exit with return code
-    sys.exit(rc)
+    if not success:
+        raise RuntimeError('At least one error occured during the test.')
+    #sys.exit(rc)
@@ -185,7 +185,9 @@ def test(installed=False, debug=False):
         os.system('rm -rf %s' % (path))
     # Exit with return code
-    sys.exit(rc)
+    if not success:
+        raise RuntimeError('At least one error occured during the test.')
+    #sys.exit(rc)
 # ======================== #
@@ -143,8 +143,10 @@ def test(installed=False):
     if installed:
         os.system('rm -rf %s' % (path))
-    # Exit with return code
-    sys.exit(rc)
+    # Throw an exception in case of failure
+    if not success:
+        raise RuntimeError('At least one error occured during the test.')
+    #sys.exit(rc)
 # ======================== #