NHSDigital
diff --git a/‎.eslintrc.json‎
Lines changed: 3 additions & 2 deletions b/‎.eslintrc.json‎
Lines changed: 3 additions & 2 deletions
diff --git a/‎frontend/src/__tests__/components/forms/PreviewEmailTemplate/__snapshots__/PreviewEmailTemplate.test.tsx.snap‎
Lines changed: 4 additions & 4 deletions b/‎frontend/src/__tests__/components/forms/PreviewEmailTemplate/__snapshots__/PreviewEmailTemplate.test.tsx.snap‎
Lines changed: 4 additions & 4 deletions
diff --git a/‎frontend/src/__tests__/components/forms/PreviewNHSAppTemplate/__snapshots__/PreviewNHSAppTemplate.test.tsx.snap‎
Lines changed: 4 additions & 4 deletions b/‎frontend/src/__tests__/components/forms/PreviewNHSAppTemplate/__snapshots__/PreviewNHSAppTemplate.test.tsx.snap‎
Lines changed: 4 additions & 4 deletions
diff --git a/‎frontend/src/__tests__/components/forms/PreviewSMSTemplate/__snapshots__/PreviewSMSTemplate.test.tsx.snap‎
Lines changed: 4 additions & 4 deletions b/‎frontend/src/__tests__/components/forms/PreviewSMSTemplate/__snapshots__/PreviewSMSTemplate.test.tsx.snap‎
Lines changed: 4 additions & 4 deletions
diff --git a/‎frontend/src/__tests__/components/molecules/PreviewTemplateDetails.test.tsx‎
Lines changed: 75 additions & 103 deletions b/‎frontend/src/__tests__/components/molecules/PreviewTemplateDetails.test.tsx‎
Lines changed: 75 additions & 103 deletions
@@ -89,12 +89,13 @@
     },
     {
       "files": [
-        "*.test.{ts,tsx}"
+        "*.{test,spec}.{ts,tsx}"
       ],
       "rules": {
         "unicorn/no-useless-undefined": "off",
         "unicorn/consistent-function-scoping": "off",
-        "max-classes-per-file": "off"
+        "max-classes-per-file": "off",
+        "sonarjs/no-nested-functions": "off"
       }
     }
   ]
 
@@ -123,7 +123,7 @@ exports[`Preview email form renders Client-side validation triggers 1`] = `
             </div>
           </dl>
           <dl
-            class="nhsuk-summary-list preview"
+            class="nhsuk-summary-list nhsuk-u-margin-bottom-4 preview"
           >
             <div
               class="nhsuk-summary-list__row"
@@ -412,7 +412,7 @@ exports[`Preview email form renders matches error snapshot 1`] = `
             </div>
           </dl>
           <dl
-            class="nhsuk-summary-list preview"
+            class="nhsuk-summary-list nhsuk-u-margin-bottom-4 preview"
           >
             <div
               class="nhsuk-summary-list__row"
@@ -674,7 +674,7 @@ exports[`Preview email form renders matches snapshot when navigating from edit s
             </div>
           </dl>
           <dl
-            class="nhsuk-summary-list preview"
+            class="nhsuk-summary-list nhsuk-u-margin-bottom-4 preview"
           >
             <div
               class="nhsuk-summary-list__row"
@@ -919,7 +919,7 @@ exports[`Preview email form renders matches snapshot when navigating from manage
             </div>
           </dl>
           <dl
-            class="nhsuk-summary-list preview"
+            class="nhsuk-summary-list nhsuk-u-margin-bottom-4 preview"
           >
             <div
               class="nhsuk-summary-list__row"
 
@@ -123,7 +123,7 @@ exports[`Preview nhs app form renders Client-side validation triggers 1`] = `
             </div>
           </dl>
           <dl
-            class="nhsuk-summary-list preview"
+            class="nhsuk-summary-list nhsuk-u-margin-bottom-4 preview"
           >
             <div
               class="nhsuk-summary-list__row"
@@ -387,7 +387,7 @@ exports[`Preview nhs app form renders matches error snapshot 1`] = `
             </div>
           </dl>
           <dl
-            class="nhsuk-summary-list preview"
+            class="nhsuk-summary-list nhsuk-u-margin-bottom-4 preview"
           >
             <div
               class="nhsuk-summary-list__row"
@@ -624,7 +624,7 @@ exports[`Preview nhs app form renders matches snapshot when navigating from edit
             </div>
           </dl>
           <dl
-            class="nhsuk-summary-list preview"
+            class="nhsuk-summary-list nhsuk-u-margin-bottom-4 preview"
           >
             <div
               class="nhsuk-summary-list__row"
@@ -844,7 +844,7 @@ exports[`Preview nhs app form renders matches snapshot when navigating from mana
             </div>
           </dl>
           <dl
-            class="nhsuk-summary-list preview"
+            class="nhsuk-summary-list nhsuk-u-margin-bottom-4 preview"
           >
             <div
               class="nhsuk-summary-list__row"
 
@@ -123,7 +123,7 @@ exports[`Review sms form renders Client-side validation triggers 1`] = `
             </div>
           </dl>
           <dl
-            class="nhsuk-summary-list preview"
+            class="nhsuk-summary-list nhsuk-u-margin-bottom-4 preview"
           >
             <div
               class="nhsuk-summary-list__row"
@@ -387,7 +387,7 @@ exports[`Review sms form renders matches error snapshot 1`] = `
             </div>
           </dl>
           <dl
-            class="nhsuk-summary-list preview"
+            class="nhsuk-summary-list nhsuk-u-margin-bottom-4 preview"
           >
             <div
               class="nhsuk-summary-list__row"
@@ -624,7 +624,7 @@ exports[`Review sms form renders matches snapshot when navigating from edit scre
             </div>
           </dl>
           <dl
-            class="nhsuk-summary-list preview"
+            class="nhsuk-summary-list nhsuk-u-margin-bottom-4 preview"
           >
             <div
               class="nhsuk-summary-list__row"
@@ -844,7 +844,7 @@ exports[`Review sms form renders matches snapshot when navigating from manage te
             </div>
           </dl>
           <dl
-            class="nhsuk-summary-list preview"
+            class="nhsuk-summary-list nhsuk-u-margin-bottom-4 preview"
           >
             <div
               class="nhsuk-summary-list__row"
 
@@ -1,10 +1,13 @@
-import { render, screen } from '@testing-library/react';
-import { PreviewTemplateDetails } from '@molecules/PreviewTemplateDetails';
+import PreviewTemplateDetailsEmail from '@molecules/PreviewTemplateDetails/PreviewTemplateDetailsEmail';
+import PreviewTemplateDetailsLetter from '@molecules/PreviewTemplateDetails/PreviewTemplateDetailsLetter';
+import PreviewTemplateDetailsNhsApp from '@molecules/PreviewTemplateDetails/PreviewTemplateDetailsNhsApp';
+import PreviewTemplateDetailsSms from '@molecules/PreviewTemplateDetails/PreviewTemplateDetailsSms';
+import { render } from '@testing-library/react';
 
-describe('PreviewTemplateDetails component', () => {
-  it('matches not yet submitted snapshot', () => {
+describe('PreviewTemplateDetailsNhsApp', () => {
+  it('matches snapshot', () => {
     const container = render(
-      <PreviewTemplateDetails
+      <PreviewTemplateDetailsNhsApp
         template={{
           id: 'template-id',
           name: 'Example template',
@@ -14,53 +17,66 @@ describe('PreviewTemplateDetails component', () => {
           createdAt: '2025-01-13T10:19:25.579Z',
           updatedAt: '2025-01-13T10:19:25.579Z',
         }}
-        templateTypeText='Channel template'
-        contentPreview={[
-          {
-            heading: 'Heading',
-            id: 'heading',
-            value: 'Test value-1',
-          },
-        ]}
+        message='app content'
       />
     );
 
     expect(container.asFragment()).toMatchSnapshot();
   });
+});
 
-  it('matches submitted snapshot', () => {
+describe('PreviewTemplateDetailsEmail', () => {
+  it('matches snapshot', () => {
     const container = render(
-      <PreviewTemplateDetails
+      <PreviewTemplateDetailsEmail
         template={{
           id: 'template-id',
-          name: 'Example template',
+          name: 'Example Email template',
+          message: 'email message message',
+          subject: 'subject',
+          templateStatus: 'SUBMITTED',
+          templateType: 'EMAIL',
+          createdAt: '2025-01-13T10:19:25.579Z',
+          updatedAt: '2025-01-13T10:19:25.579Z',
+        }}
+        message='email content'
+        subject='email subject'
+      />
+    );
+
+    expect(container.asFragment()).toMatchSnapshot();
+  });
+});
+
+describe('PreviewTemplateDetailsSms', () => {
+  it('matches snapshot', () => {
+    const container = render(
+      <PreviewTemplateDetailsSms
+        template={{
+          id: 'template-id',
+          name: 'SMS template',
+          message: 'SMS message',
           templateStatus: 'SUBMITTED',
           templateType: 'SMS',
-          message: 'text message message',
           createdAt: '2025-01-13T10:19:25.579Z',
           updatedAt: '2025-01-13T10:19:25.579Z',
         }}
-        templateTypeText='Channel template'
-        contentPreview={[
-          {
-            heading: 'Heading',
-            id: 'heading',
-            value: 'Test value-1',
-          },
-        ]}
+        message='SMS content'
       />
     );
 
     expect(container.asFragment()).toMatchSnapshot();
   });
+});
 
-  it('renders without content preview', () => {
+describe('PreviewTemplateDetailsLetter', () => {
+  it('matches snapshot without proofs', () => {
     const container = render(
-      <PreviewTemplateDetails
+      <PreviewTemplateDetailsLetter
         template={{
           id: 'template-id',
           name: 'Example template',
-          templateStatus: 'SUBMITTED',
+          templateStatus: 'PENDING_VALIDATION',
           templateType: 'LETTER',
           letterType: 'x0',
           language: 'fr',
@@ -79,27 +95,19 @@ describe('PreviewTemplateDetails component', () => {
           createdAt: '2025-01-13T10:19:25.579Z',
           updatedAt: '2025-01-13T10:19:25.579Z',
         }}
-        additionalMetaFields={[
-          {
-            title: 'Meta',
-            id: 'meta',
-            content: <p>metadata</p>,
-          },
-        ]}
-        templateTypeText='Channel template'
       />
     );
 
     expect(container.asFragment()).toMatchSnapshot();
   });
 
-  it('renders letter template when testDataCsv is not present', () => {
+  it('matches snapshot when test data CSV is absent', () => {
     const container = render(
-      <PreviewTemplateDetails.Letter
+      <PreviewTemplateDetailsLetter
         template={{
           id: 'template-id',
           name: 'Example template',
-          templateStatus: 'SUBMITTED',
+          templateStatus: 'PENDING_VALIDATION',
           templateType: 'LETTER',
           letterType: 'x0',
           language: 'fr',
@@ -119,26 +127,31 @@ describe('PreviewTemplateDetails component', () => {
     expect(container.asFragment()).toMatchSnapshot();
   });
 
-  it('renders letter template when testPersonalisationInputFile is set', () => {
+  it('matches snapshot when proofs are present, proofs failing virus scan are not displayed', () => {
     const container = render(
-      <PreviewTemplateDetails.Letter
+      <PreviewTemplateDetailsLetter
         template={{
           id: 'template-id',
-          name: 'Example template',
-          templateStatus: 'SUBMITTED',
+          name: 'Example letter',
+          templateStatus: 'PROOF_AVAILABLE',
           templateType: 'LETTER',
           letterType: 'x0',
-          language: 'fr',
+          language: 'en',
           files: {
             pdfTemplate: {
               fileName: 'file.pdf',
               currentVersion: '4C728B7D-A028-4BA2-B180-A63CDD2AE1E9',
-              virusScanStatus: 'PENDING',
+              virusScanStatus: 'PASSED',
             },
             testDataCsv: {
               fileName: 'file.csv',
               currentVersion: '622AB7FA-29BA-418A-B1B6-1E63FB299269',
-              virusScanStatus: 'PENDING',
+              virusScanStatus: 'PASSED',
+            },
+            proofs: {
+              'a.pdf': { fileName: 'a.pdf', virusScanStatus: 'PASSED' },
+              'b.pdf': { fileName: 'b.pdf', virusScanStatus: 'PASSED' },
+              'c.pdf': { fileName: 'c.pdf', virusScanStatus: 'FAILED' },
             },
           },
           createdAt: '2025-01-13T10:19:25.579Z',
@@ -150,70 +163,29 @@ describe('PreviewTemplateDetails component', () => {
     expect(container.asFragment()).toMatchSnapshot();
   });
 
-  it('renders component correctly', () => {
-    render(
-      <PreviewTemplateDetails
+  it('if status is PROOF_AVAILABLE, but no proofs are registered on the template, proof section is hidden (this is unexpected)', () => {
+    const container = render(
+      <PreviewTemplateDetailsLetter
         template={{
           id: 'template-id',
-          name: 'Example template',
-          message: 'app message message',
-          templateType: 'NHS_APP',
-          templateStatus: 'NOT_YET_SUBMITTED',
+          name: 'Example letter',
+          templateStatus: 'PROOF_AVAILABLE',
+          templateType: 'LETTER',
+          letterType: 'x0',
+          language: 'en',
+          files: {
+            pdfTemplate: {
+              fileName: 'file.pdf',
+              currentVersion: '4C728B7D-A028-4BA2-B180-A63CDD2AE1E9',
+              virusScanStatus: 'PASSED',
+            },
+          },
           createdAt: '2025-01-13T10:19:25.579Z',
           updatedAt: '2025-01-13T10:19:25.579Z',
         }}
-        templateTypeText='Channel template'
-        contentPreview={[
-          {
-            heading: 'Subject',
-            id: 'subject',
-            value: 'Subject value',
-          },
-          {
-            heading: 'Heading',
-            id: 'heading',
-            value: 'Heading value',
-          },
-          {
-            heading: 'Body text',
-            id: 'body-text',
-            value: 'Body text value',
-          },
-          {
-            heading: 'Message',
-            id: 'message',
-            value: 'Message value',
-          },
-        ]}
       />
     );
 
-    expect(screen.getByTestId('preview-message__heading')).toHaveTextContent(
-      'Example template'
-    );
-    expect(screen.getByTestId('preview__heading-0')).toHaveTextContent(
-      'Subject'
-    );
-    expect(screen.getByTestId('preview__content-0')).toHaveTextContent(
-      'Subject value'
-    );
-    expect(screen.getByTestId('preview__heading-1')).toHaveTextContent(
-      'Heading'
-    );
-    expect(screen.getByTestId('preview__content-1')).toHaveTextContent(
-      'Heading value'
-    );
-    expect(screen.getByTestId('preview__heading-2')).toHaveTextContent(
-      'Body text'
-    );
-    expect(screen.getByTestId('preview__content-2')).toHaveTextContent(
-      'Body text value'
-    );
-    expect(screen.getByTestId('preview__heading-3')).toHaveTextContent(
-      'Message'
-    );
-    expect(screen.getByTestId('preview__content-3')).toHaveTextContent(
-      'Message value'
-    );
+    expect(container.asFragment()).toMatchSnapshot();
   });
 });
Original file line number	Diff line number	Diff line change
`@@ -89,12 +89,13 @@`
`89`	`89`	`},`
`90`	`90`	`{`
`91`	`91`	`"files": [`
`92`		`- "*.test.{ts,tsx}"`
	`92`	`+ "*.{test,spec}.{ts,tsx}"`
`93`	`93`	`],`
`94`	`94`	`"rules": {`
`95`	`95`	`"unicorn/no-useless-undefined": "off",`
`96`	`96`	`"unicorn/consistent-function-scoping": "off",`
`97`		`- "max-classes-per-file": "off"`
	`97`	`+ "max-classes-per-file": "off",`
	`98`	`+ "sonarjs/no-nested-functions": "off"`
`98`	`99`	`}`
`99`	`100`	`}`
`100`	`101`	`]`