Add a DEBUG-gated testAwareDelay helper (same ICCERY_UI_TESTING=1 convention as AppPaths.testRoot) and apply it to the ensureNotRunning 100ms poll and the XY chartread 500ms pre-kill wait. Under UI tests the intervals shrink ~10x; release builds compile the #if DEBUG branch out entirely, so production timing is unchanged.
No current UI test sets MOCK_CHARTREAD_MODE=xy, so the 500ms site is future-proofing at zero risk today.
Verification
build-for-testing succeeds (host arch).
ICCeryCoreTests: 339 tests, 0 failures (unit tests don't set the flag, so behavior is identical).
Stacked on #143 — retarget to milestone/m9-monterey once #142 and #143 land.
Closes #141
Add a DEBUG-gated `testAwareDelay` helper (same `ICCERY_UI_TESTING=1` convention as `AppPaths.testRoot`) and apply it to the `ensureNotRunning` 100ms poll and the XY chartread 500ms pre-kill wait. Under UI tests the intervals shrink ~10x; release builds compile the `#if DEBUG` branch out entirely, so production timing is unchanged.
No current UI test sets `MOCK_CHARTREAD_MODE=xy`, so the 500ms site is future-proofing at zero risk today.
## Verification
- `build-for-testing` succeeds (host arch).
- ICCeryCoreTests: 339 tests, 0 failures (unit tests don't set the flag, so behavior is identical).
Stacked on #143 — retarget to `milestone/m9-monterey` once #142 and #143 land.
Add a DEBUG-gated testAwareDelay helper (same ICCERY_UI_TESTING=1
convention as AppPaths.testRoot) and apply it to the ensureNotRunning
100ms poll and the XY chartread 500ms pre-kill wait. Under UI tests the
intervals shrink ~10x; release builds compile the branch out entirely,
so production timing is unchanged. No current UI test exercises XY
cancel, so the 500ms site is future-proofing at zero risk today.
Generated with [Devin](https://devin.ai)
Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
gronod
merged commit 67fb3a9b37 into milestone/m9-monterey2026-09-12 17:00:48 +01:00
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Closes #141
Add a DEBUG-gated
testAwareDelayhelper (sameICCERY_UI_TESTING=1convention asAppPaths.testRoot) and apply it to theensureNotRunning100ms poll and the XY chartread 500ms pre-kill wait. Under UI tests the intervals shrink ~10x; release builds compile the#if DEBUGbranch out entirely, so production timing is unchanged.No current UI test sets
MOCK_CHARTREAD_MODE=xy, so the 500ms site is future-proofing at zero risk today.Verification
build-for-testingsucceeds (host arch).Stacked on #143 — retarget to
milestone/m9-montereyonce #142 and #143 land.