Skip to content

Commit 9ef13cb

Browse files
committed
CCM-11544 Test failures
1 parent 03b4bce commit 9ef13cb

File tree

2 files changed

+27
-26
lines changed

2 files changed

+27
-26
lines changed

tests/test-team/helpers/template-mgmt-common.steps.ts

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -205,15 +205,12 @@ export function assertBackLinkTop(
205205
props: CommonStepsProps & { expectedUrl: string }
206206
) {
207207
return test.step('displays back link at the top of the page with correct url', async () => {
208-
const { page, baseURL, expectedUrl } = props;
208+
const { page, expectedUrl } = props;
209209

210210
await page.loadPage();
211211

212212
await expect(page.backLinkTop).toBeVisible();
213-
await expect(page.backLinkTop).toHaveAttribute(
214-
'href',
215-
`${baseURL}/${expectedUrl}`
216-
);
213+
await expect(page.backLinkTop).toHaveAttribute('href', `/${expectedUrl}`);
217214
});
218215
}
219216

tests/test-team/template-mgmt-routing-component-tests/letter/choose-other-language-letter-template.routing-component.spec.ts

Lines changed: 25 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -310,35 +310,39 @@ test.describe('Routing - Choose other language letter templates page', () => {
310310
);
311311
});
312312

313-
test('user sees a message when no foreign language templates are available and can go to create templates', async ({
314-
page,
315-
baseURL,
316-
}) => {
313+
test.describe('user with no templates', () => {
317314
test.use({ storageState: { cookies: [], origins: [] } });
318315

319-
await loginAsUser(userWithNoTemplates, page);
316+
test('user sees a message when no foreign language templates are available and can go to create templates', async ({
317+
page,
318+
baseURL,
319+
}) => {
320+
await loginAsUser(userWithNoTemplates, page);
320321

321-
const chooseOtherLanguageLetterTemplatePage =
322-
new RoutingChooseOtherLanguageLetterTemplatePage(page);
323-
await chooseOtherLanguageLetterTemplatePage
324-
.setPathParam('messagePlanId', routingConfigForUserWithNoTemplates.id)
325-
.loadPage();
322+
const chooseOtherLanguageLetterTemplatePage =
323+
new RoutingChooseOtherLanguageLetterTemplatePage(page);
324+
await chooseOtherLanguageLetterTemplatePage
325+
.setPathParam('messagePlanId', routingConfigForUserWithNoTemplates.id)
326+
.loadPage();
326327

327-
await assertChooseTemplatePageWithNoTemplates({
328-
page: chooseOtherLanguageLetterTemplatePage,
329-
});
328+
await assertChooseTemplatePageWithNoTemplates({
329+
page: chooseOtherLanguageLetterTemplatePage,
330+
});
330331

331-
await expect(
332-
chooseOtherLanguageLetterTemplatePage.messagePlanName
333-
).toHaveText(routingConfigForUserWithNoTemplates.name);
332+
await expect(
333+
chooseOtherLanguageLetterTemplatePage.messagePlanName
334+
).toHaveText(routingConfigForUserWithNoTemplates.name);
334335

335-
await expect(
336-
chooseOtherLanguageLetterTemplatePage.noTemplatesMessage
337-
).toHaveText('You do not have any foreign language letter templates yet.');
336+
await expect(
337+
chooseOtherLanguageLetterTemplatePage.noTemplatesMessage
338+
).toHaveText(
339+
'You do not have any foreign language letter templates yet.'
340+
);
338341

339-
await chooseOtherLanguageLetterTemplatePage.goToTemplatesLink.click();
342+
await chooseOtherLanguageLetterTemplatePage.goToTemplatesLink.click();
340343

341-
await expect(page).toHaveURL(`${baseURL}/templates/message-templates`);
344+
await expect(page).toHaveURL(`${baseURL}/templates/message-templates`);
345+
});
342346
});
343347

344348
test('user can choose multiple templates for a message plan that has no language templates', async ({

0 commit comments

Comments
 (0)