[rb] update styling for rubocop

Author: titusfortner

rb/lib/selenium/webdriver/atoms.rb

@@ -27,7 +27,7 @@ def read_atom(function)
       end
 
       def execute_atom(function_name, *arguments)
-        script = format("/* #{function_name} */ return (%<atom>s).apply(null, arguments)",
+        script = format("/* #{function_name} */return (%<atom>s).apply(null, arguments)",
                         atom: read_atom(function_name))
         execute_script(script, *arguments)
       end

rb/lib/selenium/webdriver/bidi/browsing_context.rb

@@ -27,15 +27,15 @@ class BrowsingContext
         attr_accessor :id
 
         READINESS_STATE = {
-          none: "none",
-          interactive: "interactive",
-          complete: "complete"
+          none: 'none',
+          interactive: 'interactive',
+          complete: 'complete'
         }.freeze
 
         def initialize(driver:, browsing_context_id: nil, type: nil, reference_context: nil)
           unless driver.capabilities.web_socket_url
             raise Error::WebDriverError,
-                  "WebDriver instance must support BiDi protocol"
+                  'WebDriver instance must support BiDi protocol'
           end
 
           unless type.nil? || %i[window tab].include?(type)
@@ -44,7 +44,7 @@ def initialize(driver:, browsing_context_id: nil, type: nil, reference_context:
           end
 
           @bidi = driver.bidi
-          @id = browsing_context_id.nil? ? create(type, reference_context)["context"] : browsing_context_id
+          @id = browsing_context_id.nil? ? create(type, reference_context)['context'] : browsing_context_id
         end
 
         def navigate(url:, readiness_state: nil)
@@ -53,17 +53,17 @@ def navigate(url:, readiness_state: nil)
                   "Valid readiness states are :none, :interactive & :complete. Received: #{readiness_state.inspect}"
           end
 
-          navigate_result = @bidi.send_cmd("browsingContext.navigate", context: @id, url: url,
+          navigate_result = @bidi.send_cmd('browsingContext.navigate', context: @id, url: url,
                                                                        wait: READINESS_STATE[readiness_state])
 
           NavigateResult.new(
-            url: navigate_result["url"],
-            navigation_id: navigate_result["navigation"]
+            url: navigate_result['url'],
+            navigation_id: navigate_result['navigation']
           )
         end
 
         def get_tree(max_depth: nil)
-          result = @bidi.send_cmd("browsingContext.getTree", root: @id, maxDepth: max_depth).dig("contexts", 0)
+          result = @bidi.send_cmd('browsingContext.getTree', root: @id, maxDepth: max_depth).dig('contexts', 0)
 
           BrowsingContextInfo.new(
             id: result['context'],
@@ -74,13 +74,13 @@ def get_tree(max_depth: nil)
         end
 
         def close
-          @bidi.send_cmd("browsingContext.close", context: @id)
+          @bidi.send_cmd('browsingContext.close', context: @id)
         end
 
         private
 
         def create(type, reference_context)
-          @bidi.send_cmd("browsingContext.create", type: type.to_s, referenceContext: reference_context)
+          @bidi.send_cmd('browsingContext.create', type: type.to_s, referenceContext: reference_context)
         end
       end # BrowsingContext
     end # BiDi

rb/lib/selenium/webdriver/bidi/log/javascript_log_entry.rb

@@ -25,7 +25,7 @@ class JavascriptLogEntry < GenericLogEntry
 
         def initialize(level:, text:, timestamp:, type:, stack_trace:)
           super(level: level, text: text, timestamp: timestamp, type: type, stack_trace: stack_trace)
-          @type = "javascript"
+          @type = 'javascript'
         end
       end # JavascriptLogEntry
     end # BiDi

rb/lib/selenium/webdriver/bidi/log_inspector.rb

@@ -33,20 +33,20 @@ class LogInspector
         }.freeze
 
         LOG_LEVEL = {
-          DEBUG: "debug",
-          ERROR: "error",
-          INFO: "info",
-          WARNING: "warning"
+          DEBUG: 'debug',
+          ERROR: 'error',
+          INFO: 'info',
+          WARNING: 'warning'
         }.freeze
 
         def initialize(driver, browsing_context_ids = nil)
           unless driver.capabilities.web_socket_url
             raise Error::WebDriverError,
-                  "WebDriver instance must support BiDi protocol"
+                  'WebDriver instance must support BiDi protocol'
           end
 
           @bidi = driver.bidi
-          @bidi.session.subscribe("log.entryAdded", browsing_context_ids)
+          @bidi.session.subscribe('log.entryAdded', browsing_context_ids)
         end
 
         def on_console_entry(&block)
@@ -55,8 +55,8 @@ def on_console_entry(&block)
           return if enabled
 
           on_log do |params|
-            type = params["type"]
-            console_log_events(params) if type.eql?("console")
+            type = params['type']
+            console_log_events(params) if type.eql?('console')
           end
         end
 
@@ -66,8 +66,8 @@ def on_javascript_log(&block)
           return if enabled
 
           on_log do |params|
-            type = params["type"]
-            javascript_log_events(params) if type.eql?("javascript")
+            type = params['type']
+            javascript_log_events(params) if type.eql?('javascript')
           end
         end
 
@@ -78,10 +78,10 @@ def on_javascript_exception(&block)
           return if enabled
 
           on_log do |params|
-            type = params["type"]
-            level = params["level"]
+            type = params['type']
+            level = params['level']
 
-            javascript_log_events(params) if type.eql?("javascript") && level.eql?(LOG_LEVEL[:ERROR])
+            javascript_log_events(params) if type.eql?('javascript') && level.eql?(LOG_LEVEL[:ERROR])
           end
         end
 

rb/lib/selenium/webdriver/bidi/session.rb

@@ -36,14 +36,14 @@ def subscribe(events, browsing_contexts = nil)
           events_list = Array(events)
           browsing_contexts_list = browsing_contexts.nil? ? nil : Array(browsing_contexts)
 
-          @bidi.send_cmd("session.subscribe", events: events_list, contexts: browsing_contexts_list)
+          @bidi.send_cmd('session.subscribe', events: events_list, contexts: browsing_contexts_list)
         end
 
         def unsubscribe(events, browsing_contexts = nil)
           events_list = Array(events)
           browsing_contexts_list = browsing_contexts.nil? ? nil : Array(browsing_contexts)
 
-          @bidi.send_cmd("session.unsubscribe", events: events_list, contexts: browsing_contexts_list)
+          @bidi.send_cmd('session.unsubscribe', events: events_list, contexts: browsing_contexts_list)
         end
       end # Session
     end # BiDi

rb/lib/selenium/webdriver/chromium/options.rb

@@ -241,7 +241,7 @@ def check_w3c(w3c)
 
           raise Error::InvalidArgumentError,
                 "Setting 'w3c: false' is not allowed.\n" \
-                "Please update to W3C Syntax: https://www.selenium.dev/blog/2022/legacy-protocol-support/"
+                'Please update to W3C Syntax: https://www.selenium.dev/blog/2022/legacy-protocol-support/'
         end
 
         def binary_path

rb/lib/selenium/webdriver/chromium/service.rb

@@ -32,8 +32,8 @@ def extract_service_args(driver_opts)
           if driver_opts.key?(:whitelisted_ips)
             driver_args << "--whitelisted-ips=#{driver_opts.delete(:whitelisted_ips)}"
           end
-          driver_args << "--verbose" if driver_opts.key?(:verbose)
-          driver_args << "--silent" if driver_opts.key?(:silent)
+          driver_args << '--verbose' if driver_opts.key?(:verbose)
+          driver_args << '--silent' if driver_opts.key?(:silent)
           driver_args
         end
       end # Service

rb/lib/selenium/webdriver/common/profile_helper.rb

@@ -40,7 +40,7 @@ def encoded
       end
 
       def as_json(*)
-        {"zip" => encoded}
+        {'zip' => encoded}
       end
 
       def to_json(*)

rb/lib/selenium/webdriver/common/selenium_manager.rb

@@ -27,7 +27,7 @@ module WebDriver
     # @api private
     #
     class SeleniumManager
-      BIN_PATH = "../../../../../bin"
+      BIN_PATH = '../../../../../bin'
 
       class << self
         # @param [String] driver_name which driver to use.
@@ -60,7 +60,7 @@ def binary
                     end
             location = File.expand_path(path, __FILE__)
             unless location.is_a?(String) && File.exist?(location) && File.executable?(location)
-              raise Error::WebDriverError, "Unable to obtain Selenium Manager"
+              raise Error::WebDriverError, 'Unable to obtain Selenium Manager'
             end
 
             WebDriver.logger.debug("Selenium Manager found at #{location}")

rb/lib/selenium/webdriver/common/service.rb

@@ -91,8 +91,8 @@ def shutdown_supported
       protected
 
       def extract_service_args(driver_opts)
-        WebDriver.logger.deprecate("initializing Service class with :args using Hash",
-                                   ":args parameter with an Array of String values",
+        WebDriver.logger.deprecate('initializing Service class with :args using Hash',
+                                   ':args parameter with an Array of String values',
                                    id: :driver_opts)
         driver_opts.key?(:args) ? driver_opts.delete(:args) : []
       end