From 3ff3567c89a774a9e2086af07a858266d84d41fe Mon Sep 17 00:00:00 2001
From: Andrey Lushnikov <aslushnikov@gmail.com>
Date: Wed, 22 Feb 2023 17:22:56 -0800
Subject: [PATCH] test: update context menu tests (#21126)

It turns out these new contextmenu-based tests do not work on
Chromium-based browsers on Windows.

We consider context menu support a best-effort, so we skip them
for now.
---
 tests/page/page-keyboard.spec.ts | 6 ++++--
 tests/page/page-mouse.spec.ts    | 7 ++++++-
 tests/page/wheel.spec.ts         | 3 ++-
 3 files changed, 12 insertions(+), 4 deletions(-)

diff --git a/tests/page/page-keyboard.spec.ts b/tests/page/page-keyboard.spec.ts
index 0b73ecf409..1a312c46e5 100644
--- a/tests/page/page-keyboard.spec.ts
+++ b/tests/page/page-keyboard.spec.ts
@@ -664,7 +664,8 @@ async function captureLastKeydown(page) {
   return lastEvent;
 }
 
-it('should dispatch insertText after context menu was opened', async ({ server, page, browserName }) => {
+it('should dispatch insertText after context menu was opened', async ({ server, page, browserName, isWindows }) => {
+  it.skip(browserName === 'chromium' && isWindows, 'context menu support is best-effort for Linux and MacOS');
   await page.goto(server.PREFIX + '/input/textarea.html');
   await page.evaluate(() => {
     window['contextMenuPromise'] = new Promise(x => {
@@ -682,7 +683,8 @@ it('should dispatch insertText after context menu was opened', async ({ server,
   await expect.poll(() => page.locator('textarea').inputValue()).toBe('嗨');
 });
 
-it('should type after context menu was opened', async ({ server, page, browserName }) => {
+it('should type after context menu was opened', async ({ server, page, browserName, isWindows }) => {
+  it.skip(browserName === 'chromium' && isWindows, 'context menu support is best-effort for Linux and MacOS');
   await page.evaluate(() => {
     window['keys'] = [];
     window.addEventListener('keydown', event => window['keys'].push(event.key));
diff --git a/tests/page/page-mouse.spec.ts b/tests/page/page-mouse.spec.ts
index d8624c546e..cf93bd8bf1 100644
--- a/tests/page/page-mouse.spec.ts
+++ b/tests/page/page-mouse.spec.ts
@@ -253,6 +253,10 @@ it('should always round down', async ({ page }) => {
 
 it('should not crash on mouse drag with any button', async ({ page }) => {
   it.info().annotations.push({ type: 'issue', description: 'https://github.com/microsoft/playwright/issues/16609' });
+  await page.evaluate(() => {
+    // Do not show contextmenu on right click since it is poorly supported.
+    window.addEventListener('contextmenu', e => e.preventDefault(), false);
+  });
   for (const button of ['left', 'middle', 'right'] as const) {
     await page.mouse.move(50, 50);
     await page.mouse.down({ button });
@@ -260,9 +264,10 @@ it('should not crash on mouse drag with any button', async ({ page }) => {
   }
 });
 
-it('should dispatch mouse move after context menu was opened', async ({ page, browserName }) => {
+it('should dispatch mouse move after context menu was opened', async ({ page, browserName, isWindows }) => {
   it.info().annotations.push({ type: 'issue', description: 'https://github.com/microsoft/playwright/issues/20823' });
   it.fixme(browserName === 'firefox');
+  it.skip(browserName === 'chromium' && isWindows, 'context menu support is best-effort for Linux and MacOS');
   await page.evaluate(() => {
     window['contextMenuPromise'] = new Promise(x => {
       window.addEventListener('contextmenu', x, false);
diff --git a/tests/page/wheel.spec.ts b/tests/page/wheel.spec.ts
index e6aa4546ad..fb655c7e43 100644
--- a/tests/page/wheel.spec.ts
+++ b/tests/page/wheel.spec.ts
@@ -67,9 +67,10 @@ it('should dispatch wheel events @smoke', async ({ page, server }) => {
   });
 });
 
-it('should dispatch wheel events after context menu was opened', async ({ page, browserName }) => {
+it('should dispatch wheel events after context menu was opened', async ({ page, browserName, isWindows }) => {
   it.info().annotations.push({ type: 'issue', description: 'https://github.com/microsoft/playwright/issues/20823' });
   it.fixme(browserName === 'firefox');
+  it.skip(browserName === 'chromium' && isWindows, 'context menu support is best-effort for Linux and MacOS');
 
   await page.setContent(`<div style="width: 5000px; height: 5000px;"></div>`);
   await page.mouse.move(50, 60);