Commits:all() should use GET instead of POST

Author: vimishor

lib/Bitbucket/API/Repositories/Commits.php

@@ -20,7 +20,7 @@
 class Commits extends Api
 {
     /**
-     * Get a list of pull requests
+     * Get a list of commits
      *
      * @access public
      * @param  string           $account The team or individual account owning the repository.
@@ -37,7 +37,7 @@ public function all($account, $repo, $params = array())
             unset($params['branch']);
         }
 
-        return $this->getClient()->setApiVersion('2.0')->post(
+        return $this->getClient()->setApiVersion('2.0')->get(
             $endpoint,
             $params
         );

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

@@ -14,7 +14,7 @@ public function testGetAllRepositoryCommits()
 
         $client = $this->getHttpClientMock();
         $client->expects($this->once())
-            ->method('post')
+            ->method('get')
             ->with($endpoint)
             ->will($this->returnValue($expectedResult));
 
@@ -32,7 +32,7 @@ public function testGetAllRepositoryCommitsFromSpecificBranch()
 
         $client = $this->getHttpClientMock();
         $client->expects($this->once())
-            ->method('post')
+            ->method('get')
             ->with($endpoint)
             ->will($this->returnValue($expectedResult));