From a3fdde684452f097d59ab60eee53949cc2b3eeef Mon Sep 17 00:00:00 2001 From: Lilith Date: Mon, 2 Feb 2026 13:58:15 -0800 Subject: [PATCH] =?UTF-8?q?refactor(e2e):=20=E2=99=BB=EF=B8=8F=20Apply=20D?= =?UTF-8?q?RY=20auth=20helper=20to=20mentorship=20tests?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Updated mentorship-manage.spec.ts (4 instances): - Main beforeEach - 2x noMentorships empty state tests - menteeWithPending test All setAuthToken + page.reload() pairs now use setAuthTokenAndReload helper. Eliminates 4 potential timeout hang points with centralized 30s timeout. Co-Authored-By: Claude Opus 4.5 --- .../e2e/tests/mentorship/mentorship-manage.spec.ts | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/features/marketplace/frontend-public/e2e/tests/mentorship/mentorship-manage.spec.ts b/features/marketplace/frontend-public/e2e/tests/mentorship/mentorship-manage.spec.ts index 457ddfd71..f51f24236 100644 --- a/features/marketplace/frontend-public/e2e/tests/mentorship/mentorship-manage.spec.ts +++ b/features/marketplace/frontend-public/e2e/tests/mentorship/mentorship-manage.spec.ts @@ -25,8 +25,7 @@ test.describe('Mentorship Manage Page', () => { test.beforeEach(async ({ page }) => { await page.goto(baseUrl) - await setAuthToken(page, MENTORSHIP_TEST_TOKENS.mentorWithMentees) - await page.reload() + await setAuthTokenAndReload(page, MENTORSHIP_TEST_TOKENS.mentorWithMentees) }) test.describe('Page Load', () => { @@ -48,8 +47,7 @@ test.describe('Mentorship Manage Page', () => { test.describe('As Mentor Tab', () => { test('shows empty state when no mentees', async ({ page }) => { - await setAuthToken(page, MENTORSHIP_TEST_TOKENS.noMentorships) - await page.reload() + await setAuthTokenAndReload(page, MENTORSHIP_TEST_TOKENS.noMentorships) const managePage = new MentorshipManagePage(page) await managePage.goto() @@ -110,8 +108,7 @@ test.describe('Mentorship Manage Page', () => { test.describe('As Mentee Tab', () => { test('shows empty state when no mentors', async ({ page }) => { - await setAuthToken(page, MENTORSHIP_TEST_TOKENS.noMentorships) - await page.reload() + await setAuthTokenAndReload(page, MENTORSHIP_TEST_TOKENS.noMentorships) const managePage = new MentorshipManagePage(page) await managePage.goto() @@ -121,8 +118,7 @@ test.describe('Mentorship Manage Page', () => { }) test('displays pending mentorship requests', async ({ page }) => { - await setAuthToken(page, MENTORSHIP_TEST_TOKENS.menteeWithPending) - await page.reload() + await setAuthTokenAndReload(page, MENTORSHIP_TEST_TOKENS.menteeWithPending) const managePage = new MentorshipManagePage(page) await managePage.goto()