make sure we exit based on test failure rather than always exiting with 0

1 files changed, 9 insertions, 3 deletions

diff --git a/shell/hush_test/run-all b/shell/hush_test/run-all
index b79af2f67..b5db79f6c 100755
--- a/shell/hush_test/run-all
+++ b/shell/hush_test/run-all
@@ -25,6 +25,7 @@ do_test()
     test -d "$1" || return 0
 #   echo Running tests in directory "$1"
     (
+    tret=0
     cd "$1" || { echo "cannot cd $1!"; exit 1; }
     for x in run-*; do
 	test -f "$x" || continue
@@ -48,26 +49,31 @@ do_test()
 	{
 	    "$THIS_SH" "./$x" >"$name.xx" 2>&1
 	    diff -u "$name.xx" "$name.right" >"../$1-$x.fail" && rm -f "$name.xx" "../$1-$x.fail"
-	} && echo "$1/$x: ok" || echo "$1/$x: fail"
+	} && echo "$1/$x: ok" || { echo "$1/$x: fail"; ((tret+=1)); }
     done
+    exit ${tret}
     )
 }
 
 # Main part of this script
 # Usage: run-all [directories]
 
+ret=0
+
 if [ $# -lt 1 ]; then
     # All sub directories
     modules=`ls -d hush-*`
 
     for module in $modules; do
-	do_test $module
+	do_test $module || ret=1
     done
 else
     while [ $# -ge 1 ]; do
 	if [ -d $1 ]; then
-	    do_test $1
+	    do_test $1 || ret=1
 	fi
 	shift
     done
 fi
+
+exit ${ret}