analyse the full git commit message where present.

netmelody · Jul 14, 2017 · 9320047 · 9320047
1 parent cea811e
commit 9320047
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 18 deletions.
diff --git a/src/main/java/org/netmelody/cieye/spies/jenkins/JobAnalyser.java b/src/main/java/org/netmelody/cieye/spies/jenkins/JobAnalyser.java
@@ -1,20 +1,7 @@
 package org.netmelody.cieye.spies.jenkins;
 
-import static com.google.common.collect.Collections2.filter;
-import static com.google.common.collect.Collections2.transform;
-import static com.google.common.collect.Lists.newArrayList;
-import static java.util.Collections.unmodifiableSet;
-import static org.netmelody.cieye.core.domain.Percentage.percentageOf;
-import static org.netmelody.cieye.core.domain.RunningBuild.buildAt;
-
-import java.util.Collection;
-import java.util.Date;
-import java.util.HashMap;
-import java.util.HashSet;
-import java.util.List;
-import java.util.Map;
-import java.util.Set;
-
+import com.google.common.base.Function;
+import com.google.common.base.Predicate;
 import org.netmelody.cieye.core.domain.RunningBuild;
 import org.netmelody.cieye.core.domain.Sponsor;
 import org.netmelody.cieye.core.domain.Status;
@@ -27,8 +14,20 @@
 import org.netmelody.cieye.spies.jenkins.jsondomain.JobDetail;
 import org.netmelody.cieye.spies.jenkins.jsondomain.User;
 
-import com.google.common.base.Function;
-import com.google.common.base.Predicate;
+import java.util.Collection;
+import java.util.Date;
+import java.util.HashMap;
+import java.util.HashSet;
+import java.util.List;
+import java.util.Map;
+import java.util.Set;
+
+import static com.google.common.collect.Collections2.filter;
+import static com.google.common.collect.Collections2.transform;
+import static com.google.common.collect.Lists.newArrayList;
+import static java.util.Collections.unmodifiableSet;
+import static org.netmelody.cieye.core.domain.Percentage.percentageOf;
+import static org.netmelody.cieye.core.domain.RunningBuild.buildAt;
 
 public final class JobAnalyser {
 
@@ -63,6 +62,10 @@ public TargetDetail analyse(Job jobDigest) {
         return new TargetDetail(job.url, job.url, job.name, statusOf(job), startTimeOf(job), buildsFor(job), sponsorsOf(job));
     }
 
+    String lastBadBuildUrl() {
+        return communicator.lastBadBuildFor(jobEndpoint);
+    }
+
     private long startTimeOf(JobDetail job) {
         return buildStartTimeFetcher.lastStartTimeOf(job);
     }

diff --git a/src/test/java/org/netmelody/cieye/spies/jenkins/test/JenkinsCommunicatorTest.java b/src/test/java/org/netmelody/cieye/spies/jenkins/test/JenkinsCommunicatorTest.java
@@ -53,7 +53,7 @@ public void setup() {
 
         assertThat(changes, is(Matchers.<ChangeSetItem>iterableWithSize(1)));
         assertThat(changes.get(0).msg, is("add some example json from jenkins for testing"));
-        assertThat(changes.get(0).comment, is("add some example json from jenkins for testing\n"));
+        assertThat(changes.get(0).comment, is("add some example json from jenkins for testing\nand a second line of magic\n"));
     }
 
     private String contentFrom(String resourceName) {