cucumber 0.1.9 → 0.1.10

data/lib/cucumber/version.rb

@@ -2,7 +2,7 @@ module Cucumber #:nodoc:
   class VERSION #:nodoc:
     MAJOR = 0
     MINOR = 1
-    TINY  = 9
+    TINY  = 10
 
     STRING = [MAJOR, MINOR, TINY].join('.')
   end

data/rails_generators/cucumber/templates/webrat_steps.rb

@@ -1,6 +1,8 @@
 # Commonly used webrat steps
 # http://github.com/brynary/webrat
 
+require 'webrat' if !defined?(Webrat) # Because some people have it installed as a Gem
+
 When /^I press "(.*)"$/ do |button|
   clicks_button(button)
 end
@@ -17,6 +19,53 @@ When /^I select "(.*)" from "(.*)"$/ do |value, field|
   selects(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|
+  selects_datetime(time)
+end
+
+# Use this step when using multiple datetime_select helpers on a page or 
+# you want to specify which datetime to select. Given the following view:
+#   <%= f.label :preferred %><br />
+#   <%= f.datetime_select :preferred %>
+#   <%= f.label :alternative %><br />
+#   <%= f.datetime_select :alternative %>
+# The following steps would fill out the form:
+# When I select "November 23, 2004 11:20" as the "Preferred" data and time
+# And I select "November 25, 2004 10:30" as the "Alternative" data and time
+When /^I select "(.*)" as the "(.*)" date and time$/ do |datetime, datetime_label|
+  selects_datetime(datetime, :from => datetime_label)
+end
+
+# Use this step in conjuction 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|
+  selects_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|
+  selects_time(time, :from => time_label)
+end
+
+# Use this step in conjuction 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|
+  selects_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|
+  selects_date(date, :from => date_label)
+end
+
 When /^I check "(.*)"$/ do |field|
   checks(field) 
 end

data/rails_generators/feature/templates/feature.erb

@@ -1,7 +1,7 @@
 Feature: Manage <%= plural_name %>
-  In order to keep track of <%= plural_name %>
-  A <%= singular_name %> mechanic
-  Should be able to manage several <%= plural_name %>
+  In order to [goal]
+  [stakeholder]
+  wants [behaviour]
   
   Scenario: Register new <%= singular_name %>
     Given I am on the new <%= singular_name %> page

data/spec/cucumber/cli_spec.rb

@@ -5,7 +5,7 @@ module Cucumber
   describe CLI do
     
     def mock_executor(stubs = {})
-      stub('executor', {:visit_features => nil, :failed => false, :formatters= => nil}.merge(stubs))
+      stub('executor', {:visit_features => nil, :lines_for_features= => nil, :failed => false, :formatters= => nil}.merge(stubs))
     end
     
     def mock_broadcaster(stubs = {})
@@ -227,6 +227,38 @@ Defined profiles in cucumber.yml:
       cli.execute!(stub('step mother'), mock_executor, stub('features'))
     end
 
+    describe "example.feature:line file arguments" do
+
+      it "should extract line numbers" do
+        cli = CLI.new
+        cli.parse_options!(%w{example.feature:10})
+      
+        cli.options[:lines_for_features]['example.feature'].should == [10]
+      end
+    
+      it "should remove line numbers" do
+        cli = CLI.new
+        cli.parse_options!(%w{example.feature:10})
+      
+        cli.paths.should == ["example.feature"]
+      end
+
+      it "should support multiple feature:line numbers" do
+        cli = CLI.new
+        cli.parse_options!(%w{example.feature:11 another_example.feature:12})
+      
+        cli.options[:lines_for_features].should == {'another_example.feature' => [12], 'example.feature' => [11]}
+      end
+
+      it "should accept multiple line numbers for a single feature" do
+        cli = CLI.new
+        cli.parse_options!(%w{example.feature:11:12})
+      
+        cli.options[:lines_for_features].should == {'example.feature' => [11, 12]}
+      end
+    
+  end
+
     it "should search for all features in the specified directory" do
       cli = CLI.new
 

data/spec/cucumber/executor_spec.rb

@@ -3,6 +3,17 @@ require 'stringio'
 
 module Cucumber
   describe Executor do
+    
+    def mock_scenario(stubs = {})
+      @scenario ||= stub("scenario", {
+        :row? => false, 
+        :name => 'test', 
+        :accept => nil, 
+        :steps => [],
+        :pending? => true
+      }.merge(stubs))
+    end
+    
     before do # TODO: Way more setup and duplication of lib code. Use lib code!
       @io = StringIO.new
       @step_mother = StepMother.new
@@ -35,19 +46,47 @@ module Cucumber
 
 1)
 dang
-#{__FILE__}:32:in `Then /I should owe (\\d*) cucumbers/'
+#{__FILE__}:43:in `Then /I should owe (\\d*) cucumbers/'
 #{@feature_file}:9:in `Then I should owe 7 cucumbers'
 })
     end
 
-#     it "should allow calling of other steps from steps" do
-#       @executor.register_step_proc("call me please") { @x = 1 }
-#       @executor.register_step_proc("I will call you") { @executor.register_step_proc("call me please") }
-#       @executor.register_step_proc(/I should owe (\d*) cucumbers/)  { |n| @n.should == -n.to_i }
-#       @feature.accept(@executor)
-#       @formatters.each { |formatter| formatter.dump }
-#       @io.string.should == "...\n"
-#     end
+    describe "creating a world" do
+      module DoitExtension
+        def doit
+          "dunit"
+        end
+      end
+
+      module BeatitExtension
+        def beatit
+          "beatenit"
+        end
+      end
+      
+      it "should yield an Object to the world proc" do
+        @executor.register_world_proc do |world|
+          world.extend(DoitExtension)
+        end
+        @executor.register_world_proc do |world|
+          world.extend(BeatitExtension)
+        end
+        world = @executor.create_world
+        world.doit.should == "dunit"
+        world.beatit.should == "beatenit"
+      end
+    end
+
+    describe "visiting feature" do
+
+      it "should set the feature file being visited" do
+        mock_feature = mock('feature', :file => 'womble.feature', :scenarios => [])
+        @executor.visit_feature(mock_feature)
+    
+        @executor.instance_variable_get('@feature_file').should == 'womble.feature'
+      end
+
+    end
 
     describe "visiting steps" do
       def make_regex(a,b,c)
@@ -139,21 +178,22 @@ dang
       describe "without having first run the matching regular scenario" do
       
         before(:each) do
-          @scenario = Tree::Scenario.new(nil, 'test', 1)
-          @executor.line=5        
-          @executor.visit_regular_scenario(@scenario)
+          @regular_scenario = Tree::Scenario.new(nil, 'test', 1)
+          @executor.lines_for_features = Hash.new([5])
+          @executor.visit_regular_scenario(@regular_scenario)
         end
         
         it "should run the regular scenario before the row scenario" do
-          @scenario.should_receive(:accept)
-
-          @executor.visit_row_scenario(mock_row_scenario(:name => 'test', :at_line? => true, :accept => nil))
+          @regular_scenario.should_receive(:accept)
+          row_scenario = mock_row_scenario(:name => 'test', :at_line? => true)
+          row_scenario.should_receive(:accept)
+          @executor.visit_row_scenario(row_scenario)
         end
 
         it "should run the row scenario after running the regular scenario" do
-          mock_row_scenario(:at_line? => true).should_receive(:accept)
-
-          @executor.visit_row_scenario(mock_row_scenario)
+          row_scenario = mock_row_scenario(:at_line? => true)
+          row_scenario.should_receive(:accept)
+          @executor.visit_row_scenario(row_scenario)
         end
       
       end
@@ -172,18 +212,53 @@ dang
       end
     end
     
-    describe "caching visited scenarios" do
+    describe "visiting scenarios" do
+      
+      it "should check if a scenario is at the specified line number" do
+        mock_scenario = mock('scenario', :null_object => true)
+        @executor.lines_for_features = Hash.new([1])
 
-      def mock_scenario(stubs = {})
-        @scenario ||= stub("scenario", {
-          :row? => false, 
-          :name => 'test', 
-          :accept => nil, 
-          :steps => [],
-          :pending? => true
-        }.merge(stubs))
+        mock_scenario.should_receive(:at_line?).with(1)
+
+        @executor.visit_scenario(mock_scenario)
       end
-      
+    
+     describe "with specific features and lines" do
+
+        it "should check if a scenario is at the specified feature line number" do
+          @executor.instance_variable_set('@feature_file', 'sell_cucumbers.feature')
+          @executor.lines_for_features = {'sell_cucumbers.feature' => [11]}
+
+          mock_scenario.should_receive(:at_line?).with(11).and_return(false)
+
+          @executor.visit_scenario(mock_scenario)
+        end
+
+        it "should not check feature line numbers if --line is already set" do
+          @executor.instance_variable_set('@feature_file', 'sell_cucumbers.feature')
+          @executor.lines_for_features = {'sell_cucumbers.feature' => [11]}
+          @executor.lines_for_features = Hash.new([5])
+
+          mock_scenario.should_not_receive(:at_line?).with(11).and_return(false)
+
+          @executor.visit_scenario(mock_scenario)
+        end
+
+        it "should not check feature line numbers if the current feature file does not have lines specified" do
+          @executor.instance_variable_set('@feature_file', 'beetlejuice.feature')
+          @executor.lines_for_features = {'sell_cucumbers.feature' => [11]}
+
+          mock_scenario.should_not_receive(:at_line?).with(11).and_return(false)
+
+          @executor.visit_scenario(mock_scenario)
+        end
+
+      end
+    
+    end
+    
+    describe "caching visited scenarios" do
+ 
       it "should reset cache after each feature visit" do
         Tree::Scenario.stub!(:new).and_return(mock_scenario)
 
@@ -218,5 +293,6 @@ dang
         @executor.visit_features(@features)
       end
     end
+        
   end
 end

data/spec/cucumber/step_mother_spec.rb

@@ -50,13 +50,6 @@ module Cucumber
       end
     end
 
-    it "should mark a step as pending if it does not have a proc" do
-      m = StepMother.new
-      m.register_step_proc /I have no body/
-      regexp, args, proc = m.regexp_args_proc "I have no body"
-      proc.should == StepMother::PENDING
-    end
-
     it "should report that a step has a definition if the step is registered" do
       m = StepMother.new
       m.register_step_proc /I am a real step definition/ do end
@@ -69,6 +62,13 @@ module Cucumber
     
       m.has_step_definition?('I am a step without a definition').should be_false
     end
+    
+    it "should explain to the programmer that step definitions always need a proc" do
+      m = StepMother.new
+      lambda do
+        m.register_step_proc(/no milk today/)
+      end.should raise_error(MissingProc, "Step definitions must always have a proc")
+    end
 
   end
 end

data/spec/cucumber/tree/feature_spec.rb

@@ -3,10 +3,41 @@ require File.dirname(__FILE__) + '/../../spec_helper'
 module Cucumber
   module Tree
     describe Feature do
+      
       it "should have padding_length 2 when alone" do
         feature = Feature.new('header')
         feature.padding_length.should == 2
       end
+      
+      describe "creating a Scenario" do
+      
+        it "should create a new scenario for a feature" do
+          feature = Feature.new('header')
+
+          Scenario.should_receive(:new).with(feature, 'test scenario', "19")
+
+          feature.Scenario('test scenario') {}
+        end
+      
+      end
+      
+      describe "creating a Table" do
+    
+        it "should set the table header of the template scenario" do
+          feature = Feature.new('header')
+          mock_scenario = mock("scenario", :update_table_column_widths => nil)
+          Scenario.stub!(:new).and_return(mock_scenario)
+          feature.add_scenario('scenario', 5)    
+
+          mock_scenario.should_receive(:table_header=).with(["input_1", "input_2"])
+
+          feature.Table do |t| 
+            t | "input_1" | "input_2" | t 
+            t | 1 | 2 | t
+          end
+        end
+      
+      end
     end
   end
 end

data/spec/cucumber/tree/row_scenario_spec.rb

@@ -4,6 +4,10 @@ module Cucumber
   module Tree
     describe RowScenario do
       
+      def mock_scenario(stubs = {})
+        mock('scenario', {:update_table_column_widths => nil, :steps => []}.merge(stubs))
+      end
+            
       describe "pending?" do
         before :each do
           @scenario = Scenario.new(nil, '', 1)
@@ -18,8 +22,34 @@ module Cucumber
           @scenario.create_step('Given', 'a long step', 1)
           @row_scenario.should_not be_pending
         end
+      end
+            
+      describe "generating row steps" do
+        
+         it "should cache unbound steps" do
+          row_scenario = RowScenario.new(mock('feature'), mock_scenario, [], 1)        
+          
+          row_scenario.steps.should equal(row_scenario.steps)
+        end
         
+        it "should cache bound steps" do
+          mock_step = mock('step', :arity => 1)
+          row_scenario = RowScenario.new(mock('feature'), mock_scenario(:steps => [mock_step]), [], 1)        
+          
+          row_scenario.steps.should equal(row_scenario.steps)
+        end
+        
+        it "should regenerate row steps when scenario template steps have been matched" do
+          mock_step = mock('step', :arity => 0)
+          row_scenario = RowScenario.new(mock('feature'), mock_scenario(:steps => [mock_step]), [], 1)        
+          unbound_steps = row_scenario.steps
+          mock_step.stub!(:arity => 1)
+          
+          unbound_steps.should_not equal(row_scenario.steps)
+        end
+
       end
+          
     end
   end
 end

data/spec/spec_helper.rb

@@ -2,6 +2,7 @@ require 'rubygems'
 gem 'rspec'
 require 'spec'
 
+$KCODE='u'
 $:.unshift(File.dirname(__FILE__) + '/../lib')
 require 'cucumber'
 require 'cucumber/treetop_parser/feature_en'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: cucumber
 version: !ruby/object:Gem::Version 
-  version: 0.1.9
+  version: 0.1.10
 platform: ruby
 authors: 
 - "Aslak Helles\xC3\xB8y"
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2008-11-12 00:00:00 +01:00
+date: 2008-11-25 00:00:00 +01:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -33,24 +33,24 @@ dependencies:
         version: 1.2.4
     version: 
 - !ruby/object:Gem::Dependency 
-  name: rspec
+  name: diff-lcs
   type: :runtime
   version_requirement: 
   version_requirements: !ruby/object:Gem::Requirement 
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        version: 1.1.5
+        version: 1.1.2
     version: 
 - !ruby/object:Gem::Dependency 
-  name: diff-lcs
-  type: :runtime
+  name: rspec
+  type: :development
   version_requirement: 
   version_requirements: !ruby/object:Gem::Requirement 
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        version: 1.1.2
+        version: 1.1.11
     version: 
 - !ruby/object:Gem::Dependency 
   name: hoe
@@ -109,6 +109,11 @@ files:
 - examples/i18n/da/features/step_definitons/kalkulator_steps.rb
 - examples/i18n/da/features/summering.feature
 - examples/i18n/da/lib/kalkulator.rb
+- examples/i18n/de/Rakefile
+- examples/i18n/de/features/addition.feature
+- examples/i18n/de/features/division.feature
+- examples/i18n/de/features/step_definitons/calculator_steps.rb
+- examples/i18n/de/lib/calculator.rb
 - examples/i18n/en/Rakefile
 - examples/i18n/en/features/addition.feature
 - examples/i18n/en/features/division.feature
@@ -141,6 +146,11 @@ files:
 - examples/i18n/ja/features/division.feature
 - examples/i18n/ja/features/step_definitons/calculator_steps.rb
 - examples/i18n/ja/lib/calculator.rb
+- examples/i18n/lt/Rakefile
+- examples/i18n/lt/features/addition.feature
+- examples/i18n/lt/features/division.feature
+- examples/i18n/lt/features/step_definitons/calculator_steps.rb
+- examples/i18n/lt/lib/calculator.rb
 - examples/i18n/no/Rakefile
 - examples/i18n/no/features/step_definitons/kalkulator_steps.rb
 - examples/i18n/no/features/summering.feature
@@ -171,6 +181,9 @@ files:
 - examples/selenium/Rakefile
 - examples/selenium/features/search.feature
 - examples/selenium/features/step_definitons/stories_steps.rb
+- examples/test_unit/Rakefile
+- examples/test_unit/features/step_definitions/test_unit_steps.rb
+- examples/test_unit/features/test_unit.feature
 - examples/tickets/Rakefile
 - examples/tickets/cucumber.yml
 - examples/tickets/features/step_definitons/tickets_steps.rb
@@ -213,6 +226,7 @@ files:
 - lib/cucumber/languages.yml
 - lib/cucumber/model.rb
 - lib/cucumber/model/table.rb
+- lib/cucumber/platform.rb
 - lib/cucumber/rails/rspec.rb
 - lib/cucumber/rails/world.rb
 - lib/cucumber/rake/task.rb
@@ -239,6 +253,7 @@ files:
 - lib/cucumber/treetop_parser/feature_id.rb
 - lib/cucumber/treetop_parser/feature_it.rb
 - lib/cucumber/treetop_parser/feature_ja.rb
+- lib/cucumber/treetop_parser/feature_lt.rb
 - lib/cucumber/treetop_parser/feature_nl.rb
 - lib/cucumber/treetop_parser/feature_no.rb
 - lib/cucumber/treetop_parser/feature_parser.rb