Improve code coverage

Author: bitwiseman

pom.xml

@@ -550,6 +550,7 @@
               <exclude>org.kohsuke.github.PagedIterable#toArray(org.kohsuke.github.PagedIterator)</exclude>
               <exclude>org.kohsuke.github.PagedIterable#PagedIterable()</exclude>
               <exclude>org.kohsuke.github.PagedIterator#base</exclude>
+              <exclude>org.kohsuke.github.PagedIterable#iterator()</exclude>
               <exclude>org.kohsuke.github.PagedIterator#wrapUp(java.lang.Object[])</exclude>
               <exclude>org.kohsuke.github.PagedSearchIterable#_iterator(int)</exclude>
               <exclude>org.kohsuke.github.PagedSearchIterable#adapt(java.util.Iterator)</exclude>

src/main/java/org/kohsuke/github/PagedIterable.java

@@ -1,6 +1,7 @@
 package org.kohsuke.github;
 
 import java.io.IOException;
+import java.util.Iterator;
 import java.util.List;
 import java.util.Set;
 
@@ -26,8 +27,8 @@ public class PagedIterable<T> implements Iterable<T> {
     }
 
     @Nonnull
-    public final PagedIterator<T> iterator() {
-        return new PagedIterator<>(paginatedEndpoint.items());
+    public final Iterator<T> iterator() {
+        return paginatedEndpoint.iterator();
     }
 
     @Nonnull

src/main/java/org/kohsuke/github/PagedIterator.java

@@ -43,7 +43,7 @@ public T next() {
      * Get the next page of items.
      *
      * @return a list of the next page of items.
-     * @deprecated use PagedIterable.pageIterator().
+     * @deprecated use PagedIterable.pages().
      */
     @Deprecated
     public List<T> nextPage() {

src/test/java/org/kohsuke/github/AppTest.java

@@ -362,7 +362,7 @@ public void testCommit() throws Exception {
                 .getRepository("jenkins")
                 .getCommit("08c1c9970af4d609ae754fbe803e06186e3206f7");
         assertThat(commit.getParents().size(), equalTo(1));
-        assertThat(commit.listFiles().toList().size(), equalTo(1));
+        assertThat(commit.listFiles().withPageSize(50).toList().size(), equalTo(1));
         assertThat(commit.getHtmlUrl().toString(),
                 equalTo("https://github.com/jenkinsci/jenkins/commit/08c1c9970af4d609ae754fbe803e06186e3206f7"));
         assertThat(commit.getLinesAdded(), equalTo(40));

src/test/java/org/kohsuke/github/GitHubTest.java

@@ -294,10 +294,11 @@ public void searchContent() throws Exception {
 
         PagedSearchIterable<GHContent> r4 = searchBuilder.list();
 
-        GHContent c4 = r4.iterator().next();
+        GHContent c4 = r4.withPageSize(25).iterator().next();
         assertThat(c4.getPath(), not(equalTo(c2.getPath())));
         assertThat(c4.getPath(), not(equalTo(c3.getPath())));
         assertThat(r4.getTotalCount(), equalTo(r2.getTotalCount()));
+        assertThat(r4.isIncomplete(), equalTo(false));
 
         // Verify qualifier not allowed to be empty
         IllegalArgumentException e = Assert.assertThrows(IllegalArgumentException.class,

src/test/resources/org/kohsuke/github/GitHubTest/wiremock/searchContent/mappings/6-search_code.json

@@ -2,7 +2,7 @@
   "id": "a9b8870b-33d7-4164-a407-310342d68536",
   "name": "search_code",
   "request": {
-    "url": "/search/code?sort=indexed&order=desc&q=addClass+in%3Afile+language%3Ajs+repo%3Ajquery%2Fjquery",
+    "url": "/search/code?sort=indexed&order=desc&q=addClass+in%3Afile+language%3Ajs+repo%3Ajquery%2Fjquery&per_page=25",
     "method": "GET",
     "headers": {
       "Accept": {