Add method to remove group variable

Author: jdecool

lib/Gitlab/Api/Groups.php

@@ -285,6 +285,17 @@ public function updateVariable($group_id, $key, $value, $protected = null)
         return $this->put($this->getGroupPath($group_id, 'variables/'.$this->encodePath($key)), $payload);
     }
 
+    /**
+     * @param int $group_id
+     * @param string $key
+     *
+     * @return mixed
+     */
+    public function removeVariable($group_id, $key)
+    {
+        return $this->delete($this->getGroupPath($group_id, 'variables/'.$this->encodePath($key)));
+    }
+
     private function getGroupSearchResolver()
     {
         $resolver = $this->createOptionsResolver();

test/Gitlab/Tests/Api/GroupsTest.php

@@ -413,6 +413,23 @@ public function shouldUpdateVariableWithProtected()
         $this->assertEquals($expectedArray, $api->updateVariable(1, 'DEPLOY_SERVER', 'stage.example.com', true));
     }
 
+    /**
+     * @test
+     */
+    public function shouldRemoveVariable()
+    {
+        $expectedBool = true;
+
+        $api = $this->getApiMock();
+        $api->expects($this->once())
+            ->method('delete')
+            ->with('groups/1/variables/ftp_password')
+            ->will($this->returnValue($expectedBool))
+        ;
+
+        $this->assertEquals($expectedBool, $api->removeVariable(1, 'ftp_password'));
+    }
+
     protected function getApiClass()
     {
         return 'Gitlab\Api\Groups';