updated PullRequests::activity to support fetching a single PR activity

Author: vimishor

docs/repositories/pullrequests.md

@@ -77,7 +77,9 @@ $pull->delete($account_name, $repo_slug, 1);
 $pull->diff($account_name, $repo_slug, 1);
 ```
 
-### Get the log of all of a repository's pull request activity
+### Get the log of a pull request activity
+If pull request ID is omitted, the entire repository's pull request activity is returned.
+
 ```php
 $pull->activity($account_name, $repo_slug, 1);
 ```

lib/Bitbucket/API/Repositories/PullRequests.php

@@ -215,16 +215,25 @@ public function diff($account, $repo, $id)
     /**
      * Get the log of all of a repository's pull request activity
      *
+     * If `$id` is omitted the repository's pull request activity is returned.
+     * If `$id` is not omitted the pull request activity is returned.
+     *
      * @access public
      * @param  string           $account The team or individual account owning the repository.
      * @param  string           $repo    The repository identifier.
-     * @param  int              $id      ID of the pull request
+     * @param  int              $id      (Optional) ID of the pull request
      * @return MessageInterface
      */
-    public function activity($account, $repo, $id)
+    public function activity($account, $repo, $id = 0)
     {
-        return $this->getClient()->setApiVersion('2.0')->get(
-            sprintf('repositories/%s/%s/pullrequests/%d/activity', $account, $repo, $id)
-        );
+        $endpoint = sprintf('repositories/%s/%s/pullrequests/', $account, $repo);
+
+        if ($id === 0) {
+            $endpoint .= 'activity';
+        } else {
+            $endpoint .= $id.'/activity';
+        }
+
+        return $this->getClient()->setApiVersion('2.0')->get($endpoint);
     }
 }

test/Bitbucket/Tests/API/Repositories/PullRequestsTest.php

@@ -236,4 +236,22 @@ public function testGetPullRequestActivity()
 
         $this->assertEquals($expectedResult, $actual);
     }
+
+    public function testGetRepositoryPullRequestActivity()
+    {
+        $endpoint       = 'repositories/gentle/eof/pullrequests/activity';
+        $expectedResult = $this->fakeResponse(array('dummy'));
+
+        $client = $this->getHttpClientMock();
+        $client->expects($this->once())
+            ->method('get')
+            ->with($endpoint)
+            ->will($this->returnValue($expectedResult));
+
+        /** @var \Bitbucket\API\Repositories\PullRequests $pull */
+        $pull   = $this->getClassMock('Bitbucket\API\Repositories\PullRequests', $client);
+        $actual = $pull->activity('gentle', 'eof');
+
+        $this->assertEquals($expectedResult, $actual);
+    }
 }