o Making sure every resource that supports paging has a default ordering.

o Fixing mis-mapped JenkinsJob.
o Better navigation in the Jenkins app, <a> links are now proper links.

8 files changed, 45 insertions, 26 deletions

diff --git a/src/main/java/io/trygvis/esper/testing/Util.java b/src/main/java/io/trygvis/esper/testing/Util.java
index 849a6f9..1bba36c 100755
--- a/src/main/java/io/trygvis/esper/testing/Util.java
+++ b/src/main/java/io/trygvis/esper/testing/Util.java
@@ -77,11 +77,19 @@ public class Util {
     // SQL
     // -----------------------------------------------------------------------
 
+    public static List<String> ifEmpty(List<String> inputs, String defaultValue) {
+        if (inputs.isEmpty()) {
+            return inputs;
+        }
+
+        return Collections.singletonList(defaultValue);
+    }
+
     public static String orderBy(String[] inputs, String... allowed) {
         return orderBy(Arrays.asList(inputs), allowed);
     }
 
-    public static String orderBy(Iterable<String> inputs, String... allowed) {
+    public static String orderBy(List<String> inputs, String... allowed) {
         StringBuilder buffer = new StringBuilder();
 
         for (String input : inputs) {
diff --git a/src/main/java/io/trygvis/esper/testing/core/db/BuildDao.java b/src/main/java/io/trygvis/esper/testing/core/db/BuildDao.java
index c6a46c6..7c9ccb7 100755
--- a/src/main/java/io/trygvis/esper/testing/core/db/BuildDao.java
+++ b/src/main/java/io/trygvis/esper/testing/core/db/BuildDao.java
@@ -92,7 +92,7 @@ public class BuildDao {
 
     public List<BuildDto> selectBuildsByPerson(Uuid person, PageRequest page) throws SQLException {
         String sql = "SELECT " + BUILD + " FROM build b, build_participant bp WHERE bp.person=? AND b.uuid = bp.build";
-        sql += orderBy(page.orderBy, "created_date", "timestamp");
+        sql += orderBy(ifEmpty(page.orderBy, "created_date-"), "created_date", "timestamp");
         sql += " LIMIT ? OFFSET ?";
 
         try (PreparedStatement s = c.prepareStatement(sql)) {
@@ -106,7 +106,7 @@ public class BuildDao {
 
     public List<BuildDto> selectBuilds(PageRequest page) throws SQLException {
         String sql = "SELECT " + BUILD + " FROM build";
-        sql += orderBy(page.orderBy, "created_date", "timestamp");
+        sql += orderBy(ifEmpty(page.orderBy, "created_date-"), "created_date", "timestamp");
         sql += " LIMIT ? OFFSET ?";
 
         try (PreparedStatement s = c.prepareStatement(sql)) {
diff --git a/src/main/java/io/trygvis/esper/testing/core/db/PersonDao.java b/src/main/java/io/trygvis/esper/testing/core/db/PersonDao.java
index 290d6d5..2cfb16d 100755
--- a/src/main/java/io/trygvis/esper/testing/core/db/PersonDao.java
+++ b/src/main/java/io/trygvis/esper/testing/core/db/PersonDao.java
@@ -97,14 +97,14 @@ public class PersonDao {
         }
     }
 
-    public List<PersonDto> selectPersons(PageRequest pageRequest, Option<String> query) throws SQLException {
+    public List<PersonDto> selectPersons(PageRequest page, Option<String> query) throws SQLException {
         String sql = "SELECT " + PERSON + " FROM person";
 
         if (query.isSome()) {
             sql += " WHERE lower(name) LIKE '%' || ? || '%'";
         }
 
-        sql += orderBy(pageRequest.orderBy, "name", "created_date");
+        sql += orderBy(ifEmpty(page.orderBy, "created_date-"), "name", "created_date");
 
         sql += " LIMIT ? OFFSET ?";
 
@@ -115,8 +115,8 @@ public class PersonDao {
                 s.setString(i++, query.some());
             }
 
-            s.setInt(i++, pageRequest.count.orSome(10));
-            s.setInt(i, pageRequest.startIndex.orSome(0));
+            s.setInt(i++, page.count.orSome(10));
+            s.setInt(i, page.startIndex.orSome(0));
             return toList(s, person);
         }
     }
@@ -216,7 +216,7 @@ public class PersonDao {
             sql += " AND level=?";
         }
 
-        sql += orderBy(page.orderBy, "name", "created_date");
+        sql += orderBy(ifEmpty(page.orderBy, "created_date-"), "name", "created_date");
 
         sql += " LIMIT ? OFFSET ?";
 
diff --git a/src/main/java/io/trygvis/esper/testing/jenkins/JenkinsDao.java b/src/main/java/io/trygvis/esper/testing/jenkins/JenkinsDao.java
index 781df13..d21118d 100755
--- a/src/main/java/io/trygvis/esper/testing/jenkins/JenkinsDao.java
+++ b/src/main/java/io/trygvis/esper/testing/jenkins/JenkinsDao.java
@@ -22,8 +22,6 @@ public class JenkinsDao {
 
     public static final String JENKINS_SERVER = "uuid, created_date, name, url, enabled";
 
-    public static final String JENKINS_JOB = "uuid, created_date, server, file, url, job_type, display_name";
-
     public static final String JENKINS_USER = "uuid, created_date, server, absolute_url";
 
     public JenkinsDao(Connection c) {
@@ -42,6 +40,8 @@ public class JenkinsDao {
         }
     };
 
+    public static final String JENKINS_JOB = "uuid, created_date, server, file, url, job_type, display_name";
+
     public static final SqlF<ResultSet, JenkinsJobDto> jenkinsJob = new SqlF<ResultSet, JenkinsJobDto>() {
         public JenkinsJobDto apply(ResultSet rs) throws SQLException {
             int i = 1;
@@ -51,6 +51,7 @@ public class JenkinsDao {
                     UUID.fromString(rs.getString(i++)),
                     UUID.fromString(rs.getString(i++)),
                     URI.create(rs.getString(i++)),
+                    rs.getString(i++),
                     fromNull(rs.getString(i)));
         }
     };
@@ -172,7 +173,11 @@ public class JenkinsDao {
     }
 
     public List<JenkinsBuildDto> selectBuildByJob(UUID job, PageRequest page) throws SQLException {
-        try (PreparedStatement s = c.prepareStatement("SELECT " + JENKINS_BUILD + " FROM jenkins_build WHERE job=? ORDER BY created_date DESC LIMIT ? OFFSET ?")) {
+        String sql = "SELECT " + JENKINS_BUILD + " FROM jenkins_build WHERE job=?";
+        sql += orderBy(ifEmpty(page.orderBy, "created_date-"), "created_date", "timestamp");
+        sql += " LIMIT ? OFFSET ?";
+
+        try (PreparedStatement s = c.prepareStatement(sql)) {
             int i = 1;
             s.setString(i++, job.toString());
             s.setInt(i++, page.count.orSome(10));
diff --git a/src/main/java/io/trygvis/esper/testing/jenkins/JenkinsJobDto.java b/src/main/java/io/trygvis/esper/testing/jenkins/JenkinsJobDto.java
index e540a2a..2092d89 100755
--- a/src/main/java/io/trygvis/esper/testing/jenkins/JenkinsJobDto.java
+++ b/src/main/java/io/trygvis/esper/testing/jenkins/JenkinsJobDto.java
@@ -11,13 +11,15 @@ public class JenkinsJobDto extends AbstractEntity {
     public final UUID server;
     public final UUID file;
     public final URI url;
+    public final String jobType;
     public final Option<String> displayName;
 
-    public JenkinsJobDto(UUID uuid, DateTime createdDate, UUID server, UUID file, URI url, Option<String> displayName) {
+    public JenkinsJobDto(UUID uuid, DateTime createdDate, UUID server, UUID file, URI url, String jobType, Option<String> displayName) {
         super(uuid, createdDate);
         this.server = server;
         this.file = file;
         this.url = url;
+        this.jobType = jobType;
         this.displayName = displayName;
     }
 }
diff --git a/src/main/java/io/trygvis/esper/testing/util/sql/PageRequest.java b/src/main/java/io/trygvis/esper/testing/util/sql/PageRequest.java
index b7c6750..7773e50 100644..100755
--- a/src/main/java/io/trygvis/esper/testing/util/sql/PageRequest.java
+++ b/src/main/java/io/trygvis/esper/testing/util/sql/PageRequest.java
@@ -1,12 +1,14 @@
 package io.trygvis.esper.testing.util.sql;
 
-import fj.data.*;
+import fj.data.Option;
+
+import java.util.*;
 
 public class PageRequest {
     public final Option<Integer> startIndex;
     public final Option<Integer> count;
     public final List<String> orderBy;
-    public static final PageRequest FIRST_PAGE = new PageRequest(Option.<Integer>none(), Option.<Integer>none(), List.<String>nil());
+    public static final PageRequest FIRST_PAGE = new PageRequest(Option.<Integer>none(), Option.<Integer>none(), Collections.<String>emptyList());
 
     public PageRequest(Option<Integer> startIndex, Option<Integer> count, List<String> orderBy) {
         this.startIndex = startIndex;
diff --git a/src/main/java/io/trygvis/esper/testing/web/JerseyApplication.java b/src/main/java/io/trygvis/esper/testing/web/JerseyApplication.java
index 38cfd9d..b85dd7a 100755
--- a/src/main/java/io/trygvis/esper/testing/web/JerseyApplication.java
+++ b/src/main/java/io/trygvis/esper/testing/web/JerseyApplication.java
@@ -66,10 +66,10 @@ public class JerseyApplication extends Application {
 
                         List<String> list = queryParameters.get("orderBy");
 
-                        fj.data.List<String> orderBy = nil();
+                        List<String> orderBy = Collections.emptyList();
 
                         if (list != null) {
-                            orderBy = iterableList(list);
+                            orderBy = list;
                         }
 
                         return new PageRequest(
diff --git a/src/main/java/io/trygvis/esper/testing/web/resource/JenkinsResource.java b/src/main/java/io/trygvis/esper/testing/web/resource/JenkinsResource.java
index 0c0c6a9..a6d8e8e 100755
--- a/src/main/java/io/trygvis/esper/testing/web/resource/JenkinsResource.java
+++ b/src/main/java/io/trygvis/esper/testing/web/resource/JenkinsResource.java
@@ -69,9 +69,9 @@ public class JenkinsResource extends AbstractResource {
     @GET
     @Path("/job/{uuid}")
     @Produces(MediaType.APPLICATION_JSON)
-    public JenkinsJobJsonDetail getJob(@MagicParam final UUID uuid) throws Exception {
-        return sql(new JenkinsDaosCallback<SqlOption<JenkinsJobJsonDetail>>() {
-            protected SqlOption<JenkinsJobJsonDetail> run() throws SQLException {
+    public JenkinsJobDetailJson getJob(@MagicParam final UUID uuid) throws Exception {
+        return sql(new JenkinsDaosCallback<SqlOption<JenkinsJobDetailJson>>() {
+            protected SqlOption<JenkinsJobDetailJson> run() throws SQLException {
                 return daos.jenkinsDao.selectJob(uuid).map(getJenkinsJobJsonDetail);
             }
         });
@@ -149,13 +149,13 @@ public class JenkinsResource extends AbstractResource {
 
         protected SqlF<JenkinsJobDto, JenkinsJobJson> getJenkinsJobJson = new SqlF<JenkinsJobDto, JenkinsJobJson>() {
             public JenkinsJobJson apply(JenkinsJobDto job) throws SQLException {
-                return new JenkinsJobJson(job.uuid, job.createdDate, job.server, job.displayName.toNull());
+                return new JenkinsJobJson(job.uuid, job.createdDate, job.server, job.url, job.displayName.toNull());
             }
         };
 
-        protected SqlF<JenkinsJobDto,JenkinsJobJsonDetail> getJenkinsJobJsonDetail = new SqlF<JenkinsJobDto, JenkinsJobJsonDetail>() {
-            public JenkinsJobJsonDetail apply(JenkinsJobDto dto) throws SQLException {
-                return new JenkinsJobJsonDetail(
+        protected SqlF<JenkinsJobDto,JenkinsJobDetailJson> getJenkinsJobJsonDetail = new SqlF<JenkinsJobDto, JenkinsJobDetailJson>() {
+            public JenkinsJobDetailJson apply(JenkinsJobDto dto) throws SQLException {
+                return new JenkinsJobDetailJson(
                         getJenkinsJobJson.apply(dto),
                         daos.jenkinsDao.selectBuildCountByJob(dto.uuid));
             }
@@ -223,21 +223,23 @@ class JenkinsJobJson {
     public final UUID uuid;
     public final DateTime createdDate;
     public final UUID server;
+    public final URI url;
     public final String displayName;
 
-    JenkinsJobJson(UUID uuid, DateTime createdDate, UUID server, String displayName) {
+    JenkinsJobJson(UUID uuid, DateTime createdDate, UUID server, URI url, String displayName) {
         this.uuid = uuid;
         this.createdDate = createdDate;
         this.server = server;
+        this.url = url;
         this.displayName = displayName;
     }
 }
 
-class JenkinsJobJsonDetail {
+class JenkinsJobDetailJson {
     public final JenkinsJobJson job;
     public final Integer buildCount;
 
-    JenkinsJobJsonDetail(JenkinsJobJson job, Integer buildCount) {
+    JenkinsJobDetailJson(JenkinsJobJson job, Integer buildCount) {
         this.job = job;
         this.buildCount = buildCount;
     }