Skip to content

Commit 384043c

Browse files
Merge commit 'ce7d57d6eef435910da122c78246d49d7ddd87e5' into feature/CCM-12111_display-updated-at
2 parents 5a58cd7 + ce7d57d commit 384043c

File tree

26 files changed

+825
-492
lines changed

26 files changed

+825
-492
lines changed

frontend/src/__tests__/app/upload-letter-template/__snapshots__/page.test.tsx.snap

Lines changed: 0 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -1,24 +1,5 @@
11
// Jest Snapshot v1, https://goo.gl/fbAQLP
22

3-
exports[`UploadLetterTemplatePage should render UploadLetterTemplatePage with campaignId field when campaignIds is not available 1`] = `
4-
<LetterTemplateForm
5-
campaignIds={
6-
[
7-
"campaign-id",
8-
]
9-
}
10-
initialState={
11-
{
12-
"campaignId": "campaign-id",
13-
"language": "en",
14-
"letterType": "x0",
15-
"name": "",
16-
"templateType": "LETTER",
17-
}
18-
}
19-
/>
20-
`;
21-
223
exports[`UploadLetterTemplatePage should render UploadLetterTemplatePage with campaignIds field when available 1`] = `
234
<LetterTemplateForm
245
campaignIds={

frontend/src/__tests__/app/upload-letter-template/page.test.tsx

Lines changed: 3 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ describe('UploadLetterTemplatePage', () => {
3030
});
3131
mockFetchClient.mockResolvedValueOnce({
3232
data: {
33-
campaignId: 'campaign2',
33+
campaignIds: ['campaign2'],
3434
features: {},
3535
},
3636
});
@@ -49,26 +49,6 @@ describe('UploadLetterTemplatePage', () => {
4949
mockFetchClient.mockResolvedValueOnce({
5050
data: {
5151
campaignIds: ['campaign-id', 'other-campaign-id'],
52-
campaignId: 'campaign-id',
53-
features: {},
54-
},
55-
});
56-
57-
const page = await UploadLetterTemplatePage();
58-
59-
expect(await generateMetadata()).toEqual({ title: pageTitle });
60-
expect(page).toMatchSnapshot();
61-
});
62-
63-
it('should render UploadLetterTemplatePage with campaignId field when campaignIds is not available', async () => {
64-
mockGetSessionServer.mockResolvedValueOnce({
65-
accessToken: 'mocktoken',
66-
clientId: 'client1',
67-
});
68-
mockFetchClient.mockResolvedValueOnce({
69-
data: {
70-
campaignIds: undefined,
71-
campaignId: 'campaign-id',
7252
features: {},
7353
},
7454
});
@@ -107,7 +87,7 @@ describe('UploadLetterTemplatePage', () => {
10787
});
10888
mockFetchClient.mockResolvedValueOnce({
10989
data: {
110-
campaignId: 'campaign2',
90+
campaignIds: ['campaign2'],
11191
features: {},
11292
},
11393
});
@@ -130,7 +110,6 @@ describe('UploadLetterTemplatePage', () => {
130110
mockFetchClient.mockResolvedValueOnce({
131111
data: {
132112
campaignIds: [],
133-
campaignId: 'campaign-id',
134113
features: {},
135114
},
136115
});
@@ -143,7 +122,7 @@ describe('UploadLetterTemplatePage', () => {
143122
);
144123
});
145124

146-
it('should redirect to error page when neither campaignIds nor campaignId is present', async () => {
125+
it('should redirect to error page when neither campaignIds are not present', async () => {
147126
const mockRedirect = jest.mocked(redirect);
148127

149128
mockGetSessionServer.mockResolvedValueOnce({
@@ -153,7 +132,6 @@ describe('UploadLetterTemplatePage', () => {
153132
mockFetchClient.mockResolvedValueOnce({
154133
data: {
155134
campaignIds: undefined,
156-
campaignId: undefined,
157135
features: {},
158136
},
159137
});

0 commit comments

Comments
 (0)