Fix the bug

When spinach runs a pending step. It will halt the whole scenario and
move onto the next scenario.

features/pending_steps.feature

@@ -7,4 +7,3 @@ Feature: Pending steps
     Given I have a feature that has a pending step
     When I run the feature
     Then the test stops at the pending step and reported as such
-

features/steps/pending_steps.rb

@@ -2,14 +2,16 @@ class Spinach::Features::PendingSteps < Spinach::FeatureSteps
   include Integration::SpinachRunner
 
   step 'I have a feature that has a pending step' do
-    @feature = Integration::FeatureGenerator.pending_feature
+    @feature = Integration::FeatureGenerator.pending_feature_with_multiple_scenario
   end
 
   step 'I run the feature' do
     run_feature @feature
   end
 
   step 'the test stops at the pending step and reported as such' do
-    @stdout.must_match("(0) Successful")
+    @stdout.must_match("(1) Successful")
+    @stdout.must_match("(0) Failed")
+    @stdout.must_match("(1) Pending")
   end
 end

features/support/feature_generator.rb

@@ -22,9 +22,10 @@ def success_feature
       #   The feature file name
       #
       # @api private
-      def pending_feature
-        steps = pending_step_class_str + pending_step + success_step + "\nend"
-        write_feature 'features/success_feature.feature', pending_feature_str,
+      def pending_feature_with_multiple_scenario
+        feature_str = pending_feature_str + "\n" + success_scenario
+        steps = pending_step_class_str + pending_step + "\n" + failure_step_definition + success_step + "\nend"
+        write_feature 'features/success_feature.feature', feature_str,
           'features/steps/success_feature.rb', steps
       end
 
@@ -77,9 +78,14 @@ def write_feature(feature_file, feature, step_file, steps)
       end
 
       def failure_step
-        'class AFailureFeature < Spinach::FeatureSteps
+        %Q|class AFailureFeature < Spinach::FeatureSteps
       feature "A failure feature"
-      Then "I fail" do
+      %{failure_step_definition}
+      |
+      end
+
+      def failure_step_definition
+      'Then "I fail" do
         true.must_equal false
       end'
       end
@@ -88,7 +94,7 @@ def pending_feature_str
         "Feature: A pending feature
         Scenario: This is scenario will be pending
         When this is a pending step
-    Then I succeed"
+    Then I fail"
       end
 
       def success_scenario

lib/spinach/runner/scenario_runner.rb

@@ -51,7 +51,7 @@ def run
           steps.each do |step|
             Spinach.hooks.run_before_step step, step_definitions
 
-            if @exception
+            if @exception || @has_pending_step
               Spinach.hooks.run_on_skipped_step step, step_definitions
             else
               run_step(step)
@@ -84,6 +84,7 @@ def run_step(step)
         Spinach.hooks.run_on_undefined_step step, @exception, step_definitions
       rescue Spinach::StepPendingException => e
         e.step = step
+        @has_pending_step = true
         Spinach.hooks.run_on_pending_step step, e
       rescue Exception => e
         @exception = e

test/spinach/runner/scenario_runner_test.rb

@@ -154,6 +154,11 @@ class Runner
             subject.instance_variable_get(:@exception).must_be_kind_of(NilClass)
           end
 
+          it 'sets the has_pending_step' do
+            subject.run_step(@step)
+            subject.instance_variable_get(:@has_pending_step).must_equal(true)
+          end
+
           it 'runs the pending hooks' do
             Spinach.hooks.expects(:run_on_pending_step).with(@step, kind_of(Spinach::StepPendingException))
             subject.run_step(@step)