Snippets tests and tidying up group tests

Author: Matt Humphrey

lib/Gitlab/Api/Snippets.php

@@ -26,16 +26,14 @@ public function show($project_id, $snippet_id)
      * @param string $title
      * @param string $filename
      * @param string $code
-     * @param string $lifetime
      * @return mixed
      */
-    public function create($project_id, $title, $filename, $code, $lifetime = null)
+    public function create($project_id, $title, $filename, $code)
     {
         return $this->post($this->getProjectPath($project_id, 'snippets'), array(
             'title' => $title,
             'file_name' => $filename,
-            'code' => $code,
-            'lifetime' => $lifetime
+            'code' => $code
         ));
     }
 

test/Gitlab/Tests/Api/GroupsTest.php

@@ -7,7 +7,7 @@ class GroupsTest extends TestCase
     /**
      * @test
      */
-    public function shouldGetAllProjects()
+    public function shouldGetAllGroups()
     {
         $expectedArray = array(
             array('id' => 1, 'name' => 'A group'),
@@ -183,6 +183,23 @@ public function shouldRemoveMember()
         $this->assertEquals($expectedBool, $api->removeMember(1, 2));
     }
 
+    /**
+     * @test
+     */
+    public function shouldRemoveGroup()
+    {
+        $expectedBool = true;
+
+        $api = $this->getApiMock();
+        $api->expects($this->once())
+            ->method('delete')
+            ->with('groups/1')
+            ->will($this->returnValue($expectedBool))
+        ;
+
+        $this->assertEquals($expectedBool, $api->remove(1));
+    }
+
     protected function getApiClass()
     {
         return 'Gitlab\Api\Groups';

test/Gitlab/Tests/Api/SnippetsTest.php

@@ -0,0 +1,116 @@
+<?php namespace Gitlab\Tests\Api;
+
+use Gitlab\Api\AbstractApi;
+
+class SnippetsTest extends TestCase
+{
+    /**
+     * @test
+     */
+    public function shouldGetAllSnippets()
+    {
+        $expectedArray = array(
+            array('id' => 1, 'title' => 'A snippet'),
+            array('id' => 2, 'title' => 'Another snippet'),
+        );
+
+        $api = $this->getApiMock();
+        $api->expects($this->once())
+            ->method('get')
+            ->with('projects/1/snippets')
+            ->will($this->returnValue($expectedArray))
+        ;
+
+        $this->assertEquals($expectedArray, $api->all(1));
+    }
+
+    /**
+     * @test
+     */
+    public function shouldShowSnippet()
+    {
+        $expectedArray = array('id' => 2, 'title' => 'Another snippet');
+
+        $api = $this->getApiMock();
+        $api->expects($this->once())
+            ->method('get')
+            ->with('projects/1/snippets/2')
+            ->will($this->returnValue($expectedArray))
+        ;
+
+        $this->assertEquals($expectedArray, $api->show(1, 2));
+    }
+
+    /**
+     * @test
+     */
+    public function shouldCreateSnippet()
+    {
+        $expectedArray = array('id' => 3, 'title' => 'A new snippet');
+
+        $api = $this->getApiMock();
+        $api->expects($this->once())
+            ->method('post')
+            ->with('projects/1/snippets', array('title' => 'A new snippet', 'code' => 'A file', 'file_name' => 'file.txt'))
+            ->will($this->returnValue($expectedArray))
+        ;
+
+        $this->assertEquals($expectedArray, $api->create(1, 'A new snippet', 'file.txt', 'A file'));
+    }
+
+    /**
+     * @test
+     */
+    public function shouldUpdateSnippet()
+    {
+        $expectedArray = array('id' => 3, 'title' => 'Updated snippet');
+
+        $api = $this->getApiMock();
+        $api->expects($this->once())
+            ->method('put')
+            ->with('projects/1/snippets/3', array('title' => 'Updated snippet', 'code' => 'New content', 'file_name' => 'new_file.txt'))
+            ->will($this->returnValue($expectedArray))
+        ;
+
+        $this->assertEquals($expectedArray, $api->update(1, 3, array('file_name' => 'new_file.txt', 'code' => 'New content', 'title' => 'Updated snippet')));
+    }
+
+    /**
+     * @test
+     */
+    public function shouldShowContent()
+    {
+        $expectedString = 'New content';
+
+        $api = $this->getApiMock();
+        $api->expects($this->once())
+            ->method('get')
+            ->with('projects/1/snippets/3/raw')
+            ->will($this->returnValue($expectedString))
+        ;
+
+        $this->assertEquals($expectedString, $api->content(1, 3));
+    }
+
+    /**
+     * @test
+     */
+    public function shouldRemoveSnippet()
+    {
+        $expectedBool = true;
+
+        $api = $this->getApiMock();
+        $api->expects($this->once())
+            ->method('delete')
+            ->with('projects/1/snippets/3')
+            ->will($this->returnValue($expectedBool))
+        ;
+
+        $this->assertEquals($expectedBool, $api->remove(1, 3));
+    }
+
+    protected function getApiClass()
+    {
+        return 'Gitlab\Api\Snippets';
+    }
+}