[GitHub] [camel-quarkus] VratislavHais commented on a change in pull request #3170: Aws sqs test fix

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

VratislavHais commented on a change in pull request #3170:
URL: https://github.com/apache/camel-quarkus/pull/3170#discussion_r724916018



##########
File path: integration-test-groups/aws2/aws2-sqs-sns/src/test/java/org/apache/camel/quarkus/component/aws2/sqs/it/Aws2SqsSnsTest.java
##########
@@ -43,6 +49,31 @@
 @QuarkusTestResource(Aws2TestResource.class)
 class Aws2SqsSnsTest {
 
+    @AfterEach
+    public void purgeQueueAndWait() {
+        String qName = getPredefinedQueueName();
+        purgeQueue(qName);
+        // purge takes up to 60 seconds
+        // all messages delivered within those 60 seconds might get deleted
+        try {

Review comment:
       It's risky. All messages can be already deleted but purge signal can still be active. It happened to me on AWS that tests continued, sent new message and it got immediately deleted by purge signal. I tried to come up with a better solution but haven't thought of any.




-- 
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