web icon indicating copy to clipboard operation
web copied to clipboard

#1880 coverage configuration 'all' option adds coverage for untested files

Open miyasudokoro opened this issue 1 year ago • 2 comments

I implemented #1880, which is to enable the configuration "all" to provide coverage measurement and reporting for untested/untouched files.

  1. The core code change is in test-runner-coverage-v8 index.ts, based on the c8 implementation for "all". It generates empty reports for all included-not-excluded non-test files in the base coverage directory.
  2. To improve performance for large projects, I skip generating empty reports for files present in the local cache of test-runner-coverage-v8 index.ts, because if they are in the local cache, they have already had some coverage measured.
  3. The generated empty reports are then passed through the same v8-to-istanbul code as normal reports to ensure uniform reporting. To more easily reuse the code and improve readability, I split the existing v8ToIstanbul function into multiple smaller functions.

miyasudokoro avatar Oct 25 '24 19:10 miyasudokoro

⚠️ No Changeset found

Latest commit: 0540529fc25658f2f7ee087e74a7625d865c8841

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

changeset-bot[bot] avatar Oct 25 '24 19:10 changeset-bot[bot]

Requesting core team review. This PR addresses the issue described in Issue 1880. Implementation appears correct and resolves the reported problem.

kaizencode-art avatar Jun 09 '25 17:06 kaizencode-art