-
Notifications
You must be signed in to change notification settings - Fork 223
fix(test): add Docker skip guards + stabilize flaky tests under load #677
Copy link
Copy link
Labels
go:needs-researchNeeds investigationNeeds investigationsquadSquad triage inbox — Lead will assign to a memberSquad triage inbox — Lead will assign to a membersquad:fidoAssigned to FIDO (Quality Owner)Assigned to FIDO (Quality Owner)type:choreMaintenance, refactoring, cleanupMaintenance, refactoring, cleanup
Description
What
Add Docker availability skip guards and timeout adjustments to fix flaky tests under full-suite load.
Why
Issue #582 reports ~24 intermittent test failures in two categories:
- Docker-dependent tests (~15) fail when Docker Desktop isn't running
- Timing/resource contention tests (~9) fail under parallel load but pass in isolation
This affects CI reliability for everyone — false negatives on PRs waste reviewer time.
Proposed Fix
- Add
describe.skipIf(!dockerAvailable)guard to Docker-dependent test files - Increase timeout margins for speed-gate tests
- Add retry logic for
ENOTEMPTYcleanup errors
Related
- Fixes Flaky tests under full-suite load + Docker-dependent tests need skip guard #582
- Filed by @diberry
- Assigned to FIDO (quality owner)
Acceptance Criteria
- Docker-dependent tests skip gracefully when Docker is unavailable
- Speed-gate tests pass reliably under full parallel load
-
npm testshows 0 flaky failures on 3 consecutive runs
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
go:needs-researchNeeds investigationNeeds investigationsquadSquad triage inbox — Lead will assign to a memberSquad triage inbox — Lead will assign to a membersquad:fidoAssigned to FIDO (Quality Owner)Assigned to FIDO (Quality Owner)type:choreMaintenance, refactoring, cleanupMaintenance, refactoring, cleanup