From e8523abea5ba64db71837acaad099604e502a88c Mon Sep 17 00:00:00 2001 From: Dmitry Gozman Date: Mon, 9 Oct 2023 14:07:22 -0700 Subject: [PATCH] test: mark some `--headless=new` tests as fixme (#27516) --- tests/library/emulation-focus.spec.ts | 2 ++ tests/library/video.spec.ts | 3 ++- 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/tests/library/emulation-focus.spec.ts b/tests/library/emulation-focus.spec.ts index caf06ac5dc..bc4bbb801c 100644 --- a/tests/library/emulation-focus.spec.ts +++ b/tests/library/emulation-focus.spec.ts @@ -104,6 +104,8 @@ it('should change document.activeElement', async ({ page, server }) => { it('should not affect screenshots', async ({ page, server, browserName, headless, isWindows }) => { it.skip(browserName === 'webkit' && isWindows && !headless, 'WebKit/Windows/headed has a larger minimal viewport. See https://github.com/microsoft/playwright/issues/22616'); it.skip(browserName === 'firefox' && !headless, 'Firefox headed produces a different image'); + const isChromiumHeadlessNew = browserName === 'chromium' && !!headless && !!process.env.PLAYWRIGHT_CHROMIUM_USE_HEADLESS_NEW; + it.fixme(isChromiumHeadlessNew, 'Times out with --headless=new'); const page2 = await page.context().newPage(); await Promise.all([ diff --git a/tests/library/video.spec.ts b/tests/library/video.spec.ts index 931345ff41..c0cf50fb3d 100644 --- a/tests/library/video.spec.ts +++ b/tests/library/video.spec.ts @@ -444,7 +444,8 @@ it.describe('screencast', () => { }); it('should scale frames down to the requested size ', async ({ browser, browserName, server, headless, trace }, testInfo) => { - it.fixme(!headless, 'Fails on headed'); + const isChromiumHeadlessNew = browserName === 'chromium' && !!headless && !!process.env.PLAYWRIGHT_CHROMIUM_USE_HEADLESS_NEW; + it.fixme(!headless || isChromiumHeadlessNew, 'Fails on headed'); const context = await browser.newContext({ recordVideo: {