From cfbb11327027926855b8c9d5104f28b9a25fa72c Mon Sep 17 00:00:00 2001 From: Christian Boltz Date: Nov 12 2018 15:58:22 +0000 Subject: Make sure to test failures of all test runs ... instead of using a "last one wins" approach, which will hide errors in all tests except the last one. See also my comment in !213 for the initial report about this. --- diff --git a/bin/test_sudo.sh b/bin/test_sudo.sh index 338a364..af1345e 100755 --- a/bin/test_sudo.sh +++ b/bin/test_sudo.sh @@ -26,6 +26,7 @@ run_tests() { echo_FAILED fi echo + return $STATUS } echo_INFO "Testing virtual: physical" @@ -43,12 +44,14 @@ SUDO_ROLES=( ) popd > /dev/null +ALL_STATUS=0 + for _role in ${SUDO_ROLES[@]}; do _role=${_role##*/} role=${_role%%.*} echo_INFO "Testing role: $role" reset_sudo - run_tests + run_tests || ALL_STATUS=$? done -exit $STATUS +exit $ALL_STATUS