Skip to content

audit F47 [quality]: worker_cache test uses std::thread::sleep — flake risk #547

@intendednull

Description

@intendednull

File: crates/client/src/worker_cache.rs:176
Severity: quality
Obvious? yes

evict_stale_removes_expired test sleeps 10ms after setting a 1ms TTL. CLAUDE.md's condition-based-waiting skill explicitly flags this as flakey on slow CI.

Fix: inject a clock or expose a cutoff_override for the test to remove the timing dependency entirely.


Filed by /general-audit @ b901575 (2026-05-02). master: #513.

Metadata

Metadata

Assignees

No one assigned

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions