Merge pull request #55 from jsm/fix_commenting

Don't set default for job-specific messages
jenkinsci · Oct 9, 2014 · f997416 · f997416
2 parents 49c2f62 + 3875a5f
commit f997416
Showing 1 changed file with 4 additions and 13 deletions.
diff --git a/src/main/java/org/jenkinsci/plugins/ghprb/GhprbTrigger.java b/src/main/java/org/jenkinsci/plugins/ghprb/GhprbTrigger.java
@@ -527,23 +527,14 @@ public String getUnstableAs() {
         }
 
         public String getMsgSuccess(AbstractBuild<?, ?> build) {
-        	String msg = msgSuccess;
-            if (msg == null) {
-                msg = "Test PASSed.";
-            }
-
-        	msg = Ghprb.replaceMacros(build, msg);
+            String msg = msgSuccess;
+            msg = Ghprb.replaceMacros(build, msg);
             return msg;
         }
 
         public String getMsgFailure(AbstractBuild<?, ?> build) {
-        	String msg = msgFailure;
-            if (msg == null) {
-                msg = "Test FAILed.";
-            }
-
-        	msg = Ghprb.replaceMacros(build, msg);
-
+            String msg = msgFailure;
+            msg = Ghprb.replaceMacros(build, msg);
             return msg;
         }