fix(frontend): flaky CI + Svelte 5-Warnungen in Component-Tests #294

Merged

niboer merged 1 commit from fix/svelte5-test-warnungen into main

2026-07-11 13:30:26 +02:00

niboer commented

2026-07-11 13:17:29 +02:00

Owner

Allgemein

refs #282

Altes Verhalten

Flaky CI: Frontend-Unit-Tests schlagen in CI unberechenbar fehl wegen ReferenceError: document is not defined aus bits-ui body-scroll-lock. Alle 1216 Tests laufen gruen, aber ein nach jsdom-Teardown feuernder Timeout verursacht exit code 1.
invalid_raw_snippet_render: createRawSnippet-Renderfunktion in data-table.component.test.ts gibt Plain-String statt HTML-Element zurueck.
derived_inert: Svelte-5-Warnungen aus bits-ui-internem Code waehrend Component-Teardown.

Neues Verhalten

✅ Flaky CI gefixt: setTimeout-Tracker in vitest.setup.ts ueberschreibt globalThis.setTimeout/clearTimeout, tracked alle Timer-IDs. Der afterEach-Hook in setupFiles (laeuft NACH testing-library's cleanup) cleart alle pending Timer vor jsdom-Teardown.
✅ invalid_raw_snippet_render gefixt: createRawSnippet.render gibt jetzt <span>...</span> statt Plain-String.
✅ Dokumentiert: docs/design/09-bekannte-svelte5-warnungen.md mit Upstream-Referenzen (bits-ui #2083, #2080, #2050).

Begruendung

Der flaky CI betrifft jeden PR und erzwungenes manuelles Re-Running, ohne wirkliche Code-Aenderung zu signalisieren.
invalid_raw_snippet_render ist ein echter Svelte-5-Kompatibilitaetsfehler im Test.
derived_inert kann von unserer Seite nicht gefixt werden (bits-ui node_modules), wird aber dokumentiert.

Test-Hinweise

Lokal: cd frontend && npx vitest run — alle 72 Test Files muessen gruen sein, 0 unhandled errors.
CI: Der Job test-frontend sollte nicht mehr flaky fehlschlagen.

### Allgemein - refs #282 ### Altes Verhalten 1. **Flaky CI:** Frontend-Unit-Tests schlagen in CI unberechenbar fehl wegen `ReferenceError: document is not defined` aus bits-ui body-scroll-lock. Alle 1216 Tests laufen gruen, aber ein nach jsdom-Teardown feuernder Timeout verursacht exit code 1. 2. **invalid_raw_snippet_render:** `createRawSnippet`-Renderfunktion in `data-table.component.test.ts` gibt Plain-String statt HTML-Element zurueck. 3. **derived_inert:** Svelte-5-Warnungen aus bits-ui-internem Code waehrend Component-Teardown. ### Neues Verhalten 1. ✅ **Flaky CI gefixt:** `setTimeout`-Tracker in `vitest.setup.ts` ueberschreibt `globalThis.setTimeout`/`clearTimeout`, tracked alle Timer-IDs. Der `afterEach`-Hook in setupFiles (laeuft NACH testing-library's cleanup) cleart alle pending Timer vor jsdom-Teardown. 2. ✅ **invalid_raw_snippet_render gefixt:** `createRawSnippet.render` gibt jetzt `<span>...</span>` statt Plain-String. 3. ✅ **Dokumentiert:** `docs/design/09-bekannte-svelte5-warnungen.md` mit Upstream-Referenzen (bits-ui #2083, #2080, #2050). ### Begruendung - Der flaky CI betrifft jeden PR und erzwungenes manuelles Re-Running, ohne wirkliche Code-Aenderung zu signalisieren. - `invalid_raw_snippet_render` ist ein echter Svelte-5-Kompatibilitaetsfehler im Test. - `derived_inert` kann von unserer Seite nicht gefixt werden (bits-ui node_modules), wird aber dokumentiert. ### Test-Hinweise 1. Lokal: `cd frontend && npx vitest run` — alle 72 Test Files muessen gruen sein, 0 unhandled errors. 2. CI: Der Job `test-frontend` sollte nicht mehr flaky fehlschlagen.

niboer added 1 commit

2026-07-11 13:17:29 +02:00

fix(frontend): flaky CI + Svelte 5-Warnungen in Component-Tests

CI Backend / test-backend (pull_request) Has been cancelled

Details

CI Frontend / lint-frontend (pull_request) Has been cancelled

Details

CI Frontend / test-frontend (pull_request) Has been cancelled

Details

CI Meta / conventional-commit (pull_request) Has been cancelled

Details

E2E / e2e (pull_request) Has been cancelled

Details

CI Backend / lint-backend (pull_request) Has been cancelled

Details

448900772d

refs #282

Aktuelles:
- Flaky CI: Uncaught ReferenceError: document is not defined aus
  bits-ui body-scroll-lock Cleanup-Timeout nach jsdom-Teardown
- invalid_raw_snippet_render in data-table.component.test.ts
- derived_inert-Warnungen aus bits-ui, nicht test-failend

Neu:
- setTimeout-Tracker + afterEach-Cleaner in vitest.setup.ts verhindert
  Race-Condition zwischen bits-ui-Timeout und jsdom-Teardown
- createRawSnippet.render gibt einzelnes <span>-Element zurueck
- docs/design/09-bekannte-svelte5-warnungen.md dokumentiert
  derived_inert als bits-ui-Upstream-Bug (#2083, #2080, #2050)

Wichtige Aenderung: vitest.setup.ts ueberschreibt globalThis.setTimeout
zum Tracken aller Timer-IDs. afterEach in setupFiles laeuft nach
testing-library's cleanup und cleart alle pending Timer vor
jsdom-Teardown.