Support iterating through downstream builds (if present) to compose the message that will be posted to Github

pom.xml

@@ -88,6 +88,12 @@
             <scope>test</scope>
         </dependency>
 
+        <dependency>
+        	<groupId>com.cloudbees.plugins</groupId>
+        	<artifactId>build-flow-plugin</artifactId>
+        	<version>0.12</version>
+        	<optional>true</optional>
+        </dependency>
     </dependencies>
 
     <build>

src/main/java/org/jenkinsci/plugins/ghprb/GhprbBuilds.java

@@ -7,6 +7,11 @@
 import hudson.plugins.git.util.BuildData;
 
 import org.apache.commons.io.FileUtils;
+
+import org.jenkinsci.plugins.ghprb.manager.GhprbBuildManager;
+import org.jenkinsci.plugins.ghprb.manager.configuration.JobConfiguration;
+import org.jenkinsci.plugins.ghprb.manager.factory.GhprbBuildManagerFactoryUtil;
+
 import org.kohsuke.github.GHCommitState;
 import org.kohsuke.github.GHIssueState;
 import org.kohsuke.github.GHPullRequest;
@@ -138,8 +143,9 @@ public void onCompleted(AbstractBuild<?,?> build, PrintStream logger) {
             } else {
                 msg.append(GhprbTrigger.getDscp().getMsgFailure(build));
             }
+
             msg.append("\nRefer to this link for build results (access rights to CI server needed): \n");
-            msg.append(publishedURL).append(build.getUrl());
+            msg.append(generateCustomizedMessage(build));
 
             int numLines = GhprbTrigger.getDscp().getlogExcerptLines();
             if (state != GHCommitState.SUCCESS && numLines > 0) {
@@ -177,4 +183,25 @@ public void onCompleted(AbstractBuild<?,?> build, PrintStream logger) {
             }
         }
     }
+
+    private String generateCustomizedMessage(AbstractBuild build) {
+        JobConfiguration jobConfiguration =
+            JobConfiguration.builder()
+                .printStackTrace(trigger.isDisplayBuildErrorsOnDownstreamBuilds())
+                .build();
+
+        GhprbBuildManager buildManager =
+            GhprbBuildManagerFactoryUtil.getBuildManager(build, jobConfiguration);
+
+        StringBuilder sb = new StringBuilder();
+
+        sb.append(buildManager.calculateBuildUrl());
+
+        if (build.getResult() != Result.SUCCESS) {
+            sb.append(
+                buildManager.getTestResults());
+        }
+
+        return sb.toString();
+    }
 }

src/main/java/org/jenkinsci/plugins/ghprb/GhprbTrigger.java

@@ -54,6 +54,7 @@ public class GhprbTrigger extends Trigger<AbstractProject<?, ?>> {
 	private final String commentFilePath;
     private String whitelist;
     private Boolean autoCloseFailedPullRequests;
+    private Boolean displayBuildErrorsOnDownstreamBuilds;
     private List<GhprbBranch> whiteListTargetBranches;
     private transient Ghprb helper;
     private String project;
@@ -68,6 +69,7 @@ public GhprbTrigger(String adminlist,
                         Boolean useGitHubHooks,
                         Boolean permitAll,
                         Boolean autoCloseFailedPullRequests,
+                        Boolean displayBuildErrorsOnDownstreamBuilds,
                         String commentFilePath,
                         List<GhprbBranch> whiteListTargetBranches,
                         Boolean allowMembersOfWhitelistedOrgsAsAdmin) throws ANTLRException {
@@ -81,6 +83,7 @@ public GhprbTrigger(String adminlist,
         this.useGitHubHooks = useGitHubHooks;
         this.permitAll = permitAll;
         this.autoCloseFailedPullRequests = autoCloseFailedPullRequests;
+        this.displayBuildErrorsOnDownstreamBuilds = displayBuildErrorsOnDownstreamBuilds;
         this.whiteListTargetBranches = whiteListTargetBranches;
         this.commentFilePath = commentFilePath;
         this.allowMembersOfWhitelistedOrgsAsAdmin = allowMembersOfWhitelistedOrgsAsAdmin;
@@ -287,9 +290,16 @@ public Boolean isAutoCloseFailedPullRequests() {
         if (autoCloseFailedPullRequests == null) {
             Boolean autoClose = getDescriptor().getAutoCloseFailedPullRequests();
             return (autoClose != null && autoClose);
-        } else {
-            return autoCloseFailedPullRequests;
         }
+        return autoCloseFailedPullRequests;
+    }
+
+    public Boolean isDisplayBuildErrorsOnDownstreamBuilds() {
+        if (displayBuildErrorsOnDownstreamBuilds == null) {
+            Boolean displayErrors = getDescriptor().getDisplayBuildErrorsOnDownstreamBuilds();
+            return (displayErrors != null && displayErrors);
+        }
+        return displayBuildErrorsOnDownstreamBuilds;
     }
 
     public List<GhprbBranch> getWhiteListTargetBranches() {
@@ -351,6 +361,7 @@ public static final class DescriptorImpl extends TriggerDescriptor {
         private String msgFailure = "Test FAILed.";
         private List<GhprbBranch> whiteListTargetBranches;
         private Boolean autoCloseFailedPullRequests = false;
+        private Boolean displayBuildErrorsOnDownstreamBuilds = false;
 
 
 
@@ -399,6 +410,7 @@ public boolean configure(StaplerRequest req, JSONObject formData) throws FormExc
             logExcerptLines = formData.getInt("logExcerptLines");
             unstableAs = formData.getString("unstableAs");
             autoCloseFailedPullRequests = formData.getBoolean("autoCloseFailedPullRequests");
+            displayBuildErrorsOnDownstreamBuilds = formData.getBoolean("displayBuildErrorsOnDownstreamBuilds");
             msgSuccess = formData.getString("msgSuccess");
             msgFailure = formData.getString("msgFailure");
 
@@ -481,6 +493,10 @@ public Boolean getAutoCloseFailedPullRequests() {
             return autoCloseFailedPullRequests;
         }
 
+        public Boolean getDisplayBuildErrorsOnDownstreamBuilds() {
+            return displayBuildErrorsOnDownstreamBuilds;
+        }
+
         public String getServerAPIUrl() {
             return serverAPIUrl;
         }

src/main/java/org/jenkinsci/plugins/ghprb/manager/GhprbBuildManager.java

@@ -0,0 +1,31 @@
+package org.jenkinsci.plugins.ghprb.manager;
+
+import java.util.Iterator;
+
+/**
+ * @author mdelapenya (Manuel de la Peña)
+ */
+public interface GhprbBuildManager {
+
+	/**
+	 * Calculate the build URL of a build
+	 * 
+	 * @return the build URL
+	 */
+	String calculateBuildUrl();
+
+	/**
+	 * Returns downstream builds as an iterator
+	 * 
+	 * @return the iterator
+	 */
+	Iterator downstreamProjects();
+
+	/**
+	 * Print tests result of a build
+	 *
+	 * @return the tests result
+	 */
+	String getTestResults();
+
+}

src/main/java/org/jenkinsci/plugins/ghprb/manager/configuration/JobConfiguration.java

@@ -0,0 +1,46 @@
+package org.jenkinsci.plugins.ghprb.manager.configuration;
+
+/**
+ * @author Miguel Pastor
+ */
+public class JobConfiguration {
+
+	public static PrintStackTrace builder() {
+		return new JobConfigurationBuilder();
+	}
+
+	private boolean printStacktrace;
+
+	private JobConfiguration() {
+	}
+
+	public boolean printStackTrace() {
+		return this.printStacktrace;
+	}
+
+	public static interface PrintStackTrace {
+		Build printStackTrace(boolean print);
+	}
+
+	public static interface Build {
+		JobConfiguration build();
+	}
+
+
+	public static final class JobConfigurationBuilder implements Build, PrintStackTrace {
+
+		private JobConfiguration jobConfiguration = new JobConfiguration();
+
+		public JobConfiguration build() {
+			return jobConfiguration;
+		}
+
+		public Build printStackTrace(boolean print) {
+			jobConfiguration.printStacktrace = print;
+			return this;
+		}
+	}
+
+}
+
+

src/main/java/org/jenkinsci/plugins/ghprb/manager/factory/GhprbBuildManagerFactoryUtil.java

@@ -0,0 +1,48 @@
+package org.jenkinsci.plugins.ghprb.manager.factory;
+
+
+import com.cloudbees.plugins.flow.FlowRun;
+
+import hudson.model.AbstractBuild;
+import org.jenkinsci.plugins.ghprb.manager.configuration.JobConfiguration;
+import org.jenkinsci.plugins.ghprb.manager.GhprbBuildManager;
+import org.jenkinsci.plugins.ghprb.manager.impl.GhprbDefaultBuildManager;
+import org.jenkinsci.plugins.ghprb.manager.impl.downstreambuilds.BuildFlowBuildManager;
+
+/**
+ * @author mdelapenya (Manuel de la Peña)
+ */
+public class GhprbBuildManagerFactoryUtil {
+
+	/**
+	 * Gets an instance of a library that is able to calculate build urls
+	 * depending of build type.
+	 * 
+	 * If the class representing the build type is not present on the classloader
+	 * then default implementation is returned.
+	 * 
+	 * @param build
+	 * @return
+	 */
+	public static GhprbBuildManager getBuildManager(AbstractBuild build) {
+		JobConfiguration jobConfiguration =
+			JobConfiguration.builder()
+				.printStackTrace(false)
+				.build();
+
+		return getBuildManager(build, jobConfiguration);
+	}
+
+	public static GhprbBuildManager getBuildManager(AbstractBuild build, JobConfiguration jobConfiguration) {
+		try {
+			if (build instanceof FlowRun) {
+				return new BuildFlowBuildManager(build, jobConfiguration);
+			}
+		}
+		catch (NoClassDefFoundError ncdfe) {
+		}
+
+		return new GhprbDefaultBuildManager(build);
+	}
+
+}