Skip to content

poll: fix flaky TestWaitOn_WithCompare - #296

Open
thaJeztah wants to merge 1 commit into
gotestyourself:mainfrom
thaJeztah:deflake_TestWaitOn_WithCompare
Open

thaJeztah wants to merge 1 commit into
gotestyourself:mainfrom
thaJeztah:deflake_TestWaitOn_WithCompare

Conversation

@thaJeztah

Copy link
Copy Markdown
Contributor

Increase the timeout in TestWaitOn_WithCompare to avoid racing the first poll attempt on slow or heavily loaded systems.

It was flaky in CI;

=== Failed
=== FAIL: poll TestWaitOn_WithCompare (0.01s)
    poll_test.go:113: assertion failed: string "timeout hit after 10ms: first check never completed" does not contain "assertion failed: 3 (int) != 4 (int)"

Increase the timeout in TestWaitOn_WithCompare to avoid racing the first
poll attempt on slow or heavily loaded systems.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah

Copy link
Copy Markdown
Contributor Author

cc @dnephin @vdemeester PTAL 🤗 (👋)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant