[rb] update syntax for rubocop

Author: titusfortner

rb/lib/selenium/server.rb

@@ -210,6 +210,7 @@ def stop
       begin
         Net::HTTP.get(@host, '/selenium-server/driver/?cmd=shutDownSeleniumServer', @port)
       rescue Errno::ECONNREFUSED
+        nil
       end
 
       stop_process if @process

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

@@ -34,7 +34,8 @@ class BrowsingContext
 
         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"
+            raise Error::WebDriverError,
+                  "WebDriver instance must support BiDi protocol"
           end
 
           unless type.nil? || %i[window tab].include?(type)

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

@@ -23,8 +23,8 @@ class BiDi
       class ConsoleLogEntry < GenericLogEntry
         attr_accessor :method, :realm, :args
 
-        def initialize(level:, text:, timestamp:, type:, method:, realm:, args:, stack_trace:)
-          super(level: level, text: text, timestamp: timestamp, type: type, stack_trace: stack_trace)
+        def initialize(method:, realm:, args:, **opts)
+          super(**opts)
           @method = method
           @realm = realm
           @args = args

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

@@ -41,7 +41,8 @@ class LogInspector
 
         def initialize(driver, browsing_context_ids = nil)
           unless driver.capabilities.web_socket_url
-            raise Error::WebDriverError, "WebDriver instance must support BiDi protocol"
+            raise Error::WebDriverError,
+                  "WebDriver instance must support BiDi protocol"
           end
 
           @bidi = driver.bidi

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

@@ -255,19 +255,19 @@ def add_virtual_authenticator(options)
       #   driver.first(id: 'foo')
       #
 
-      alias_method :first, :find_element
+      alias first find_element
 
       #
       #   driver.all(class: 'bar') #=> [#<WebDriver::Element:0x1011c3b88, ...]
       #
 
-      alias_method :all, :find_elements
+      alias all find_elements
 
       #
       #   driver.script('function() { ... };')
       #
 
-      alias_method :script, :execute_script
+      alias script execute_script
 
       # Get the first element matching the given selector. If given a
       # String or Symbol, it will be used as the id of the element.

rb/lib/selenium/webdriver/common/driver_extensions/has_location.rb

@@ -29,7 +29,7 @@ def location
         def location=(*)
           raise Error::UnsupportedOperationError, 'The W3C standard does not currently support setting location'
         end
-        alias_method :set_location, :location
+        alias set_location location
       end # HasLocation
     end # DriverExtensions
   end # WebDriver

rb/lib/selenium/webdriver/common/element.rb

@@ -43,7 +43,7 @@ def inspect
       def ==(other)
         other.is_a?(self.class) && ref == other.ref
       end
-      alias_method :eql?, :==
+      alias eql? ==
 
       def hash
         [@id, @bridge].hash
@@ -207,7 +207,7 @@ def text
       def send_keys(*args)
         bridge.send_keys_to_element @id, Keys.encode(args)
       end
-      alias_method :send_key, :send_keys
+      alias send_key send_keys
 
       #
       # If this element is a text entry element, this will clear the value. Has no effect on other
@@ -275,7 +275,7 @@ def submit
       def css_value(prop)
         bridge.element_value_of_css_property @id, prop
       end
-      alias_method :style, :css_value
+      alias style css_value
 
       #
       # Get the location of this element.
@@ -333,18 +333,18 @@ def shadow_root
       #   element.first(id: 'foo')
       #
 
-      alias_method :first, :find_element
+      alias first find_element
 
       #
       #   element.all(class: 'bar')
       #
 
-      alias_method :all, :find_elements
+      alias all find_elements
 
       #
       #   element['class'] or element[:class] #=> "someclass"
       #
-      alias_method :[], :attribute
+      alias [] attribute
 
       #
       # @api private

rb/lib/selenium/webdriver/common/html5/shared_web_storage.rb

@@ -26,8 +26,8 @@ module SharedWebStorage
         def key?(key)
           keys.include? key
         end
-        alias_method :member?, :key?
-        alias_method :has_key?, :key?
+        alias member? key?
+        alias has_key? key?
 
         def fetch(key)
           return self[key] if key? key

rb/lib/selenium/webdriver/common/interactions/pointer_actions.rb

@@ -95,9 +95,9 @@ def pointer_up(button = :left, device: nil, **opts)
       # @return [ActionBuilder] A self reference.
       #
 
-      def move_to(element, right_by = nil, down_by = nil, device: nil, duration: default_move_duration, **opts)
-        pointer = pointer_input(device)
-        pointer.create_pointer_move(duration: duration,
+      def move_to(element, right_by = nil, down_by = nil, **opts)
+        pointer = pointer_input(opts.delete(:device))
+        pointer.create_pointer_move(duration: opts.delete(:duration) || default_move_duration,
                                     x: right_by || 0,
                                     y: down_by || 0,
                                     origin: element,

rb/lib/selenium/webdriver/common/interactions/pointer_move.rb

@@ -33,12 +33,12 @@ class PointerMove < Interaction
         POINTER = :pointer
         ORIGINS = [VIEWPORT, POINTER].freeze
 
-        def initialize(source, duration, x, y, element: nil, origin: nil, **opts)
+        def initialize(source, duration, x, y, **opts)
           super(source)
           @duration = duration * 1000
           @x_offset = x
           @y_offset = y
-          @origin = element || origin || :viewport
+          @origin = opts.delete(:element) || opts.delete(:origin) || :viewport
           @type = :pointerMove
           @opts = opts
         end