fix: verhoog polling-budget save-as-test van 500ms naar 5s (#1363) #1364

Merged

brenno merged 1 commit from fix/1363-save-as-test-flaky into main

2026-08-08 11:08:08 +00:00

brenno commented

2026-08-08 10:40:35 +00:00

Owner

Samenvatting

De test Cmd+S op een document zonder pad valt terug op Opslaan als… faalde intermitterend op de Forgejo linux-gate: het polling-budget van 500ms op filePath was te krap onder I/O-contentie (vier containers op één dind).
Neemt het bewezen 5s-budget (500×10ms) over van de zuster-test in document_editor_screen_test.dart, die exact hetzelfde patroon met dezelfde atomic-write al gebruikt sinds een eerdere fix.
Geen productiewijziging — de productiecode is correct; alleen de test-polling was te krap.

Waarom geen regressietest

Dit is zelf de testfix. Er is geen productiebug om vast te houden — de _save()-flow werkt correct, de test wachtte alleen niet lang genoeg op de async write + state-update.

Testplan

flutter test test/document_new_and_save_as_test.dart --test-randomize-ordering-seed random — 4/4 groen
make check — 8963 tests, 87.1% dekking, alle poorten groen
make check-secrets — 0 geheimen
make sast — 0 findingen

Closes #1363

Generated with Devin

## Samenvatting - De test `Cmd+S op een document zonder pad valt terug op Opslaan als…` faalde intermitterend op de Forgejo linux-gate: het polling-budget van 500ms op `filePath` was te krap onder I/O-contentie (vier containers op één dind). - Neemt het bewezen 5s-budget (500×10ms) over van de zuster-test in `document_editor_screen_test.dart`, die exact hetzelfde patroon met dezelfde atomic-write al gebruikt sinds een eerdere fix. - Geen productiewijziging — de productiecode is correct; alleen de test-polling was te krap. ## Waarom geen regressietest Dit is zelf de testfix. Er is geen productiebug om vast te houden — de `_save()`-flow werkt correct, de test wachtte alleen niet lang genoeg op de async write + state-update. #### Testplan - [x] `flutter test test/document_new_and_save_as_test.dart --test-randomize-ordering-seed random` — 4/4 groen - [x] `make check` — 8963 tests, 87.1% dekking, alle poorten groen - [x] `make check-secrets` — 0 geheimen - [x] `make sast` — 0 findingen Closes #1363 Generated with [Devin](https://devin.ai)

brenno added 1 commit

2026-08-08 10:40:35 +00:00

fix: verhoog polling-budget save-as-test van 500ms naar 5s (#1363 )

scans / scans (pull_request) Successful in 1m51s

Details

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

Details

linux-gate / gate-linux (pull_request) Successful in 28m0s

Details

79feb9a4e3

De Cmd+S-op-een-document-zonder-pad-test pollde 500ms op filePath, te
krap voor de I/O-contentie op de Forgejo linux-gate (vier containers op
één dind). Neemt het bewezen 5s-budget over van de zuster-test in
document_editor_screen_test.dart, die hetzelfde patroon met dezelfde
write al gebruikt.

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

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

brenno merged commit 265b7b7309 into main

2026-08-08 11:08:08 +00:00

brenno referenced this pull request from a commit

2026-08-08 11:08:10 +00:00

Merge pull request 'fix: verhoog polling-budget save-as-test van 500ms naar 5s (#1363)' (#1364) from fix/1363-save-as-test-flaky into main