aldettinger commented on a change in pull request #3352:
URL: https://github.com/apache/camel-quarkus/pull/3352#discussion_r764779841
##########
File path:
integration-test-groups/aws2/aws2-sqs/src/test/java/org/apache/camel/quarkus/component/aws2/sqs/it/Aws2SqsTest.java
##########
@@ -160,35 +160,35 @@ public void sqsAutoCreateDelayedQueue() {
}
private void createDelayQueueAndVerifyExistence(String queueName, int
delay) {
- RestAssured.post("/aws2-sqs-sns/sqs/queue/autocreate/delayed/" +
queueName + "/" + delay)
+ RestAssured.post("/aws2-sqs/queue/autocreate/delayed/" + queueName +
"/" + delay)
.then()
.statusCode(200)
.extract()
.body()
.as(String[].class);
- Awaitility.await().pollInterval(1, TimeUnit.SECONDS).atMost(120,
TimeUnit.SECONDS).until(
+ Awaitility.await().pollInterval(1, TimeUnit.SECONDS).atMost(10,
TimeUnit.SECONDS).until(
Review comment:
It looks the sqs subset of sqs-sns to take far less time. What is the
reason of narrowing down the timeout from 120 to 10 ? Do we save that much time
?
--
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]
