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

fmariani 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 25d8c1a5c4d8 Make SqsConsumerTest more robust
25d8c1a5c4d8 is described below

commit 25d8c1a5c4d87bd56cd84f564ed4425f122ca3f7
Author: Croway <[email protected]>
AuthorDate: Tue Feb 10 17:25:06 2026 +0100

    Make SqsConsumerTest more robust
---
 .../camel/component/aws2/sqs/Sqs2ConsumerTest.java     | 18 +++++++++++-------
 1 file changed, 11 insertions(+), 7 deletions(-)

diff --git 
a/components/camel-aws/camel-aws2-sqs/src/test/java/org/apache/camel/component/aws2/sqs/Sqs2ConsumerTest.java
 
b/components/camel-aws/camel-aws2-sqs/src/test/java/org/apache/camel/component/aws2/sqs/Sqs2ConsumerTest.java
index 4f6df74ebb78..1e0d694b64fc 100644
--- 
a/components/camel-aws/camel-aws2-sqs/src/test/java/org/apache/camel/component/aws2/sqs/Sqs2ConsumerTest.java
+++ 
b/components/camel-aws/camel-aws2-sqs/src/test/java/org/apache/camel/component/aws2/sqs/Sqs2ConsumerTest.java
@@ -49,6 +49,7 @@ import static org.assertj.core.api.Assertions.catchException;
 import static org.mockito.Mockito.doReturn;
 import static 
software.amazon.awssdk.services.sqs.model.MessageSystemAttributeName.ALL;
 import static 
software.amazon.awssdk.services.sqs.model.MessageSystemAttributeName.MESSAGE_GROUP_ID;
+import static 
software.amazon.awssdk.services.sqs.model.MessageSystemAttributeName.SENDER_ID;
 import static 
software.amazon.awssdk.services.sqs.model.MessageSystemAttributeName.SENT_TIMESTAMP;
 import static 
software.amazon.awssdk.services.sqs.model.MessageSystemAttributeName.SEQUENCE_NUMBER;
 
@@ -385,6 +386,7 @@ class Sqs2ConsumerTest extends CamelTestSupport {
     void shouldRequest66MessagesWithSevenReceiveRequestSortedBySenderId() 
throws Exception {
         // given
         generateSequenceNumber = false;
+        configuration.setSortAttributeName("SenderId");
         var expectedMessages = IntStream.range(0, 66)
                 .mapToObj(i -> Message.builder()
                         .body(Integer.toString(i))
@@ -401,14 +403,16 @@ class Sqs2ConsumerTest extends CamelTestSupport {
             // then
             assertThat(polledMessagesCount).isEqualTo(66);
             
assertThat(receiveMessageBodies()).containsExactly(expectedMessages.stream().map(Message::body).toArray());
+            var expectedRequest = expectedReceiveRequestBuilder()
+                    .messageSystemAttributeNames(List.of(SENT_TIMESTAMP, 
MESSAGE_GROUP_ID, SENDER_ID));
             
assertThat(sqsClientMock.getReceiveRequests()).containsExactlyInAnyOrder(
-                    expectedReceiveRequest(10),
-                    expectedReceiveRequest(10),
-                    expectedReceiveRequest(10),
-                    expectedReceiveRequest(10),
-                    expectedReceiveRequest(10),
-                    expectedReceiveRequest(10),
-                    expectedReceiveRequest(6));
+                    expectedRequest.maxNumberOfMessages(10).build(),
+                    expectedRequest.maxNumberOfMessages(10).build(),
+                    expectedRequest.maxNumberOfMessages(10).build(),
+                    expectedRequest.maxNumberOfMessages(10).build(),
+                    expectedRequest.maxNumberOfMessages(10).build(),
+                    expectedRequest.maxNumberOfMessages(10).build(),
+                    expectedRequest.maxNumberOfMessages(6).build());
             assertThat(sqsClientMock.getQueues()).isEmpty();
         }
     }

Reply via email to