[ https://issues.apache.org/jira/browse/CAMEL-19547?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17738582#comment-17738582 ]
Nicolas Filotto commented on CAMEL-19547: ----------------------------------------- Are you sure that it will help to fix the flakiness of the test? Personally, I would expect to have a condition tested by Awaitility otherwise I'm afraid we will get the same behavior but with a more complex code, don't you agree? > camel-xmpp: replace Thread.sleep in tests > ----------------------------------------- > > Key: CAMEL-19547 > URL: https://issues.apache.org/jira/browse/CAMEL-19547 > Project: Camel > Issue Type: Task > Components: camel-xmpp, tests > Affects Versions: 4.0.0 > Reporter: Otavio Rodolfo Piske > Priority: Minor > Labels: easy, help-wanted > > We have many tests which use Thread.sleep for synchronization. This is bug > prone and can introduce flakiness when running on environments with different > capacities. > Ideally we should replace these with: > * [Awaitility|http://www.awaitility.org/] > * Java's native syncronization mechanism (Latches, Phasers, Locks, etc) > * Nothing (i.e.; in some cases the sleep can simply be removed) > > -- This message was sent by Atlassian Jira (v8.20.10#820010)