-
Notifications
You must be signed in to change notification settings - Fork 29
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
Upgraded to http4s 0.20.0 + Fixed flaky failing test. #22
Conversation
GeneralMetrics().activeRequests.distribution().min shouldBe 0L | ||
} | ||
|
||
reporter.clear() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
as far as I could tell it doesn't seem to be used in this test at all!
|
||
val test = IO { | ||
eventually(timeout(5 seconds)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Due to the way the test is setup, requests *> test
the test is only run after all requests have completed therefore there's no need to use eventually.
|
||
reporter.clear() | ||
|
||
GeneralMetrics().activeRequests.distribution().max should be > 1L |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Initial constraint was that it had to be =10 but it would some times fail
Bigger than 1 seems reasonable as it proves that one or more requests were active at the same time!
@cmcmteixeira done! Thanks for proposing this PR |
No description provided.