From 6e73bf189e921e61289005228b697c631fd577f4 Mon Sep 17 00:00:00 2001 From: Nick Grosenbacher Date: Mon, 21 Oct 2024 13:59:35 -0400 Subject: [PATCH] SWC-7102 - Fix chromium e2e tests (locally) --- e2e/helpers/testUser.ts | 2 +- e2e/homepage.spec.ts | 16 +++++++--------- 2 files changed, 8 insertions(+), 10 deletions(-) diff --git a/e2e/helpers/testUser.ts b/e2e/helpers/testUser.ts index 6063521975..8fc228b2c4 100644 --- a/e2e/helpers/testUser.ts +++ b/e2e/helpers/testUser.ts @@ -99,7 +99,7 @@ export async function loginTestUser( await acceptSiteCookies(page) // Login - await page.getByRole('link', { name: 'Log in to Synapse' }).first().click() + await page.getByRole('button', { name: 'Login' }).first().click() await page.getByRole('button', { name: 'Sign in with your email' }).click() const usernameInput = page.getByLabel('Username or Email Address') diff --git a/e2e/homepage.spec.ts b/e2e/homepage.spec.ts index f5f347669a..e44c04e169 100644 --- a/e2e/homepage.spec.ts +++ b/e2e/homepage.spec.ts @@ -9,27 +9,25 @@ test.describe('Homepage - Unauthenticated', () => { await page.goto('/') await waitForInitialPageLoad(page) + await expect(page.getByRole('button', { name: 'Login' })).toHaveCount(1) await expect( - page.getByRole('link', { name: 'Log in to Synapse' }), - ).toHaveCount(1) - await expect( - page.getByRole('link', { name: 'View Your Dashboard' }), + page.getByRole('button', { name: 'View Dashboard' }), ).toHaveCount(0) }) }) testAuth.describe('Homepage - Authenticated', () => { testAuth( - 'should show View Your Dashboard button when logged in', + 'should show View Dashboard button when logged in', async ({ userPage }) => { await userPage.goto('/Home:x') await waitForInitialPageLoad(userPage) + await expect(userPage.getByRole('button', { name: 'Login' })).toHaveCount( + 0, + ) await expect( - userPage.getByRole('link', { name: 'Log in to Synapse' }), - ).toHaveCount(0) - await expect( - userPage.getByRole('link', { name: 'View Your Dashboard' }), + userPage.getByRole('button', { name: 'View Dashboard' }), ).toHaveCount(1) }, )