Skip to content

test: fix flaky integration tests#93

Merged
dhunt3800 merged 7 commits into
mainfrom
pr-flaky-integration-tests
May 27, 2026
Merged

test: fix flaky integration tests#93
dhunt3800 merged 7 commits into
mainfrom
pr-flaky-integration-tests

Conversation

@dhunt3800
Copy link
Copy Markdown
Contributor

No description provided.

@dhunt3800 dhunt3800 self-assigned this May 21, 2026
dhunt3800 added 2 commits May 21, 2026 18:33
Signed-off-by: Dane Hunt <dhunt3800@gmail.com>
Signed-off-by: Dane Hunt <dhunt3800@gmail.com>
@dhunt3800 dhunt3800 force-pushed the pr-flaky-integration-tests branch from ba505ff to bee2be9 Compare May 21, 2026 22:33
@dhunt3800 dhunt3800 linked an issue May 21, 2026 that may be closed by this pull request
dhunt3800 added 2 commits May 22, 2026 12:17
Signed-off-by: Dane Hunt <dhunt3800@gmail.com>
Signed-off-by: Dane Hunt <dhunt3800@gmail.com>
@dhunt3800 dhunt3800 force-pushed the pr-flaky-integration-tests branch from 81df72d to fc24afa Compare May 22, 2026 16:34
dhunt3800 added 2 commits May 22, 2026 12:41
Signed-off-by: Dane Hunt <dhunt3800@gmail.com>
Signed-off-by: Dane Hunt <dhunt3800@gmail.com>
@dhunt3800 dhunt3800 marked this pull request as ready for review May 22, 2026 17:01
Signed-off-by: Dane Hunt <dhunt3800@gmail.com>
@dhunt3800 dhunt3800 force-pushed the pr-flaky-integration-tests branch from 241b14b to 1a28b78 Compare May 27, 2026 13:59
@rsperl
Copy link
Copy Markdown
Contributor

rsperl commented May 27, 2026

The only test that didn't pass the first time was TestProduceSingleConsumeRetry, and it passed when run again by itself. The only change I would suggest is using t.Log* instead of the existing log.Print* and fmt.Print*, but that's really not a part of the PR and really minor.

@dhunt3800 dhunt3800 merged commit b77e51c into main May 27, 2026
4 checks passed
@dhunt3800 dhunt3800 deleted the pr-flaky-integration-tests branch May 27, 2026 20:20
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.

Fix root cause of integration tests failing locally first run

3 participants