CCM-11148: add automated tests skeleton

Author: bhansell1

frontend/src/__tests__/app/message-plans/page.test.tsx

@@ -8,18 +8,15 @@ import {
 } from '@molecules/MessagePlans/MessagePlans';
 import { serverIsFeatureEnabled } from '@utils/server-features';
 import MessagePlansPage, { generateMetadata } from '@app/message-plans/page';
-import { redirect } from 'next/navigation';
 import { ReactElement } from 'react';
 import { RoutingConfig } from 'nhs-notify-backend-client';
 
-jest.mock('next/navigation');
 jest.mock('@utils/form-actions');
 jest.mock('@utils/server-features');
 
 const countRoutingConfigsMock = jest.mocked(countRoutingConfigs);
 const serverIsFeatureEnabledMock = jest.mocked(serverIsFeatureEnabled);
 const getRoutingConfigsMock = jest.mocked(getRoutingConfigs);
-const redirectMock = jest.mocked(redirect);
 
 const buildRoutingConfig = (rc: Partial<RoutingConfig>): RoutingConfig => ({
   campaignId: 'abc',
@@ -48,15 +45,6 @@ describe('MessagePlansPage', () => {
     });
   });
 
-  test('redirects to invalid page when routing feature is disabled', async () => {
-    serverIsFeatureEnabledMock.mockReset();
-    serverIsFeatureEnabledMock.mockResolvedValueOnce(false);
-
-    await MessagePlansPage();
-
-    expect(redirectMock).toHaveBeenCalledWith('/invalid-template', 'replace');
-  });
-
   test('renders the page with drafts and production routing plans', async () => {
     getRoutingConfigsMock.mockResolvedValueOnce([
       buildRoutingConfig({
@@ -88,8 +76,6 @@ describe('MessagePlansPage', () => {
 
     expect(page.props).toBeDefined();
 
-    expect(redirectMock).not.toHaveBeenCalled();
-
     expect(getRoutingConfigsMock).toHaveBeenCalledTimes(1);
 
     expect(countRoutingConfigsMock).toHaveBeenNthCalledWith(1, 'DRAFT');

frontend/src/__tests__/components/molecules/__snapshots__/MessagePlans.test.tsx.snap

@@ -18,6 +18,7 @@ exports[`MessagePlans matches snapshot 1`] = `
         </h1>
         <details
           class="nhsuk-details"
+          id="message-plans-status-info"
         >
           <summary
             class="nhsuk-details__summary"

frontend/src/app/message-plans/page.tsx

@@ -4,8 +4,6 @@ import content from '@content/content';
 import { MessagePlans } from '@molecules/MessagePlans/MessagePlans';
 import { Metadata } from 'next';
 import { countRoutingConfigs, getRoutingConfigs } from '@utils/form-actions';
-import { redirect, RedirectType } from 'next/navigation';
-import { serverIsFeatureEnabled } from '@utils/server-features';
 
 export async function generateMetadata(): Promise<Metadata> {
   return {
@@ -14,12 +12,6 @@ export async function generateMetadata(): Promise<Metadata> {
 }
 
 const MessagePlansPage = async () => {
-  const routing = await serverIsFeatureEnabled('routing');
-
-  if (!routing) {
-    return redirect('/invalid-template', RedirectType.replace);
-  }
-
   const [routingConfigurations, draftCount, completedCount] = await Promise.all(
     [
       getRoutingConfigs(),

frontend/src/components/molecules/MessagePlans/MessagePlans.tsx

@@ -37,7 +37,7 @@ export const MessagePlans = (props: MessagePlansProps) => {
       <div className='nhsuk-grid-row'>
         <div className='nhsuk-grid-column-full'>
           <h1>{pageHeading}</h1>
-          <Details>
+          <Details id='message-plans-status-info'>
             <Details.Summary>
               {messagePlansPage.draftAndProductionInfo.heading}
             </Details.Summary>

lambdas/backend-client/src/__tests__/routing-config-api-client.test.ts

@@ -1,14 +1,21 @@
 import axios from 'axios';
-import { RoutingConfigurationApiClient } from '../routing-config-api-client';
+import {
+  RoutingConfigurationApiClient,
+  routingConfigurationApiClient,
+} from '../routing-config-api-client';
 import MockAdapter from 'axios-mock-adapter';
 
-describe('ClientConfiguration', () => {
+describe('RoutingConfigurationApiClient', () => {
   const axiosMock = new MockAdapter(axios);
 
   beforeEach(() => {
     axiosMock.reset();
   });
 
+  it('should export client', () => {
+    expect(routingConfigurationApiClient).not.toBeUndefined();
+  });
+
   describe('count', () => {
     it('should return error when failing to fetch from API', async () => {
       axiosMock

tests/test-team/pages/routing-message-plans-page.ts

@@ -0,0 +1,28 @@
+import { Locator, type Page } from '@playwright/test';
+import { TemplateMgmtBasePageNonDynamic } from './template-mgmt-base-page-non-dynamic';
+
+export class RoutingMessagePlansPage extends TemplateMgmtBasePageNonDynamic {
+  static readonly pageUrlSegment = 'message-plans';
+
+  readonly messagePlanStatusInfo: Locator;
+
+  readonly newMessagePlanButton: Locator;
+
+  readonly draftMessagePlansTable: Locator;
+
+  readonly productionMessagePlansTable: Locator;
+
+  constructor(page: Page) {
+    super(page);
+    this.messagePlanStatusInfo = page.locator('#message-plans-status-info');
+    this.newMessagePlanButton = page.locator('#create-message-plan-button');
+    this.draftMessagePlansTable = page.locator('#message-plans-list-draft');
+    this.productionMessagePlansTable = page.locator(
+      '#message-plans-list-production'
+    );
+  }
+
+  async clickNewMessagePlanButton() {
+    await this.newMessagePlanButton.click();
+  }
+}

tests/test-team/template-mgmt-routing-component-tests/message-plans.routing-component.spec.ts

@@ -0,0 +1,73 @@
+import { test, expect } from '@playwright/test';
+import { RoutingMessagePlansPage } from '../pages/routing-message-plans-page';
+import {
+  assertFooterLinks,
+  assertGoBackLink,
+  assertSignOutLink,
+  assertHeaderLogoLink,
+  assertSkipToMainContent,
+} from '../helpers/template-mgmt-common.steps';
+import { createAuthHelper, testUsers } from 'helpers/auth/cognito-auth-helper';
+import { RoutingConfigFactory } from 'helpers/factories/routing-config-factory';
+import { RoutingConfigStorageHelper } from 'helpers/db/routing-config-storage-helper';
+import { RoutingConfigDbEntry } from 'helpers/types';
+
+type MessagePlansPageData = {
+  draft: RoutingConfigDbEntry;
+  production: RoutingConfigDbEntry;
+  deleted: RoutingConfigDbEntry;
+};
+
+async function createRoutingConfigs(): Promise<MessagePlansPageData> {
+  const authHelper = createAuthHelper();
+  const user = await authHelper.getTestUser(testUsers.User1.userId);
+
+  return {
+    draft: {
+      ...RoutingConfigFactory.create(user, { status: 'DRAFT' }).dbEntry,
+    },
+    production: {
+      ...RoutingConfigFactory.create(user, { status: 'COMPLETED' }).dbEntry,
+    },
+    deleted: {
+      ...RoutingConfigFactory.create(user, { status: 'DELETED' }).dbEntry,
+    },
+  };
+}
+
+test.describe('Message plans Page', () => {
+  const routingStorageHelper = new RoutingConfigStorageHelper();
+  let routingConfigs: MessagePlansPageData;
+
+  test.beforeAll(async () => {
+    routingConfigs = await createRoutingConfigs();
+    await routingStorageHelper.seed(Object.values(routingConfigs));
+  });
+
+  test('should land on "Message plans" page when navigating to "/templates/message-plans" url', async ({
+    page,
+    baseURL,
+  }) => {
+    const messagePlanPage = new RoutingMessagePlansPage(page);
+
+    await messagePlanPage.loadPage();
+
+    await expect(page).toHaveURL(`${baseURL}/templates/message-plans`);
+    await expect(messagePlanPage.pageHeading).toHaveText('Message plans');
+  });
+
+  test('common page tests', async ({ page, baseURL }) => {
+    const props = {
+      page: new RoutingMessagePlansPage(page),
+      id: '',
+      baseURL,
+      expectedUrl: 'templates/message-plans',
+    };
+
+    await assertSkipToMainContent(props);
+    await assertHeaderLogoLink(props);
+    await assertFooterLinks(props);
+    await assertSignOutLink(props);
+    await assertGoBackLink(props);
+  });
+});