Merge branch 'trunk' into use_rbs_trace

Author: aguspe

Rakefile

@@ -510,7 +510,9 @@ namespace :node do
 
   desc 'Generate Node documentation'
   task :docs do |_task, arguments|
-    abort('Aborting documentation update: nightly versions should not update docs.') if node_version.include?('nightly')
+    if node_version.include?('nightly') && !arguments.to_a.include?('force')
+      abort('Aborting documentation update: nightly versions should not update docs.')
+    end
 
     puts 'Generating Node documentation'
     FileUtils.rm_rf('build/docs/api/javascript/')
@@ -608,7 +610,7 @@ namespace :py do
 
   desc 'Generate Python documentation'
   task :docs do |_task, arguments|
-    if python_version.match?(/^\d+\.\d+\.\d+\.\d+$/)
+    if python_version.match?(/^\d+\.\d+\.\d+\.\d+$/) && !arguments.to_a.include?('force')
       abort('Aborting documentation update: nightly versions should not update docs.')
     end
     puts 'Generating Python documentation'
@@ -776,7 +778,9 @@ namespace :rb do
 
   desc 'Generate Ruby documentation'
   task :docs do |_task, arguments|
-    abort('Aborting documentation update: nightly versions should not update docs.') if ruby_version.include?('nightly')
+    if ruby_version.include?('nightly') && !arguments.to_a.include?('force')
+      abort('Aborting documentation update: nightly versions should not update docs.')
+    end
     puts 'Generating Ruby documentation'
 
     FileUtils.rm_rf('build/docs/api/rb/')
@@ -872,7 +876,7 @@ namespace :dotnet do
 
   desc 'Generate .NET documentation'
   task :docs do |_task, arguments|
-    if dotnet_version.include?('nightly')
+    if dotnet_version.include?('nightly') && !arguments.to_a.include?('force')
       abort('Aborting documentation update: nightly versions should not update docs.')
     end
 
@@ -987,7 +991,7 @@ namespace :java do
 
   desc 'Generate Java documentation'
   task :docs do |_task, arguments|
-    if java_version.include?('SNAPSHOT')
+    if java_version.include?('SNAPSHOT') && !arguments.to_a.include?('force')
       abort('Aborting documentation update: snapshot versions should not update docs.')
     end
 

common/repositories.bzl

@@ -50,8 +50,8 @@ js_library(
 
     http_archive(
         name = "linux_beta_firefox",
-        url = "https://ftp.mozilla.org/pub/firefox/releases/139.0b4/linux-x86_64/en-US/firefox-139.0b4.tar.xz",
-        sha256 = "21bfb99fe445e64ad8fb86822fd6d77c9c0d387530b787f3c3fc99b97723d664",
+        url = "https://ftp.mozilla.org/pub/firefox/releases/139.0b6/linux-x86_64/en-US/firefox-139.0b6.tar.xz",
+        sha256 = "39f087356b885fb4bbfebe315d09bac3979798a7e0bff0b7c788707c0c7d4d11",
         build_file_content = """
 load("@aspect_rules_js//js:defs.bzl", "js_library")
 package(default_visibility = ["//visibility:public"])
@@ -72,8 +72,8 @@ js_library(
 
     dmg_archive(
         name = "mac_beta_firefox",
-        url = "https://ftp.mozilla.org/pub/firefox/releases/139.0b4/mac/en-US/Firefox%20139.0b4.dmg",
-        sha256 = "c61d0fe161c6cb778a3e564f4cffa70cc7fb6bb5770586fa66c7db8ab9c31cf1",
+        url = "https://ftp.mozilla.org/pub/firefox/releases/139.0b6/mac/en-US/Firefox%20139.0b6.dmg",
+        sha256 = "d2fa2ef24f80cbadab7dfb31639f76d8a530acd1653bb8a437ef128937ce1019",
         build_file_content = """
 load("@aspect_rules_js//js:defs.bzl", "js_library")
 package(default_visibility = ["//visibility:public"])
@@ -123,10 +123,10 @@ js_library(
 
     pkg_archive(
         name = "mac_edge",
-        url = "https://msedge.sf.dl.delivery.mp.microsoft.com/filestreamingservice/files/7a2f13f4-e241-4582-94b2-6c56bef6b23d/MicrosoftEdge-136.0.3240.50.pkg",
-        sha256 = "3500c3a7f548e222344c3c110e0a0b190a033adb10d020ce58b8c64a384ad8b6",
+        url = "https://msedge.sf.dl.delivery.mp.microsoft.com/filestreamingservice/files/0d424762-5771-4a00-ada6-d1a3fb2e2054/MicrosoftEdge-136.0.3240.64.pkg",
+        sha256 = "483324afe4db695c9497822e3e220065379d92077b8f29abb1e8d8f6d045909c",
         move = {
-            "MicrosoftEdge-136.0.3240.50.pkg/Payload/Microsoft Edge.app": "Edge.app",
+            "MicrosoftEdge-136.0.3240.64.pkg/Payload/Microsoft Edge.app": "Edge.app",
         },
         build_file_content = """
 load("@aspect_rules_js//js:defs.bzl", "js_library")
@@ -143,8 +143,8 @@ js_library(
 
     deb_archive(
         name = "linux_edge",
-        url = "https://packages.microsoft.com/repos/edge/pool/main/m/microsoft-edge-stable/microsoft-edge-stable_136.0.3240.50-1_amd64.deb",
-        sha256 = "19d27dddf6d4a46e1e03b3d0fa1eb4af0d5870f12d3816e0431b983331f947e9",
+        url = "https://packages.microsoft.com/repos/edge/pool/main/m/microsoft-edge-stable/microsoft-edge-stable_136.0.3240.64-1_amd64.deb",
+        sha256 = "cf2a2ea4d76d5cae66bc8e68ca9f40255ad6c1894c28e55a6813a70ce8c73b4a",
         build_file_content = """
 load("@aspect_rules_js//js:defs.bzl", "js_library")
 package(default_visibility = ["//visibility:public"])
@@ -182,8 +182,8 @@ js_library(
 
     http_archive(
         name = "mac_edgedriver",
-        url = "https://msedgedriver.azureedge.net/136.0.3240.50/edgedriver_mac64.zip",
-        sha256 = "9c592695119986af76bd0182bc355716e29844dd97052f77865f55fffbf6db3d",
+        url = "https://msedgedriver.azureedge.net/136.0.3240.64/edgedriver_mac64.zip",
+        sha256 = "729731958bfeee999006572973c8eee6d4bd402e34c5b410fb29bfbaeeafb8bd",
         build_file_content = """
 load("@aspect_rules_js//js:defs.bzl", "js_library")
 package(default_visibility = ["//visibility:public"])

py/selenium/webdriver/common/bidi/log.py

@@ -68,3 +68,12 @@ def from_json(cls, json):
             stacktrace=json["stackTrace"],
             type_=json["type"],
         )
+
+
+class LogLevel:
+    """Represents log level."""
+
+    DEBUG = "debug"
+    INFO = "info"
+    WARN = "warn"
+    ERROR = "error"

py/test/selenium/webdriver/common/bidi_script_tests.py

@@ -15,6 +15,7 @@
 # specific language governing permissions and limitations
 # under the License.
 
+from selenium.webdriver.common.bidi.log import LogLevel
 from selenium.webdriver.common.by import By
 from selenium.webdriver.support.ui import WebDriverWait
 
@@ -31,7 +32,7 @@ def test_logs_console_messages(driver, pages):
     WebDriverWait(driver, 5).until(lambda _: log_entries)
 
     log_entry = log_entries[0]
-    assert log_entry.level == "info"
+    assert log_entry.level == LogLevel.INFO
     assert log_entry.method == "log"
     assert log_entry.text == "Hello, world!"
     assert log_entry.type_ == "console"
@@ -55,7 +56,7 @@ def log_error(entry):
     assert len(log_entries) == 1
 
     log_entry = log_entries[0]
-    assert log_entry.level == "error"
+    assert log_entry.level == LogLevel.ERROR
     assert log_entry.method == "error"
     assert log_entry.text == "I am console error"
     assert log_entry.type_ == "console"
@@ -105,7 +106,7 @@ def test_javascript_error_messages(driver, pages):
 
     log_entry = log_entries[0]
     assert log_entry.text == "Error: Not working"
-    assert log_entry.level == "error"
+    assert log_entry.level == LogLevel.ERROR
     assert log_entry.type_ == "javascript"
 
 

rb/lib/selenium/webdriver/firefox/service.rb

@@ -29,9 +29,9 @@ class Service < WebDriver::Service
         # @rbs (?path: nil, ?port: nil, ?log: nil, ?args: nil) -> void
         def initialize(path: nil, port: nil, log: nil, args: nil)
           args ||= []
-          unless args.any? { |arg| arg.include?('--connect-existing') }
+          unless args.any? { |arg| arg.include?('--connect-existing') || arg.include?('--websocket-port') }
             args << '--websocket-port'
-            args << WebDriver::PortProber.above(9222).to_s
+            args << '0'
           end
           super
         end

rb/spec/integration/selenium/webdriver/chrome/service_spec.rb

@@ -22,7 +22,9 @@
 module Selenium
   module WebDriver
     module Chrome
-      describe Service, exclusive: [{bidi: false, reason: 'Not yet implemented with BiDi'}, {browser: :chrome}] do
+      describe Service,
+               {exclude: {driver: :remote},
+                exclusive: [{bidi: false, reason: 'Not yet implemented with BiDi'}, {browser: :chrome}]} do
         let(:service) { described_class.new }
         let(:service_manager) { service.launch }
 

rb/spec/integration/selenium/webdriver/edge/service_spec.rb

@@ -22,7 +22,9 @@
 module Selenium
   module WebDriver
     module Edge
-      describe Service, exclusive: [{bidi: false, reason: 'Not yet implemented with BiDi'}, {browser: :edge}] do
+      describe Service,
+               {exclude: {driver: :remote},
+                exclusive: [{bidi: false, reason: 'Not yet implemented with BiDi'}, {browser: :edge}]} do
         let(:service) { described_class.new }
         let(:service_manager) { service.launch }
 

rb/spec/integration/selenium/webdriver/firefox/service_spec.rb

@@ -22,7 +22,9 @@
 module Selenium
   module WebDriver
     module Firefox
-      describe Service, exclusive: [{bidi: false, reason: 'Not yet implemented with BiDi'}, {browser: :firefox}] do
+      describe Service,
+               {exclude: {driver: :remote},
+                exclusive: [{bidi: false, reason: 'Not yet implemented with BiDi'}, {browser: :firefox}]} do
         let(:service) { described_class.new }
         let(:service_manager) { service.launch }
 
@@ -37,21 +39,6 @@ module Firefox
         it 'can be started outside driver' do
           expect(service_manager.uri).to be_a(URI)
         end
-
-        context 'with BiDi enabled', exclusive: {bidi: true, reason: 'only executed when bidi is enabled'} do
-          it 'ensures two service instances use different websocket port' do
-            service1 = described_class.new
-            service2 = described_class.new
-
-            ws_index1 = service1.args.index('--websocket-port')
-            ws_index2 = service2.args.index('--websocket-port')
-
-            port1 = service1.args[ws_index1 + 1].to_i
-            port2 = service2.args[ws_index2 + 1].to_i
-
-            expect(port1).not_to eq(port2)
-          end
-        end
       end
     end # Firefox
   end # WebDriver

rb/spec/integration/selenium/webdriver/ie/service_spec.rb

@@ -22,7 +22,9 @@
 module Selenium
   module WebDriver
     module IE
-      describe Service, exclusive: [{bidi: false, reason: 'Not yet implemented with BiDi'}, {browser: :ie}] do
+      describe Service, {exclusive: [{bidi: false, reason: 'Not yet implemented with BiDi'}, {browser: :ie}],
+               exclude: {driver: :remote}} do
+
         let(:service) { described_class.new }
         let(:service_manager) { service.launch }
 

rb/spec/integration/selenium/webdriver/remote/driver_spec.rb

@@ -23,6 +23,8 @@ module Selenium
   module WebDriver
     module Remote
       describe Driver, exclusive: [{bidi: false, reason: 'Not yet implemented with BiDi'}, {driver: :remote}] do
+        let(:tempfile) { create_tempfile }
+
         it 'exposes session_id' do
           expect(driver.session_id).to be_a(String)
         end
@@ -35,15 +37,15 @@ module Remote
            flaky: {browser: :safari, ci: :github, reason: 'unreliable with downloads'} do
           driver.navigate.to url_for('upload.html')
 
-          driver.find_element(id: 'upload').send_keys(__FILE__)
+          driver.find_element(id: 'upload').send_keys(tempfile.path)
           driver.find_element(id: 'go').submit
           wait.until { driver.find_element(id: 'upload_label').displayed? }
 
           driver.switch_to.frame('upload_target')
           wait.until { driver.find_element(xpath: '//body') }
 
           body = driver.find_element(xpath: '//body')
-          expect(body.text.scan('Licensed to the Software Freedom Conservancy').count).to eq(2)
+          expect(body.text.scan('This is a dummy test file').count).to eq(1)
         end
 
         it 'lists downloads', exclude: {browser: :safari, reason: 'grid hangs'} do