Fix the test_valgrind.sh script running tests under valgrind

The current script just checks if there is the
"ERROR SUMMARY: 0 errors from 0 contexts"
line in the output of a test.

It turned out that there can be many lines starting with
"ERROR SUMMARY:" in the output of one test - some with
errors and some without, so the current approach is wrong.
We have to check if there are no other lines starting with
"ERROR SUMMARY:" than this:
"ERROR SUMMARY: 0 errors from 0 contexts" instead.

Signed-off-by: Lukasz Dorau <[email protected]>

Author: ldorau

test/test_valgrind.sh

@@ -56,16 +56,20 @@ hwloc-gather-cpuid ./cpuid
 
 echo "Running: \"valgrind $OPTION\" for the following tests:"
 
-for tf in $(ls -1 ./test/umf_test-*); do
-	[ ! -x $tf ] && continue
-	echo -n "$tf "
-	LOG=${tf}.log
-	HWLOC_CPUID_PATH=./cpuid valgrind $OPTION $tf >$LOG 2>&1 || echo -n "(valgrind failed) "
-	if grep -q -e "ERROR SUMMARY: 0 errors from 0 contexts" $LOG; then
+for test in $(ls -1 ./test/umf_test-*); do
+	[ ! -x $test ] && continue
+	echo -n "$test "
+	LOG=${test}.log
+	ERR=${test}.err
+	HWLOC_CPUID_PATH=./cpuid valgrind $OPTION $test >$LOG 2>&1 || echo -n "(valgrind failed) "
+	# grep for "ERROR SUMMARY" with errors (there can be many lines with "ERROR SUMMARY")
+	grep -e "ERROR SUMMARY:" $LOG | grep -v -e "ERROR SUMMARY: 0 errors from 0 contexts" > $ERR || true
+	if [ $(cat $ERR | wc -l) -eq 0 ]; then
 		echo "- OK"
-		rm $LOG
+		rm -f $LOG $ERR
 	else
-		echo "- FAILED! : $(grep -e "ERROR SUMMARY:" $LOG | cut -d' ' -f2-)"
+		echo "- FAILED!"
+		cat $ERR | cut -d' ' -f2-
 		FAIL=1
 	fi || true
 done