Add tests and modify options

Author: aguspe

rb/lib/selenium/webdriver/common/driver.rb

@@ -323,8 +323,8 @@ def create_bridge(caps:, url:, http_client: nil)
         if caps['webSocketUrl']
           ws_options =
             {
-              response_timeout: caps['wsResponseTimeout'],
-              response_interval: caps['wsResponseInterval']
+              response_timeout: caps[:'ws:response_timeout'],
+              response_interval: caps[:'ws:response_interval']
             }.compact
 
           klass = Remote::BiDiBridge

rb/lib/selenium/webdriver/common/options.rb

@@ -26,7 +26,7 @@ class Options
 
       GRID_OPTIONS = %i[enable_downloads].freeze
 
-      WEBSOCKET_OPTIONS = %i[ws_response_timeout ws_response_interval].freeze
+      WEBSOCKET_OPTIONS = %i[ws:response_timeout ws:response_interval].freeze
 
       class << self
         attr_reader :driver_path

rb/spec/integration/selenium/webdriver/bidi/browsing_context_spec.rb

@@ -132,6 +132,22 @@ class BiDi
           browsing_context.activate
           expect(driver.execute_script('return document.hasFocus();')).to be_truthy
         end
+
+        it 'times out if a bidi command takes too long to receive a response' do
+          reset_driver!(web_socket_url: true, 'ws:response_timeout': 1) do |driver|
+            expect {
+              driver.navigate.to url_for('sleep?time=2')
+            }.to raise_error(Selenium::WebDriver::Error::TimeoutError)
+          end
+        end
+
+        it 'does not time out if a bidi command is fast enough' do
+          reset_driver!(web_socket_url: true, 'ws:response_timeout': 5) do |driver|
+            expect {
+              driver.navigate.to url_for('sleep?time=1')
+            }.not_to raise_error
+          end
+        end
       end
     end # BiDi
   end # WebDriver