rb - update string literals

Author: titusfortner

rb/.rubocop_todo.yml

@@ -1,6 +1,6 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2016-06-10 10:26:32 -0700 using RuboCop version 0.37.2.
+# on 2016-06-10 14:40:50 -0700 using RuboCop version 0.37.2.
 # The point is for the user to remove these configuration records
 # one by one as the offenses are removed from the code base.
 # Note that changes in the inspected code, or installation of new
@@ -38,10 +38,3 @@ Metrics/ModuleLength:
 # Offense count: 15
 Metrics/PerceivedComplexity:
   Max: 16
-
-# Offense count: 1398
-# Cop supports --auto-correct.
-# Configuration parameters: EnforcedStyle, SupportedStyles, ConsistentQuotesInMultiline.
-# SupportedStyles: single_quotes, double_quotes
-Style/StringLiterals:
-  Enabled: false

rb/lib/selenium/server.rb

@@ -73,8 +73,8 @@ def download(required_version)
         return download_file_name if File.exist? download_file_name
 
         begin
-          open(download_file_name, "wb") do |destination|
-            net_http.start("selenium-release.storage.googleapis.com") do |http|
+          open(download_file_name, 'wb') do |destination|
+            net_http.start('selenium-release.storage.googleapis.com') do |http|
               resp = http.request_get("/#{required_version[/(\d+\.\d+)\./, 1]}/#{download_file_name}") do |response|
                 total = response.content_length
                 progress = 0
@@ -113,8 +113,8 @@ def download(required_version)
 
       def latest
         require 'rexml/document'
-        net_http.start("selenium-release.storage.googleapis.com") do |http|
-          REXML::Document.new(http.get("/").body).root.get_elements("//Contents/Key").map do |e|
+        net_http.start('selenium-release.storage.googleapis.com') do |http|
+          REXML::Document.new(http.get('/').body).root.get_elements('//Contents/Key').map do |e|
             e.text[/selenium-server-standalone-(\d+\.\d+\.\d+)\.jar/, 1]
           end.compact.max
         end
@@ -124,7 +124,7 @@ def net_http
         http_proxy = ENV['http_proxy'] || ENV['HTTP_PROXY']
 
         if http_proxy
-          http_proxy = "http://#{http_proxy}" unless http_proxy.start_with?("http://")
+          http_proxy = "http://#{http_proxy}" unless http_proxy.start_with?('http://')
           uri = URI.parse(http_proxy)
 
           Net::HTTP::Proxy(uri.host, uri.port)
@@ -174,7 +174,7 @@ def initialize(jar, opts = {})
       raise Errno::ENOENT, jar unless File.exist?(jar)
 
       @jar        = jar
-      @host       = "127.0.0.1"
+      @host       = '127.0.0.1'
       @port       = opts.fetch(:port, 4444)
       @timeout    = opts.fetch(:timeout, 30)
       @background = opts.fetch(:background, false)
@@ -192,7 +192,7 @@ def start
 
     def stop
       begin
-        Net::HTTP.get(@host, "/selenium-server/driver/?cmd=shutDownSeleniumServer", @port)
+        Net::HTTP.get(@host, '/selenium-server/driver/?cmd=shutDownSeleniumServer', @port)
       rescue Errno::ECONNREFUSED
       end
 
@@ -232,11 +232,11 @@ def stop_process
 
     def process
       @process ||= (
-        cp = ChildProcess.build("java", "-jar", @jar, "-port", @port.to_s, *@additional_args)
+        cp = ChildProcess.build('java', '-jar', @jar, '-port', @port.to_s, *@additional_args)
         io = cp.io
 
         if @log.is_a?(String)
-          @log_file = File.open(@log, "w")
+          @log_file = File.open(@log, 'w')
           io.stdout = io.stderr = @log_file
         elsif @log
           io.inherit!

rb/lib/selenium/webdriver.rb

@@ -45,7 +45,7 @@ module WebDriver
     # @api private
 
     def self.root
-      @root ||= File.expand_path("../..", __FILE__)
+      @root ||= File.expand_path('../..', __FILE__)
     end
 
     #

rb/lib/selenium/webdriver/chrome.rb

@@ -26,7 +26,7 @@
 module Selenium
   module WebDriver
     module Chrome
-      MISSING_TEXT = "Unable to find the chromedriver executable. Please download the server from http://chromedriver.storage.googleapis.com/index.html and place it somewhere on your PATH. More info at https://github.com/SeleniumHQ/selenium/wiki/ChromeDriver.".freeze
+      MISSING_TEXT = 'Unable to find the chromedriver executable. Please download the server from http://chromedriver.storage.googleapis.com/index.html and place it somewhere on your PATH. More info at https://github.com/SeleniumHQ/selenium/wiki/ChromeDriver.'.freeze
 
       def self.driver_path=(path)
         Platform.assert_executable path
@@ -35,7 +35,7 @@ def self.driver_path=(path)
 
       def self.driver_path
         @driver_path ||= begin
-          path = Platform.find_binary("chromedriver")
+          path = Platform.find_binary('chromedriver')
           raise Error::WebDriverError, MISSING_TEXT unless path
           Platform.assert_executable path
 

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

@@ -89,7 +89,7 @@ def create_capabilities(opts)
 
           if args
             unless args.is_a? Array
-              raise ArgumentError, ":args must be an Array of Strings"
+              raise ArgumentError, ':args must be an Array of Strings'
             end
 
             chrome_options['args'] = args.map(&:to_s)
@@ -113,7 +113,7 @@ def create_capabilities(opts)
           caps['proxy'] = proxy if proxy
 
           # legacy options - for chromedriver < 17.0.963.0
-          caps["chrome.switches"] = chrome_options['args'] if chrome_options.member?('args')
+          caps['chrome.switches'] = chrome_options['args'] if chrome_options.member?('args')
           %w[binary detach extensions nativeEvents noWebsiteTestingDefaults prefs profile verbose].each do |key|
             caps["chrome.#{key}"] = chrome_options[key] if chrome_options.member?(key)
           end

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

@@ -52,17 +52,17 @@ def add_encoded_extension(encoded)
         #
 
         def []=(key, value)
-          parts = key.split(".")
+          parts = key.split('.')
           parts[0..-2].inject(prefs) { |a, e| a[e] ||= {} }[parts.last] = value
         end
 
         def [](key)
-          parts = key.split(".")
+          parts = key.split('.')
           parts.inject(prefs) { |a, e| a.fetch(e) }
         end
 
         def layout_on_disk
-          dir = @model ? create_tmp_copy(@model) : Dir.mktmpdir("webdriver-chrome-profile")
+          dir = @model ? create_tmp_copy(@model) : Dir.mktmpdir('webdriver-chrome-profile')
           FileReaper << dir
 
           write_prefs_to dir
@@ -72,7 +72,7 @@ def layout_on_disk
 
         def as_json(opts = nil)
           extensions = @extensions.map do |crx_path|
-            File.open(crx_path, "rb") { |crx_file| Base64.strict_encode64 crx_file.read }
+            File.open(crx_path, 'rb') { |crx_file| Base64.strict_encode64 crx_file.read }
           end
 
           extensions.concat(@encoded_extensions)
@@ -86,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

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

@@ -38,7 +38,7 @@ def start_process
         end
 
         def stop_server
-          connect_to_server { |http| http.get("/shutdown") }
+          connect_to_server { |http| http.get('/shutdown') }
         end
 
         def cannot_connect_error_text

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

@@ -53,9 +53,9 @@ def parse_cookie_string(str)
           'secure'  => false
         }
 
-        str.split(";").each do |attribute|
-          if attribute.include? "="
-            key, value = attribute.strip.split("=", 2)
+        str.split(';').each do |attribute|
+          if attribute.include? '='
+            key, value = attribute.strip.split('=', 2)
             if result['name'].empty?
               result['name']  = key
               result['value'] = value
@@ -64,11 +64,11 @@ def parse_cookie_string(str)
             elsif key && value
               result[key] = value
             end
-          elsif attribute == "secure"
+          elsif attribute == 'secure'
             result['secure'] = true
           end
 
-          unless [nil, "", "0"].include?(result['expires'])
+          unless [nil, '', '0'].include?(result['expires'])
             # firefox stores expiry as number of seconds
             result['expires'] = Time.at(result['expires'].to_i)
           end

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

@@ -32,7 +32,7 @@ def network_connection_type
         end
 
         def network_connection_type=(connection_type)
-          raise ArgumentError, "Invalid connection type" unless valid_type? connection_type
+          raise ArgumentError, 'Invalid connection type' unless valid_type? connection_type
 
           connection_value = type_to_values[connection_type]
 

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

@@ -48,7 +48,7 @@ def screenshot_as(format)
           when :base64
             bridge.getScreenshot
           when :png
-            bridge.getScreenshot.unpack("m")[0]
+            bridge.getScreenshot.unpack('m')[0]
           else
             raise Error::UnsupportedOperationError, "unsupported format: #{format.inspect}"
           end