Auf- und Abstieg: zwei Tests prüfen weniger, als ihr Name sagt (PhaseDialog, MigrationSmokeTest) #110

New issue

Open

opened 2026-10-18 17:55:45 +00:00 by claude · 0 comments

claude commented

2026-10-18 17:55:45 +00:00

Collaborator

Gefunden im Abschluss-Review von #88 (in main eb18a9b). Bestehende Tests ändern — braucht Ollis Ja vor dem Bau.

PhaseDialog — sperrt auch den Saisonstart, aber keinen anderen Wechsel (portal/src/test/PhaseDialog.test.tsx (179)): die zweite Hälfte klickt „Meldungen beendet“ und erwartet Speichern frei. Da die Sperrliste nur RUNNING enthält, wäre das auch ohne jede Sperrlogik so — die Hälfte fällt bei keinem denkbaren Fehler um. Vorschlag: zusätzlich eine Sperre für eine andere Phase setzen und prüfen, dass sie nur dort greift.
MigrationSmokeTest V46 — seedCategoriesBeforeV46 legt beide Saisons als CLOSED an (backend/src/test/java/ch/sffs/tt/migration/MigrationSmokeTest.java (336-341)). Laut V46-Kommentar gelten die Zahlen „in jeder Phase“; geprüft ist nur eine. Vorschlag: 2026/2027 in einer offenen Phase anlegen (z.B. RUNNING), damit die Phasenunabhängigkeit belegt ist.

Gefunden im Abschluss-Review von #88 (in main eb18a9b). **Bestehende Tests ändern — braucht Ollis Ja vor dem Bau.** 1. **PhaseDialog** — `sperrt auch den Saisonstart, aber keinen anderen Wechsel` (`portal/src/test/PhaseDialog.test.tsx` (179)): die zweite Hälfte klickt „Meldungen beendet“ und erwartet Speichern frei. Da die Sperrliste nur `RUNNING` enthält, wäre das auch ohne jede Sperrlogik so — die Hälfte fällt bei keinem denkbaren Fehler um. Vorschlag: zusätzlich eine Sperre für eine andere Phase setzen und prüfen, dass sie nur dort greift. 2. **MigrationSmokeTest V46** — `seedCategoriesBeforeV46` legt beide Saisons als `CLOSED` an (`backend/src/test/java/ch/sffs/tt/migration/MigrationSmokeTest.java` (336-341)). Laut V46-Kommentar gelten die Zahlen „in jeder Phase“; geprüft ist nur eine. Vorschlag: 2026/2027 in einer offenen Phase anlegen (z.B. `RUNNING`), damit die Phasenunabhängigkeit belegt ist.