[GitHub] [camel-quarkus] aldettinger commented on a change in pull request #3352: ✅ Test AWS2 SQS in isolation

[GitBox <gi...@apache.org>]

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: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org