[Feature] Golden tests cover 8 of 24 slide types and sit outside make check #617

New issue

Closed

opened 2026-07-22 16:21:48 +00:00 by brenno · 1 comment

brenno commented

2026-07-22 16:21:48 +00:00

Owner

Found in the pre-publication test review.

Evidence: test/golden/slide_golden_test.dart has nine tests over eight types (title, section, bullets, twoBullets, table, quote, code, image, plus a TLP variant). lib/models/slide.dart:19-48 defines 24. The goldens are also excluded from the main gate — Makefile:126, --exclude-tags golden — so even those eight run only if someone types make test-golden.

Why this matters now: every type built after the first round — chart, cockpit, timeline, scorecard, finding, checklist, scopeMatrix, discoveries, findingsSummary, question — has no visual regression test. A theme change or an adjustment in SlidePreviewWidget can shift their layout with nothing turning red and make check staying green. This review found a Scorecard slide rendering entirely blank (separate issue); a golden would have caught it. docs/CHECKS.md describes the goldens as guarding "per slide type", which is not currently true.

Proposal: for (final type in SlideType.values) in the golden file with one representative fixture per type, then run UPDATE=1 once. Until that exists, replace "each slide type" in docs/CHECKS.md with the eight names.

Found in the pre-publication test review. **Evidence:** `test/golden/slide_golden_test.dart` has nine tests over eight types (`title, section, bullets, twoBullets, table, quote, code, image`, plus a TLP variant). `lib/models/slide.dart:19-48` defines 24. The goldens are also excluded from the main gate — `Makefile:126`, `--exclude-tags golden` — so even those eight run only if someone types `make test-golden`. **Why this matters now:** every type built after the first round — `chart, cockpit, timeline, scorecard, finding, checklist, scopeMatrix, discoveries, findingsSummary, question` — has no visual regression test. A theme change or an adjustment in `SlidePreviewWidget` can shift their layout with nothing turning red and `make check` staying green. This review found a Scorecard slide rendering entirely blank (separate issue); a golden would have caught it. `docs/CHECKS.md` describes the goldens as guarding "per slide type", which is not currently true. **Proposal:** `for (final type in SlideType.values)` in the golden file with one representative fixture per type, then run `UPDATE=1` once. Until that exists, replace "each slide type" in `docs/CHECKS.md` with the eight names.

brenno added the

enhancement

label

2026-07-22 16:21:48 +00:00

brenno added the

in-progress

label

2026-07-22 21:44:03 +00:00

brenno referenced this issue

2026-07-22 21:47:05 +00:00

test(golden): een golden voor elk van de 24 slidetypes (#617) #696

brenno referenced this issue from a commit

2026-07-22 21:47:05 +00:00

test(golden): een golden voor elk van de 24 slidetypes (#617)

brenno referenced this issue from a commit

2026-07-22 21:47:22 +00:00

Merge pull request 'test(golden): een golden voor elk van de 24 slidetypes (#617)' (#696) from test/goldens-alle-slidetypes-617 into main

brenno removed the

in-progress

label

2026-07-22 21:47:22 +00:00

brenno commented

2026-07-22 21:47:23 +00:00

Author

Owner

Opgelost in PR #696, op main.

De lus loopt over SlideType.values, met dezelfde fixture die de markdown-ronde-trip en sinds #615 ook de rasterizer gebruiken. Belangrijker dan de zestien PNG's die erbij komen: een nieuw slidetype krijgt zijn golden nu zonder dat iemand eraan hoeft te denken. Dat was het echte gat — de acht die er waren, waren de acht van de eerste ronde.

De negen bestaande tests blijven staan; die dragen rijkere inhoud dan een standaardfixture kan (een TLP-markering met watermerk, een ontbrekende afbeelding). De lus is de ondergrens, niet de vervanging.

Wat ik bewust niet heb gedaan: de goldens in make check trekken. Je noemt de uitsluiting terecht, maar een pixelvergelijking in de dagelijkse poort faalt op elke machine behalve deze — dan is de poort niet strenger maar onbruikbaar voor een tweede ontwikkelaar. CHECKS.md beschrijft de route die dit wél oplost: een single-platform CI-job zodra er een runner is, en dat hangt aan #592.

CHECKS.md is verder rechtgezet: er stond "each slide type" terwijl het er acht waren, en de twee kanttekeningen staan er nu bij.

Opgelost in PR #696, op main. De lus loopt over `SlideType.values`, met dezelfde fixture die de markdown-ronde-trip en sinds #615 ook de rasterizer gebruiken. Belangrijker dan de zestien PNG's die erbij komen: **een nieuw slidetype krijgt zijn golden nu zonder dat iemand eraan hoeft te denken.** Dat was het echte gat — de acht die er waren, waren de acht van de eerste ronde. De negen bestaande tests blijven staan; die dragen rijkere inhoud dan een standaardfixture kan (een TLP-markering met watermerk, een ontbrekende afbeelding). De lus is de ondergrens, niet de vervanging. **Wat ik bewust niet heb gedaan: de goldens in `make check` trekken.** Je noemt de uitsluiting terecht, maar een pixelvergelijking in de dagelijkse poort faalt op elke machine behalve deze — dan is de poort niet strenger maar onbruikbaar voor een tweede ontwikkelaar. `CHECKS.md` beschrijft de route die dit wél oplost: een single-platform CI-job zodra er een runner is, en dat hangt aan #592. `CHECKS.md` is verder rechtgezet: er stond "each slide type" terwijl het er acht waren, en de twee kanttekeningen staan er nu bij.

brenno closed this issue

2026-07-22 21:47:23 +00:00

brenno referenced this issue

2026-07-22 22:01:28 +00:00

fix(ux): het polijstwerk uit #646 — vinkje, exportpad en de naam van de uitgever #700