test(golden): voeg ontbrekende slidetype-goldens toe (controlStatus, menu) #1197

Merged

brenno merged 1 commit from fix/missing-controlstatus-golden into main

2026-08-04 07:44:33 +00:00

brenno commented

2026-08-04 07:38:24 +00:00

Owner

Waarom

De test elk slidetype heeft een golden in test/golden/slide_golden_test.dart itereert over SlideType.values en verwacht per type een test/golden/goldens/type_<naam>.png. Bij het toevoegen van de diatypes controlStatus en menu (keuze-menu, fase 3 van #1162) aan de enum zijn de bijbehorende goldens niet meegecommit, waardoor make test-golden lokaal faalt. Deze poort draait niet per PR in CI (alleen op een v*-tag), dus het is niet eerder rood geworden. Staat los van #1163.

Wat

test/golden/goldens/type_controlStatus.png (nieuw)
test/golden/goldens/type_menu.png (nieuw)

Gericht gegenereerd met --update-goldens --plain-name per type. git status bevestigt dat uitsluitend deze twee bestanden bijkomen — geen enkele bestaande golden drift (deze machine is de referentie). De volledige golden-suite is groen (48/48).

De menu-golden rendert dun: de gedeelde fixture slideMetInhoud valt voor menu op de default (title: 'T') terug, en Slide.create(menu) zaait geen voorbeeldblokken (die leven in Slide.bullets), zodat alleen de titel tekent. Dat is de werkelijke huidige rendering. Een rijkere menu-fixture raakt ook de round-trip- en rasterizer-tests en hoort bij het menu-spoor zelf, niet bij deze golden-reparatie — als losse opvolging genoteerd.

Poort

Dit raakt alleen test-golden-artefacten (PNG's), geen bestandsformaat, opslag, afhankelijkheid, netwerk of publieke belofte — bewaker-weging niet van toepassing. Getoetst: make test-golden groen (48/48).

## Waarom De test `elk slidetype heeft een golden` in `test/golden/slide_golden_test.dart` itereert over `SlideType.values` en verwacht per type een `test/golden/goldens/type_<naam>.png`. Bij het toevoegen van de diatypes **`controlStatus`** en **`menu`** (keuze-menu, fase 3 van #1162) aan de enum zijn de bijbehorende goldens niet meegecommit, waardoor `make test-golden` lokaal faalt. Deze poort draait niet per PR in CI (alleen op een `v*`-tag), dus het is niet eerder rood geworden. Staat los van #1163. ## Wat - `test/golden/goldens/type_controlStatus.png` (nieuw) - `test/golden/goldens/type_menu.png` (nieuw) Gericht gegenereerd met `--update-goldens --plain-name` per type. `git status` bevestigt dat **uitsluitend deze twee bestanden** bijkomen — geen enkele bestaande golden drift (deze machine is de referentie). De volledige golden-suite is groen (48/48). ## Kanttekening bij de menu-golden De menu-golden rendert dun: de gedeelde fixture `slideMetInhoud` valt voor `menu` op de default (`title: 'T'`) terug, en `Slide.create(menu)` zaait geen voorbeeldblokken (die leven in `Slide.bullets`), zodat alleen de titel tekent. Dat is de werkelijke huidige rendering. Een rijkere menu-fixture raakt ook de round-trip- en rasterizer-tests en hoort bij het menu-spoor zelf, niet bij deze golden-reparatie — als losse opvolging genoteerd. ## Poort Dit raakt alleen test-golden-artefacten (PNG's), geen bestandsformaat, opslag, afhankelijkheid, netwerk of publieke belofte — bewaker-weging niet van toepassing. Getoetst: `make test-golden` groen (48/48).

brenno added 1 commit

2026-08-04 07:38:25 +00:00

test(golden): voeg ontbrekende slidetype-goldens toe (controlStatus, menu)

scans / scans (pull_request) Successful in 2m50s

Details

static-gate / static-gate (pull_request) Successful in 5m25s

Details

d2cc35f70d

De test `elk slidetype heeft een golden` in slide_golden_test.dart itereert
over SlideType.values en verwacht per type een `type_<naam>.png`. Bij het
toevoegen van de diatypes `controlStatus` (#1162) en `menu` (keuze-menu,
fase 3 van #1162) aan de enum zijn de bijbehorende goldens niet meegecommit,
waardoor `make test-golden` lokaal faalt. Deze poort draait niet in CI (alleen
op een v*-tag), dus het is niet eerder rood geworden.

Gericht gegenereerd met --update-goldens per type; git status bevestigt dat
uitsluitend deze twee bestanden bijkomen — geen enkele bestaande golden drift
(deze machine is de referentie).

De menu-golden rendert dun: de gedeelde fixture `slideMetInhoud` valt voor menu
op de default (`title: 'T'`) terug en `Slide.create(menu)` zaait geen
voorbeeldblokken (die leven in `bullets`), zodat alleen de titel tekent. Dat is
de werkelijke huidige rendering; een rijkere menu-fixture raakt ook de
round-trip- en rasterizer-tests en hoort bij het menu-spoor zelf, niet bij deze
golden-reparatie.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

brenno merged commit a871be5929 into main

2026-08-04 07:44:33 +00:00

brenno deleted branch fix/missing-controlstatus-golden

2026-08-04 07:44:35 +00:00

brenno referenced this pull request from a commit

2026-08-04 07:44:35 +00:00

Merge pull request 'test(golden): voeg ontbrekende slidetype-goldens toe (controlStatus, menu)' (#1197) from fix/missing-controlstatus-golden into main