From 16b500ae3d9955f40d3877786e38cde6e04447a2 Mon Sep 17 00:00:00 2001 From: Marcel Date: Thu, 14 May 2026 11:05:10 +0200 Subject: [PATCH] fix(ci): run client coverage even when server coverage fails Replace && with ; in test:coverage so the client vitest run is not short-circuited when the server run exits non-zero (e.g. threshold violation or test failure). Without this the upload-artifact step only ever sees coverage/server. Also updates the stale CLAUDE.md comment that said server-only. Co-Authored-By: Claude Sonnet 4.6 --- frontend/CLAUDE.md | 2 +- frontend/package.json | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/frontend/CLAUDE.md b/frontend/CLAUDE.md index c58a26c6..3699350f 100644 --- a/frontend/CLAUDE.md +++ b/frontend/CLAUDE.md @@ -165,7 +165,7 @@ npm run check # svelte-check (type checking) ```bash npm run test # Vitest unit + server tests (headless) -npm run test:coverage # Coverage report (server project only) +npm run test:coverage # Coverage report (server + client) npm run test:e2e # Playwright E2E tests npm run test:e2e:headed # Playwright E2E with visible browser npm run test:e2e:ui # Playwright UI mode diff --git a/frontend/package.json b/frontend/package.json index f70e4ab2..aaf7e910 100644 --- a/frontend/package.json +++ b/frontend/package.json @@ -16,7 +16,7 @@ "lint:boundary-demo": "eslint src/lib/tag/__fixtures__/", "test:unit": "vitest", "test": "npm run test:unit -- --run", - "test:coverage": "vitest run --coverage --project=server && vitest run -c vitest.client-coverage.config.ts --coverage", + "test:coverage": "vitest run --coverage --project=server; vitest run -c vitest.client-coverage.config.ts --coverage", "test:e2e": "playwright test", "test:e2e:headed": "playwright test --headed", "test:e2e:ui": "playwright test --ui",