CCM-11544 Routing playwright tests

nicki-nhs · nicki-nhs · commit a5c4476a2769 · 2025-12-08T19:07:08.000Z
diff --git a/frontend/src/__tests__/app/message-plans/choose-email-template/__snapshots__/page.test.tsx.snap b/frontend/src/__tests__/app/message-plans/choose-email-template/__snapshots__/page.test.tsx.snap
@@ -44,7 +44,7 @@ exports[`ChooseEmailTemplate page renders Email template selection 1`] = `
           />
           <dl
             class="nhsuk-summary-list"
-            data-testid="previous-selection-summary"
+            data-testid="previous-selection-details"
           >
             <div
               class="nhsuk-summary-list__row"
diff --git a/frontend/src/__tests__/app/message-plans/choose-nhs-app-template/__snapshots__/page.test.tsx.snap b/frontend/src/__tests__/app/message-plans/choose-nhs-app-template/__snapshots__/page.test.tsx.snap
@@ -44,7 +44,7 @@ exports[`ChooseNHSAppTemplate page renders NHS App template selection 1`] = `
           />
           <dl
             class="nhsuk-summary-list"
-            data-testid="previous-selection-summary"
+            data-testid="previous-selection-details"
           >
             <div
               class="nhsuk-summary-list__row"
diff --git a/frontend/src/__tests__/app/message-plans/choose-standard-english-letter-template/__snapshots__/page.test.tsx.snap b/frontend/src/__tests__/app/message-plans/choose-standard-english-letter-template/__snapshots__/page.test.tsx.snap
@@ -44,7 +44,7 @@ exports[`ChooseStandardEnglishLetterTemplate page renders letter template select
           />
           <dl
             class="nhsuk-summary-list"
-            data-testid="previous-selection-summary"
+            data-testid="previous-selection-details"
           >
             <div
               class="nhsuk-summary-list__row"
diff --git a/frontend/src/__tests__/app/message-plans/choose-text-message-template/__snapshots__/page.test.tsx.snap b/frontend/src/__tests__/app/message-plans/choose-text-message-template/__snapshots__/page.test.tsx.snap
@@ -44,7 +44,7 @@ exports[`ChooseTextMessageTemplate page renders sms template selection 1`] = `
           />
           <dl
             class="nhsuk-summary-list"
-            data-testid="previous-selection-summary"
+            data-testid="previous-selection-details"
           >
             <div
               class="nhsuk-summary-list__row"
diff --git a/frontend/src/__tests__/components/forms/ChooseChannelTemplate/ChooseChannelTemplate.test.tsx b/frontend/src/__tests__/components/forms/ChooseChannelTemplate/ChooseChannelTemplate.test.tsx
@@ -259,7 +259,7 @@ describe('ChooseChannelTemplate', () => {
       });
 
       expect(
-        screen.queryByTestId('previous-selection-summary')
+        screen.queryByTestId('previous-selection-details')
       ).not.toBeInTheDocument();
     });
 
@@ -285,7 +285,7 @@ describe('ChooseChannelTemplate', () => {
     it('displays previously selected template', () => {
       renderComponent();
 
-      const summary = screen.getByTestId('previous-selection-summary');
+      const summary = screen.getByTestId('previous-selection-details');
       expect(summary).toBeInTheDocument();
       expect(
         screen.getByText('Previously selected template')
@@ -341,7 +341,7 @@ describe('ChooseChannelTemplate', () => {
         ),
       });
 
-      const summary = screen.getByTestId('previous-selection-summary');
+      const summary = screen.getByTestId('previous-selection-details');
 
       expect(summary).toBeInTheDocument();
       expect(
diff --git a/frontend/src/__tests__/components/forms/ChooseChannelTemplate/__snapshots__/ChooseChannelTemplate.test.tsx.snap b/frontend/src/__tests__/components/forms/ChooseChannelTemplate/__snapshots__/ChooseChannelTemplate.test.tsx.snap
@@ -323,7 +323,7 @@ exports[`ChooseChannelTemplate renders email form 1`] = `
           />
           <dl
             class="nhsuk-summary-list"
-            data-testid="previous-selection-summary"
+            data-testid="previous-selection-details"
           >
             <div
               class="nhsuk-summary-list__row"
@@ -1096,7 +1096,7 @@ exports[`ChooseChannelTemplate renders letter form 1`] = `
           />
           <dl
             class="nhsuk-summary-list"
-            data-testid="previous-selection-summary"
+            data-testid="previous-selection-details"
           >
             <div
               class="nhsuk-summary-list__row"
@@ -1353,7 +1353,7 @@ exports[`ChooseChannelTemplate renders nhs app form 1`] = `
           />
           <dl
             class="nhsuk-summary-list"
-            data-testid="previous-selection-summary"
+            data-testid="previous-selection-details"
           >
             <div
               class="nhsuk-summary-list__row"
@@ -1610,7 +1610,7 @@ exports[`ChooseChannelTemplate renders sms form 1`] = `
           />
           <dl
             class="nhsuk-summary-list"
-            data-testid="previous-selection-summary"
+            data-testid="previous-selection-details"
           >
             <div
               class="nhsuk-summary-list__row"
@@ -1867,7 +1867,7 @@ exports[`ChooseChannelTemplate when templates are available renders multiple opt
           />
           <dl
             class="nhsuk-summary-list"
-            data-testid="previous-selection-summary"
+            data-testid="previous-selection-details"
           >
             <div
               class="nhsuk-summary-list__row"
@@ -2381,7 +2381,7 @@ exports[`ChooseChannelTemplate when there is a conditional template selected ren
           />
           <dl
             class="nhsuk-summary-list"
-            data-testid="previous-selection-summary"
+            data-testid="previous-selection-details"
           >
             <div
               class="nhsuk-summary-list__row"
@@ -2822,7 +2822,7 @@ exports[`ChooseChannelTemplate when there is a default template selected renders
           />
           <dl
             class="nhsuk-summary-list"
-            data-testid="previous-selection-summary"
+            data-testid="previous-selection-details"
           >
             <div
               class="nhsuk-summary-list__row"
diff --git a/frontend/src/components/forms/ChooseChannelTemplate/ChooseChannelTemplate.tsx b/frontend/src/components/forms/ChooseChannelTemplate/ChooseChannelTemplate.tsx
@@ -71,7 +71,7 @@ export function ChooseChannelTemplate(props: ChooseChannelTemplateProps) {
             formAttributes={{ onSubmit: formValidate }}
           >
             {selectedTemplateId && (
-              <SummaryList data-testid='previous-selection-summary'>
+              <SummaryList data-testid='previous-selection-details'>
                 <SummaryList.Row>
                   <SummaryList.Key>
                     {content.previousSelectionLabel}
diff --git a/tests/test-team/pages/routing/email/choose-email-template-page.ts b/tests/test-team/pages/routing/email/choose-email-template-page.ts
@@ -1,7 +1,7 @@
 import { type Page } from '@playwright/test';
-import { TemplateMgmtBasePageDynamic } from 'pages/template-mgmt-base-page-dynamic';
+import { TemplateMgmtChooseTemplateForMessagePlanBasePage } from 'pages/template-mgmt-choose-template-base-page';
 
-export class RoutingChooseEmailTemplatePage extends TemplateMgmtBasePageDynamic {
+export class RoutingChooseEmailTemplatePage extends TemplateMgmtChooseTemplateForMessagePlanBasePage {
   static readonly pageUrlSegments = ['message-plans/choose-email-template'];
 
   constructor(page: Page) {
diff --git a/tests/test-team/pages/routing/email/preview-email-page.ts b/tests/test-team/pages/routing/email/preview-email-page.ts
@@ -1,7 +1,7 @@
 import { type Page } from '@playwright/test';
-import { TemplateMgmtBasePageDynamic } from 'pages/template-mgmt-base-page-dynamic';
+import { TemplateMgmtPreviewBasePage } from 'pages/template-mgmt-preview-base-page';
 
-export class RoutingPreviewEmailTemplatePage extends TemplateMgmtBasePageDynamic {
+export class RoutingPreviewEmailTemplatePage extends TemplateMgmtPreviewBasePage {
   static readonly pageUrlSegments = [
     'message-plans/choose-email-template',
     'preview-template',
diff --git a/tests/test-team/pages/routing/letter/choose-large-print-letter-template-page.ts b/tests/test-team/pages/routing/letter/choose-large-print-letter-template-page.ts
@@ -0,0 +1,12 @@
+import { type Page } from '@playwright/test';
+import { TemplateMgmtChooseTemplateForMessagePlanBasePage } from 'pages/template-mgmt-choose-template-base-page';
+
+export class RoutingChooseLargePrintLetterTemplatePage extends TemplateMgmtChooseTemplateForMessagePlanBasePage {
+  static readonly pageUrlSegments = [
+    'message-plans/choose-large-print-letter-template',
+  ];
+
+  constructor(page: Page) {
+    super(page);
+  }
+}
diff --git a/tests/test-team/pages/routing/letter/choose-other-language-letter-template-page.ts b/tests/test-team/pages/routing/letter/choose-other-language-letter-template-page.ts
@@ -0,0 +1,22 @@
+import { Locator, type Page } from '@playwright/test';
+import { TemplateMgmtChooseTemplateForMessagePlanBasePage } from 'pages/template-mgmt-choose-template-base-page';
+
+export class RoutingChooseOtherLanguageLetterTemplatePage extends TemplateMgmtChooseTemplateForMessagePlanBasePage {
+  static readonly pageUrlSegments = [
+    'message-plans/choose-other-language-letter-template',
+  ];
+
+  public readonly tableRows: Locator;
+
+  constructor(page: Page) {
+    super(page);
+
+    this.formError = page.locator('#language-templates--error-message');
+    this.templatesTable = page.getByTestId('language-templates-table');
+    this.tableRows = this.templatesTable.locator('tbody tr');
+  }
+
+  getCheckbox(templateId: string): Locator {
+    return this.page.getByTestId(`${templateId}-checkbox`);
+  }
+}
diff --git a/tests/test-team/pages/routing/letter/choose-standard-letter-template-page.ts b/tests/test-team/pages/routing/letter/choose-standard-letter-template-page.ts
@@ -1,7 +1,7 @@
 import { type Page } from '@playwright/test';
-import { TemplateMgmtBasePageDynamic } from 'pages/template-mgmt-base-page-dynamic';
+import { TemplateMgmtChooseTemplateForMessagePlanBasePage } from 'pages/template-mgmt-choose-template-base-page';
 
-export class RoutingChooseStandardLetterTemplatePage extends TemplateMgmtBasePageDynamic {
+export class RoutingChooseStandardLetterTemplatePage extends TemplateMgmtChooseTemplateForMessagePlanBasePage {
   static readonly pageUrlSegments = [
     'message-plans/choose-standard-english-letter-template',
   ];
diff --git a/tests/test-team/pages/routing/letter/preview-large-print-letter-template-page.ts b/tests/test-team/pages/routing/letter/preview-large-print-letter-template-page.ts
@@ -0,0 +1,13 @@
+import { type Page } from '@playwright/test';
+import { TemplateMgmtPreviewBasePage } from 'pages/template-mgmt-preview-base-page';
+
+export class RoutingPreviewLargePrintLetterTemplatePage extends TemplateMgmtPreviewBasePage {
+  static readonly pageUrlSegments = [
+    'message-plans/choose-large-print-letter-template',
+    'preview-template',
+  ];
+
+  constructor(page: Page) {
+    super(page);
+  }
+}
diff --git a/tests/test-team/pages/routing/letter/preview-other-language-letter-template-page.ts b/tests/test-team/pages/routing/letter/preview-other-language-letter-template-page.ts
@@ -0,0 +1,13 @@
+import { type Page } from '@playwright/test';
+import { TemplateMgmtPreviewBasePage } from 'pages/template-mgmt-preview-base-page';
+
+export class RoutingPreviewOtherLanguageLetterTemplatePage extends TemplateMgmtPreviewBasePage {
+  static readonly pageUrlSegments = [
+    'message-plans/choose-other-language-letter-template',
+    'preview-template',
+  ];
+
+  constructor(page: Page) {
+    super(page);
+  }
+}
diff --git a/tests/test-team/pages/routing/letter/preview-standard-letter-page.ts b/tests/test-team/pages/routing/letter/preview-standard-letter-page.ts
@@ -1,7 +1,7 @@
 import { type Page } from '@playwright/test';
-import { TemplateMgmtBasePageDynamic } from 'pages/template-mgmt-base-page-dynamic';
+import { TemplateMgmtPreviewBasePage } from 'pages/template-mgmt-preview-base-page';
 
-export class RoutingPreviewStandardLetterTemplatePage extends TemplateMgmtBasePageDynamic {
+export class RoutingPreviewStandardLetterTemplatePage extends TemplateMgmtPreviewBasePage {
   static readonly pageUrlSegments = [
     'message-plans/choose-standard-english-letter-template',
     'preview-template',
diff --git a/tests/test-team/pages/routing/nhs-app/choose-nhs-app-template-page.ts b/tests/test-team/pages/routing/nhs-app/choose-nhs-app-template-page.ts
@@ -1,7 +1,7 @@
 import { type Page } from '@playwright/test';
-import { TemplateMgmtBasePageDynamic } from 'pages/template-mgmt-base-page-dynamic';
+import { TemplateMgmtChooseTemplateForMessagePlanBasePage } from 'pages/template-mgmt-choose-template-base-page';
 
-export class RoutingChooseNhsAppTemplatePage extends TemplateMgmtBasePageDynamic {
+export class RoutingChooseNhsAppTemplatePage extends TemplateMgmtChooseTemplateForMessagePlanBasePage {
   static readonly pageUrlSegments = ['message-plans/choose-nhs-app-template'];
 
   constructor(page: Page) {
diff --git a/tests/test-team/pages/routing/nhs-app/preview-nhs-app-page.ts b/tests/test-team/pages/routing/nhs-app/preview-nhs-app-page.ts
@@ -1,7 +1,7 @@
 import { type Page } from '@playwright/test';
-import { TemplateMgmtBasePageDynamic } from 'pages/template-mgmt-base-page-dynamic';
+import { TemplateMgmtPreviewBasePage } from 'pages/template-mgmt-preview-base-page';
 
-export class RoutingPreviewNhsAppTemplatePage extends TemplateMgmtBasePageDynamic {
+export class RoutingPreviewNhsAppTemplatePage extends TemplateMgmtPreviewBasePage {
   static readonly pageUrlSegments = [
     'message-plans/choose-nhs-app-template',
     'preview-template',
diff --git a/tests/test-team/pages/routing/sms/choose-sms-template-page.ts b/tests/test-team/pages/routing/sms/choose-sms-template-page.ts
@@ -1,7 +1,7 @@
 import { type Page } from '@playwright/test';
-import { TemplateMgmtBasePageDynamic } from 'pages/template-mgmt-base-page-dynamic';
+import { TemplateMgmtChooseTemplateForMessagePlanBasePage } from 'pages/template-mgmt-choose-template-base-page';
 
-export class RoutingChooseTextMessageTemplatePage extends TemplateMgmtBasePageDynamic {
+export class RoutingChooseTextMessageTemplatePage extends TemplateMgmtChooseTemplateForMessagePlanBasePage {
   static readonly pageUrlSegments = [
     'message-plans/choose-text-message-template',
   ];
diff --git a/tests/test-team/pages/routing/sms/preview-sms-template-page.ts b/tests/test-team/pages/routing/sms/preview-sms-template-page.ts
@@ -1,7 +1,7 @@
 import { type Page } from '@playwright/test';
-import { TemplateMgmtBasePageDynamic } from 'pages/template-mgmt-base-page-dynamic';
+import { TemplateMgmtPreviewBasePage } from 'pages/template-mgmt-preview-base-page';
 
-export class RoutingPreviewSmsTemplatePage extends TemplateMgmtBasePageDynamic {
+export class RoutingPreviewSmsTemplatePage extends TemplateMgmtPreviewBasePage {
   static readonly pageUrlSegments = [
     'message-plans/choose-text-message-template',
     'preview-template',
diff --git a/tests/test-team/pages/template-mgmt-choose-template-base-page.ts b/tests/test-team/pages/template-mgmt-choose-template-base-page.ts
@@ -0,0 +1,42 @@
+import { Locator, Page } from '@playwright/test';
+import { TemplateMgmtBasePageDynamic } from './template-mgmt-base-page-dynamic';
+
+export abstract class TemplateMgmtChooseTemplateForMessagePlanBasePage extends TemplateMgmtBasePageDynamic {
+  readonly messagePlanName: Locator;
+
+  readonly tableHintText: Locator;
+
+  readonly saveAndContinueButton: Locator;
+
+  readonly noTemplatesMessage: Locator;
+
+  readonly goToTemplatesLink: Locator;
+
+  readonly previousSelectionDetails: Locator;
+
+  formError: Locator;
+
+  templatesTable: Locator;
+
+  constructor(page: Page) {
+    super(page);
+    this.messagePlanName = page.locator('.nhsuk-caption-l');
+    this.tableHintText = page.getByTestId('table-hint');
+    this.saveAndContinueButton = page.getByTestId('submit-button');
+    this.noTemplatesMessage = page.getByTestId('no-templates-message');
+    this.goToTemplatesLink = page.getByTestId('go-to-templates-link');
+    this.previousSelectionDetails = page.getByTestId(
+      'previous-selection-details'
+    );
+    this.formError = page.locator('#channel-template--error-message');
+    this.templatesTable = page.getByTestId('channel-templates-table');
+  }
+
+  getPreviewLink(templateId: string): Locator {
+    return this.page.getByTestId(`${templateId}-preview-link`);
+  }
+
+  getRadioButton(templateId: string): Locator {
+    return this.page.getByTestId(`${templateId}-radio`);
+  }
+}
diff --git a/tests/test-team/pages/template-mgmt-preview-base-page.ts b/tests/test-team/pages/template-mgmt-preview-base-page.ts
@@ -2,19 +2,19 @@ import { Locator, Page } from '@playwright/test';
 import { TemplateMgmtBasePageDynamic } from './template-mgmt-base-page-dynamic';
 
 export abstract class TemplateMgmtPreviewBasePage extends TemplateMgmtBasePageDynamic {
-  readonly backToAllTemplatesLinks: Locator;
+  readonly templateCaption: Locator;
 
-  constructor(page: Page) {
-    super(page);
+  readonly templateId: Locator;
 
-    this.backToAllTemplatesLinks = page.getByText('Back to all templates');
-  }
+  readonly campaignId: Locator;
 
-  async clickBackToAllTemplatesTopLink() {
-    await this.backToAllTemplatesLinks.nth(0).click();
-  }
+  readonly summaryList: Locator;
 
-  async clickBackToAllTemplatesBottomLink() {
-    await this.backToAllTemplatesLinks.nth(1).click();
+  constructor(page: Page) {
+    super(page);
+    this.templateCaption = page.locator('.nhsuk-caption-l');
+    this.templateId = page.getByTestId('preview-template-id');
+    this.campaignId = page.locator('[id="campaign-id"]');
+    this.summaryList = page.locator('dl.nhsuk-summary-list');
   }
 }
diff --git a/tests/test-team/template-mgmt-routing-component-tests/email/choose-email-template.routing-component.spec.ts b/tests/test-team/template-mgmt-routing-component-tests/email/choose-email-template.routing-component.spec.ts
diff --git a/tests/test-team/template-mgmt-routing-component-tests/letter/choose-large-print-letter-template.routing-component.spec.ts b/tests/test-team/template-mgmt-routing-component-tests/letter/choose-large-print-letter-template.routing-component.spec.ts
diff --git a/tests/test-team/template-mgmt-routing-component-tests/letter/choose-other-language-letter-template.routing-component.spec.ts b/tests/test-team/template-mgmt-routing-component-tests/letter/choose-other-language-letter-template.routing-component.spec.ts
diff --git a/tests/test-team/template-mgmt-routing-component-tests/letter/choose-standard-letter-template.routing-component.spec.ts b/tests/test-team/template-mgmt-routing-component-tests/letter/choose-standard-letter-template.routing-component.spec.ts
diff --git a/tests/test-team/template-mgmt-routing-component-tests/letter/preview-large-print-letter-template.routing-component.spec.ts b/tests/test-team/template-mgmt-routing-component-tests/letter/preview-large-print-letter-template.routing-component.spec.ts
diff --git a/tests/test-team/template-mgmt-routing-component-tests/letter/preview-other-language-letter-template.routing-component.spec.ts b/tests/test-team/template-mgmt-routing-component-tests/letter/preview-other-language-letter-template.routing-component.spec.ts
diff --git a/tests/test-team/template-mgmt-routing-component-tests/letter/preview-standard-letter-template.routing-component.spec.ts b/tests/test-team/template-mgmt-routing-component-tests/letter/preview-standard-letter-template.routing-component.spec.ts
diff --git a/tests/test-team/template-mgmt-routing-component-tests/nhs-app/choose-nhs-app-template.routing-component.spec.ts b/tests/test-team/template-mgmt-routing-component-tests/nhs-app/choose-nhs-app-template.routing-component.spec.ts
diff --git a/tests/test-team/template-mgmt-routing-component-tests/routing-common.steps.ts b/tests/test-team/template-mgmt-routing-component-tests/routing-common.steps.ts
diff --git a/tests/test-team/template-mgmt-routing-component-tests/sms/choose-sms-template.routing-component.spec.ts b/tests/test-team/template-mgmt-routing-component-tests/sms/choose-sms-template.routing-component.spec.ts

Original file line number	Diff line number	Diff line change
@@ -44,7 +44,7 @@ exports[`ChooseEmailTemplate page renders Email template selection 1`] = `
`44`	`44`	`/>`
`45`	`45`	`<dl`
`46`	`46`	`class="nhsuk-summary-list"`
`47`		`- data-testid="previous-selection-summary"`
	`47`	`+ data-testid="previous-selection-details"`
`48`	`48`	`>`
`49`	`49`	`<div`
`50`	`50`	`class="nhsuk-summary-list__row"`
Original file line number	Diff line number	Diff line change
@@ -44,7 +44,7 @@ exports[`ChooseNHSAppTemplate page renders NHS App template selection 1`] = `
`44`	`44`	`/>`
`45`	`45`	`<dl`
`46`	`46`	`class="nhsuk-summary-list"`
`47`		`- data-testid="previous-selection-summary"`
	`47`	`+ data-testid="previous-selection-details"`
`48`	`48`	`>`
`49`	`49`	`<div`
`50`	`50`	`class="nhsuk-summary-list__row"`
Original file line number	Diff line number	Diff line change
@@ -44,7 +44,7 @@ exports[`ChooseStandardEnglishLetterTemplate page renders letter template select
`44`	`44`	`/>`
`45`	`45`	`<dl`
`46`	`46`	`class="nhsuk-summary-list"`
`47`		`- data-testid="previous-selection-summary"`
	`47`	`+ data-testid="previous-selection-details"`
`48`	`48`	`>`
`49`	`49`	`<div`
`50`	`50`	`class="nhsuk-summary-list__row"`
Original file line number	Diff line number	Diff line change
@@ -44,7 +44,7 @@ exports[`ChooseTextMessageTemplate page renders sms template selection 1`] = `
`44`	`44`	`/>`
`45`	`45`	`<dl`
`46`	`46`	`class="nhsuk-summary-list"`
`47`		`- data-testid="previous-selection-summary"`
	`47`	`+ data-testid="previous-selection-details"`
`48`	`48`	`>`
`49`	`49`	`<div`
`50`	`50`	`class="nhsuk-summary-list__row"`
Original file line number	Diff line number	Diff line change
`@@ -71,7 +71,7 @@ export function ChooseChannelTemplate(props: ChooseChannelTemplateProps) {`
`71`	`71`	`formAttributes={{ onSubmit: formValidate }}`
`72`	`72`	`>`
`73`	`73`	`{selectedTemplateId && (`
`74`		`- <SummaryList data-testid='previous-selection-summary'>`
	`74`	`+ <SummaryList data-testid='previous-selection-details'>`
`75`	`75`	`<SummaryList.Row>`
`76`	`76`	`<SummaryList.Key>`
`77`	`77`	`{content.previousSelectionLabel}`