Merge pull request #275 from derinbay/failed-tests-report

Add display failed tests option to slack notifications on job configuration
jenkinsci · Mar 3, 2017 · 364bca9 · 364bca9
2 parents ee51c74 + 2cac73c
commit 364bca9
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/src/main/java/jenkins/plugins/slack/ActiveNotifier.java b/src/main/java/jenkins/plugins/slack/ActiveNotifier.java
@@ -379,9 +379,10 @@ public MessageBuilder appendFailedTests() {
                     .getAction(AbstractTestResultAction.class);
             if (action != null) {
                 int failed = action.getFailCount();
-                message.append("\n").append(failed).append(" failed Tests:\n");
+                message.append("\n").append(failed).append(" Failed Tests:\n");
                 for(TestResult result : action.getFailedTests()) {
-                    message.append("\t").append(result.getName()).append("\n");
+                    message.append("\t").append(result.getName()).append(" after ")
+                            .append(result.getDurationString()).append("\n");
                 }
             }
             return this;

diff --git a/src/main/resources/jenkins/plugins/slack/SlackNotifier/config.jelly b/src/main/resources/jenkins/plugins/slack/SlackNotifier/config.jelly
@@ -36,7 +36,7 @@
         <f:entry title="Include Test Summary">
             <f:checkbox name="includeTestSummary" value="true" checked="${instance.includeTestSummary()}"/>
         </f:entry>
-        <f:entry title="Include failed Tests">
+        <f:entry title="Include Failed Tests">
             <f:checkbox name="includeFailedTests" value="true" checked="${instance.includeFailedTests()}"/>
         </f:entry>