Add method to create group variable

Author: jdecool

lib/Gitlab/Api/Groups.php

@@ -242,6 +242,28 @@ public function variable($group_id, $key)
         return $this->get($this->getGroupPath($group_id, 'variables/'.$this->encodePath($key)));
     }
 
+    /**
+     * @param int $group_id
+     * @param string $key
+     * @param string $value
+     * @param bool $protected
+     *
+     * @return mixed
+     */
+    public function addVariable($group_id, $key, $value, $protected = null)
+    {
+        $payload = array(
+            'key' => $key,
+            'value' => $value,
+        );
+
+        if ($protected) {
+            $payload['protected'] = $protected;
+        }
+
+        return $this->post($this->getGroupPath($group_id, 'variables'), $payload);
+    }
+
     private function getGroupSearchResolver()
     {
         $resolver = $this->createOptionsResolver();

test/Gitlab/Tests/Api/GroupsTest.php

@@ -328,6 +328,47 @@ public function shouldGetVariable()
         $this->assertEquals($expectedArray, $api->variable(1, 'ftp_username'));
     }
 
+    public function shouldAddVariable()
+    {
+        $expectedKey   = 'ftp_port';
+        $expectedValue = '21';
+
+        $expectedArray = array(
+            'key'   => $expectedKey,
+            'value' => $expectedValue,
+        );
+
+        $api = $this->getApiMock();
+        $api->expects($this->once())
+            ->method('post')
+            ->with('groups/1/variables', $expectedArray)
+            ->will($this->returnValue($expectedArray))
+        ;
+
+        $this->assertEquals($expectedArray, $api->addVariable(1, $expectedKey, $expectedValue));
+    }
+
+    /**
+     * @test
+     */
+    public function shouldAddVariableWithProtected()
+    {
+        $expectedArray = array(
+            'key'   => 'DEPLOY_SERVER',
+            'value' => 'stage.example.com',
+            'protected' => true,
+        );
+
+        $api = $this->getApiMock();
+        $api->expects($this->once())
+            ->method('post')
+            ->with('groups/1/variables', $expectedArray)
+            ->will($this->returnValue($expectedArray))
+        ;
+
+        $this->assertEquals($expectedArray, $api->addVariable(1, 'DEPLOY_SERVER', 'stage.example.com', true));
+    }
+
     protected function getApiClass()
     {
         return 'Gitlab\Api\Groups';