better github action workflows (should detect failure also under maco…

…s-latest)
fdaPDE · Dec 6, 2023 · dd7d768 · dd7d768
1 parent d999940
commit dd7d768
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 4 deletions.
diff --git a/.github/workflows/test-linux-clang.yml b/.github/workflows/test-linux-clang.yml
@@ -19,3 +19,11 @@ jobs:
         export Eigen3_DIR=/usr/share/cmake/Modules # reguired to let eigen3 work with CMake in alpine
         chmod +x run_tests.sh
         ./run_tests.sh --compiler clang
+        echo "TEST_RESULT=$(echo $?)" >> $GITHUB_ENV
+    - name: check_tests_passed
+      run: |
+        if [ "$TEST_RESULT" -eq "0" ]; then
+           exit 0
+        else
+           exit 1
+        fi
diff --git a/.github/workflows/test-linux-gcc.yml b/.github/workflows/test-linux-gcc.yml
@@ -19,3 +19,11 @@ jobs:
         export Eigen3_DIR=/usr/share/cmake/Modules # reguired to let eigen3 work with CMake in alpine
         chmod +x run_tests.sh
         ./run_tests.sh --compiler gcc
+        echo "TEST_RESULT=$(echo $?)" >> $GITHUB_ENV
+    - name: check_tests_passed
+      run: |
+        if [ "$env.TEST_RESULT" -eq "0" ]; then
+           exit 0
+        else
+           exit 1
+        fi
diff --git a/.github/workflows/test-macos-clang.yml b/.github/workflows/test-macos-clang.yml
@@ -15,11 +15,15 @@ jobs:
       run: |
         brew install eigen
     - name: run_tests
-      id: test_result
       run: |
         cd test/
         chmod +x run_tests.sh
         ./run_tests.sh --compiler clang
-    - name: check tests passed
-        if: steps.test_result.outputs.result != 0
-        run: exit 1
+        echo "TEST_RESULT=$(echo $?)" >> $GITHUB_ENV
+    - name: check_tests_passed
+      run: |
+        if [ "$TEST_RESULT" -eq "0" ]; then
+           exit 0
+        else
+           exit 1
+        fi