Merge pull request #747 from jenkinsci/larger-pagesize

Request files for each PR in larger batch
jenkinsci · Nov 4, 2019 · 96ada00 · 96ada00
2 parents f4eb4bc + 42f21b7
commit 96ada00
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/main/java/org/jenkinsci/plugins/ghprb/GhprbPullRequest.java b/src/main/java/org/jenkinsci/plugins/ghprb/GhprbPullRequest.java
@@ -466,7 +466,7 @@ boolean containsWatchedPaths(GHPullRequest pr) {
             }
 
             List<String> paths = new ArrayList<String>();
-            for (GHPullRequestFileDetail fileDetail : pr.listFiles()) {
+            for (GHPullRequestFileDetail fileDetail : pr.listFiles().withPageSize(100)) {
                 paths.add(fileDetail.getFilename());
             }
 

diff --git a/src/test/java/org/jenkinsci/plugins/ghprb/GhprbPullRequestTest.java b/src/test/java/org/jenkinsci/plugins/ghprb/GhprbPullRequestTest.java
@@ -92,6 +92,7 @@ public void setup() throws IOException {
         @SuppressWarnings("unchecked")
         PagedIterator<GHPullRequestFileDetail> pagedIterator = mock(PagedIterator.class);
         given(pagedIterable.iterator()).willReturn(pagedIterator);
+        given(pagedIterable.withPageSize(100)).willReturn(pagedIterable);
         given(pr.listFiles()).willReturn(pagedIterable);
 
         // Create the list of file paths to return