Coverage config hides untested files

Open Beginner friendly
#24,051 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
78/100
Issue type
Bug
Clarity
Clearly specified
Activity status
Active
Tech stack
typescript, vite
Domain
testing-qa

Research direction

Start with vite/vite.config.ts and inspect the existing coverage.enabled and exclude settings. Enable all-file coverage, run the package coverage checks, and record the resulting baseline before adding thresholds; done means untested source files appear in the report and the real baseline is documented.

Written by the indexing model from the issue text.

Description

Cloudflare Workers javascript Tests

vite/vite.config.ts sets coverage.enabled and an exclude list, but not all: true. V8 coverage only reports files a test imported, so a source file with no test at all contributes nothing and shows no row. The Cloudflare 95.5% figure is over touched files only.

Files with no matching unit test today, by package: cloudflare 17, deno 21, bun 3, vercel-edge 3, hono 4, effect 2.

Work item. Set coverage.all: true in the base config and record the real baseline before setting any thresholds. Expect the reported numbers to drop.

Dominant language
TypeScript
Stars
8.7k
Forks
1.9k
Avg merge
1d 17h
Merged PRs (30d)
521

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from getsentry/sentry-javascript

All issues in getsentry/sentry-javascript

Similar issues

More TypeScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.