Fix E2E tests broken by redesigned navigation menu (#18315)
## Summary
- **Settings selector**: The Settings navigation item is now rendered as
a `<button>` (via `NavigationDrawerItem` with `onClick`) instead of an
`<a>` link (with `to`). Updated `leftMenu.ts` POM and
`create-kanban-view.spec.ts` to use `getByRole('button', { name:
'Settings' })`.
- **create-record URL field**: The Linkedin field interaction was
missing an initial label click to trigger the hover portal rendering.
Added `recordFieldList.getByText('Linkedin').first().click()` before the
value click, matching the pattern used by the working Emails field.
## Test plan
- [ ] E2E `signup_invite_email.spec.ts` passes (uses
`leftMenu.goToSettings()`)
- [ ] E2E `create-kanban-view.spec.ts` passes (uses Settings click
directly)
- [ ] E2E `create-record.spec.ts` passes (Linkedin URL field
interaction)
- [ ] Existing passing E2E tests remain green
Made with [Cursor](https://cursor.com)
This commit is contained in:
@@ -80,6 +80,7 @@ test('Create and update record', async ({ page }) => {
|
||||
await page.getByPlaceholder('Intro').press('Enter');
|
||||
|
||||
// Fill URL
|
||||
await recordFieldList.getByText('Linkedin').first().click();
|
||||
const urlInput = recordFieldList.getByText('Linkedin').nth(1);
|
||||
await expect(urlInput).toBeVisible();
|
||||
await urlInput.click({ force: true });
|
||||
@@ -87,11 +88,12 @@ test('Create and update record', async ({ page }) => {
|
||||
await page.getByPlaceholder('URL').press('Enter');
|
||||
|
||||
// Click on 4th star to rate
|
||||
recordFieldList.getByText('Performance Rating').first().click({ force: true });
|
||||
await recordFieldList.getByText('Performance Rating').first().click({ force: true });
|
||||
const ratingContainer = recordFieldList.locator('div[aria-label="Rating"]');
|
||||
await ratingContainer.locator('svg').nth(3).click({force: true});
|
||||
|
||||
// Fill phone field
|
||||
await recordFieldList.getByText('Phones').first().click();
|
||||
const phoneInput = recordFieldList.getByText('Phones').nth(1);
|
||||
await expect(phoneInput).toBeVisible();
|
||||
await phoneInput.click({ force: true });
|
||||
|
||||
Reference in New Issue
Block a user