From 311a6bf0a9d4997f7f82e788125d568adf51ea78 Mon Sep 17 00:00:00 2001 From: MDW Date: Fri, 17 Jan 2025 00:31:18 +0100 Subject: [PATCH] Fix ci: travis - Better test if phpunit completed # Fix ci: travis - Better test if phpunit completed --- .travis.yml | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/.travis.yml b/.travis.yml index 7129c93d9f1..0020c7b067b 100644 --- a/.travis.yml +++ b/.travis.yml @@ -528,12 +528,11 @@ script: - | echo "Unit testing" - # Ensure we catch errors with -e. Set this to +e instead of -e if you want to go to the end to see dolibarr.log file. - set -e - phpunit -d memory_limit=-1 -c test/phpunit/phpunittest.xml test/phpunit/AllTests.php - phpunitresult=$? + # Execute phpunit, check its exit status and that the phpunit output shows a test summary + phpunit -d memory_limit=-1 -c test/phpunit/phpunittest.xml test/phpunit/AllTests.php | tee /dev/tty | grep -qE "(OK .*[0-9]+ tests.*[0-9]+ assertions|Tests: [0-9]+)" ; phpunitresult=$((PIPESTATUS[0]?PIPESTATUS[0]:PIPESTATUS[2])) echo "Phpunit return code = $phpunitresult" - set +e + # Comment next line if you want to go to the end to see the dolibarr.log file. + [ $phpunitresult == 0 ] || exit $phpunitresult echo after_script: