Skip to content

Commit 2241409

Browse files
committed
CCM-8868: Fix unit test
1 parent f5e7906 commit 2241409

File tree

20 files changed

+102
-20
lines changed

20 files changed

+102
-20
lines changed

frontend/src/__tests__/app/choose-a-template-type/page.test.tsx

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,9 @@ import ChooseATemplateTypePage, {
33
generateMetadata,
44
} from '@app/choose-a-template-type/page';
55
import { TemplateFormState } from 'nhs-notify-web-template-management-utils';
6+
import content from '@content/content';
7+
8+
const { pageTitle } = content.components.chooseTemplate;
69

710
jest.mock('next/navigation', () => ({
811
redirect: () => {
@@ -31,11 +34,11 @@ jest.mock('react', () => {
3134
});
3235

3336
test('ChooseATemplateTypePage', async () => {
34-
generateMetadata();
3537
const page = await ChooseATemplateTypePage();
3638

3739
const container = render(page);
3840

41+
expect(await generateMetadata()).toEqual({ title: pageTitle });
3942
expect(container.asFragment()).toMatchSnapshot();
4043
});
4144

frontend/src/__tests__/app/create-email-template/page.test.tsx

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,16 +4,19 @@
44
import CreateEmailTemplatePage, {
55
generateMetadata,
66
} from '@app/create-email-template/page';
7+
import content from '@content/content';
8+
9+
const { pageTitle } = content.components.templateFormEmail;
710

811
jest.mock('@forms/EmailTemplateForm/EmailTemplateForm');
912

1013
describe('CreateEmailTemplatePage', () => {
1114
beforeEach(jest.resetAllMocks);
1215

1316
it('should render CreateEmailTemplatePage', async () => {
14-
generateMetadata();
1517
const page = await CreateEmailTemplatePage();
1618

19+
expect(await generateMetadata()).toEqual({ title: pageTitle });
1720
expect(page).toMatchSnapshot();
1821
});
1922
});

frontend/src/__tests__/app/create-nhs-app-template/page.test.tsx

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,16 +4,19 @@
44
import CreateNHSAppTemplatePage, {
55
generateMetadata,
66
} from '@app/create-nhs-app-template/page';
7+
import content from '@content/content';
8+
9+
const { pageTitle } = content.components.templateFormNhsApp;
710

811
jest.mock('@forms/NhsAppTemplateForm/NhsAppTemplateForm');
912

1013
describe('CreateNHSAppTemplatePage', () => {
1114
beforeEach(jest.resetAllMocks);
1215

1316
it('should render CreateNHSAppTemplatePage', async () => {
14-
generateMetadata();
1517
const page = await CreateNHSAppTemplatePage();
1618

19+
expect(await generateMetadata()).toEqual({ title: pageTitle });
1720
expect(page).toMatchSnapshot();
1821
});
1922
});

frontend/src/__tests__/app/create-text-message-template/page.test.tsx

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,16 +4,19 @@
44
import CreateSMSTemplatePage, {
55
generateMetadata,
66
} from '@app/create-text-message-template/page';
7+
import content from '@content/content';
8+
9+
const { pageTitle } = content.components.templateFormSms;
710

811
jest.mock('@forms/SmsTemplateForm/SmsTemplateForm');
912

1013
describe('CreateSMSTemplatePage', () => {
1114
beforeEach(jest.resetAllMocks);
1215

1316
it('should render CreateSMSTemplatePage', async () => {
14-
generateMetadata();
1517
const page = await CreateSMSTemplatePage();
1618

19+
expect(await generateMetadata()).toEqual({ title: pageTitle });
1720
expect(page).toMatchSnapshot();
1821
});
1922
});

frontend/src/__tests__/app/edit-email-template/page.test.tsx

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,9 @@ import { getTemplate } from '@utils/form-actions';
88
import { redirect } from 'next/navigation';
99
import { EmailTemplateForm } from '@forms/EmailTemplateForm/EmailTemplateForm';
1010
import { EmailTemplate } from 'nhs-notify-web-template-management-utils';
11+
import content from '@content/content';
12+
13+
const { editPageTitle } = content.components.templateFormEmail;
1114

1215
jest.mock('@utils/form-actions');
1316
jest.mock('next/navigation');
@@ -62,7 +65,6 @@ describe('EditEmailTemplatePage', () => {
6265
});
6366

6467
it('should render CreateEmailTemplatePage component when template is found', async () => {
65-
generateMetadata();
6668
getTemplateMock.mockResolvedValueOnce(template);
6769

6870
const emailTemplate: EmailTemplate = {
@@ -78,6 +80,7 @@ describe('EditEmailTemplatePage', () => {
7880
}),
7981
});
8082

83+
expect(await generateMetadata()).toEqual({ title: editPageTitle });
8184
expect(getTemplateMock).toHaveBeenCalledWith('template-id');
8285

8386
expect(page).toEqual(<EmailTemplateForm initialState={emailTemplate} />);

frontend/src/__tests__/app/edit-nhs-app-template/page.test.tsx

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,9 @@ import EditNhsAppTemplatePage, {
1010
} from '@app/edit-nhs-app-template/[templateId]/page';
1111
import { TemplateDto } from 'nhs-notify-backend-client';
1212
import { EMAIL_TEMPLATE, LETTER_TEMPLATE, SMS_TEMPLATE } from '../../helpers';
13+
import content from '@content/content';
14+
15+
const { editPageTitle } = content.components.templateFormNhsApp;
1316

1417
jest.mock('@forms/NhsAppTemplateForm/NhsAppTemplateForm');
1518
jest.mock('@utils/form-actions');
@@ -22,7 +25,6 @@ describe('EditNhsAppTemplatePage', () => {
2225
beforeEach(jest.resetAllMocks);
2326

2427
test('page loads', async () => {
25-
generateMetadata();
2628
const template = {
2729
id: 'template-id',
2830
templateType: 'NHS_APP',
@@ -47,6 +49,7 @@ describe('EditNhsAppTemplatePage', () => {
4749
}),
4850
});
4951

52+
expect(await generateMetadata()).toEqual({ title: editPageTitle });
5053
expect(page).toEqual(<NhsAppTemplateForm initialState={nhsAppTemplate} />);
5154
});
5255

frontend/src/__tests__/app/edit-text-message-template/page.test.tsx

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,9 @@ import { redirect } from 'next/navigation';
99
import { SmsTemplateForm } from '@forms/SmsTemplateForm/SmsTemplateForm';
1010
import { TemplateDto } from 'nhs-notify-backend-client';
1111
import { SMSTemplate } from 'nhs-notify-web-template-management-utils';
12+
import content from '@content/content';
13+
14+
const { editPageTitle } = content.components.templateFormSms;
1215

1316
jest.mock('@utils/form-actions');
1417
jest.mock('next/navigation');
@@ -62,7 +65,6 @@ describe('EditSmsTemplatePage', () => {
6265
});
6366

6467
it('should render CreateSmsTemplate component when templateId is found', async () => {
65-
generateMetadata();
6668
getTemplateMock.mockResolvedValueOnce(templateDTO);
6769

6870
const smsTemplate: SMSTemplate = {
@@ -77,6 +79,7 @@ describe('EditSmsTemplatePage', () => {
7779
}),
7880
});
7981

82+
expect(await generateMetadata()).toEqual({ title: editPageTitle });
8083
expect(getTemplateMock).toHaveBeenCalledWith('template-id');
8184

8285
expect(page).toEqual(<SmsTemplateForm initialState={smsTemplate} />);

frontend/src/__tests__/app/email-template-submitted/page.test.tsx

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,9 @@ import { TemplateSubmitted } from '@molecules/TemplateSubmitted/TemplateSubmitte
88
import { getTemplate } from '@utils/form-actions';
99
import { redirect } from 'next/navigation';
1010
import { TemplateDto } from 'nhs-notify-backend-client';
11+
import content from '@content/content';
12+
13+
const { pageTitle } = content.components.templateSubmitted;
1114

1215
jest.mock('@molecules/TemplateSubmitted/TemplateSubmitted');
1316
jest.mock('@utils/form-actions');
@@ -50,7 +53,6 @@ describe('EmailTemplateSubmittedPage', () => {
5053
});
5154

5255
test('should handle invalid template', async () => {
53-
generateMetadata();
5456
getTemplateMock.mockResolvedValueOnce(undefined);
5557

5658
await EmailTemplateSubmittedPage({
@@ -59,6 +61,7 @@ describe('EmailTemplateSubmittedPage', () => {
5961
}),
6062
});
6163

64+
expect(await generateMetadata()).toEqual({ title: pageTitle.EMAIL });
6265
expect(getTemplateMock).toHaveBeenCalledWith('invalid-template');
6366

6467
expect(redirectMock).toHaveBeenCalledWith('/invalid-template', 'replace');

frontend/src/__tests__/app/manage-templates/page.test.tsx

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,9 +28,11 @@ describe('ManageTemplatesPage', () => {
2828
jest.resetAllMocks();
2929
});
3030
test('renders the page without templates', async () => {
31-
generateMetadata();
3231
render(await ManageTemplatesPage());
3332

33+
expect(await generateMetadata()).toEqual({
34+
title: manageTemplatesContent.pageTitle,
35+
});
3436
expect(screen.getByTestId('page-content-wrapper')).toBeInTheDocument();
3537
expect(screen.getByTestId('page-heading')).toBeInTheDocument();
3638
expect(screen.getByRole('button')).toBeInTheDocument();

frontend/src/__tests__/app/nhs-app-template-submitted/page.test.tsx

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,9 @@ import { TemplateSubmitted } from '@molecules/TemplateSubmitted/TemplateSubmitte
88
import { getTemplate } from '@utils/form-actions';
99
import { redirect } from 'next/navigation';
1010
import { TemplateDto } from 'nhs-notify-backend-client';
11+
import content from '@content/content';
12+
13+
const { pageTitle } = content.components.templateSubmitted;
1114

1215
jest.mock('@molecules/TemplateSubmitted/TemplateSubmitted');
1316
jest.mock('@utils/form-actions');
@@ -20,7 +23,6 @@ describe('NhsAppTemplateSubmittedPage', () => {
2023
beforeEach(jest.resetAllMocks);
2124

2225
test('should load page', async () => {
23-
generateMetadata();
2426
const template = {
2527
id: 'template-id',
2628
templateType: 'NHS_APP',
@@ -41,6 +43,9 @@ describe('NhsAppTemplateSubmittedPage', () => {
4143
}),
4244
});
4345

46+
expect(await generateMetadata()).toEqual({
47+
title: pageTitle.NHS_APP,
48+
});
4449
expect(getTemplateMock).toHaveBeenCalledWith('template-id');
4550

4651
expect(page).toEqual(

0 commit comments

Comments
 (0)