test: update some test conditions (#31336)

microsoft · Jun 17, 2024 · a002572 · a002572
1 parent 2a7f17d
commit a002572
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 7 deletions.
diff --git a/tests/page/page-drag.spec.ts b/tests/page/page-drag.spec.ts
@@ -18,10 +18,12 @@ import type { ElementHandle, Route } from 'playwright-core';
 import { test as it, expect } from './pageTest';
 import { attachFrame } from '../config/utils';
 
+it.skip(({ browserName, browserMajorVersion }) => browserName === 'chromium' && browserMajorVersion < 91);
+it.fixme(({ headless, isLinux }) => isLinux && !headless, 'Stray mouse events on Linux headed mess up the tests.');
+it.fixme(({ headless, isWindows, browserName }) => isWindows && !headless && browserName === 'webkit', 'WebKit win also send stray mouse events.');
+
 it.describe('Drag and drop', () => {
-  it.skip(({ browserName, browserMajorVersion }) => browserName === 'chromium' && browserMajorVersion < 91);
   it.skip(({ isAndroid }) => isAndroid, 'No drag&drop on Android.');
-  it.fixme(({ headless, isLinux }) => isLinux && !headless, 'Stray mouse events on Linux headed mess up the tests.');
 
   it('should work @smoke', async ({ page, server }) => {
     await page.goto(server.PREFIX + '/drag-n-drop.html');
@@ -258,9 +260,7 @@ it.describe('Drag and drop', () => {
     expect(await page.$eval('#target', target => target.contains(document.querySelector('#source')))).toBe(true);
   });
 
-  it('should be able to drag the mouse in a frame', async ({ page, server, isAndroid }) => {
-    it.fixme(isAndroid);
-
+  it('should be able to drag the mouse in a frame', async ({ page, server }) => {
     await page.goto(server.PREFIX + '/frames/one-frame.html');
     const eventsHandle = await trackEvents(await page.frames()[1].$('html'));
     await page.mouse.move(30, 30);
@@ -336,7 +336,6 @@ it.describe('Drag and drop', () => {
 });
 
 it('should work if not doing a drag', async ({ page, isLinux, headless }) => {
-  it.fixme(isLinux && !headless, 'Stray mouse events on Linux headed mess up the tests.');
   const eventsHandle = await trackEvents(await page.$('html'));
   await page.mouse.move(50, 50);
   await page.mouse.down();

diff --git a/tests/page/page-request-intercept.spec.ts b/tests/page/page-request-intercept.spec.ts
@@ -177,7 +177,7 @@ it('should give access to the intercepted response body', async ({ page, server,
 
 it('should intercept multipart/form-data request body', async ({ page, server, asset, browserName }) => {
   it.info().annotations.push({ type: 'issue', description: 'https://github.com/microsoft/playwright/issues/14624' });
-  it.fail(browserName !== 'firefox');
+  it.fixme(browserName !== 'firefox');
   await page.goto(server.PREFIX + '/input/fileupload.html');
   const filePath = path.relative(process.cwd(), asset('file-to-upload.txt'));
   await page.locator('input[type=file]').setInputFiles(filePath);