Enforce test coverage thresholds#139
Merged
Merged
Conversation
Adds V8 coverage to the vitest run and wires it into the test script, so both CI (pnpm test) and verify enforce a coverage floor. Thresholds sit a few points below current coverage (statements/lines 90, functions 92, branches 84) to guard against regressions without being brittle. Closes #134
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
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
Adds V8 coverage to the vitest run and enforces a minimum coverage floor, wired into the
testscript so both CI (pnpm test) andverifyapply it. The dependency (@vitest/coverage-v8) is committed with the lockfile this time.How verified
pnpm testruns coverage and passes at current levels (statements 93.15, branches 87.1, functions 95.62, lines 93.34).Closes #134