CCCM-11494 component test

Author: nicki-nhs

frontend/src/components/molecules/MessagePlanChannelTemplate/MessagePlanChannelTemplate.tsx

@@ -42,7 +42,10 @@ function MessagePlanChannelTemplateBase({
   const hasTemplates = templateCount > 0;
 
   return (
-    <div className={styles['channel-template-outer']}>
+    <div
+      className={styles['channel-template-outer']}
+      data-testid={`channel-template-${testIdSuffix}`}
+    >
       <div className={styles['channel-template-inner']}>
         <h3 className='nhsuk-heading-s'>{`${channelTemplateType}${required ? '' : ' (optional)'}`}</h3>
 

frontend/src/content/content.ts

@@ -1,6 +1,10 @@
 import type { ContentBlock } from '@molecules/ContentRenderer/ContentRenderer';
 import { getBasePath } from '@utils/get-base-path';
-import { TemplateStatus, TemplateType } from 'nhs-notify-backend-client';
+import {
+  LetterType,
+  TemplateStatus,
+  TemplateType,
+} from 'nhs-notify-backend-client';
 
 const generatePageTitle = (title: string): string => {
   return `${title} - NHS Notify`;

tests/test-team/helpers/factories/routing-config-factory.ts

@@ -31,9 +31,7 @@ export const RoutingConfigFactory = {
           defaultTemplateId: null,
         },
       ],
-      cascadeGroupOverrides: routingConfig.cascadeGroupOverrides ?? [
-        { name: 'standard' },
-      ],
+      cascadeGroupOverrides: routingConfig.cascadeGroupOverrides ?? [],
       name: routingConfig.name ?? 'Test config',
     };
 

tests/test-team/helpers/types.ts

@@ -24,7 +24,7 @@ export const expectedChannelLabels: Record<Channel, string> = {
   NHSAPP: 'NHS App',
   SMS: 'Text message (SMS)',
   EMAIL: 'Email',
-  LETTER: 'Letter',
+  LETTER: 'Standard English letter',
 };
 
 export const allChannels: Channel[] = ['NHSAPP', 'EMAIL', 'SMS', 'LETTER'];

tests/test-team/pages/routing/choose-templates-page.ts

@@ -18,6 +18,8 @@ export class RoutingChooseTemplatesPage extends TemplateMgmtBasePageDynamic {
 
   public readonly saveAndCloseButton: Locator;
 
+  public readonly conditionalLetterTemplates: Locator;
+
   constructor(page: Page) {
     super(page);
     this.errorSummary = page.locator('.nhsuk-error-summary');
@@ -29,29 +31,29 @@ export class RoutingChooseTemplatesPage extends TemplateMgmtBasePageDynamic {
     this.channelBlocks = page.locator('[data-testid^="message-plan-block-"]');
     this.moveToProductionButton = page.getByTestId('move-to-production-cta');
     this.saveAndCloseButton = page.getByTestId('save-and-close-cta');
+    this.conditionalLetterTemplates = page.getByTestId(
+      'message-plan-conditional-templates'
+    );
   }
 
-  public messagePlanChannel(channel: string) {
+  public messagePlanItem(identifier: string) {
     return {
-      block: this.page.getByTestId(`message-plan-block-${channel}`),
-      number: this.page
-        .getByTestId(`message-plan-block-${channel}`)
-        .locator('[class*=message-plan-block-number]'),
+      templateItem: this.page.getByTestId(`channel-template-${identifier}`),
       heading: this.page
-        .getByTestId(`message-plan-block-${channel}`)
+        .getByTestId(`channel-template-${identifier}`)
         .getByRole('heading', { level: 3 }),
-      templateName: this.page.getByTestId(`template-name-${channel}`),
-      fallbackConditions: this.page.getByTestId(
-        `message-plan-fallback-conditions-${channel}`
-      ),
+      templateName: this.page
+        .getByTestId(`template-name-${identifier}`)
+        .first(),
+      templateNames: this.page.getByTestId(`template-name-${identifier}`),
       changeTemplateLink: this.page.getByTestId(
-        `change-template-link-${channel}`
+        `change-template-link-${identifier}`
       ),
       chooseTemplateLink: this.page.getByTestId(
-        `choose-template-link-${channel}`
+        `choose-template-link-${identifier}`
       ),
       removeTemplateLink: this.page.getByTestId(
-        `remove-template-link-${channel}`
+        `remove-template-link-${identifier}`
       ),
       async clickChooseTemplateLink() {
         await this.chooseTemplateLink.click();
@@ -65,6 +67,19 @@ export class RoutingChooseTemplatesPage extends TemplateMgmtBasePageDynamic {
     };
   }
 
+  public messagePlanChannel(channel: string) {
+    return {
+      ...this.messagePlanItem(channel),
+      block: this.page.getByTestId(`message-plan-block-${channel}`),
+      number: this.page
+        .getByTestId(`message-plan-block-${channel}`)
+        .locator('[class*=message-plan-block-number]'),
+      fallbackConditions: this.page.getByTestId(
+        `message-plan-fallback-conditions-${channel}`
+      ),
+    };
+  }
+
   public readonly nhsApp = this.messagePlanChannel('NHSAPP');
 
   public readonly sms = this.messagePlanChannel('SMS');
@@ -73,6 +88,23 @@ export class RoutingChooseTemplatesPage extends TemplateMgmtBasePageDynamic {
 
   public readonly letter = this.messagePlanChannel('LETTER');
 
+  public alternativeLetterFormats() {
+    const conditionalTemplates = this.page.getByTestId(
+      'message-plan-conditional-templates'
+    );
+    return {
+      conditionalTemplates,
+      fallbackConditions: conditionalTemplates.getByTestId(
+        'message-plan-fallback-conditions-LETTER'
+      ),
+      listItems: conditionalTemplates.locator(
+        '[class*=message-plan-conditional-templates__list-item]'
+      ),
+      largePrint: this.messagePlanItem('x1'),
+      otherLanguages: this.messagePlanItem('foreign-language'),
+    };
+  }
+
   async clickMoveToProduction() {
     await this.moveToProductionButton.click();
   }

tests/test-team/template-mgmt-api-tests/create-routing-configuration.api.spec.ts

@@ -77,7 +77,6 @@ test.describe('POST /v1/routing-configuration', () => {
   }) => {
     const payload = RoutingConfigFactory.create(user1, {
       cascadeGroupOverrides: [
-        { name: 'standard' },
         { name: 'translations', language: ['ar'] },
         { name: 'accessible', accessibleFormat: ['x0'] },
       ],