CCM-11537: Tests

ClareJonesBJSS · ClareJonesBJSS · commit 3c2ceda851ae · 2025-11-10T16:59:35.000Z
diff --git a/frontend/src/app/message-plans/choose-email-template/[routingConfigId]/page.tsx b/frontend/src/app/message-plans/choose-email-template/[routingConfigId]/page.tsx
@@ -35,7 +35,6 @@ export default async function ChooseEmailTemplate(props: MessagePlanPageProps) {
 
   const availableTemplateList = await getTemplates({
     templateType: 'EMAIL',
-    templateStatus: 'SUBMITTED',
   });
 
   return (
diff --git a/frontend/src/app/message-plans/choose-nhs-app-template/[routingConfigId]/page.tsx b/frontend/src/app/message-plans/choose-nhs-app-template/[routingConfigId]/page.tsx
@@ -37,7 +37,6 @@ export default async function ChooseNhsAppTemplate(
 
   const availableTemplateList = await getTemplates({
     templateType: 'NHS_APP',
-    templateStatus: 'SUBMITTED',
   });
 
   return (
diff --git a/frontend/src/app/message-plans/choose-text-message-template/[routingConfigId]/page.tsx b/frontend/src/app/message-plans/choose-text-message-template/[routingConfigId]/page.tsx
@@ -37,7 +37,6 @@ export default async function ChooseTextMessageTemplate(
 
   const availableTemplateList = await getTemplates({
     templateType: 'SMS',
-    templateStatus: 'SUBMITTED',
   });
 
   return (
diff --git a/frontend/src/components/forms/ChooseChannelTemplate/ChooseChannelTemplate.tsx b/frontend/src/components/forms/ChooseChannelTemplate/ChooseChannelTemplate.tsx
@@ -54,7 +54,7 @@ export function ChooseChannelTemplate(props: ChooseChannelTemplateProps) {
             formAttributes={{ onSubmit: formValidate }}
           >
             {selectedTemplateId && (
-              <SummaryList>
+              <SummaryList data-testid='previous-selection-summary'>
                 <SummaryList.Row>
                   <SummaryList.Key>
                     {content.previousSelectionLabel}
diff --git a/frontend/src/components/molecules/ChannelTemplates/ChannelTemplates.tsx b/frontend/src/components/molecules/ChannelTemplates/ChannelTemplates.tsx
@@ -99,7 +99,7 @@ export function ChannelTemplates({
                         })}
                         id={`preview-template-link-${index}`}
                         aria-label={`${tableContent.action.preview.text}`}
-                        data-testid='preview-link'
+                        data-testid={`${template.id}-preview-link`}
                       >
                         {tableContent.action.preview.text}
                       </Link>
diff --git a/tests/test-team/helpers/factories/template-factory.ts b/tests/test-team/helpers/factories/template-factory.ts
@@ -6,7 +6,8 @@ export const TemplateFactory = {
   createEmailTemplate: (
     id: string,
     user: TestUser,
-    name: string = 'test'
+    name: string = 'test',
+    templateStatus: string = 'NOT_YET_SUBMITTED'
   ): Template => {
     return TemplateFactory.create({
       campaignId: user.campaignId,
@@ -17,6 +18,7 @@ export const TemplateFactory = {
       owner: `CLIENT#${user.clientId}`,
       subject: 'test-subject',
       templateType: 'EMAIL',
+      templateStatus,
     });
   },
 
diff --git a/tests/test-team/pages/routing/choose-email-template-page.ts b/tests/test-team/pages/routing/choose-email-template-page.ts
@@ -0,0 +1,10 @@
+import { type Page } from '@playwright/test';
+import { TemplateMgmtBasePageDynamic } from 'pages/template-mgmt-base-page-dynamic';
+
+export class ChooseEmailTemplatePage extends TemplateMgmtBasePageDynamic {
+  static readonly pageUrlSegment = 'message-plans/choose-email-template';
+
+  constructor(page: Page) {
+    super(page);
+  }
+}
diff --git a/tests/test-team/pages/routing/choose-nhs-app-template-page.ts b/tests/test-team/pages/routing/choose-nhs-app-template-page.ts
@@ -0,0 +1,10 @@
+import { type Page } from '@playwright/test';
+import { TemplateMgmtBasePageDynamic } from 'pages/template-mgmt-base-page-dynamic';
+
+export class ChooseNhsAppTemplatePage extends TemplateMgmtBasePageDynamic {
+  static readonly pageUrlSegment = 'message-plans/choose-nhs-app-template';
+
+  constructor(page: Page) {
+    super(page);
+  }
+}
diff --git a/tests/test-team/pages/routing/choose-standard-english-letter-template-page.ts b/tests/test-team/pages/routing/choose-standard-english-letter-template-page.ts
@@ -0,0 +1,11 @@
+import { type Page } from '@playwright/test';
+import { TemplateMgmtBasePageDynamic } from 'pages/template-mgmt-base-page-dynamic';
+
+export class ChooseStandardEnglishLetterTemplatePage extends TemplateMgmtBasePageDynamic {
+  static readonly pageUrlSegment =
+    'message-plans/choose-standard-english-letter-template';
+
+  constructor(page: Page) {
+    super(page);
+  }
+}
diff --git a/tests/test-team/pages/routing/choose-text-message-template-page.ts b/tests/test-team/pages/routing/choose-text-message-template-page.ts
@@ -0,0 +1,10 @@
+import { type Page } from '@playwright/test';
+import { TemplateMgmtBasePageDynamic } from 'pages/template-mgmt-base-page-dynamic';
+
+export class ChooseTextMessageTemplatePage extends TemplateMgmtBasePageDynamic {
+  static readonly pageUrlSegment = 'message-plans/choose-text-message-template';
+
+  constructor(page: Page) {
+    super(page);
+  }
+}
diff --git a/tests/test-team/template-mgmt-routing-component-tests/choose-email-template.routing-component.spec.ts b/tests/test-team/template-mgmt-routing-component-tests/choose-email-template.routing-component.spec.ts
@@ -0,0 +1,276 @@
+import { test, expect } from '@playwright/test';
+import { RoutingConfigStorageHelper } from 'helpers/db/routing-config-storage-helper';
+import {
+  assertFooterLinks,
+  assertSignOutLink,
+  assertHeaderLogoLink,
+  assertSkipToMainContent,
+  assertGoBackLinkNotPresent,
+} from '../helpers/template-mgmt-common.steps';
+import { RoutingConfigFactory } from 'helpers/factories/routing-config-factory';
+import {
+  createAuthHelper,
+  TestUser,
+  testUsers,
+} from 'helpers/auth/cognito-auth-helper';
+import { TemplateStorageHelper } from 'helpers/db/template-storage-helper';
+import { randomUUID } from 'node:crypto';
+import { TemplateFactory } from 'helpers/factories/template-factory';
+import { ChooseEmailTemplatePage } from 'pages/routing/choose-email-template-page';
+
+const routingConfigStorageHelper = new RoutingConfigStorageHelper();
+const templateStorageHelper = new TemplateStorageHelper();
+
+const invalidMessagePlanId = 'invalid-id';
+const notFoundMessagePlanId = randomUUID();
+
+function createMessagePlans(user: TestUser) {
+  return {
+    EMAIL_ROUTING_CONFIG: RoutingConfigFactory.createForMessageOrder(
+      user,
+      'NHSAPP,EMAIL'
+    ).dbEntry,
+    NON_EMAIL_ROUTING_CONFIG: RoutingConfigFactory.createForMessageOrder(
+      user,
+      'NHSAPP'
+    ).dbEntry,
+  };
+}
+
+function createTemplates(user: TestUser) {
+  return {
+    EMAIL1: TemplateFactory.createEmailTemplate(
+      randomUUID(),
+      user,
+      'Email template 1'
+    ),
+    EMAIL2: TemplateFactory.createEmailTemplate(
+      randomUUID(),
+      user,
+      'Email template 2'
+    ),
+    EMAIL3: TemplateFactory.createEmailTemplate(
+      randomUUID(),
+      user,
+      'Email template 3'
+    ),
+    APP: TemplateFactory.createNhsAppTemplate(
+      randomUUID(),
+      user,
+      'App template'
+    ),
+  };
+}
+
+test.describe('Routing - Choose email template page', () => {
+  let messagePlans: ReturnType<typeof createMessagePlans>;
+  let templates: ReturnType<typeof createTemplates>;
+
+  test.beforeAll(async () => {
+    const user = await createAuthHelper().getTestUser(testUsers.User1.userId);
+
+    messagePlans = createMessagePlans(user);
+    templates = createTemplates(user);
+
+    await routingConfigStorageHelper.seed(Object.values(messagePlans));
+    // Seed templates later to test empty page
+  });
+
+  test.afterAll(async () => {
+    await routingConfigStorageHelper.deleteSeeded();
+    await templateStorageHelper.deleteSeededTemplates();
+  });
+
+  test('common page tests', async ({ page, baseURL }) => {
+    const props = {
+      page: new ChooseEmailTemplatePage(page),
+      id: messagePlans.EMAIL_ROUTING_CONFIG.id,
+      baseURL,
+    };
+    await assertSkipToMainContent(props);
+    await assertHeaderLogoLink(props);
+    await assertFooterLinks(props);
+    await assertSignOutLink(props);
+    await assertGoBackLinkNotPresent(props);
+  });
+
+  test('loads the choose email template page for a message plan with an email channel', async ({
+    page,
+    baseURL,
+  }) => {
+    const chooseEmailTemplatePage = new ChooseEmailTemplatePage(page);
+    await chooseEmailTemplatePage.loadPage(
+      messagePlans.EMAIL_ROUTING_CONFIG.id
+    );
+    await expect(page).toHaveURL(
+      `${baseURL}/templates/message-plans/choose-email-template/${messagePlans.EMAIL_ROUTING_CONFIG.id}`
+    );
+
+    await test.step('loads page when no templates exist', async () => {
+      await expect(chooseEmailTemplatePage.pageHeading).toHaveText(
+        'Choose an email template'
+      );
+
+      await expect(
+        page.getByText(messagePlans.EMAIL_ROUTING_CONFIG.name)
+      ).toBeVisible();
+
+      await expect(
+        page.getByText('You do not have any templates yet.')
+      ).toBeVisible();
+
+      const goToTemplatesLink = page.getByRole('link', {
+        name: 'Go to templates',
+      });
+      await expect(goToTemplatesLink).toBeVisible();
+      await expect(goToTemplatesLink).toHaveAttribute(
+        'href',
+        '/templates/message-templates'
+      );
+
+      const goBackLink = page.getByRole('link', { name: 'Go back' });
+      await expect(goBackLink).toBeVisible();
+      await expect(goBackLink).toHaveAttribute(
+        'href',
+        `/templates/message-plans/overview/${messagePlans.EMAIL_ROUTING_CONFIG.id}`
+      );
+
+      await expect(page.getByTestId('submit-button')).toBeHidden();
+    });
+
+    await templateStorageHelper.seedTemplateData(Object.values(templates));
+    await chooseEmailTemplatePage.loadPage(
+      messagePlans.EMAIL_ROUTING_CONFIG.id
+    );
+
+    await test.step('displays list of email templates to choose from', async () => {
+      const table = page.getByTestId('channel-templates-table');
+      await expect(table).toBeVisible();
+      await expect(
+        table.getByTestId('channel-templates-table-header-template-select')
+      ).toHaveText('Select');
+      await expect(
+        table.getByTestId('channel-templates-table-header-template-name')
+      ).toHaveText('Name');
+      await expect(
+        table.getByTestId('channel-templates-table-header-template-type')
+      ).toHaveText('Type');
+      await expect(
+        table.getByTestId('channel-templates-table-header-template-last-edited')
+      ).toHaveText('Last edited');
+      await expect(
+        table.getByTestId('channel-templates-table-header-template-action')
+      ).toHaveText('');
+
+      for (const template of [
+        templates.EMAIL1,
+        templates.EMAIL2,
+        templates.EMAIL3,
+      ]) {
+        await expect(table.getByText(template.name)).toBeVisible();
+
+        const radioButton = table.getByTestId(`${template.id}-radio`);
+        await expect(radioButton).toBeVisible();
+        await expect(radioButton).toHaveAttribute('value', template.id);
+        await expect(radioButton).not.toBeChecked();
+
+        await expect(
+          table.getByTestId(`${template.id}-template-type`)
+        ).toHaveText('Email');
+
+        const previewLink = table.getByTestId(`${template.id}-preview-link`);
+        await expect(previewLink).toBeVisible();
+        await expect(previewLink).toHaveText('Preview');
+        await expect(previewLink).toHaveAttribute(
+          'href',
+          `/message-plans/choose-email-template/${messagePlans.EMAIL_ROUTING_CONFIG.id}/preview-template/${template.id}`
+        );
+      }
+
+      await expect(table.getByText(templates.APP.name)).toBeHidden();
+
+      const submitButton = page.getByTestId('submit-button');
+      await expect(submitButton).toBeVisible();
+      await expect(submitButton).toHaveAttribute('name', 'Save and continue');
+
+      const goBackLink = page.getByRole('link', { name: 'Go back' });
+      await expect(goBackLink).toBeVisible();
+      await expect(goBackLink).toHaveAttribute(
+        'href',
+        `/message-plans/overview/${messagePlans.EMAIL_ROUTING_CONFIG.id}`
+      );
+    });
+
+    await test.step('errors on no selection', async () => {
+      await page.getByTestId('submit-button').click();
+
+      await expect(page).toHaveURL(
+        `${baseURL}/templates/message-plans/choose-email-template/${messagePlans.EMAIL_ROUTING_CONFIG.id}`
+      );
+
+      await expect(chooseEmailTemplatePage.errorSummary).toBeVisible();
+      await expect(chooseEmailTemplatePage.errorSummaryList).toHaveText([
+        'Choose an email template',
+      ]);
+    });
+
+    await test.step('pre-selects previously selected template', async () => {
+      await page.getByTestId(`${templates.EMAIL2.id}-radio`).check();
+      await page.getByTestId('submit-button').click();
+
+      await expect(page).toHaveURL(
+        `${baseURL}/templates/message-plans/overview/${messagePlans.EMAIL_ROUTING_CONFIG.id}`
+      );
+
+      await chooseEmailTemplatePage.loadPage(
+        messagePlans.EMAIL_ROUTING_CONFIG.id
+      );
+
+      // Check summary list is present and displays the name of the previously selected template
+      const summaryList = page.getByTestId('previous-selection-summary');
+      await expect(summaryList).toBeVisible();
+      await expect(summaryList).toContainText('Previously selected template');
+      await expect(summaryList).toContainText(templates.EMAIL2.name);
+
+      const selectedRadio = page.getByTestId(`${templates.EMAIL2.id}-radio`);
+      await expect(selectedRadio).toBeChecked();
+    });
+  });
+
+  test.describe('redirects to invalid message plan page', () => {
+    test('when message plan cannot be found', async ({ page, baseURL }) => {
+      const chooseEmailTemplatePage = new ChooseEmailTemplatePage(page);
+
+      await chooseEmailTemplatePage.loadPage(notFoundMessagePlanId);
+
+      await expect(page).toHaveURL(
+        `${baseURL}/templates/message-plans/invalid`
+      );
+    });
+
+    test('when routing config ID is invalid', async ({ page, baseURL }) => {
+      const chooseEmailTemplatePage = new ChooseEmailTemplatePage(page);
+
+      await chooseEmailTemplatePage.loadPage(invalidMessagePlanId);
+
+      await expect(page).toHaveURL(
+        `${baseURL}/templates/message-plans/invalid`
+      );
+    });
+
+    test('when routing config does not have an email channel', async ({
+      page,
+      baseURL,
+    }) => {
+      const chooseEmailTemplatePage = new ChooseEmailTemplatePage(page);
+
+      await chooseEmailTemplatePage.loadPage(
+        messagePlans.NON_EMAIL_ROUTING_CONFIG.id
+      );
+
+      await expect(page).toHaveURL(
+        `${baseURL}/templates/message-plans/invalid`
+      );
+    });
+  });
+});
diff --git a/tests/test-team/template-mgmt-routing-component-tests/choose-nhs-app-template.routing-component.spec.ts b/tests/test-team/template-mgmt-routing-component-tests/choose-nhs-app-template.routing-component.spec.ts
diff --git a/tests/test-team/template-mgmt-routing-component-tests/choose-standard-english-letter-template.routing-component.spec.ts b/tests/test-team/template-mgmt-routing-component-tests/choose-standard-english-letter-template.routing-component.spec.ts
diff --git a/tests/test-team/template-mgmt-routing-component-tests/choose-text-message-template.routing-component.spec.ts b/tests/test-team/template-mgmt-routing-component-tests/choose-text-message-template.routing-component.spec.ts

Original file line number	Diff line number	Diff line change
`@@ -54,7 +54,7 @@ export function ChooseChannelTemplate(props: ChooseChannelTemplateProps) {`
`54`	`54`	`formAttributes={{ onSubmit: formValidate }}`
`55`	`55`	`>`
`56`	`56`	`{selectedTemplateId && (`
`57`		`- <SummaryList>`
	`57`	`+ <SummaryList data-testid='previous-selection-summary'>`
`58`	`58`	`<SummaryList.Row>`
`59`	`59`	`<SummaryList.Key>`
`60`	`60`	`{content.previousSelectionLabel}`
Original file line number	Diff line number	Diff line change
`@@ -99,7 +99,7 @@ export function ChannelTemplates({`
`99`	`99`	`})}`
`100`	`100`	id={`preview-template-link-${index}`}
`101`	`101`	aria-label={`${tableContent.action.preview.text}`}
`102`		`- data-testid='preview-link'`
	`102`	+ data-testid={`${template.id}-preview-link`}
`103`	`103`	`>`
`104`	`104`	`{tableContent.action.preview.text}`
`105`	`105`	`</Link>`