unit tests pass

Author: alexnuttall

frontend/src/__tests__/utils/form-actions.test.ts

@@ -14,6 +14,7 @@ import {
   createLetterTemplate,
   setTemplateToDeleted,
   setTemplateToSubmitted,
+  requestTemplateProof,
 } from '@utils/form-actions';
 import { getAccessTokenServer } from '@utils/amplify-utils';
 import { TemplateDto } from 'nhs-notify-backend-client';
@@ -583,4 +584,66 @@ describe('form-actions', () => {
       );
     });
   });
+
+  describe('requestTemplateProof', () => {
+    test('sends proof request successfully', async () => {
+      const responseData = {
+        templateType: 'LETTER',
+        id: 'new-template-id',
+        templateStatus: 'NOT_YET_SUBMITTED',
+        name: 'template-name',
+        letterType: 'q1',
+        language: 'ar',
+        files: {
+          pdfTemplate: {
+            fileName: 'template.pdf',
+            currentVersion: 'pdf-version',
+            virusScanStatus: 'PASSED',
+          },
+        },
+        createdAt: '2025-01-13T10:19:25.579Z',
+        updatedAt: '2025-01-13T10:19:25.579Z',
+      } satisfies TemplateDto;
+
+      mockedTemplateClient.requestProof.mockResolvedValueOnce({
+        data: responseData,
+      });
+
+      const response = await requestTemplateProof('id');
+
+      expect(mockedTemplateClient.requestProof).toHaveBeenCalledWith(
+        'id',
+        'token'
+      );
+
+      expect(response).toEqual(responseData);
+    });
+
+    test('requestTemplateProof - should throw error when saving unexpectedly fails', async () => {
+      mockedTemplateClient.requestProof.mockResolvedValueOnce({
+        error: {
+          code: 400,
+          message: 'Bad request',
+        },
+      });
+
+      await expect(requestTemplateProof('id')).rejects.toThrow(
+        'Failed to request proof'
+      );
+
+      expect(mockedTemplateClient.requestProof).toHaveBeenCalledWith(
+        'id',
+        'token'
+      );
+    });
+
+    test('deleteTemplate - should throw error when no token', async () => {
+      authIdTokenServerMock.mockReset();
+      authIdTokenServerMock.mockResolvedValueOnce(undefined);
+
+      await expect(requestTemplateProof('id')).rejects.toThrow(
+        'Failed to get access token'
+      );
+    });
+  });
 });

frontend/src/components/forms/RequestProof/server-action.ts

@@ -26,12 +26,7 @@ export async function requestProof(route: string, formData: FormData) {
   }
 
   try {
-    await requestTemplateProof(
-      templateId,
-      validatedTemplate.personalisationParameters ?? [],
-      validatedTemplate.files.pdfTemplate.currentVersion,
-      validatedTemplate.files.testDataCsv?.currentVersion
-    );
+    await requestTemplateProof(templateId);
   } catch (error) {
     logger.error('Failed to request template proof', {
       error,

frontend/src/utils/form-actions.ts

@@ -116,22 +116,15 @@ export async function setTemplateToDeleted(templateId: string): Promise<void> {
 }
 
 export async function requestTemplateProof(
-  templateId: string,
-  personalisationParameters: string[],
-  pdfVersionId: string,
-  testDataVersionId: string | undefined
+  templateId: string
 ): Promise<TemplateDto> {
   const token = await getAccessTokenServer();
 
   if (!token) {
     throw new Error('Failed to get access token');
   }
 
-  const { data, error } = await templateClient.requestProof(templateId, token, {
-    personalisationParameters,
-    pdfVersionId,
-    testDataVersionId,
-  });
+  const { data, error } = await templateClient.requestProof(templateId, token);
 
   if (error) {
     logger.error('Failed to request proof', { error });