Keine Tests im Projekt #30

Open
opened 2026-09-09 20:12:51 +00:00 by Aceless · 1 comment
Owner

Das Verzeichnis app/test/ existiert nicht - es gibt weder Unit-, Widget- noch Integrationstests. flutter_test ist in den dev_dependencies deklariert und ungenutzt.

Gerade die Stellen mit reiner Rechenlogik liessen sich guenstig absichern und haben nachweislich Fehler:

  • ModernWeekField._weekNumber - liefert falsche Kalenderwochen
  • MealPlanNotifier._getWeekStart - Montags-/Sonntagslogik inkl. Monats- und Jahreswechsel
  • RecipeState.copyWith / _applyFilters - das Filter-Reset-Problem waere sofort aufgefallen
  • Recipe.fromJson / MealPlanEntry.fromJson - Feldnamen und Defaults, vgl. das camelCase-Problem in AppInfoService

Vorschlag: Mit Unit-Tests fuer diese vier Bereiche starten, danach Widget-Tests fuer LoginScreen und RecipeEditScreen (Validierung, Speichern-Pfad). flutter test und flutter analyze zusammen in CI verankern.

Das Verzeichnis `app/test/` existiert nicht - es gibt weder Unit-, Widget- noch Integrationstests. `flutter_test` ist in den `dev_dependencies` deklariert und ungenutzt. Gerade die Stellen mit reiner Rechenlogik liessen sich guenstig absichern und haben nachweislich Fehler: - `ModernWeekField._weekNumber` - liefert falsche Kalenderwochen - `MealPlanNotifier._getWeekStart` - Montags-/Sonntagslogik inkl. Monats- und Jahreswechsel - `RecipeState.copyWith` / `_applyFilters` - das Filter-Reset-Problem waere sofort aufgefallen - `Recipe.fromJson` / `MealPlanEntry.fromJson` - Feldnamen und Defaults, vgl. das camelCase-Problem in `AppInfoService` **Vorschlag:** Mit Unit-Tests fuer diese vier Bereiche starten, danach Widget-Tests fuer `LoginScreen` und `RecipeEditScreen` (Validierung, Speichern-Pfad). `flutter test` und `flutter analyze` zusammen in CI verankern.
Author
Owner

Teilweise erledigt: test/ existiert jetzt mit 22 gruenen Tests fuer isoWeekNumber, startOfWeek, dayIndexInWeek und die copyWith-Sentinel-Logik. Offen bleiben die Widget-Tests fuer LoginScreen und RecipeEditScreen sowie die CI-Einbindung von flutter test und flutter analyze. Bleibt deshalb offen.

Teilweise erledigt: test/ existiert jetzt mit 22 gruenen Tests fuer isoWeekNumber, startOfWeek, dayIndexInWeek und die copyWith-Sentinel-Logik. Offen bleiben die Widget-Tests fuer LoginScreen und RecipeEditScreen sowie die CI-Einbindung von flutter test und flutter analyze. Bleibt deshalb offen.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
Aceless/RecipePlanner#30
No description provided.