Merge pull request #308 from PierreBtz/pbeitz/dev

#307 Add a sorting order for the comment retrieval

Author: aheritier

src/main/java/org/zendesk/client/v2/SortOrder.java

@@ -0,0 +1,15 @@
+package org.zendesk.client.v2;
+
+public enum SortOrder {
+    ASCENDING("asc"), DESCENDING("desc");
+
+    private String queryParameter;
+
+    SortOrder(String queryParameter) {
+        this.queryParameter = queryParameter;
+    }
+
+    public String getQueryParameter() {
+        return queryParameter;
+    }
+}

src/main/java/org/zendesk/client/v2/Zendesk.java

@@ -1048,8 +1048,15 @@ public Iterable<Comment> getRequestComments(long id) {
     }
 
     public Iterable<Comment> getTicketComments(long id) {
-        return new PagedIterable<>(tmpl("/tickets/{id}/comments.json").set("id", id),
-                handleList(Comment.class, "comments"));
+        return getTicketComments(id, SortOrder.ASCENDING);
+    }
+
+    public Iterable<Comment> getTicketComments(long id, SortOrder order) {
+        return new PagedIterable<>(
+              tmpl("/tickets/{id}/comments.json?sort_order={order}")
+                .set("id", id)
+                .set("order", order.getQueryParameter()),
+              handleList(Comment.class, "comments"));
     }
 
     public Comment getRequestComment(org.zendesk.client.v2.model.Request request, Comment comment) {

src/test/java/org/zendesk/client/v2/RealSmokeTest.java

@@ -52,6 +52,7 @@
 import static org.hamcrest.CoreMatchers.nullValue;
 import static org.hamcrest.Matchers.greaterThan;
 import static org.hamcrest.Matchers.lessThanOrEqualTo;
+import static org.hamcrest.core.StringContains.containsString;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNotEquals;
 import static org.junit.Assert.assertNotNull;
@@ -1106,4 +1107,54 @@ public void createTicketForm() throws Exception {
         assertEquals(givenName, createdForm.getRawName());
         assertEquals(givenName, createdForm.getRawDisplayName());
     }
+
+    @Test
+    public void getTicketCommentsShouldBeAscending() throws Exception {
+        createClientWithTokenOrPassword();
+
+        Ticket t = new Ticket(
+              new Ticket.Requester(config.getProperty("requester.name"), config.getProperty("requester.email")),
+              "This is an automated test ticket", new Comment("1"));
+        Ticket ticket = null;
+        try {
+            ticket = instance.createTicket(t);
+            instance.createComment(ticket.getId(), new Comment("2"));
+            Iterable<Comment> ticketCommentsIt = instance.getTicketComments(ticket.getId());
+            List<Comment> comments = new ArrayList<>();
+            ticketCommentsIt.forEach(comments::add);
+
+            assertThat(comments.size(), is(2));
+            assertThat(comments.get(0).getBody(), containsString("1"));
+            assertThat(comments.get(1).getBody(), containsString("2"));
+        } finally {
+            if (ticket != null) {
+                instance.deleteTicket(ticket.getId());
+            }
+        }
+    }
+
+    @Test
+    public void getTicketCommentsDescending() throws Exception {
+        createClientWithTokenOrPassword();
+
+        Ticket t = new Ticket(
+              new Ticket.Requester(config.getProperty("requester.name"), config.getProperty("requester.email")),
+              "This is an automated test ticket", new Comment("1"));
+        Ticket ticket = null;
+        try {
+            ticket = instance.createTicket(t);
+            instance.createComment(ticket.getId(), new Comment("2"));
+            Iterable<Comment> ticketCommentsIt = instance.getTicketComments(ticket.getId(), SortOrder.DESCENDING);
+            List<Comment> comments = new ArrayList<>();
+            ticketCommentsIt.forEach(comments::add);
+
+            assertThat(comments.size(), is(2));
+            assertThat(comments.get(0).getBody(), containsString("2"));
+            assertThat(comments.get(1).getBody(), containsString("1"));
+        } finally {
+            if (ticket != null) {
+                instance.deleteTicket(ticket.getId());
+            }
+        }
+    }
 }