Thread.sleep used in some unit tests; consider replacing with Awaitility

apache / pulsar-adapters

Apache Pulsar Adapters

https://pulsar.apache.org/

24 stars 31 forks source link

Thread.sleep used in some unit tests; consider replacing with Awaitility #23

Open dlg99 opened 3 years ago

dlg99 commented 3 years ago

Thread.sleep used in some unit tests (search code for it); consider replacing with Awaitility.

This is a follow up on https://github.com/apache/pulsar-adapters/pull/22#discussion_r654194531

Priority is fairly low IMO - occurences of this are very limited, test run reasonably fast AFAICT, it's unclear what specifically the tests have to wait for etc.

mattisonchao commented 3 years ago

I will fix it @dlg99