Add support for fetching statuses (#565)

* Add support for fetching statuses

* fix style ci

* Add docs, add test, linter cleanup

* Merge in master

* Update language

* Fix test

Author: slaughter550

lib/Github/Api/PullRequest.php

@@ -10,7 +10,8 @@
 /**
  * API for accessing Pull Requests from your Git/Github repositories.
  *
- * @link   http://developer.github.com/v3/pulls/
+ * @see   http://developer.github.com/v3/pulls/
+ *
  * @author Joseph Bielawski <[email protected]>
  */
 class PullRequest extends AbstractApi
@@ -60,7 +61,7 @@ public function all($username, $repository, array $params = array())
     {
         $parameters = array_merge(array(
             'page' => 1,
-            'per_page' => 30
+            'per_page' => 30,
         ), $params);
 
         return $this->get('/repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/pulls', $parameters);
@@ -92,6 +93,24 @@ public function files($username, $repository, $id)
         return $this->get('/repos/'.rawurlencode($username).'/'.rawurlencode($repository).'/pulls/'.rawurlencode($id).'/files');
     }
 
+    /**
+     * All statuses which are the statuses of its head branch.
+     *
+     * @see http://developer.github.com/v3/pulls/
+     *
+     * @param string $username   the username
+     * @param string $repository the repository
+     * @param string $id         the ID of the pull request for which statuses are retrieved
+     *
+     * @return array array of statuses for the project
+     */
+    public function status($username, $repository, $id)
+    {
+        $link = $this->show($username, $repository, $id)['_links']['statuses']['href'];
+
+        return $this->get($link);
+    }
+
     public function comments()
     {
         return new Comments($this->client);

test/Github/Tests/Api/PullRequestTest.php

@@ -101,6 +101,28 @@ public function shouldShowFilesFromPullRequest()
         $this->assertEquals($expectedArray, $api->files('ezsystems', 'ezpublish', '15'));
     }
 
+    /**
+     * @test
+     */
+    public function shouldShowStatusesFromPullRequest()
+    {
+        $expectedArray = array(array('id' => 'id', 'sha' => '123123'));
+        $expectedArray['_links']['statuses']['href'] = '/repos/ezsystems/ezpublish/pulls/15/statuses';
+
+        $api = $this->getApiMock();
+        $api->expects($this->at(0))
+            ->method('get')
+            ->with('/repos/ezsystems/ezpublish/pulls/15')
+            ->will($this->returnValue($expectedArray));
+
+        $api->expects($this->at(1))
+            ->method('get')
+            ->with('/repos/ezsystems/ezpublish/pulls/15/statuses')
+            ->will($this->returnValue($expectedArray));
+
+        $this->assertEquals($expectedArray, $api->status('ezsystems', 'ezpublish', '15'));
+    }
+
     /**
      * @test
      */
@@ -187,10 +209,10 @@ public function shouldMergePullRequestWithMergeMethod()
     public function shouldCreatePullRequestUsingTitle()
     {
         $data = array(
-            'base'  => 'master',
-            'head'  => 'virtualtestbranch',
+            'base' => 'master',
+            'head' => 'virtualtestbranch',
             'title' => 'TITLE: Testing pull-request creation from PHP Github API',
-            'body'  => 'BODY: Testing pull-request creation from PHP Github API'
+            'body' => 'BODY: Testing pull-request creation from PHP Github API',
         );
 
         $api = $this->getApiMock();
@@ -207,9 +229,9 @@ public function shouldCreatePullRequestUsingTitle()
     public function shouldCreatePullRequestUsingIssueId()
     {
         $data = array(
-            'base'  => 'master',
-            'head'  => 'virtualtestbranch',
-            'issue' => 25
+            'base' => 'master',
+            'head' => 'virtualtestbranch',
+            'issue' => 25,
         );
 
         $api = $this->getApiMock();
@@ -227,9 +249,9 @@ public function shouldCreatePullRequestUsingIssueId()
     public function shouldNotCreatePullRequestWithoutBase()
     {
         $data = array(
-            'head'  => 'virtualtestbranch',
+            'head' => 'virtualtestbranch',
             'title' => 'TITLE: Testing pull-request creation from PHP Github API',
-            'body'  => 'BODY: Testing pull-request creation from PHP Github API'
+            'body' => 'BODY: Testing pull-request creation from PHP Github API',
         );
 
         $api = $this->getApiMock();
@@ -246,9 +268,9 @@ public function shouldNotCreatePullRequestWithoutBase()
     public function shouldNotCreatePullRequestWithoutHead()
     {
         $data = array(
-            'base'  => 'master',
+            'base' => 'master',
             'title' => 'TITLE: Testing pull-request creation from PHP Github API',
-            'body'  => 'BODY: Testing pull-request creation from PHP Github API'
+            'body' => 'BODY: Testing pull-request creation from PHP Github API',
         );
 
         $api = $this->getApiMock();
@@ -265,8 +287,8 @@ public function shouldNotCreatePullRequestWithoutHead()
     public function shouldNotCreatePullRequestUsingTitleButWithoutBody()
     {
         $data = array(
-            'base'  => 'master',
-            'head'  => 'virtualtestbranch',
+            'base' => 'master',
+            'head' => 'virtualtestbranch',
             'title' => 'TITLE: Testing pull-request creation from PHP Github API',
         );
 
@@ -284,8 +306,8 @@ public function shouldNotCreatePullRequestUsingTitleButWithoutBody()
     public function shouldNotCreatePullRequestWithoutIssueIdOrTitle()
     {
         $data = array(
-            'base'  => 'master',
-            'head'  => 'virtualtestbranch',
+            'base' => 'master',
+            'head' => 'virtualtestbranch',
         );
 
         $api = $this->getApiMock();