fix(test): HTML-fallback-test gebruikt een comment, niet een tag (#1777) #1785

Merged

brenno merged 1 commit from fix/ci-html-fallback-test-1777 into main

2026-08-25 06:44:52 +00:00

brenno commented

2026-08-25 06:36:00 +00:00

Owner

Wat

De integratietest "Visueel: rauwe HTML schakelt automatisch naar Bron-modus met uitleg" in document_editor_screen_test.dart faalde op alle drie de CI-jobs (macOS, Windows, Linux) sinds #1777.

Waarom

#1777 veranderde _rawHtml van <!--|</?[A-Za-z][^>]*> naar alleen <!-- — HTML-tags round-trippen als platte tekst en zijn geen beperking meer, alleen HTML-commentaar triggert nog de fallback. De test gebruikte nog <div>rauwe html</div> (een tag) en verwachtte fallback naar Bron-modus, maar dat gebeurt niet meer.

De unit tests in markdown_visual_compatibility_test.dart waren in #1777 al aangepast; deze integratietest was gemist.

Fix

Vervang <div>rauwe html</div> door  (een HTML-comment die nog steeds fallback triggert). De testnaam en intentie blijven gelijk: een constructie die de brug niet verliesvrij aankan valt terug op Bron-modus met uitleg.

Bewaker

Overgeslagen (expliciet): test-only fix, raakt geen bestandsformaat, opslag, afhankelijkheid, uitgaand verkeer of publieke belofte.

Test plan

flutter test test/document_editor_screen_test.dart — 47 tests groen
make check — 10569 tests, 87.1% coverage
make check-secrets — 0 leaks
make sast — 0 findings

Generated with Devin

## Wat De integratietest "Visueel: rauwe HTML schakelt automatisch naar Bron-modus met uitleg" in `document_editor_screen_test.dart` faalde op alle drie de CI-jobs (macOS, Windows, Linux) sinds #1777. ## Waarom #1777 veranderde `_rawHtml` van `` (een HTML-comment die nog steeds fallback triggert). De testnaam en intentie blijven gelijk: een constructie die de brug niet verliesvrij aankan valt terug op Bron-modus met uitleg. ## Bewaker Overgeslagen (expliciet): test-only fix, raakt geen bestandsformaat, opslag, afhankelijkheid, uitgaand verkeer of publieke belofte. #### Test plan - [x] `flutter test test/document_editor_screen_test.dart` — 47 tests groen - [x] `make check` — 10569 tests, 87.1% coverage - [x] `make check-secrets` — 0 leaks - [x] `make sast` — 0 findings Generated with [Devin](https://devin.ai)

brenno added 1 commit

2026-08-25 06:36:01 +00:00

fix(test): HTML-fallback-test gebruikt een comment, niet een tag (#1777 )

scans / scans (pull_request) Successful in 3m44s

Details

static-gate / static-gate (pull_request) Successful in 8m15s

Details

29f225cb4d

De integratietest "rauwe HTML schakelt automatisch naar Bron-modus" gebruikte
nog `<div>rauwe html</div>`, maar sinds #1777 round-trippen HTML-tags als
platte tekst en zijn ze geen beperking meer — alleen HTML-commentaar
(`<!-- … -->`) triggert nog de fallback. De test faalde daardoor op alle
drie de CI-jobs (macOS, Windows, Linux).

Vervang de tag door een comment. De testnaam en intentie blijven gelijk:
een constructie die de brug niet verliesvrij aankan valt terug op Bron-modus.

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>

brenno merged commit e65fb3e206 into main

2026-08-25 06:44:52 +00:00

brenno referenced this pull request from a commit

2026-08-25 06:44:52 +00:00

fix(test): HTML-fallback-test gebruikt een comment, niet een tag (#1785)