Fix page-click.spec.ts headful test failures

src/PlaywrightSharp.Nunit/TestExpectations/TestExpectations.local.json

@@ -227,12 +227,6 @@
     "parameters": ["headful"],
     "expectations": ["FAIL"]
   },
-  {
-    "testIdPattern": "[page-click.spec.ts] *",
-    "platforms": ["darwin", "linux", "win32"],
-    "parameters": ["headful"],
-    "expectations": ["FAIL"]
-  },
   {
     "testIdPattern": "[page-fill.spec.ts] *",
     "platforms": ["darwin", "linux", "win32"],

src/PlaywrightSharp.Tests/PageClickTests.cs

@@ -1,6 +1,7 @@
 using System;
 using System.Collections.Generic;
 using System.Drawing;
+using System.Linq;
 using System.Threading.Tasks;
 using NUnit.Framework;
 using PlaywrightSharp.Helpers;
@@ -261,7 +262,13 @@
             Assert.That(await Page.EvaluateAsync<bool?>("result.check"), Is.Null);
             await Page.ClickAsync("input#agree");
             Assert.That(await Page.EvaluateAsync<bool>("result.check"), Is.True);
-            Assert.That(await Page.EvaluateAsync<string[]>("result.events"), Is.EqualTo(new[] {
+            var events = await Page.EvaluateAsync<string[]>("result.events");
+            if (!TestConstants.IsHeadless)
+            {
+                events = events.Where(e => e != "mouseout" && e != "mouseleave").ToArray();
+            }
+
+            Assert.That(events, Is.EqualTo(new[] {
                 "mouseover",
                 "mouseenter",
                 "mousemove",
@@ -549,6 +556,8 @@
                 document.body.style.margin = '0';
             }");
 
+            // rafraf for Firefox to kick in the animation.
+            await Page.EvaluateAsync("() => new Promise(f => requestAnimationFrame(() => requestAnimationFrame(f)))");
             await Page.ClickAsync("button");
             Assert.That(await Page.EvaluateAsync<string>("window.result"), Is.EqualTo("Clicked"));
             Assert.That(await Page.EvaluateAsync<int>("pageX"), Is.EqualTo(300));
@@ -642,7 +651,14 @@
         [Test, Timeout(TestConstants.DefaultTestTimeout)]
         public async Task ShouldWaitForSelectToBeEnabled()
         {
-            await Page.SetContentAsync("<select onclick=\"javascript: window.__CLICKED = true;\" disabled><option selected>Hello</option></select>");
+            await Page.SetContentAsync(@"
+                <select disabled><option selected>Hello</option></select>
+                <script>
+                    document.querySelector('select').addEventListener('mousedown', event => {
+                        window.__CLICKED=true;
+                        event.preventDefault();
+                    });
+                </script>");
             var clickTask = Page.ClickAsync("select");
             await GiveItAChanceToClick(Page);
             Assert.That(await Page.EvaluateAsync<bool?>("window.__CLICKED"), Is.Null);
@@ -894,7 +910,7 @@
         public async Task ShouldClickTheButtonWhenWindowInnerWidthIsCorrupted()
         {
             await Page.GoToAsync(TestConstants.ServerUrl + "/input/button.html");
-            await Page.EvaluateAsync(@"() => window.innerWith = 0");
+            await Page.EvaluateAsync(@"() => Object.defineProperty(window, 'innerWidth', { value: 0 })");
 
             await Page.ClickAsync("button");
             Assert.That(await Page.EvaluateAsync<string>("result"), Is.EqualTo("Clicked"));

src/PlaywrightSharp.Tests/TestConstants.cs

@@ -117,6 +117,8 @@ internal static Task<IBrowser> LaunchDefaultHeadful(
         internal static readonly bool IsChromium = Product.Equals(ChromiumProduct, StringComparison.OrdinalIgnoreCase);
         internal static readonly bool IsMacOSX = RuntimeInformation.IsOSPlatform(OSPlatform.OSX);
         internal static readonly bool IsWindows = RuntimeInformation.IsOSPlatform(OSPlatform.Windows);
+        internal static readonly bool IsHeadless = !string.Equals(
+            Environment.GetEnvironmentVariable("HEADLESS"), "false", StringComparison.OrdinalIgnoreCase);
 
         public static readonly IEnumerable<string> NestedFramesDumpResult = new List<string>()
         {