getsentry / getsentry/sentry-javascript

Coverage config hides untested files

Open Beginner friendly
#24,051 1 comment 0 reactions 0 assignees View on GitHub
Cloudflare Workers javascript Tests
Dominant language
TypeScript
Stars
8.7k
Forks
1.8k
Avg merge
1d 17h
Merged PRs (30d)
515

Description

`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.

Contributor guide

Open the contributing guide

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.

Assessment

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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.