Skip to content

Commit db8a444

Browse files
fullstackjamclaude
andcommitted
fix(test): skip v8 coverage on pool-workers suite
workerd has no node:inspector/promises module, which @vitest/coverage-v8 requires. Coverage now runs only on the happy-dom suite; the pool-workers suite still runs (verifying behavior) but does not contribute to the coverage report. Trade-off: covered % drops because many tests moved to pool-workers. Acceptable — tests still execute, and switching the coverage backend (e.g. istanbul) is a separate decision. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 0f06f57 commit db8a444

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515
"test:run": "vitest run && vitest run --config vitest.workers.config.ts",
1616
"test:happy-dom": "vitest run",
1717
"test:workers": "vitest run --config vitest.workers.config.ts",
18-
"test:coverage": "vitest run --coverage && vitest run --config vitest.workers.config.ts --coverage",
18+
"test:coverage": "vitest run --coverage && vitest run --config vitest.workers.config.ts",
1919
"lint": "eslint .",
2020
"lint:fix": "eslint . --fix",
2121
"format": "prettier --write .",

0 commit comments

Comments
 (0)