cucumber-rails 1.6.0 → 2.2.0

data/dev_tasks/yard.rake

@@ -1,14 +1,15 @@
+# frozen_string_literal: true
+
 require 'yard'
 require 'yard/rake/yardoc_task'
 
-SITE_DIR = File.expand_path(File.dirname(__FILE__) + '/../cucumber.github.com')
+SITE_DIR = File.expand_path("#{File.dirname(__FILE__)}/../cucumber.github.com")
 API_DIR = File.join(SITE_DIR, 'api', 'cucumber-rails', 'ruby', 'yardoc')
 
 namespace :api do
   file :dir do
-    unless File.directory?(SITE_DIR)
-      raise "You need to git clone git@github.com:cucumber/cucumber.github.com.git #{SITE_DIR}"
-    end
+    raise "You need to git clone git@github.com:cucumber/cucumber.github.com.git #{SITE_DIR}" unless File.directory?(SITE_DIR)
+
     sh('git pull origin master')
     mkdir_p API_DIR
   end
@@ -18,18 +19,10 @@ namespace :api do
   YARD::Rake::YardocTask.new(:yard) do |yard|
     dir = API_DIR
     mkdir_p dir
-    yard.options = ["--out", dir]
+    yard.options = ['--out', dir]
   end
   task yard: :dir
 
-  task :release do
-    Dir.chdir(SITE_DIR) do
-      sh('git add .')
-      sh("git commit -m 'Update API docs for cucumber-rails v#{cucumber-rails::VERSION}'")
-      sh('git push origin master')
-    end
-  end
-
   desc "Generate YARD docs for Cucumber-Rails' API"
-  task doc: [:yard, :release]
+  task doc: :yard
 end

data/dev_tasks/yard/default/layout/html/setup.rb

@@ -1,4 +1,9 @@
+# frozen_string_literal: true
+
 def init
   super
-  options[:serializer].serialize('/images/bubble_32x32.png', IO.read(File.dirname(__FILE__) + '/bubble_32x32.png'))
+  options[:serializer].serialize(
+    '/images/bubble_32x32.png',
+    IO.read("#{File.dirname(__FILE__)}/bubble_32x32.png")
+  )
 end

data/features/allow_rescue.feature

@@ -1,54 +1,59 @@
 Feature: Allow Cucumber to rescue exceptions
 
   Background: A controller that raises an exception
-    Given I have created a new Rails app "rails-3-app" and installed cucumber-rails
-    And I write to "app/controllers/posts_controller.rb" with:
+    Given I have created a new Rails app and installed cucumber-rails
+    When I write to "app/controllers/posts_controller.rb" with:
       """
       class PostsController < ApplicationController
         def index
-          raise "There is an error in index"
+          raise 'There is an error in index'
         end
       end
       """
     And I write to "config/routes.rb" with:
       """
-      Rails3App::Application.routes.draw do
+      TestApp::Application.routes.draw do
         resources :posts
       end
       """
 
   Scenario: Allow rescue
-    Given I write to "features/posts.feature" with:
+    When I write to "features/posts.feature" with:
       """
-      Feature: posts
+      Feature: Posts
         @allow-rescue
         Scenario: See posts
           When I look at the posts
+          Then I should see the public error page
       """
     And I write to "features/step_definitions/posts_steps.rb" with:
       """
-      When /^I look at the posts$/ do
+      When('I look at the posts') do
         visit '/posts'
       end
+
+      Then('I should see the public error page') do
+        expect(page).to have_content "We're sorry, but something went wrong."
+      end
       """
     And I run `bundle exec rake db:migrate`
     And I run `bundle exec cucumber`
     Then the feature run should pass with:
       """
       1 scenario (1 passed)
-      1 step (1 passed)
+      2 steps (2 passed)
       """
 
   Scenario: Don't allow rescue
-    Given I write to "features/posts.feature" with:
+    When I write to "features/posts.feature" with:
       """
-      Feature: posts
+      Feature: Posts
         Scenario: See them
           When I look at the posts
       """
     And I write to "features/step_definitions/posts_steps.rb" with:
       """
-      When /^I look at the posts$/ do
+      When('I look at the posts') do
         visit '/posts'
       end
       """

data/features/annotations.feature

@@ -1,23 +1,20 @@
-@requires-rails-version-42
 Feature: Annotations
 
-  I order to track my development progress
+  In order to track my development progress
   As a developer
   I should be able to list annotations in my features
 
-  Background:
-    Given I have created a new Rails app "test-app" with no database and installed cucumber-rails
-    And I write to "features/tests.feature" with:
+  Scenario: See annotations in feature file
+    Given I have created a new Rails app with no database and installed cucumber-rails
+    When I write to "features/products.feature" with:
       """
-      Feature: Tests
-        Scenario: Tests
+      Feature: Products
+        Scenario: Test a Product
           # TODO: When I go to the products page
       """
-
-  Scenario: See annotations in .feature file
     When I run `bundle exec rake notes`
     Then it should pass with:
       """
-      features/tests.feature:
-        * [ 3] [TODO] When I go to the products page
+      features/products.feature:
+        * [3] [TODO] When I go to the products page
       """

data/features/capybara_javascript_drivers.feature

@@ -1,40 +1,44 @@
-@focus
 Feature: Capybara Javascript Drivers
 
-  Background: A simple calendar app
+  Scenario: Use a particular driver
     Given I have created a new Rails app and installed cucumber-rails
     And I force selenium to run Firefox in headless mode
-    And I run `bundle exec rails g scaffold appointment name:string when:datetime`
-    And I write to "features/step_definitions/date_time_steps.rb" with:
-      """
-      When /^(?:|I )select "([^"]+)" as the "([^"]+)" time$/ do |time, selector|
-        select_time(time, from: selector)
-      end
-
-      When /^(?:|I )select "([^"]+)" as the "([^"]+)" date$/ do |date, selector|
-        select_date(date, from: selector)
-      end
-
-      When /^(?:|I )select "([^"]+)" as the "([^"]+)" date and time$/ do |datetime, selector|
-        select_datetime(datetime, from: selector)
-      end
-      """
-
-  Scenario: Use a particular driver
-    Given I write to "features/create_appointment.feature" with:
+    When I run `bundle exec rails g scaffold appointment name:string when:datetime`
+    And I write to "features/create_appointment.feature" with:
       """
       @javascript
       Feature: Create appointments
-        Scenario: Constitution on May 17
+        Scenario: Visit the Constitution on May 17
           Given I am on the new appointment page
           And I fill in "Norway's constitution" for "Name"
           And I select "2015-02-20 15:10:00 UTC" as the "When" date and time
           And I press "Create Appointment"
           Then I should see "Norway's constitution"
           And I should see "2015-02-20 15:10:00 UTC"
+      """
+    And I write to "features/create_appointment_steps.rb" with:
+      """
+      Given('I am on the new appointment page') do
+        visit new_appointment_path
+      end
+
+      When('I fill in {string} for {string}') do |value, field|
+        fill_in(field, with: value)
+      end
+
+      When('I press {string}') do |button|
+        click_button(button)
+      end
+
+      Then('I should see {string}') do |text|
+        expect(page).to have_content(text)
+      end
 
+      When('I select {string} as the {string} date and time') do |datetime, selector|
+        select_datetime(datetime, from: selector)
+      end
       """
-    When I run `bundle exec rake db:migrate`
+    And I run `bundle exec rake db:migrate`
     And I run `bundle exec rake cucumber`
     Then the feature run should pass with:
       """
@@ -43,31 +47,86 @@ Feature: Capybara Javascript Drivers
       """
 
   Scenario: Mixed DB access
-    Given I write to "features/create_appointment.feature" with:
+    Given I have created a new Rails app and installed cucumber-rails
+    And I force selenium to run Firefox in headless mode
+    When I run `bundle exec rails g scaffold appointment name:string when:datetime`
+    And I write to "features/create_appointment.feature" with:
       """
       @javascript
       Feature: Create appointments
-        Scenario: Constitution on May 17
+        Scenario: Visit the Constitution on May 17
           Given a random appointment
-          And I am viewing random appointment
+          And I am viewing the appointment
           Then I should see "Random appointment"
-
       """
-    And I write to "features/step_definitions/custom_steps.rb" with:
+    And I write to "features/step_definitions/create_appointment_steps.rb" with:
       """
-      Given /^a random appointment$/ do
+      Given('a random appointment') do
         @appointment = Appointment.create!(name: 'Random appointment', when: DateTime.now)
       end
 
-      Given /^I am viewing random appointment$/ do
+      Given('I am viewing the appointment') do
         visit appointment_path(@appointment)
       end
-      """
 
-    When I run `bundle exec rake db:migrate`
+      Then('I should see {string}') do |text|
+        expect(page).to have_content(text)
+      end
+      """
+    And I run `bundle exec rake db:migrate`
     And I run `bundle exec rake cucumber`
     Then the feature run should pass with:
       """
       1 scenario (1 passed)
       3 steps (3 passed)
       """
+
+  Scenario: Use a particular driver without a DB
+    Given I have created a new Rails app and installed cucumber-rails without database_cleaner
+    And I force selenium to run Firefox in headless mode
+    When I run `bundle exec rails g scaffold appointment name:string when:datetime`
+    And I write to "features/create_appointment.feature" with:
+      """
+      @javascript
+      Feature: Create appointments
+        Scenario: Visit the Constitution on May 17
+          Given I am on the new appointment page
+          And I fill in "Norway's constitution" for "Name"
+          And I select "2015-02-20 15:10:00 UTC" as the "When" date and time
+          And I press "Create Appointment"
+          Then I should see "Norway's constitution"
+          And I should see "2015-02-20 15:10:00 UTC"
+      """
+    And I write to "features/create_appointment_steps.rb" with:
+      """
+      Given('I am on the new appointment page') do
+        visit new_appointment_path
+      end
+
+      When('I fill in {string} for {string}') do |value, field|
+        fill_in(field, with: value)
+      end
+
+      When('I press {string}') do |button|
+        click_button(button)
+      end
+
+      Then('I should see {string}') do |text|
+        expect(page).to have_content(text)
+      end
+
+      When('I select {string} as the {string} date and time') do |datetime, selector|
+        select_datetime(datetime, from: selector)
+      end
+      """
+    And I append to "features/support/env.rb" with:
+      """
+        Cucumber::Rails::Database.autorun_database_cleaner = false
+      """
+    And I run `bundle exec rake db:migrate`
+    And I run `bundle exec rake cucumber`
+    Then the feature run should pass with:
+      """
+      1 scenario (1 passed)
+      6 steps (6 passed)
+      """

data/features/choose_javascript_database_strategy.feature

@@ -1,6 +1,6 @@
 Feature: Choose javascript database strategy
 
-  When running a scenario with the @javascript tag, Capybara will fire up a web server
+  Whilst running a scenario with the @javascript tag, Capybara will fire up a web server
   in the same process in a separate thread to your cukes. By default, this means ActiveRecord will give it a
   separate database connection, which in turn means data you put into your database from
   Cucumber step definitions (e.g. using FactoryBot) won't be visible to the web server
@@ -26,9 +26,23 @@ Feature: Choose javascript database strategy
   Background:
     Given I have created a new Rails app and installed cucumber-rails
     And I have a "Widget" ActiveRecord model object
+    When I write to "features/step_definitions/widget_steps.rb" with:
+      """
+      When('I create {int} widgets') do |number|
+        number.times { Widget.create! }
+      end
+
+      Then('I should have {int} widgets') do |number|
+        expect(Widget.count).to eq(number)
+      end
+
+      Then('the DatabaseCleaner strategy should be {word}') do |strategy_name|
+        expect(DatabaseCleaner.cleaners.values.first.strategy.to_s).to match(/#{strategy_name}/i)
+      end
+      """
 
   Scenario: Set the strategy to truncation and run a javascript scenario.
-    Given I append to "features/env.rb" with:
+    When I append to "features/env.rb" with:
       """
       DatabaseCleaner.strategy = :transaction
       Cucumber::Rails::Database.javascript_strategy = :truncation
@@ -37,13 +51,13 @@ Feature: Choose javascript database strategy
       """
       Feature:
         Background:
-          Given I have created 2 widgets
+          When I create 2 widgets
 
         @javascript
         Scenario:
-          Then the DatabaseCleaner strategy should be truncation
           When I create 3 widgets
-          Then I should have 5 widgets
+          Then the DatabaseCleaner strategy should be truncation
+          And I should have 5 widgets
 
         @javascript
         Scenario:
@@ -54,29 +68,15 @@ Feature: Choose javascript database strategy
           Then the DatabaseCleaner strategy should be transaction
           And I should have 2 widgets
       """
-    And I write to "features/step_definitions/widget_steps.rb" with:
+    And I run the cukes
+    Then the feature run should pass with:
       """
-      Given /created? (\d) widgets/ do |num|
-        num.to_i.times { Widget.create! }
-      end
-
-      Then /should have (\d) widgets/ do |num|
-        Widget.count.should == num.to_i
-      end
-
-     Then /^the DatabaseCleaner strategy should be (\w+)$/ do |strategy_name|
-        DatabaseCleaner.connections.first.strategy.to_s.should =~ /#{strategy_name}/i
-      end
+      3 scenarios (3 passed)
+      10 steps (10 passed)
       """
-    When I run the cukes
-    Then the feature run should pass with:
-       """
-       3 scenarios (3 passed)
-       10 steps (10 passed)
-       """
 
- Scenario: Set the strategy to deletion and run a javascript scenario.
-    Given I append to "features/env.rb" with:
+  Scenario: Set the strategy to deletion and run a javascript scenario.
+    When I append to "features/env.rb" with:
       """
       Cucumber::Rails::Database.javascript_strategy = :deletion
       """
@@ -85,7 +85,7 @@ Feature: Choose javascript database strategy
       @javascript
       Feature:
         Background:
-          Given I have created 2 widgets
+          When I create 2 widgets
 
         Scenario:
           When I create 3 widgets
@@ -94,27 +94,17 @@ Feature: Choose javascript database strategy
         Scenario:
           Then I should have 2 widgets
       """
-    And I write to "features/step_definitions/widget_steps.rb" with:
+    And I run the cukes
+    Then the feature run should pass with:
       """
-      Given /created? (\d) widgets/ do |num|
-        num.to_i.times { Widget.create! }
-      end
-
-      Then /should have (\d) widgets/ do |num|
-        Widget.count.should == num.to_i
-      end
+      2 scenarios (2 passed)
+      5 steps (5 passed)
       """
-    When I run the cukes
-    Then the feature run should pass with:
-       """
-       2 scenarios (2 passed)
-       5 steps (5 passed)
-     """
 
   Scenario: Set the strategy to truncation with an except option and run a javascript scenario.
-    Given I append to "features/env.rb" with:
+    When I append to "features/env.rb" with:
       """
-     Cucumber::Rails::Database.javascript_strategy = :truncation, {:except=>%w[widgets]}
+      Cucumber::Rails::Database.javascript_strategy = :truncation, { except: %w[widgets] }
       """
     And I write to "features/widgets.feature" with:
       """
@@ -127,19 +117,9 @@ Feature: Choose javascript database strategy
         Scenario:
           Then I should have 3 widgets
       """
-    And I write to "features/step_definitions/widget_steps.rb" with:
+    And I run the cukes
+    Then the feature run should pass with:
       """
-      Given /created? (\d) widgets/ do |num|
-        num.to_i.times { Widget.create! }
-      end
-
-      Then /should have (\d) widgets/ do |num|
-        Widget.count.should == num.to_i
-      end
+      2 scenarios (2 passed)
+      3 steps (3 passed)
       """
-    When I run the cukes
-    Then the feature run should pass with:
-       """
-       2 scenarios (2 passed)
-       3 steps (3 passed)
-       """