test(export): laat elk slidetype door de rasterizer, niet negen eromheen (#615) #686

Merged

brenno merged 1 commit from test/export-alle-slidetypes-615 into main 2026-07-22 20:42:03 +00:00

Conversation 0 Commits 1 Files changed 3 +82 -45

brenno commented 2026-07-22 20:41:50 +00:00

Owner

Copy link

Sluit #615.

Negen van de 24 types kwamen door géén enkele exporttest. De test loopt nu uitputtend over SlideType.values en toetst per type dat er werkelijk iets getekend is — meer dan één kleur in het beeld.

Waarom in de rasterizer en niet in de exportservice. Die service krijgt al gerenderde PNG's aangereikt; de rasterizer is de enige weg naar PDF en PPTX. Daar moet "lege pagina bij de ontvanger" zich laten vangen, niet een laag hoger.

Vandaag rastert elk type iets — dit is bewaking, geen reparatie. Maar wel eerst rood laten worden: met een vers aangemaakte scorecard-dia, precies het geval uit #583, valt hij om met "een scorecard-dia rastert als een egale vlakte — dat is een lege pagina in de PDF en de PPTX". Zonder die stap zou ik niet weten of _distinctColours > 1 scherp genoeg is.

De fixture is gedeeld. markdown_round_trip_test.dart liep als enige al uitputtend over het enum en had daarvoor een _metInhoud(type)-helper. Die staat nu in test/slide_fixtures.dart en wordt door beide gebruikt. Eén lijst, want twee lijsten lopen uiteen — en dan dekt de ene een type dat de andere stilzwijgend overslaat, en dat is precies de vorm van deze bug.

Poort

make check groen (niet door tail gepijpt).

Sluit #615. Negen van de 24 types kwamen door géén enkele exporttest. De test loopt nu uitputtend over `SlideType.values` en toetst per type dat er werkelijk iets getekend is — meer dan één kleur in het beeld. **Waarom in de rasterizer en niet in de exportservice.** Die service krijgt al gerenderde PNG's aangereikt; de rasterizer is de enige weg naar PDF en PPTX. Daar moet "lege pagina bij de ontvanger" zich laten vangen, niet een laag hoger. **Vandaag rastert elk type iets** — dit is bewaking, geen reparatie. Maar wel eerst rood laten worden: met een vers aangemaakte scorecard-dia, precies het geval uit #583, valt hij om met *"een scorecard-dia rastert als een egale vlakte — dat is een lege pagina in de PDF en de PPTX"*. Zonder die stap zou ik niet weten of `_distinctColours > 1` scherp genoeg is. **De fixture is gedeeld.** `markdown_round_trip_test.dart` liep als enige al uitputtend over het enum en had daarvoor een `_metInhoud(type)`-helper. Die staat nu in `test/slide_fixtures.dart` en wordt door beide gebruikt. Eén lijst, want twee lijsten lopen uiteen — en dan dekt de ene een type dat de andere stilzwijgend overslaat, en dat is precies de vorm van deze bug. ## Poort `make check` groen (niet door `tail` gepijpt).

brenno added 1 commit 2026-07-22 20:41:50 +00:00

test(export): laat elk slidetype door de rasterizer, niet negen eromheen (#615) ... 1adfb3fc19

Negen van de 24 types — section, twoBullets, twoImages, code, cockpit,
question, timeline, finding, signOff — kwamen door géén enkele exporttest.
Export is wat de gebruiker uit OciDeck meeneemt, dus een type dat als lege
pagina rastert is een fout die hij als eerste ziet in plaats van de suite.

De test loopt uitputtend over `SlideType.values` en toetst per type dat er
werkelijk iets getekend is: meer dan één kleur in het beeld. De rasterizer
is de enige weg naar PDF en PPTX, dus dit is de plek waar "lege pagina bij
de ontvanger" zich moet laten vangen — niet in de exportservice, die
al gerenderde PNG's aangereikt krijgt.

Vandaag rastert elk type iets; de test is de bewaking, geen reparatie. Wel
eerst rood laten worden: met een vers aangemaakte scorecard-dia — precies
het geval uit #583 — valt hij om met "rastert als een egale vlakte".

De fixture komt uit `markdown_round_trip_test.dart`, dat als enige al
uitputtend over het enum liep. Die staat nu in `test/slide_fixtures.dart`,
gedeeld door beide. Eén lijst, want twee lijsten lopen uiteen — en dan
dekt de ene een type dat de andere stilzwijgend overslaat, wat precies de
vorm van deze bug is.

Sluit #615.

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

brenno merged commit 3aed187b16 into main 2026-07-22 20:42:03 +00:00

brenno referenced this pull request from a commit 2026-07-22 20:42:04 +00:00

Merge pull request 'test(export): laat elk slidetype door de rasterizer, niet negen eromheen (#615)' (#686) from test/export-alle-slidetypes-615 into main

brenno referenced this pull request 2026-07-22 20:42:04 +00:00

[Bug] Nine of 24 slide types never pass through an export test #615

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

accepted

Assessed and will be built

  

bug

Something is broken

  

declined

Assessed and will not be built

  

docs

Documentation only

  

duplicate

Already tracked elsewhere

  

enhancement

New feature or improvement request

  

good first issue

Small, self-contained, and a good place to start

  

in-progress

Someone is working on this right now. Do not start it in parallel.

  

needs-info

Waiting for the reporter

  

privacy

Personal data can leak. Handled before anything else.

  

security

A weakness someone could use. Handled before anything else.

  

triage

Awaiting assessment

No labels

accepted

bug

declined

docs

duplicate

enhancement

good first issue

in-progress

needs-info

privacy

security

triage

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

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

LibreKAT/Ocideck!686

No description provided.