CalvinKirs opened a new pull request, #3392:
URL: https://github.com/apache/bookkeeper/pull/3392
### Descriptions of the changes in this PR:
Using Thread.sleep in a test is just generally a bad idea. It creates
brittle tests that can fail unpredictably depending on environment ("Passes on
my machine!") or load. So I used Awaitility for asynchroneous testing.
Such
as:https://github.com/apache/bookkeeper/runs/7219059790?check_suite_focus=true
(He may not be a perfect guarantee, but it can improve the stability ratio.)
### Motivation
Using `Thread.sleep` in a test is generally a bad idea. It creates brittle
tests that can fail unpredictably depending on the environment ("Passes on my
machine!") or load
### Changes
Useing `Awaitility` for asynchronous testing.
Useing `Awaitility` test `AutoRecoveryMainTest`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
