From 87752e24441bc70669e5666c628261194782c483 Mon Sep 17 00:00:00 2001 From: Max Schmitt Date: Tue, 11 Jun 2024 21:22:05 +0200 Subject: [PATCH] fixes --- tests/electron/playwright.config.ts | 2 ++ tests/page/page-set-input-files.spec.ts | 6 +++--- tests/webview2/playwright.config.ts | 1 + 3 files changed, 6 insertions(+), 3 deletions(-) diff --git a/tests/electron/playwright.config.ts b/tests/electron/playwright.config.ts index b630b38e565388..515e830de8896e 100644 --- a/tests/electron/playwright.config.ts +++ b/tests/electron/playwright.config.ts @@ -54,6 +54,7 @@ config.projects.push({ name: 'electron-api', use: { browserName: 'chromium', + headless: false, }, testDir: path.join(testDir, 'electron'), metadata, @@ -65,6 +66,7 @@ config.projects.push({ snapshotPathTemplate: '{testDir}/{testFileDir}/{testFileName}-snapshots/{arg}-chromium{ext}', use: { browserName: 'chromium', + headless: false, }, testDir: path.join(testDir, 'page'), metadata, diff --git a/tests/page/page-set-input-files.spec.ts b/tests/page/page-set-input-files.spec.ts index b2a22b57e3f2b1..fb2db478c9c22a 100644 --- a/tests/page/page-set-input-files.spec.ts +++ b/tests/page/page-set-input-files.spec.ts @@ -16,7 +16,7 @@ */ import { test as it, expect } from './pageTest'; -import { attachFrame } from '../config/utils'; +import { attachFrame, chromiumVersionLessThan } from '../config/utils'; import path from 'path'; import fs from 'fs'; @@ -37,7 +37,7 @@ it('should upload the file', async ({ page, server, asset }) => { }, input)).toBe('contents of the file'); }); -it('should upload a folder', async ({ page, server, browserName, headless, browserMajorVersion }) => { +it('should upload a folder', async ({ page, server, browserName, headless, browserVersion }) => { await page.goto(server.PREFIX + '/input/folderupload.html'); const input = await page.$('input'); const dir = path.join(it.info().outputDir, 'file-upload-test'); @@ -51,7 +51,7 @@ it('should upload a folder', async ({ page, server, browserName, headless, brows await input.setInputFiles(dir); expect(new Set(await page.evaluate(e => [...e.files].map(f => f.webkitRelativePath), input))).toEqual(new Set([ // https://issues.chromium.org/issues/345393164 - ...((browserName === 'chromium' && headless && browserMajorVersion < 128) ? [] : ['file-upload-test/sub-dir/really.txt']), + ...((browserName === 'chromium' && headless && chromiumVersionLessThan(browserVersion, '127.0.6533.0')) ? [] : ['file-upload-test/sub-dir/really.txt']), 'file-upload-test/file1.txt', 'file-upload-test/file2', ])); diff --git a/tests/webview2/playwright.config.ts b/tests/webview2/playwright.config.ts index 0710378a5036b2..6a7ec6ed1bdb5a 100644 --- a/tests/webview2/playwright.config.ts +++ b/tests/webview2/playwright.config.ts @@ -56,6 +56,7 @@ config.projects.push({ snapshotPathTemplate: '{testDir}/{testFileDir}/{testFileName}-snapshots/{arg}-chromium{ext}', use: { browserName: 'chromium', + headless: false, }, testDir: path.join(testDir, 'page'), metadata,