CCM-11492 Pass message plan ID through for URLs

Author: nicki-nhs

frontend/src/__tests__/components/molecules/MessagePlanBlock.test.tsx

@@ -24,7 +24,11 @@ describe('MessagePlanBlock', () => {
     const channelItem = buildCascadeItem('EMAIL');
 
     const { container } = render(
-      <MessagePlanBlock index={0} channelItem={channelItem} />
+      <MessagePlanBlock
+        index={0}
+        channelItem={channelItem}
+        routingConfigId='test-routing-config-id'
+      />
     );
 
     const stepNumber = container.querySelector('.message-plan-block-number');
@@ -39,7 +43,11 @@ describe('MessagePlanBlock', () => {
     const channelItem = buildCascadeItem('NHSAPP');
 
     const { container } = render(
-      <MessagePlanBlock index={2} channelItem={channelItem} />
+      <MessagePlanBlock
+        index={2}
+        channelItem={channelItem}
+        routingConfigId='test-routing-config-id'
+      />
     );
 
     const stepNumber = container.querySelector('.message-plan-block-number');
@@ -52,7 +60,13 @@ describe('MessagePlanBlock', () => {
   it('should render the channel template section with the correct channel subheading', () => {
     const channelItem = buildCascadeItem('EMAIL');
 
-    render(<MessagePlanBlock index={0} channelItem={channelItem} />);
+    render(
+      <MessagePlanBlock
+        index={0}
+        channelItem={channelItem}
+        routingConfigId='test-routing-config-id'
+      />
+    );
 
     expect(
       screen.getByRole('heading', { level: 3, name: 'Email' })
@@ -68,6 +82,7 @@ describe('MessagePlanBlock', () => {
           index={0}
           channelItem={channelItem}
           template={mockTemplate}
+          routingConfigId='test-routing-config-id'
         />
       );
       expect(screen.getByText('Test email template')).toBeInTheDocument();
@@ -81,14 +96,15 @@ describe('MessagePlanBlock', () => {
           index={0}
           channelItem={channelItem}
           template={mockTemplate}
+          routingConfigId='test-routing-config-id'
         />
       );
 
       expect(
         screen.getByRole('link', { name: 'Change Email template' })
       ).toBeInTheDocument();
       expect(
-        screen.getByRole('link', { name: 'Remove Email template' })
+        screen.getByRole('button', { name: 'Remove Email template' })
       ).toBeInTheDocument();
       expect(
         screen.queryByRole('link', { name: 'Choose Email template' })
@@ -100,7 +116,13 @@ describe('MessagePlanBlock', () => {
     it('should show Choose link (and no Change/Remove links)', () => {
       const channelItem = buildCascadeItem('SMS');
 
-      render(<MessagePlanBlock index={0} channelItem={channelItem} />);
+      render(
+        <MessagePlanBlock
+          index={0}
+          channelItem={channelItem}
+          routingConfigId='test-routing-config-id'
+        />
+      );
 
       expect(
         screen.getByRole('link', { name: 'Choose Text message (SMS) template' })
@@ -128,6 +150,7 @@ describe('MessagePlanBlock', () => {
             index={0}
             channelItem={channelItem}
             template={mockTemplate}
+            routingConfigId='test-routing-config-id'
           />
         );
         expect(asFragment()).toMatchSnapshot();
@@ -141,7 +164,11 @@ describe('MessagePlanBlock', () => {
       it('should match snapshot', async () => {
         const channelItem = buildCascadeItem(channel);
         const { asFragment } = render(
-          <MessagePlanBlock index={0} channelItem={channelItem} />
+          <MessagePlanBlock
+            index={0}
+            channelItem={channelItem}
+            routingConfigId='test-routing-config-id'
+          />
         );
         expect(asFragment()).toMatchSnapshot();
       });

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

@@ -3,8 +3,15 @@ import { MessagePlanChannelTemplate } from '@molecules/MessagePlanChannelTemplat
 import type { TemplateDto } from 'nhs-notify-backend-client';
 
 describe('MessagePlanChannelTemplate', () => {
+  const routingConfigId = 'test-routing-config-id';
+
   it('should display the channel heading', () => {
-    render(<MessagePlanChannelTemplate channel='EMAIL' />);
+    render(
+      <MessagePlanChannelTemplate
+        channel='EMAIL'
+        routingConfigId={routingConfigId}
+      />
+    );
 
     expect(
       screen.getByRole('heading', { level: 3, name: 'Email' })
@@ -13,7 +20,13 @@ describe('MessagePlanChannelTemplate', () => {
 
   describe('when the channel is not required', () => {
     beforeEach(() => {
-      render(<MessagePlanChannelTemplate channel='LETTER' required={false} />);
+      render(
+        <MessagePlanChannelTemplate
+          channel='LETTER'
+          required={false}
+          routingConfigId={routingConfigId}
+        />
+      );
     });
 
     it('should display the heading with the "(optional)" suffix', () => {
@@ -25,7 +38,12 @@ describe('MessagePlanChannelTemplate', () => {
 
   describe('when no template is selected', () => {
     beforeEach(() => {
-      render(<MessagePlanChannelTemplate channel='NHSAPP' />);
+      render(
+        <MessagePlanChannelTemplate
+          channel='NHSAPP'
+          routingConfigId={routingConfigId}
+        />
+      );
     });
 
     it('should show the "Choose template" link with accessible name and href', () => {
@@ -34,7 +52,7 @@ describe('MessagePlanChannelTemplate', () => {
       });
       expect(link).toHaveAttribute(
         'href',
-        '/message-plans/choose-nhs-app-template'
+        `/message-plans/choose-nhs-app-template/${routingConfigId}`
       );
     });
 
@@ -56,7 +74,11 @@ describe('MessagePlanChannelTemplate', () => {
 
     beforeEach(() => {
       render(
-        <MessagePlanChannelTemplate channel='SMS' template={testTemplate} />
+        <MessagePlanChannelTemplate
+          channel='SMS'
+          template={testTemplate}
+          routingConfigId={routingConfigId}
+        />
       );
     });
 
@@ -70,7 +92,7 @@ describe('MessagePlanChannelTemplate', () => {
       });
       expect(link).toHaveAttribute(
         'href',
-        '/message-plans/choose-text-message-template'
+        `/message-plans/choose-text-message-template/${routingConfigId}`
       );
     });
 
@@ -95,7 +117,10 @@ describe('MessagePlanChannelTemplate', () => {
     'should match snapshot for empty state (%s)',
     (channel) => {
       const { container } = render(
-        <MessagePlanChannelTemplate channel={channel} />
+        <MessagePlanChannelTemplate
+          channel={channel}
+          routingConfigId={routingConfigId}
+        />
       );
 
       expect(container).toMatchSnapshot();
@@ -111,7 +136,11 @@ describe('MessagePlanChannelTemplate', () => {
       } as TemplateDto;
 
       const { container } = render(
-        <MessagePlanChannelTemplate channel={channel} template={testTemplate} />
+        <MessagePlanChannelTemplate
+          channel={channel}
+          template={testTemplate}
+          routingConfigId={routingConfigId}
+        />
       );
       expect(container).toMatchSnapshot();
     }