ci: Capture coverage information even when tests fail

Ensure that code coverage information is calculated and stored even when tests fail. Signed-off-by: Sam Friedman <[email protected]>
golioth · Jan 15, 2025 · c0ddbc5 · c0ddbc5
1 parent c2ac22d
commit c0ddbc5
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/.github/workflows/hil-integration-linux.yml b/.github/workflows/hil-integration-linux.yml
@@ -104,6 +104,9 @@ jobs:
             --alluredir=allure-reports                  \
             --platform linux
 
+      - name: Capture coverage
+        if: success() || failure()
+        run: |
           gcovr                                                         \
             --gcov-ignore-parse-errors=negative_hits.warn_once_per_file \
             --merge-mode-functions=separate                             \
@@ -112,6 +115,7 @@ jobs:
 
       - name: Upload coverage
         uses: actions/upload-artifact@v4
+        if: success() || failure()
         with:
           name: linux-hil-test-coverage-${{ matrix.test }}
           path: |

diff --git a/.github/workflows/hil-integration-nsim.yml b/.github/workflows/hil-integration-nsim.yml
@@ -135,6 +135,7 @@ jobs:
             --allure-board=${{ inputs.platform }}
 
       - name: Capture coverage
+        if: success() || failure()
         run: |
           gcovr -r modules/lib/golioth-firmware-sdk                      \
             --gcov-ignore-parse-errors=negative_hits.warn_once_per_file  \
@@ -143,7 +144,7 @@ jobs:
             hil-out/${{ matrix.test }}
 
       - name: Upload test coverage artifacts
-        if: always()
+        if: success() || failure()
         uses: ./modules/lib/golioth-firmware-sdk/.github/actions/safe-upload-artifacts
         with:
           secrets-json: ${{ toJson(secrets) }}