Skip to content

Commit b4f8884

Browse files
committed
CCM-14583: update destination when approving letter authoring
1 parent 7536664 commit b4f8884

File tree

4 files changed

+22
-6
lines changed

4 files changed

+22
-6
lines changed

frontend/src/__tests__/app/preview-letter-template/server-action.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,15 +13,15 @@ describe('submitAuthoringLetterAction', () => {
1313
jest.resetAllMocks();
1414
});
1515

16-
it('should redirect to submit-letter-template page with valid form data', async () => {
16+
it('should redirect to get-ready-to-approve-letter-template page with valid form data', async () => {
1717
const formData = new FormData();
1818
formData.append('templateId', 'template-123');
1919
formData.append('lockNumber', '1');
2020

2121
await submitAuthoringLetterAction({}, formData);
2222

2323
expect(redirectMock).toHaveBeenCalledWith(
24-
'/submit-letter-template/template-123?lockNumber=1'
24+
'/get-ready-to-approve-letter-template/template-123'
2525
);
2626
});
2727

frontend/src/app/preview-letter-template/[templateId]/server-action.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,7 @@ export async function submitAuthoringLetterAction(
2222
};
2323
}
2424

25-
const { templateId, lockNumber } = result.data;
25+
const { templateId } = result.data;
2626

27-
// destination TBD
28-
redirect(`/submit-letter-template/${templateId}?lockNumber=${lockNumber}`);
27+
redirect(`/get-ready-to-approve-letter-template/${templateId}`);
2928
}

tests/test-team/.context/test-context.json

Lines changed: 0 additions & 1 deletion
This file was deleted.

tests/test-team/template-mgmt-component-tests/letter/template-mgmt-preview-letter-page.component.spec.ts

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -666,6 +666,24 @@ test.describe('Preview Letter template Page', () => {
666666
await expect(previewPage.campaignAction).toBeHidden();
667667
});
668668

669+
test('when user approves a template, then they are taken to "get ready to approve template" page', async ({
670+
page,
671+
baseURL,
672+
}) => {
673+
const previewPage = new TemplateMgmtPreviewLetterPage(page).setPathParam(
674+
'templateId',
675+
templates.authoringValid.id
676+
);
677+
678+
await previewPage.loadPage();
679+
680+
await previewPage.clickContinueButton();
681+
682+
await expect(page).toHaveURL(
683+
`${baseURL}/templates/get-ready-to-approve-letter-template/${templates.authoringValid.id}`
684+
);
685+
});
686+
669687
test.describe('Letter render section', () => {
670688
test('displays letter render section with tabs when initialRender exists', async ({
671689
page,

0 commit comments

Comments
 (0)