Merge pull request GitLabPHP#133 from EdwinHoksberg/buildVariables

Add support for build variables

Author: m4tthumphrey

lib/Gitlab/Api/Projects.php

@@ -417,4 +417,60 @@ public function removeService($project_id, $service_name)
     {
         return $this->delete($this->getProjectPath($project_id, 'services/'.$this->encodePath($service_name)));
     }
+
+    /**
+     * @param int $project_id
+     * @return mixed
+     */
+    public function variables($project_id)
+    {
+        return $this->get($this->getProjectPath($project_id, 'variables'));
+    }
+
+    /**
+     * @param int $project_id
+     * @param string $key
+     * @return mixed
+     */
+    public function variable($project_id, $key)
+    {
+        return $this->get($this->getProjectPath($project_id, 'variables/'.$this->encodePath($key)));
+    }
+
+    /**
+     * @param int $project_id
+     * @param string $key
+     * @param string $value
+     * @return mixed
+     */
+    public function addVariable($project_id, $key, $value)
+    {
+        return $this->post($this->getProjectPath($project_id, 'variables'), array(
+            'key'   => $key,
+            'value' => $value
+        ));
+    }
+
+    /**
+     * @param int $project_id
+     * @param string $key
+     * @param string $value
+     * @return mixed
+     */
+    public function updateVariable($project_id, $key, $value)
+    {
+        return $this->put($this->getProjectPath($project_id, 'variables/'.$this->encodePath($key)), array(
+            'value' => $value,
+        ));
+    }
+
+    /**
+     * @param int $project_id
+     * @param string $key
+     * @return mixed
+     */
+    public function removeVariable($project_id, $key)
+    {
+        return $this->delete($this->getProjectPath($project_id, 'variables/'.$this->encodePath($key)));
+    }
 }

test/Gitlab/Tests/Api/ProjectsTest.php

@@ -254,7 +254,7 @@ public function shouldGetBuild()
 
         $this->assertEquals($expectedArray, $api->build(1, 2));
     }
-    
+
     /**
      * @test
      */
@@ -757,6 +757,106 @@ public function shouldRemoveService()
         $this->assertEquals($expectedBool, $api->removeService(1, 'hipchat'));
     }
 
+    /**
+     * @test
+     */
+    public function shouldGetVariables()
+    {
+        $expectedArray = array(
+            array('key' => 'ftp_username', 'value' => 'ftp'),
+            array('key' => 'ftp_password', 'value' => 'somepassword')
+        );
+
+        $api = $this->getApiMock();
+        $api->expects($this->once())
+            ->method('get')
+            ->with('projects/1/variables')
+            ->will($this->returnValue($expectedArray))
+        ;
+
+        $this->assertEquals($expectedArray, $api->variables(1));
+    }
+
+    /**
+     * @test
+     */
+    public function shouldGetVariable()
+    {
+        $expectedArray = array('key' => 'ftp_username', 'value' => 'ftp');
+
+        $api = $this->getApiMock();
+        $api->expects($this->once())
+            ->method('get')
+            ->with('projects/1/variables/ftp_username')
+            ->will($this->returnValue($expectedArray))
+        ;
+
+        $this->assertEquals($expectedArray, $api->variable(1, 'ftp_username'));
+    }
+
+    /**
+     * @test
+     */
+    public function shouldAddVariable()
+    {
+        $expectedKey   = 'ftp_port';
+        $expectedValue = '21';
+
+        $expectedArray = array(
+            'key'   => $expectedKey,
+            'value' => $expectedValue,
+        );
+
+        $api = $this->getApiMock();
+        $api->expects($this->once())
+            ->method('post')
+            ->with('projects/1/variables', $expectedArray)
+            ->will($this->returnValue($expectedArray))
+        ;
+
+        $this->assertEquals($expectedArray, $api->addVariable(1, $expectedKey, $expectedValue));
+    }
+
+    /**
+     * @test
+     */
+    public function shouldUpdateVariable()
+    {
+        $expectedKey   = 'ftp_port';
+        $expectedValue = '22';
+
+        $expectedArray = array(
+            'key'   => 'ftp_port',
+            'value' => '22',
+        );
+
+        $api = $this->getApiMock();
+        $api->expects($this->once())
+            ->method('put')
+            ->with('projects/1/variables/'.$expectedKey, array('value' => $expectedValue))
+            ->will($this->returnValue($expectedArray))
+        ;
+
+        $this->assertEquals($expectedArray, $api->updateVariable(1, $expectedKey, $expectedValue));
+    }
+
+    /**
+     * @test
+     */
+    public function shouldRemoveVariable()
+    {
+        $expectedBool = true;
+
+        $api = $this->getApiMock();
+        $api->expects($this->once())
+            ->method('delete')
+            ->with('projects/1/variables/ftp_password')
+            ->will($this->returnValue($expectedBool))
+        ;
+
+        $this->assertEquals($expectedBool, $api->removeVariable(1, 'ftp_password'));
+    }
+
     protected function getMultipleProjectsRequestMock($path, $expectedArray = array(), $page = 1, $per_page = 20, $order_by = 'created_at', $sort = 'asc')
     {
         $api = $this->getApiMock();