Revert to all-steps-first scenario execution

To fix the missmatch between the PrettyFormatter from the Gherkin
library and Cucumber-JVM, by reverting the scenario execution to
all-steps-first execution as in Cucumber-JVM 1.1.2. This is possible
since the JSONFormatter in the Gherkin library in Gherkin 2.12.0
supports all-steps-first execution.

Author: brasmusson

core/src/main/java/cucumber/runtime/model/CucumberScenario.java

@@ -37,7 +37,7 @@ public void run(Formatter formatter, Reporter reporter, Runtime runtime) {
 
         runBackground(formatter, reporter, runtime);
         format(formatter);
-        runSteps(formatter, reporter, runtime);
+        runSteps(reporter, runtime);
 
         runtime.runAfterHooks(reporter, tags);
         runtime.disposeBackendWorlds();
@@ -46,7 +46,7 @@ public void run(Formatter formatter, Reporter reporter, Runtime runtime) {
     private void runBackground(Formatter formatter, Reporter reporter, Runtime runtime) {
         if (cucumberBackground != null) {
             cucumberBackground.format(formatter);
-            cucumberBackground.runSteps(formatter, reporter, runtime);
+            cucumberBackground.runSteps(reporter, runtime);
         }
     }
 }

core/src/main/java/cucumber/runtime/model/CucumberScenarioOutline.java

@@ -38,7 +38,6 @@ public List<CucumberExamples> getCucumberExamplesList() {
     @Override
     public void run(Formatter formatter, Reporter reporter, Runtime runtime) {
         format(formatter);
-        formatSteps(formatter);
         for (CucumberExamples cucumberExamples : cucumberExamplesList) {
             cucumberExamples.format(formatter);
             List<CucumberScenario> exampleScenarios = cucumberExamples.createExampleScenarios();

core/src/main/java/cucumber/runtime/model/StepContainer.java

@@ -29,22 +29,18 @@ public void step(Step step) {
 
     void format(Formatter formatter) {
         statement.replay(formatter);
-    }
-
-    void formatSteps(Formatter formatter) {
         for (Step step : getSteps()) {
             formatter.step(step);
         }
     }
 
-    void runSteps(Formatter formatter, Reporter reporter, Runtime runtime) {
+    void runSteps(Reporter reporter, Runtime runtime) {
         for (Step step : getSteps()) {
-            runStep(step, formatter, reporter, runtime);
+            runStep(step, reporter, runtime);
         }
     }
 
-    void runStep(Step step, Formatter formatter, Reporter reporter, Runtime runtime) {
-        formatter.step(step);
+    void runStep(Step step, Reporter reporter, Runtime runtime) {
         runtime.runStep(cucumberFeature.getUri(), step, reporter, cucumberFeature.getI18n());
     }
 }