[JENKINS-26148] Using default implementation of StepExecution.stop.

jenkinsci · Sep 4, 2017 · a7248f0 · a7248f0
1 parent 3dfefde
commit a7248f0
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/pom.xml b/pom.xml
@@ -4,7 +4,7 @@
     <parent>
         <groupId>org.jenkins-ci.plugins</groupId>
         <artifactId>plugin</artifactId>
-        <version>2.15</version>
+        <version>2.33</version>
         <relativePath />
     </parent>
     <groupId>org.jenkins-ci.plugins</groupId>
@@ -40,12 +40,13 @@
     <properties>
         <jenkins.version>1.642.3</jenkins.version>
         <scm-api.version>2.0.7</scm-api.version>
+        <workflow-step-api-plugin.version>2.13-20170904.014626-1</workflow-step-api-plugin.version> <!-- TODO https://github.com/jenkinsci/workflow-step-api-plugin/pull/30 -->
     </properties>
     <dependencies>
         <dependency>
             <groupId>org.jenkins-ci.plugins.workflow</groupId>
             <artifactId>workflow-step-api</artifactId>
-            <version>2.3</version>
+            <version>${workflow-step-api-plugin.version}</version>
         </dependency>
         <dependency>
             <groupId>org.jenkins-ci.plugins.workflow</groupId>
@@ -66,7 +67,7 @@
         <dependency>
             <groupId>org.jenkins-ci.plugins.workflow</groupId>
             <artifactId>workflow-step-api</artifactId>
-            <version>2.3</version>
+            <version>${workflow-step-api-plugin.version}</version>
             <classifier>tests</classifier>
             <scope>test</scope>
         </dependency>

diff --git a/...in/java/org/jenkinsci/plugins/workflow/support/steps/build/BuildTriggerStepExecution.java b/...in/java/org/jenkinsci/plugins/workflow/support/steps/build/BuildTriggerStepExecution.java
@@ -170,7 +170,7 @@ private List<ParameterValue> completeDefaultParameters(List<ParameterValue> para
 
     @SuppressFBWarnings(value="RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE", justification="TODO 1.653+ switch to Jenkins.getInstanceOrNull")
     @Override
-    public void stop(Throwable cause) {
+    public void stop(Throwable cause) throws Exception {
         StepContext context = getContext();
         Jenkins jenkins = Jenkins.getInstance();
         if (jenkins == null) {
@@ -208,7 +208,7 @@ public void stop(Throwable cause) {
         }
 
         if (!interrupted) {
-            context.onFailure(cause);
+            super.stop(cause);
         }
     }
     private static boolean maybeInterrupt(Executor e, Throwable cause, StepContext context) {