Add connection error handling method to reduce ABC-Metric of the request method

Author: rgisiger

rb/lib/selenium/webdriver/remote/http/default.rb

@@ -88,9 +88,7 @@ def request(verb, url, headers, payload, redirects = 0)
               sleep 2
               retry
             rescue Errno::ECONNREFUSED => e
-              raise e.class, "using proxy: #{http.proxy_uri}" if http.proxy?
-
-              raise
+              handle_connection_refused(e)
             end
 
             if response.is_a? Net::HTTPRedirection
@@ -104,6 +102,12 @@ def request(verb, url, headers, payload, redirects = 0)
             end
           end
 
+          def handle_connection_refused(error)
+            raise error.class, "using proxy: #{http.proxy_uri}" if http.proxy?
+
+            raise
+          end
+
           def new_request_for(verb, url, headers, payload)
             req = Net::HTTP.const_get(verb.to_s.capitalize).new(url.path, headers)
 

rb/sig/lib/selenium/webdriver/remote/http/default.rbs

@@ -32,6 +32,8 @@ module Selenium
 
           def request: (untyped verb, untyped url, untyped headers, untyped payload, ?::Integer redirects) -> untyped
 
+          def handle_connection_refused: (Errno::ECONNREFUSED exception) -> untyped
+
           def new_request_for: (untyped verb, untyped url, untyped headers, untyped payload) -> untyped
 
           def response_for: (untyped request) -> untyped