apupier commented on code in PR #24651:
URL: https://github.com/apache/camel/pull/24651#discussion_r3585616641
##########
components/camel-hazelcast/src/test/java/org/apache/camel/component/hazelcast/HazelcastReplicatedmapConsumerTest.java:
##########
@@ -84,7 +84,7 @@ public void testAdd() throws InterruptedException {
out.expectedMessageCount(1);
map.put("4711", "my-foo");
- MockEndpoint.assertIsSatisfied(context, 5000, TimeUnit.MILLISECONDS);
+ MockEndpoint.assertIsSatisfied(context, 30, TimeUnit.SECONDS);
Review Comment:
this test is usually taking less than 100ms, the problem is very unlikely
not due to the amount of time needed
##########
components/camel-hazelcast/src/test/java/org/apache/camel/component/hazelcast/HazelcastReplicatedmapConsumerTest.java:
##########
@@ -110,11 +110,11 @@ public void testRemove() throws InterruptedException {
// Wait for the ADDED event to be fully processed before removing,
// otherwise the remove may execute before the async ADDED event is
delivered.
MockEndpoint added = getMockEndpoint("mock:added");
- Awaitility.await().atMost(5, TimeUnit.SECONDS)
+ Awaitility.await().atMost(10, TimeUnit.SECONDS)
.until(() -> added.getReceivedCounter() >= 1);
map.remove("4711");
- MockEndpoint.assertIsSatisfied(context, 5000, TimeUnit.MILLISECONDS);
+ MockEndpoint.assertIsSatisfied(context, 30, TimeUnit.SECONDS);
this.checkHeaders(out.getExchanges().get(0).getIn().getHeaders(),
HazelcastConstants.REMOVED);
Review Comment:
this test is usually taking less than 150ms. i tivery unlikely that
increasing the tiemout from 5 to 10 or 30 seconds will change something
--
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]
