cucumber-core 5.0.2 → 9.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3ab18c1635eeda0fbd575d01a83bafc96a0073a1ec0ba1249e4bf26b9c60f914
-  data.tar.gz: 1dccc021f15d75d76c2c3974049f768ee926e0dd7ced151cc9b46a2b422b6906
+  metadata.gz: aeb7e9695298dfc63cddf549b2522341be7321f72fbd62b5d1c9f27aaf4209e5
+  data.tar.gz: d382bca49b619bede78e599748bf07b69c3041d1d4a0124ce77dde76d58865cf
 SHA512:
-  metadata.gz: 81461cf45c9616e3e36fc70a25da1b3c0e30e709cc12e675994e6fd0f730ff4e68f9680453e31ecb4814d69a9a79a98ceeb0691a7365fc660fe9e4cad7462092
-  data.tar.gz: 3a8cc3c74351d91f90c36f509a813b56cab9a333057de9b3b596de4b1feaa8dc912d87827d2682f1fc3f2b2eca2f82c75f38bd2f75204119eb21bcbcf0b8507a
+  metadata.gz: a0fbe65ed295c80bdea85ad91574edeefb12b4689ada4cad11aab513c7c5acc0f080f5872b9a733b22d005e3db7db32e06ff8a24fced4041bb5e334e50fd2bce
+  data.tar.gz: b143b5bbefc63fb2dc5b36595c46588a90dfc23dbeaa5847ef9388796597d0635847f78a76237393d5d7d7ac946f84976871c0043850f712dac770cb2e634697

data/CHANGELOG.md

@@ -1,5 +1,78 @@
 Please see [CONTRIBUTING.md](https://github.com/cucumber/cucumber/blob/master/CONTRIBUTING.md) on how to contribute to Cucumber.
 
+## [In GIT](https://github.com/cucumber/cucumber-ruby-core/compare/v9.0.0...master)
+
+### Added
+
+### Changed
+
+### Deprecate
+
+### Removed
+
+### Fixed
+
+### Dependencies
+
+## [9.0.0](https://github.com/cucumber/cucumber-ruby-core/compare/v8.0.1...v9.0.0)
+
+### Dependencies
+
+* Upgraded to gherkin v18 and messages v15
+* Updated other dependencies (look at the diff for details)
+
+## [8.0.1](https://github.com/cucumber/cucumber-ruby-core/compare/v8.0.0...v8.0.1)
+
+### Fixed
+
+* Make releases from docker using secrets from keybase
+
+## [8.0.0](https://github.com/cucumber/cucumber-ruby-core/compare/v7.1.0...v8.0.0)
+
+This release failed
+
+### Dependencies
+
+* Updated dependencies (look at the diff for details)
+
+## [7.1.0](https://github.com/cucumber/cucumber-ruby-core/compare/v7.0.0...v7.1.0)
+
+### Changed
+
+* Updated gems:
+  * `cucumber-gherkin` ~> 14.0.1
+  * `cucumber-messages` ~> 12.2.0
+
+## [7.0.0](https://github.com/cucumber/cucumber-ruby-core/compare/v6.0.0...v7.0.0)
+
+### Changed
+
+* Updated monorepo libraries:
+  - cucumber-gherkin ~> 13
+  - cucumber-messages ~> 12
+
+## [6.0.0](https://github.com/cucumber/cucumber-ruby-core/compare/v5.0.2...v6.0.0)
+
+### Changed
+
+* Update to Gherkin 10
+
+### Added
+
+* Add `envelope` event, which are used when emitting `Cucumber::Messages`
+* Add `TestCaseCreated` and `TestStepCreated` events, emitted when compiling a `Pickle`
+* Add `Id` field to `TestCase` and `TestStep`
+* Added rubocop (with todo file), and removed backports gems
+  ([#186](https://github.com/cucumber/cucumber-ruby-core/pull/186),
+  [#182](https://github.com/cucumber/cucumber-ruby-core/issues/182)
+  [tas50](https://github.com/tas50),
+  [luke-hill](https://github.com/luke-hill))
+
+### Removed
+
+* Remove location for MultiLine arguments
+
+
 ## [5.0.2](https://github.com/cucumber/cucumber-ruby-core/compare/v5.0.1...v5.0.2)
 
 ### Changed

data/CONTRIBUTING.md

@@ -1,13 +1,13 @@
 Release Process
 ===============
 
+* Upgrade gems with `scripts/update-gemspec`
 * Bump the version number in `lib/cucumber/core/version.rb`
 * Update `CHANGELOG.md` with the upcoming version number and create a new `In Git` section
+* Remove empty sections from `CHANGELOG.md`
 * Now release it:
 
 ```
-bundle update
-bundle exec rake
 git commit -m "Release X.Y.Z"
-rake release
+make release
 ```

data/README.md

@@ -1,8 +1,6 @@
 # cucumber-core
 
-[![Chat with us](https://badges.gitter.im/Join%20Chat.svg)](https://gitter.im/cucumber/cucumber-ruby?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge)
 [![CircleCI](https://circleci.com/gh/cucumber/cucumber-ruby-core.svg?style=svg)](https://circleci.com/gh/cucumber/cucumber-ruby-core)
-[![Build Status](https://travis-ci.org/cucumber/cucumber-ruby-core.svg?branch=master)](https://travis-ci.org/cucumber/cucumber-ruby-core)
 [![Code Climate](https://codeclimate.com/github/cucumber/cucumber-ruby-core.svg)](https://codeclimate.com/github/cucumber/cucumber-ruby-core)
 [![Coverage Status](https://coveralls.io/repos/cucumber/cucumber-ruby-core/badge.svg?branch=master)](https://coveralls.io/r/cucumber/cucumber-ruby-core?branch=master)
 

data/lib/cucumber/core.rb

@@ -4,6 +4,8 @@ require 'cucumber/core/gherkin/parser'
 require 'cucumber/core/gherkin/document'
 require 'cucumber/core/compiler'
 require 'cucumber/core/test/runner'
+require 'cucumber/messages'
+require 'gherkin/query'
 
 module Cucumber
   module Core
@@ -11,21 +13,22 @@ module Cucumber
     def execute(gherkin_documents, filters = [], event_bus = EventBus.new)
       yield event_bus if block_given?
       receiver = Test::Runner.new(event_bus)
-      compile gherkin_documents, receiver, filters
+      compile gherkin_documents, receiver, filters, event_bus
       self
     end
 
     def compile(gherkin_documents, last_receiver, filters = [], event_bus = EventBus.new)
       first_receiver = compose(filters, last_receiver)
-      compiler = Compiler.new(first_receiver)
-      parse gherkin_documents, compiler, event_bus
+      gherkin_query = ::Gherkin::Query.new
+      compiler = Compiler.new(first_receiver, gherkin_query, event_bus)
+      parse gherkin_documents, compiler, event_bus, gherkin_query
       self
     end
 
     private
 
-    def parse(gherkin_documents, compiler, event_bus)
-      parser = Core::Gherkin::Parser.new(compiler, event_bus)
+    def parse(gherkin_documents, compiler, event_bus, gherkin_query)
+      parser = Core::Gherkin::Parser.new(compiler, event_bus, gherkin_query)
       gherkin_documents.each do |document|
         parser.document document
       end
@@ -38,6 +41,5 @@ module Cucumber
         filter.with_receiver(receiver)
       end
     end
-
   end
 end

data/lib/cucumber/core/compiler.rb

@@ -6,16 +6,20 @@ require 'cucumber/core/test/tag'
 require 'cucumber/core/test/doc_string'
 require 'cucumber/core/test/data_table'
 require 'cucumber/core/test/empty_multiline_argument'
+require 'cucumber/messages'
 
 module Cucumber
   module Core
     # Compiles the Pickles into test cases
     class Compiler
-      attr_reader :receiver
-      private     :receiver
+      attr_reader :receiver, :gherkin_query, :id_generator
+      private     :receiver, :gherkin_query, :id_generator
 
-      def initialize(receiver)
+      def initialize(receiver, gherkin_query, event_bus = nil)
         @receiver = receiver
+        @id_generator = Cucumber::Messages::IdGenerator::UUID.new
+        @gherkin_query = gherkin_query
+        @event_bus = event_bus
       end
 
       def pickle(pickle)
@@ -33,38 +37,55 @@ module Cucumber
       def create_test_case(pickle)
         uri = pickle.uri
         test_steps = pickle.steps.map { |step| create_test_step(step, uri) }
-        lines = pickle.locations.map { |location| location.line }.sort.reverse
-        tags = pickle.tags.map { |tag| Test::Tag.new(Test::Location.new(uri, tag.location.line), tag.name) }
-        Test::Case.new(pickle.name, test_steps, Test::Location.new(uri, lines), tags, pickle.language)
+        lines = source_lines_for_pickle(pickle).sort.reverse
+        tags = pickle.tags.map { |tag| Test::Tag.new(Test::Location.new(uri, source_line_for_pickle_tag(tag)), tag.name) }
+        test_case = Test::Case.new(id_generator.new_id, pickle.name, test_steps, Test::Location.new(uri, lines), tags, pickle.language)
+        @event_bus&.test_case_created(test_case, pickle)
+        test_case
       end
 
       def create_test_step(pickle_step, uri)
-        lines = pickle_step.locations.map { |location| location.line }.sort.reverse
+        lines = source_lines_for_pickle_step(pickle_step).sort.reverse
         multiline_arg = create_multiline_arg(pickle_step, uri)
-        Test::Step.new(pickle_step.text, Test::Location.new(uri, lines), multiline_arg)
+        step = Test::Step.new(id_generator.new_id, pickle_step.text, Test::Location.new(uri, lines), multiline_arg)
+        @event_bus&.test_step_created(step, pickle_step)
+        step
       end
 
-      def create_multiline_arg(pickle_step, uri)
+      def create_multiline_arg(pickle_step, _uri)
         if pickle_step.argument
           if pickle_step.argument.doc_string
             doc_string = pickle_step.argument.doc_string
             Test::DocString.new(
               doc_string.content,
-              doc_string.contentType,
-              Test::Location.new(uri, doc_string.location.line)
+              doc_string.media_type
             )
           elsif pickle_step.argument.data_table
             data_table = pickle_step.argument.data_table
-            first_cell = data_table.rows.first.cells.first
             Test::DataTable.new(
-              data_table.rows.map { |row| row.cells.map { |cell| cell.value } },
-              Test::Location.new(uri, first_cell.location.line)
+              data_table.rows.map { |row| row.cells.map { |cell| cell.value } }
             )
           end
         else
           Test::EmptyMultilineArgument.new
         end
       end
+
+      def source_lines_for_pickle(pickle)
+        pickle.ast_node_ids.map { |id| source_line(id) }
+      end
+
+      def source_lines_for_pickle_step(pickle_step)
+        pickle_step.ast_node_ids.map { |id| source_line(id) }
+      end
+
+      def source_line_for_pickle_tag(tag)
+        source_line(tag.ast_node_id)
+      end
+
+      def source_line(id)
+        gherkin_query.location(id).line
+      end
     end
   end
 end

data/lib/cucumber/core/event.rb

@@ -1,5 +1,3 @@
-require 'backports/2.1.0/array/to_h'
-
 module Cucumber
   module Core
     class Event
@@ -24,7 +22,7 @@ module Cucumber
           end
 
           define_method(:to_h) do
-            attributes.reduce({}) { |result, attribute| 
+            attributes.reduce({}) { |result, attribute|
               value = self.send(attribute)
               result[attribute] = value
               result
@@ -50,8 +48,8 @@ module Cucumber
 
         def underscore(string)
           string.to_s.gsub(/::/, '/').
-          gsub(/([A-Z]+)([A-Z][a-z])/,'\1_\2').
-          gsub(/([a-z\d])([A-Z])/,'\1_\2').
+          gsub(/([A-Z]+)([A-Z][a-z])/, '\1_\2').
+          gsub(/([a-z\d])([A-Z])/, '\1_\2').
           tr("-", "_").
           downcase
         end

data/lib/cucumber/core/event_bus.rb

@@ -44,7 +44,7 @@ module Cucumber
 
       def broadcast_queued_events_to(handler, event_type)
         @event_queue.select { |event|
-          event.class == event_type
+          event.instance_of?(event_type)
         }.each { |event|
           handler.call(event)
         }

data/lib/cucumber/core/events.rb

@@ -5,6 +5,10 @@ module Cucumber
   module Core
     module Events
 
+      class Envelope < Event.new(:envelope)
+        attr_reader :envelope
+      end
+
       # Signals that a gherkin source has been parsed
       class GherkinSourceParsed < Event.new(:gherkin_document)
         # @return [GherkinDocument] the GherkinDocument Ast Node
@@ -12,6 +16,24 @@ module Cucumber
 
       end
 
+      # Signals that a Test::Step was created from a PickleStep
+      class TestStepCreated < Event.new(:test_step, :pickle_step)
+        # The created test step
+        attr_reader :test_step
+
+        # The source pickle step
+        attr_reader :pickle_step
+      end
+
+      # Signals that a Test::Case was created from a Pickle
+      class TestCaseCreated < Event.new(:test_case, :pickle)
+        # The created test step
+        attr_reader :test_case
+
+        # The source pickle step
+        attr_reader :pickle
+      end
+
       # Signals that a {Test::Case} is about to be executed
       class TestCaseStarted < Event.new(:test_case)
 
@@ -39,7 +61,6 @@ module Cucumber
 
       end
 
-
       # Signals that a {Test::Case} has finished executing
       class TestCaseFinished < Event.new(:test_case, :result)
 
@@ -55,7 +76,10 @@ module Cucumber
       # that will be used by the {EventBus} by default.
       def self.registry
         build_registry(
+          Envelope,
           GherkinSourceParsed,
+          TestStepCreated,
+          TestCaseCreated,
           TestCaseStarted,
           TestStepStarted,
           TestStepFinished,

data/lib/cucumber/core/gherkin/parser.rb

@@ -7,24 +7,26 @@ module Cucumber
       ParseError = Class.new(StandardError)
 
       class Parser
-        attr_reader :receiver, :event_bus
-        private     :receiver, :event_bus
+        attr_reader :receiver, :event_bus, :gherkin_query
+        private     :receiver, :event_bus, :gherkin_query
 
-        def initialize(receiver, event_bus)
+        def initialize(receiver, event_bus, gherkin_query)
           @receiver = receiver
           @event_bus = event_bus
+          @gherkin_query = gherkin_query
         end
 
         def document(document)
           messages = ::Gherkin.from_source(document.uri, document.body, gherkin_options(document))
           messages.each do |message|
-            if !message.gherkinDocument.nil?
-              event_bus.gherkin_source_parsed(message.gherkinDocument)
+            event_bus.envelope(message)
+            gherkin_query.update(message)
+            if !message.gherkin_document.nil?
+              event_bus.gherkin_source_parsed(message.gherkin_document)
             elsif !message.pickle.nil?
               receiver.pickle(message.pickle)
-            elsif !message.attachment.nil?
-              # Parse error
-              raise Core::Gherkin::ParseError.new("#{document.uri}: #{message.attachment.data}")
+            elsif message.parse_error
+              raise Core::Gherkin::ParseError.new("#{document.uri}: #{message.parse_error.message}")
             else
               raise "Unknown message: #{message.to_hash}"
             end

data/lib/cucumber/core/gherkin/writer.rb

@@ -60,12 +60,14 @@ module Cucumber
           end
 
           def statements
-            prepare_statements language_statement,
+            prepare_statements(
+              language_statement,
               comments_statement,
               tag_statement,
               name_statement,
               description_statement,
               NEW_LINE
+            )
           end
 
           def language_statement
@@ -85,10 +87,12 @@ module Cucumber
 
           private
           def statements
-            prepare_statements comments_statement,
+            prepare_statements(
+              comments_statement,
               tag_statement,
               name_statement,
               description_statement
+            )
           end
         end
 
@@ -104,10 +108,12 @@ module Cucumber
 
           private
           def statements
-            prepare_statements comments_statement,
+            prepare_statements(
+              comments_statement,
               name_statement,
               description_statement,
               NEW_LINE
+            )
           end
         end
 
@@ -123,10 +129,12 @@ module Cucumber
 
           private
           def statements
-            prepare_statements comments_statement,
+            prepare_statements(
+              comments_statement,
               tag_statement,
               name_statement,
               description_statement
+            )
           end
         end
 
@@ -235,12 +243,14 @@ module Cucumber
 
           private
           def statements
-            prepare_statements NEW_LINE,
+            prepare_statements(
+              NEW_LINE,
               comments_statement,
               tag_statement,
               name_statement,
               description_statement,
               row_statements(2)
+            )
           end
         end
       end