Merge pull request #53 from lyrixx/pr-pagination

Added pagination support for PullRequest

Author: pilot

lib/Github/Api/PullRequest.php

@@ -21,12 +21,20 @@ class PullRequest extends AbstractApi
      * @param  string $repository        the repository
      * @param  string $state             the state of the fetched pull requests.
      *                                   The API seems to automatically default to 'open'
+     * @param integer $page              the page
+     * @param integer $perPage           the per page
      *
      * @return array                     array of pull requests for the project
      */
-    public function all($username, $repository, $state = null)
+    public function all($username, $repository, $state = null, $page = 1, $perPage = 30)
     {
-        return $this->get('repos/'.urlencode($username).'/'.urlencode($repository).'/pulls', array('state' => $state));
+        $parameters = array(
+            'page' => $page,
+            'per_page' => $perPage,
+            'state' => $state,
+        );
+
+        return $this->get('repos/'.urlencode($username).'/'.urlencode($repository).'/pulls', $parameters);
     }
 
     /**

test/Github/Tests/Api/PullRequestTest.php

@@ -30,7 +30,7 @@ public function shouldGetOpenPullRequests()
         $api = $this->getApiMock();
         $api->expects($this->once())
             ->method('get')
-            ->with('repos/ezsystems/ezpublish/pulls', array('state' => 'open'))
+            ->with('repos/ezsystems/ezpublish/pulls', array('state' => 'open', 'per_page' => 30, 'page' => 1))
             ->will($this->returnValue($expectedArray));
 
         $this->assertEquals($expectedArray, $api->all('ezsystems', 'ezpublish', 'open'));
@@ -46,7 +46,7 @@ public function shouldGetClosedPullRequests()
         $api = $this->getApiMock();
         $api->expects($this->once())
             ->method('get')
-            ->with('repos/ezsystems/ezpublish/pulls', array('state' => 'closed'))
+            ->with('repos/ezsystems/ezpublish/pulls', array('state' => 'closed', 'per_page' => 30, 'page' => 1))
             ->will($this->returnValue($expectedArray));
 
         $this->assertEquals($expectedArray, $api->all('ezsystems', 'ezpublish', 'closed'));