- Bug TW-1296

- TW-1296 make test/run_all exit with non-zero code if a test fail (thanks to
          Jakub Wilk).

ChangeLog

@@ -37,6 +37,8 @@ Bugs
 - TW-1282 incorrect URLs in man task-sync (thanks to Jeremiah Marks).
 - TW-1288 Added missing locking for task modifications (thanks to Kosta H,
           Ralph Bean, Adam Coddington).
+- TW-1296 make test/run_all exit with non-zero code if a test fail (thanks to
+          Jakub Wilk).
 - #1511 sync init crashes if client certification file is empty or invalid
         (thanks to Marton Suranyi).
 - #1508 Show command highlight configuration (thanks to Nicolas Appriou).

test/run_all.in

@@ -1,5 +1,6 @@
 #! /bin/sh
 
+rc=0
 if [ x"$1" = x"--verbose" ];
 then
   for i in ${TESTBLOB}
@@ -9,9 +10,10 @@ then
     while read LINE
     do
       echo "$LINE"
-    done < test.log
+    done < test.log || rc=1
     rm test.log
   done
+  exit $rc
 else
   date > all.log
 
@@ -37,7 +39,7 @@ else
       COUNT=`expr $COUNT + 1`
     fi
 
-    $i >> all.log 2>&1
+    $i >> all.log 2>&1 || rc=1
   done
 
   if [ $BAR -eq 1 ]; then
@@ -53,4 +55,5 @@ else
   printf "Fail:    %5d\n" `grep -c '^not' all.log`
   printf "Skipped: %5d\n" `grep -c '^skip' all.log`
   printf "Runtime: %5d seconds\n" $RUNTIME
+  exit $rc
 fi

test/test.cpp

@@ -28,6 +28,7 @@
 #include <iomanip>
 #include <string.h>
 #include <math.h>
+#include <stdlib.h>
 #include <test.h>
 
 ///////////////////////////////////////////////////////////////////////////////
@@ -84,6 +85,7 @@ UnitTest::~UnitTest ()
             << " skipped. "
             << std::setprecision (3) << percentPassed
             << "% passed.\n";
+  exit (_failed > 0);
 }
 
 ///////////////////////////////////////////////////////////////////////////////