Add browser test for skipping a meal and marking eating out #58

New issue

Open

opened 2026-08-18 23:53:40 +02:00 by myrmidex · 0 comments

myrmidex commented

2026-08-18 23:53:40 +02:00

Owner

Gap

#51 added an optional skip reason ("eating out") to the schedule calendar. It is covered by feature tests (ScheduleCalendarTest, SkipScheduledUserDishForDateActionTest) but has no browser test — no browser test in tests/Browser/Schedule/ mentions skipping at all.

This is a calendar interaction driven by Livewire and Alpine, so the browser layer is where the real behaviour lives.

Scope

In tests/Browser/Schedule/SkipMealTest.php:

Skipping a meal marks it skipped in the calendar
Choosing "eating out" as the reason is visually distinguishable from a plain skip
Un-skipping restores the meal
The reason persists across a page reload

Blocked by #54

Write in Pest browser syntax after the Dusk → Pest migration. This flow is a good first candidate for the new setup — it is visual, interactive, and recent, so it doubles as proof the migration works.

Acceptance

Skip covered
Eating-out reason covered and visually distinct
Un-skip covered
Persistence across reload covered
Green in CI

## Gap #51 added an optional skip reason ("eating out") to the schedule calendar. It is covered by feature tests (`ScheduleCalendarTest`, `SkipScheduledUserDishForDateActionTest`) but has no browser test — no browser test in `tests/Browser/Schedule/` mentions skipping at all. This is a calendar interaction driven by Livewire and Alpine, so the browser layer is where the real behaviour lives. ## Scope In `tests/Browser/Schedule/SkipMealTest.php`: - Skipping a meal marks it skipped in the calendar - Choosing "eating out" as the reason is visually distinguishable from a plain skip - Un-skipping restores the meal - The reason persists across a page reload ## Blocked by #54 Write in Pest browser syntax after the Dusk → Pest migration. This flow is a good first candidate for the new setup — it is visual, interactive, and recent, so it doubles as proof the migration works. ## Acceptance - [ ] Skip covered - [ ] Eating-out reason covered and visually distinct - [ ] Un-skip covered - [ ] Persistence across reload covered - [ ] Green in CI