svn commit: [25858] trunk/busybox/shell/hush_test

[vapier at busybox.net]

Author: vapier
Date: 2009-03-28 15:43:47 +0000 (Sat, 28 Mar 2009)
New Revision: 25858

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

Modified:
   trunk/busybox/shell/hush_test/run-all


Changeset:
Modified: trunk/busybox/shell/hush_test/run-all
===================================================================
--- trunk/busybox/shell/hush_test/run-all	2009-03-28 13:59:26 UTC (rev 25857)
+++ trunk/busybox/shell/hush_test/run-all	2009-03-28 15:43:47 UTC (rev 25858)
@@ -25,6 +25,7 @@
     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 @@
 	{
 	    "$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}