Test timing delays increased to fix flakiness
Workflow publish history integration tests just got more reliable — setTimeout delays bumped from 1ms to 5ms to prevent race conditions.
Integration tests in the workflow publish history module were occasionally failing due to tight timing gaps between test assertions. A 1ms delay between record insertions was too short, causing intermittent flakiness that wastes CI time and erodes confidence in the test suite. Delays have been increased to 5ms, giving the test environment enough breathing room to execute reliably. This fix lives in the WorkflowPublishHistoryRepository integration tests within the core package.
View Original GitHub Description
Summary
I saw this flake recently, so bumping the time a bit. This is obviously not great, but relatively common in our codebase.
Increases setTimeout delays from 1ms to 5ms in WorkflowPublishHistoryRepository integration tests to fix flaky failures caused by insufficient timing gaps between record insertions.
Review / Merge checklist
- I have seen this code, I have run this code, and I take responsibility for this code.
- PR title and summary are descriptive. (conventions)
- Docs updated or follow-up ticket created.
- Tests included.
- PR Labeled with
release/backport(if the PR is an urgent fix that needs to be backported)