Skip to content

infra: tox -e coverage-check - #407

Merged
speller26 merged 1 commit into
mainfrom
coverage
Sep 19, 2026
Merged

speller26 merged 1 commit into
mainfrom
coverage

Conversation

@speller26

Copy link
Copy Markdown
Member

Added a coverage-check testenv to tox.ini, so tox will now fail if coverage falls below 100%

Issue #, if available:

Description of changes:

Testing done:

Merge Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.

General

Tests

  • I have added tests that prove my fix is effective or that my feature works (if appropriate)
  • I have checked that my tests are not configured for a specific region or account (if appropriate)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Added a `coverage-check` testenv to `tox.ini`, so `tox` will now fail if coverage falls below 100%
@speller26
speller26 requested a review from a team as a code owner September 18, 2026 23:12
@codecov

codecov Bot commented Sep 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (b5413c6) to head (4553caa).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #407   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           49        49           
  Lines         4964      4964           
  Branches       580       580           
=========================================
  Hits          4964      4964           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@speller26
speller26 merged commit c881cfd into main Sep 19, 2026
39 of 48 checks passed
@speller26
speller26 deleted the coverage branch September 19, 2026 00:12
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.

2 participants