ci(coverage): ratchet floor to 82% + add local test-cov recipe#1
Merged
Conversation
Coverage on the initial-push CI run combined to 82.1% — below the 85% tri-OS floor that ci.yml's plan job sets. Lower the floor to 82 (single ratchet for both single-OS and tri-OS paths) as a no-regression bar. Target is 90%; raise the ratchet as tests for spec-edge branches in _canonicalize / _constructor / _url are added. Add a `just test-cov` recipe that runs the suite under coverage and produces a term-missing summary plus an HTML report under htmlcov/, so contributors can drill into uncovered branches without rebuilding the CI invocation locally. `just cov-open` opens the report. Note the CI vs local gap (82.1% in CI, 96.7% locally on the same suite) — likely a path-mapping miss in `[tool.coverage.paths]` when site- packages from the wheel install doesn't combine cleanly with src/. Investigated separately; the ratchet here just unsticks the merge gate.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
just test-covrecipe runs the suite under coverage and produces a term-missing summary + HTML report athtmlcov/index.html.just cov-openlaunches the report.Context
Local
just test-covreports 96.7% combined coverage on the same suite that CI combines to 82.1%. That gap looks like a[tool.coverage.paths]mapping miss when matrix jobs measure the installed wheel under site-packages and the combine job's checkout is at src/. Tracking that separately; this PR just unsticks the merge gate.Test plan
just test-covproduces report locally (580 passed, 19 skipped, 96.7%)Ensure combined coverage thresholdjobCI greenaggregate goes green so the ruleset allows merge