Phase 0a: A1 Angular HTTPS-Cert-Race fixen (einer der 4 pre-existing failures) #38

New issue

Closed

opened 2026-06-18 06:17:57 +00:00 by Artur · 1 comment

Artur commented

2026-06-18 06:17:57 +00:00

Owner

Problembeschreibung

Test A1 ("Bootstrap — Angular lädt und bootstrapt erfolgreich") in angular-comprehensive.test.ts schlägt fehl, wenn die gesamte Test-Suite parallel läuft. Fehler:

[ScriptLoader] Failed to load script: https://localhost/angular/a1-bootstrap.js — unable to verify the first certificate
Empty chunk: https://localhost/angular/a1-bootstrap.js

Der Angular-Test-Server (Bun.serve ohne TLS) served korrekt via HTTP auf localhost. Im parallelen Testlauf generiert der RequestManager jedoch eine HTTPS-URL für den Script-Fetch, was zu einem Zertifikatsfehler führt. In Isolation läuft der Test fehlerfrei — es ist ein Race-Condition-Problem in der parallelen Test-Infrastruktur.

Lösungsansätze

Option A: TLS in Bun.serve (empfohlen)

Den Mock-Server mit einem selbstsignierten Zertifikat starten, sodass HTTPS-Requests akzeptiert werden.

Vorteil: Löst die Ursache — HTTPS funktioniert dann tatsächlich.
Nachteil: Braucht Zertifikats-Dateien im Repo.

Option B: URL-Force auf HTTP (Workaround)

In DynamicScripts/ScriptLoader prüfen ob die Ziel-URL localhost ist und explizit http:// erzwingen statt https://.

Option C: Fetch-Interceptor für Test-Seiten

page.goto() returned echte HTTP-Response, aber dynamische Script-Nachladeversuche gehen via HTTPS. Im RequestManager einen Interceptor für localhost-URLs einbauen, der HTTP erzwingt.

Akzeptanzkriterien

bun test läuft ohne A1-Fehler (alle 29 Angular-Tests pass)
A1 besteht sowohl in Isolation als auch im parallelen Suite-Lauf
Keine Regression in anderen Tests (1422+ bestehende Tests bleiben grün)

Betroffene Dateien

tests/interaction/angular-comprehensive.test.ts — Mock-Server + Test-Seite
src/js/script-loader.ts — fetchContent/executeRaw
src/js/dynamic-scripts.ts — _processQueue fetch
Optional: scripts/gen-cert.sh — TLS-Zertifikat-Generierung

Cross-Referenzen

#37 (buildVarDecls fix, der React-UMD enabled hat)
Test-Suite: angular-comprehensive.test.ts (29 Tests, 28 pass, 1 flaky)

## Problembeschreibung Test A1 ("Bootstrap — Angular lädt und bootstrapt erfolgreich") in `angular-comprehensive.test.ts` schlägt fehl, wenn die gesamte Test-Suite parallel läuft. Fehler: ``` [ScriptLoader] Failed to load script: https://localhost/angular/a1-bootstrap.js — unable to verify the first certificate Empty chunk: https://localhost/angular/a1-bootstrap.js ``` Der Angular-Test-Server (`Bun.serve` ohne TLS) served korrekt via HTTP auf localhost. Im parallelen Testlauf generiert der RequestManager jedoch eine HTTPS-URL für den Script-Fetch, was zu einem Zertifikatsfehler führt. In Isolation läuft der Test fehlerfrei — es ist ein Race-Condition-Problem in der parallelen Test-Infrastruktur. ## Lösungsansätze ### Option A: TLS in Bun.serve (empfohlen) Den Mock-Server mit einem selbstsignierten Zertifikat starten, sodass HTTPS-Requests akzeptiert werden. **Vorteil:** Löst die Ursache — HTTPS funktioniert dann tatsächlich. **Nachteil:** Braucht Zertifikats-Dateien im Repo. ### Option B: URL-Force auf HTTP (Workaround) In `DynamicScripts`/`ScriptLoader` prüfen ob die Ziel-URL `localhost` ist und explizit `http://` erzwingen statt `https://`. ### Option C: Fetch-Interceptor für Test-Seiten `page.goto()` returned echte HTTP-Response, aber dynamische Script-Nachladeversuche gehen via HTTPS. Im RequestManager einen Interceptor für localhost-URLs einbauen, der HTTP erzwingt. ## Akzeptanzkriterien - [ ] `bun test` läuft ohne A1-Fehler (alle 29 Angular-Tests pass) - [ ] A1 besteht sowohl in Isolation als auch im parallelen Suite-Lauf - [ ] Keine Regression in anderen Tests (1422+ bestehende Tests bleiben grün) ## Betroffene Dateien - `tests/interaction/angular-comprehensive.test.ts` — Mock-Server + Test-Seite - `src/js/script-loader.ts` — fetchContent/executeRaw - `src/js/dynamic-scripts.ts` — _processQueue fetch - Optional: `scripts/gen-cert.sh` — TLS-Zertifikat-Generierung ## Cross-Referenzen - #37 (buildVarDecls fix, der React-UMD enabled hat) - Test-Suite: angular-comprehensive.test.ts (29 Tests, 28 pass, 1 flaky)

Artur added the

bug

label

2026-06-18 06:17:57 +00:00

Artur referenced this issue

2026-06-18 06:17:58 +00:00

Phase 0b: Completed Issues aufräumen und schließen (24 Issues mit Implementierung) #39

Artur commented

2026-06-18 06:28:50 +00:00

Author

Owner

✅ Implementiert. Commit 3517c7c.
Root cause: test HTML used relative script URLs → DynamicScriptHandler resolved against wrong base URL during parallel execution.
Fix: server generates absolute URLs (http://localhost:PORT/...) in fixture HTML.
Also: changed Page default baseUrl + fallback URLs from https://localhost to http://localhost.
Test: 1423 pass, A1 now ✅ in parallel suite.

✅ Implementiert. Commit 3517c7c. Root cause: test HTML used relative script URLs → DynamicScriptHandler resolved against wrong base URL during parallel execution. Fix: server generates absolute URLs (http://localhost:PORT/...) in fixture HTML. Also: changed Page default baseUrl + fallback URLs from https://localhost to http://localhost. Test: 1423 pass, A1 now ✅ in parallel suite.

Artur closed this issue

2026-06-18 06:28:50 +00:00

Artur referenced this issue

2026-06-18 10:37:20 +00:00

Sprint 2: Inline Scripts blockieren Module Execution (per HTML Spec) #51

Artur referenced this issue from a commit

2026-06-18 10:38:01 +00:00

Sprint 2: Inline Config Routing — Proxy ownKeys Trap + globalThis Sync + Prologue Erweiterung