mediawiki_selenium 1.0.0.pre.2 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 93a8424d3894106aa9575f18912096df4d6bda83
-  data.tar.gz: 226dbf63109a48e9ede7a3d2346a7356e697782b
+  metadata.gz: 834dc185ba694c50b41bd95e0364c4c75071a8bf
+  data.tar.gz: 5a0425ce60b219c451b046653945a3561947f7e0
 SHA512:
-  metadata.gz: 81739dccd7bedc37d73e438b4d58bf301580e5d0d6ae3d4fbb7d7da2221c92e3be1ddfd5d68a2461cbada9d999f078d0f0c65120ebf841708a97b54c3ac1a83a
-  data.tar.gz: dbcb5db3ae8620ed8ff25279440ea3f381e0cfd4f7b378c7b5c75069ba15c71a94f0bea73670db0b45f393e4980e358ca6903895fba3fd6de9696ca862886270
+  metadata.gz: 44f9e8fbe4cae9d9e30044ac2986601cb72859af136f07723755d58084ce6225766ddc21bd3d16e59e6bef183ef969213ff6f1e5a1a789ede3e0b0854165ccd5
+  data.tar.gz: 956616b547ae43072e3fd3a417fc67910e85539b7da1a354fb07c65d81a0d2fcb00aa48ad0c2b06c61a0bb1151ff8c2cd7033c94f10e203d22b535b88424ef1f

data/.rubocop.yml

@@ -1 +1,23 @@
-inherit_from: .rubocop_todo.yml
+AllCops:
+  StyleGuideCopsOnly: true
+
+Metrics/LineLength:
+  Max: 100
+
+Metrics/MethodLength:
+  Enabled: false
+
+Style/Alias:
+  Enabled: false
+
+Style/DotPosition:
+  EnforcedStyle: trailing
+
+Style/FileName:
+  Exclude: ['bin/*']
+
+Style/SignalException:
+  Enabled: false
+
+Style/TrivialAccessors:
+  ExactNameMatch: true

data/.ruby-gemset

@@ -0,0 +1 @@
+mediawiki_selenium

data/.ruby-version

@@ -0,0 +1 @@
+2.1.5

data/Gemfile

@@ -1,6 +1,3 @@
-#ruby=ruby-2.1.3
-#ruby-gemset=mediawiki_selenium
-
-source "https://rubygems.org"
+source 'https://rubygems.org'
 
 gemspec

data/UPGRADE.md

@@ -5,7 +5,7 @@
 First, update the `Gemfile` in your project's root directory to specify the
 new version.
 
-    gem 'mediawiki_selenium', '~> 1.0.0.pre.2'
+    gem 'mediawiki_selenium', '~> 1.0.0'
 
 ## Upgrade gems and dependencies
 

data/bin/mediawiki-selenium-init

@@ -1,5 +1,5 @@
 #!/usr/bin/env ruby
 
-require "mediawiki_selenium"
+require 'mediawiki_selenium'
 
 MediawikiSelenium::Initializer.new.install

data/lib/mediawiki_selenium/browser_factory/base.rb

@@ -1,4 +1,4 @@
-require "watir-webdriver"
+require 'watir-webdriver'
 
 module MediawikiSelenium
   module BrowserFactory
@@ -15,8 +15,8 @@ module MediawikiSelenium
         # @example Always configure Firefox's language according to `:browser_language`
         #   module MediawikiSelenium::BrowserFactory
         #     class Firefox < Base
-        #       bind(:browser_language) do |language, options|
-        #         options[:desired_capabilities][:firefox_profile]["intl.accept_languages"] = language
+        #       bind(:browser_language) do |lang, options|
+        #         options[:desired_capabilities][:firefox_profile]["intl.accept_languages"] = lang
         #       end
         #     end
         #   end
@@ -27,7 +27,7 @@ module MediawikiSelenium
         #                                  the browser options.
         #
         def bind(*names, &blk)
-          raise ArgumentError, "no block given" unless block_given?
+          raise ArgumentError, 'no block given' unless block_given?
 
           key = names.length == 1 ? names.first : names
           default_bindings[key] ||= []
@@ -41,7 +41,7 @@ module MediawikiSelenium
         #
         def bindings
           if superclass <= Base
-            default_bindings.merge(superclass.bindings) { |key, old, new| old + new }
+            default_bindings.merge(superclass.bindings) { |_key, old, new| old + new }
           else
             default_bindings
           end
@@ -116,7 +116,7 @@ module MediawikiSelenium
       # @return [Hash]
       #
       def bindings
-        self.class.bindings.merge(@bindings) { |key, old, new| old + new }
+        self.class.bindings.merge(@bindings) { |_key, old, new| old + new }
       end
 
       # Instantiate a browser using the given environmental configuration.
@@ -182,7 +182,7 @@ module MediawikiSelenium
       # @param env [Environment] Environment.
       # @param status [Symbol] Status of the executed scenario.
       #
-      def teardown(env, status)
+      def teardown(_env, _status)
         # abstract
       end
 
@@ -196,7 +196,7 @@ module MediawikiSelenium
         Selenium::WebDriver::Remote::Capabilities.send(browser_name)
       end
 
-      def finalize_options!(options)
+      def finalize_options!(_options)
         # abstract
       end
 

data/lib/mediawiki_selenium/browser_factory/chrome.rb

@@ -10,7 +10,7 @@ module MediawikiSelenium
     #
     class Chrome < Base
       bind(:browser_language) do |language, options|
-        options[:prefs]["intl.accept_languages"] = language
+        options[:prefs]['intl.accept_languages'] = language
       end
 
       bind(:browser_user_agent) do |user_agent, options|

data/lib/mediawiki_selenium/browser_factory/firefox.rb

@@ -12,16 +12,16 @@ module MediawikiSelenium
     class Firefox < Base
       bind(:browser_timeout) do |timeout, options|
         timeout = timeout.to_i
-        options[:profile]["dom.max_script_run_time"] = timeout
-        options[:profile]["dom.max_chrome_script_run_time"] = timeout
+        options[:profile]['dom.max_script_run_time'] = timeout
+        options[:profile]['dom.max_chrome_script_run_time'] = timeout
       end
 
       bind(:browser_language) do |language, options|
-        options[:profile]["intl.accept_languages"] = language
+        options[:profile]['intl.accept_languages'] = language
       end
 
       bind(:browser_user_agent) do |user_agent, options|
-        options[:profile]["general.useragent.override"] = user_agent
+        options[:profile]['general.useragent.override'] = user_agent
       end
 
       protected

data/lib/mediawiki_selenium/browser_factory/phantomjs.rb

@@ -10,11 +10,11 @@ module MediawikiSelenium
     #
     class Phantomjs < Base
       bind(:browser_language) do |language, options|
-        options[:desired_capabilities]["phantomjs.page.customHeaders.Accept-Language"] = language
+        options[:desired_capabilities]['phantomjs.page.customHeaders.Accept-Language'] = language
       end
 
       bind(:browser_user_agent) do |user_agent, options|
-        options[:desired_capabilities]["phantomjs.page.settings.userAgent"] = user_agent
+        options[:desired_capabilities]['phantomjs.page.settings.userAgent'] = user_agent
       end
     end
   end

data/lib/mediawiki_selenium/browser_factory.rb

@@ -1,9 +1,9 @@
 module MediawikiSelenium
   module BrowserFactory
-    autoload :Base, "mediawiki_selenium/browser_factory/base"
-    autoload :Firefox, "mediawiki_selenium/browser_factory/firefox"
-    autoload :Chrome, "mediawiki_selenium/browser_factory/chrome"
-    autoload :Phantomjs, "mediawiki_selenium/browser_factory/phantomjs"
+    autoload :Base, 'mediawiki_selenium/browser_factory/base'
+    autoload :Firefox, 'mediawiki_selenium/browser_factory/firefox'
+    autoload :Chrome, 'mediawiki_selenium/browser_factory/chrome'
+    autoload :Phantomjs, 'mediawiki_selenium/browser_factory/phantomjs'
 
     # Resolves and instantiates a new factory for the given browser name.
     #
@@ -17,7 +17,7 @@ module MediawikiSelenium
     # @return [BrowserFactory::Base]
     #
     def self.new(browser_name)
-      factory_class = const_get(browser_name.to_s.split("_").map(&:capitalize).join(""))
+      factory_class = const_get(browser_name.to_s.split('_').map(&:capitalize).join(''))
       factory_class.new(browser_name)
     end
   end

data/lib/mediawiki_selenium/environment.rb

@@ -1,4 +1,4 @@
-require "yaml"
+require 'yaml'
 
 module MediawikiSelenium
   # Provides an interface that unifies environmental configuration, page
@@ -85,11 +85,11 @@ module MediawikiSelenium
       # @see load
       #
       def load_default
-        load(ENV["MEDIAWIKI_ENVIRONMENT"], ENV)
+        load(ENV['MEDIAWIKI_ENVIRONMENT'], ENV)
       end
     end
 
-    self.default_configuration = "environments.yml"
+    self.default_configuration = 'environments.yml'
 
     def initialize(*configs)
       @_config = configs.map { |config| normalize_config(config) }.reduce(:merge)
@@ -168,7 +168,7 @@ module MediawikiSelenium
     # @return [Symbol]
     #
     def browser_name
-      lookup(:browser, default: "firefox").downcase.to_sym
+      lookup(:browser, default: 'firefox').downcase.to_sym
     end
 
     # A reference to this environment. Can be used in conjunction with {#[]}
@@ -227,7 +227,7 @@ module MediawikiSelenium
     # Whether browsers should be left open after each scenario completes.
     #
     def keep_browser_open?
-      lookup(:keep_browser_open, default: "false") == "true"
+      lookup(:keep_browser_open, default: 'false') == 'true'
     end
 
     # Returns the configured value for the given env variable name.
@@ -334,10 +334,10 @@ module MediawikiSelenium
     # @yieldparam browser [Watir::Browser] Browser object, before it's closed.
     #
     def teardown(status = :passed)
-      @_factory_cache.each do |_, factory|
+      @_factory_cache.each do |(_, browser_name), factory|
         factory.each do |browser|
           yield browser if block_given?
-          browser.close unless keep_browser_open?
+          browser.close unless keep_browser_open? && browser_name != :phantomjs
         end
 
         factory.teardown(self, status)
@@ -354,7 +354,8 @@ module MediawikiSelenium
       if scenario.respond_to? :feature
         "#{scenario.feature.title}: #{scenario.title}"
       elsif scenario.respond_to? :scenario_outline
-        "#{scenario.scenario_outline.feature.title}: #{scenario.scenario_outline.title}: #{scenario.name}"
+        outline = scenario.scenario_outline
+        "#{outline.feature.title}: #{outline.title}: #{scenario.name}"
       else
         scenario.name
       end
@@ -379,7 +380,7 @@ module MediawikiSelenium
     # @return [String]
     #
     def user_label(id = nil)
-      user(id).gsub("_", " ")
+      user(id).gsub('_', ' ')
     end
 
     # Navigates the current browser to the given wiki.
@@ -414,9 +415,9 @@ module MediawikiSelenium
         # Prefixing relative paths with an explicit "./" guarantees proper
         # parsing of paths like "Special:Page" that would otherwise be
         # confused for URI schemes.
-        if path.include?(":")
+        if path.include?(':')
           path_uri = URI.parse(path)
-          path = "./#{path}" if path_uri.class == URI::Generic && !path.start_with?("/")
+          path = "./#{path}" if path_uri.class == URI::Generic && !path.start_with?('/')
         end
 
         url = URI.parse(url).merge(path).to_s
@@ -463,11 +464,11 @@ module MediawikiSelenium
     private
 
     def browser_config
-      lookup_all(browser_factory.all_binding_keys, default: nil).reject { |k, v| v.nil? }
+      lookup_all(browser_factory.all_binding_keys, default: nil).reject { |_k, v| v.nil? }
     end
 
     def password_variable
-      name = lookup(:mediawiki_password_variable, default: "")
+      name = lookup(:mediawiki_password_variable, default: '')
       name.empty? ? :mediawiki_password : normalize_key(name)
     end
 
@@ -485,7 +486,7 @@ module MediawikiSelenium
 
       begin
         @_config = @_config.merge(overrides)
-        yield *overrides.values if block_given?
+        yield(*overrides.values) if block_given?
       ensure
         @_config = original_config
       end

data/lib/mediawiki_selenium/initializer.rb

@@ -1,4 +1,4 @@
-require "thor"
+require 'thor'
 
 module MediawikiSelenium
   # Creates the directory structure and configuration for a brand new
@@ -8,12 +8,12 @@ module MediawikiSelenium
     include Thor::Actions
 
     def self.source_root
-      File.expand_path("../../../templates", __FILE__)
+      File.expand_path('../../../templates', __FILE__)
     end
 
-    desc "install", "Creates tests/browser directory structure and default configuration"
+    desc 'install', 'Creates tests/browser directory structure and default configuration'
     def install
-      directory("tests")
+      directory('tests')
     end
   end
 end

data/lib/mediawiki_selenium/page_factory.rb

@@ -1,4 +1,4 @@
-require "page-object"
+require 'page-object'
 
 module MediawikiSelenium
   # Handles on-demand browser instantiation and assignment of the `@browser`

data/lib/mediawiki_selenium/remote_browser_factory.rb

@@ -1,5 +1,7 @@
-require "rest_client"
-require "uri"
+require 'mediawiki_selenium/support/sauce'
+
+require 'rest_client'
+require 'uri'
 
 module MediawikiSelenium
   # Constructs remote browser sessions to be run via Sauce Labs. Adds the
@@ -12,7 +14,7 @@ module MediawikiSelenium
   #
   module RemoteBrowserFactory
     REQUIRED_CONFIG = [:sauce_ondemand_username, :sauce_ondemand_access_key]
-    URL = "http://ondemand.saucelabs.com/wd/hub"
+    URL = 'http://ondemand.saucelabs.com/wd/hub'
 
     class << self
       def extend_object(factory)
@@ -51,13 +53,15 @@ module MediawikiSelenium
           url: "https://saucelabs.com/rest/v1/#{username}/jobs/#{sid}",
           user: username,
           password: key,
-          headers: { content_type: "application/json" },
+          headers: { content_type: 'application/json' },
           payload: {
             public: true,
             passed: status == :passed,
-            build: env.lookup(:build_number, default: nil),
+            build: env.lookup(:build_number, default: nil)
           }.to_json
         )
+
+        Cucumber::Formatter::Sauce.current_session_id = sid
       end
     end
 
@@ -68,9 +72,9 @@ module MediawikiSelenium
       when :firefox
         options[:desired_capabilities][:firefox_profile] = options.delete(:profile)
       when :chrome
-        options[:desired_capabilities]["chromeOptions"] ||= {}
-        options[:desired_capabilities]["chromeOptions"]["prefs"] = options.delete(:prefs)
-        options[:desired_capabilities]["chromeOptions"]["args"] = options.delete(:args)
+        options[:desired_capabilities]['chromeOptions'] ||= {}
+        options[:desired_capabilities]['chromeOptions']['prefs'] = options.delete(:prefs)
+        options[:desired_capabilities]['chromeOptions']['args'] = options.delete(:args)
       end
     end
 

data/lib/mediawiki_selenium/step_definitions/login_steps.rb

@@ -1,14 +1,3 @@
-=begin
-This file is subject to the license terms in the LICENSE file found in the
-mediawiki_selenium top-level directory and at
-https://git.wikimedia.org/blob/mediawiki%2Fselenium/HEAD/LICENSE. No part of
-mediawiki_selenium, including this file, may be copied, modified, propagated, or
-distributed except according to the terms contained in the LICENSE file.
-Copyright 2013 by the Mediawiki developers. See the CREDITS file in the
-mediawiki_selenium top-level directory and at
-https://git.wikimedia.org/blob/mediawiki%2Fselenium/HEAD/CREDITS.
-=end
-
 Given(/^I am logged in(?: as (\w+))$/) do |user|
   as_user(user) do |user, password|
     visit(LoginPage).login_with(user, password)

data/lib/mediawiki_selenium/step_definitions/navigation_steps.rb

@@ -1,14 +1,3 @@
-=begin
-This file is subject to the license terms in the LICENSE file found in the
-mediawiki_selenium top-level directory and at
-https://git.wikimedia.org/blob/mediawiki%2Fselenium/HEAD/LICENSE. No part of
-mediawiki_selenium, including this file, may be copied, modified, propagated, or
-distributed except according to the terms contained in the LICENSE file.
-Copyright 2013 by the Mediawiki developers. See the CREDITS file in the
-mediawiki_selenium top-level directory and at
-https://git.wikimedia.org/blob/mediawiki%2Fselenium/HEAD/CREDITS.
-=end
-
 Given(/^I am at a random page.*$/) do
   visit RandomPage
 end

data/lib/mediawiki_selenium/step_definitions/preferences_steps.rb

@@ -1,14 +1,3 @@
-=begin
-This file is subject to the license terms in the LICENSE file found in the
-mediawiki_selenium top-level directory and at
-https://git.wikimedia.org/blob/mediawiki%2Fselenium/HEAD/LICENSE. No part of
-mediawiki_selenium, including this file, may be copied, modified, propagated, or
-distributed except according to the terms contained in the LICENSE file.
-Copyright 2013 by the Mediawiki developers. See the CREDITS file in the
-mediawiki_selenium top-level directory and at
-https://git.wikimedia.org/blob/mediawiki%2Fselenium/HEAD/CREDITS.
-=end
-
 Given(/^I have reset my preferences$/) do
   visit(ResetPreferencesPage).submit_element.click
 end

data/lib/mediawiki_selenium/step_definitions/resource_loader_steps.rb

@@ -1,14 +1,3 @@
-=begin
-This file is subject to the license terms in the LICENSE file found in the
-mediawiki_selenium top-level directory and at
-https://git.wikimedia.org/blob/mediawiki%2Fselenium/HEAD/LICENSE. No part of
-mediawiki_selenium, including this file, may be copied, modified, propagated, or
-distributed except according to the terms contained in the LICENSE file.
-Copyright 2013 by the Mediawiki developers. See the CREDITS file in the
-mediawiki_selenium top-level directory and at
-https://git.wikimedia.org/blob/mediawiki%2Fselenium/HEAD/CREDITS.
-=end
-
 Then(/^page has no ResourceLoader errors$/) do
   result = browser.execute_script(<<-end)
 return (function() {

data/lib/mediawiki_selenium/step_definitions/upload_file_steps.rb

@@ -1,6 +1,6 @@
 
 When(/^upload bogus file (.+)$/) do |file_name|
-  require "tempfile"
+  require 'tempfile'
   path = "#{Dir.tmpdir}/#{file_name}"
 
   system("touch #{path}")
@@ -14,10 +14,10 @@ end
 
 
 When(/^upload file (.+)$/) do |file_name|
-  require "tempfile"
+  require 'tempfile'
   path = "#{Dir.tmpdir}/#{file_name}"
 
-  require "chunky_png"
+  require 'chunky_png'
   ChunkyPNG::Image.new(Random.new.rand(255), Random.new.rand(255), Random.new.rand(255)).save path
 
   if browser.driver.browser == :chrome

data/lib/mediawiki_selenium/step_definitions.rb

@@ -1,5 +1,5 @@
-require "mediawiki_selenium/step_definitions/login_steps"
-require "mediawiki_selenium/step_definitions/navigation_steps"
-require "mediawiki_selenium/step_definitions/preferences_steps"
-require "mediawiki_selenium/step_definitions/resource_loader_steps"
-require "mediawiki_selenium/step_definitions/upload_file_steps"
+require 'mediawiki_selenium/step_definitions/login_steps'
+require 'mediawiki_selenium/step_definitions/navigation_steps'
+require 'mediawiki_selenium/step_definitions/preferences_steps'
+require 'mediawiki_selenium/step_definitions/resource_loader_steps'
+require 'mediawiki_selenium/step_definitions/upload_file_steps'

data/lib/mediawiki_selenium/support/env.rb

@@ -1,21 +1,10 @@
-=begin
-This file is subject to the license terms in the LICENSE file found in the
-mediawiki_selenium top-level directory and at
-https://git.wikimedia.org/blob/mediawiki%2Fselenium/HEAD/LICENSE. No part of
-mediawiki_selenium, including this file, may be copied, modified, propagated, or
-distributed except according to the terms contained in the LICENSE file.
-Copyright 2013 by the Mediawiki developers. See the CREDITS file in the
-mediawiki_selenium top-level directory and at
-https://git.wikimedia.org/blob/mediawiki%2Fselenium/HEAD/CREDITS.
-=end
-
 # before all
-require "bundler/setup"
-require "page-object/page_factory"
-require "watir-webdriver"
+require 'bundler/setup'
+require 'page-object/page_factory'
+require 'watir-webdriver'
 
-require "mediawiki_selenium/support/modules/api_helper"
-require "mediawiki_selenium/support/modules/strict_pending"
+require 'mediawiki_selenium/support/modules/api_helper'
+require 'mediawiki_selenium/support/modules/strict_pending'
 
 World { MediawikiSelenium::Environment.load_default }
 

data/lib/mediawiki_selenium/support/hooks.rb

@@ -1,31 +1,20 @@
-=begin
-This file is subject to the license terms in the LICENSE file found in the
-mediawiki_selenium top-level directory and at
-https://git.wikimedia.org/blob/mediawiki%2Fselenium/HEAD/LICENSE. No part of
-mediawiki_selenium, including this file, may be copied, modified, propagated, or
-distributed except according to the terms contained in the LICENSE file.
-Copyright 2013 by the Mediawiki developers. See the CREDITS file in the
-mediawiki_selenium top-level directory and at
-https://git.wikimedia.org/blob/mediawiki%2Fselenium/HEAD/CREDITS.
-=end
-
-Before("@custom-browser") do |scenario|
+Before('@custom-browser') do |scenario|
   @scenario = scenario
 end
 
 AfterConfiguration do |config|
   # Install a formatter that can be used to show feature-related warnings
-  pretty_format, io = config.formats.find { |(format, io)| format == "pretty" }
-  config.formats << ["MediawikiSelenium::WarningsFormatter", io] if pretty_format
+  pretty_format, io = config.formats.find { |(format, _io)| format == 'pretty' }
+  config.formats << ['MediawikiSelenium::WarningsFormatter', io] if pretty_format
 
   # Initiate headless mode
-  if ENV["HEADLESS"] == "true"
-    require "headless"
+  if ENV['HEADLESS'] == 'true' && ENV['BROWSER'] != 'phantomjs'
+    require 'headless'
 
     headless_options = {}.tap do |options|
-      options[:display] = ENV["HEADLESS_DISPLAY"] if ENV.include?("HEADLESS_DISPLAY")
-      options[:reuse] = false if ENV["HEADLESS_REUSE"] == "false"
-      options[:destroy_at_exit] = false if ENV["HEADLESS_DESTROY_AT_EXIT"] == "false"
+      options[:display] = ENV['HEADLESS_DISPLAY'] if ENV.include?('HEADLESS_DISPLAY')
+      options[:reuse] = false if ENV['HEADLESS_REUSE'] == 'false'
+      options[:destroy_at_exit] = false if ENV['HEADLESS_DESTROY_AT_EXIT'] == 'false'
     end
 
     headless = Headless.new(headless_options)
@@ -46,8 +35,8 @@ Before do |scenario|
   dependencies = tags.map { |tag| tag.match(/^@extension-(.+)$/) { |m| m[1].downcase } }.compact
 
   unless dependencies.empty?
-    extensions = api.meta(:siteinfo, siprop: "extensions").data["extensions"]
-    extensions = extensions.map { |ext| ext["name"] }.compact.map(&:downcase)
+    extensions = api.meta(:siteinfo, siprop: 'extensions').data['extensions']
+    extensions = extensions.map { |ext| ext['name'] }.compact.map(&:downcase)
     missing = dependencies - extensions
 
     if missing.any?
@@ -55,7 +44,7 @@ Before do |scenario|
 
       if scenario.feature.respond_to?(:mw_warn)
         warning = "Skipped feature due to missing wiki extensions: #{missing.join(", ")}"
-        scenario.feature.mw_warn(warning, "missing wiki extensions")
+        scenario.feature.mw_warn(warning, 'missing wiki extensions')
       end
     end
   end
@@ -81,16 +70,16 @@ end
 
 After do |scenario|
   if scenario.respond_to?(:status)
-    require "fileutils"
+    require 'fileutils'
 
     teardown(scenario.status) do |browser|
-      if scenario.failed? && lookup(:screenshot_failures, default: false) == "true"
-        screen_dir = lookup(:screenshot_failures_path, default: "screenshots")
+      if scenario.failed? && lookup(:screenshot_failures, default: false) == 'true'
+        screen_dir = lookup(:screenshot_failures_path, default: 'screenshots')
         FileUtils.mkdir_p screen_dir
         name = test_name(scenario).gsub(/ /, '_')
         path = "#{screen_dir}/#{name}.png"
         browser.screenshot.save path
-        embed path, "image/png"
+        embed path, 'image/png'
       end
 
     end

data/lib/mediawiki_selenium/support/modules/api_helper.rb

@@ -1,4 +1,4 @@
-require "mediawiki_api"
+require 'mediawiki_api'
 
 module MediawikiSelenium
   # Provides more direct access to the API client from hooks and step
@@ -26,11 +26,9 @@ module MediawikiSelenium
     # @param id [Symbol] ID of alternative user.
     #
     def ensure_account(id)
-      begin
-        api.create_account(user(id), password(id))
-      rescue MediawikiApi::ApiError => e
-        raise e unless e.code == "userexists"
-      end
+      api.create_account(user(id), password(id))
+    rescue MediawikiApi::ApiError => e
+      raise e unless e.code == 'userexists'
     end
 
     # Extends parent implementation to also override the API URL. If no API
@@ -53,7 +51,7 @@ module MediawikiSelenium
     private
 
     def api_url_from(wiki_url)
-      URI.parse(wiki_url).merge("/w/api.php").to_s
+      URI.parse(wiki_url).merge('/w/api.php').to_s
     end
   end
 end

data/lib/mediawiki_selenium/support/pages/api_page.rb

@@ -1,5 +1,5 @@
-require "page-object"
-require "mediawiki_api"
+require 'page-object'
+require 'mediawiki_api'
 
 class APIPage
   include PageObject
@@ -7,12 +7,12 @@ class APIPage
   def client
     return @client if defined?(@client)
 
-    unless ENV["MEDIAWIKI_API_URL"]
-      raise "Environment variable MEDIAWIKI_API_URL must be set in order to use the API"
+    unless ENV['MEDIAWIKI_API_URL']
+      raise 'Environment variable MEDIAWIKI_API_URL must be set in order to use the API'
     end
 
-    @client = MediawikiApi::Client.new(ENV["MEDIAWIKI_API_URL"])
-    @client.log_in ENV["MEDIAWIKI_USER"], ENV["MEDIAWIKI_PASSWORD"]
+    @client = MediawikiApi::Client.new(ENV['MEDIAWIKI_API_URL'])
+    @client.log_in ENV['MEDIAWIKI_USER'], ENV['MEDIAWIKI_PASSWORD']
 
     @client
   end