From f3852d4511ceacade89d8cf376faae53555edd9b Mon Sep 17 00:00:00 2001 From: Andrey Lushnikov Date: Wed, 26 Apr 2023 20:11:02 +0000 Subject: [PATCH] test: tree herding (#22660) Disable tests according to the filed bugs. https://github.com/microsoft/playwright/issues/22616 https://github.com/microsoft/playwright/issues/22617 https://github.com/microsoft/playwright/issues/22618 --- tests/library/emulation-focus.spec.ts | 3 ++- tests/library/video.spec.ts | 9 ++++++--- 2 files changed, 8 insertions(+), 4 deletions(-) diff --git a/tests/library/emulation-focus.spec.ts b/tests/library/emulation-focus.spec.ts index d81506b5b9..564c4d565c 100644 --- a/tests/library/emulation-focus.spec.ts +++ b/tests/library/emulation-focus.spec.ts @@ -101,7 +101,8 @@ it('should change document.activeElement', async ({ page, server }) => { expect(active).toEqual(['INPUT', 'TEXTAREA']); }); -it('should not affect screenshots', async ({ page, server, browserName, headless }) => { +it('should not affect screenshots', async ({ page, server, browserName, headless, isWindows }) => { + it.fixme(browserName === 'webkit' && isWindows && !headless, 'https://github.com/microsoft/playwright/issues/22616'); it.skip(browserName === 'firefox' && !headless, 'Firefox headede produces a different image'); const page2 = await page.context().newPage(); diff --git a/tests/library/video.spec.ts b/tests/library/video.spec.ts index 84fb8db623..9d3b6be14a 100644 --- a/tests/library/video.spec.ts +++ b/tests/library/video.spec.ts @@ -162,7 +162,8 @@ it.describe('screencast', () => { expect(error.message).toContain('"videoSize" option requires "videosPath" to be specified'); }); - it('should work with old options', async ({ browser, browserName, trace }, testInfo) => { + it('should work with old options', async ({ browser, browserName, trace, headless, isWindows }, testInfo) => { + it.fixme(browserName === 'firefox' && !headless && isWindows, 'https://github.com/microsoft/playwright/issues/22618'); const videosPath = testInfo.outputPath(''); // Firefox does not have a mobile variant and has a large minimum size (500 on windows and 450 elsewhere). const size = browserName === 'firefox' ? { width: 500, height: 400 } : { width: 320, height: 240 }; @@ -186,7 +187,8 @@ it.describe('screencast', () => { expect(error.message).toContain('recordVideo.dir: expected string, got undefined'); }); - it('should capture static page', async ({ browser, browserName, trace }, testInfo) => { + it('should capture static page', async ({ browser, browserName, trace, headless, isWindows }, testInfo) => { + it.fixme(browserName === 'firefox' && !headless && isWindows, 'https://github.com/microsoft/playwright/issues/22618'); // Firefox does not have a mobile variant and has a large minimum size (500 on windows and 450 elsewhere). const size = browserName === 'firefox' ? { width: 500, height: 400 } : { width: 320, height: 240 }; const context = await browser.newContext({ @@ -709,10 +711,11 @@ it.describe('screencast', () => { expectAll(pixels, almostRed); }); - it('should capture full viewport on hidpi', async ({ browserType, browserName, headless, isWindows }, testInfo) => { + it('should capture full viewport on hidpi', async ({ browserType, browserName, headless, isWindows, isLinux }, testInfo) => { it.info().annotations.push({ type: 'issue', description: 'https://github.com/microsoft/playwright/issues/22411' }); it.fixme(browserName === 'chromium' && !headless, 'The square is not on the video'); it.fixme(browserName === 'firefox' && isWindows, 'https://github.com/microsoft/playwright/issues/14405'); + it.fixme(browserName === 'webkit' && isLinux, 'https://github.com/microsoft/playwright/issues/22617'); const size = { width: 600, height: 400 }; const browser = await browserType.launch();