List all scenarios making the exit code non-zero

Currently only failed scenarios are listed in the summary, but also
scenarios of other result types can force the exit code to be non-zero.
Therefore list the scenarios of all result type the each would have made
the exit code non-zero.

Author: brasmusson

features/docs/cli/dry_run.feature

@@ -64,6 +64,9 @@ Feature: Dry Run
             Undefined step: "this step is undefined" (Cucumber::Undefined)
             features/test.feature:3:in `Given this step is undefined'
 
+      Undefined Scenarios:
+      cucumber features/test.feature:2 # Scenario: 
+
       1 scenario (1 undefined)
       1 step (1 undefined)
 

features/docs/cli/strict_mode.feature

@@ -28,6 +28,9 @@ Feature: Strict mode
             Undefined step: "this step passes" (Cucumber::Undefined)
             features/missing.feature:3:in `Given this step passes'
 
+      Undefined Scenarios:
+      cucumber features/missing.feature:2
+
       1 scenario (1 undefined)
       1 step (1 undefined)
       """
@@ -45,6 +48,9 @@ Feature: Strict mode
             ./features/step_definitions/steps.rb:3:in `/^this step is pending$/'
             features/pending.feature:3:in `Given this step is pending'
 
+      Pending Scenarios:
+      cucumber features/pending.feature:2
+
       1 scenario (1 pending)
       1 step (1 pending)
       """

lib/cucumber/formatter/console_issues.rb

@@ -6,28 +6,42 @@ class ConsoleIssues
       include Console
 
       def initialize(config)
-        @failures = []
+        @issues = Hash.new { |h, k| h[k] = [] }
         @config = config
         @config.on_event(:test_case_finished) do |event|
-          @failures << event.test_case if event.result.failed?
+          @issues[event.result.to_sym] << event.test_case unless event.result.ok?(@config.strict?)
         end
       end
 
       def to_s
-        return if @failures.empty?
-        result = [ format_string('Failing Scenarios:', :failed) ] + @failures.map { |failure|
-          source = @config.source? ? format_string(" # #{failure.keyword}: #{failure.name}", :comment) : ''
-          format_string("cucumber #{profiles_string}" + failure.location, :failed) + source
-        }
-        result.join("\n")
+        return if @issues.empty?
+        result = Cucumber::Core::Test::Result::TYPES.map { |type| scenario_listing(type, @issues[type]) }
+        result.flatten.join("\n")
       end
 
       def any?
-        @failures.any?
+        @issues.any?
       end
 
       private
 
+      def scenario_listing(type, test_cases)
+        return [] if test_cases.empty?
+        [ format_string("#{type_heading(type)} Scenarios:", type) ] + test_cases.map { |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
+
+      def type_heading(type)
+        case type
+        when :failed
+          'Failing'
+        else
+          type.to_s.slice(0, 1).capitalize + type.to_s.slice(1..-1)
+        end
+      end
+
       def profiles_string
         return if @config.custom_profiles.empty?
         @config.custom_profiles.map { |profile| "-p #{profile}" }.join(' ') + ' '