Skip to content

Commit 6f42c63

Browse files
author
Pankaj Agrawal
committed
Fix correct place holder for queuename and account
1 parent 145174b commit 6f42c63

File tree

2 files changed

+10
-2
lines changed

2 files changed

+10
-2
lines changed

powertools-sqs/src/main/java/software/amazon/lambda/powertools/sqs/internal/BatchContext.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -75,8 +75,8 @@ private void deleteSuccessMessage() {
7575
private String url() {
7676
String[] arnArray = success.get(0).getEventSourceArn().split(":");
7777
return client.getQueueUrl(GetQueueUrlRequest.builder()
78-
.queueOwnerAWSAccountId(arnArray[1])
79-
.queueName(arnArray[2])
78+
.queueOwnerAWSAccountId(arnArray[4])
79+
.queueName(arnArray[5])
8080
.build())
8181
.queueUrl();
8282
}

powertools-sqs/src/test/java/software/amazon/lambda/powertools/sqs/PowertoolsSqsBatchProcessorTest.java

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
import org.junit.jupiter.api.Test;
1010
import org.junit.jupiter.params.ParameterizedTest;
1111
import org.junit.jupiter.params.provider.ValueSource;
12+
import org.mockito.ArgumentCaptor;
1213
import software.amazon.awssdk.services.sqs.SqsClient;
1314
import software.amazon.awssdk.services.sqs.model.DeleteMessageBatchRequest;
1415
import software.amazon.awssdk.services.sqs.model.GetQueueUrlRequest;
@@ -107,6 +108,13 @@ void shouldBatchProcessAndDeleteSuccessMessageOnPartialFailures() {
107108

108109
verify(interactionClient).listQueues();
109110
verify(sqsClient).deleteMessageBatch(any(DeleteMessageBatchRequest.class));
111+
112+
ArgumentCaptor<GetQueueUrlRequest> captor = ArgumentCaptor.forClass(GetQueueUrlRequest.class);
113+
verify(sqsClient).getQueueUrl(captor.capture());
114+
115+
assertThat(captor.getValue())
116+
.hasFieldOrPropertyWithValue("queueName", "my-queue")
117+
.hasFieldOrPropertyWithValue("queueOwnerAWSAccountId", "123456789012");
110118
}
111119

112120
@Test

0 commit comments

Comments
 (0)