Merge branch 'trunk' into sm_firefox_latest_version

Author: bonigarcia

.github/workflows/bazel.yml

@@ -116,7 +116,7 @@ jobs:
           node-version: ${{ inputs.node-version }}
       - name: Setup Bazel with caching
         if: inputs.caching
-        uses: bazel-contrib/setup-bazel@0.8.5
+        uses: bazel-contrib/setup-bazel@0.9.1
         with:
           bazelisk-cache: true
           bazelrc: common --color=yes
@@ -130,7 +130,7 @@ jobs:
           repository-cache: true
       - name: Setup Bazel without caching
         if: inputs.caching == false
-        uses: bazel-contrib/setup-bazel@0.8.5
+        uses: bazel-contrib/setup-bazel@0.9.1
         with:
           bazelrc: common --color=yes
       - name: Setup Fluxbox and Xvfb

.github/workflows/ci-python.yml

@@ -103,10 +103,16 @@ jobs:
           - browser: safari
             os: macos
           - browser: chrome
-            os: macos
+            os: ubuntu
+          - browser: edge
+            os: ubuntu
+          - browser: firefox
+            os: ubuntu
     with:
       name: Integration Tests (${{ matrix.browser }}, ${{ matrix.os }})
       browser: ${{ matrix.browser }}
       os: ${{ matrix.os }}
       cache-key: py-browser-${{ matrix.browser }}
-      run: bazel test --flaky_test_attempts 3 //py:test-${{ matrix.browser }}
+      run: |
+        bazel test --local_test_jobs 1 --flaky_test_attempts 3 //py:common-${{ matrix.browser }}-bidi
+        bazel test --local_test_jobs 1 --flaky_test_attempts 3 //py:test-${{ matrix.browser }}

.github/workflows/ci.yml

@@ -21,7 +21,7 @@ jobs:
         with:
           fetch-depth: 50
       - name: Setup Bazel
-        uses: bazel-contrib/setup-bazel@0.8.5
+        uses: bazel-contrib/setup-bazel@0.9.1
         with:
           bazelisk-cache: true
           cache-version: 2

.skipped-tests

@@ -31,6 +31,8 @@
 -//py:test-chrome-test/selenium/webdriver/chrome/chrome_launcher_tests.py
 -//py:test-chrome-test/selenium/webdriver/chrome/chrome_service_tests.py
 -//py:test-chrome-test/selenium/webdriver/chrome/proxy_tests.py
+-//py:test-edge-test/selenium/webdriver/edge/edge_launcher_tests.py
+-//py:test-edge-test/selenium/webdriver/edge/edge_service_tests.py
 -//rb/spec/integration/selenium/webdriver/chrome:service-chrome
 -//rb/spec/integration/selenium/webdriver/chrome:service-chrome-bidi
 -//rb/spec/integration/selenium/webdriver/chrome:service-chrome-remote

MODULE.bazel

@@ -207,7 +207,7 @@ maven.install(
         "org.assertj:assertj-core:3.26.3",
         "org.bouncycastle:bcpkix-jdk18on:1.78.1",
         "org.eclipse.mylyn.github:org.eclipse.egit.github.core:2.1.5",
-        "org.hsqldb:hsqldb:2.7.3",
+        "org.hsqldb:hsqldb:2.7.4",
         "org.jspecify:jspecify:1.0.0",
         "org.junit.jupiter:junit-jupiter-api:5.11.3",
         "org.junit.jupiter:junit-jupiter-engine:5.11.3",

README.md

@@ -317,6 +317,18 @@ Run unit tests with:
 bazel test //py:unit
 ```
 
+To run common tests with a specific browser:
+
+```sh
+bazel test //py:common-<browsername>
+```
+
+To run common tests with a specific browser (include BiDi tests):
+
+```sh
+bazel test //py:common-<browsername>-bidi
+```
+
 To run tests with a specific browser:
 
 ```sh

dotnet/src/webdriver/DefaultFileDetector.cs

@@ -17,6 +17,10 @@
 // under the License.
 // </copyright>
 
+using System.Diagnostics.CodeAnalysis;
+
+#nullable enable
+
 namespace OpenQA.Selenium
 {
     /// <summary>
@@ -31,7 +35,7 @@ public class DefaultFileDetector : IFileDetector
         /// </summary>
         /// <param name="keySequence">The sequence to test for file existence.</param>
         /// <returns>This method always returns <see langword="false"/> in this implementation.</returns>
-        public bool IsFile(string keySequence)
+        public bool IsFile([NotNullWhen(true)] string? keySequence)
         {
             return false;
         }

dotnet/src/webdriver/DetachedShadowRootException.cs

@@ -18,7 +18,8 @@
 // </copyright>
 
 using System;
-using System.Runtime.Serialization;
+
+#nullable enable
 
 namespace OpenQA.Selenium
 {
@@ -41,7 +42,7 @@ public DetachedShadowRootException()
         /// a specified error message.
         /// </summary>
         /// <param name="message">The message that describes the error.</param>
-        public DetachedShadowRootException(string message)
+        public DetachedShadowRootException(string? message)
             : base(message)
         {
         }
@@ -54,7 +55,7 @@ public DetachedShadowRootException(string message)
         /// <param name="message">The error message that explains the reason for the exception.</param>
         /// <param name="innerException">The exception that is the cause of the current exception,
         /// or <see langword="null"/> if no inner exception is specified.</param>
-        public DetachedShadowRootException(string message, Exception innerException)
+        public DetachedShadowRootException(string? message, Exception? innerException)
             : base(message, innerException)
         {
         }

dotnet/src/webdriver/DevTools/CommandResponseException.cs

@@ -19,6 +19,8 @@
 
 using System;
 
+#nullable enable
+
 namespace OpenQA.Selenium.DevTools
 {
     /// <summary>
@@ -39,7 +41,7 @@ public CommandResponseException()
         /// Initializes a new instance of the <see cref="CommandResponseException"/> class with the specified message.
         /// </summary>
         /// <param name="message">The message of the exception.</param>
-        public CommandResponseException(string message)
+        public CommandResponseException(string? message)
             : base(message)
         {
         }
@@ -49,7 +51,7 @@ public CommandResponseException(string message)
         /// </summary>
         /// <param name="message">The message of the exception.</param>
         /// <param name="innerException">The inner exception for this exception.</param>
-        public CommandResponseException(string message, Exception innerException)
+        public CommandResponseException(string? message, Exception? innerException)
             : base(message, innerException)
         {
         }

dotnet/src/webdriver/DriverServiceNotFoundException.cs

@@ -18,12 +18,13 @@
 // </copyright>
 
 using System;
-using System.Runtime.Serialization;
+
+#nullable enable
 
 namespace OpenQA.Selenium
 {
     /// <summary>
-    /// The exception that is thrown when an element is not visible.
+    /// The exception that is thrown when the driver service is not available.
     /// </summary>
     [Serializable]
     public class DriverServiceNotFoundException : WebDriverException
@@ -41,7 +42,7 @@ public DriverServiceNotFoundException()
         /// a specified error message.
         /// </summary>
         /// <param name="message">The message that describes the error.</param>
-        public DriverServiceNotFoundException(string message)
+        public DriverServiceNotFoundException(string? message)
             : base(message)
         {
         }
@@ -54,7 +55,7 @@ public DriverServiceNotFoundException(string message)
         /// <param name="message">The error message that explains the reason for the exception.</param>
         /// <param name="innerException">The exception that is the cause of the current exception,
         /// or <see langword="null"/> if no inner exception is specified.</param>
-        public DriverServiceNotFoundException(string message, Exception innerException)
+        public DriverServiceNotFoundException(string? message, Exception? innerException)
             : base(message, innerException)
         {
         }