cucumber 0.1.16 → 0.2.0

data/spec/cucumber/parser/feature_parser_spec.rb

@@ -0,0 +1,282 @@
+require File.dirname(__FILE__) + '/../../spec_helper'
+require 'cucumber/parser'
+
+module Cucumber
+  module Parser
+    describe Feature do
+      before do
+        @parser = FeatureParser.new
+      end
+
+      def parse(text)
+        feature = @parser.parse_or_fail(text)
+      end
+
+      def parse_file(file)
+        @parser.parse_file(File.dirname(__FILE__) + "/../treetop_parser/" + file)
+      end
+
+      def parse_example_file(file)
+        @parser.parse_file(File.dirname(__FILE__) + "/../../../examples/" + file)
+      end
+
+      describe "Header" do
+        it "should parse Feature with blurb" do
+          parse(%{Feature: hi
+with blurb
+})
+        end
+      end
+
+      describe "Comments" do
+        it "should parse a file with only a one line comment" do
+          parse(%{# My comment
+Feature: hi
+}).to_sexp.should ==
+          [:feature, "Feature: hi\n",
+            [:comment, "# My comment\n"]]
+        end
+        
+        it "should parse a comment within a scenario" do
+          pending "Store comment in node and output it in pretty formatter"
+          parse(%{Feature: Hi
+  Scenario: Hello
+    Given foo
+    # When bar
+    Then baz
+}).to_sexp.should == 
+          [:feature, "Feature: Hi", 
+            [:scenario, 2, "Scenario:", "Hello", 
+              [:step, 3, "Given", "foo"],
+              [:comment, "# When bar\n"], 
+              [:step, 5, "Then", "baz"] 
+            ]
+          ]
+        end
+
+        it "should parse a file with only a multiline comment" do
+          parse(%{# Hello
+# World
+Feature: hi
+}).to_sexp.should ==
+          [:feature, "Feature: hi\n",
+            [:comment, "# Hello\n# World\n"]]
+        end
+
+        it "should parse a file with no comments" do
+          parse("Feature: hi\n").to_sexp.should ==
+          [:feature, "Feature: hi\n"]
+        end
+
+        it "should parse a file with only a multiline comment with newlines" do
+          parse("# Hello\n\n# World\n").to_sexp.should == 
+          [:feature, "", 
+            [:comment, "# Hello\n\n# World\n"]]
+        end
+      end
+
+      describe "Tags" do
+        it "should parse a file with tags on a feature" do
+          parse("# My comment\n@hello @world Feature: hi\n").to_sexp.should ==
+          [:feature, "Feature: hi\n",
+            [:comment, "# My comment\n"],
+            [:tag, "hello"],
+            [:tag, "world"]]
+        end
+
+        it "should parse a file with tags on a scenario" do
+          parse(%{# FC
+  @ft
+Feature: hi
+
+  @st1 @st2   
+  Scenario: First
+    Given Pepper
+
+@st3 
+   @st4    @ST5  @#^%&ST6**!
+  Scenario: Second}).to_sexp.should ==
+          [:feature, "Feature: hi",
+            [:comment, "# FC\n  "],
+            [:tag, "ft"],
+            [:scenario, 6, 'Scenario:', 'First',
+              [:tag, "st1"], [:tag, "st2"],
+              [:step_invocation, 7, "Given", "Pepper"]
+            ],
+            [:scenario, 11, 'Scenario:', 'Second',
+              [:tag, "st3"], [:tag, "st4"], [:tag, "ST5"], [:tag, "#^%&ST6**!"]]]
+        end
+      end
+      
+      describe "Background" do
+        it "should have steps" do
+          parse("Feature: Hi\nBackground:\nGiven I am a step\n").to_sexp.should ==
+          [:feature, "Feature: Hi",
+            [:background, 2, "Background:",
+              [:step, 3, "Given", "I am a step"]]]
+        end
+      end
+
+      describe "Scenarios" do
+        it "can be empty" do
+          parse("Feature: Hi\n\nScenario: Hello\n").to_sexp.should ==
+          [:feature, "Feature: Hi",
+            [:scenario, 3, "Scenario:", "Hello"]]
+        end
+
+        it "should allow whitespace lines after the Scenario line" do
+          parse(%{Feature: Foo
+
+Scenario: bar
+
+  Given baz})
+        end
+            
+        it "should have steps" do
+          parse("Feature: Hi\nScenario: Hello\nGiven I am a step\n").to_sexp.should ==
+          [:feature, "Feature: Hi",
+            [:scenario, 2, "Scenario:", "Hello",
+              [:step_invocation, 3, "Given", "I am a step"]]]
+        end
+
+        it "should have steps with inline table" do
+          parse(%{Feature: Hi
+Scenario: Hello
+Given I have a table
+|a|b|
+}).to_sexp.should ==
+          [:feature, "Feature: Hi",
+            [:scenario, 2, "Scenario:", "Hello",
+              [:step_invocation, 3, "Given", "I have a table",
+                [:table,
+                  [:row,
+                    [:cell, "a"],
+                    [:cell, "b"]]]]]]
+        end
+
+        it "should have steps with inline py_string" do
+          parse(%{Feature: Hi
+Scenario: Hello
+Given I have a string
+
+
+   """
+  hello
+  world
+  """
+
+}).to_sexp.should ==
+          [:feature, "Feature: Hi",
+            [:scenario, 2, "Scenario:", "Hello",
+              [:step_invocation, 3, "Given", "I have a string",
+                [:py_string, "hello\nworld"]]]]
+        end
+      end
+
+      describe "Scenario Outlines" do
+        it "can be empty" do
+          parse(%{Feature: Hi
+Scenario Outline: Hello
+Given a <what> cucumber
+Examples:
+|what|
+|green|
+}).to_sexp.should ==
+          [:feature, "Feature: Hi",
+            [:scenario_outline, "Scenario Outline:", "Hello",
+              [:step, 3, "Given", "a <what> cucumber"],
+              [:examples, "Examples:", "",
+                [:table, 
+                  [:row, 
+                    [:cell, "what"]], 
+                    [:row, [:cell, "green"]]]]]]
+        end
+
+        it "should have line numbered steps with inline table" do
+          parse(%{Feature: Hi
+Scenario Outline: Hello
+
+Given I have a table
+
+|<a>|<b>|
+Examples:
+|a|b|
+|c|d|
+}).to_sexp.should ==
+          [:feature, "Feature: Hi",
+            [:scenario_outline, "Scenario Outline:", "Hello",
+              [:step, 4, "Given", "I have a table",
+                [:table, 
+                  [:row, 
+                    [:cell, "<a>"], 
+                    [:cell, "<b>"]]]],
+            [:examples, "Examples:", "",
+              [:table,
+                [:row, 
+                  [:cell, "a"], 
+                  [:cell, "b"]],
+                [:row, 
+                  [:cell, "c"], 
+                  [:cell, "d"]]]]]]
+        end
+
+        it "should have examples" do
+          parse("Feature: Hi
+
+  Scenario Outline: Hello
+
+  Given I have a table
+    |1|2|
+
+  Examples:
+|x|y|
+|5|6|
+
+").to_sexp.should ==
+          [:feature, "Feature: Hi",
+            [:scenario_outline, "Scenario Outline:", "Hello",
+              [:step, 5, "Given", "I have a table",
+                [:table,
+                  [:row,
+                    [:cell, "1"],
+                    [:cell, "2"]]]],
+              [:examples, "Examples:", "",
+                [:table,
+                  [:row,
+                    [:cell, "x"],
+                    [:cell, "y"]],
+                  [:row,
+                    [:cell, "5"],
+                    [:cell, "6"]]]]]]
+        end
+
+        it "should set line numbers on feature" do
+          feature = parse_file("empty_feature.feature:11:12")
+          feature.instance_variable_get('@lines').should == [11, 12]
+        end
+      end
+
+      describe "Syntax" do
+        it "should parse empty_feature" do
+          parse_file("empty_feature.feature")
+        end
+
+        it "should parse empty_scenario" do
+          parse_file("empty_scenario.feature")
+        end
+
+        it "should parse empty_scenario_outline" do
+          parse_file("empty_scenario_outline.feature")
+        end
+
+        it "should parse fit_scenario" do
+          parse_file("multiline_steps.feature")
+        end
+
+        it "should parse scenario_outline" do
+          parse_file("scenario_outline.feature")
+        end
+      end
+    end
+  end
+end

data/spec/cucumber/parser/table_parser_spec.rb

@@ -0,0 +1,48 @@
+require File.dirname(__FILE__) + '/../../spec_helper'
+require 'treetop'
+require 'cucumber/parser'
+
+module Cucumber
+  module Parser
+    describe 'Tables' do
+      before do
+        @parser = FeatureParser.new
+      end
+      
+      def parse(text)
+        @parser.__send__(:prepare_to_parse, text)
+        @parser.root = :table
+        table = @parser._nt_table
+        table.raw
+      end
+
+      it "should parse a 1x2 table with newline" do
+        parse(" | 1 | 2 | \n").should == [%w{1 2}]
+      end
+
+      it "should parse a 1x2 table without newline" do
+        parse("| 1 | 2 |").should == [%w{1 2}]
+      end
+
+      it "should parse a 1x2 table without spaces" do
+        parse("|1|2|").should == [%w{1 2}]
+      end
+
+      it "should parse a 2x2 table" do
+        parse("| 1 | 2 |\n| 3 | 4 |\n").should == [%w{1 2}, %w{3 4}]
+      end
+
+      it "should parse a 2x2 table with several newlines" do
+        parse("| 1 | 2 |\n| 3 | 4 |\n\n\n").should == [%w{1 2}, %w{3 4}]
+      end
+
+      it "should parse a 2x2 table with empty cells" do
+        parse("| 1 |  |\n|| 4 |\n").should == [['1', nil], [nil, '4']]
+      end
+
+      it "should not parse a 2x2 table that isn't closed" do
+        parse("| 1 |  |\n|| 4 ").should_not == [['1', nil], [nil, '4']]
+      end
+    end
+  end
+end

data/spec/cucumber/step_definition_spec.rb

@@ -0,0 +1,81 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+require 'cucumber/ast'
+require 'cucumber/step_mother'
+require 'cucumber/step_definition'
+
+module Cucumber
+  describe StepDefinition do
+    before do
+      extend StepMother
+      @world = new_world!
+      $inside = nil
+    end
+
+    it "should allow calling of other steps" do
+      Given /Outside/ do
+        Given "Inside"
+      end
+      Given /Inside/ do
+        $inside = true
+      end
+
+      step_match("Outside").invoke(@world, nil)
+      $inside.should == true
+    end
+
+    it "should allow calling of other steps with inline arg" do
+      Given /Outside/ do
+        Given "Inside", Ast::Table.new([['inside']])
+      end
+      Given /Inside/ do |table|
+        $inside = table.raw[0][0]
+      end
+
+      step_match("Outside").invoke(@world, nil)
+      $inside.should == 'inside'
+    end
+
+    it "should raise Undefined when inside step is not defined" do
+      Given /Outside/ do
+        Given 'Inside'
+      end
+
+      step = mock('Step')
+      step.should_receive(:exception=)
+      lambda do
+        @world.__cucumber_current_step = step
+        step_match('Outside').invoke(@world, nil)
+      end.should raise_error(Undefined, 'Undefined step: "Inside"')
+    end
+
+    it "should allow forced pending" do
+      Given /Outside/ do
+        pending("Do me!")
+      end
+
+      lambda do
+        step_match("Outside").invoke(@world, nil)
+      end.should raise_error(Pending, "Do me!")
+    end
+    
+    it "should have a #to_s suitable for automcompletion" do
+      stepdef = Given /Hello (.*)/ do
+      end
+      
+      stepdef.to_s.should    == '/Hello (.*)/ # spec/cucumber/step_definition_spec.rb:63'
+      stepdef.to_s(2).should == '/Hello (.*)/   # spec/cucumber/step_definition_spec.rb:63'
+    end
+    
+    it "should allow announce" do
+      v = mock('visitor')
+      v.should_receive(:announce).with('wasup')
+      self.visitor = v
+      world = new_world!
+      Given /Loud/ do
+        announce 'wasup'
+      end
+      step_match("Loud").invoke(world, nil)
+    end
+  end
+end

data/spec/cucumber/step_mother_spec.rb

@@ -1,74 +1,73 @@
 require File.dirname(__FILE__) + '/../spec_helper'
 
+require 'cucumber/step_mother'
+
 module Cucumber
   describe StepMother do
-    it "should report file and line numbers for both duplicate step definitions" do
-      m = StepMother.new
-      
-      m.register_step_proc /Three (.*) mice/ do |disability|
-      end
-
-      begin
-        m.register_step_proc /Three (.*) mice/ do |disability|
-        end
-        violated("Should raise error")
-      rescue => e
-        e.message.should =~ %r{Duplicate step definitions:.+step_mother_spec\.rb:8:in `/Three \(\.\*\) mice/'.+step_mother_spec\.rb\:12:in `/Three \(\.\*\) mice/'}m
-
-      end
+    before do
+      @step_mother = Object.new
+      @step_mother.extend(StepMother)
+      @visitor = mock('Visitor')
     end
 
-    it "should report file and line numbers for multiple step definitions" do
-      m = StepMother.new
-      
-      m.register_step_proc /Three (.*) mice/ do |disability|
+    it "should format step names" do
+      @step_mother.Given(/it (.*) in (.*)/) do |what, month|
       end
-
-      m.register_step_proc /Three blind (.*)/ do |animal|
+      @step_mother.Given(/nope something else/) do |what, month|
       end
+      format = @step_mother.step_match("it snows in april").format_args("[%s]")
+      format.should == "it [snows] in [april]"
+    end
 
-      begin
-        m.regexp_args_proc('Three blind mice')
-        violated("Should raise error")
-      rescue => e
-        e.message.should =~ %r{Multiple step definitions match "Three blind mice":
+    it "should raise Ambiguous error when multiple step definitions match" do
+      @step_mother.Given(/Three (.*) mice/) {|disability|}
+      @step_mother.Given(/Three blind (.*)/) {|animal|}
 
-.+step_mother_spec\.rb:24:in `/Three \(\.\*\) mice/'
-.+step_mother_spec\.rb:27:in `/Three blind \(\.\*\)/'
+      lambda do
+        @step_mother.step_match("Three blind mice")
+      end.should raise_error(Ambiguous, %{Ambiguous match of "Three blind mice":
 
-}m
-      end
-    end
-    
-    it "should mark step as pending when it doesn't match any procs" do
-      pending "think some more about what to expect here" do
-        m = StepMother.new
-        step = mock('step')
-        step.should_receive(:pending!)
-        raise "FIXME"
-        m.execute(step)
-      end
+spec/cucumber/step_mother_spec.rb:23:in `/Three (.*) mice/'
+spec/cucumber/step_mother_spec.rb:24:in `/Three blind (.*)/'
+
+})
     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
+    it "should not raise Ambiguous error when multiple step definitions match, but --guess is enabled" do
+      @step_mother.options = {:guess => true}
+      @step_mother.Given(/Three (.*) mice/) {|disability|}
+      @step_mother.Given(/Three (.*)/) {|animal|}
 
-      m.has_step_definition?('I am a real step definition').should be_true
+      lambda do
+        @step_mother.step_match("Three blind mice")
+      end.should_not raise_error
     end
-
-    it "should report that a step does not have a definition if it has not been registered" do
-      m = StepMother.new
     
-      m.has_step_definition?('I am a step without a definition').should be_false
+    it "should pick right step definition when --guess is enabled and equal number of capture groups" do
+      @step_mother.options = {:guess => true}
+      right = @step_mother.Given(/Three (.*) mice/) {|disability|}
+      wrong = @step_mother.Given(/Three (.*)/) {|animal|}
+      @step_mother.step_match("Three blind mice").step_definition.should == right
     end
     
-    it "should explain to the programmer that step definitions always need a proc" do
-      m = StepMother.new
+    it "should pick right step definition when --guess is enabled and unequal number of capture groups" do
+      @step_mother.options = {:guess => true}
+      right = @step_mother.Given(/Three (.*) mice ran (.*)/) {|disability|}
+      wrong = @step_mother.Given(/Three (.*)/) {|animal|}
+      @step_mother.step_match("Three blind mice ran far").step_definition.should == right
+    end
+    
+    it "should raise Undefined error when no step definitions match" do
       lambda do
-        m.register_step_proc(/no milk today/)
-      end.should raise_error(MissingProc, "Step definitions must always have a proc")
+        @step_mother.step_match("Three blind mice")
+      end.should raise_error(Undefined)
     end
 
+    it "should raise Redundant error when same regexp is registered twice" do
+      @step_mother.Given(/Three (.*) mice/) {|disability|}
+      lambda do
+        @step_mother.Given(/Three (.*) mice/) {|disability|}
+      end.should raise_error(Redundant)
+    end
   end
 end