Skip to content

fix: exclude build output from coverage#462

Merged
marcolink merged 2 commits intomasterfrom
fix/coverage-ignore-lib
Apr 1, 2026
Merged

fix: exclude build output from coverage#462
marcolink merged 2 commits intomasterfrom
fix/coverage-ignore-lib

Conversation

@marcolink
Copy link
Copy Markdown
Collaborator

Summary

  • restrict Jest coverage collection to source files under src/**
  • ignore generated lib/** output during coverage runs
  • remove redundant build step before coverage upload in CI

Why

The Apr 1 CI change started running pnpm build before pnpm test:coverage. That generated lib/**, and Jest/Coveralls began counting both source and compiled output, dropping reported coverage from about 90% to about 56%.

Verification

  • pnpm exec jest --coverage --coverageReporters=text-summary --coverageReporters=json-summary
  • lines: 94.61%
  • test suites: 20 passed

@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 23861677723

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+34.2%) to 90.741%

Totals Coverage Status
Change from base Build 23857345948: 34.2%
Covered Lines: 457
Relevant Lines: 483

💛 - Coveralls

@marcolink marcolink merged commit ad82392 into master Apr 1, 2026
6 checks passed
@marcolink marcolink deleted the fix/coverage-ignore-lib branch April 1, 2026 17:27
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