update unit tests

Author: alexnuttall

lambdas/backend-api/src/__tests__/templates/api/proof.test.ts

@@ -3,13 +3,16 @@ import { mock } from 'jest-mock-extended';
 import { createHandler } from '@backend-api/templates/api/proof';
 import { TemplateClient } from '@backend-api/templates/app/template-client';
 import { LetterTemplate } from 'nhs-notify-web-template-management-utils';
+import { ProofingQueue } from '@backend-api/templates/infra/proofing-queue';
 
 const setup = () => {
   const templateClient = mock<TemplateClient>();
 
-  const handler = createHandler({ templateClient });
+  const proofingQueue = mock<ProofingQueue>();
 
-  return { handler, mocks: { templateClient } };
+  const handler = createHandler({ templateClient, proofingQueue });
+
+  return { handler, mocks: { templateClient, proofingQueue } };
 };
 
 describe('Template API - request proof', () => {
@@ -97,7 +100,8 @@ describe('Template API - request proof', () => {
 
     expect(mocks.templateClient.requestProof).toHaveBeenCalledWith(
       'template-id',
-      { userId: 'sub', clientId: 'nhs-notify-client-id' }
+      { userId: 'sub', clientId: 'nhs-notify-client-id' },
+      mocks.proofingQueue
     );
   });
 
@@ -145,9 +149,13 @@ describe('Template API - request proof', () => {
       body: JSON.stringify({ statusCode: 200, data: response }),
     });
 
-    expect(mocks.templateClient.requestProof).toHaveBeenCalledWith('id', {
-      userId: 'sub',
-      clientId: 'notify-client-id',
-    });
+    expect(mocks.templateClient.requestProof).toHaveBeenCalledWith(
+      'id',
+      {
+        userId: 'sub',
+        clientId: 'notify-client-id',
+      },
+      mocks.proofingQueue
+    );
   });
 });

lambdas/backend-api/src/__tests__/templates/app/template-client.test.ts

@@ -40,7 +40,6 @@ const setup = () => {
   const templateClient = new TemplateClient(
     templateRepository,
     letterUploadRepository,
-    queueMock,
     defaultLetterSupplier,
     clientConfigRepository,
     logger
@@ -1673,7 +1672,11 @@ describe('templateClient', () => {
         data: { features: { proofing: false } },
       });
 
-      const result = await templateClient.requestProof(templateId, user);
+      const result = await templateClient.requestProof(
+        templateId,
+        user,
+        mocks.queueMock
+      );
 
       expect(
         mocks.templateRepository.proofRequestUpdate
@@ -1696,7 +1699,11 @@ describe('templateClient', () => {
         error: { errorMeta: { description: 'err', code: 500 } },
       });
 
-      const result = await templateClient.requestProof(templateId, user);
+      const result = await templateClient.requestProof(
+        templateId,
+        user,
+        mocks.queueMock
+      );
 
       expect(
         mocks.templateRepository.proofRequestUpdate
@@ -1740,7 +1747,11 @@ describe('templateClient', () => {
         },
       });
 
-      const result = await templateClient.requestProof(templateId, user);
+      const result = await templateClient.requestProof(
+        templateId,
+        user,
+        mocks.queueMock
+      );
 
       expect(mocks.templateRepository.proofRequestUpdate).toHaveBeenCalledWith(
         templateId,
@@ -1801,7 +1812,11 @@ describe('templateClient', () => {
         },
       });
 
-      const result = await templateClient.requestProof(templateId, user);
+      const result = await templateClient.requestProof(
+        templateId,
+        user,
+        mocks.queueMock
+      );
 
       expect(mocks.templateRepository.proofRequestUpdate).toHaveBeenCalledWith(
         templateId,
@@ -1852,7 +1867,11 @@ describe('templateClient', () => {
         },
       });
 
-      const result = await templateClient.requestProof(templateId, user);
+      const result = await templateClient.requestProof(
+        templateId,
+        user,
+        mocks.queueMock
+      );
 
       expect(mocks.templateRepository.proofRequestUpdate).toHaveBeenCalledWith(
         templateId,
@@ -1920,7 +1939,11 @@ describe('templateClient', () => {
         },
       });
 
-      const result = await templateClient.requestProof(templateId, user);
+      const result = await templateClient.requestProof(
+        templateId,
+        user,
+        mocks.queueMock
+      );
 
       expect(mocks.templateRepository.proofRequestUpdate).toHaveBeenCalledWith(
         templateId,
@@ -1994,7 +2017,11 @@ describe('templateClient', () => {
 
       mocks.queueMock.send.mockResolvedValueOnce({ data: { $metadata: {} } });
 
-      const result = await templateClient.requestProof(templateId, user);
+      const result = await templateClient.requestProof(
+        templateId,
+        user,
+        mocks.queueMock
+      );
 
       expect(mocks.templateRepository.proofRequestUpdate).toHaveBeenCalledWith(
         templateId,