Merge pull request spotify#50 from spotify/pr/add-params

backuitist · web-flow · commit 995446639b52 · 2021-02-01T16:21:15.000+01:00
feat: add `page` &amp; `per_page` to PullRequestParameters
diff --git a/src/main/java/com/spotify/github/v3/prs/requests/PullRequestParameters.java b/src/main/java/com/spotify/github/v3/prs/requests/PullRequestParameters.java
@@ -58,5 +58,16 @@ public interface PullRequestParameters extends Parameters {
    */
   Optional<String> direction();
 
+  /**
+   * Results per page (max 100)
+   */
+  @SuppressWarnings("checkstyle:methodname")
+  Optional<Integer> per_page();
+
+  /**
+   * Page number of the results to fetch.
+   */
+  Optional<Integer> page();
+
   /** Serialize declared non-Object methods as key=value joined by & */
 }
diff --git a/src/test/java/com/spotify/github/v3/prs/requests/PullRequestParametersTest.java b/src/test/java/com/spotify/github/v3/prs/requests/PullRequestParametersTest.java
@@ -36,11 +36,14 @@ public void testFullSerialize() {
             .head("bogus-head")
             .sort("bogus-sort")
             .state("bogus-state")
+            .page(2)
+            .per_page(5)
             .build();
 
     assertThat(
         params.serialize(),
         is(
-            "base=bogus-base&direction=bogus-direction&head=bogus-head&sort=bogus-sort&state=bogus-state"));
+            "base=bogus-base&direction=bogus-direction&head=bogus-head&page=2" +
+                    "&per_page=5&sort=bogus-sort&state=bogus-state"));
   }
 }
