Fix CI coverage reporting and cov config

Update GitHub Actions and tox/pyproject coverage settings so coverage output is picked up reliably. Read the tox output from tox-output.log and point the codecov action at ./.coverage.py312.xml. Stop excluding site-packages in pyproject (commented out) because it caused our installed package to be omitted from reports. Also tidy tox.ini coverage command formatting and ensure XML/term reports are emitted per envname.

Author: C-Achard

.github/workflows/testing-ci.yml

@@ -64,13 +64,13 @@ jobs:
           echo "## Coverage Summary" >> "$GITHUB_STEP_SUMMARY"
           echo '```text' >> "$GITHUB_STEP_SUMMARY"
           # Pull the coverage table from tox output:
-          sed -n '/^Name\s\+Stmts\s\+Miss\s\+Cover/,$p' tox-output.txt >> "$GITHUB_STEP_SUMMARY" || true
+          sed -n '/^Name\s\+Stmts\s\+Miss\s\+Cover/,$p' tox-output.log >> "$GITHUB_STEP_SUMMARY" || true
           echo '```' >> "$GITHUB_STEP_SUMMARY"
 
       - name: Upload coverage to Codecov
         if: matrix.os == 'ubuntu-latest' && matrix.python == '3.12' && github.event_name == 'pull_request' && (github.base_ref == 'main' || github.base_ref == 'master')
         uses: codecov/codecov-action@v5
         with:
           token: ${{ secrets.CODECOV_TOKEN }}
-          files: ./coverage.py312.xml
+          files: ./.coverage.py312.xml
           fail_ci_if_error: true

pyproject.toml

@@ -139,6 +139,7 @@ omit = [
 branch = true
 omit = [
   "*/__pycache__/*",
-  "*/site-packages/*",
+  # "*/site-packages/*",
+  # breaks CI coverage reporting as it excludes our own installed package
 ]
 source = [ "dlclivegui", "tests" ]

tox.ini

@@ -24,9 +24,9 @@ setenv =
 # Keep behavior aligned with your GitHub Actions job:
 commands =
     pytest -m "not hardware" --maxfail=1 --disable-warnings \
-        --cov=dlclivegui
-        --cov-report=xml:{toxinidir}/.coverage.{envname}.xml
-        --cov-report=term-missing
+        --cov=dlclivegui \
+        --cov-report=xml:{toxinidir}/.coverage.{envname}.xml \
+        --cov-report=term-missing \
         {posargs}
 
 # Let CI variables pass through (useful for debugging and some GUI/headless setups):