Add method to list group variables

Author: jdecool

lib/Gitlab/Api/Groups.php

@@ -218,6 +218,19 @@ public function subgroups($groupId, array $parameters = [])
         return $this->get('groups/'.$this->encodePath($groupId).'/subgroups', $resolver->resolve($parameters));
     }
 
+    /**
+     * @param int $group_id
+     * @param array $parameters
+     *
+     * @return mixed
+     */
+    public function variables($group_id, array $parameters = [])
+    {
+        $resolver = $this->createOptionsResolver();
+
+        return $this->get($this->getGroupPath($group_id, 'variables'), $resolver->resolve($parameters));
+    }
+
     private function getGroupSearchResolver()
     {
         $resolver = $this->createOptionsResolver();

test/Gitlab/Tests/Api/GroupsTest.php

@@ -294,6 +294,23 @@ public function shouldGetAllSubgroups()
         $this->assertEquals($expectedArray, $api->subgroups(1, ['page' => 1, 'per_page' => 10]));
     }
 
+    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('groups/1/variables')
+            ->will($this->returnValue($expectedArray))
+        ;
+
+        $this->assertEquals($expectedArray, $api->variables(1));
+    }
+
     protected function getApiClass()
     {
         return 'Gitlab\Api\Groups';