diff options
Diffstat (limited to 'src/main/java/io/trygvis/esper/testing/jenkins/JenkinsClient.java')
-rwxr-xr-x | src/main/java/io/trygvis/esper/testing/jenkins/JenkinsClient.java | 30 |
1 files changed, 27 insertions, 3 deletions
diff --git a/src/main/java/io/trygvis/esper/testing/jenkins/JenkinsClient.java b/src/main/java/io/trygvis/esper/testing/jenkins/JenkinsClient.java index bd7c82b..0b2883f 100755 --- a/src/main/java/io/trygvis/esper/testing/jenkins/JenkinsClient.java +++ b/src/main/java/io/trygvis/esper/testing/jenkins/JenkinsClient.java @@ -22,6 +22,7 @@ import java.util.List; import static fj.data.Option.*; import static io.trygvis.esper.testing.Util.*; import static io.trygvis.esper.testing.util.HttpClient.inputStreamOnly; +import static java.lang.System.currentTimeMillis; import static org.apache.commons.lang.StringUtils.*; public class JenkinsClient { @@ -36,7 +37,10 @@ public class JenkinsClient { jenkinsEntryXmlClient = new HttpClient<>(http, inputStreamOnly(new F<InputStream, Option<List<JenkinsEntryXml>>>() { public Option<List<JenkinsEntryXml>> f(InputStream inputStream) { + long start = currentTimeMillis(); Feed feed = (Feed) parser.parse(inputStream).getRoot(); + long end = currentTimeMillis(); + logger.info("Parsed document in " + (end - start) + "ms."); List<JenkinsEntryXml> list = new ArrayList<>(); @@ -47,6 +51,10 @@ public class JenkinsClient { } } + long end2 = currentTimeMillis(); + + logger.info("Converted document to JenkinsEntryXml in " + (end2 - end) + "ms."); + return some(list); } })); @@ -133,8 +141,8 @@ public class JenkinsClient { switch (name) { case "matrixBuild": - case "matrixRun": case "mavenModuleSetBuild": + case "mavenBuild": case "freeStyleBuild": return JenkinsBuildXml.parse(root); default: @@ -166,8 +174,24 @@ public class JenkinsClient { Option<Integer> duration = childText(root, "duration").bind(Util.parseInt); Option<Long> timestamp = childText(root, "timestamp").bind(Util.parseLong); - if(url.isNone() || number.isNone() || result.isNone() || duration.isNone() || timestamp.isNone()) { - logger.warn("Missing required fields."); + if(url.isNone()) { + logger.warn("Missing required field: <url>"); + return none(); + } + if(number.isNone()) { + logger.warn("Missing required field: <number>"); + return none(); + } + if(result.isNone()) { + logger.warn("Missing required field: <result>"); + return none(); + } + if(duration.isNone()) { + logger.warn("Missing required field: <duration>"); + return none(); + } + if(timestamp.isNone()) { + logger.warn("Missing required field: <timestamp>"); return none(); } |