cucumber 3.0.1 → 3.2.0

data/lib/cucumber/filters/gated_receiver.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
+
 module Cucumber
   module Filters
-
     class GatedReceiver
       def initialize(receiver)
         @receiver = receiver
@@ -21,6 +21,5 @@ module Cucumber
         self
       end
     end
-
   end
 end

data/lib/cucumber/filters/prepare_world.rb

@@ -1,13 +1,12 @@
 # frozen_string_literal: true
+
 require 'cucumber/core/filter'
 require 'cucumber/core/ast/location'
 require 'cucumber/running_test_case'
 
 module Cucumber
   module Filters
-
     class PrepareWorld < Core::Filter.new(:runtime)
-
       def test_case(test_case)
         CaseFilter.new(runtime, test_case).test_case.describe_to receiver
       end
@@ -25,7 +24,7 @@ module Cucumber
           end
           around_hooks = [init_scenario] + @original_test_case.around_hooks
 
-          empty_hook = proc {} #no op - legacy format adapter expects a before hooks
+          empty_hook = proc {} # no op - legacy format adapter expects a before hooks
           empty_hook_location = Cucumber::Core::Ast::Location.from_source_location(*empty_hook.source_location)
           default_hook = Cucumber::Hooks.before_hook(@original_test_case.source, empty_hook_location, &empty_hook)
           steps = [default_hook] + @original_test_case.test_steps
@@ -39,8 +38,6 @@ module Cucumber
           @scenario ||= RunningTestCase.new(test_case)
         end
       end
-
     end
-
   end
 end

data/lib/cucumber/filters/quit.rb

@@ -1,9 +1,9 @@
 # frozen_string_literal: true
+
 module Cucumber
   module Filters
-
     class Quit
-      def initialize(receiver=nil)
+      def initialize(receiver = nil)
         @receiver = receiver
       end
 
@@ -23,7 +23,5 @@ module Cucumber
         self.class.new(receiver)
       end
     end
-
   end
 end
-

data/lib/cucumber/filters/randomizer.rb

@@ -1,12 +1,12 @@
 # frozen_string_literal: true
+
 require 'digest/sha2'
 
 module Cucumber
   module Filters
-
-    # Batches up all test cases, randomizes them, and then sends them on
+    # Batches up all test cases, randomizes them, and then sends them on
     class Randomizer
-      def initialize(seed, receiver=nil)
+      def initialize(seed, receiver = nil)
         @receiver = receiver
         @test_cases = []
         @seed = seed
@@ -33,14 +33,13 @@ module Cucumber
 
       def shuffled_test_cases
         digester = Digest::SHA2.new(256)
-        @test_cases.map.with_index.
-          sort_by { |_, index| digester.digest((@seed + index).to_s) }.
-          map { |test_case, _| test_case }
+        @test_cases.map.with_index
+                   .sort_by { |_, index| digester.digest((@seed + index).to_s) }
+                   .map { |test_case, _| test_case }
       end
 
       attr_reader :seed
       private :seed
     end
-
   end
 end

data/lib/cucumber/filters/retry.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'cucumber/core/filter'
 require 'cucumber/running_test_case'
 require 'cucumber/events'
@@ -6,7 +7,6 @@ require 'cucumber/events'
 module Cucumber
   module Filters
     class Retry < Core::Filter.new(:configuration)
-
       def test_case(test_case)
         configuration.on_event(:test_case_finished) do |event|
           next unless retry_required?(test_case, event)
@@ -25,7 +25,7 @@ module Cucumber
       end
 
       def test_case_counts
-        @test_case_counts ||= Hash.new {|h,k| h[k] = 0 }
+        @test_case_counts ||= Hash.new { |h, k| h[k] = 0 }
       end
     end
   end

data/lib/cucumber/filters/tag_limits.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'cucumber/filters/gated_receiver'
 require 'cucumber/filters/tag_limits/test_case_index'
 require 'cucumber/filters/tag_limits/verifier'
@@ -12,7 +13,7 @@ module Cucumber
     end
 
     class TagLimits
-      def initialize(tag_limits, receiver=nil)
+      def initialize(tag_limits, receiver = nil)
         @tag_limits = tag_limits
         @gated_receiver = GatedReceiver.new(receiver)
         @test_case_index = TestCaseIndex.new
@@ -41,6 +42,5 @@ module Cucumber
       attr_reader :test_case_index
       attr_reader :verifier
     end
-
   end
 end

data/lib/cucumber/filters/tag_limits/test_case_index.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
+
 module Cucumber
   module Filters
     class TagLimits
-
       class TestCaseIndex
         def initialize
           @index = Hash.new { |hash, key| hash[key] = [] }
@@ -26,7 +26,6 @@ module Cucumber
 
         attr_accessor :index
       end
-
     end
   end
 end

data/lib/cucumber/filters/tag_limits/verifier.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
+
 module Cucumber
   module Filters
     class TagLimits
-
       class Verifier
         def initialize(tag_limits)
           @tag_limits = tag_limits
@@ -53,7 +53,6 @@ module Cucumber
           attr_reader :limit
           attr_reader :locations
         end
-
       end
     end
   end

data/lib/cucumber/formatter/ansicolor.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'cucumber/platform'
 require 'cucumber/term/ansicolor'
 
@@ -53,21 +54,21 @@ module Cucumber
     module ANSIColor
       include Cucumber::Term::ANSIColor
 
-      ALIASES = Hash.new do |h,k|
+      ALIASES = Hash.new do |h, k|
         if k.to_s =~ /(.*)_param/
           h[$1] + ',bold'
         end
       end.merge({
-        'undefined' => 'yellow',
-        'pending'   => 'yellow',
-        'flaky'     => 'yellow',
-        'failed'    => 'red',
-        'passed'    => 'green',
-        'outline'   => 'cyan',
-        'skipped'   => 'cyan',
-        'comment'   => 'grey',
-        'tag'       => 'cyan'
-      })
+                  'undefined' => 'yellow',
+                  'pending'   => 'yellow',
+                  'flaky'     => 'yellow',
+                  'failed'    => 'red',
+                  'passed'    => 'green',
+                  'outline'   => 'cyan',
+                  'skipped'   => 'cyan',
+                  'comment'   => 'grey',
+                  'tag'       => 'cyan'
+                })
 
       if ENV['CUCUMBER_COLORS'] # Example: export CUCUMBER_COLORS="passed=red:failed=yellow"
         ENV['CUCUMBER_COLORS'].split(':').each do |pair|
@@ -95,7 +96,7 @@ module Cucumber
           end
           # This resets the colour to the non-param colour
           def #{method_name}_param(string=nil, &proc)
-            #{ALIASES[method_name+'_param'].split(",").join("(") + "(string, &proc" + ")" * ALIASES[method_name+'_param'].split(",").length} + #{ALIASES[method_name].split(",").join(' + ')}
+            #{ALIASES[method_name + '_param'].split(",").join("(") + "(string, &proc" + ")" * ALIASES[method_name + '_param'].split(",").length} + #{ALIASES[method_name].split(",").join(' + ')}
           end
           EOF
           eval(code)

data/lib/cucumber/formatter/backtrace_filter.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
-require 'cucumber/platform'
 
+require 'cucumber/platform'
 
 module Cucumber
   module Formatter
@@ -49,6 +49,5 @@ module Cucumber
         @exception
       end
     end
-
   end
 end

data/lib/cucumber/formatter/console.rb

@@ -1,11 +1,11 @@
 # frozen_string_literal: true
+
 require 'cucumber/formatter/ansicolor'
 require 'cucumber/formatter/duration'
 require 'cucumber/gherkin/i18n'
 
 module Cucumber
   module Formatter
-
     # This module contains helper methods that are used by formatters that
     # print output to the terminal.
     #
@@ -111,7 +111,7 @@ module Cucumber
       # http://blade.nagaokaut.ac.jp/cgi-bin/scat.rb/ruby/ruby-talk/10655
       def linebreaks(s, max)
         return s unless max && max > 0
-        s.gsub(/.{1,#{max}}(?:\s|\Z)/){($& + 5.chr).gsub(/\n\005/,"\n").gsub(/\005/,"\n")}.rstrip
+        s.gsub(/.{1,#{max}}(?:\s|\Z)/) { ($& + 5.chr).gsub(/\n\005/, "\n").gsub(/\005/, "\n") }.rstrip
       end
 
       def collect_snippet_data(test_step, result)
@@ -127,9 +127,9 @@ module Cucumber
         return unless options[:snippets]
         return if runtime.steps(:undefined).empty?
 
-        snippet_text_proc = lambda { |step_keyword, step_name, multiline_arg|
+        snippet_text_proc = lambda do |step_keyword, step_name, multiline_arg|
           runtime.snippet_text(step_keyword, step_name, multiline_arg)
-        }
+        end
         do_print_snippets(snippet_text_proc)
       end
 
@@ -165,8 +165,8 @@ module Cucumber
         # no-op
       end
 
-      #define @delayed_messages = [] in your Formatter if you want to
-      #activate this feature
+      # define @delayed_messages = [] in your Formatter if you want to
+      # activate this feature
       def puts(*messages)
         if @delayed_messages
           @delayed_messages += messages
@@ -182,7 +182,7 @@ module Cucumber
       end
 
       def print_messages
-        @delayed_messages.each {|message| print_message(message)}
+        @delayed_messages.each { |message| print_message(message) }
         empty_messages
       end
 
@@ -211,12 +211,12 @@ module Cucumber
         profiles_sentence = profiles.size == 1 ? profiles.first :
           "#{profiles[0...-1].join(', ')} and #{profiles.last}"
 
-        @io.puts "Using the #{profiles_sentence} profile#{'s' if profiles.size> 1}..."
+        @io.puts "Using the #{profiles_sentence} profile#{'s' if profiles.size > 1}..."
       end
 
       private
 
-      FORMATS = Hash.new{ |hash, format| hash[format] = method(format).to_proc }
+      FORMATS = Hash.new { |hash, format| hash[format] = method(format).to_proc }
 
       def format_for(*keys)
         key = keys.join('_').to_sym
@@ -241,9 +241,9 @@ module Cucumber
 
       def snippet_text(step_keyword, step_name, multiline_arg)
         keyword = Cucumber::Gherkin::I18n.code_keyword_for(step_keyword).strip
-        config.snippet_generators.map { |generator|
+        config.snippet_generators.map do |generator|
           generator.call(keyword, step_name, multiline_arg, config.snippet_type)
-        }.join("\n")
+        end.join("\n")
       end
 
       class SnippetData
@@ -252,7 +252,6 @@ module Cucumber
           @actual_keyword, @step = actual_keyword, step
         end
       end
-
     end
   end
 end

data/lib/cucumber/formatter/console_counts.rb

@@ -29,14 +29,14 @@ module Cucumber
       end
 
       def status_counts(summary)
-        counts = Core::Test::Result::TYPES.map { |status|
+        counts = Core::Test::Result::TYPES.map do |status|
           count = summary.total(status)
           [status, count]
-        }.select { |status, count|
+        end.select do |status, count|
           count > 0
-        }.map { |status, count|
+        end.map do |status, count|
           format_string("#{count} #{status}", status)
-        }
+        end
         "(#{counts.join(", ")})" if counts.any?
       end
     end

data/lib/cucumber/formatter/console_issues.rb

@@ -14,7 +14,7 @@ module Cucumber
             @previous_test_case = event.test_case
             @issues[event.result.to_sym] << event.test_case unless event.result.ok?(@config.strict)
           elsif event.result.passed?
-            @issues[:flaky] << event.test_case unless Core::Test::Result::Flaky.ok?(@config.strict)
+            @issues[:flaky] << event.test_case unless Core::Test::Result::Flaky.ok?(@config.strict.strict?(:flaky))
             @issues[:failed].delete(event.test_case)
           end
         end
@@ -34,10 +34,10 @@ module Cucumber
 
       def scenario_listing(type, test_cases)
         return [] if test_cases.empty?
-        [ format_string("#{type_heading(type)} Scenarios:", type) ] + test_cases.map { |test_case|
+        [format_string("#{type_heading(type)} Scenarios:", type)] + test_cases.map do |test_case|
           source = @config.source? ? format_string(" # #{test_case.keyword}: #{test_case.name}", :comment) : ''
           format_string("cucumber #{profiles_string}" + test_case.location, type) + source
-        }
+        end
       end
 
       def type_heading(type)

data/lib/cucumber/formatter/duration.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Cucumber
   module Formatter
     module Duration

data/lib/cucumber/formatter/duration_extractor.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
+
 module Cucumber
   module Formatter
-
     class DurationExtractor
       attr_reader :result_duration
       def initialize(result)

data/lib/cucumber/formatter/fail_fast.rb

@@ -1,20 +1,23 @@
 # frozen_string_literal: true
+
 require 'cucumber/formatter/io'
 require 'cucumber/formatter/console'
 
 module Cucumber
   module Formatter
-
     class FailFast
-
       def initialize(configuration)
+        @previous_test_case = nil
         configuration.on_event :test_case_finished do |event|
-          _test_case, result = *event.attributes
-          Cucumber.wants_to_quit = true unless result.ok?(configuration.strict)
+          test_case, result = *event.attributes
+          if test_case != @previous_test_case
+            @previous_test_case = event.test_case
+            Cucumber.wants_to_quit = true unless result.ok?(configuration.strict)
+          elsif result.passed?
+            Cucumber.wants_to_quit = false
+          end
         end
       end
-
     end
-
   end
 end

data/lib/cucumber/formatter/fanout.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
+
 module Cucumber
   module Formatter
-
     # Forwards any messages sent to this object to all recipients
     # that respond to that message.
     class Fanout < BasicObject
@@ -21,8 +21,6 @@ module Cucumber
       def respond_to_missing?(name, include_private = false)
         recipients.any? { |recipient| recipient.respond_to?(name, include_private) }
       end
-
     end
-
   end
 end

data/lib/cucumber/formatter/hook_query_visitor.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Cucumber
   module Formatter
     class HookQueryVisitor