Merge pull request #488 from DEFRA/fix/df-912-dlq-vis-timeout

fix/df-912: Changed DLQ VisibilityTimeout to zero

Author: jbarnsley10

src/messaging/event.js

@@ -50,7 +50,7 @@ export function receiveDlqMessages(dlq) {
   const command = new ReceiveMessageCommand({
     QueueUrl: queueUrl,
     MaxNumberOfMessages: 10,
-    VisibilityTimeout: 1,
+    VisibilityTimeout: 0,
     WaitTimeSeconds: 0
   })
   return sqsClient.send(command)

src/messaging/event.test.js

@@ -69,7 +69,7 @@ describe('event', () => {
       await receiveDlqMessages('form-submissions')
       expect(snsMock).toHaveReceivedCommandWith(ReceiveMessageCommand, {
         QueueUrl: expect.any(String),
-        VisibilityTimeout: 1,
+        VisibilityTimeout: 0,
         WaitTimeSeconds: 0
       })
     })
@@ -83,7 +83,7 @@ describe('event', () => {
       await receiveDlqMessages('save-and-exit')
       expect(snsMock).toHaveReceivedCommandWith(ReceiveMessageCommand, {
         QueueUrl: expect.any(String),
-        VisibilityTimeout: 1,
+        VisibilityTimeout: 0,
         WaitTimeSeconds: 0
       })
     })