gherkin 2.5.4-x86-mingw32 → 2.6.0-x86-mingw32

data/features/step_definitions/json_formatter_steps.rb

@@ -11,15 +11,18 @@ class Hash
 end
 
 Given /^a JSON formatter$/ do
-  @io = StringIO.new
-  @formatter = Gherkin::Formatter::JSONFormatter.new(@io)
+  @out = StringIO.new
+  @formatter = Gherkin::Formatter::JSONFormatter.new(@out)
 end
 
 Then /^the outputted JSON should be:$/ do |expected_json|
   require 'json'
-  puts JSON.pretty_generate(JSON.parse(@io.string))
+  @formatter.close
+  actual_json = @out.string
+  puts actual_json
+  puts JSON.pretty_generate(JSON.parse(actual_json))
   expected = JSON.parse(expected_json)
-  actual   = JSON.parse(@io.string)
+  actual   = JSON.parse(actual_json)
   actual.should == expected
 end
 

data/features/step_definitions/json_parser_steps.rb

@@ -12,7 +12,7 @@ Given /^a JSON lexer$/ do
 end
 
 Given /^the following JSON is parsed:$/ do |text|
-  @json_parser.parse(JSON.pretty_generate(JSON.parse(text)), "unknown.json", 0)
+  @json_parser.parse(JSON.pretty_generate(JSON.parse(text)))
 end
 
 Then /^the outputted text should be:$/ do |expected_text|

data/features/step_definitions/pretty_formatter_steps.rb

@@ -20,11 +20,12 @@ module PrettyPlease
     json_formatter      = Gherkin::Formatter::JSONFormatter.new(json)
     gherkin_parser      = Gherkin::Parser::Parser.new(json_formatter, true)
     parse(gherkin_parser, gherkin, feature_path)
+    json_formatter.close
 
     io                  = StringIO.new
     pretty_formatter    = Gherkin::Formatter::PrettyFormatter.new(io, true, false)
     json_parser         = Gherkin::JSONParser.new(pretty_formatter, pretty_formatter)
-    json_parser.parse(json.string, "#{feature_path}.json", 0)
+    json_parser.parse(json.string)
     
     io.string
   end

data/gherkin.gemspec

@@ -2,7 +2,8 @@
 
 Gem::Specification.new do |s|
   s.name        = "gherkin"
-  s.version     = "2.5.4"
+  # If bumping the major version here, comment out the cucumber dependency below and uncomment in the Gemfile to build against a local cucumber.
+  s.version     = "2.6.0"
   s.authors     = ["Mike Sassak", "Gregory Hnatiuk", "Aslak Hellesøy"]
   s.description = "A fast Gherkin lexer/parser based on the Ragel State Machine Compiler."
   s.summary     = "#{s.name}-#{s.version}"
@@ -45,10 +46,10 @@ Gem::Specification.new do |s|
 
   s.add_runtime_dependency('json', '>= 1.4.6')
 
-  s.add_development_dependency('cucumber', '>= 1.1.0')
+  s.add_development_dependency('cucumber', '>= 1.1.1')
   s.add_development_dependency('rake', '>= 0.9.2')
   s.add_development_dependency('bundler', '>= 1.0.21')
-  s.add_development_dependency('rspec', '>= 2.6.0')
+  s.add_development_dependency('rspec', '>= 2.7.0')
 
   unless ENV['RUBY_CC_VERSION'] || defined?(JRUBY_VERSION)
     s.add_development_dependency('therubyracer', '>= 0.9.8') if ENV['GHERKIN_JS']

data/lib/gherkin/formatter/json_formatter.rb

@@ -5,26 +5,31 @@ require 'base64'
 
 module Gherkin
   module Formatter
+    # This class doesn't really generate JSON - instead it populates an Array that can easily
+    # be turned into JSON.
     class JSONFormatter
       native_impl('gherkin')
       
       include Base64
-      attr_reader :gherkin_object
       
-      # Creates a new instance that writes the resulting JSON to +io+.
-      # If +io+ is nil, the JSON will not be written, but instead a Ruby
-      # object can be retrieved with #gherkin_object
       def initialize(io)
+        raise "Must be writeable" unless io.respond_to?(:write)
         @io = io
+        @feature_hashes = []
+      end
+
+      def close
+        @io.write(@feature_hashes.to_json)
       end
 
       def uri(uri)
-        # We're ignoring the uri - we don't want it as part of the JSON
-        # (The pretty formatter uses it just for visual niceness - comments)
+        @uri = uri
       end
 
       def feature(feature)
-        @gherkin_object = feature.to_hash
+        @feature_hash = feature.to_hash
+        @feature_hash['uri'] = @uri
+        @feature_hashes << @feature_hash
       end
 
       def background(background)
@@ -68,13 +73,12 @@ module Gherkin
       end
 
       def eof
-        @io.write(@gherkin_object.to_json) if @io
       end
 
     private
 
       def feature_elements
-        @gherkin_object['elements'] ||= []
+        @feature_hash['elements'] ||= []
       end
 
       def feature_element

data/lib/gherkin/formatter/model.rb

@@ -31,11 +31,12 @@ module Gherkin
       end
 
       class TagStatement < DescribedStatement
-        attr_reader :tags
+        attr_reader :tags, :id
 
-        def initialize(comments, tags, keyword, name, description, line)
+        def initialize(comments, tags, keyword, name, description, line, id)
           super(comments, keyword, name, description, line)
           @tags = tags
+          @id = id
         end
 
         def first_non_comment_line
@@ -46,10 +47,6 @@ module Gherkin
       class Feature < TagStatement
         native_impl('gherkin')
 
-        def initialize(comments, tags, keyword, name, description, line)
-          super(comments, tags, keyword, name, description, line)
-        end
-
         def replay(formatter)
           formatter.feature(self)
         end
@@ -71,8 +68,8 @@ module Gherkin
       class Scenario < TagStatement
         native_impl('gherkin')
 
-        def initialize(comments, tags, keyword, name, description, line)
-          super(comments, tags, keyword, name, description, line)
+        def initialize(comments, tags, keyword, name, description, line, id)
+          super(comments, tags, keyword, name, description, line, id)
           @type = "scenario"
         end
 
@@ -84,8 +81,8 @@ module Gherkin
       class ScenarioOutline < TagStatement
         native_impl('gherkin')
 
-        def initialize(comments, tags, keyword, name, description, line)
-          super(comments, tags, keyword, name, description, line)
+        def initialize(comments, tags, keyword, name, description, line, id)
+          super(comments, tags, keyword, name, description, line, id)
           @type = "scenario_outline"
         end
 
@@ -97,23 +94,47 @@ module Gherkin
       class Examples < TagStatement
         native_impl('gherkin')
 
-        attr_accessor :rows
+        attr_accessor :rows # needs to remain mutable for filters
 
-        def initialize(comments, tags, keyword, name, description, line, rows)
-          super(comments, tags, keyword, name, description, line)
+        def initialize(comments, tags, keyword, name, description, line, id, rows)
+          super(comments, tags, keyword, name, description, line, id)
           @rows = rows
         end
 
         def replay(formatter)
           formatter.examples(self)
         end
+        
+        class Builder
+          def initialize(*args)
+            @args = *args
+            @rows = nil
+          end
+        
+          def row(comments, cells, line, id)
+            @rows ||= []
+            @rows << ExamplesTableRow.new(comments, cells, line, id)
+          end
+        
+          def replay(formatter)
+            build.replay(formatter)
+          end
+          
+          def build
+            Examples.new(*(@args << @rows))
+          end
+        end
       end
 
       class Step < BasicStatement
         native_impl('gherkin')
 
-        attr_accessor :rows
-        attr_accessor :doc_string
+        attr_reader :rows, :doc_string
+        
+        def initialize(comments, keyword, name, line, rows, doc_string)
+          super(comments, keyword, name, line)
+          @rows, @doc_string = rows, doc_string
+        end
 
         def line_range
           range = super
@@ -136,6 +157,31 @@ module Gherkin
             Argument.new(offset, val)
           end
         end
+        
+        class Builder
+          def initialize(*args)
+            @args = *args
+            @rows = nil
+            @doc_string = nil
+          end
+        
+          def row(comments, cells, line, id)
+            @rows ||= []
+            @rows << DataTableRow.new(comments, cells, line)
+          end
+          
+          def doc_string(string, content_type, line)
+            @doc_string = Formatter::Model::DocString.new(string, content_type, line)
+          end
+        
+          def replay(formatter)
+            build.replay(formatter)
+          end
+          
+          def build
+            Step.new(*(@args << @rows << @doc_string))
+          end
+        end
       end
 
       class Comment < Hashable
@@ -169,10 +215,10 @@ module Gherkin
       class DocString < Hashable
         native_impl('gherkin')
 
-        attr_reader :content_type, :value, :line
+        attr_reader :value, :content_type, :line
         
-        def initialize(content_type, value, line)
-          @content_type, @value, @line = content_type, value, line
+        def initialize(value, content_type, line)
+          @value, @content_type, @line = value, content_type, line
         end
 
         def line_range
@@ -182,8 +228,6 @@ module Gherkin
       end
 
       class Row < Hashable
-        native_impl('gherkin')
-
         attr_reader :comments, :cells, :line
 
         def initialize(comments, cells, line)
@@ -191,6 +235,20 @@ module Gherkin
         end
       end
 
+      class DataTableRow < Row
+        native_impl('gherkin')
+      end
+
+      class ExamplesTableRow < Row
+        native_impl('gherkin')
+        attr_reader :id
+
+        def initialize(comments, cells, line, id)
+          super(comments, cells, line)
+          @id = id
+        end
+      end
+
       class Match < Hashable
         native_impl('gherkin')