ci(privacy): de beeldtests draaiden nergens echt #364

Merged

brenno merged 1 commit from ci/dartcv-lib-path into main 2026-07-19 15:51:46 +00:00

Conversation 0 Commits 1 Files changed 3 +57

brenno commented 2026-07-19 15:51:37 +00:00

Owner

Copy link

Vervolg op #363, dat de beeldcontrole introduceerde. De detectietests daarvan sloegen zichzelf overal over — lokaal én in CI — en stonden dus groen zonder iets te bewaken.

Waarom

De beeldcontrole draait op OpenCV via FFI, en die native bibliotheek zit in de app-bundel. Een kale flutter test op de Dart-VM heeft haar niet.

Dat is geen theoretisch risico. Tijdens het bouwen van de feature stonden precies deze tests één keer groen terwijl de bibliotheek niet laadde: elke aanroep viel in de foutafhandeling en de scanner meldde nul gezichten. Groen om de verkeerde reden is erger dan rood, en dat is nu net het gedrag waar deze hele controle tegen ontworpen is.

Wat er verandert

Makefile — zoekt de bibliotheek zelf op in build/ en exporteert DARTCV_LIB_PATH. Wie ooit een platformbuild heeft gedaan, draait de detectietests dus vanzelf echt; niemand hoeft een variabele te onthouden. Gezocht op bestandsnaam en niet op een vast pad, want dat verschilt per platform (framework / .so / .dll) én per bouwmodus.

CI — alleen de macOS-taak bouwt. Eén platform waar ze werkelijk draaien volstaat om een regressie in de detector te vangen, en een Linux-build zou GTK-pakketten op de runner vragen voor dezelfde twee tests. Die stap faalt hard als de bibliotheek niet staat waar hij wordt verwacht — anders valt een toekomstige opencv_core met een gewijzigd pad stilletjes terug op overslaan, en zijn we terug bij het probleem dat deze PR oplost.

CHECKS.md — beschrijft dat deze ene suite een platformbuild nodig heeft, en waarom.

Gemeten

flutter test test/image_face_scan_test.dart
  zonder:  +7 ~2   (twee overgeslagen)
  met:     +9      (alles draait)

De regeldekking van de volledige suite stijgt mee, van 35.940 naar 35.980 regels. make check groen.

Kosten

De macOS-taak krijgt er een debug-build bij, lokaal ongeveer vier minuten. Dat is de prijs voor twee tests die de precisie van een gezichtsdetector bewaken — een detector die op elke huisdierfoto zou aanslaan als hij stukging, zonder dat iemand het merkte.

🤖 Generated with Claude Code

Vervolg op #363, dat de beeldcontrole introduceerde. De detectietests daarvan sloegen zichzelf **overal** over — lokaal én in CI — en stonden dus groen zonder iets te bewaken. ## Waarom De beeldcontrole draait op OpenCV via FFI, en die native bibliotheek zit in de app-bundel. Een kale `flutter test` op de Dart-VM heeft haar niet. Dat is geen theoretisch risico. Tijdens het bouwen van de feature stonden precies deze tests één keer groen terwijl de bibliotheek niet laadde: elke aanroep viel in de foutafhandeling en de scanner meldde nul gezichten. Groen om de verkeerde reden is erger dan rood, en dat is nu net het gedrag waar deze hele controle tegen ontworpen is. ## Wat er verandert **Makefile** — zoekt de bibliotheek zelf op in `build/` en exporteert `DARTCV_LIB_PATH`. Wie ooit een platformbuild heeft gedaan, draait de detectietests dus vanzelf echt; niemand hoeft een variabele te onthouden. Gezocht op bestandsnaam en niet op een vast pad, want dat verschilt per platform (framework / `.so` / `.dll`) én per bouwmodus. **CI** — alleen de macOS-taak bouwt. Eén platform waar ze werkelijk draaien volstaat om een regressie in de detector te vangen, en een Linux-build zou GTK-pakketten op de runner vragen voor dezelfde twee tests. Die stap **faalt hard** als de bibliotheek niet staat waar hij wordt verwacht — anders valt een toekomstige `opencv_core` met een gewijzigd pad stilletjes terug op overslaan, en zijn we terug bij het probleem dat deze PR oplost. **CHECKS.md** — beschrijft dat deze ene suite een platformbuild nodig heeft, en waarom. ## Gemeten ``` flutter test test/image_face_scan_test.dart zonder: +7 ~2 (twee overgeslagen) met: +9 (alles draait) ``` De regeldekking van de volledige suite stijgt mee, van 35.940 naar 35.980 regels. `make check` groen. ## Kosten De macOS-taak krijgt er een debug-build bij, lokaal ongeveer vier minuten. Dat is de prijs voor twee tests die de precisie van een gezichtsdetector bewaken — een detector die op elke huisdierfoto zou aanslaan als hij stukging, zonder dat iemand het merkte. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

brenno added 1 commit 2026-07-19 15:51:37 +00:00

ci(privacy): de beeldtests draaiden nergens echt ... f90453d56d

De detectietests voor de beeldcontrole sloegen zichzelf overal over — lokaal
én in CI. Ze stonden groen zonder iets te bewaken.

De oorzaak is structureel: die controle draait op OpenCV via FFI, en de
native bibliotheek zit in de app-bundel. Een kale `flutter test` op de
Dart-VM heeft haar niet. En dat is geen theoretisch risico: tijdens de bouw
van deze feature stonden die tests één keer groen terwijl de bibliotheek
niet laadde, elke aanroep in de foutafhandeling viel en de scanner nul
gezichten meldde. Groen om de verkeerde reden is erger dan rood.

`dartcv` leest het pad uit DARTCV_LIB_PATH. In plaats van iedereen die
variabele te laten onthouden, zoekt de Makefile de bibliotheek nu zelf op in
`build/` en zet hem. Wie ooit een platformbuild heeft gedaan, draait de
detectietests dus vanzelf echt. Gezocht op bestandsnaam en niet op een vast
pad, want dat verschilt per platform (framework, `.so`, `.dll`) én per
bouwmodus.

In CI bouwt alleen de macOS-taak. Eén platform waar ze werkelijk draaien
volstaat om een regressie in de detector te vangen, en een Linux-build zou
GTK-pakketten op de runner vragen voor dezelfde twee tests. Die stap faalt
hard als de bibliotheek niet staat waar hij wordt verwacht — anders valt een
toekomstige opencv_core met een ander pad stilletjes terug op overslaan, en
zijn we terug bij het probleem dat deze commit oplost.

Meetbaar: `flutter test test/image_face_scan_test.dart` gaf `+7 ~2`
(twee overgeslagen) en geeft nu `+9`. De regeldekking van de suite stijgt
mee, van 35.940 naar 35.980.

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

brenno merged commit 300a37bc86 into main 2026-07-19 15:51:46 +00:00

brenno referenced this pull request from a commit 2026-07-19 15:51:47 +00:00

Merge pull request "ci(privacy): de beeldtests draaiden nergens echt" (#364) from ci/dartcv-lib-path into main

brenno referenced this pull request 2026-07-19 16:05:43 +00:00

docs(ociwacht): de losse einden vastgelegd in plaats van in een gesprek #367

brenno referenced this pull request 2026-07-19 16:12:27 +00:00

docs(ociwacht): het DLP-onderzoek vastgelegd als naslag #369

brenno referenced this pull request from a commit 2026-07-19 16:53:26 +00:00

ci(privacy): de beeldtests draaien nu ook op Linux en Windows

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.

  

release-blocker

Must land before the next release. scripts/release_auto.sh refuses to start while this is open.

  

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

release-blocker

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!364

No description provided.