iox-##692 Fix bug in build script

Signed-off-by: Ibrahim Kuru <ibrahim.kuru@apex.ai>
eclipse-iceoryx · Jan 12, 2023 · f40ff85 · f40ff85
1 parent 33310a3
commit f40ff85
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 6 deletions.
diff --git a/.github/workflows/build-test.yml b/.github/workflows/build-test.yml
@@ -123,7 +123,7 @@ jobs:
       - name: Run Thread Sanitizer
         run: ./tools/ci/build-test-ubuntu-with-sanitizers.sh clang tsan
 
-  build-test-macos-with-sanitizers:
+  build-test-macos-with-address-sanitizer:
     # prevent stuck jobs consuming runners for 3 hours
     timeout-minutes: 180
     runs-on: macos-12

diff --git a/tools/run_tests.sh b/tools/run_tests.sh
@@ -121,7 +121,14 @@ fi
 set_sanitizer_options
 
 make_c() {
-    make $1 || test_failed=1; test "$CONTINUE_ON_ERROR" != "true" && exit 1
+    make $1 || test_failed=1
+    if [ "$test_failed" == "1" ]; then
+        echo "------->>>>> Test: $1 failed!!!!"
+        if [ "$CONTINUE_ON_ERROR" != "true" ]; then
+            echo "Exiting immediately (CONTINUE_ON_ERROR=false)"
+            exit 1
+        fi
+    fi
 }
 
 execute_test() {
@@ -157,10 +164,6 @@ execute_test() {
         ;;
     esac
 
-    if [ "$test_failed" != "0" ]; then
-        echo "Tests Failed!"
-    fi
-
     return $((test_failed))
 }