From 3875a5f0a011c176b8d34ea5e46b52809fc5545e Mon Sep 17 00:00:00 2001 From: Jon San Miguel Date: Wed, 8 Oct 2014 17:35:27 -0700 Subject: [PATCH] Don't set default for job-specific messages --- .../jenkinsci/plugins/ghprb/GhprbTrigger.java | 17 ++++------------- 1 file changed, 4 insertions(+), 13 deletions(-) diff --git a/src/main/java/org/jenkinsci/plugins/ghprb/GhprbTrigger.java b/src/main/java/org/jenkinsci/plugins/ghprb/GhprbTrigger.java index 086b65221..b94427c0b 100644 --- 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; }