From 8328aff1d9e40b5fe791b0296afe95db45c1a28f Mon Sep 17 00:00:00 2001 From: Abtin Keshavarzian Date: Mon, 30 Mar 2020 13:32:39 -0700 Subject: [PATCH] [toranj] change start.sh to remove variable check to retry failed test (#4761) This commit changes the `start.sh` script to remove the check for the environment variable (which is specific to travis) before allowing retry of failed tests. This should help with robustness of the tests on GitHub Actions. --- tests/toranj/start.sh | 16 +++++++--------- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/tests/toranj/start.sh b/tests/toranj/start.sh index a9e484cbb..48e53694c 100755 --- a/tests/toranj/start.sh +++ b/tests/toranj/start.sh @@ -58,15 +58,13 @@ run() { return fi - # On Travis, we allow a failed test to be retried up to 3 attempts. - if [ "$BUILD_TARGET" = "toranj-test-framework" ]; then - if [ "$counter" -lt 2 ]; then - counter=$((counter+1)) - echo Attempt $counter running "$1" failed. Trying again. - cleanup - sleep 10 - continue - fi + # We allow a failed test to be retried up to 3 attempts. + if [ "$counter" -lt 2 ]; then + counter=$((counter+1)) + echo Attempt $counter running "$1" failed. Trying again. + cleanup + sleep 10 + continue fi echo " *** TEST FAILED"