Seitentests beantworten jede Adresse – ein falscher Pfad fällt dort nicht auf #136

New issue

Open

opened 2026-10-23 20:15:57 +00:00 by claude · 0 comments

claude commented

2026-10-23 20:15:57 +00:00

Collaborator

Aus dem Review von Etappe 1 der URL-Neuordnung (23.10., Branch feature/url-etappe-1).

Mehrere Frontend-Tests ersetzen den Server durch eine Attrappe, die jede Anfrage beantwortet, egal welche Adresse sie hat:

portal/src/test/MatchTaskPage.*.test.tsx: jedes GET liefert das Begegnungs-Detail
portal/src/test/VenuePage.test.tsx, RegistrationsPage.test.tsx, SubstituteAppearancesPage.test.tsx, LoginPage.test.tsx
portal/src/test/EloCutoffPage.test.tsx und DashboardPage.test.tsx prüfen nur includes('/elo-determinations'), nicht ?seasonId=

Ruft eine Seite einen falschen oder alten Pfad auf, bleibt der Test grün. In Etappe 1 hat das nur der E2E-Lauf mit Treffervergleich abgefangen. Am 23.10. ist so schon eine Stelle durchgerutscht (EloCutoffPage.test.tsx prüfte ein Teilstück des alten Pfads).

Vorschlag: Je Seite mindestens ein Test, der die aufgerufene Adresse prüft (toHaveBeenCalledWith('/home-venues') o. ä.). Sinnvoll vor den nächsten Etappen der URL-Neuordnung, die mehr Pfade bewegen. Teständerungen nur mit Ollis Ja.

Aus dem Review von Etappe 1 der URL-Neuordnung (23.10., Branch `feature/url-etappe-1`). Mehrere Frontend-Tests ersetzen den Server durch eine Attrappe, die **jede** Anfrage beantwortet, egal welche Adresse sie hat: - `portal/src/test/MatchTaskPage.*.test.tsx`: jedes GET liefert das Begegnungs-Detail - `portal/src/test/VenuePage.test.tsx`, `RegistrationsPage.test.tsx`, `SubstituteAppearancesPage.test.tsx`, `LoginPage.test.tsx` - `portal/src/test/EloCutoffPage.test.tsx` und `DashboardPage.test.tsx` prüfen nur `includes('/elo-determinations')`, nicht `?seasonId=` Ruft eine Seite einen falschen oder alten Pfad auf, bleibt der Test grün. In Etappe 1 hat das nur der E2E-Lauf mit Treffervergleich abgefangen. Am 23.10. ist so schon eine Stelle durchgerutscht (`EloCutoffPage.test.tsx` prüfte ein Teilstück des alten Pfads). **Vorschlag:** Je Seite mindestens ein Test, der die aufgerufene Adresse prüft (`toHaveBeenCalledWith('/home-venues')` o. ä.). Sinnvoll vor den nächsten Etappen der URL-Neuordnung, die mehr Pfade bewegen. Teständerungen nur mit Ollis Ja.