cucumber 6.0.0 → 8.0.0.rc.1

data/lib/cucumber/cli/options.rb

@@ -24,8 +24,12 @@ module Cucumber
                                                               "#{INDENT}the usage formatter, except that steps are not printed."],
         'junit'       => ['Cucumber::Formatter::Junit',       "Generates a report similar to Ant+JUnit. Use\n" \
                                                               "#{INDENT}junit,fileattribute=true to include a file attribute."],
-        'json'        => ['Cucumber::Formatter::Json',        '[DEPRECATED] Prints the feature as JSON'],
-        'message'     => ['Cucumber::Formatter::Message',     'Outputs protobuf messages'],
+        'json'        => ['Cucumber::Formatter::Json',        "Prints the feature as JSON.\n" \
+                                                              "#{INDENT}The JSON format is in maintenance mode.\n" \
+                                                              "#{INDENT}Please consider using the message formatter\n"\
+                                                              "#{INDENT}with the standalone json-formatter\n" \
+                                                              "#{INDENT}(https://github.com/cucumber/cucumber/tree/master/json-formatter)."],
+        'message'     => ['Cucumber::Formatter::Message',     'Prints each message in NDJSON form, which can then be consumed by other tools.'],
         'html'        => ['Cucumber::Formatter::HTML',        'Outputs HTML report'],
         'summary'     => ['Cucumber::Formatter::Summary',     'Summary output of feature and scenarios']
       }.freeze
@@ -59,13 +63,13 @@ module Cucumber
         '--lines', '--port', '-I', '--snippet-type'
       ].freeze
       ORDER_TYPES = %w[defined random].freeze
-      TAG_LIMIT_MATCHER = /(?<tag_name>\@\w+):(?<limit>\d+)/x
+      TAG_LIMIT_MATCHER = /(?<tag_name>@\w+):(?<limit>\d+)/x
 
       def self.parse(args, out_stream, error_stream, options = {})
         new(out_stream, error_stream, options).parse!(args)
       end
 
-      def initialize(out_stream = STDOUT, error_stream = STDERR, options = {})
+      def initialize(out_stream = $stdout, error_stream = $stderr, options = {})
         @out_stream   = out_stream
         @error_stream = error_stream
 
@@ -176,6 +180,7 @@ Specify SEED to reproduce the shuffling from a previous run.
       def check_formatter_stream_conflicts
         streams = @options[:formats].uniq.map { |(_, _, stream)| stream }
         return if streams == streams.uniq
+
         raise 'All but one formatter must use --out, only one can print to each stream (or STDOUT)'
       end
 
@@ -199,6 +204,7 @@ Specify SEED to reproduce the shuffling from a previous run.
 
       def truthy_string?(str)
         return false if str.nil?
+
         str !~ /^(false|no|0)$/i
       end
 
@@ -363,6 +369,7 @@ Specify SEED to reproduce the shuffling from a previous run.
       def require_files(v)
         @options[:require] << v
         return unless Cucumber::JRUBY && File.directory?(v)
+
         require 'java'
         $CLASSPATH << v
       end
@@ -381,6 +388,7 @@ Specify SEED to reproduce the shuffling from a previous run.
         require 'gherkin/dialect'
 
         return indicate_invalid_language_and_exit(lang) unless ::Gherkin::DIALECTS.key?(lang)
+
         list_keywords_and_exit(lang)
       end
 
@@ -399,6 +407,7 @@ Specify SEED to reproduce the shuffling from a previous run.
       def add_tag(value)
         raise("Found tags option '#{value}'. '~@tag' is no longer supported, use 'not @tag' instead.") if value.include?('~')
         raise("Found tags option '#{value}'. '@tag1,@tag2' is no longer supported, use '@tag or @tag2' instead.") if value.include?(',')
+
         @options[:tag_expressions] << value.gsub(/(@\w+)(:\d+)?/, '\1')
         add_tag_limits(value)
       end
@@ -411,6 +420,7 @@ Specify SEED to reproduce the shuffling from a previous run.
 
       def add_tag_limit(tag_limits, tag_name, limit)
         raise "Inconsistent tag limits for #{tag_name}: #{tag_limits[tag_name]} and #{limit}" if tag_limits[tag_name] && tag_limits[tag_name] != limit
+
         tag_limits[tag_name] = limit
       end
 

data/lib/cucumber/cli/profile_loader.rb

@@ -75,11 +75,7 @@ Defined profiles in cucumber.yml:
       def process_configuration_file_with_erb
         require 'erb'
         begin
-          @cucumber_erb = if RUBY_VERSION >= '2.6'
-                            ERB.new(IO.read(cucumber_file), trim_mode: '%').result(binding)
-                          else
-                            ERB.new(IO.read(cucumber_file), nil, '%').result(binding)
-                          end
+          @cucumber_erb = ERB.new(IO.read(cucumber_file), trim_mode: '%').result(binding)
         rescue StandardError
           raise(YmlLoadError, "cucumber.yml was found, but could not be parsed with ERB.  Please refer to cucumber's documentation on correct profile usage.\n#{$ERROR_INFO.inspect}")
         end

data/lib/cucumber/cli/rerun_file.rb

@@ -10,7 +10,7 @@ module Cucumber
       end
 
       def self.real_path(path)
-        path[1..-1] # remove leading @
+        path[1..]
       end
 
       def initialize(path)

data/lib/cucumber/configuration.rb

@@ -176,12 +176,12 @@ module Cucumber
     end
 
     def support_to_load
-      support_files = all_files_to_load.select { |f| f =~ %r{/support/} }
+      support_files = all_files_to_load.select { |f| f =~ /\/support\// }
 
       # env_files are separated from other_files so we can ensure env files
       # load first.
       #
-      env_files = support_files.select { |f| f =~ %r{/support/env\..*} }
+      env_files = support_files.select { |f| f =~ /\/support\/env\..*/ }
       other_files = support_files - env_files
       env_files.reverse + other_files.reverse
     end
@@ -189,7 +189,7 @@ module Cucumber
     def all_files_to_load
       files = require_dirs.map do |path|
         path = path.tr('\\', '/') # In case we're on windows. Globs don't work with backslashes.
-        path = path.gsub(/\/$/, '') # Strip trailing slash. # rubocop:disable Style/RegexpLiteral
+        path = path.gsub(/\/$/, '') # Strip trailing slash.
         File.directory?(path) ? Dir["#{path}/**/*"] : path
       end.flatten.uniq
       remove_excluded_files_from(files)
@@ -200,7 +200,7 @@ module Cucumber
     end
 
     def step_defs_to_load
-      all_files_to_load.reject { |f| f =~ %r{/support/} }
+      all_files_to_load.reject { |f| f =~ /\/support\// }
     end
 
     def formatter_factories
@@ -283,6 +283,7 @@ module Cucumber
 
     def with_default_features_path(paths)
       return default_features_paths if paths.empty?
+
       paths
     end
 

data/lib/cucumber/constantize.rb

@@ -2,7 +2,7 @@
 
 require 'cucumber/platform'
 module Cucumber
-  module Constantize #:nodoc:
+  module Constantize # :nodoc:
     def constantize(camel_cased_word)
       try = 0
       begin

data/lib/cucumber/deprecate.rb

@@ -20,6 +20,7 @@ module Cucumber
     class CliOption
       def self.deprecate(stream, option, message, remove_after_version)
         return if stream.nil?
+
         stream.puts(
           AnsiString.failure_message(
             "\nWARNING: #{option} is deprecated" \
@@ -31,7 +32,7 @@ module Cucumber
 
     module ForUsers
       def self.call(message, method, remove_after_version)
-        STDERR.puts AnsiString.failure_message(
+        $stderr.puts AnsiString.failure_message(
           "\nWARNING: ##{method} is deprecated" \
           " and will be removed after version #{remove_after_version}. #{message}.\n" \
           "(Called from #{caller(3..3).first})"

data/lib/cucumber/errors.rb

@@ -9,7 +9,7 @@ module Cucumber
       return result.with_message(with_prefix(result.message)) if result.is_a?(self)
 
       begin
-        raise new(with_prefix(step_name)) # rubocop:disable Style/RaiseArgs
+        raise self, with_prefix(step_name)
       rescue StandardError => e
         e
       end

data/lib/cucumber/events/hook_test_step_created.rb

@@ -6,8 +6,7 @@ module Cucumber
   module Events
     # Event fired when a step is created from a hook
     class HookTestStepCreated < Core::Event.new(:test_step, :hook)
-      attr_reader :test_step
-      attr_reader :hook
+      attr_reader :test_step, :hook
     end
   end
 end

data/lib/cucumber/events/step_activated.rb

@@ -15,12 +15,6 @@ module Cucumber
       #
       # @return [Cucumber::StepMatch]
       attr_reader :step_match
-
-      # @private
-      def initialize(test_step, step_match)
-        @test_step = test_step
-        @step_match = step_match
-      end
     end
   end
 end

data/lib/cucumber/events/step_definition_registered.rb

@@ -10,11 +10,6 @@ module Cucumber
       #
       # @return [RbSupport::RbStepDefinition]
       attr_reader :step_definition
-
-      # _@private
-      def initialize(step_definition)
-        @step_definition = step_definition
-      end
     end
   end
 end

data/lib/cucumber/events/test_case_created.rb

@@ -6,8 +6,7 @@ module Cucumber
   module Events
     # Event fired when a Test::Case is created from a Pickle
     class TestCaseCreated < Core::Event.new(:test_case, :pickle)
-      attr_reader :test_case
-      attr_reader :pickle
+      attr_reader :test_case, :pickle
     end
   end
 end

data/lib/cucumber/events/test_run_finished.rb

@@ -5,7 +5,8 @@ require 'cucumber/core/events'
 module Cucumber
   module Events
     # Event fired after all test cases have finished executing
-    class TestRunFinished < Core::Event.new
+    class TestRunFinished < Core::Event.new(:success)
+      attr_reader :success
     end
   end
 end

data/lib/cucumber/events/test_step_created.rb

@@ -6,8 +6,7 @@ module Cucumber
   module Events
     # Event fired when a TestStep is created from a PickleStep
     class TestStepCreated < Core::Event.new(:test_step, :pickle_step)
-      attr_reader :test_step
-      attr_reader :pickle_step
+      attr_reader :test_step, :pickle_step
     end
   end
 end

data/lib/cucumber/events/undefined_parameter_type.rb

@@ -3,8 +3,7 @@ require 'cucumber/core/events'
 module Cucumber
   module Events
     class UndefinedParameterType < Core::Event.new(:type_name, :expression)
-      attr_reader :type_name
-      attr_reader :expression
+      attr_reader :type_name, :expression
     end
   end
 end

data/lib/cucumber/events.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-Dir[File.dirname(__FILE__) + '/events/*.rb'].map(&method(:require))
+Dir["#{File.dirname(__FILE__)}/events/*.rb"].map(&method(:require))
 
 module Cucumber
   # Events tell you what's happening while Cucumber runs your features.
@@ -11,7 +11,7 @@ module Cucumber
   # To subscribe to an event, use {Cucumber::Configuration#on_event}
   #
   # @example
-  #   AfterConfiguration do |config|
+  #   InstallPlugin do |config|
   #     config.on_event :test_case_finished do |event|
   #       puts event.result
   #     end

data/lib/cucumber/file_specs.rb

@@ -5,7 +5,7 @@ require 'cucumber/core/test/location'
 
 module Cucumber
   class FileSpecs
-    FILE_COLON_LINE_PATTERN = /^([\w\W]*?)(?::([\d:]+))?$/ #:nodoc:
+    FILE_COLON_LINE_PATTERN = /^([\w\W]*?)(?::([\d:]+))?$/ # :nodoc:
 
     def initialize(file_specs)
       Cucumber.logger.debug("Features:\n")
@@ -32,6 +32,7 @@ module Cucumber
 
       def locations
         return [Core::Test::Location.new(@file)] if @lines.empty?
+
         @lines.map { |line| Core::Test::Location.new(@file, line) }
       end
     end

data/lib/cucumber/filters/activate_steps.rb

@@ -52,6 +52,7 @@ module Cucumber
             end
             configuration.notify :step_activated, test_step, match
             return SkippingStepMatch.new if configuration.dry_run?
+
             match
           end
 

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

@@ -47,9 +47,7 @@ module Cucumber
             locations.count
           end
 
-          attr_reader :tag_name
-          attr_reader :limit
-          attr_reader :locations
+          attr_reader :tag_name, :limit, :locations
         end
       end
     end

data/lib/cucumber/filters/tag_limits.rb

@@ -38,9 +38,7 @@ module Cucumber
 
       private
 
-      attr_reader :gated_receiver
-      attr_reader :test_case_index
-      attr_reader :verifier
+      attr_reader :gated_receiver, :test_case_index, :verifier
     end
   end
 end

data/lib/cucumber/formatter/ansicolor.rb

@@ -3,28 +3,48 @@
 require 'cucumber/platform'
 require 'cucumber/term/ansicolor'
 
-if Cucumber::WINDOWS_MRI
-  unless ENV['ANSICON']
-    STDERR.puts %{*** WARNING: You must use ANSICON 1.31 or higher (https://github.com/adoxa/ansicon/) to get coloured output on Windows}
-    Cucumber::Term::ANSIColor.coloring = false
-  end
-end
-
-Cucumber::Term::ANSIColor.coloring = false if !STDOUT.tty? && !ENV.key?('AUTOTEST')
+Cucumber::Term::ANSIColor.coloring = false if !$stdout.tty? && !ENV.key?('AUTOTEST')
 
 module Cucumber
   module Formatter
-    # Defines aliases for coloured output. You don't invoke any methods from this
-    # module directly, but you can change the output colours by defining
-    # a <tt>CUCUMBER_COLORS</tt> variable in your shell, very much like how you can
-    # tweak the familiar POSIX command <tt>ls</tt> with
-    # <a href="http://mipsisrisc.com/rambling/2008/06/27/lscolorsls_colors-now-with-linux-support/">$LSCOLORS/$LS_COLORS</a>
+    # This module allows to format cucumber related outputs using ANSI escape sequences.
+    #
+    # For example, it provides a `passed` method which returns the string with
+    # the ANSI escape sequence to format it green per default.
+    #
+    # To use this, include or extend it in your class.
+    #
+    # Example:
+    #
+    #   require 'cucumber/formatter/ansicolor'
+    #
+    #   class MyFormatter
+    #     extend Cucumber::Term::ANSIColor
+    #
+    #     def on_test_step_finished(event)
+    #       $stdout.puts undefined(event.test_step) if event.result.undefined?
+    #       $stdout.puts passed(event.test_step) if event.result.passed?
+    #     end
+    #   end
+    #
+    # This module also allows the user to customize the format of cucumber outputs
+    # using environment variables.
+    #
+    # For instance, if your shell has a black background and a green font (like the
+    # "Homebrew" settings for OS X' Terminal.app), you may want to override passed
+    # steps to be white instead of green.
+    #
+    # Example:
+    #
+    #   export CUCUMBER_COLORS="passed=white,bold:passed_param=white,bold,underline"
     #
     # The colours that you can change are:
     #
     # * <tt>undefined</tt>     - defaults to <tt>yellow</tt>
     # * <tt>pending</tt>       - defaults to <tt>yellow</tt>
     # * <tt>pending_param</tt> - defaults to <tt>yellow,bold</tt>
+    # * <tt>flaky</tt>         - defaults to <tt>yellow</tt>
+    # * <tt>flaky_param</tt>   - defaults to <tt>yellow,bold</tt>
     # * <tt>failed</tt>        - defaults to <tt>red</tt>
     # * <tt>failed_param</tt>  - defaults to <tt>red,bold</tt>
     # * <tt>passed</tt>        - defaults to <tt>green</tt>
@@ -36,26 +56,18 @@ module Cucumber
     # * <tt>comment</tt>       - defaults to <tt>grey</tt>
     # * <tt>tag</tt>           - defaults to <tt>cyan</tt>
     #
-    # For instance, if your shell has a black background and a green font (like the
-    # "Homebrew" settings for OS X' Terminal.app), you may want to override passed
-    # steps to be white instead of green.
-    #
-    # Although not listed, you can also use <tt>grey</tt>.
-    #
-    # Examples: (On Windows, use SET instead of export.)
-    #
-    #   export CUCUMBER_COLORS="passed=white"
-    #   export CUCUMBER_COLORS="passed=white,bold:passed_param=white,bold,underline"
-    #
     # To see what colours and effects are available, just run this in your shell:
     #
-    #   ruby -e "require 'rubygems'; require 'term/ansicolor'; puts Cucumber::Term::ANSIColor.attributes"
+    #   ruby -e "require 'rubygems'; require 'cucumber/term/ansicolor'; puts Cucumber::Term::ANSIColor.attributes"
     #
     module ANSIColor
       include Cucumber::Term::ANSIColor
 
+      # :stopdoc:
       ALIASES = Hash.new do |h, k|
-        h[Regexp.last_match(1)] + ',bold' if k.to_s =~ /(.*)_param/
+        next unless k.to_s =~ /(.*)_param/
+
+        "#{h[Regexp.last_match(1)]},bold"
       end.merge(
         'undefined' => 'yellow',
         'pending'   => 'yellow',
@@ -67,15 +79,22 @@ module Cucumber
         'comment'   => 'grey',
         'tag'       => 'cyan'
       )
+      # :startdoc:
 
-      if ENV['CUCUMBER_COLORS'] # Example: export CUCUMBER_COLORS="passed=red:failed=yellow"
-        ENV['CUCUMBER_COLORS'].split(':').each do |pair|
+      # Apply the custom color scheme
+      #
+      # example:
+      #
+      #   apply_custom_colors('passed=white')
+      def apply_custom_colors(colors)
+        colors.split(':').each do |pair|
           a = pair.split('=')
           ALIASES[a[0]] = a[1]
         end
       end
+      apply_custom_colors(ENV['CUCUMBER_COLORS']) if ENV['CUCUMBER_COLORS']
 
-      # Eval to define the color-named methods required by Term::ANSIColor.
+      # Define the color-named methods required by Term::ANSIColor.
       #
       # Examples:
       #
@@ -87,53 +106,18 @@ module Cucumber
       #     red(bold(string, &proc)) + red
       #   end
       ALIASES.each_key do |method_name|
-        next if method_name =~ /.*_param/
-        code = <<-COLOR
-          def #{method_name}(string=nil, &proc)
-            #{ALIASES[method_name].split(',').join('(') + '(string, &proc' + ')' * ALIASES[method_name].split(',').length}
-          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(' + ')}
-          end
-        COLOR
-        eval(code) # rubocop:disable Security/Eval
-      end
+        next if method_name.end_with?('_param')
 
-      def self.define_grey #:nodoc:
-        gem 'genki-ruby-terminfo'
-        require 'terminfo'
-        case TermInfo.default_object.tigetnum('colors')
-        when 0
-          raise "Your terminal doesn't support colours."
-        when 1
-          ::Cucumber::Term::ANSIColor.coloring = false
-          alias_method :grey, :white
-        when 2..8
-          alias_method :grey, :white # rubocop:disable Lint/DuplicateMethods
-        else
-          define_real_grey
-        end
-      rescue Exception => e # rubocop:disable Lint/RescueException
-        if e.class.name == 'TermInfo::TermInfoError'
-          STDERR.puts '*** WARNING ***'
-          STDERR.puts "You have the genki-ruby-terminfo gem installed, but you haven't set your TERM variable."
-          STDERR.puts 'Try setting it to TERM=xterm-256color to get grey colour in output.'
-          STDERR.puts "\n"
-          alias_method :grey, :white
-        else
-          define_real_grey
+        define_method(method_name) do |text = nil, &proc|
+          apply_styles(ALIASES[method_name], text, &proc)
         end
-      end
 
-      def self.define_real_grey #:nodoc:
-        define_method :grey do |string|
-          ::Cucumber::Term::ANSIColor.coloring? ? "\e[90m#{string}\e[0m" : string
+        define_method("#{method_name}_param") do |text = nil, &proc|
+          apply_styles(ALIASES["#{method_name}_param"], text, &proc) + apply_styles(ALIASES[method_name])
         end
       end
 
-      define_grey
-
+      # :stopdoc:
       def cukes(n)
         ('(::) ' * n).strip
       end
@@ -149,6 +133,15 @@ module Cucumber
       def yellow_cukes(n)
         blink(yellow(cukes(n)))
       end
+      # :startdoc:
+
+      private
+
+      def apply_styles(styles, text = nil, &proc)
+        styles.split(',').reverse.reduce(text) do |result, method_name|
+          send(method_name, result, &proc)
+        end
+      end
     end
   end
 end

data/lib/cucumber/formatter/ast_lookup.rb

@@ -45,11 +45,11 @@ module Cucumber
             break
           end
           break if node.previous_node.nil?
+
           node = node.previous_node
         end
         keyword = dialect.given_keywords.reject { |kw| kw == '* ' }[0] if keyword.nil?
-        keyword = Cucumber::Gherkin::I18n.code_keyword_for(keyword)
-        keyword
+        Cucumber::Gherkin::I18n.code_keyword_for(keyword)
       end
 
       ScenarioSource = Struct.new(:type, :scenario)

data/lib/cucumber/formatter/backtrace_filter.rb

@@ -33,7 +33,7 @@ module Cucumber
       def exception
         return @exception if ::Cucumber.use_full_backtrace
 
-        pwd_pattern = /#{::Regexp.escape(::Dir.pwd)}\//m # rubocop:disable Style/RegexpLiteral
+        pwd_pattern = /#{::Regexp.escape(::Dir.pwd)}\//m
         backtrace = @exception.backtrace.map { |line| line.gsub(pwd_pattern, './') }
 
         filtered = (backtrace || []).reject do |line|

data/lib/cucumber/formatter/console.rb

@@ -34,7 +34,7 @@ module Cucumber
 
       def format_step(keyword, step_match, status, source_indent)
         comment = if source_indent
-                    c = ('# ' + step_match.location.to_s).indent(source_indent)
+                    c = indent("# #{step_match.location}", source_indent)
                     format_string(c, :comment)
                   else
                     ''
@@ -99,17 +99,20 @@ module Cucumber
         @io.puts(format_string(string, status))
       end
 
-      def exception_message_string(e, indent)
+      def exception_message_string(e, indent_amount)
         message = "#{e.message} (#{e.class})".dup.force_encoding('UTF-8')
         message = linebreaks(message, ENV['CUCUMBER_TRUNCATE_OUTPUT'].to_i)
 
-        "#{message}\n#{e.backtrace.join("\n")}".indent(indent)
+        indent("#{message}\n#{e.backtrace.join("\n")}", indent_amount)
       end
 
       # http://blade.nagaokaut.ac.jp/cgi-bin/scat.rb/ruby/ruby-talk/10655
       def linebreaks(msg, max)
         return msg unless max && max > 0
-        msg.gsub(/.{1,#{max}}(?:\s|\Z)/) { ($& + 5.chr).gsub(/\n\005/, "\n").gsub(/\005/, "\n") }.rstrip
+
+        msg.gsub(/.{1,#{max}}(?:\s|\Z)/) do
+          (Regexp.last_match(0) + 5.chr).gsub(/\n\005/, "\n").gsub(/\005/, "\n")
+        end.rstrip
       end
 
       def collect_snippet_data(test_step, ast_lookup)
@@ -150,6 +153,7 @@ module Cucumber
 
       def print_passing_wip(config, passed_test_cases, ast_lookup)
         return unless config.wip?
+
         messages = passed_test_cases.map do |test_case|
           scenario_source = ast_lookup.scenario_source(test_case)
           keyword = scenario_source.type == :Scenario ? scenario_source.scenario.keyword : scenario_source.scenario_outline.keyword
@@ -170,6 +174,7 @@ module Cucumber
       def attach(src, media_type)
         return unless media_type == 'text/x.cucumber.log+plain'
         return unless @io
+
         @io.puts
         @io.puts(format_string(src, :tag))
         @io.flush
@@ -177,6 +182,7 @@ module Cucumber
 
       def print_profile_information
         return if @options[:skip_profile_information] || @options[:profiles].nil? || @options[:profiles].empty?
+
         do_print_profile_information(@options[:profiles])
       end
 
@@ -208,6 +214,14 @@ module Cucumber
         ].join("\n")
       end
 
+      def indent(string, padding)
+        if padding >= 0
+          string.gsub(/^/, ' ' * padding)
+        else
+          string.gsub(/^ {0,#{-padding}}/, '')
+        end
+      end
+
       private
 
       FORMATS = Hash.new { |hash, format| hash[format] = method(format).to_proc }
@@ -216,6 +230,7 @@ module Cucumber
         key = keys.join('_').to_sym
         fmt = FORMATS[key]
         raise "No format for #{key.inspect}: #{FORMATS.inspect}" if fmt.nil?
+
         fmt
       end
 
@@ -238,6 +253,7 @@ module Cucumber
 
       class SnippetData
         attr_reader :actual_keyword, :step
+
         def initialize(actual_keyword, step)
           @actual_keyword = actual_keyword
           @step = step

data/lib/cucumber/formatter/console_issues.rb

@@ -23,6 +23,7 @@ module Cucumber
 
       def to_s
         return if @issues.empty?
+
         result = Core::Test::Result::TYPES.map { |type| scenario_listing(type, @issues[type]) }
         result.flatten.join("\n")
       end
@@ -35,6 +36,7 @@ module Cucumber
 
       def scenario_listing(type, test_cases)
         return [] if test_cases.empty?
+
         [format_string("#{type_heading(type)} Scenarios:", type)] + test_cases.map do |test_case|
           scenario_source = @ast_lookup.scenario_source(test_case)
           keyword = scenario_source.type == :Scenario ? scenario_source.scenario.keyword : scenario_source.scenario_outline.keyword
@@ -54,7 +56,10 @@ module Cucumber
 
       def profiles_string
         return if @config.custom_profiles.empty?
-        @config.custom_profiles.map { |profile| "-p #{profile}" }.join(' ') + ' '
+
+        profiles = @config.custom_profiles.map { |profile| "-p #{profile}" }.join(' ')
+
+        "#{profiles} "
       end
     end
   end