Merge pull request #258 from kayladnls/gistComments

Gist comments

Author: pilot

lib/Github/Api/Gist/Comments.php

@@ -0,0 +1,37 @@
+<?php
+
+namespace Github\Api\Gist;
+
+use Github\Api\AbstractApi;
+
+/**
+ * @link   https://developer.github.com/v3/gists/comments/
+ * @author Kayla Daniels <[email protected]>
+ */
+class Comments extends AbstractApi
+{
+    public function all($gist)
+    {
+        return $this->get('gists/'.rawurlencode($gist).'/comments');
+    }
+
+    public function show($gist, $comment)
+    {
+        return $this->get('gists/'.rawurlencode($gist).'/comments/'.rawurlencode($comment));
+    }
+
+    public function create($gist, $body)
+    {
+        return $this->post('gists/'.rawurlencode($gist).'/comments', array($body));
+    }
+
+    public function update($gist, $comment_id, $body)
+    {
+        return $this->patch('gists/'.rawurlencode($gist).'/comments/'.rawurlencode($comment_id), array($body));
+    }
+
+    public function remove($gist, $comment)
+    {
+        return $this->delete('gists/'.rawurlencode($gist).'/comments/'.rawurlencode($comment));
+    }
+}

lib/Github/Api/Gists.php

@@ -2,8 +2,8 @@
 
 namespace Github\Api;
 
-use Github\Api\AbstractApi;
 use Github\Exception\MissingArgumentException;
+use Github\Api\Gist\Comments;
 
 /**
  * Creating, editing, deleting and listing gists.
@@ -73,4 +73,16 @@ public function unstar($id)
     {
         return $this->delete('gists/'.rawurlencode($id).'/star');
     }
+
+    /**
+     * Get a gist's comments.
+     *
+     * @link http://developer.github.com/v3/gists/comments/
+     *
+     * @return Comments
+     */
+    public function comments()
+    {
+        return new Comments($this->client);
+    }
 }

test/Github/Tests/Api/Gist/CommentsTest.php

@@ -0,0 +1,94 @@
+<?php
+
+namespace Github\Tests\Api\Gist;
+
+use Github\Tests\Api\TestCase;
+
+class CommentsTest extends TestCase
+{
+    /**
+     * @test
+     */
+    public function shouldGetAllGistComments()
+    {
+        $expectedValue = array(array('comment1data'), array('comment2data'));
+
+        $api = $this->getApiMock();
+        $api->expects($this->once())
+            ->method('get')
+            ->with('gists/123/comments')
+            ->will($this->returnValue($expectedValue));
+
+        $this->assertEquals($expectedValue, $api->all('123'));
+    }
+
+    /**
+     * @test
+     */
+    public function shouldShowGistComment()
+    {
+        $expectedValue = array('comment1');
+
+        $api = $this->getApiMock();
+        $api->expects($this->once())
+            ->method('get')
+            ->with('gists/123/comments/123')
+            ->will($this->returnValue($expectedValue));
+
+        $this->assertEquals($expectedValue, $api->show(123, 123));
+    }
+
+    /**
+     * @test
+     */
+    public function shouldCreateGistComment()
+    {
+        $expectedValue = array('comment1data');
+
+        $api = $this->getApiMock();
+        $api->expects($this->once())
+            ->method('post')
+            ->with('gists/123/comments', array('Test body'))
+            ->will($this->returnValue($expectedValue));
+
+        $this->assertEquals($expectedValue, $api->create('123', 'Test body'));
+    }
+
+    /**
+     * @test
+     */
+    public function shouldUpdateGistComment()
+    {
+        $expectedValue = array('comment1data');
+        $data = array('body test');
+
+        $api = $this->getApiMock();
+        $api->expects($this->once())
+            ->method('patch')
+            ->with('gists/123/comments/233', $data)
+            ->will($this->returnValue($expectedValue));
+
+        $this->assertEquals($expectedValue, $api->update(123, 233, 'body test'));
+    }
+
+    /**
+     * @test
+     */
+    public function shouldRemoveComment()
+    {
+        $expectedValue = array('someOutput');
+
+        $api = $this->getApiMock();
+        $api->expects($this->once())
+            ->method('delete')
+            ->with('gists/123/comments/233')
+            ->will($this->returnValue($expectedValue));
+
+        $this->assertEquals($expectedValue, $api->remove(123, 233));
+    }
+
+    protected function getApiClass()
+    {
+        return 'Github\Api\Gist\Comments';
+    }
+}

test/Github/Tests/Api/GistsTest.php

@@ -68,6 +68,16 @@ public function shouldShowCommits()
         $this->assertEquals($expectedArray, $api->commits(123));
     }
 
+    /**
+     * @test
+     */
+    public function shouldGetCommentsApiObject()
+    {
+        $api = $this->getApiMock();
+
+        $this->assertInstanceOf('Github\Api\Gist\Comments', $api->comments());
+    }
+
     /**
      * @test
      */
@@ -216,4 +226,4 @@ protected function getApiClass()
     {
         return 'Github\Api\Gists';
     }
-}
+}