StepwiseExtension: Mark only subsequent features as skipped in case o…

…f failure (spockframework#893)

fixes spockframework#844

spock-core/src/main/java/org/spockframework/runtime/extension/builtin/StepwiseExtension.java

@@ -53,10 +53,12 @@ public void error(ErrorInfo error) {
         // but not sub- and super classes
         if (!error.getMethod().getParent().equals(spec)) return;
 
-        // we can just set skip flag on all features, even though
-        // some of them might already have run
-        for (FeatureInfo feature : spec.getFeatures())
-          feature.setSkipped(true);
+        // mark all subsequent features as skipped
+        List<FeatureInfo> features = spec.getFeatures();
+        int indexOfFailedFeature = features.indexOf(error.getMethod().getFeature());
+        for (int i = indexOfFailedFeature + 1; i < features.size(); i++) {
+          features.get(i).setSkipped(true);
+        }
       }
     });
   }