rb - make all the spacing consistent

Author: titusfortner

rb/.rubocop_todo.yml

@@ -6,11 +6,6 @@
 # Note that changes in the inspected code, or installation of new
 # versions of RuboCop, may require this file to be generated again.
 
-# Offense count: 2
-Lint/AmbiguousRegexpLiteral:
-  Exclude:
-    - 'spec/integration/selenium/webdriver/firefox/marionette_spec.rb'
-
 # Offense count: 2
 # Cop supports --auto-correct.
 Lint/DeprecatedClassMethods:
@@ -69,7 +64,7 @@ Metrics/CyclomaticComplexity:
 # Configuration parameters: AllowHeredoc, AllowURI, URISchemes.
 # URISchemes: http, https
 Metrics/LineLength:
-  Max: 318
+  Max: 319
 
 # Offense count: 61
 # Configuration parameters: CountComments.
@@ -103,17 +98,6 @@ Style/Alias:
     - 'lib/selenium/webdriver/support/color.rb'
     - 'spec/integration/selenium/webdriver/spec_support/guards.rb'
 
-# Offense count: 6
-# Cop supports --auto-correct.
-# Configuration parameters: EnforcedStyle, SupportedStyles.
-# SupportedStyles: with_first_parameter, with_fixed_indentation
-Style/AlignParameters:
-  Exclude:
-    - 'lib/selenium/webdriver/remote/bridge.rb'
-    - 'spec/integration/selenium/webdriver/element_spec.rb'
-    - 'spec/integration/selenium/webdriver/spec_support/test_environment.rb'
-    - 'spec/unit/selenium/webdriver/remote/http/common_spec.rb'
-
 # Offense count: 3
 Style/AsciiComments:
   Exclude:
@@ -145,13 +129,6 @@ Style/ClassAndModuleChildren:
 Style/ClassCheck:
   Enabled: false
 
-# Offense count: 3
-# Cop supports --auto-correct.
-Style/ClosingParenthesisIndentation:
-  Exclude:
-    - 'spec/integration/selenium/webdriver/spec_support/test_environment.rb'
-    - 'spec/integration/selenium/webdriver/storage_spec.rb'
-
 # Offense count: 3
 # Cop supports --auto-correct.
 # Configuration parameters: Keywords.
@@ -161,14 +138,6 @@ Style/CommentAnnotation:
     - 'lib/selenium/webdriver/remote/w3c_bridge.rb'
     - 'spec/integration/selenium/webdriver/window_spec.rb'
 
-# Offense count: 3
-# Cop supports --auto-correct.
-Style/CommentIndentation:
-  Exclude:
-    - 'lib/selenium/webdriver/edge/legacy_support.rb'
-    - 'spec/integration/selenium/webdriver/spec_support/test_environment.rb'
-    - 'spec/integration/selenium/webdriver/target_locator_spec.rb'
-
 # Offense count: 1
 Style/ConstantName:
   Exclude:
@@ -198,67 +167,6 @@ Style/Documentation:
 Style/DotPosition:
   Enabled: false
 
-# Offense count: 17
-# Cop supports --auto-correct.
-# Configuration parameters: AllowAdjacentOneLineDefs.
-Style/EmptyLineBetweenDefs:
-  Exclude:
-    - 'lib/selenium/webdriver/support/abstract_event_listener.rb'
-
-# Offense count: 19
-# Cop supports --auto-correct.
-Style/EmptyLines:
-  Enabled: false
-
-# Offense count: 43
-# Cop supports --auto-correct.
-# Configuration parameters: EnforcedStyle, SupportedStyles.
-# SupportedStyles: empty_lines, no_empty_lines
-Style/EmptyLinesAroundBlockBody:
-  Enabled: false
-
-# Offense count: 102
-# Cop supports --auto-correct.
-# Configuration parameters: EnforcedStyle, SupportedStyles.
-# SupportedStyles: empty_lines, no_empty_lines
-Style/EmptyLinesAroundClassBody:
-  Enabled: false
-
-# Offense count: 3
-# Cop supports --auto-correct.
-Style/EmptyLinesAroundMethodBody:
-  Exclude:
-    - 'lib/selenium/webdriver/common/touch_screen.rb'
-    - 'lib/selenium/webdriver/remote/bridge.rb'
-    - 'lib/selenium/webdriver/remote/w3c_bridge.rb'
-
-# Offense count: 151
-# Cop supports --auto-correct.
-# Configuration parameters: EnforcedStyle, SupportedStyles.
-# SupportedStyles: empty_lines, no_empty_lines
-Style/EmptyLinesAroundModuleBody:
-  Enabled: false
-
-# Offense count: 29
-# Cop supports --auto-correct.
-# Configuration parameters: AllowForAlignment, ForceEqualSignAlignment.
-Style/ExtraSpacing:
-  Exclude:
-    - 'lib/selenium/webdriver/chrome/profile.rb'
-    - 'lib/selenium/webdriver/common/action_builder.rb'
-    - 'lib/selenium/webdriver/common/error.rb'
-    - 'lib/selenium/webdriver/common/log_entry.rb'
-    - 'lib/selenium/webdriver/common/mouse.rb'
-    - 'lib/selenium/webdriver/common/target_locator.rb'
-    - 'lib/selenium/webdriver/firefox/profile.rb'
-    - 'spec/integration/selenium/webdriver/firefox/marionette_spec.rb'
-    - 'spec/unit/selenium/webdriver/action_builder_spec.rb'
-    - 'spec/unit/selenium/webdriver/chrome/bridge_spec.rb'
-    - 'spec/unit/selenium/webdriver/firefox/bridge_spec.rb'
-    - 'spec/unit/selenium/webdriver/ie/bridge_spec.rb'
-    - 'spec/unit/selenium/webdriver/search_context_spec.rb'
-    - 'spec/unit/selenium/webdriver/socket_poller_spec.rb'
-
 # Offense count: 1
 # Configuration parameters: ExpectMatchingDefinition, Regex, IgnoreExecutableScripts.
 Style/FileName:
@@ -283,13 +191,6 @@ Style/FormatString:
     - 'lib/selenium/webdriver/safari/server.rb'
     - 'lib/selenium/webdriver/support/color.rb'
 
-# Offense count: 1
-# Cop supports --auto-correct.
-# Configuration parameters: EnforcedStyle, SupportedStyles, IndentationWidth.
-# SupportedStyles: special_inside_parentheses, consistent, align_brackets
-Style/IndentArray:
-  Enabled: false
-
 # Offense count: 1
 # Cop supports --auto-correct.
 # Configuration parameters: EnforcedStyle, SupportedStyles.
@@ -298,21 +199,6 @@ Style/IndentationConsistency:
   Exclude:
     - 'spec/unit/selenium/webdriver/remote/http/default_spec.rb'
 
-# Offense count: 8
-# Cop supports --auto-correct.
-# Configuration parameters: Width.
-Style/IndentationWidth:
-  Exclude:
-    - 'lib/selenium/webdriver/remote/w3c_bridge.rb'
-    - 'lib/selenium/webdriver/support/color.rb'
-    - 'spec/integration/selenium/webdriver/spec_support/guards.rb'
-
-# Offense count: 1
-# Cop supports --auto-correct.
-Style/LeadingCommentSpace:
-  Exclude:
-    - 'lib/selenium/webdriver/common/driver_extensions/has_location.rb'
-
 # Offense count: 6
 # Cop supports --auto-correct.
 Style/LineEndConcatenation:
@@ -334,13 +220,6 @@ Style/MethodCallParentheses:
 Style/MethodName:
   Enabled: false
 
-# Offense count: 52
-# Cop supports --auto-correct.
-# Configuration parameters: EnforcedStyle, SupportedStyles, IndentationWidth.
-# SupportedStyles: aligned, indented
-Style/MultilineMethodCallIndentation:
-  Enabled: false
-
 # Offense count: 3
 # Cop supports --auto-correct.
 # Configuration parameters: EnforcedStyle, SupportedStyles, IndentationWidth.
@@ -468,49 +347,6 @@ Style/SingleLineMethods:
     - 'spec/unit/selenium/webdriver/firefox/extension_spec.rb'
     - 'spec/unit/selenium/webdriver/wait_spec.rb'
 
-# Offense count: 5
-# Cop supports --auto-correct.
-Style/SpaceAfterComma:
-  Exclude:
-    - 'lib/selenium/webdriver/common/error.rb'
-    - 'lib/selenium/webdriver/common/platform.rb'
-    - 'lib/selenium/webdriver/common/touch_action_builder.rb'
-    - 'spec/unit/selenium/webdriver/spec_helper.rb'
-
-# Offense count: 9
-# Cop supports --auto-correct.
-# Configuration parameters: AllowForAlignment.
-Style/SpaceAroundOperators:
-  Exclude:
-    - 'lib/selenium/webdriver/common/action_builder.rb'
-    - 'lib/selenium/webdriver/common/driver.rb'
-    - 'lib/selenium/webdriver/common/element.rb'
-    - 'lib/selenium/webdriver/common/log_entry.rb'
-    - 'lib/selenium/webdriver/firefox/profile.rb'
-    - 'lib/selenium/webdriver/remote/http/curb.rb'
-    - 'spec/unit/selenium/server_spec.rb'
-
-# Offense count: 5
-# Cop supports --auto-correct.
-Style/SpaceBeforeComma:
-  Exclude:
-    - 'lib/selenium/webdriver/remote/capabilities.rb'
-    - 'spec/unit/selenium/webdriver/safari/bridge_spec.rb'
-
-# Offense count: 1
-# Cop supports --auto-correct.
-Style/SpaceInsideParens:
-  Exclude:
-    - 'spec/unit/selenium/webdriver/action_builder_spec.rb'
-
-# Offense count: 1
-# Cop supports --auto-correct.
-# Configuration parameters: EnforcedStyle, SupportedStyles.
-# SupportedStyles: space, no_space
-Style/SpaceInsideStringInterpolation:
-  Exclude:
-    - 'lib/selenium/webdriver/remote/response.rb'
-
 # Offense count: 2
 # Cop supports --auto-correct.
 # Configuration parameters: SupportedStyles.
@@ -532,13 +368,6 @@ Style/StringLiterals:
 Style/StringLiteralsInInterpolation:
   Enabled: false
 
-# Offense count: 22
-# Cop supports --auto-correct.
-# Configuration parameters: EnforcedStyle, SupportedStyles.
-# SupportedStyles: final_newline, final_blank_line
-Style/TrailingBlankLines:
-  Enabled: false
-
 # Offense count: 9
 # Cop supports --auto-correct.
 # Configuration parameters: EnforcedStyleForMultiline, SupportedStyles.
@@ -554,15 +383,6 @@ Style/TrailingCommaInLiteral:
     - 'spec/unit/selenium/webdriver/proxy_spec.rb'
     - 'spec/unit/selenium/webdriver/safari/bridge_spec.rb'
 
-# Offense count: 5
-# Cop supports --auto-correct.
-Style/TrailingWhitespace:
-  Exclude:
-    - 'lib/selenium/webdriver/common/alert.rb'
-    - 'lib/selenium/webdriver/remote/bridge.rb'
-    - 'lib/selenium/webdriver/remote/commands.rb'
-    - 'spec/unit/selenium/webdriver/support/select_spec.rb'
-
 # Offense count: 5
 # Cop supports --auto-correct.
 # Configuration parameters: ExactNameMatch, AllowPredicates, AllowDSLWriters, IgnoreClassMethods, Whitelist.

rb/lib/selenium/server.rb

@@ -22,7 +22,6 @@
 require 'net/http'
 
 module Selenium
-
   #
   # Wraps the remote server jar
   #
@@ -262,6 +261,5 @@ def poll_for_shutdown
     def socket
       @socket ||= WebDriver::SocketPoller.new(@host, @port, @timeout)
     end
-
   end # Server
 end # Selenium

rb/lib/selenium/webdriver.rb

@@ -83,6 +83,5 @@ def self.root
     def self.for(*args)
       WebDriver::Driver.for(*args)
     end
-
   end # WebDriver
 end # Selenium

rb/lib/selenium/webdriver/android.rb

@@ -20,9 +20,8 @@
 module Selenium
   module WebDriver
     module Android
-
     end # Android
   end # WebDriver
 end # Selenium
 
-require 'selenium/webdriver/android/bridge'
+require 'selenium/webdriver/android/bridge'

rb/lib/selenium/webdriver/android/bridge.rb

@@ -20,13 +20,11 @@
 module Selenium
   module WebDriver
     module Android
-
       #
       # @api private
       #
 
       class Bridge < Remote::Bridge
-
         DEFAULT_URL = "http://#{Platform.localhost}:8080/wd/hub/"
 
         def initialize(opts = {})
@@ -61,7 +59,6 @@ def driver_extensions
         def capabilities
           @capabilities ||= Remote::Capabilities.android
         end
-
       end # Bridge
     end # Android
   end # WebDriver

rb/lib/selenium/webdriver/chrome.rb

@@ -51,7 +51,6 @@ def self.path=(path)
       def self.path
         @path ||= nil
       end
-
     end # Chrome
   end # WebDriver
 end # Selenium

rb/lib/selenium/webdriver/chrome/bridge.rb

@@ -20,10 +20,8 @@
 module Selenium
   module WebDriver
     module Chrome
-
       # @api private
       class Bridge < Remote::Bridge
-
         def initialize(opts = {})
           http_client = opts.delete(:http_client)
 
@@ -132,7 +130,6 @@ def extract_service_args(opts)
 
           args
         end
-
       end # Bridge
     end # Chrome
   end # WebDriver

rb/lib/selenium/webdriver/chrome/profile.rb

@@ -20,7 +20,6 @@
 module Selenium
   module WebDriver
     module Chrome
-
       #
       # @private
       #
@@ -87,7 +86,7 @@ def write_prefs_to(dir)
           prefs_file = prefs_file_for(dir)
 
           FileUtils.mkdir_p File.dirname(prefs_file)
-          File.open(prefs_file, "w") { |file| file << JSON.generate(prefs)  }
+          File.open(prefs_file, "w") { |file| file << JSON.generate(prefs) }
         end
 
         def prefs
@@ -103,7 +102,6 @@ def prefs_file_for(dir)
           File.join dir, 'Default', 'Preferences'
         end
       end # Profile
-
     end # Chrome
   end # WebDriver
 end # Selenium

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

@@ -20,7 +20,6 @@
 module Selenium
   module WebDriver
     module Chrome
-
       #
       # @api private
       #
@@ -45,7 +44,6 @@ def stop_server
         def cannot_connect_error_text
           "unable to connect to chromedriver #{@host}:#{@port}"
         end
-
       end # Service
     end # Chrome
   end # WebDriver