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

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


The following commit(s) were added to refs/heads/main by this push:
     new e98287b6712f CAMEL-23761: Add SqsProducerDelayedQueueIT for the 
delayed queue pattern (#26395)
e98287b6712f is described below

commit e98287b6712fa61bb5ad69710657e55a2aea1e05
Author: Guillaume Nodet <[email protected]>
AuthorDate: Mon Sep 14 16:32:43 2026 +0200

    CAMEL-23761: Add SqsProducerDelayedQueueIT for the delayed queue pattern 
(#26395)
    
    CAMEL-23761: Add SqsProducerDelayedQueueIT and fix visibilityTimeout usage
    
    The SqsProducerDelayedQueueIT test verifies the delayed queue pattern:
    a queue is created with DELAY_SECONDS=20, a message is sent to it, and
    reception must take at least 20 seconds.
    
    The root cause of the failure: the receive endpoint used
    defaultVisibilityTimeout=0 which calls 
SetQueueAttributes({VISIBILITY_TIMEOUT:0}).
    On Localstack, this resets DELAY_SECONDS to 0, defeating the delay.
    
    Fix: use visibilityTimeout=0 (per-request ReceiveMessage timeout) instead
    of defaultVisibilityTimeout=0 (queue-level attribute). This avoids calling
    SetQueueAttributes and preserves the queue's DELAY_SECONDS attribute.
    
    Co-authored-by: Claude Sonnet 4.6 <[email protected]>
---
 .../sqs/integration/SqsProducerDelayedQueueIT.java | 66 ++++++++++++++++++++++
 1 file changed, 66 insertions(+)

diff --git 
a/components/camel-aws/camel-aws2-sqs/src/test/java/org/apache/camel/component/aws2/sqs/integration/SqsProducerDelayedQueueIT.java
 
b/components/camel-aws/camel-aws2-sqs/src/test/java/org/apache/camel/component/aws2/sqs/integration/SqsProducerDelayedQueueIT.java
new file mode 100644
index 000000000000..674ec748b65d
--- /dev/null
+++ 
b/components/camel-aws/camel-aws2-sqs/src/test/java/org/apache/camel/component/aws2/sqs/integration/SqsProducerDelayedQueueIT.java
@@ -0,0 +1,66 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.camel.component.aws2.sqs.integration;
+
+import java.time.Duration;
+import java.time.Instant;
+import java.util.UUID;
+import java.util.concurrent.TimeUnit;
+
+import org.awaitility.Awaitility;
+import org.junit.jupiter.api.Test;
+import software.amazon.awssdk.services.sqs.model.ListQueuesResponse;
+
+import static org.assertj.core.api.Assertions.assertThat;
+
+class SqsProducerDelayedQueueIT extends Aws2SQSBaseTest {
+
+    @Test
+    void delayedQueue() throws Exception {
+        int delay = 20;
+        String delayedQueueName = sharedNameGenerator.getName() + "_delayed";
+        Instant start = Instant.now();
+
+        // Create the delayed queue via autoCreateQueue
+        template.requestBody(
+                String.format(
+                        
"aws2-sqs://%s?autoCreateQueue=true&delayQueue=true&delaySeconds=%d&operation=listQueues",
+                        delayedQueueName, delay),
+                null, ListQueuesResponse.class);
+
+        // Send a message to the delayed queue
+        String msg = "sqs" + UUID.randomUUID().toString().replace("-", "");
+        template.requestBody("aws2-sqs://" + delayedQueueName, msg, 
String.class);
+
+        // Poll until the message is received; use visibilityTimeout 
(per-request) instead of
+        // defaultVisibilityTimeout (queue-level attribute) to avoid 
overwriting the queue's
+        // DELAY_SECONDS attribute via SetQueueAttributes.
+        Awaitility.await().pollInterval(1, TimeUnit.SECONDS).atMost(120, 
TimeUnit.SECONDS)
+                .until(() -> 
msg.equals(receiveMessageFromQueue(delayedQueueName)));
+
+        assertThat(Duration.between(start, 
Instant.now()).getSeconds()).isGreaterThanOrEqualTo(delay);
+    }
+
+    private String receiveMessageFromQueue(String queueName) {
+        // visibilityTimeout=0 sets the per-request visibility timeout in 
ReceiveMessage,
+        // which does not call SetQueueAttributes and therefore does not 
disturb DELAY_SECONDS.
+        return consumer.receiveBody(
+                
String.format("aws2-sqs://%s?deleteAfterRead=false&deleteIfFiltered=false&visibilityTimeout=0",
+                        queueName),
+                10000, String.class);
+    }
+}

Reply via email to