Skip to content

Commit 8f0a274

Browse files
committed
wip!
1 parent f272a9f commit 8f0a274

File tree

2 files changed

+8
-11
lines changed

2 files changed

+8
-11
lines changed

.github/workflows/pr.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -171,7 +171,7 @@ jobs:
171171
test_target_name: e2e.esbuild_node22
172172
env:
173173
E2E_SHARD_TOTAL: 1
174-
TESTBRIDGE_TEST_ONLY: tests/basic/{build,rebuild}.ts
174+
TESTBRIDGE_TEST_ONLY: tests/vitest/larger-project-coverage.ts
175175

176176
e2e-package-managers:
177177
needs: build

tests/legacy-cli/e2e/tests/vitest/larger-project-coverage.ts

Lines changed: 7 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -63,16 +63,13 @@ export default async function () {
6363
const { stdout: jsdomStdout } = await ng('test', '--no-watch', '--coverage');
6464
assert.match(jsdomStdout, expectedMessage, `Expected ${totalTests} tests to pass in JSDOM mode.`);
6565

66-
// TODO: Investigate why coverage-final.json is empty on Windows in JSDOM mode.
67-
// For now, skip the coverage report check on Windows.
68-
if (process.platform !== 'win32') {
69-
// Assert that every generated file is in the coverage report by reading the JSON output.
70-
const jsdomSummary = JSON.parse(await readFile(coverageJsonPath));
71-
const jsdomSummaryKeys = Object.keys(jsdomSummary);
72-
for (const file of generatedFiles) {
73-
const found = jsdomSummaryKeys.some((key) => key.endsWith(file));
74-
assert.ok(found, `Expected ${file} to be in the JSDOM coverage report.`);
75-
}
66+
// Assert that every generated file is in the coverage report by reading the JSON output.
67+
const jsdomSummary = JSON.parse(await readFile(coverageJsonPath));
68+
console.log({ out: JSON.stringify({ jsdomSummary }) });
69+
const jsdomSummaryKeys = Object.keys(jsdomSummary);
70+
for (const file of generatedFiles) {
71+
const found = jsdomSummaryKeys.some((key) => key.endsWith(file));
72+
assert.ok(found, `Expected ${file} to be in the JSDOM coverage report.`);
7673
}
7774

7875
// Setup for browser mode

0 commit comments

Comments
 (0)