From e75d0653c861239d92c1ad342217744d237dae7f Mon Sep 17 00:00:00 2001
From: wpbonelli <wbonelli@ucar.edu>
Date: Tue, 15 Oct 2024 19:05:12 -0400
Subject: [PATCH] ci(commit.yml): fix silently failing test step (#2335)

The CI test step was suppressing errors, I think because we ran coverage immediately after. Move coverage to a subsequent step. This reveals a few failing tests which are not addressed in this changeset.
---
 .github/workflows/commit.yml | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/.github/workflows/commit.yml b/.github/workflows/commit.yml
index b28e3af4d..6a2d8292d 100644
--- a/.github/workflows/commit.yml
+++ b/.github/workflows/commit.yml
@@ -181,10 +181,13 @@ jobs:
         working-directory: autotest
         run: |
           pytest -v -m="not example" -n=auto --cov=flopy --cov-append --cov-report=xml --durations=0 --keep-failed=.failed --dist loadfile
-          coverage report
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 
+      - name: Report coverage
+        working-directory: autotest
+        run: coverage report
+
       - name: Upload failed test outputs
         uses: actions/upload-artifact@v4
         if: failure()