test: fix flaky unselect test #33060
Conversation
| test('should select with multiple attribute', async ({ openRecorder }) => { | ||
| const { page, recorder } = await openRecorder(); | ||
|
|
||
| await recorder.setContentAndWait(`<select id="age" multiple onchange="console.log('[' + [...age.selectedOptions].map(x => x.value).join(',') + ']')"><option value="1">1</option><option value="2">2</option></select>`); | ||
|
|
||
| const locator = await recorder.hoverOverElement('select'); | ||
| expect(locator).toBe(`locator('#age')`); | ||
| await clickMultipleSelectOption(page.getByRole('option', { name: '1' })); | ||
| await page.locator('select').focus(); | ||
| await page.selectOption('select', ['1']); |
There was a problem hiding this comment.
Note: if you don't focus, it throws here that activeModel is null.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
| @@ -464,3 +464,41 @@ it('should click in custom element', async ({ page }) => { | |||
| await page.locator('input').click(); | |||
| expect(await page.evaluate('window.__clicked')).toBe(true); | |||
| }); | |||
|
|
|||
| for (const mode of ['same-context', 'different-context'] as const) { | |||
| it(`concurrent selectOption in ${mode} should work`, async ({ browser, browserName, isLinux }) => { | |||
Test results for "tests others"5 flaky20557 passed, 495 skipped Merge workflow run. |
Test results for "tests 1"2 failed 5 flaky35878 passed, 624 skipped Merge workflow run. |
Test results for "tests 2"49 failed 81 flaky238945 passed, 9481 skipped Merge workflow run. |
This is flaky since it was added in #32127.
Follow-up to #32127 (comment).
Added an isolated test for the WebKit/Linux bug in #33064.