[U-Boot] [PATCH] test/run: Exit on error from tests

[Tom Rini]

Currently, result isn't saved outside of the scope of run_test, so we
always exit on error here.  An easier way to deal with having test/run
show failures is to exit on failure.

Cc: Simon Glass <sjg at chromium.org>
Fixes: 2f52018c3cbb ("test/run: Report and return failure")
Signed-off-by: Tom Rini <trini at konsulko.com>
---
Automation is only as good as the humans behind it.  If you forget, like
I did, that you finally fixed Jenkins to email you every time, that lack
of emails about my non-travis jobs failing wasn't "no problems", it was
"You forgot to migrate all the Docker stuff to the new build server".
So, I'm going to apply this ASAP as I would not have let in the
u-boot-dm PR, had my setup been functioning as expected.
---
 test/run | 18 ++++--------------
 1 file changed, 4 insertions(+), 14 deletions(-)

diff --git a/test/run b/test/run
index caee4f83f2c1..47dfed7bdca7 100755
--- a/test/run
+++ b/test/run
@@ -1,24 +1,14 @@
 #!/bin/bash
 
-run_test() {
-	$@
-	[ $? -ne 0 ] && result=$((result+1))
-	echo "result $result"
-}
+# Exit on error
+set +e
 
 # Run all tests that the standard sandbox build can support
-run_test ./test/py/test.py --bd sandbox --build
+./test/py/test.py --bd sandbox --build
 
 # Run tests which require sandbox_spl
-run_test ./test/py/test.py --bd sandbox_spl --build -k \
+./test/py/test.py --bd sandbox_spl --build -k \
 		test/py/tests/test_ofplatdata.py
 
 # Run tests for the flat DT version of sandbox
 ./test/py/test.py --bd sandbox_flattree --build
-
-if [ $result == 0 ]; then
-	echo "Tests passed!"
-else
-	echo "Tests FAILED"
-	exit 1
-fi
-- 
2.7.4