selenium-webdriver 3.141.0 → 3.142.7

data/lib/selenium/webdriver/chrome/service.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Licensed to the Software Freedom Conservancy (SFC) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information
@@ -23,28 +25,26 @@ module Selenium
       #
 
       class Service < WebDriver::Service
-        DEFAULT_PORT = 9515
-        @executable = 'chromedriver'.freeze
-        @missing_text = <<-ERROR.gsub(/\n +| {2,}/, ' ').freeze
+        @default_port = 9515
+        @executable = 'chromedriver'
+        @missing_text = <<~ERROR
           Unable to find chromedriver. Please download the server from
-          http://chromedriver.storage.googleapis.com/index.html and place it somewhere on your PATH.
+          https://chromedriver.storage.googleapis.com/index.html and place it somewhere on your PATH.
           More info at https://github.com/SeleniumHQ/selenium/wiki/ChromeDriver.
         ERROR
+        @shutdown_supported = true
 
-        private
-
-        def start_process
-          @process = build_process(@executable_path, "--port=#{@port}", *@extra_args)
-          @process.leader = true unless Platform.windows?
-          @process.start
+        def self.driver_path=(path)
+          Platform.assert_executable path if path.is_a?(String)
+          @driver_path = path
         end
 
-        def cannot_connect_error_text
-          "unable to connect to chromedriver #{@host}:#{@port}"
-        end
+        private
 
+        # Note: This processing is deprecated
         def extract_service_args(driver_opts)
           driver_args = super
+          driver_opts = driver_opts.dup
           driver_args << "--log-path=#{driver_opts.delete(:log_path)}" if driver_opts.key?(:log_path)
           driver_args << "--url-base=#{driver_opts.delete(:url_base)}" if driver_opts.key?(:url_base)
           driver_args << "--port-server=#{driver_opts.delete(:port_server)}" if driver_opts.key?(:port_server)

data/lib/selenium/webdriver/chrome.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Licensed to the Software Freedom Conservancy (SFC) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information
@@ -18,7 +20,6 @@
 require 'net/http'
 
 require 'selenium/webdriver/chrome/bridge'
-require 'selenium/webdriver/chrome/service'
 require 'selenium/webdriver/chrome/driver'
 require 'selenium/webdriver/chrome/profile'
 require 'selenium/webdriver/chrome/options'
@@ -27,12 +28,15 @@ module Selenium
   module WebDriver
     module Chrome
       def self.driver_path=(path)
-        Platform.assert_executable path
-        @driver_path = path
+        WebDriver.logger.deprecate 'Selenium::WebDriver::Chrome#driver_path=',
+                                   'Selenium::WebDriver::Chrome::Service#driver_path='
+        Selenium::WebDriver::Chrome::Service.driver_path = path
       end
 
       def self.driver_path
-        @driver_path ||= nil
+        WebDriver.logger.deprecate 'Selenium::WebDriver::Chrome#driver_path',
+                                   'Selenium::WebDriver::Chrome::Service#driver_path'
+        Selenium::WebDriver::Chrome::Service.driver_path
       end
 
       def self.path=(path)
@@ -46,3 +50,5 @@ module Selenium
     end # Chrome
   end # WebDriver
 end # Selenium
+
+require 'selenium/webdriver/chrome/service'

data/lib/selenium/webdriver/common/action_builder.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Licensed to the Software Freedom Conservancy (SFC) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information

data/lib/selenium/webdriver/common/alert.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Licensed to the Software Freedom Conservancy (SFC) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information

data/lib/selenium/webdriver/common/bridge_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Licensed to the Software Freedom Conservancy (SFC) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information
@@ -31,6 +33,7 @@ module Selenium
         when Hash
           element_id = element_id_from(arg)
           return Element.new(self, element_id) if element_id
+
           arg.each { |k, v| arg[k] = unwrap_script_result(v) }
         else
           arg
@@ -43,12 +46,12 @@ module Selenium
 
       def parse_cookie_string(str)
         result = {
-          'name'    => '',
-          'value'   => '',
-          'domain'  => '',
-          'path'    => '',
+          'name' => '',
+          'value' => '',
+          'domain' => '',
+          'path' => '',
           'expires' => '',
-          'secure'  => false
+          'secure' => false
         }
 
         str.split(';').each do |attribute|

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

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Licensed to the Software Freedom Conservancy (SFC) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information
@@ -73,7 +75,7 @@ module Selenium
       end
 
       def inspect
-        format '#<%s:0x%x browser=%s>', self.class, hash * 2, bridge.browser.inspect
+        format '#<%<class>s:0x%<hash>x browser=%<browser>s>', class: self.class, hash: hash * 2, browser: bridge.browser.inspect
       end
 
       #
@@ -95,12 +97,12 @@ module Selenium
       end
 
       #
-      # @return [Options]
-      # @see Options
+      # @return [Manager]
+      # @see Manager
       #
 
       def manage
-        bridge.options
+        bridge.manage
       end
 
       #
@@ -281,13 +283,26 @@ module Selenium
       # @see SearchContext
       #
 
-      def ref
-        nil
-      end
+      def ref; end
 
       private
 
       attr_reader :bridge
+
+      def service_url(opts)
+        @service = opts.delete(:service)
+        %i[driver_opts driver_path port].each do |key|
+          next unless opts.key? key
+
+          WebDriver.logger.deprecate(":#{key}", ':service with an instance of Selenium::WebDriver::Service')
+        end
+        @service ||= Service.send(browser,
+                                  args: opts.delete(:driver_opts),
+                                  path: opts.delete(:driver_path),
+                                  port: opts.delete(:port))
+        @service.start
+        @service.uri
+      end
     end # Driver
   end # WebDriver
 end # Selenium

data/lib/selenium/webdriver/common/driver_extensions/downloads_files.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Licensed to the Software Freedom Conservancy (SFC) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information

data/lib/selenium/webdriver/common/driver_extensions/has_addons.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Licensed to the Software Freedom Conservancy (SFC) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information

data/lib/selenium/webdriver/common/driver_extensions/has_debugger.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Licensed to the Software Freedom Conservancy (SFC) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information

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

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Licensed to the Software Freedom Conservancy (SFC) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information
@@ -24,9 +26,7 @@ module Selenium
         end
 
         def location=(loc)
-          unless loc.is_a?(Location)
-            raise TypeError, "expected #{Location}, got #{loc.inspect}:#{loc.class}"
-          end
+          raise TypeError, "expected #{Location}, got #{loc.inspect}:#{loc.class}" unless loc.is_a?(Location)
 
           @bridge.set_location loc.latitude, loc.longitude, loc.altitude
         end

data/lib/selenium/webdriver/common/driver_extensions/has_network_conditions.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Licensed to the Software Freedom Conservancy (SFC) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information

data/lib/selenium/webdriver/common/driver_extensions/has_network_connection.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Licensed to the Software Freedom Conservancy (SFC) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information
@@ -48,7 +50,7 @@ module Selenium
         end
 
         def valid_type?(type)
-          type_to_values.keys.include? type
+          type_to_values.key? type
         end
       end # HasNetworkConnection
     end # DriverExtensions

data/lib/selenium/webdriver/common/driver_extensions/has_permissions.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Licensed to the Software Freedom Conservancy (SFC) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information

data/lib/selenium/webdriver/common/driver_extensions/has_remote_status.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Licensed to the Software Freedom Conservancy (SFC) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information

data/lib/selenium/webdriver/common/driver_extensions/has_session_id.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Licensed to the Software Freedom Conservancy (SFC) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information

data/lib/selenium/webdriver/common/driver_extensions/has_touch_screen.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Licensed to the Software Freedom Conservancy (SFC) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information

data/lib/selenium/webdriver/common/driver_extensions/has_web_storage.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Licensed to the Software Freedom Conservancy (SFC) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information

data/lib/selenium/webdriver/common/driver_extensions/rotatable.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Licensed to the Software Freedom Conservancy (SFC) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information
@@ -23,7 +25,7 @@ module Selenium
 
     module DriverExtensions
       module Rotatable
-        ORIENTATIONS = [:landscape, :portrait].freeze
+        ORIENTATIONS = %i[landscape portrait].freeze
 
         #
         # Change the screen orientation

data/lib/selenium/webdriver/common/driver_extensions/takes_screenshot.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Licensed to the Software Freedom Conservancy (SFC) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information

data/lib/selenium/webdriver/common/driver_extensions/uploads_files.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Licensed to the Software Freedom Conservancy (SFC) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information
@@ -48,9 +50,7 @@ module Selenium
         #
 
         def file_detector=(detector)
-          unless detector.nil? || detector.respond_to?(:call)
-            raise ArgumentError, 'detector must respond to #call'
-          end
+          raise ArgumentError, 'detector must respond to #call' unless detector.nil? || detector.respond_to?(:call)
 
           bridge.file_detector = detector
         end

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

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Licensed to the Software Freedom Conservancy (SFC) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information
@@ -32,7 +34,7 @@ module Selenium
       end
 
       def inspect
-        format '#<%s:0x%x id=%s>', self.class, hash * 2, @id.inspect
+        format '#<%<class>s:0x%<hash>x id=%<id>s>', class: self.class, hash: hash * 2, id: @id.inspect
       end
 
       def ==(other)

data/lib/selenium/webdriver/common/error.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Licensed to the Software Freedom Conservancy (SFC) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information
@@ -17,7 +19,7 @@
 
 module Selenium
   module WebDriver
-    module Error
+    module Error # rubocop:disable Metrics/ModuleLength
 
       #
       # Returns exception from code (Integer - OSS, String - W3C).
@@ -29,7 +31,7 @@ module Selenium
         when nil, 0
           nil
         when Integer
-          ERRORS.fetch(code)
+          Object.const_get(ERRORS.fetch(code).to_s)
         when String
           klass_name = code.split(' ').map(&:capitalize).join.sub(/Error$/, '')
           const_get("#{klass_name}Error", false)
@@ -210,6 +212,13 @@ module Selenium
 
       class ElementNotInteractableError < WebDriverError; end
 
+      #
+      # A command could not be completed because TLS certificate is expired
+      # or invalid.
+      #
+
+      class InsecureCertificateError < WebDriverError; end
+
       #
       # The arguments passed to a command are either invalid or malformed.
       #
@@ -263,30 +272,30 @@ module Selenium
       class UnsupportedOperationError < WebDriverError; end
 
       # Aliases for OSS dialect.
-      ScriptTimeoutError  = ScriptTimeOutError
-      TimeoutError        = TimeOutError
-      NoAlertOpenError    = NoAlertPresentError
+      ScriptTimeoutError  = Class.new(ScriptTimeOutError)
+      TimeoutError        = Class.new(TimeOutError)
+      NoAlertOpenError    = Class.new(NoAlertPresentError)
 
       # Aliases for backwards compatibility.
-      ObsoleteElementError      = StaleElementReferenceError
-      UnhandledError            = UnknownError
-      UnexpectedJavascriptError = JavascriptError
-      ElementNotDisplayedError  = ElementNotVisibleError
+      ObsoleteElementError      = Class.new(StaleElementReferenceError)
+      UnhandledError            = Class.new(UnknownError)
+      UnexpectedJavascriptError = Class.new(JavascriptError)
+      ElementNotDisplayedError  = Class.new(ElementNotVisibleError)
 
       #
       # @api private
       #
 
       ERRORS = {
-        1  => IndexOutOfBoundsError,
-        2  => NoCollectionError,
-        3  => NoStringError,
-        4  => NoStringLengthError,
-        5  => NoStringWrapperError,
-        6  => NoSuchDriverError,
-        7  => NoSuchElementError,
-        8  => NoSuchFrameError,
-        9  => UnknownCommandError,
+        1 => IndexOutOfBoundsError,
+        2 => NoCollectionError,
+        3 => NoStringError,
+        4 => NoStringLengthError,
+        5 => NoStringWrapperError,
+        6 => NoSuchDriverError,
+        7 => NoSuchElementError,
+        8 => NoSuchFrameError,
+        9 => UnknownCommandError,
         10 => StaleElementReferenceError,
         11 => ElementNotVisibleError,
         12 => InvalidElementStateError,
@@ -322,6 +331,53 @@ module Selenium
         63 => UnableToCaptureScreenError
       }.freeze
 
+      DEPRECATED_ERRORS = {
+        IndexOutOfBoundsError: nil,
+        NoCollectionError: nil,
+        NoStringError: nil,
+        NoStringLengthError: nil,
+        NoStringWrapperError: nil,
+        NoSuchDriverError: nil,
+        ElementNotVisibleError: ElementNotInteractableError,
+        InvalidElementStateError: ElementNotInteractableError,
+        ElementNotSelectableError: ElementNotInteractableError,
+        ExpectedError: nil,
+        NoSuchDocumentError: nil,
+        NoScriptResultError: nil,
+        XPathLookupError: InvalidSelectorError,
+        NoSuchCollectionError: nil,
+        UnhandledAlertError: UnexpectedAlertOpenError,
+        NoAlertPresentError: NoSuchAlertError,
+        NoAlertOpenError: NoSuchAlertError,
+        ScriptTimeOutError: ScriptTimeoutError,
+        InvalidElementCoordinatesError: nil,
+        IMENotAvailableError: nil,
+        IMEEngineActivationFailedError: nil,
+        InvalidXpathSelectorError: InvalidSelectorError,
+        InvalidXpathSelectorReturnTyperError: InvalidSelectorError,
+        TimeOutError: TimeoutError,
+        ObsoleteElementError: StaleElementReferenceError,
+        UnhandledError: UnknownError,
+        UnexpectedJavascriptError: JavascriptError,
+        ElementNotDisplayedError: ElementNotInteractableError
+      }.freeze
+
+      DEPRECATED_ERRORS.keys.each do |oss_error|
+        remove_const oss_error
+      end
+
+      def self.const_missing(const_name)
+        super unless DEPRECATED_ERRORS.key?(const_name)
+        if DEPRECATED_ERRORS[const_name]
+          WebDriver.logger.deprecate("Selenium::WebDriver::Error::#{const_name}",
+                                     "#{DEPRECATED_ERRORS[const_name]} (ensure the driver supports W3C WebDriver specification)")
+          DEPRECATED_ERRORS[const_name]
+        else
+          WebDriver.logger.deprecate("Selenium::WebDriver::Error::#{const_name}")
+          WebDriverError
+        end
+      end
+
     end # Error
   end # WebDriver
 end # Selenium

data/lib/selenium/webdriver/common/file_reaper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Licensed to the Software Freedom Conservancy (SFC) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information
@@ -41,9 +43,7 @@ module Selenium
         def reap(file)
           return unless reap?
 
-          unless tmp_files.include?(file)
-            raise Error::WebDriverError, "file not added for reaping: #{file.inspect}"
-          end
+          raise Error::WebDriverError, "file not added for reaping: #{file.inspect}" unless tmp_files.include?(file)
 
           FileUtils.rm_rf tmp_files.delete(file)
         end

data/lib/selenium/webdriver/common/html5/local_storage.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Licensed to the Software Freedom Conservancy (SFC) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information

data/lib/selenium/webdriver/common/html5/session_storage.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Licensed to the Software Freedom Conservancy (SFC) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information

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

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Licensed to the Software Freedom Conservancy (SFC) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information
@@ -30,6 +32,7 @@ module Selenium
         def fetch(key)
           return self[key] if key? key
           return yield(key) if block_given?
+
           raise KeyError, "missing key #{key.inspect}"
         end
 
@@ -40,7 +43,7 @@ module Selenium
         def each
           return enum_for(:each) unless block_given?
 
-          keys.each do |k| # rubocop:disable Performance/HashEachMethods
+          keys.each do |k|
             yield k, self[k]
           end
         end

data/lib/selenium/webdriver/common/interactions/input_device.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Licensed to the Software Freedom Conservancy (SFC) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information
@@ -30,6 +32,7 @@ module Selenium
 
         def add_action(action)
           raise TypeError, "#{action.inspect} is not a valid action" unless action.class < Interaction
+
           @actions << action
         end
 

data/lib/selenium/webdriver/common/interactions/interaction.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Licensed to the Software Freedom Conservancy (SFC) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information
@@ -25,6 +27,7 @@ module Selenium
 
         def initialize(source)
           raise TypeError, "#{source.type} is not a valid input type" unless Interactions::SOURCE_TYPES.include? source.type
+
           @source = source
         end
       end

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

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Licensed to the Software Freedom Conservancy (SFC) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information
@@ -13,7 +15,7 @@
 # "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
 # KIND, either express or implied.  See the License for the
 # specific language governing permissions and limitations
-# under the License
+# under the License.
 
 module Selenium
   module WebDriver

data/lib/selenium/webdriver/common/interactions/key_actions.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Licensed to the Software Freedom Conservancy (SFC) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information

data/lib/selenium/webdriver/common/interactions/key_input.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Licensed to the Software Freedom Conservancy (SFC) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information
@@ -27,6 +29,7 @@ module Selenium
 
         def encode
           return nil if no_actions?
+
           {type: type, id: name, actions: @actions.map(&:encode)}
         end
 
@@ -49,6 +52,7 @@ module Selenium
 
           def assert_type(type)
             raise TypeError, "#{type.inspect} is not a valid key subtype" unless KeyInput::SUBTYPES.key? type
+
             KeyInput::SUBTYPES[type]
           end
 

data/lib/selenium/webdriver/common/interactions/none_input.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Licensed to the Software Freedom Conservancy (SFC) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information
@@ -25,6 +27,7 @@ module Selenium
 
         def encode
           return nil if no_actions?
+
           {type: type, id: name, actions: @actions.map(&:encode)}
         end
       end # NoneInput

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

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Licensed to the Software Freedom Conservancy (SFC) under one
 # or more contributor license agreements.  See the NOTICE file
 # distributed with this work for additional information