Skip to content

Commit 5dc3be5

Browse files
davehentonclaude
andcommitted
Fix coverage: run vitest per-package, merge lcov reports with path prefixes
Running vitest from the workspace root with --coverage.enabled doesn't properly apply per-package globals config. Switch to per-package vitest runs using pnpm --filter, merge lcov reports with corrected paths, and install the coverage provider before tests to avoid node_modules issues. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 3d9a500 commit 5dc3be5

1 file changed

Lines changed: 12 additions & 3 deletions

File tree

.github/workflows/ci.yml

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -36,19 +36,28 @@ jobs:
3636
- uses: pnpm/action-setup@v5
3737
with:
3838
run_install: true
39-
- run: pnpm test
4039
- name: Install coverage provider
4140
if: matrix.node-version == 22
4241
run: pnpm add -Dw @vitest/coverage-v8
42+
- run: pnpm test
4343
- name: Generate coverage
4444
if: matrix.node-version == 22
45-
run: pnpm exec vitest run --coverage.enabled --coverage.reporter=lcov
45+
run: |
46+
pnpm --filter openapi-fetch exec vitest run --coverage.enabled --coverage.reporter=lcov
47+
pnpm --filter openapi-typescript exec vitest run --coverage.enabled --coverage.reporter=lcov
48+
pnpm --filter openapi-react-query exec vitest run --coverage.enabled --coverage.reporter=lcov
49+
- name: Merge coverage reports
50+
if: matrix.node-version == 22
51+
run: |
52+
sed 's|^SF:|SF:packages/openapi-fetch/|' packages/openapi-fetch/coverage/lcov.info > coverage-merged.info
53+
sed 's|^SF:|SF:packages/openapi-typescript/|' packages/openapi-typescript/coverage/lcov.info >> coverage-merged.info
54+
sed 's|^SF:|SF:packages/openapi-react-query/|' packages/openapi-react-query/coverage/lcov.info >> coverage-merged.info
4655
- name: Upload coverage to Qlty
4756
if: matrix.node-version == 22
4857
uses: qltysh/qlty-action/coverage@v2
4958
with:
5059
token: ${{ secrets.QLTY_COVERAGE_TOKEN }}
51-
files: coverage/lcov.info
60+
files: coverage-merged.info
5261
test-e2e:
5362
runs-on: ubuntu-latest
5463
steps:

0 commit comments

Comments
 (0)