chore: roll driver to 1.41.0-beta (#2821)

Reference #2772

README.md

@@ -3,9 +3,9 @@
 
 |          | Linux | macOS | Windows |
 |   :---   | :---: | :---: | :---:   |
-| Chromium <!-- GEN:chromium-version -->120.0.6099.56<!-- GEN:stop --> | ✅ | ✅ | ✅ |
+| Chromium <!-- GEN:chromium-version -->121.0.6167.57<!-- GEN:stop --> | ✅ | ✅ | ✅ |
 | WebKit <!-- GEN:webkit-version -->17.4<!-- GEN:stop --> | ✅ | ✅ | ✅ |
-| Firefox <!-- GEN:firefox-version -->119.0<!-- GEN:stop --> | ✅ | ✅ | ✅ |
+| Firefox <!-- GEN:firefox-version -->121.0<!-- GEN:stop --> | ✅ | ✅ | ✅ |
 
 Playwright for .NET is the official language port of [Playwright](https://playwright.dev), the library to automate [Chromium](https://www.chromium.org/Home), [Firefox](https://www.mozilla.org/en-US/firefox/new/) and [WebKit](https://webkit.org/) with a single API. Playwright is built to enable cross-browser web automation that is **ever-green**, **capable**, **reliable** and **fast**.
 

src/Common/Version.props

@@ -2,7 +2,7 @@
   <PropertyGroup>
     <AssemblyVersion>1.40.0</AssemblyVersion>
     <PackageVersion>$(AssemblyVersion)</PackageVersion>
-    <DriverVersion>1.40.0</DriverVersion>
+    <DriverVersion>1.41.0-beta-1705092460000</DriverVersion>
     <ReleaseVersion>$(AssemblyVersion)</ReleaseVersion>
     <FileVersion>$(AssemblyVersion)</FileVersion>
     <NoDefaultExcludes>true</NoDefaultExcludes>

src/Playwright.Tests/BrowserContextRouteTests.cs

@@ -356,4 +356,29 @@ await context.RouteAsync("**/empty.html", (route) =>
         await page.GotoAsync(Server.EmptyPage);
         Assert.AreEqual(new List<int>() { 3, 2, 1 }, interceped);
     }
+
+    [PlaywrightTest("browsercontext-route.spec.ts", "should work if handler with times parameter was removed from another handler")]
+    public async Task ShouldWorkIfHandlerWithTimesParameterWasRemovedFromAnotherHandler()
+    {
+        await using var context = await Browser.NewContextAsync();
+        var page = await context.NewPageAsync();
+        var intercepted = new List<string>();
+        async Task handler(IRoute route)
+        {
+            intercepted.Add("first");
+            route.ContinueAsync();
+        };
+        await context.RouteAsync("**/*", handler, new() { Times = 1 });
+        await context.RouteAsync("**/*", async (route) =>
+        {
+            intercepted.Add("second");
+            await context.UnrouteAsync("**/*", handler);
+            await route.FallbackAsync();
+        });
+        await page.GotoAsync(Server.EmptyPage);
+        Assert.AreEqual(new List<string>() { "second" }, intercepted);
+        intercepted.Clear();
+        await page.GotoAsync(Server.EmptyPage);
+        Assert.AreEqual(new List<string>() { "second" }, intercepted);
+    }
 }

src/Playwright.Tests/PageKeyboardTests.cs

@@ -49,7 +49,7 @@ await Page.EvaluateAsync<string>(@"() => {
                 window.keyPromise = new Promise(resolve => document.addEventListener('keydown', event => resolve(event.key)));
             }");
         await Page.Keyboard.PressAsync("Meta");
-        Assert.AreEqual(TestConstants.IsFirefox && !TestConstants.IsMacOSX ? "OS" : "Meta", await Page.EvaluateAsync<string>("keyPromise"));
+        Assert.AreEqual("Meta", await Page.EvaluateAsync<string>("keyPromise"));
     }
 
     [PlaywrightTest("page-keyboard.spec.ts", "should move with the arrow keys")]
@@ -466,29 +466,9 @@ public async Task ShouldPressTheMetaKey2()
         string key = result.key;
         string code = result.code;
         bool metaKey = result.metaKey;
-
-        if (TestConstants.IsFirefox && !TestConstants.IsMacOSX)
-        {
-            Assert.AreEqual("OS", key);
-        }
-        else
-        {
-            Assert.AreEqual("Meta", key);
-        }
-
-        if (TestConstants.IsFirefox)
-        {
-            Assert.AreEqual("MetaLeft", code);
-        }
-
-        if (TestConstants.IsFirefox && !TestConstants.IsMacOSX)
-        {
-            Assert.False(metaKey);
-        }
-        else
-        {
-            Assert.True(metaKey);
-        }
+        Assert.AreEqual("Meta", key);
+        Assert.AreEqual("MetaLeft", code);
+        Assert.True(metaKey);
     }
 
     [PlaywrightTest("page-keyboard.spec.ts", "should work after a cross origin navigation")]

src/Playwright.Tests/PageNetworkResponseTests.cs

@@ -331,20 +331,4 @@ public async Task ShouldReportIfRequestWasFromServiceWorker()
             Assert.True(res.FromServiceWorker);
         }
     }
-
-    [PlaywrightTest("", "should collect stale handles")]
-    public async Task ShouldCollectStaleHandels()
-    {
-        Page.Request += (sender, request) => { };
-        var response = await Page.GotoAsync(Server.EmptyPage);
-        for (var i = 0; i < 1000; i++)
-        {
-            await Page.EvaluateAsync(@"async () => {
-                const response = await fetch('/');
-                await response.text();
-            }");
-        }
-        var exception = await PlaywrightAssert.ThrowsAsync<PlaywrightException>(async () => await response.AllHeadersAsync());
-        StringAssert.Contains("The object has been collected to prevent unbounded heap growth.", exception.Message);
-    }
 }

src/Playwright.Tests/PageRouteTests.cs

@@ -766,4 +766,27 @@ await page.RouteAsync("**/empty.html", (route) =>
         await page.GotoAsync(Server.EmptyPage);
         Assert.AreEqual(new List<int>() { 3, 2, 1 }, interceped);
     }
+
+    [PlaywrightTest("page-route.spec.ts", "should work if handler with times parameter was removed from another handler")]
+    public async Task ShouldWorkIfHandlerWithTimesParameterWasRemovedFromAnotherHandler()
+    {
+        var intercepted = new List<string>();
+        async Task handler(IRoute route)
+        {
+            intercepted.Add("first");
+            route.ContinueAsync();
+        };
+        await Page.RouteAsync("**/*", handler, new() { Times = 1 });
+        await Page.RouteAsync("**/*", async (route) =>
+        {
+            intercepted.Add("second");
+            await Page.UnrouteAsync("**/*", handler);
+            await route.FallbackAsync();
+        });
+        await Page.GotoAsync(Server.EmptyPage);
+        Assert.AreEqual(new List<string>() { "second" }, intercepted);
+        intercepted.Clear();
+        await Page.GotoAsync(Server.EmptyPage);
+        Assert.AreEqual(new List<string>() { "second" }, intercepted);
+    }
 }

src/Playwright.Tests/PageScreenshotTests.cs

@@ -447,4 +447,32 @@ public async Task PathOptionShouldThrowForUnsupportedMimeType()
         var exception = await PlaywrightAssert.ThrowsAsync<ArgumentException>(() => Page.ScreenshotAsync(new() { Path = "file.txt" }));
         StringAssert.Contains("path: unsupported mime type \"text/plain\"", exception.Message);
     }
+
+    [PlaywrightTest("page-screenshot.spec.ts", "should hide elements based on attr")]
+    public async Task ShouldHideElementsBasedOnAttr()
+    {
+        await Page.SetViewportSizeAsync(500, 500);
+        await Page.GotoAsync(Server.Prefix + "/grid.html");
+        await Page.Locator("div").Nth(5).EvaluateAsync("element => element.setAttribute('data-test-screenshot', 'hide')");
+        var screenshot = await Page.ScreenshotAsync(new() { Style = @"[data-test-screenshot=""hide""] {
+            visibility: hidden;
+        }" });
+        Assert.True(ScreenshotHelper.PixelMatch("hide-should-work.png", screenshot));
+        var visibility = await Page.Locator("div").Nth(5).EvaluateAsync<string>("element => element.style.visibility");
+        Assert.AreEqual("", visibility);
+    }
+
+    [PlaywrightTest("page-screenshot.spec.ts", "should remove elements based on attr")]
+    public async Task ShouldRemoveElementsBasedOnAttr()
+    {
+        await Page.SetViewportSizeAsync(500, 500);
+        await Page.GotoAsync(Server.Prefix + "/grid.html");
+        await Page.Locator("div").Nth(5).EvaluateAsync("element => element.setAttribute('data-test-screenshot', 'remove')");
+        var screenshot = await Page.ScreenshotAsync(new() { Style = @"[data-test-screenshot=""remove""] {
+            visibility: none;
+        }" });
+        Assert.True(ScreenshotHelper.PixelMatch("remove-should-work.png", screenshot));
+        var display = await Page.Locator("div").Nth(5).EvaluateAsync<string>("element => element.style.display");
+        Assert.AreEqual("", display);
+    }
 }