aslakhellesoy-cucumber 0.1.9 → 0.1.10

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,13 +5,17 @@ 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 = {})
       stub(Broadcaster, {:register => nil}.merge(stubs))
     end
     
+    def mock_features(stubs ={})
+      stub('features', {:<< => nil}.merge(stubs))
+    end
+    
     before(:each) do
       Kernel.stub!(:exit)
     end
@@ -101,6 +105,40 @@ Defined profiles in cucumber.yml:
       cli.options[:source].should be_false
     end
 
+    it "should accept --verbose option" do
+      cli = CLI.new
+      cli.parse_options!(%w{--verbose})
+
+      cli.options[:verbose].should be_true
+    end
+
+    describe "verbose mode" do
+      
+      before(:each) do
+        @out = StringIO.new
+        @cli = CLI.new(@out)
+        @cli.stub!(:require)
+        Dir.stub!(:[])
+      end
+
+      it "should show ruby files required" do
+        @cli.parse_options!(%w{--verbose --require example.rb})
+        @cli.execute!(stub('step mother'), mock_executor, mock_features)
+        
+        @out.string.should include('example.rb')
+      end
+      
+      it "should show feature files parsed" do
+        TreetopParser::FeatureParser.stub!(:new).and_return(mock("feature parser", :parse_feature => nil))
+          
+        @cli.parse_options!(%w{--verbose example.feature})
+        @cli.execute!(stub('step mother'), mock_executor, mock_features)
+
+        @out.string.should include('example.feature')
+      end
+      
+    end
+
     it "should accept --out option" do
       cli = CLI.new
       File.should_receive(:open).with('jalla.txt', 'w')
@@ -180,6 +218,106 @@ Defined profiles in cucumber.yml:
       cli.execute!(stub('step mother'), mock_executor, stub('features'))
     end
 
+    describe "external formatter" do
+
+      describe "exists and valid constructor" do
+        
+        before(:each) do
+          @mock_formatter_class = mock('formatter class')
+          Kernel.stub!(:const_get).and_return(@mock_formatter_class)
+        end
+        
+        it "should create the formatter" do
+          cli = CLI.new
+          mock_formatter = mock('magical formatter')
+          cli.parse_options!(%w{--format magical})
+
+          @mock_formatter_class.should_receive(:new)
+
+          cli.execute!(stub('step mother'), mock_executor, stub('features'))
+        end
+
+        it "should register the formatter with broadcaster" do
+          cli = CLI.new
+          broadcaster = Broadcaster.new
+          mock_formatter = mock('magical formatter')
+          Broadcaster.stub!(:new).and_return(broadcaster, stub("output broadcaster", :register => nil))
+          @mock_formatter_class.stub!(:new).and_return(mock_formatter)
+          cli.parse_options!(%w{--format magical})
+
+          broadcaster.should_receive(:register).with(mock_formatter)
+        
+          cli.execute!(stub('step mother'), mock_executor, stub('features'))
+        end
+      
+      end
+          
+      describe "exists but invalid constructor" do
+
+        before(:each) do
+          @out = StringIO.new
+          @error = StringIO.new
+          @cli = CLI.new(@out, @error)
+          
+          mock_formatter_class = stub('formatter class')
+          mock_formatter_class.stub!(:new).and_raise("No such method")
+          Kernel.stub!(:const_get).and_return(mock_formatter_class)
+          
+          @cli.parse_options!(%w{--format exists_but_evil}) 
+        end
+        
+        it "should show exception" do
+          Kernel.stub!(:exit)
+
+          @cli.execute!(stub('step mother'), mock_executor, stub('features'))
+
+          @error.string.should include("No such method")
+        end
+        
+        it "should exit" do
+          Kernel.should_receive(:exit)
+
+          @cli.execute!(stub('step mother'), mock_executor, stub('features'))
+        end
+                
+      end
+          
+      describe "non-existent" do
+
+        before(:each) do
+          @out = StringIO.new
+          @error = StringIO.new
+          @cli = CLI.new(@out, @error)
+          
+          @cli.parse_options!(%w{--format invalid})
+        end
+
+        it "should display a format error" do
+          Kernel.stub!(:exit)
+
+          @cli.execute!(stub('step mother'), mock_executor, stub('features'))
+          
+          @error.string.should include("Invalid format: invalid\n")
+        end
+        
+        it "should display --help" do
+          Kernel.stub!(:exit)
+
+          @cli.execute!(stub('step mother'), mock_executor, stub('features'))
+          
+          @out.string.should include("Usage: cucumber")
+        end
+
+        it "should exit" do
+          Kernel.should_receive(:exit)
+
+          @cli.execute!(stub('step mother'), mock_executor, stub('features'))
+        end
+        
+      end
+            
+    end
+
     it "should accept multiple --scenario options" do
       cli = CLI.new
       cli.parse_options!(['--scenario', "User logs in", '--scenario', "User signs up"])
@@ -227,6 +365,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
 
@@ -235,7 +405,7 @@ Defined profiles in cucumber.yml:
 
       Dir.should_receive(:[]).with("feature_directory/**/*.feature").any_number_of_times.and_return([])
       
-      cli.execute!(stub('step mother'), mock_executor, stub('features', :<< => nil))
+      cli.execute!(stub('step mother'), mock_executor, mock_features)
     end
 
   end

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