CI: native-scan-integratietest automatisch draaien + dode DARTCV_LIB_PATH-steiger opruimen #899

New issue

Closed

opened 2026-07-26 15:20:03 +00:00 by brenno · 1 comment

brenno commented

2026-07-26 15:20:03 +00:00

Owner

Volgt uit de native-scan-fix (branch fix/native-face-scan-integration-test, herstelt
FaceDetectorYN via de OpenCV-modules in pubspec).

Bij die fix is integration_test/native_face_scan_test.dart toegevoegd — de enige
plek waar de native gezichtsscan écht getoetst wordt, want onder flutter test laadt
dartcv4 2.x zijn code-assets niet. Lokaal is die alleen op macOS groen gedraaid.

Te doen

De integratietest automatisch in CI draaien, per desktopplatform:
- Linux op de forge (.forgejo/), met een xvfb-display (xvfb-run flutter test integration_test -d linux);
- macOS en Windows op de spiegel (.github/workflows/ci.yml, -d macos / -d windows).
  Dit is tegelijk het bewijs dat de modulebuild op Linux/Windows slaagt en de scan
  daar werkt.
De dode DARTCV_LIB_PATH-steiger opruimen. dartcv4 2.x leest die var nergens;
de per-platform "build + detect + set DARTCV_LIB_PATH"-stappen in ci.yml en het
DARTCV_LIB / export DARTCV_LIB_PATH-blok in de Makefile zijn zinloos geworden.
Houd de platform-flutter build als compile-rooktoets; verwijder de detectie/export.
(Hierdoor verviel PR #891, die alleen de macOS-detectie repareerde.)

Waarom apart

De fix zelf (kapotte privacy-functie) moest snel landen en is geverifieerd. De
CI-automatisering vraagt xvfb/runner-werk dat alleen op de spiegel te toetsen is.

Volgt uit de native-scan-fix (branch `fix/native-face-scan-integration-test`, herstelt FaceDetectorYN via de OpenCV-modules in pubspec). Bij die fix is `integration_test/native_face_scan_test.dart` toegevoegd — de enige plek waar de native gezichtsscan écht getoetst wordt, want onder `flutter test` laadt dartcv4 2.x zijn code-assets niet. Lokaal is die alleen op **macOS** groen gedraaid. ## Te doen 1. **De integratietest automatisch in CI draaien**, per desktopplatform: - Linux op de forge (`.forgejo/`), met een `xvfb`-display (`xvfb-run flutter test integration_test -d linux`); - macOS en Windows op de spiegel (`.github/workflows/ci.yml`, `-d macos` / `-d windows`). Dit is tegelijk het bewijs dat de modulebuild op Linux/Windows slaagt en de scan daar werkt. 2. **De dode DARTCV_LIB_PATH-steiger opruimen.** dartcv4 2.x leest die var nergens; de per-platform "build + detect + set DARTCV_LIB_PATH"-stappen in `ci.yml` en het `DARTCV_LIB` / `export DARTCV_LIB_PATH`-blok in de `Makefile` zijn zinloos geworden. Houd de platform-`flutter build` als compile-rooktoets; verwijder de detectie/export. (Hierdoor verviel PR #891, die alleen de macOS-detectie repareerde.) ## Waarom apart De fix zelf (kapotte privacy-functie) moest snel landen en is geverifieerd. De CI-automatisering vraagt xvfb/runner-werk dat alleen op de spiegel te toetsen is.

brenno added the

triage

in-progress

labels

2026-07-26 15:20:03 +00:00

brenno referenced this issue from a commit

2026-07-26 16:18:23 +00:00

ci: draai de native-scan-integratietest per platform + verwijder DARTCV_LIB_PATH-steiger (#899)

brenno referenced this issue

2026-07-26 16:18:43 +00:00

ci: native-scan-integratietest per platform + DARTCV_LIB_PATH-steiger verwijderen (#899) #901

brenno referenced this issue from a commit

2026-07-26 16:48:48 +00:00

Merge pull request 'ci: native-scan-integratietest per platform + DARTCV_LIB_PATH-steiger verwijderen (#899)' (#901) from fix/899-native-scan-ci into main

brenno commented

2026-07-26 16:49:04 +00:00

Author

Owner

Uitgevoerd en gemerged via #901 (merge 91bc0c70): de native-scan-integratietest draait nu per platform in de spiegel-CI (Linux via xvfb, macOS+Windows in de matrix — alle drie groen op run #197) en de dode DARTCV_LIB_PATH-steiger is verwijderd.

Uitgevoerd en gemerged via #901 (merge 91bc0c70): de native-scan-integratietest draait nu per platform in de spiegel-CI (Linux via xvfb, macOS+Windows in de matrix — alle drie groen op run #197) en de dode DARTCV_LIB_PATH-steiger is verwijderd.

brenno closed this issue

2026-07-26 16:49:04 +00:00