exchange

commit 2805413486701578c642a24faa1a0c698f8f3bd5
parent 4b3c21a36e5220be99fd990e005dc452ef4dbbe0
Author: Christian Grothoff <christian@grothoff.org>
Date:   Mon,  1 Apr 2024 12:44:10 +0200

actually fail tests on error

Diffstat:
M
contrib/ci/jobs/2-test/test.sh
 | 
1
+
M
src/templating/dotest.sh
 | 
16
+++++++++++-----
M
src/templating/run-original-tests.sh
 | 
9
---------

3 files changed, 12 insertions(+), 14 deletions(-)
diff --git a/contrib/ci/jobs/2-test/test.sh b/contrib/ci/jobs/2-test/test.sh
@@ -14,6 +14,7 @@ make clean
 make -j$(( $nump / 2 ))
 cd src/templating/
 ./run-original-tests.sh
+make clean
 cd -
 make -j$(( $nump / 2 ))
 make install
diff --git a/src/templating/dotest.sh b/src/templating/dotest.sh
@@ -1,20 +1,26 @@
 #!/bin/sh
 
+# Exit, with error message (hard failure)
+exit_fail() {
+    echo " FAIL: " "$@" >&2
+    exit 1
+}
+
 mustach=${mustach:-../mustach}
 echo starting test
 if test "$NOVALGRIND" = 1
 then
-	$mustach "$@" > resu.last
+	$mustach "$@" > resu.last || exit_fail "ERROR! mustach command failed ($?)!"
 else
-	valgrind $mustach "$@" > resu.last 2> vg.last
+	valgrind $mustach "$@" > resu.last 2> vg.last || exit_fail "ERROR! mustach command failed ($?)!"
 	sed -i 's:^==[0-9]*== ::' vg.last
-	awk '/^ *total heap usage: .* allocs, .* frees,.*/{if($$4-$$6)exit(1)}' vg.last || echo "ERROR! Alloc/Free issue"
+	awk '/^ *total heap usage: .* allocs, .* frees,.*/{if($$4-$$6)exit(1)}' vg.last || exit_fail "ERROR! Alloc/Free issue"
 fi
 if diff -w resu.ref resu.last
 then
 	echo "result ok"
 else
-	echo "ERROR! Result differs"
+	exit_fail "ERROR! Result differs"
 fi
 echo
-
+exit 0
diff --git a/src/templating/run-original-tests.sh b/src/templating/run-original-tests.sh
@@ -4,15 +4,6 @@ set -eux
 
 export CFLAGS="-g"
 
-function build()
-{
-    make clean
-    make
-}
-
-# Install rebuild-on-exit handler (except for kill -9)
-trap build EXIT
-
 echo "Ensuring clean state on entry to upstream tests ..."
 make clean