hyper-spec 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 2934f3c0c220b8178be5364c372de4474fba2b74
-  data.tar.gz: add452cd5e700a0540b8c97253176f55ff9eecfc
+  metadata.gz: a644db6a221bc30f343b05e079c705269a90871d
+  data.tar.gz: f6c9c28bbb47e6e39c5f7f3b88304da9df0ac8b8
 SHA512:
-  metadata.gz: 0793765d331c7070e355331de5528ed2333d98a48900569877a4a10c7e198a214ff5150db6eddf9ec6d4956b41f39b268a43573669f4e3500615abee86a9253e
-  data.tar.gz: 528748ad43262c8bb341306c3308be71054f81f152d96bb6cec3ef17059dcc86796b518a6e46b13b7ec08e5594a9dba22ca8e8e0cadd085131f4034ba3b65626
+  metadata.gz: de46555fb469983fba0b02f24fdd2439826f4dbca419b1e2fe53281fa62ed9512a077ec579900c0638ac1d85ac7d5f4ce130e8168c1b14343cc06775ae4c0872
+  data.tar.gz: ac2b3168d21fa3993e0af01f25fc5fe4129532a8f77cf4ff167092355d17be25d0a587286c9188bebee650a3e785c718bf1e39bc943c9227dcdb1503381956f0

data/.gitignore

@@ -42,3 +42,10 @@ bower.json
 
 # Ignore Byebug command history file.
 .byebug_history
+
+# Ignore test app files
+/spec/test_app/log/
+/spec/test_app/tmp/
+
+# Ignore built gem
+*.gem

data/.rubocop.yml

@@ -0,0 +1,107 @@
+Style/BlockDelimiters:
+  EnforcedStyle: line_count_based
+  SupportedStyles:
+    # The `line_count_based` style enforces braces around single line blocks and
+    # do..end around multi-line blocks.
+    - line_count_based
+    # The `semantic` style enforces braces around functional blocks, where the
+    # primary purpose of the block is to return a value and do..end for
+    # procedural blocks, where the primary purpose of the block is its
+    # side-effects.
+    #
+    # This looks at the usage of a block's method to determine its type (e.g. is
+    # the result of a `map` assigned to a variable or passed to another
+    # method) but exceptions are permitted in the `ProceduralMethods`,
+    # `FunctionalMethods` and `IgnoredMethods` sections below.
+    - semantic
+    # The `braces_for_chaining` style enforces braces around single line blocks
+    # and do..end around multi-line blocks, except for multi-line blocks whose
+    # return value is being chained with another method (in which case braces
+    # are enforced).
+    - braces_for_chaining
+  ProceduralMethods:
+    # Methods that are known to be procedural in nature but look functional from
+    # their usage, e.g.
+    #
+    #   time = Benchmark.realtime do
+    #     foo.bar
+    #   end
+    #
+    # Here, the return value of the block is discarded but the return value of
+    # `Benchmark.realtime` is used.
+    - benchmark
+    - bm
+    - bmbm
+    - create
+    - each_with_object
+    - measure
+    - new
+    - realtime
+    - tap
+    - with_object
+  FunctionalMethods:
+    # Methods that are known to be functional in nature but look procedural from
+    # their usage, e.g.
+    #
+    #   let(:foo) { Foo.new }
+    #
+    # Here, the return value of `Foo.new` is used to define a `foo` helper but
+    # doesn't appear to be used from the return value of `let`.
+    - let
+    - let!
+    - subject
+    - watch
+  IgnoredMethods:
+    # Methods that can be either procedural or functional and cannot be
+    # categorised from their usage alone, e.g.
+    #
+    #   foo = lambda do |x|
+    #     puts "Hello, #{x}"
+    #   end
+    #
+    #   foo = lambda do |x|
+    #     x * 100
+    #   end
+    #
+    # Here, it is impossible to tell from the return value of `lambda` whether
+    # the inner block's return value is significant.
+    - lambda
+    - proc
+    - it
+
+Style/Documentation:
+  Description: 'Document classes and non-namespace modules.'
+  Enabled: false
+  Exclude:
+    - 'spec/**/*'
+    - 'test/**/*'
+
+# Multi-line method chaining should be done with trailing dots.
+Style/DotPosition:
+  EnforcedStyle: leading
+  SupportedStyles:
+    - leading
+    - trailing
+
+Style/FrozenStringLiteralComment:
+  Description: >-
+                 Add the frozen_string_literal comment to the top of files
+                 to help transition from Ruby 2.3.0 to Ruby 3.0.
+  Enabled: false
+
+Style/MultilineBlockChain:
+  Description: 'Avoid multi-line chains of blocks.'
+  StyleGuide: 'https://github.com/bbatsov/ruby-style-guide#single-line-blocks'
+  Enabled: false
+
+Style/MutableConstant:
+  Description: 'Do not assign mutable objects to constants.'
+  Enabled: false
+
+Metrics/AbcSize:
+  # The ABC size is a calculated magnitude, so this number can be a Fixnum or
+  # a Float.
+  Max: 20
+
+Metrics/LineLength:
+  Max: 100

data/Gemfile.lock

@@ -1,9 +1,16 @@
 PATH
   remote: .
   specs:
-    hyper-spec (0.1.0)
-      hyper-react (>= 0.10.0)
-      rails
+    hyper-spec (0.1.2)
+      capybara
+      opal
+      parser
+      poltergeist
+      pry
+      rspec-rails
+      selenium-webdriver (= 2.53.4)
+      timecop
+      unparser (= 0.2.5)
 
 GEM
   remote: https://rubygems.org/
@@ -60,14 +67,14 @@ GEM
       babel-source (>= 4.0, < 6)
       execjs (~> 2.0)
     builder (3.2.3)
-    capybara (2.10.1)
+    capybara (2.13.0)
       addressable
       mime-types (>= 1.16)
       nokogiri (>= 1.3.3)
       rack (>= 1.0.0)
       rack-test (>= 0.5.4)
       xpath (~> 2.0)
-    childprocess (0.6.1)
+    childprocess (0.6.2)
       ffi (~> 1.0, >= 1.0.11)
     chromedriver-helper (1.0.0)
       archive-zip (~> 0.7.0)
@@ -84,7 +91,7 @@ GEM
     equalizer (0.0.11)
     erubis (2.7.0)
     execjs (2.7.0)
-    ffi (1.9.17)
+    ffi (1.9.18)
     globalid (0.3.7)
       activesupport (>= 4.1.0)
     hike (1.2.3)
@@ -142,17 +149,18 @@ GEM
     paggio (0.2.6)
     parser (2.3.3.1)
       ast (~> 2.2)
-    poltergeist (1.11.0)
+    poltergeist (1.14.0)
       capybara (~> 2.1)
       cliver (~> 0.3.1)
       websocket-driver (>= 0.2.0)
+    powerpack (0.1.1)
     procto (0.0.3)
     pry (0.10.4)
       coderay (~> 1.1.0)
       method_source (~> 0.8.1)
       slop (~> 3.4)
     public_suffix (2.0.5)
-    puma (3.6.2)
+    puma (3.7.0)
     rack (2.0.1)
     rack-test (0.6.3)
       rack (>= 1.0)
@@ -179,6 +187,7 @@ GEM
       method_source
       rake (>= 0.8.7)
       thor (>= 0.18.1, < 2.0)
+    rainbow (2.2.1)
     rake (10.5.0)
     react-rails (1.4.2)
       babel-transpiler (>= 0.7.0)
@@ -192,7 +201,7 @@ GEM
       rspec-core (~> 3.5.0)
       rspec-expectations (~> 3.5.0)
       rspec-mocks (~> 3.5.0)
-    rspec-collection_matchers (1.1.2)
+    rspec-collection_matchers (1.1.3)
       rspec-expectations (>= 2.99.0.beta1)
     rspec-core (3.5.4)
       rspec-support (~> 3.5.0)
@@ -216,7 +225,14 @@ GEM
     rspec-steps (2.1.1)
       rspec (>= 3.0, < 3.99)
     rspec-support (3.5.0)
-    rubyzip (1.2.0)
+    rubocop (0.47.1)
+      parser (>= 2.3.3.1, < 3.0)
+      powerpack (~> 0.1)
+      rainbow (>= 1.99.1, < 3.0)
+      ruby-progressbar (~> 1.7)
+      unicode-display_width (~> 1.0, >= 1.0.1)
+    ruby-progressbar (1.8.1)
+    rubyzip (1.2.1)
     selenium-webdriver (2.53.4)
       childprocess (~> 0.5)
       rubyzip (~> 1.0)
@@ -224,7 +240,7 @@ GEM
     shoulda (3.5.0)
       shoulda-context (~> 1.0, >= 1.0.1)
       shoulda-matchers (>= 1.4.1, < 3.0)
-    shoulda-context (1.2.1)
+    shoulda-context (1.2.2)
     shoulda-matchers (2.8.0)
       activesupport (>= 3.0.0)
     slop (3.6.0)
@@ -249,6 +265,7 @@ GEM
     timecop (0.8.1)
     tzinfo (1.2.2)
       thread_safe (~> 0.1)
+    unicode-display_width (1.1.3)
     unparser (0.2.5)
       abstract_type (~> 0.0.7)
       adamantium (~> 0.2.0)
@@ -269,32 +286,26 @@ PLATFORMS
 
 DEPENDENCIES
   bundler (~> 1.12)
-  capybara
   chromedriver-helper
+  hyper-react (>= 0.10.0)
   hyper-spec!
   method_source
   opal-browser
   opal-rails
-  parser
-  poltergeist
-  pry
   puma
   rails (~> 5.0.0)
   rake (~> 10.0)
-  react-rails (~> 1.4.0)
+  react-rails (< 1.10.0)
   rspec-collection_matchers
   rspec-expectations
   rspec-its
   rspec-mocks
-  rspec-rails
   rspec-steps
-  selenium-webdriver (= 2.53.4)
+  rubocop
   shoulda
   shoulda-matchers
   spring-commands-rspec
   therubyracer (= 0.12.2)
-  timecop
-  unparser (= 0.2.5)
 
 BUNDLED WITH
    1.12.5

data/README.md

@@ -350,6 +350,11 @@ See the Timecop [README](https://github.com/travisjeffery/timecop/blob/master/RE
 
 There is one confusing thing to note:  On the server if you `sleep` then you will sleep for the specified number of seconds when viewed *outside* of the test.  However inside the test environment if you look at Time.now, you will see it advancing according to the scale factor.  Likewise if you have a `after` or `every` block on the client, you will wait according to *simulated* time.
 
+## Common Problems
+
+If you are getting failures on Poltergeist but not Firefox, make sure you are not requiring `browser` in your components.rb.
+Requiring `browser/interval` or `browser/delay` is okay.
+
 ## Development
 
 After checking out the repo, run bundle install and you should be good to go.

data/hyper-spec.gemspec

@@ -1,68 +1,72 @@
 # 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 'hyper-spec/version'
 
-Gem::Specification.new do |spec|
-  spec.name          = "hyper-spec"
+Gem::Specification.new do |spec| # rubocop:disable Metrics/BlockLength
+  spec.name          = 'hyper-spec'
   spec.version       = HyperSpec::VERSION
-  spec.authors       = ["catmando"]
-  spec.email         = ["mitch@catprint.com"]
+  spec.authors       = ['catmando', 'adamcreekroad']
+  spec.email         = ['mitch@catprint.com']
 
-  spec.summary       = "Drive your Hyperloop client and server specs from RSpec and Capybara"
-  spec.description   = "A Hyperloop application consists of isomorphic React Components, Active Record Models, Stores, Operations and Policiespec. "\
-                       "Test them all from Rspec, regardless if the code runs on the client or server."
-  spec.homepage      = "https://github.com/ruby-hyperloop/hyper-spec"
-  spec.license       = "MIT"
+  spec.summary       =
+    'Drive your Hyperloop client and server specs from RSpec and Capybara'
+  spec.description   =
+    'A Hyperloop application consists of isomorphic React Components, '\
+    'Active Record Models, Stores, Operations and Policiespec. '\
+    'Test them all from Rspec, regardless if the code runs on the client or server.'
+  spec.homepage      = 'https://github.com/ruby-hyperloop/hyper-spec'
+  spec.license       = 'MIT'
 
-  spec.files         = `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
-  spec.bindir        = "exe"
-  spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
-  spec.require_paths = ["lib"]
-
-  spec.add_development_dependency "bundler", "~> 1.12"
-  spec.add_development_dependency "rake", "~> 10.0"
+  # Prevent pushing this gem to RubyGemspec.org. To allow pushes either set the 'allowed_push_host'
+  # to allow pushing to a single host or delete this section to allow pushing to any host.
+  spec.metadata['allowed_push_host'] = 'https://rubygems.org'
 
-  spec.add_dependency 'hyper-react', '>= 0.10.0'
+  spec.files         =
+    `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
+  spec.bindir        = 'exe'
+  spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
+  spec.require_paths = ['lib']
 
-  spec.add_development_dependency 'rspec-rails'
+  # Test app dependencies
+  spec.add_development_dependency 'bundler', '~> 1.12'
+  spec.add_development_dependency 'hyper-react', '>= 0.10.0'
+  spec.add_development_dependency 'rails', '~>5.0.0'
+  spec.add_development_dependency 'rake', '~> 10.0'
+  spec.add_development_dependency 'react-rails', '< 1.10.0'
 
-  # For Test Rails App
-  spec.add_development_dependency 'rails', '~>5.0.0' #'4.2.4'
-  spec.add_development_dependency 'react-rails', '~> 1.4.0'
-  spec.add_development_dependency 'opal-rails'
-  # spec.add_development_dependency 'opal-activesupport'
-  # spec.add_development_dependency 'factory_girl_rails'
-  # spec.add_development_dependency 'reactrb-rails-generator'
-  # spec.add_development_dependency 'rspec-wait'
-  spec.add_development_dependency 'puma'
+  # Keep linter-rubocop happy
+  spec.add_development_dependency 'rubocop'
 
   if RUBY_PLATFORM == 'java'
     spec.add_development_dependency 'therubyrhino'
   else
     spec.add_development_dependency 'therubyracer', '0.12.2'
 
-    # The following allow react code to be tested from the server side
-    spec.add_development_dependency 'rspec-mocks'
-    spec.add_development_dependency 'rspec-expectations'
+    # Actual dependencies
+    spec.add_dependency 'capybara'
+    spec.add_dependency 'opal'
+    spec.add_dependency 'parser'
+    spec.add_dependency 'poltergeist'
+    spec.add_dependency 'pry'
+    spec.add_dependency 'rspec-rails'
+    spec.add_dependency 'selenium-webdriver', '2.53.4'
+    spec.add_dependency 'timecop'
+    spec.add_dependency 'unparser', '0.2.5'
 
-    # spec.add_development_dependency 'factory_girl_rails'
-    spec.add_development_dependency 'shoulda'
-    spec.add_development_dependency 'shoulda-matchers'
-    spec.add_development_dependency 'rspec-its'
-    spec.add_development_dependency 'rspec-collection_matchers'
-    spec.add_development_dependency 'capybara'
-    spec.add_development_dependency 'selenium-webdriver', '2.53.4'
-    spec.add_development_dependency 'poltergeist'
-    spec.add_development_dependency 'spring-commands-rspec'
+    # Test app dependencies
     spec.add_development_dependency 'chromedriver-helper'
-    spec.add_development_dependency 'parser'
-    spec.add_development_dependency 'unparser',  '0.2.5'
-    spec.add_development_dependency 'pry'
     spec.add_development_dependency 'method_source'
     spec.add_development_dependency 'opal-browser'
+    spec.add_development_dependency 'opal-rails'
+    spec.add_development_dependency 'puma'
+    spec.add_development_dependency 'rspec-collection_matchers'
+    spec.add_development_dependency 'rspec-expectations'
+    spec.add_development_dependency 'rspec-its'
+    spec.add_development_dependency 'rspec-mocks'
     spec.add_development_dependency 'rspec-steps'
-    spec.add_dependency "rails"
-    spec.add_development_dependency 'timecop'
+    spec.add_development_dependency 'shoulda'
+    spec.add_development_dependency 'shoulda-matchers'
+    spec.add_development_dependency 'spring-commands-rspec'
   end
 end

data/lib/hyper-spec.rb

@@ -1,286 +1,88 @@
-require "hyper-spec/version"
-require "hyper-spec/engine"
-
-require 'opal'
-
-require 'hyper-spec/component_helpers'
-
-# DELETE require 'pry'
-# DELETE begin
-# DELETE   require File.expand_path('../test_app/config/environment', __FILE__)
-# DELETE rescue LoadError
-# DELETE   puts 'Could not load test application. Please ensure you have run `bundle exec rake test_app`'
-# DELETE end
-# DELETE require 'rspec/rails'
-# DELETE require 'timecop'
-# DELETE require "rspec/wait"
-# DELETE #require 'pusher-fake/support/base'
-
-# DELETE Dir["./spec/support/**/*.rb"].sort.each { |f| require f }
-
-RSpec.configure do |config|
-  # DELETE config.color = true
-  # DELETE config.fail_fast = ENV['FAIL_FAST'] || false
-  # DELETE config.fixture_path = File.join(File.expand_path(File.dirname(__FILE__)), "fixtures")
-  # DELETE config.infer_spec_type_from_file_location!
-  config.mock_with :rspec
-  # DELETE config.raise_errors_for_deprecations!
-
-  # If you're not using ActiveRecord, or you'd prefer not to run each of your
-  # examples within a transaction, comment the following line or assign false
-  # instead of true.
-  # DELETE config.use_transactional_fixtures = true
-
-  # config.after :each do
-  #   Rails.cache.clear
-  # end
-
-  config.after(:each) do |example|
-    unless example.exception
-      # DELETE #Object.send(:remove_const, :Application) rescue nil
-      # DELETE ObjectSpace.each_object(Class).each do |klass|
-      # DELETE   if klass < HyperMesh::Regulation
-      # DELETE     klass.instance_variables.each { |v| klass.instance_variable_set(v, nil) }
-      # DELETE   end
-      # DELETE end
-      PusherFake::Channel.reset if defined? PusherFake
-    end
-  end
-
-  # DELETE config.filter_run_including focus: true
-  # DELETE config.filter_run_excluding opal: true
-  # DELETE config.run_all_when_everything_filtered = true
-end
-
-# DELETE FACTORY_GIRL = false
-
-# DELETE #require 'rails_helper'
-# DELETE require 'rspec'
-# DELETE require 'rspec/expectations'
-# DELETE begin
-# DELETE   require 'factory_girl_rails'
-# DELETE rescue LoadError
-# DELETE end
-# DELETE require 'shoulda/matchers'
-# DELETE require 'database_cleaner'
 require 'capybara/rspec'
-#hmmm.... where should this go???? require 'capybara/rails'
 require 'capybara/poltergeist'
+require 'opal'
 require 'selenium-webdriver'
 
-module React
-  module IsomorphicHelpers
-    def self.load_context(ctx, controller, name = nil)
-      @context = Context.new("#{controller.object_id}-#{Time.now.to_i}", ctx, controller, name)
-    end
-  end
-end
-
-# DELETE #Capybara.default_max_wait_time = 4.seconds
-
-Capybara.server = :puma
-
-module WaitForAjax
-
-  def wait_for_ajax
-    Timeout.timeout(Capybara.default_max_wait_time) do
-      begin
-        sleep 0.25
-      end until finished_all_ajax_requests?
-    end
-  end
-
-  def running?
-    result = page.evaluate_script("(function(active) {console.log('jquery is active? '+active); return active})(jQuery.active)")
-    result && !result.zero?
-  rescue Exception => e
-    puts "wait_for_ajax failed while testing state of jQuery.active: #{e}"
-  end
-
-  def finished_all_ajax_requests?
-    unless running?
-      sleep 0.25 # this was 1 second, not sure if its necessary to be so long...
-      !running?
-    end
-  rescue Capybara::NotSupportedByDriverError
-    true
-  rescue Exception => e
-    e.message == "jQuery is not defined"
-  end
-
-end
-
-RSpec.configure do |config|
-  config.include WaitForAjax
-end
+require 'hyper-spec/component_test_helpers'
+require 'hyper-spec/rails/engine'
+require 'hyper-spec/version'
+require 'hyper-spec/wait_for_ajax'
+require 'react/isomorphic_helpers'
+require 'selenium/web_driver/firefox/profile'
 
 RSpec.configure do |config|
+  config.include HyperSpec::ComponentTestHelpers
+  config.include HyperSpec::WaitForAjax
+  config.include Capybara::DSL
 
-  Capybara.default_max_wait_time = 10
+  config.mock_with :rspec
 
-  config.before(:each) do |x|
-    HyperMesh.class_eval do
-      def self.on_server?
-        true
+  if defined?(HyperMesh)
+    config.before(:each) do
+      HyperMesh.class_eval do
+        def self.on_server?
+          true
+        end
       end
     end
-  end if defined? HyperMesh
+  end
 
-  config.before(:each, :js => true) do
+  config.before(:each, js: true) do
     size_window
   end
 
-  config.after(:each, :js => true) do
-    page.instance_variable_set("@hyper_spec_mounted", false)
+  config.after(:each, js: true) do
+    page.instance_variable_set('@hyper_spec_mounted', false)
   end
 
-
-  if false # delete? THIS BLOCK ????
-    # rspec-expectations config goes here. You can use an alternate
-    # assertion/expectation library such as wrong or the stdlib/minitest
-    # assertions if you prefer.
-    config.expect_with :rspec do |expectations|
-      # Enable only the newer, non-monkey-patching expect syntax.
-      # For more details, see:
-      #   - http://myronmars.to/n/dev-blog/2012/06/rspecs-new-expectation-syntax
-      expectations.syntax = [:should, :expect]
-    end
-
-    # rspec-mocks config goes here. You can use an alternate test double
-    # library (such as bogus or mocha) by changing the `mock_with` option here.
-    config.mock_with :rspec do |mocks|
-      # Enable only the newer, non-monkey-patching expect syntax.
-      # For more details, see:
-      #   - http://teaisaweso.me/blog/2013/05/27/rspecs-new-message-expectation-syntax/
-      mocks.syntax = :expect
-
-      # Prevents you from mocking or stubbing a method that does not exist on
-      # a real object. This is generally recommended.
-      mocks.verify_partial_doubles = true
-    end
-
-    config.include FactoryGirl::Syntax::Methods if defined? FactoryGirl
-
-    config.use_transactional_fixtures = false
-
-    config.before(:suite) do
-      DatabaseCleaner.clean_with(:truncation)
-    end
-
-    config.before(:each) do
-      DatabaseCleaner.strategy = :transaction
-    end
-
-    config.before(:each, :js => true) do
-      DatabaseCleaner.strategy = :truncation
-    end
-
-    config.before(:each) do
-      DatabaseCleaner.start
-    end
-
-    config.after(:each) do |example|
-      unless example.exception
-        # Clear session data
-        Capybara.reset_sessions!
-        # Rollback transaction
-        DatabaseCleaner.clean
-      end
+  config.after(:each) do |example|
+    unless example.exception
+      PusherFake::Channel.reset if defined? PusherFake
     end
   end
+end
 
+# Capybara config
+RSpec.configure do |_config|
+  Capybara.default_max_wait_time = 10
 
-  config.include Capybara::DSL
-
-  Capybara.register_driver :chrome do |app|
-    #caps = Selenium::WebDriver::Remote::Capabilities.chrome("chromeOptions" => {"excludeSwitches" => [ "ignore-certificate-errors" ]})
-    caps = Selenium::WebDriver::Remote::Capabilities.chrome("chromeOptions" => {"args" => [ "--window-size=200,200" ]})
-    Capybara::Selenium::Driver.new(app, :browser => :chrome, :desired_capabilities => caps)
-  end
+  # # In case Google ever fixes chromedriver to work with Opal...
+  # Capybara.register_driver :chrome do |app|
+  #   caps = Selenium::WebDriver::Remote::Capabilities.chrome(
+  #     'chromeOptions' => { 'args' => ['--window-size=200,200'] }
+  #   )
+  #
+  #   Capybara::Selenium::Driver.new(app, browser: :chrome, desired_capabilities: caps)
+  # end
 
-  options = {
-    js_errors: false,
-    timeout: 180,
-    inspector: true,
-    phantomjs_options: ['--load-images=no', '--ignore-ssl-errors=yes']
-  }
-  options.merge!({phantomjs_logger: StringIO.new, logger: StringIO.new,}) unless ENV['SHOW_LOGS']
   Capybara.register_driver :poltergeist do |app|
-    Capybara::Poltergeist::Driver.new(app, options)
-  end
-
-  class Selenium::WebDriver::Firefox::Profile
-
-    def self.firebug_version
-      @firebug_version ||= '2.0.13-fx'
-    end
-
-    def self.firebug_version=(version)
-      @firebug_version = version
-    end
-
-    def frame_position
-      @frame_position ||= 'detached'
-    end
-
-    def frame_position=(position)
-      @frame_position = ["left", "right", "top", "detached"].detect do |side|
-        position && position[0].downcase == side[0]
-      end || "detached"
-    end
-
-    def enable_firebug(version = nil)
-      version ||= Selenium::WebDriver::Firefox::Profile.firebug_version
-      add_extension(File.expand_path("../bin/firebug-#{version}.xpi", __FILE__))
-
-      # For some reason, Firebug seems to trigger the Firefox plugin check
-      # (navigating to https://www.mozilla.org/en-US/plugincheck/ at startup).
-      # This prevents it. See http://code.google.com/p/selenium/issues/detail?id=4619.
-      self["extensions.blocklist.enabled"] = false
-
-      # Prevent "Welcome!" tab
-      self["extensions.firebug.showFirstRunPage"] = false
-
-      # Enable for all sites.
-      self["extensions.firebug.allPagesActivation"] = "on"
-
-      # Enable all features.
-      ['console', 'net', 'script'].each do |feature|
-        self["extensions.firebug.#{feature}.enableSites"] = true
+    options = {
+      js_errors: false, timeout: 180, inspector: true,
+      phantomjs_options: ['--load-images=no', '--ignore-ssl-errors=yes']
+    }.tap do |hash|
+      unless ENV['SHOW_LOGS']
+        hash[:phantomjs_logger] = StringIO.new
+        hash[:logger] = StringIO.new
       end
-
-      # Closed by default, will open detached.
-      self["extensions.firebug.framePosition"] = frame_position
-      self["extensions.firebug.previousPlacement"] = 3
-      self["extensions.firebug.defaultPanelName"] = "console"
-
-      # Disable native "Inspect Element" menu item.
-      self["devtools.inspector.enabled"] = false
-      self["extensions.firebug.hideDefaultInspector"] = true
     end
+
+    Capybara::Poltergeist::Driver.new(app, options)
   end
 
   Capybara.register_driver :selenium_with_firebug do |app|
     profile = Selenium::WebDriver::Firefox::Profile.new
     profile.frame_position = ENV['DRIVER'] && ENV['DRIVER'][2]
     profile.enable_firebug
-    Capybara::Selenium::Driver.new(app, :browser => :firefox, :profile => profile)
-  end
-
-  Capybara.javascript_driver = :poltergeist
-
-  Capybara.register_driver :chrome do |app|
-    Capybara::Selenium::Driver.new(app, :browser => :chrome)
-  end
 
-  if ENV['DRIVER'] =~ /^ff/
-    Capybara.javascript_driver = :selenium_with_firebug
-  elsif ENV['DRIVER'] == 'chrome'
-    Capybara.javascript_driver = :chrome
-  else
-    Capybara.javascript_driver = :poltergeist
+    Capybara::Selenium::Driver.new(app, browser: :firefox, profile: profile)
   end
 
-  include ComponentTestHelpers
-
+  Capybara.javascript_driver =
+    if ENV['DRIVER'] =~ /^ff/
+      :selenium_with_firebug
+    # elsif ENV['DRIVER'] == 'chrome'
+    #   Capybara.javascript_driver = :chrome
+    else
+      :poltergeist
+    end
 end