mediawiki_selenium 1.0.0.pre.2 → 1.0.0

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

@@ -1,25 +1,26 @@
-require "page-object"
+require 'page-object'
 
 class LoginPage
   include PageObject
 
-  page_url "Special:UserLogin"
+  page_url 'Special:UserLogin'
 
-  div(:feedback, class: "errorbox")
-  button(:login, id: "wpLoginAttempt")
-  li(:logout, id: "pt-logout")
-  text_field(:password, id: "wpPassword1")
-  a(:password_strength, text: "password strength")
-  a(:phishing, text: "phishing")
-  text_field(:username, id: "wpName1")
+  div(:feedback, class: 'errorbox')
+  button(:login, id: 'wpLoginAttempt')
+  li(:logout, id: 'pt-logout')
+  text_field(:password, id: 'wpPassword1')
+  a(:password_strength, text: 'password strength')
+  a(:phishing, text: 'phishing')
+  text_field(:username, id: 'wpName1')
   a(:username_displayed, title: /Your user page/)
 
   def logged_in_as_element
-    @browser.div(id: "mw-content-text").p.b
+    @browser.div(id: 'mw-content-text').p.b
   end
+
   def login_with(username, password, wait_for_logout_element = true)
-    self.username_element.when_present.send_keys(username)
-    self.password_element.when_present.send_keys(password)
+    username_element.when_present.send_keys(username)
+    password_element.when_present.send_keys(password)
     login_element.when_present.click
     logout_element.when_present(10) if wait_for_logout_element
   end

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

@@ -1,7 +1,7 @@
-require "page-object"
+require 'page-object'
 
 class RandomPage
   include PageObject
 
-  page_url "Special:Random"
+  page_url 'Special:Random'
 end

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

@@ -1,9 +1,9 @@
-require "page-object"
+require 'page-object'
 
 class ResetPreferencesPage
   include PageObject
 
-  page_url "Special:Preferences/reset"
+  page_url 'Special:Preferences/reset'
 
-  button(:submit, class: "mw-htmlform-submit")
+  button(:submit, class: 'mw-htmlform-submit')
 end

data/lib/mediawiki_selenium/support/pages.rb

@@ -1,4 +1,4 @@
-require "mediawiki_selenium/support/pages/api_page"
-require "mediawiki_selenium/support/pages/login_page"
-require "mediawiki_selenium/support/pages/random_page"
-require "mediawiki_selenium/support/pages/reset_preferences_page"
+require 'mediawiki_selenium/support/pages/api_page'
+require 'mediawiki_selenium/support/pages/login_page'
+require 'mediawiki_selenium/support/pages/random_page'
+require 'mediawiki_selenium/support/pages/reset_preferences_page'

data/lib/mediawiki_selenium/support/sauce.rb

@@ -1,30 +1,26 @@
-=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
-
-require "cucumber/formatter/junit"
+require 'cucumber/formatter/junit'
 
 module Cucumber::Formatter
   class Sauce < Junit
 
+    class << self
+      attr_accessor :current_session_id
+    end
+
     private
 
     def format_exception(exception)
-      if ENV["HEADLESS"] == "true"
-        sauce_job_page = ""
-      elsif $session_id
-        sauce_job_page = "Sauce Labs job URL: http://saucelabs.com/jobs/#{$session_id}\n"
+      if ENV['HEADLESS'] == 'true'
+        job_url = ''
+      elsif self.class.current_session_id
+        job_url = "Sauce Labs job URL: http://saucelabs.com/jobs/#{self.class.current_session_id}\n"
       else
-        sauce_job_page = "Uh-oh. Could not find link to Sauce Labs job URL."
+        job_url = 'Uh-oh. Could not find link to Sauce Labs job URL.'
       end
-      ([sauce_job_page] + ["#{exception.message} (#{exception.class})"] + exception.backtrace).join("\n")
+
+      msgs = [job_url] + ["#{exception.message} (#{exception.class})"] + exception.backtrace
+
+      msgs.join("\n")
     end
   end
 end

data/lib/mediawiki_selenium/support.rb

@@ -1,4 +1,4 @@
-require "mediawiki_selenium/support/env"
-require "mediawiki_selenium/support/hooks"
-require "mediawiki_selenium/support/pages"
-require "mediawiki_selenium/support/sauce"
+require 'mediawiki_selenium/support/env'
+require 'mediawiki_selenium/support/hooks'
+require 'mediawiki_selenium/support/pages'
+require 'mediawiki_selenium/support/sauce'

data/lib/mediawiki_selenium/version.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
-
 module MediawikiSelenium
-  VERSION = "1.0.0.pre.2"
+  VERSION = '1.0.0'
 end

data/lib/mediawiki_selenium/warnings_formatter.rb

@@ -1,32 +1,32 @@
-require "cucumber/formatter/console"
+require 'cucumber/formatter/console'
 
 module MediawikiSelenium
   class WarningsFormatter
     include Cucumber::Formatter::Console
 
-    def initialize(runtime, io, options)
+    def initialize(_runtime, io, _options)
       @io = io
       @warning_counts = Hash.new(0)
     end
 
     def after_feature(feature)
-      if feature.mw_warnings.any?
-        feature.mw_warnings.each do |type, messages|
-          messages.each { |msg| @io.puts format_string(msg, :pending) }
-          @warning_counts[type] += messages.length
-        end
+      return unless feature.mw_warnings.any?
 
-        @io.puts
+      feature.mw_warnings.each do |type, messages|
+        messages.each { |msg| @io.puts format_string(msg, :pending) }
+        @warning_counts[type] += messages.length
       end
+
+      @io.puts
     end
 
-    def after_features(features)
-      if @warning_counts.any?
-        @warning_counts.each do |type, count|
-          message = "#{count} warning#{count > 1 ? "s" : ""}"
-          message += " due to #{type}" unless type == :default
-          @io.puts format_string(message, :pending)
-        end
+    def after_features(*)
+      return unless @warning_counts.any?
+
+      @warning_counts.each do |type, count|
+        message = "#{count} warning#{count > 1 ? "s" : ""}"
+        message += " due to #{type}" unless type == :default
+        @io.puts format_string(message, :pending)
       end
     end
 

data/lib/mediawiki_selenium.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
-
 module MediawikiSelenium
-  autoload :VERSION, "mediawiki_selenium/version"
-  autoload :ApiHelper, "mediawiki_selenium/support/modules/api_helper"
-  autoload :BrowserFactory, "mediawiki_selenium/browser_factory"
-  autoload :ConfigurationError, "mediawiki_selenium/configuration_error"
-  autoload :Environment, "mediawiki_selenium/environment"
-  autoload :Initializer, "mediawiki_selenium/initializer"
-  autoload :PageFactory, "mediawiki_selenium/page_factory"
-  autoload :RemoteBrowserFactory, "mediawiki_selenium/remote_browser_factory"
+  autoload :VERSION, 'mediawiki_selenium/version'
+  autoload :ApiHelper, 'mediawiki_selenium/support/modules/api_helper'
+  autoload :BrowserFactory, 'mediawiki_selenium/browser_factory'
+  autoload :ConfigurationError, 'mediawiki_selenium/configuration_error'
+  autoload :Environment, 'mediawiki_selenium/environment'
+  autoload :Initializer, 'mediawiki_selenium/initializer'
+  autoload :PageFactory, 'mediawiki_selenium/page_factory'
+  autoload :RemoteBrowserFactory, 'mediawiki_selenium/remote_browser_factory'
 end

data/mediawiki_selenium.gemspec

@@ -1,39 +1,46 @@
 # coding: utf-8
-lib = File.expand_path("../lib", __FILE__)
+lib = File.expand_path('../lib', __FILE__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
-require "mediawiki_selenium/version"
+require 'mediawiki_selenium/version'
 
 Gem::Specification.new do |spec|
-  spec.name          = "mediawiki_selenium"
+  spec.name          = 'mediawiki_selenium'
   spec.version       = MediawikiSelenium::VERSION
-  spec.authors       = ["Chris McMahon", "Dan Duvall", "Jeff Hall", "Nikolas Everett", "Tobias Gritschacher", "Željko Filipin"]
-  spec.email         = ["cmcmahon@wikimedia.org", "dduvall@wikimedia.org", "jhall@wikimedia.org", "neverett@wikimedia.org", "tobias.gritschacher@wikimedia.de", "zeljko.filipin@gmail.com"]
-  spec.description   = %q{Several MediaWiki extensions share code that makes it easy to run Selenium tests. This gem
-makes it easy to update the shared code.}
-  spec.summary       = %q{An easy way to run MediaWiki Selenium tests.}
-  spec.homepage      = "https://gerrit.wikimedia.org/r/#/admin/projects/mediawiki/selenium"
-  spec.license       = "GPL-2"
+  spec.authors       = ['Chris McMahon', 'Dan Duvall', 'Jeff Hall', 'Nikolas Everett',
+                        'Tobias Gritschacher', 'Željko Filipin']
+  spec.email         = ['cmcmahon@wikimedia.org', 'dduvall@wikimedia.org', 'jhall@wikimedia.org',
+                        'neverett@wikimedia.org', 'tobias.gritschacher@wikimedia.de',
+                        'zeljko.filipin@gmail.com']
 
-  spec.bindir      = "bin"
+  spec.description = (<<-end).split.join(' ')
+    Several MediaWiki extensions share code that makes it easy to run Selenium tests. This gem
+    makes it easy to update the shared code.
+  end
+
+  spec.summary       = 'An easy way to run MediaWiki Selenium tests.'
+  spec.homepage      = 'https://gerrit.wikimedia.org/r/#/admin/projects/mediawiki/selenium'
+  spec.license       = 'GPL-2'
+
+  spec.bindir      = 'bin'
   spec.executables << 'mediawiki-selenium-init'
 
-  spec.files         = `git ls-files`.split($/)
-  spec.require_paths = ["lib"]
+  spec.files         = `git ls-files`.split($INPUT_RECORD_SEPARATOR)
+  spec.require_paths = ['lib']
 
-  spec.add_runtime_dependency "cucumber", "~> 1.3", ">= 1.3.10"
-  spec.add_runtime_dependency "headless", "~> 1.0", ">= 1.0.1"
-  spec.add_runtime_dependency "json", "~> 1.8", ">= 1.8.1"
-  spec.add_runtime_dependency "mediawiki_api", "~> 0.2", ">= 0.2.1"
-  spec.add_runtime_dependency "page-object", "~> 1.0"
-  spec.add_runtime_dependency "rest-client", "~> 1.6", ">= 1.6.7"
-  spec.add_runtime_dependency "rspec-expectations", "~> 2.14", ">= 2.14.4"
-  spec.add_runtime_dependency "syntax", "~> 1.2", ">= 1.2.0"
-  spec.add_runtime_dependency "thor", "~> 0.19", ">= 0.19.1"
+  spec.add_runtime_dependency 'cucumber', '~> 1.3', '>= 1.3.10'
+  spec.add_runtime_dependency 'headless', '~> 1.0', '>= 1.0.1'
+  spec.add_runtime_dependency 'json', '~> 1.8', '>= 1.8.1'
+  spec.add_runtime_dependency 'mediawiki_api', '~> 0.2', '>= 0.2.1'
+  spec.add_runtime_dependency 'page-object', '~> 1.0'
+  spec.add_runtime_dependency 'rest-client', '~> 1.6', '>= 1.6.7'
+  spec.add_runtime_dependency 'rspec-expectations', '~> 2.14', '>= 2.14.4'
+  spec.add_runtime_dependency 'syntax', '~> 1.2', '>= 1.2.0'
+  spec.add_runtime_dependency 'thor', '~> 0.19', '>= 0.19.1'
 
-  spec.add_development_dependency "bundler", "~> 1.6", ">= 1.6.3"
-  spec.add_development_dependency "yard", "~> 0.8", ">= 0.8.7.4"
-  spec.add_development_dependency "redcarpet", "~> 3.2", ">= 3.2.0"
-  spec.add_development_dependency "rubocop", "~> 0.26", ">= 0.26.1"
-  spec.add_development_dependency "rspec-core", "~> 2.14", ">= 2.14.4"
-  spec.add_development_dependency "rspec-mocks", "~> 2.14", ">= 2.14.4"
+  spec.add_development_dependency 'bundler', '~> 1.6', '>= 1.6.3'
+  spec.add_development_dependency 'yard', '~> 0.8', '>= 0.8.7.4'
+  spec.add_development_dependency 'redcarpet', '~> 3.2', '>= 3.2.0'
+  spec.add_development_dependency 'rubocop', '~> 0.26', '>= 0.26.1'
+  spec.add_development_dependency 'rspec-core', '~> 2.14', '>= 2.14.4'
+  spec.add_development_dependency 'rspec-mocks', '~> 2.14', '>= 2.14.4'
 end

data/spec/api_helper_spec.rb

@@ -1,4 +1,4 @@
-require "spec_helper"
+require 'spec_helper'
 
 module MediawikiSelenium
   describe ApiHelper do
@@ -10,52 +10,52 @@ module MediawikiSelenium
         mediawiki_url_b: alternative_wiki_url,
         mediawiki_api_url: api_url,
         mediawiki_api_url_b: alternative_api_url,
-        mediawiki_user: "mw user",
-        mediawiki_password: "mw password"
+        mediawiki_user: 'mw user',
+        mediawiki_password: 'mw password'
       }
     end
 
-    describe "#api" do
+    describe '#api' do
       subject { env.api }
 
-      let(:wiki_url) { "http://an.example/wiki/" }
-      let(:api_url) { "http://an.example/api" }
-      let(:alternative_wiki_url) { "http://another.example/wiki/" }
-      let(:alternative_api_url) { "" }
+      let(:wiki_url) { 'http://an.example/wiki/' }
+      let(:api_url) { 'http://an.example/api' }
+      let(:alternative_wiki_url) { 'http://another.example/wiki/' }
+      let(:alternative_api_url) { '' }
 
       let(:client) { double(MediawikiApi::Client) }
 
       before do
         expect(MediawikiApi::Client).to receive(:new).with(api_url).and_return(client)
-        expect(client).to receive(:log_in).with("mw user", "mw password")
+        expect(client).to receive(:log_in).with('mw user', 'mw password')
       end
 
-      context "called for the first time" do
-        it "returns a new client" do
+      context 'called for the first time' do
+        it 'returns a new client' do
           expect(subject).to be(client)
         end
       end
 
-      context "called subsequently" do
+      context 'called subsequently' do
         before do
           env.api
         end
 
-        it "returns a cached client" do
+        it 'returns a cached client' do
           expect(subject).to be(client)
         end
       end
     end
 
-    describe "#on_wiki" do
-      let(:wiki_url) { "http://an.example/wiki/" }
-      let(:api_url) { "http://an.example/api" }
-      let(:alternative_wiki_url) { "http://another.example/wiki/" }
+    describe '#on_wiki' do
+      let(:wiki_url) { 'http://an.example/wiki/' }
+      let(:api_url) { 'http://an.example/api' }
+      let(:alternative_wiki_url) { 'http://another.example/wiki/' }
 
-      context "and the given alternative API URL is configured" do
-        let(:alternative_api_url) { "http://another.example/api" }
+      context 'and the given alternative API URL is configured' do
+        let(:alternative_api_url) { 'http://another.example/api' }
 
-        it "executes in the new environment using the alternative wiki and API URL" do
+        it 'executes in the new environment using the alternative wiki and API URL' do
           expect { |block| env.on_wiki(:b, &block) }.
             to yield_with_args(alternative_wiki_url, alternative_api_url)
 
@@ -66,16 +66,16 @@ module MediawikiSelenium
         end
       end
 
-      context "and no explicit API URL is configured for the wiki" do
-        let(:alternative_api_url) { "" }
+      context 'and no explicit API URL is configured for the wiki' do
+        let(:alternative_api_url) { '' }
 
-        it "constructs one at /w/api.php relative to the wiki URL" do
+        it 'constructs one at /w/api.php relative to the wiki URL' do
           expect { |block| env.on_wiki(:b, &block) }.
-            to yield_with_args(alternative_wiki_url, "http://another.example/w/api.php")
+            to yield_with_args(alternative_wiki_url, 'http://another.example/w/api.php')
 
           env.on_wiki(:b) do
             expect(env[:mediawiki_url]).to eq(alternative_wiki_url)
-            expect(env[:mediawiki_api_url]).to eq("http://another.example/w/api.php")
+            expect(env[:mediawiki_api_url]).to eq('http://another.example/w/api.php')
           end
         end
       end

data/spec/browser_factory/base_spec.rb

@@ -1,4 +1,4 @@
-require "spec_helper"
+require 'spec_helper'
 
 module MediawikiSelenium::BrowserFactory
   describe Base do
@@ -6,28 +6,28 @@ module MediawikiSelenium::BrowserFactory
     let(:factory) { factory_class.new(browser_name) }
     let(:browser_name) { :lynx }
 
-    describe ".bind" do
+    describe '.bind' do
       subject { factory_class.bind(option_name, &block) }
 
       let(:option_name) { :foo }
       let(:block) { proc { } }
 
-      it "adds a new default binding for the given option" do
+      it 'adds a new default binding for the given option' do
         subject
         expect(factory_class.default_bindings).to include(option_name)
         expect(factory_class.default_bindings[option_name]).to include(block)
       end
 
-      context "given no block" do
+      context 'given no block' do
         subject { factory_class.bind(option_name) }
 
-        it "raises an ArgumentError" do
+        it 'raises an ArgumentError' do
           expect { subject }.to raise_error(ArgumentError)
         end
       end
     end
 
-    describe ".bindings" do
+    describe '.bindings' do
       subject { factory_class.bindings }
 
       before do
@@ -38,59 +38,59 @@ module MediawikiSelenium::BrowserFactory
         expect(subject).to include(Base.default_bindings)
       end
 
-      it "includes its own bindings" do
+      it 'includes its own bindings' do
         expect(subject).to include(factory_class.default_bindings)
       end
     end
 
-    describe ".default_bindings" do
+    describe '.default_bindings' do
       subject { factory_class.default_bindings }
 
       it { is_expected.to be_a(Hash) }
 
-      context "before any bindings are defined" do
+      context 'before any bindings are defined' do
         it { is_expected.to be_empty }
       end
 
-      context "when bindings are defined" do
+      context 'when bindings are defined' do
         before do
           factory_class.bind(:foo) {}
           factory_class.bind(:bar) {}
           factory_class.bind(:bar) {}
         end
 
-        it "includes all defined bindings" do
+        it 'includes all defined bindings' do
           expect(subject).to include(:foo, :bar)
         end
 
-        it "includes all bindings for the same option name" do
+        it 'includes all bindings for the same option name' do
           expect(subject[:bar].length).to be(2)
         end
       end
     end
 
-    describe "#bind" do
+    describe '#bind' do
       subject { factory.bind(option_name, &block) }
       before { subject }
 
       let(:option_name) { :foo }
       let(:block) { proc { } }
 
-      it "adds a new binding for the given option" do
+      it 'adds a new binding for the given option' do
         expect(factory.bindings).to include(option_name)
         expect(factory.bindings[option_name]).to include(block)
       end
 
-      context "given no block" do
+      context 'given no block' do
         subject { factory.bind(option_name) }
 
-        it "will default to an empty block" do
+        it 'will default to an empty block' do
           expect(factory.bindings[option_name]).not_to include(nil)
         end
       end
     end
 
-    describe "#bindings" do
+    describe '#bindings' do
       subject { factory.bindings }
 
       before do
@@ -98,19 +98,19 @@ module MediawikiSelenium::BrowserFactory
         factory.bind(:bar)
       end
 
-      it "includes the class-level bindings" do
+      it 'includes the class-level bindings' do
         expect(subject).to include(factory_class.bindings)
       end
 
-      it "includes its own bindings" do
+      it 'includes its own bindings' do
         expect(subject).to include(:bar)
       end
     end
 
-    describe "#browser_for" do
+    describe '#browser_for' do
       subject { factory.browser_for(config) }
 
-      let(:config) { { foo: "x" } }
+      let(:config) { { foo: 'x' } }
 
       let(:watir_browser) { double(Watir::Browser) }
       let(:capabilities) { double(Selenium::WebDriver::Remote::Capabilities) }
@@ -122,27 +122,27 @@ module MediawikiSelenium::BrowserFactory
           at_least(:once).and_return(browser_name)
       end
 
-      it "creates a new Watir::Browser" do
+      it 'creates a new Watir::Browser' do
         expect(Watir::Browser).to receive(:new).once.and_return(watir_browser)
         expect(subject).to be(watir_browser)
       end
 
-      context "called more than once" do
+      context 'called more than once' do
         let(:config1) { config }
 
-        context "with the same configuration" do
+        context 'with the same configuration' do
           let(:config2) { config }
 
-          it "returns a cached browser" do
+          it 'returns a cached browser' do
             expect(Watir::Browser).to receive(:new).once.and_return(watir_browser)
             expect(factory.browser_for(config1)).to be(factory.browser_for(config2))
           end
         end
 
-        context "with different configuration" do
-          let(:config2) { { foo: "y" } }
+        context 'with different configuration' do
+          let(:config2) { { foo: 'y' } }
 
-          it "returns two distinct browsers" do
+          it 'returns two distinct browsers' do
             expect(Watir::Browser).to receive(:new).twice
 
             factory.browser_for(config1)
@@ -152,7 +152,7 @@ module MediawikiSelenium::BrowserFactory
       end
     end
 
-    describe "#browser_options" do
+    describe '#browser_options' do
       subject { factory.browser_options(config) }
 
       let(:config) { {} }
@@ -171,38 +171,42 @@ module MediawikiSelenium::BrowserFactory
       it { is_expected.to be_a(Hash) }
       it { is_expected.to include(desired_capabilities: capabilities, http_client: client) }
 
-      context "with a binding" do
-        context "and corresponding configuration" do
-          let(:config) { { foo: "x" } }
+      context 'with a binding' do
+        context 'and corresponding configuration' do
+          let(:config) { { foo: 'x' } }
 
-          it "invokes the binding with the configured value" do
-            expect { |block| factory.bind(:foo, &block) && subject }.to yield_with_args("x", options)
+          it 'invokes the binding with the configured value' do
+            expect { |block| factory.bind(:foo, &block) && subject }.
+              to yield_with_args('x', options)
           end
         end
 
-        context "but no configuration" do
+        context 'but no configuration' do
           let(:config) { {} }
 
-          it "never invokes the binding" do
-            expect { |block| factory.bind(:foo, &block) && subject }.to_not yield_control
+          it 'never invokes the binding' do
+            expect { |block| factory.bind(:foo, &block) && subject }.
+              to_not yield_control
           end
         end
       end
 
-      context "with a multi-option binding" do
-        context "and complete configuration for all options" do
-          let(:config) { { foo: "x", bar: "y" } }
+      context 'with a multi-option binding' do
+        context 'and complete configuration for all options' do
+          let(:config) { { foo: 'x', bar: 'y' } }
 
-          it "invokes the binding with the configured values" do
-            expect { |block| factory.bind(:foo, :bar, &block) && subject }.to yield_with_args("x", "y", options)
+          it 'invokes the binding with the configured values' do
+            expect { |block| factory.bind(:foo, :bar, &block) && subject }.
+              to yield_with_args('x', 'y', options)
           end
         end
 
-        context "but incomplete configuration for all options" do
-          let(:config) { { foo: "x" } }
+        context 'but incomplete configuration for all options' do
+          let(:config) { { foo: 'x' } }
 
-          it "never invokes the binding" do
-            expect { |block| factory.bind(:foo, :bar, &block) && subject }.to_not yield_control
+          it 'never invokes the binding' do
+            expect { |block| factory.bind(:foo, :bar, &block) && subject }.
+              to_not yield_control
           end
         end
       end