This is an automated email from the ASF dual-hosted git repository.

orpiske pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git

commit da72108cc0b5ee56f091b45ba9e621ffa8376643
Author: Otavio Rodolfo Piske <angusyo...@gmail.com>
AuthorDate: Fri Jun 3 14:28:09 2022 +0200

    (chores) camel-ignite removed usages of Thread.sleep
---
 .../java/org/apache/camel/component/ignite/IgniteEventsTest.java  | 8 +++++---
 .../org/apache/camel/component/ignite/IgniteMessagingTest.java    | 6 ++++--
 2 files changed, 9 insertions(+), 5 deletions(-)

diff --git 
a/components/camel-ignite/src/test/java/org/apache/camel/component/ignite/IgniteEventsTest.java
 
b/components/camel-ignite/src/test/java/org/apache/camel/component/ignite/IgniteEventsTest.java
index b1b8a046dee..b5c55d0a224 100644
--- 
a/components/camel-ignite/src/test/java/org/apache/camel/component/ignite/IgniteEventsTest.java
+++ 
b/components/camel-ignite/src/test/java/org/apache/camel/component/ignite/IgniteEventsTest.java
@@ -34,10 +34,13 @@ import org.apache.ignite.configuration.IgniteConfiguration;
 import org.apache.ignite.events.Event;
 import org.apache.ignite.events.EventType;
 import org.assertj.core.api.Assertions;
+import org.hamcrest.Matchers;
 import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.TestInstance;
 
+import static org.awaitility.Awaitility.await;
+
 @TestInstance(TestInstance.Lifecycle.PER_CLASS)
 public class IgniteEventsTest extends AbstractIgniteTest {
 
@@ -71,9 +74,8 @@ public class IgniteEventsTest extends AbstractIgniteTest {
         cache.withExpiryPolicy(CreatedExpiryPolicy.factoryOf(new 
Duration(TimeUnit.MILLISECONDS, 100)).create())
                 .put(resourceUid, "123");
 
-        Thread.sleep(150);
-
-        cache.get(resourceUid);
+        await().atMost(150, TimeUnit.MILLISECONDS)
+                .until(() -> cache.get(resourceUid), Matchers.nullValue());
 
         assertMockEndpointsSatisfied();
 
diff --git 
a/components/camel-ignite/src/test/java/org/apache/camel/component/ignite/IgniteMessagingTest.java
 
b/components/camel-ignite/src/test/java/org/apache/camel/component/ignite/IgniteMessagingTest.java
index 60750d13545..0817b78cded 100644
--- 
a/components/camel-ignite/src/test/java/org/apache/camel/component/ignite/IgniteMessagingTest.java
+++ 
b/components/camel-ignite/src/test/java/org/apache/camel/component/ignite/IgniteMessagingTest.java
@@ -32,6 +32,7 @@ import org.apache.camel.Processor;
 import org.apache.camel.component.ignite.messaging.IgniteMessagingComponent;
 import org.apache.ignite.lang.IgniteBiPredicate;
 import org.assertj.core.api.Assertions;
+import org.hamcrest.Matchers;
 import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.TestInstance;
@@ -78,8 +79,9 @@ public class IgniteMessagingTest extends AbstractIgniteTest 
implements Serializa
 
         template.requestBodyAndHeader("ignite-messaging:" + TOPIC1, 1, 
IgniteConstants.IGNITE_MESSAGING_TOPIC, "TOPIC2");
 
-        Thread.sleep(1000);
-        Assertions.assertThat(messages1.size()).isEqualTo(0);
+        await().atMost(1, TimeUnit.SECONDS)
+                .until(() -> messages1.size(), Matchers.equalTo(0));
+
         Assertions.assertThat(messages2.size()).isEqualTo(1);
     }
 

Reply via email to