Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix: Flaky tests #320

Merged
merged 2 commits into from
Aug 11, 2023
Merged

Fix: Flaky tests #320

merged 2 commits into from
Aug 11, 2023

Conversation

scohen
Copy link
Collaborator

@scohen scohen commented Aug 11, 2023

This PR fixes a couple of our flaky unit tests. Thank you to whomever put the seed in the output of ExUnit, that was what was needed to reproduce the flakiness.

Fixes #279

The two tests in this app were stomping on each others'
configurations, and would occasionally cause spurious messages to be
sent to one another.

Deregistering and clearing config on exit fixes the problems I also
made the project URI a little more "real"
@scohen scohen changed the title Flaky tests Fix: Flaky tests Aug 11, 2023
The property test would occasionally fail with a
"TooManyDuplicatesError". I bumped up the initial generation size to
50 and picked the ranges from a larger list, which seems to have
alleviated the problem.
@scohen scohen merged commit d8a85f0 into main Aug 11, 2023
5 checks passed
@scohen scohen deleted the flaky_tests branch August 11, 2023 21:51
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.

Coordinator test is flaky
1 participant