move aborting step further back

mmuetzel · Jan 25, 2024 · c373668 · c373668
1 parent 7117a3c
commit c373668
Showing 1 changed file with 37 additions and 37 deletions.
diff --git a/.github/workflows/make.yaml b/.github/workflows/make.yaml
@@ -148,13 +148,6 @@ jobs:
         timeout-minutes: 5
         run: cat ./.build/test/fntests.log
 
-      - name: analyze test suite results
-        # Make sure the test summary lists 0 "FAIL"s and no "REGRESSION"
-        run:  |
-          [ -n "$(grep -e "FAIL\s*0" ./test-suite.log)" ] || exit 1
-          [ -z "$(grep -e "REGRESSION" ./test-suite.log)" ] || exit 1
-          echo No unknown failing tests.
-
       - name: test history file creation
         # see bug #62365
         # Pipe to an interactive session to trigger appending the command to
@@ -176,6 +169,13 @@ jobs:
           LD_LIBRARY_PATH="/usr/local/lib/octave/$(octave --eval "disp(version())")" \
             ./embedded
 
+      - name: analyze test suite results
+        # Make sure the test summary lists 0 "FAIL"s and no "REGRESSION"
+        run:  |
+          [ -n "$(grep -e "FAIL\s*0" ./test-suite.log)" ] || exit 1
+          [ -z "$(grep -e "REGRESSION" ./test-suite.log)" ] || exit 1
+          echo No unknown failing tests.
+
 
   alpine:
     runs-on: ubuntu-latest
@@ -373,15 +373,6 @@ jobs:
         timeout-minutes: 5
         run: cat ./.build/test/fntests.log
 
-      - name: analyze test suite results
-        # Make sure the test summary lists 0 "FAIL"s and no "REGRESSION"
-        run:  |
-          [ -n "$(grep -e "FAIL\s*0" ./test-suite.log)" ] \
-            || echo "::warning::At least one test failed"
-          [ -z "$(grep -e "REGRESSION" ./test-suite.log)" ] \
-            || echo "::warning::At least one regression in test suite"
-          echo Finished analyzing test suite results.
-
       - name: test history file creation
         # see bug #62365
         # Pipe to an interactive session to trigger appending the command to
@@ -400,6 +391,15 @@ jobs:
           mkoctfile --link-stand-alone embedded.cc -o embedded
           LD_LIBRARY_PATH="/usr/local/lib/octave/$(octave --eval "disp(version())")" ./embedded
 
+      - name: analyze test suite results
+        # Make sure the test summary lists 0 "FAIL"s and no "REGRESSION"
+        run:  |
+          [ -n "$(grep -e "FAIL\s*0" ./test-suite.log)" ] \
+            || echo "::warning::At least one test failed"
+          [ -z "$(grep -e "REGRESSION" ./test-suite.log)" ] \
+            || echo "::warning::At least one regression in test suite"
+          echo Finished analyzing test suite results.
+
 
   macos:
     runs-on: ${{ matrix.os }}
@@ -593,13 +593,6 @@ jobs:
         timeout-minutes: 5
         run: cat ./.build/test/fntests.log
 
-      - name: analyze test suite results
-        # Make sure the test summary lists 0 "FAIL"s and no "REGRESSION"
-        run: |
-          [ -n "$(grep -e "FAIL\s*0" ./test-suite.log)" ] || exit 1
-          [ -z "$(grep -e "REGRESSION" ./test-suite.log)" ] || exit 1
-          echo No unknown failing tests.
-
       - name: test history file creation
         # see bug #62365
         # Pipe to an interactive session to trigger appending the command to
@@ -622,6 +615,13 @@ jobs:
             mkoctfile --link-stand-alone embedded.cc -o embedded
           ./embedded
 
+      - name: analyze test suite results
+        # Make sure the test summary lists 0 "FAIL"s and no "REGRESSION"
+        run: |
+          [ -n "$(grep -e "FAIL\s*0" ./test-suite.log)" ] || exit 1
+          [ -z "$(grep -e "REGRESSION" ./test-suite.log)" ] || exit 1
+          echo No unknown failing tests.
+
 
   mingw:
     runs-on: ${{ matrix.os }}
@@ -848,13 +848,6 @@ jobs:
         timeout-minutes: 5
         run: cat ./.build/test/fntests.log
 
-      - name: analyze test suite results
-        # Make sure the test summary lists 0 "FAIL"s and no "REGRESSION"
-        run: |
-          [ -n "$(grep -e "FAIL\s*0" ./test-suite.log)" ] || echo "::warning::At least one test failed"
-          [ -z "$(grep -e "REGRESSION" ./test-suite.log)" ] || echo "::warning::At least one regression in test suite"
-          echo Finished analyzing test suite results.
-
       - name: test history file creation
         # see bug #62365
         # Pipe to an interactive session to trigger appending the command to
@@ -877,6 +870,13 @@ jobs:
           mkoctfile --link-stand-alone embedded.cc -o embedded
           ./embedded.exe
 
+      - name: analyze test suite results
+        # Make sure the test summary lists 0 "FAIL"s and no "REGRESSION"
+        run: |
+          [ -n "$(grep -e "FAIL\s*0" ./test-suite.log)" ] || echo "::warning::At least one test failed"
+          [ -z "$(grep -e "REGRESSION" ./test-suite.log)" ] || echo "::warning::At least one regression in test suite"
+          echo Finished analyzing test suite results.
+
       - name: compress build artifact
         if: matrix.msystem == 'MINGW64'
         continue-on-error: true
@@ -1108,13 +1108,6 @@ jobs:
         timeout-minutes: 5
         run: cat ./.build/test/fntests.log
 
-      - name: analyze test suite results
-        # Make sure the test summary lists 0 "FAIL"s and no "REGRESSION"
-        run: |
-          [ -n "$(grep -e "FAIL\s*0" ./test-suite.log)" ] || echo "::warning::At least one test failed"
-          [ -z "$(grep -e "REGRESSION" ./test-suite.log)" ] || echo "::warning::At least one regression in test suite"
-          echo Finished analyzing test suite results.
-
       - name: test history file creation
         # see bug #62365
         # Pipe to an interactive session to trigger appending the command to
@@ -1133,3 +1126,10 @@ jobs:
           cd examples/code
           mkoctfile --link-stand-alone embedded.cc -o embedded
           ./embedded.exe
+
+      - name: analyze test suite results
+        # Make sure the test summary lists 0 "FAIL"s and no "REGRESSION"
+        run: |
+          [ -n "$(grep -e "FAIL\s*0" ./test-suite.log)" ] || echo "::warning::At least one test failed"
+          [ -z "$(grep -e "REGRESSION" ./test-suite.log)" ] || echo "::warning::At least one regression in test suite"
+          echo Finished analyzing test suite results.