Two polling/wait intervals in ArgyllRunner (ICCeryCore) exist only to give real hardware time and add up across the ~40 UI-test launches:
ensureNotRunning poll — Packages/ICCeryCore/Sources/ICCeryCore/Argyll/ArgyllRunner.swift:203 — 100ms per attempt, up to 30 attempts, fires on every tool re-run via the duplicate-id guard (#50/#52).
XY chartread pre-kill wait — ArgyllRunner.swift:595 — 500ms Task.sleep after sending q\n before kill. No current UI test sets MOCK_CHARTREAD_MODE=xy, so this is future-proofing at zero risk today.
Change
Add a DEBUG-gated helper that reads the existing ICCERY_UI_TESTING=1 env var (same convention as AppPaths.testRoot in Packages/ICCeryCore/Sources/ICCeryCore/Paths/AppPaths.swift:54-71) and shrinks these intervals ~10x under tests:
/// DEBUG-only fast path: under `ICCERY_UI_TESTING=1` polling/wait/// intervals shrink — same env convention as `AppPaths.testRoot`.privatestaticfunctestAwareDelay(_nanos:UInt64)->UInt64{#ifDEBUGifProcessInfo.processInfo.environment["ICCERY_UI_TESTING"]=="1"{returnnanos/10}#endifreturnnanos}
Apply at the two sites: Task.sleep(nanoseconds: Self.testAwareDelay(100_000_000)) and ...Self.testAwareDelay(500_000_000).
Release builds compile the #if DEBUG branch out entirely — zero production impact. No mutable static state (env read per call).
Acceptance
ICCeryCoreTests unaffected (no behavior change for non-ICCERY_UI_TESTING builds).
Out of scope: SwiftUI animation disabling (no explicit animation code exists in Sources/ICCery).
## Problem
Two polling/wait intervals in `ArgyllRunner` (ICCeryCore) exist only to give real hardware time and add up across the ~40 UI-test launches:
- `ensureNotRunning` poll — `Packages/ICCeryCore/Sources/ICCeryCore/Argyll/ArgyllRunner.swift:203` — 100ms per attempt, up to 30 attempts, fires on every tool re-run via the duplicate-id guard (#50/#52).
- XY chartread pre-kill wait — `ArgyllRunner.swift:595` — 500ms `Task.sleep` after sending `q\n` before kill. No current UI test sets `MOCK_CHARTREAD_MODE=xy`, so this is future-proofing at zero risk today.
## Change
Add a DEBUG-gated helper that reads the existing `ICCERY_UI_TESTING=1` env var (same convention as `AppPaths.testRoot` in `Packages/ICCeryCore/Sources/ICCeryCore/Paths/AppPaths.swift:54-71`) and shrinks these intervals ~10x under tests:
```swift
/// DEBUG-only fast path: under `ICCERY_UI_TESTING=1` polling/wait
/// intervals shrink — same env convention as `AppPaths.testRoot`.
private static func testAwareDelay(_ nanos: UInt64) -> UInt64 {
#if DEBUG
if ProcessInfo.processInfo.environment["ICCERY_UI_TESTING"] == "1" {
return nanos / 10
}
#endif
return nanos
}
```
Apply at the two sites: `Task.sleep(nanoseconds: Self.testAwareDelay(100_000_000))` and `...Self.testAwareDelay(500_000_000)`.
Release builds compile the `#if DEBUG` branch out entirely — zero production impact. No mutable static state (env read per call).
## Acceptance
- ICCeryCoreTests unaffected (no behavior change for non-`ICCERY_UI_TESTING` builds).
- UI suite still green.
## Scope
- `Packages/ICCeryCore/Sources/ICCeryCore/Argyll/ArgyllRunner.swift` only.
- Out of scope: SwiftUI animation disabling (no explicit animation code exists in `Sources/ICCery`).
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.
Problem
Two polling/wait intervals in
ArgyllRunner(ICCeryCore) exist only to give real hardware time and add up across the ~40 UI-test launches:ensureNotRunningpoll —Packages/ICCeryCore/Sources/ICCeryCore/Argyll/ArgyllRunner.swift:203— 100ms per attempt, up to 30 attempts, fires on every tool re-run via the duplicate-id guard (#50/#52).ArgyllRunner.swift:595— 500msTask.sleepafter sendingq\nbefore kill. No current UI test setsMOCK_CHARTREAD_MODE=xy, so this is future-proofing at zero risk today.Change
Add a DEBUG-gated helper that reads the existing
ICCERY_UI_TESTING=1env var (same convention asAppPaths.testRootinPackages/ICCeryCore/Sources/ICCeryCore/Paths/AppPaths.swift:54-71) and shrinks these intervals ~10x under tests:Apply at the two sites:
Task.sleep(nanoseconds: Self.testAwareDelay(100_000_000))and...Self.testAwareDelay(500_000_000).Release builds compile the
#if DEBUGbranch out entirely — zero production impact. No mutable static state (env read per call).Acceptance
ICCERY_UI_TESTINGbuilds).Scope
Packages/ICCeryCore/Sources/ICCeryCore/Argyll/ArgyllRunner.swiftonly.Sources/ICCery).