Upgraded to pickle 0.2.0, cucumber 0.4.4 and fixed an inaccurate scenario step

Author: Michael MacDonald

README

@@ -2,9 +2,14 @@
 
 This is a very basic and contrived example designed to demonstrate the benefits of using Ian White's "Pickle gem":http://github.com/ianwhite/pickle with Cucumber to improve the way you BDD.
 
+== Requirements
+
+cucumber >= 0.4.4
+pickle   >= 0.2.0
+
 == Getting Started
 
-1. rake gems:install
+1. rake gems:install RAILS_ENV=cucumber
 2. rake db:migrate
 3. rake db:test:prepare
 4. cucumber

config/environments/cucumber.rb

@@ -1,4 +1,4 @@
-# IMPORTANT: This file was generated by Cucumber 0.4.2
+# IMPORTANT: This file was generated by Cucumber 0.4.4
 # Edit at your own peril - it's recommended to regenerate this file
 # in the future when you upgrade to a newer version of Cucumber.
 
@@ -19,9 +19,8 @@
 # ActionMailer::Base.deliveries array.
 config.action_mailer.delivery_method = :test
 
-config.gem 'cucumber',    :lib => false,        :version => '>=0.4.2' unless File.directory?(File.join(Rails.root, 'vendor/plugins/cucumber'))
-config.gem 'webrat',      :lib => false,        :version => '>=0.5.0' unless File.directory?(File.join(Rails.root, 'vendor/plugins/webrat'))
-config.gem 'rspec',       :lib => false,        :version => '>=1.2.8' unless File.directory?(File.join(Rails.root, 'vendor/plugins/rspec'))
-config.gem 'rspec-rails', :lib => false,        :version => '>=1.2.7.1' unless File.directory?(File.join(Rails.root, 'vendor/plugins/rspec-rails'))
-
-config.gem 'pickle'
+config.gem 'cucumber',    :lib => false,        :version => '>=0.4.4' unless File.directory?(File.join(Rails.root, 'vendor/plugins/cucumber'))
+config.gem 'webrat',      :lib => false,        :version => '>=0.5.3' unless File.directory?(File.join(Rails.root, 'vendor/plugins/webrat'))
+config.gem 'rspec',       :lib => false,        :version => '>=1.2.9' unless File.directory?(File.join(Rails.root, 'vendor/plugins/rspec'))
+config.gem 'rspec-rails', :lib => false,        :version => '>=1.2.9' unless File.directory?(File.join(Rails.root, 'vendor/plugins/rspec-rails'))
+config.gem 'pickle', :version => '>=0.2.0' unless File.directory?(File.join(Rails.root, 'vendor/plugins/pickle'))

features/person.feature

@@ -16,7 +16,7 @@ Feature: Pickle demonstration with Person objects
     Given a company exists
     And another company: "rubyflare" exists 
     And a person exists with company: company "rubyflare"
-    And another person exists with company: the company
+    And another person exists with company: the first company
     Then the first person should be one of company: "rubyflare"'s employees
 
   Scenario: email

features/step_definitions/email_steps.rb

@@ -1,4 +1,6 @@
 # this file generated by script/generate pickle email
+#
+# add email mappings in features/support/email.rb
 
 ActionMailer::Base.delivery_method = :test
 ActionMailer::Base.perform_deliveries = true
@@ -7,6 +9,7 @@
   ActionMailer::Base.deliveries.clear
 end
 
+# Clear the deliveries array, useful if your background sends email that you want to ignore
 Given(/^all emails? (?:have|has) been delivered$/) do
   ActionMailer::Base.deliveries.clear
 end
@@ -16,17 +19,15 @@
 end
 
 Then(/^(\d)+ emails? should be delivered to (.*)$/) do |count, to|
-  to =~ /^#{capture_model}$/ && to = model($1).email
-  emails("to: \"#{to}\"").size.should == count.to_i
+  emails("to: \"#{email_for(to)}\"").size.should == count.to_i
 end
 
 Then(/^(\d)+ emails? should be delivered with #{capture_fields}$/) do |count, fields|
   emails(fields).size.should == count.to_i
 end
 
 Then(/^#{capture_email} should be delivered to (.+)$/) do |email_ref, to|
-  to =~ /^#{capture_model}$/ && to = model($1).email
-  email(email_ref, "to: \"#{to}\"").should_not be_nil
+  email(email_ref, "to: \"#{email_for(to)}\"").should_not be_nil
 end
 
 Then(/^#{capture_email} should have #{capture_fields}$/) do |email_ref, fields|

features/step_definitions/pickle_steps.rb

@@ -11,37 +11,63 @@
 end
 
 # create models from a table
-Given /^the following #{capture_plural_factory} exist$/ do |plural_factory, table|
+Given(/^the following #{capture_plural_factory} exists?:?$/) do |plural_factory, table|
   name = plural_factory.singularize
   table.hashes.each { |hash| create_model(name, hash) }
 end
 
 # find a model
 Then(/^#{capture_model} should exist(?: with #{capture_fields})?$/) do |name, fields|
-  find_model(name, fields).should_not be_nil
+  find_model!(name, fields)
+end
+
+# not find a model
+Then(/^#{capture_model} should not exist(?: with #{capture_fields})?$/) do |name, fields|
+  find_model(name, fields).should be_nil
+end
+
+# find models with a table
+Then(/^the following #{capture_plural_factory} should exists?:?$/) do |plural_factory, table|
+  name = plural_factory.singularize
+  table.hashes.each { |hash| find_model!(name, hash)}
 end
 
 # find exactly n models
 Then(/^(\d+) #{capture_plural_factory} should exist(?: with #{capture_fields})?$/) do |count, plural_factory, fields|
   find_models(plural_factory.singularize, fields).size.should == count.to_i
 end
 
+# assert equality of models
+Then(/^#{capture_model} should be #{capture_model}$/) do |a, b|
+  model!(a).should == model!(b)
+end
+
 # assert model is in another model's has_many assoc
-Then(/^#{capture_model} should be (?:in|one of|amongst) #{capture_model}'s (\w+)$/) do |target, owner, association|
-  model(owner).send(association).should include(model(target))
+Then(/^#{capture_model} should be (?:in|one of|amongst) #{capture_model}(?:'s)? (\w+)$/) do |target, owner, association|
+  model!(owner).send(association).should include(model!(target))
+end
+
+# assert model is not in another model's has_many assoc
+Then(/^#{capture_model} should not be (?:in|one of|amongst) #{capture_model}(?:'s)? (\w+)$/) do |target, owner, association|
+  model!(owner).send(association).should_not include(model!(target))
 end
 
 # assert model is another model's has_one/belongs_to assoc
-Then(/^#{capture_model} should be #{capture_model}'s (\w+)$/) do |target, owner, association|
-  model(owner).send(association).should == model(target)
+Then(/^#{capture_model} should be #{capture_model}(?:'s)? (\w+)$/) do |target, owner, association|
+  model!(owner).send(association).should == model!(target)
+end
+
+# assert model is not another model's has_one/belongs_to assoc
+Then(/^#{capture_model} should not be #{capture_model}(?:'s)? (\w+)$/) do |target, owner, association|
+  model!(owner).send(association).should_not == model!(target)
 end
 
 # assert model.predicate? 
 Then(/^#{capture_model} should (?:be|have) (?:an? )?#{capture_predicate}$/) do |name, predicate|
-  model(name).should send("be_#{predicate.gsub(' ', '_')}")
+  model!(name).should send("be_#{predicate.gsub(' ', '_')}")
 end
 
 # assert not model.predicate?
 Then(/^#{capture_model} should not (?:be|have) (?:an? )?#{capture_predicate}$/) do |name, predicate|
-  model(name).should_not send("be_#{predicate.gsub(' ', '_')}")
+  model!(name).should_not send("be_#{predicate.gsub(' ', '_')}")
 end

features/step_definitions/webrat_steps.rb

@@ -1,4 +1,4 @@
-# IMPORTANT: This file was generated by Cucumber 0.4.2
+# IMPORTANT: This file was generated by Cucumber 0.4.4
 # Edit at your own peril - it's recommended to regenerate this file
 # in the future when you upgrade to a newer version of Cucumber.
 # Consider adding your own code to a new file instead of editing this one.
@@ -8,31 +8,31 @@
 # Commonly used webrat steps
 # http://github.com/brynary/webrat
 
-Given /^I am on (.+)$/ do |page_name|
+Given /^(?:|I )am on (.+)$/ do |page_name|
   visit path_to(page_name)
 end
 
-When /^I go to (.+)$/ do |page_name|
+When /^(?:|I )go to (.+)$/ do |page_name|
   visit path_to(page_name)
 end
 
-When /^I press "([^\"]*)"$/ do |button|
+When /^(?:|I )press "([^\"]*)"$/ do |button|
   click_button(button)
 end
 
-When /^I follow "([^\"]*)"$/ do |link|
+When /^(?:|I )follow "([^\"]*)"$/ do |link|
   click_link(link)
 end
 
-When /^I follow "([^\"]*)" within "([^\"]*)"$/ do |link, parent|
+When /^(?:|I )follow "([^\"]*)" within "([^\"]*)"$/ do |link, parent|
   click_link_within(parent, link)
 end
 
-When /^I fill in "([^\"]*)" with "([^\"]*)"$/ do |field, value|
+When /^(?:|I )fill in "([^\"]*)" with "([^\"]*)"$/ do |field, value|
   fill_in(field, :with => value)
 end
 
-When /^I fill in "([^\"]*)" for "([^\"]*)"$/ do |value, field|
+When /^(?:|I )fill in "([^\"]*)" for "([^\"]*)"$/ do |value, field|
   fill_in(field, :with => value)
 end
 
@@ -47,19 +47,19 @@
 # TODO: Add support for checkbox, select og option
 # based on naming conventions.
 #
-When /^I fill in the following:$/ do |fields|
+When /^(?:|I )fill in the following:$/ do |fields|
   fields.rows_hash.each do |name, value|
     When %{I fill in "#{name}" with "#{value}"}
   end
 end
 
-When /^I select "([^\"]*)" from "([^\"]*)"$/ do |value, field|
+When /^(?:|I )select "([^\"]*)" from "([^\"]*)"$/ do |value, field|
   select(value, :from => field)
 end
 
 # Use this step in conjunction with Rail's datetime_select helper. For example:
 # When I select "December 25, 2008 10:00" as the date and time
-When /^I select "([^\"]*)" as the date and time$/ do |time|
+When /^(?:|I )select "([^\"]*)" as the date and time$/ do |time|
   select_datetime(time)
 end
 
@@ -72,92 +72,92 @@
 # The following steps would fill out the form:
 # When I select "November 23, 2004 11:20" as the "Preferred" date and time
 # And I select "November 25, 2004 10:30" as the "Alternative" date and time
-When /^I select "([^\"]*)" as the "([^\"]*)" date and time$/ do |datetime, datetime_label|
+When /^(?:|I )select "([^\"]*)" as the "([^\"]*)" date and time$/ do |datetime, datetime_label|
   select_datetime(datetime, :from => datetime_label)
 end
 
 # Use this step in conjunction with Rail's time_select helper. For example:
 # When I select "2:20PM" as the time
 # Note: Rail's default time helper provides 24-hour time-- not 12 hour time. Webrat
 # will convert the 2:20PM to 14:20 and then select it.
-When /^I select "([^\"]*)" as the time$/ do |time|
+When /^(?:|I )select "([^\"]*)" as the time$/ do |time|
   select_time(time)
 end
 
 # Use this step when using multiple time_select helpers on a page or you want to
 # specify the name of the time on the form.  For example:
 # When I select "7:30AM" as the "Gym" time
-When /^I select "([^\"]*)" as the "([^\"]*)" time$/ do |time, time_label|
+When /^(?:|I )select "([^\"]*)" as the "([^\"]*)" time$/ do |time, time_label|
   select_time(time, :from => time_label)
 end
 
 # Use this step in conjunction with Rail's date_select helper.  For example:
 # When I select "February 20, 1981" as the date
-When /^I select "([^\"]*)" as the date$/ do |date|
+When /^(?:|I )select "([^\"]*)" as the date$/ do |date|
   select_date(date)
 end
 
 # Use this step when using multiple date_select helpers on one page or
 # you want to specify the name of the date on the form. For example:
 # When I select "April 26, 1982" as the "Date of Birth" date
-When /^I select "([^\"]*)" as the "([^\"]*)" date$/ do |date, date_label|
+When /^(?:|I )select "([^\"]*)" as the "([^\"]*)" date$/ do |date, date_label|
   select_date(date, :from => date_label)
 end
 
-When /^I check "([^\"]*)"$/ do |field|
+When /^(?:|I )check "([^\"]*)"$/ do |field|
   check(field)
 end
 
-When /^I uncheck "([^\"]*)"$/ do |field|
+When /^(?:|I )uncheck "([^\"]*)"$/ do |field|
   uncheck(field)
 end
 
-When /^I choose "([^\"]*)"$/ do |field|
+When /^(?:|I )choose "([^\"]*)"$/ do |field|
   choose(field)
 end
 
-When /^I attach the file at "([^\"]*)" to "([^\"]*)"$/ do |path, field|
+When /^(?:|I )attach the file at "([^\"]*)" to "([^\"]*)"$/ do |path, field|
   attach_file(field, path)
 end
 
-Then /^I should see "([^\"]*)"$/ do |text|
+Then /^(?:|I )should see "([^\"]*)"$/ do |text|
   response.should contain(text)
 end
 
-Then /^I should see "([^\"]*)" within "([^\"]*)"$/ do |text, selector|
+Then /^(?:|I )should see "([^\"]*)" within "([^\"]*)"$/ do |text, selector|
   within(selector) do |content|
     content.should contain(text)
   end
 end
 
-Then /^I should see \/([^\/]*)\/$/ do |regexp|
+Then /^(?:|I )should see \/([^\/]*)\/$/ do |regexp|
   regexp = Regexp.new(regexp)
   response.should contain(regexp)
 end
 
-Then /^I should see \/([^\/]*)\/ within "([^\"]*)"$/ do |regexp, selector|
+Then /^(?:|I )should see \/([^\/]*)\/ within "([^\"]*)"$/ do |regexp, selector|
   within(selector) do |content|
     regexp = Regexp.new(regexp)
     content.should contain(regexp)
   end
 end
 
-Then /^I should not see "([^\"]*)"$/ do |text|
+Then /^(?:|I )should not see "([^\"]*)"$/ do |text|
   response.should_not contain(text)
 end
 
-Then /^I should not see "([^\"]*)" within "([^\"]*)"$/ do |text, selector|
+Then /^(?:|I )should not see "([^\"]*)" within "([^\"]*)"$/ do |text, selector|
   within(selector) do |content|
     content.should_not contain(text)
   end
 end
 
-Then /^I should not see \/([^\/]*)\/$/ do |regexp|
+Then /^(?:|I )should not see \/([^\/]*)\/$/ do |regexp|
   regexp = Regexp.new(regexp)
   response.should_not contain(regexp)
 end
 
-Then /^I should not see \/([^\/]*)\/ within "([^\"]*)"$/ do |regexp, selector|
+Then /^(?:|I )should not see \/([^\/]*)\/ within "([^\"]*)"$/ do |regexp, selector|
   within(selector) do |content|
     regexp = Regexp.new(regexp)
     content.should_not contain(regexp)
@@ -180,7 +180,7 @@
   field_labeled(label).should_not be_checked
 end
 
-Then /^I should be on (.+)$/ do |page_name|
+Then /^(?:|I )should be on (.+)$/ do |page_name|
   URI.parse(current_url).path.should == path_to(page_name)
 end
 

features/support/email.rb

@@ -0,0 +1,21 @@
+module EmailHelpers
+  # Maps a name to an email address. Used by email_steps
+
+  def email_for(to)
+    case to
+  
+    # add your own name => email address mappings here
+  
+    when /^#{capture_model}$/
+      model($1).email
+  
+    when /^"(.*)"$/
+      $1
+  
+    else
+      to
+    end
+  end
+end
+
+World(EmailHelpers)

features/support/env.rb

@@ -1,4 +1,4 @@
-# IMPORTANT: This file was generated by Cucumber 0.4.2
+# IMPORTANT: This file was generated by Cucumber 0.4.4
 # Edit at your own peril - it's recommended to regenerate this file
 # in the future when you upgrade to a newer version of Cucumber.
 # Consider adding your own code to a new file instead of editing this one.
@@ -45,21 +45,3 @@
   config.open_error_files = false # Set to true if you want error pages to pop up in the browser
 end
 
-require 'pickle/world'
-# Example of configuring pickle:
-#
-# Pickle.configure do |config|
-#   config.adapters = [:machinist]
-#   config.map 'I', 'myself', 'me', 'my', :to => 'user: "me"'
-# end
-require 'pickle/path/world'
-require 'pickle/email/world'
-
-# settings for machinist
-
-require "#{RAILS_ROOT}/features/support/blueprints"   # load blueprints
-# original setting
-Before { Sham.reset }                     # reset Sham in between scenarios
-# might want to switch to:
-# Before { Sham.reset(:before_each) }
-# and should we have one for :before_all (see related entry in machinist readme)