Skip to content

ci: configure pytest reporting for consumer tracking#136

Merged
sammorley-short merged 1 commit intomainfrom
sms/add-pytest-reporting
Mar 5, 2026
Merged

ci: configure pytest reporting for consumer tracking#136
sammorley-short merged 1 commit intomainfrom
sms/add-pytest-reporting

Conversation

@sammorley-short
Copy link
Copy Markdown
Contributor

@sammorley-short sammorley-short commented Mar 5, 2026

Configure pytest and mypy reporting in CI

Summary

Adds CI reporting artifacts for pytest and mypy to improve visibility into test results and code quality metrics. Reports are uploaded as GitHub Actions artifacts and retained for 14 days.

Changes

Pytest reporting & coverage (run_unit_tests job)

  • Run pytest with JSON report (pytest-json-report) and coverage (pytest-cov) enabled.
  • Upload pytest_report.json and htmlcov/ as artifacts per Python version matrix entry.

Mypy reporting (run_quality_checks job)

  • Generate an HTML report from mypy via --html-report.
  • Upload mypy-report/ as an artifact.

New dependencies (pyproject.toml)

  • pytest-cov ^5.0.0 — coverage reporting for pytest.
  • pytest-json-report ^1.5.0 — machine-readable JSON test reports.
  • lxml ^6.0.2 — required by mypy's HTML report generation.

Coverage configuration (.coveragerc)

  • Added .coveragerc to omit __init__.py files from coverage measurement.

Notes

  • All report upload steps use if: always() so artifacts are available even when tests or type-checking fail.
  • Artifact retention is set to 14 days.
  • Minor whitespace cleanup in pyproject.toml (removed extraneous blank lines).

@cla-bot cla-bot Bot added the cla-signed label Mar 5, 2026
@sammorley-short sammorley-short force-pushed the sms/add-pytest-reporting branch from f539531 to 78b60d5 Compare March 5, 2026 00:20
@sammorley-short sammorley-short merged commit 80b11bb into main Mar 5, 2026
6 checks passed
@sammorley-short sammorley-short deleted the sms/add-pytest-reporting branch March 5, 2026 23:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants