Use Result#non_zero_exit_status? in the JUnit Formatter

lib/cucumber/formatter/junit.rb

@@ -36,7 +36,7 @@ def before_test_case(test_case)
       def after_test_step(test_step, result)
         return if @failing_step_source
 
-        @failing_step_source = test_step.source.last if result.failed? || (@options[:strict] && (result.pending? || result.undefined?))
+        @failing_step_source = test_step.source.last if result.non_zero_exit_status?(@options[:strict])
       end
 
       def after_test_case(test_case, result)
@@ -86,7 +86,7 @@ def end_feature
 
       def create_output_string(test_case, scenario, result, row_name)
         output = "#{test_case.keyword}: #{scenario}\n\n"
-        return output unless result.failed? || (@options[:strict] && (result.pending? || result.undefined?))
+        return output unless result.non_zero_exit_status?(@options[:strict])
         if test_case.keyword == "Scenario"
           output += "#{@failing_step_source.keyword}" unless hook?(@failing_step_source)
           output += "#{@failing_step_source.name}\n"
@@ -105,10 +105,9 @@ def build_testcase(result, scenario_designation, output)
         @time += duration
         classname = @current_feature.name
         name = scenario_designation
-        pending = (result.pending? || result.undefined?) && (!@options[:strict])
 
         @builder.testcase(:classname => classname, :name => name, :time => "%.6f" % duration) do
-          if result.skipped? || pending
+          if !result.passed? && !result.non_zero_exit_status?(@options[:strict])
             @builder.skipped
             @skipped += 1
           elsif !result.passed?