Skip to content

ci: add pytest-cov coverage threshold gate (#286) - #339

Merged
YaronZaki merged 1 commit into
Quantarq:mainfrom
tolulopedd26:feat/coverage-threshold
Jul 20, 2026
Merged

ci: add pytest-cov coverage threshold gate (#286)#339
YaronZaki merged 1 commit into
Quantarq:mainfrom
tolulopedd26:feat/coverage-threshold

Conversation

@tolulopedd26

Copy link
Copy Markdown
Contributor

Summary

Adds a coverage threshold gate to the CI pipeline using pytest-cov. Tests now fail if coverage drops below 80%.

Changes

  • Modified quantara/web_app/pytest.ini to add coverage options:
    • --cov=web_app — measure coverage of the web_app module
    • --cov-report=term-missing — show missing lines in terminal
    • --cov-report=xml:coverage.xml — generate XML report for CI
    • --cov-fail-under=80 — fail if coverage drops below 80%
  • Modified .github/workflows/ci.yml to upload coverage report as artifact

Testing

  • Coverage options validated in pytest.ini
  • CI workflow updated with artifact upload
  • No breaking changes to existing tests

Checklist

  • Code compiles without errors
  • No breaking changes
  • Coverage threshold set at 80% (adjustable)

Closes #286

@tolulopedd26
tolulopedd26 force-pushed the feat/coverage-threshold branch from 98e54e8 to 5e811e6 Compare July 20, 2026 15:36

@YaronZaki YaronZaki left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@YaronZaki
YaronZaki merged commit 5a04a8a into Quantarq:main Jul 20, 2026
9 checks passed
@grantfox-oss grantfox-oss Bot mentioned this pull request Jul 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Coverage threshold on PR

2 participants