Deprecate constructors and run methods of TestStep

Author: mpkorstanje

core/src/main/java/cucumber/api/TestStep.java

@@ -17,11 +17,24 @@ public abstract class TestStep {
             "org.junit.AssumptionViolatedException",
             "org.junit.internal.AssumptionViolatedException"
     };
+
     static {
         Arrays.sort(ASSUMPTION_VIOLATED_EXCEPTIONS);
     }
+
+    /**
+     * @deprecated not part of the public api
+     */
+    @Deprecated
     protected final DefinitionMatch definitionMatch;
 
+    /**
+     * Creates a new test step from the matching step definition
+     *
+     * @param definitionMatch the matching step definition
+     * @deprecated not part of the public api
+     */
+    @Deprecated
     public TestStep(DefinitionMatch definitionMatch) {
         this.definitionMatch = definitionMatch;
     }
@@ -52,6 +65,15 @@ public List<cucumber.runtime.Argument> getDefinitionArgument() {
 
     public abstract HookType getHookType();
 
+    /**
+     * @param bus       to which events should be broadcast
+     * @param language  in which the step is defined
+     * @param scenario  of which this step is part
+     * @param skipSteps if this step should be skipped
+     * @return result of running this step
+     * @deprecated not part of the public api
+     */
+    @Deprecated
     public Result run(EventBus bus, String language, Scenario scenario, boolean skipSteps) {
         Long startTime = bus.getTime();
         bus.send(new TestStepStarted(startTime, this));
@@ -69,10 +91,12 @@ public Result run(EventBus bus, String language, Scenario scenario, boolean skip
         return result;
     }
 
+    @Deprecated
     protected Result.Type nonExceptionStatus(boolean skipSteps) {
         return skipSteps ? Result.Type.SKIPPED : Result.Type.PASSED;
     }
 
+    @Deprecated
     protected Result.Type executeStep(String language, Scenario scenario, boolean skipSteps) throws Throwable {
         if (!skipSteps) {
             definitionMatch.runStep(language, scenario);