acceptance_test 1.5.5 → 1.6.0

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    MzIxZjhmY2Y3ODZhODMyOWYwNGJmOTVmMWVkZjI0MjkxN2UyMTU0Mw==
+    NWRiYjdjNWFjMTIwNDRlNjU4M2M0ZWM4ODgyYmI3MzZkZjI2MWUyZQ==
   data.tar.gz: !binary |-
-    OTQ3YmY0MGU2Y2IyNTViNTMxNWNlZGJlZTJiYTEyODcxNWI4MjNmNQ==
+    YjczYjJkOGMxODM0ZWQ3YTY5YmIxNmQ4MmZhZTI2NmRjNmFlOWJiZA==
 SHA512:
   metadata.gz: !binary |-
-    OWE3ZGNkMTA2YTVhYzc0NDc0Y2YyODEzODQ0ZmQ2NmI5NzUwYTMyNjI1NGE4
-    YmJjMTBmNTcwNjNkMjMyYTY3N2Y0OTQ5MjU5ZTU4NmFlMmExODIzMzQ2YzBi
-    M2Q5NmJhYjE4NzhiMmU1OTJhZDBmZjc4NTk5YTFjYTEwOWI2NGY=
+    MDYxNGE0YWJiNzdiMjBhMjRhMzQyMzQ3ZWRlNTQ1ZmU2MzljZjk1Mzc1NTgw
+    MGYxN2M3YTU5ZThhZDM4NDUyMzFhZjM4OWQ2YzgyZTQ2OTNiYTNmNmY3NzZm
+    MTI2ZDlhOTY4YzM4YmVmMTM1NzE3NTJhOWQ5MTcxZWJmNDg2YmM=
   data.tar.gz: !binary |-
-    Yzc0ZWI0ZWVmNjRkOTkyZTAwN2E2MWU5ZTRlYmFhNDJkMWY4MjE2MGQ2YzJh
-    MWQ0NWU2ODQzOGUxNTA1NjBiYjc5YTA3OWFhYjUyZTljZDI5OWY0ZTM4OTli
-    NWIyN2FhNDc5ZjVhODZjOWE0NzkzNWRjZjRjMTkzN2U1NzVkM2E=
+    ZmQ2YTQ0NmU0ZDNhMjIyZjFiYmI1YTg1MjI2Y2Y5NzEzODVjMDYzZjliYjdh
+    ZTZmOWVhNzgzN2E4ZTI4MDYzNTI2NDI5NDU4ZjNjN2Y3Yzk5YWFmMDA5Y2Ix
+    YWU1NmVkNzE4MWNiMjQxNGQ2YmFiN2I4MjU4ZjcxYmNlNGExM2M=

data/CHANGES

@@ -135,4 +135,8 @@
 
 == Version 1.5.5
 
-* Bug fixes
+* Bug fixes
+
+== Version 1.6.0
+
+* Refactor the code

data/Gemfile

@@ -12,7 +12,7 @@ end
 
 group :acceptance_test do
   gem "activesupport", "~> 4.1.6"
-  gem "capybara", "~> 2.4.1"
+  gem "capybara", "~> 2.4.3"
   gem "capybara-firebug", "~> 2.1.0"
   gem "selenium-webdriver", "~> 2.43.0"
 
@@ -27,6 +27,9 @@ group :acceptance_test do
   # Debian: sudo apt-get install libqt4-dev
   # Fedora: yum install qt-webkit-devell
 
+  # for chrome support:
+  # brew install chromedriver
+
   # brew install phantomjs
   gem "poltergeist", "~> 1.5.1"
 end

data/Gemfile.lock

@@ -8,7 +8,7 @@ GEM
       thread_safe (~> 0.1)
       tzinfo (~> 1.1)
     builder (3.2.2)
-    capybara (2.4.1)
+    capybara (2.4.3)
       mime-types (>= 1.16)
       nokogiri (>= 1.3.3)
       rack (>= 1.0.0)
@@ -20,7 +20,7 @@ GEM
     capybara-webkit (1.3.0)
       capybara (>= 2.0.2, < 2.5.0)
       json
-    childprocess (0.5.3)
+    childprocess (0.5.5)
       ffi (~> 1.0, >= 1.0.11)
     cliver (0.3.2)
     cucumber (1.3.17)
@@ -31,7 +31,7 @@ GEM
       multi_test (>= 0.1.1)
     debugger-ruby_core_source (1.3.5)
     diff-lcs (1.2.5)
-    ffi (1.9.3)
+    ffi (1.9.6)
     file_utils (1.0.7)
     gemcutter (0.7.1)
     gemspec_deps_gen (1.1.2)
@@ -42,9 +42,9 @@ GEM
     i18n (0.6.11)
     json (1.8.1)
     metaclass (0.0.4)
-    mime-types (2.3)
+    mime-types (2.4.1)
     mini_portile (0.6.0)
-    minitest (5.4.1)
+    minitest (5.4.2)
     mocha (1.1.0)
       metaclass (~> 0.0.1)
     multi_json (1.10.1)
@@ -64,16 +64,16 @@ GEM
       rspec-core (~> 3.1.0)
       rspec-expectations (~> 3.1.0)
       rspec-mocks (~> 3.1.0)
-    rspec-core (3.1.3)
+    rspec-core (3.1.6)
       rspec-support (~> 3.1.0)
     rspec-example_steps (3.0.2)
       rspec-core (>= 3.0.0)
-    rspec-expectations (3.1.1)
+    rspec-expectations (3.1.2)
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.1.0)
-    rspec-mocks (3.1.0)
+    rspec-mocks (3.1.3)
       rspec-support (~> 3.1.0)
-    rspec-support (3.1.0)
+    rspec-support (3.1.2)
     ruby-debug-base19x (0.11.30.pre12)
       debugger-ruby_core_source (>= 1.1.4)
       rake (>= 0.8.1)
@@ -92,7 +92,7 @@ GEM
     tzinfo (1.2.2)
       thread_safe (~> 0.1)
     websocket (1.2.1)
-    websocket-driver (0.3.4)
+    websocket-driver (0.3.5)
     xpath (2.0.0)
       nokogiri (~> 1.3)
 
@@ -101,7 +101,7 @@ PLATFORMS
 
 DEPENDENCIES
   activesupport (~> 4.1.6)
-  capybara (~> 2.4.1)
+  capybara (~> 2.4.3)
   capybara-firebug (~> 2.1.0)
   capybara-webkit (~> 1.3.0)
   cucumber (~> 1.3.17)

data/Rakefile

@@ -40,9 +40,3 @@ end
 #   system "mkdir -p $GEM_HOME/gems/debugger-ruby_core_source-1.2.3/lib"
 #   system "cp -R ~/debugger-ruby_core_source/lib $GEM_HOME/gems/debugger-ruby_core_source-1.2.3"
 # end
-
-task :turnip do
-  ARGV.shift
-
-  system "bundle exec rspec -r turnip/rspec #{ARGV.join(' ')}"
-end

data/features/wikipedia/step_definitions/steps.rb

@@ -3,10 +3,10 @@ require 'yaml'
 
 require 'acceptance_test'
 
-require 'acceptance_test/gherkin_helper'
+require 'acceptance_test/gherkin_ext'
 
 data_reader = lambda {|source_path| CSV.read(File.expand_path(source_path)) }
-GherkinHelper.instance.enable_external_source data_reader
+GherkinExt.instance.enable_external_source data_reader
 
 acceptance_test = AcceptanceTest.instance
 

data/lib/acceptance_test/acceptance_test.rb

@@ -1,83 +1,72 @@
 require 'uri'
-require 'fileutils'
 require 'singleton'
 
-require 'capybara'
-require "capybara/dsl"
 require 'active_support/core_ext/hash'
 
-require 'acceptance_test/gherkin_helper'
-require 'acceptance_test/turnip_helper'
+require 'acceptance_test/driver_manager'
+require 'acceptance_test/gherkin_ext'
+require 'acceptance_test/turnip_ext'
 
 class AcceptanceTest
   include Singleton
 
-  attr_reader :config
+  attr_reader :config, :driver_manager
+
+  def initialize
+    @driver_manager = DriverManager.new
+  end
 
   def configure config={}
     if config
       @config = config.kind_of?(HashWithIndifferentAccess) ? config : HashWithIndifferentAccess.new(config)
     else
       @config = HashWithIndifferentAccess.new
-
-      config[:screenshot_dir] = File.expand_path('tmp')
     end
 
-    set_app_host
+    @config[:browser] = 'firefox' unless @config[:browser]
+    @config[:screenshot_dir] = File.expand_path('tmp') unless @config[:screenshot_dir]
+  end
+
+  def setup
+    Capybara.app_host = AcceptanceTest.instance.config[:webapp_url]
 
     Capybara.configure do |conf|
       conf.default_wait_time = timeout_in_seconds
-      conf.match = :first
-
-      conf.ignore_hidden_elements = false
     end
 
-    ENV['APP_HOST'] ||= config[:webapp_url]
     ENV['WAIT_TIME'] ||= Capybara.default_wait_time.to_s
 
-    # try to load capybara-related rspec library
-    begin
-      require 'capybara/rspec'
-
-      RSpec.configure do |config|
-        config.filter_run_excluding :exclude => true
-      end
+    Capybara.default_driver = :selenium
+  end
 
-      RSpec.configure do |config|
-        config.include Capybara::DSL
-      end
+  def teardown
+    Capybara.app_host = nil
 
-      RSpec::Core::ExampleGroup.send :include, Capybara::DSL
-    rescue
-      ;
+    Capybara.configure do |conf|
+      conf.default_wait_time = 5
     end
 
-    Capybara.default_driver = :selenium
-  end
-
-  def set_app_host
-    Capybara.app_host = AcceptanceTest.instance.config[:webapp_url]
+    Capybara.default_driver = :rack_test
   end
 
-  def enable_external_source data_reader
-    GherkinHelper.instance.enable_external_source data_reader
-  end
+  def before_test metadata={}, page=nil
+    setup unless Capybara.app_host
 
-  def extend_turnip
-    TurnipHelper.instance.extend_turnip
-  end
+    tag = driver(metadata)
 
-  def before metadata={}
-    driver = driver(metadata)
+    if tag
+      driver_name = driver_manager.register_driver tag, config[:browser].to_sym, config[:selenium_url]
 
-    if driver
-      register_driver driver
+      if driver_name and Capybara.drivers[driver_name]
+        Capybara.current_driver = driver_name
+        Capybara.javascript_driver = driver_name
 
-      select_driver driver
+        page.instance_variable_set(:@mode, driver_name) if page
+      end
     end
   end
 
-  def after metadata={}, exception=nil, page=nil
+  def after_test metadata={}, exception=nil, page=nil
     driver = driver(metadata)
 
     if driver and exception and page and not [:webkit].include? driver
@@ -94,6 +83,7 @@ class AcceptanceTest
     end
 
     Capybara.current_driver = Capybara.default_driver
+    Capybara.javascript_driver = Capybara.default_driver
   end
 
   def create_shared_context name
@@ -120,7 +110,7 @@ class AcceptanceTest
     if tags.size > 0
       tag = tags.first.to_sym
 
-      if AcceptanceTest.supported_drivers.include? tag
+      if driver_manager.supported_drivers.include? tag
         metadata[:driver] = tag
       end
     end
@@ -128,12 +118,24 @@ class AcceptanceTest
     metadata
   end
 
+  def extend_turnip
+    shared_context_name = "#{random_name}AcceptanceTest"
+
+    create_shared_context shared_context_name
+
+    TurnipExt.shared_context_with_turnip shared_context_name
+  end
+
+  def enable_external_source data_reader
+    GherkinExt.enable_external_source data_reader
+  end
+
   def driver metadata
     driver = ENV['DRIVER'].nil? ? nil : ENV['DRIVER'].to_sym
 
     driver = metadata[:driver] if driver.nil?
 
-    self.class.supported_drivers.each do |supported_driver|
+    driver_manager.supported_drivers.each do |supported_driver|
       driver = supported_driver if metadata[supported_driver]
       break if driver
     end
@@ -143,13 +145,9 @@ class AcceptanceTest
     driver
   end
 
-  def selenium_driver? driver
-    driver.to_s =~ /selenium/
-  end
-
-  def self.supported_drivers
-    [:webkit, :selenium, :poltergeist, :selenium_remote]
-  end
+  # def selenium_driver? driver
+  #   driver.to_s =~ /selenium/
+  # end
 
   def configure_rspec object=nil
     acceptance_test = self
@@ -165,11 +163,11 @@ class AcceptanceTest
     end
 
     rspec_conf.around(:each) do |example|
-      acceptance_test.before(example.metadata)
+      acceptance_test.before_test(example.metadata, page)
 
       example.run
 
-      acceptance_test.after(example.metadata, example.exception, page)
+      acceptance_test.after_test(example.metadata, example.exception, page)
     end
   end
 
@@ -187,92 +185,8 @@ class AcceptanceTest
     end
   end
 
-  def register_driver driver
-    case driver
-      when :webkit
-        require "capybara-webkit"
-
-      when :selenium
-        # nothing
-
-      when :selenium_with_firebug
-        require 'capybara/firebug'
-
-      #Capybara.register_driver :selenium_with_firebug do |app|
-      #  profile = Selenium::WebDriver::Firefox::Profile.new
-      #  profile.enable_firebug
-      #  Capybara::Selenium::Driver.new(app, :browser => :firefox, :profile => profile)
-      #end
-
-      #Selenium::WebDriver::Firefox::Profile.firebug_version = '1.11.2'
-      when :selenium_chrome
-        unless Capybara.drivers[:selenium_chrome]
-          Capybara.register_driver :selenium_chrome do |app|
-            Capybara::Selenium::Driver.new(app, :browser => :chrome)
-          end
-        end
-
-      when :poltergeist
-        require 'capybara/poltergeist'
-
-        unless Capybara.drivers[:poltergeist]
-          Capybara.register_driver :poltergeist do |app|
-            Capybara::Poltergeist::Driver.new(app, { debug: false })
-          end
-        end
-
-      when :selenium_remote
-        unless Capybara.drivers[:selenium_remote]
-          Capybara.register_driver :selenium_remote do |app|
-            Capybara::Selenium::Driver.new(app, {:browser => :remote, :url => config[:selenium_url]})
-
-            #profile = Selenium::WebDriver::Firefox::Profile.new
-            #profile.enable_firebug
-            #
-            #Capybara::Driver::Selenium.new(app, {
-            #  :browser => :remote,
-            #  :url => selenium_url,
-            #  :desired_capabilities => Selenium::WebDriver::Remote::Capabilities.firefox(:firefox_profile => profile)
-            #})
-          end
-        end
-      when :selenium_safari
-        unless Capybara.drivers[:selenium_safari]
-          Capybara.register_driver :selenium_safari do |app|
-            Capybara::Selenium::Driver.new(app, :browser => :safari)
-          end
-        end
-      else
-        # nothing
-    end
-  end
-
-  def select_driver driver
-    if selenium_driver?(driver)
-      if driver == :selenium_remote
-        Capybara.current_driver = driver
-        Capybara.javascript_driver = driver
-      else
-        if acceptance_config_exist?
-          Capybara.current_driver = driver
-          Capybara.javascript_driver = driver
-        else
-          if Capybara.drivers[driver]
-            Capybara.current_driver = driver
-            Capybara.javascript_driver = driver
-          end
-        end
-      end
-    else
-      if Capybara.drivers[driver]
-        Capybara.current_driver = driver
-        Capybara.javascript_driver = driver
-      end
-    end
-  end
-
-  def acceptance_config_exist?
-    not config.nil?
+  def random_name
+    ('a'..'z').to_a.shuffle[0, 12].join
   end
 
   # def self.get_localhost
@@ -285,4 +199,7 @@ class AcceptanceTest
   # ensure
   #   Socket.do_not_reverse_lookup = orig
   # end
+
+  # ip = `ifconfig | grep 'inet ' | grep -v 127.0.0.1 | cut -d ' ' -f2`.strip
+  # Capybara.app_host = http://#{ip}:#{Capybara.server_port}
 end

data/lib/acceptance_test/driver_manager.rb

@@ -0,0 +1,138 @@
+require 'capybara'
+#require "capybara/dsl"
+
+class DriverManager
+
+  def initialize
+    Capybara.configure do |conf|
+      #conf.default_wait_time = timeout_in_seconds
+      conf.match = :first
+
+      conf.ignore_hidden_elements = false
+    end
+
+    # try to load capybara-related rspec library
+    begin
+      require 'capybara/rspec'
+
+      RSpec.configure do |conf|
+        conf.filter_run_excluding :exclude => true
+      end
+
+      RSpec.configure do |conf|
+        conf.include Capybara::DSL
+      end
+
+      RSpec::Core::ExampleGroup.send :include, Capybara::DSL
+    rescue
+      ;
+    end
+  end
+
+  def supported_drivers
+    [:selenium, :selenium_remote, :webkit, :poltergeist]
+  end
+
+  def register_driver tag, browser, selenium_url
+    driver_name, properties = *recognize_driver(tag, browser)
+
+    if driver_name
+      unless Capybara.drivers[driver_name]
+        if driver_name == :poltergeist
+          Capybara.register_driver :poltergeist do |app|
+            Capybara::Poltergeist::Driver.new(app, { debug: false })
+          end
+        else
+          if driver_name =~ /remote/
+            properties[:browser] = :remote
+            properties[:url] = selenium_url
+            properties[:desired_capabilities] = capabilities if capabilities
+
+            Capybara.register_driver driver_name do |app|
+              Capybara::Selenium::Driver.new(app, properties)
+            end
+          else
+            properties[:browser] = browser
+
+            Capybara.register_driver driver_name do |app|
+              Capybara::Selenium::Driver.new(app, properties)
+            end
+          end
+        end
+      end
+    end
+
+    driver_name
+  end
+
+  private
+
+  def recognize_driver tag, browser
+    properties = {}
+
+    driver_name =
+        case tag
+          when :webkit
+            require "capybara-webkit"
+
+            :webkit
+
+          when :selenium
+
+            case browser
+              when :firefox
+                :selenium_firefox
+
+              when :firefox_with_firebug
+                require 'capybara/firebug'
+
+                :selenium_firefox_with_firebug
+
+              when :chrome
+                :selenium_chrome
+
+              when :safari
+                :selenium_safari
+
+              when :ie, :internet_explorer
+                :selenium_ie
+
+              else
+                :unsupported
+            end
+
+          when :poltergeist
+            require 'capybara/poltergeist'
+
+            :poltergeist
+
+          when :selenium_remote
+            case browser
+              when :firefox
+                :selenium_remote_firefox
+
+              when :firefox_with_firebug
+                require 'capybara/firebug'
+
+                profile = Selenium::WebDriver::Firefox::Profile.new
+                profile.enable_firebug
+
+                properties[:desired_capabilities] = Selenium::WebDriver::Remote::Capabilities.firefox(:firefox_profile => profile)
+
+                :selenium_remote_firefox_with_firebug
+
+              when :ie
+                properties[:desired_capabilities] = Selenium::WebDriver::Remote::Capabilities.internet_explorer
+
+                :selenium_remote_ie
+              else
+                :unsupported
+            end
+          else
+            :unsupported
+        end
+
+    [driver_name, properties]
+  end
+
+end