[TEST] Increase coverage thresholds from 2-4% to industry-standard 40-70%

[Delqhi]

Problem

Coverage thresholds in frontend/vitest.config.js are far too low:

thresholds: {
  lines: 2,
  functions: 4,
  branches: 2,
  statements: 2
}

Actual coverage: 14.73% lines, 20.91% branches, 16.69% functions, 13.97% statements

These thresholds were set as a temporary workaround (PR #85) with autoUpdate: true to prevent CI from failing. This was acceptable as a starting point, but the thresholds should be raised incrementally.

Reference Plan

docs/COVERAGE-THRESHOLDS.md outlines a 4-phase plan:

• Phase 1: 2% → realistic baseline (current)
• Phase 2: 10% — quick wins in critical components
• Phase 3: 25% — core business logic covered
• Phase 4: 40% — production-ready (industry standard)

Acceptance Criteria

• Move to Phase 2 (10% lines, 10% branches) — add tests for critical paths (auth, navigation, data display)
• Remove autoUpdate: true once at Phase 2
• Each phase has a documented set of components/functions to cover
• CI runs coverage check on every PR

Aufwand

~1-2 days for Phase 2

Related

• refactor(frontend): Split 5 oversized files (>500 lines each) #83 (closed): Expand test coverage to 40% — superseded by phased plan
• ops(politician): Run initial sync + build status dashboard + alerting #85 (closed): vitest thresholds with autoUpdate — Phase 1 complete

[Delqhi]

Progress update

Added 29 new server-side unit tests today and pushed to main:

• server/__tests__/utils/middleware/requireAuthWhenOnboardingComplete.test.js (4 tests) — onboarding auth gate
• server/__tests__/utils/chats/extractImageUrls.test.js (12 tests) — screenshot URL OCR detection
• server/__tests__/utils/research/vaneClient.test.js (13 tests) — Vane client integration

All new tests pass. This moves us toward Phase 2 (realistic 10% baseline) but does not yet change the frontend vitest.config.js thresholds. Next step for this issue would be to raise the thresholds and add frontend coverage for the critical onboarding/chat components.

[Delqhi]

Phase-2-Fortschritt

Nach dem heutigen Push liegt die Frontend-Coverage bei 31,29 % lines / 38,4 % branches / 30,65 % functions / 31,66 % statements (1.142 Tests in 145 Suites, alle grün).

Damit ist die Phase-2-Zielmarke von 10 % deutlich überschritten. Die Test-Anzahl wurde durch 18 neue Test-Dateien für Chat-UI-Komponenten erhöht.

Offene Akzeptanzkriterien für Phase 2:

• Thresholds in von 2-4 % auf 10-15 % anheben
• entfernen, damit zukünftige PRs nicht mehr automatisch sinken
• Coverage-Check in CI als required step etablieren

Issue bleibt offen, bis die Thresholds und CI-Integration angepasst sind.

[Delqhi]

Thresholds angehoben und autoUpdate entfernt

Mit Commit 8c5341b wurde angepasst:

• Thresholds auf 20 % für statements, branches, functions, lines gesetzt
• entfernt, damit Coverage-Senkungen in Zukunft den Build failen lassen
• Aktuelle Coverage: 52,09 % statements / 52,29 % branches / 51,36 % functions / 52,52 % lines – damit liegen alle Metriken deutlich über den neuen 20 %-Thresholds.

Verifiziert:

• in läuft grün
• 183 Test-Dateien, 1.245 Tests

Issue wird geschlossen. Weitere Threshold-Erhöhungen (z.B. auf 40 %) können als separates Issue verfolgt werden.