From b0ac15e7a303d42f59a1add650b24311a6c13e0f Mon Sep 17 00:00:00 2001 From: Nikolay Borisenko <22616990+nvborisenko@users.noreply.github.com> Date: Thu, 18 Sep 2025 19:19:01 +0300 Subject: [PATCH 1/2] [dotnet] [bidi] Enable WebExtension tests at least for Firefox --- .../BiDi/WebExtension/WebExtensionTest.cs | 18 ++++++++++-------- 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/dotnet/test/common/BiDi/WebExtension/WebExtensionTest.cs b/dotnet/test/common/BiDi/WebExtension/WebExtensionTest.cs index 327ce5d4d29f3..daef8b8d2227d 100644 --- a/dotnet/test/common/BiDi/WebExtension/WebExtensionTest.cs +++ b/dotnet/test/common/BiDi/WebExtension/WebExtensionTest.cs @@ -18,21 +18,23 @@ // using NUnit.Framework; -using System; using System.IO; using System.Threading.Tasks; namespace OpenQA.Selenium.BiDi.WebExtension; -[Ignore(""" - The following test suite wants to set driver arguments via Options, but it breaks CDP/DevTools tests. - The desired arguments (for Chromium only?): - --enable-unsafe-extension-debugging - --remote-debugging-pipe - Ignoring these tests for now. Hopefully https://github.com/SeleniumHQ/selenium/issues/15536 will be resolved soon. - """)] +[IgnoreBrowser(Selenium.Browser.Chrome, ChromiumIgnoreReason)] +[IgnoreBrowser(Selenium.Browser.Edge, ChromiumIgnoreReason)] class WebExtensionTest : BiDiTestFixture { + public const string ChromiumIgnoreReason = """ + The following test suite wants to set driver arguments via Options, but it breaks CDP/DevTools tests. + The desired arguments (for Chromium only?): + --enable-unsafe-extension-debugging + --remote-debugging-pipe + Ignoring these tests for now. Hopefully https://github.com/SeleniumHQ/selenium/issues/15536 will be resolved soon. + """; + [Test] public async Task CanInstallPathWebExtension() { From 6ab82866e2f899e0e25a72ea59f52e1afa58146e Mon Sep 17 00:00:00 2001 From: Nikolay Borisenko <22616990+nvborisenko@users.noreply.github.com> Date: Thu, 18 Sep 2025 19:23:40 +0300 Subject: [PATCH 2/2] Private ignore reason --- dotnet/test/common/BiDi/WebExtension/WebExtensionTest.cs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dotnet/test/common/BiDi/WebExtension/WebExtensionTest.cs b/dotnet/test/common/BiDi/WebExtension/WebExtensionTest.cs index daef8b8d2227d..5065abe2554be 100644 --- a/dotnet/test/common/BiDi/WebExtension/WebExtensionTest.cs +++ b/dotnet/test/common/BiDi/WebExtension/WebExtensionTest.cs @@ -27,7 +27,7 @@ namespace OpenQA.Selenium.BiDi.WebExtension; [IgnoreBrowser(Selenium.Browser.Edge, ChromiumIgnoreReason)] class WebExtensionTest : BiDiTestFixture { - public const string ChromiumIgnoreReason = """ + const string ChromiumIgnoreReason = """ The following test suite wants to set driver arguments via Options, but it breaks CDP/DevTools tests. The desired arguments (for Chromium only?): --enable-unsafe-extension-debugging