Merge pull request GitLabPHP#113 from Fixico/add-commit-builds

Add commit builds to Repository API

Author: m4tthumphrey

lib/Gitlab/Api/Repositories.php

@@ -90,6 +90,24 @@ public function createTag($project_id, $name, $ref, $message = null)
         ));
     }
 
+    /**
+     * @param int $project_id
+     * @param string $sha
+     * @param string $scope
+     * @param int $page
+     * @param int $per_page
+     *
+     * @return mixed
+     */
+    public function commitBuilds($project_id, $sha, $scope = null, $page = 0, $per_page = self::PER_PAGE)
+    {
+        return $this->get($this->getProjectPath($project_id, 'repository/commits/'.$this->encodePath($sha).'/builds'), array(
+            'page' => $page,
+            'per_page' => $per_page,
+            'scope' => $scope
+        ));
+    }
+
     /**
      * @param int $project_id
      * @param int $page

test/Gitlab/Tests/Api/RepositoriesTest.php

@@ -170,6 +170,46 @@ public function shouldGetCommits()
         $this->assertEquals($expectedArray, $api->commits(1));
     }
 
+    /**
+     * @test
+     */
+    public function shouldGetCommitBuilds()
+    {
+        $expectedArray = array(
+            array('id' => 'abcd1234', 'status' => 'failed'),
+            array('id' => 'efgh5678', 'status' => 'success')
+        );
+
+        $api = $this->getApiMock();
+        $api->expects($this->once())
+            ->method('get')
+            ->with('projects/1/repository/commits/abcd12345/builds', array('page' => 0, 'per_page' => AbstractApi::PER_PAGE, 'scope' => null))
+            ->will($this->returnValue($expectedArray))
+        ;
+
+        $this->assertEquals($expectedArray, $api->commitBuilds(1, 'abcd12345'));
+    }
+
+    /**
+     * @test
+     */
+    public function shouldGetCommitBuildsWithScope()
+    {
+        $expectedArray = array(
+            array('id' => 'abcd1234', 'status' => 'success'),
+        );
+
+        $api = $this->getApiMock();
+        $api->expects($this->once())
+            ->method('get')
+            ->with('projects/1/repository/commits/abcd12345/builds', array('page' => 0, 'per_page' => AbstractApi::PER_PAGE, 'scope' => 'success'))
+            ->will($this->returnValue($expectedArray))
+        ;
+
+        $this->assertEquals($expectedArray, $api->commitBuilds(1, 'abcd12345', 'success'));
+    }
+
+
     /**
      * @test
      */