Skip to content

Commit 238ef82

Browse files
mchestrclaude
andcommitted
fix: update E2E tests for multi-service UI changes
Updated public-flows.spec.ts tests to use correct testids after multi-service authentication refactor: - 'home page has sign in button and initiates flow': Look for 'sign-in-with-plex' instead of 'toggle-plex' since only Plex is configured in E2E environment (toggle only appears with both Plex and Jellyfin) - 'setup page redirects if already set up': Look for collapsed 'sign-in-button' instead of nested 'sign-in-with-plex' for simpler, more reliable test Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
1 parent 10a2f6f commit 238ef82

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

e2e/public-flows.spec.ts

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,9 @@ test.describe('Public Flows', () => {
1414
// Click the button to expand and reveal sign-in options
1515
await signInButton.click();
1616

17-
// Verify Plex toggle is now visible
18-
const plexToggle = page.getByTestId('toggle-plex');
19-
await expect(plexToggle).toBeVisible({ timeout: WAIT_TIMEOUTS.PAGE_CONTENT });
17+
// Verify Plex sign-in button is now visible (only Plex configured in E2E environment)
18+
const plexSignInButton = page.getByTestId('sign-in-with-plex');
19+
await expect(plexSignInButton).toBeVisible({ timeout: WAIT_TIMEOUTS.PAGE_CONTENT });
2020
});
2121

2222
test('invite page shows invalid state for unknown code', async ({ page }) => {
@@ -37,7 +37,8 @@ test.describe('Public Flows', () => {
3737
// Wait for loading screen to disappear
3838
await waitForLoadingGone(page);
3939

40-
await expect(page.getByTestId('sign-in-with-plex')).toBeVisible();
40+
// Verify we're redirected to home page with sign-in button
41+
await expect(page.getByTestId('sign-in-button')).toBeVisible();
4142
});
4243

4344
test('denied page is accessible', async ({ page }) => {

0 commit comments

Comments
 (0)