Report coverage as separate build thing

Author: Kelly Johnson

src/main/java/org/jenkinsci/plugins/ghprb/extensions/status/GhprbSimpleStatus.java

@@ -224,17 +224,24 @@ public void onBuildComplete(Run<?, ?> build,
                 sb.append(buildManager.getOneLineTestResults());
             }
             if (getAddCoverageResults()) {
-                if (getAddTestResults()) {
-                    sb.append(" ");
-                }
                 listener.getLogger().println("Adding coverage results to commit status...");
+
+                String coverageResultString;
+                GHCommitState coverageState;
                 Map<String, String> envVars = Ghprb.getEnvVars(build, listener);
                 if (!envVars.containsKey("newCodeCoveragePercentage")) {
-                    sb.append("No coverage results found.");
+                    coverageResultString = "No coverage results found.";
+                    coverageState = GHCommitState.SUCCESS;
                 } else {
-                    sb.append(Integer.parseInt(envVars.get("newCodeCoveragePercentage")));
-                    sb.append("% line coverage on new code.");
+                    String newCodeCoveragePercentage = envVars.get("newCodeCoveragePercentage");
+                    coverageResultString = newCodeCoveragePercentage + "% line coverage on new code.";
+                    int coveragePercent = Integer.parseInt(newCodeCoveragePercentage);
+                    coverageState = coveragePercent > 1 ? GHCommitState.SUCCESS : GHCommitState.ERROR;
                 }
+
+                String context = Util.fixEmpty(commitStatusContext);
+                context = Ghprb.replaceMacros(build, listener, context) + " Coverage";
+                createCommitStatus(build, listener, coverageResultString, repo, coverageState, context);
             }
         }
 
@@ -245,7 +252,8 @@ private void createCommitStatus(Run<?, ?> build,
                                     TaskListener listener,
                                     String message,
                                     GHRepository repo,
-                                    GHCommitState state) throws GhprbCommitStatusException {
+                                    GHCommitState state,
+                                    String context) throws GhprbCommitStatusException {
 
         Map<String, String> envVars = Ghprb.getEnvVars(build, listener);
 
@@ -266,9 +274,6 @@ private void createCommitStatus(Run<?, ?> build,
             url = Ghprb.replaceMacros(build, listener, statusUrl);
         }
 
-        String context = Util.fixEmpty(commitStatusContext);
-        context = Ghprb.replaceMacros(build, listener, context);
-
         listener.getLogger().println(String.format("Setting status of %s to %s with url %s and message: '%s'",
                 sha1,
                 state,
@@ -286,6 +291,16 @@ private void createCommitStatus(Run<?, ?> build,
         }
     }
 
+    private void createCommitStatus(Run<?, ?> build,
+                                    TaskListener listener,
+                                    String message,
+                                    GHRepository repo,
+                                    GHCommitState state) throws GhprbCommitStatusException {
+        String context = Util.fixEmpty(commitStatusContext);
+        context = Ghprb.replaceMacros(build, listener, context);
+        createCommitStatus(build, listener, message, repo, state, context);
+    }
+
     public void createCommitStatus(Job<?, ?> project,
                                    int prId,
                                    String commitSha,