github_api 0.2.0 → 0.2.1

data/spec/github/repos/collaborators_spec.rb

@@ -1,6 +1,169 @@
 require 'spec_helper'
 
 describe Github::Repos::Collaborators do
-  
-  pending
-end
+
+  let(:github) { Github.new }
+  let(:user)   { 'peter-murach' }
+  let(:repo)   { 'github' }
+  let(:collaborator) { 'octocat' }
+
+  describe "collaborators" do
+    context "resource found" do
+      before do
+        stub_get("/repos/#{user}/#{repo}/collaborators").
+          to_return(:body => fixture('repos/collaborators.json'), :status => 200, :headers => {:content_type => "application/json; charset=utf-8"})
+      end
+
+      it "should fail to get resource without username" do
+        github.user, github.repo = nil, nil
+        expect { github.repos.collaborators }.to raise_error(ArgumentError)
+      end
+
+      it "should get the resources" do
+        github.repos.collaborators user, repo
+        a_get("/repos/#{user}/#{repo}/collaborators").should have_been_made
+      end
+
+      it "should return array of resources" do
+        collaborators = github.repos.collaborators user, repo
+        collaborators.should be_an Array
+        collaborators.should have(1).items
+      end
+
+      it "should be a mash type" do
+        collaborators = github.repos.collaborators user, repo
+        collaborators.first.should be_a Hashie::Mash
+      end
+
+      it "should get collaborator information" do
+        collaborators = github.repos.collaborators user, repo
+        collaborators.first.login.should == 'octocat'
+      end
+
+      it "should yield to a block" do
+        github.repos.should_receive(:collaborators).with(user, repo).and_yield('web')
+        github.repos.collaborators(user, repo) { |param| 'web' }
+      end
+    end
+
+    context "resource not found" do
+      before do
+        stub_get("/repos/#{user}/#{repo}/collaborators").
+          to_return(:body => "", :status => [404, "Not Found"], :headers => {:content_type => "application/json; charset=utf-8"})
+      end
+
+      it "should return 404 with a message 'Not Found'" do
+        expect {
+          github.repos.collaborators user, repo
+        }.to raise_error(Github::ResourceNotFound)
+      end
+    end
+  end # collaborators
+
+
+  describe "collaborator?" do
+    context "resource found " do
+      before do
+        stub_get("/repos/#{user}/#{repo}/collaborators/#{collaborator}").
+          to_return(:body => '', :status => 204, :headers => {:content_type => "application/json; charset=utf-8"})
+      end
+
+      it "should fail to get resource without collaborator name" do
+        expect {
+          github.repos.collaborator?(user, repo, nil)
+        }.to raise_error(ArgumentError)
+      end
+
+      it "should get the resource" do
+        github.repos.collaborator? user, repo, collaborator
+        a_get("/repos/#{user}/#{repo}/collaborators/#{collaborator}").should have_been_made
+      end
+
+      it "should find collaborator" do
+        github.repos.should_receive(:collaborator?).with(user, repo, collaborator).and_return true
+        github.repos.collaborator? user, repo, collaborator
+      end
+    end
+
+    context "resource not found" do
+      before do
+        stub_get("/repos/#{user}/#{repo}/collaborators/#{collaborator}").
+          to_return(:body => '', :status => 404, :headers => {:content_type => "application/json; charset=utf-8"})
+      end
+
+      it "should fail to retrieve resource" do
+        github.repos.should_receive(:collaborator?).with(user, repo, collaborator).and_return false
+        github.repos.collaborator? user, repo, collaborator
+      end
+    end
+  end # collaborator?
+
+  describe "add_collaborator" do
+
+    context "resouce added" do
+      before do
+        stub_put("/repos/#{user}/#{repo}/collaborators/#{collaborator}").
+          to_return(:body => '', :status => 204, :headers => {:content_type => "application/json; charset=utf-8"})
+      end
+
+      it "should fail to add resource if 'collaborator' input is missing" do
+        expect {
+          github.repos.add_collaborator user, repo, nil
+        }.to raise_error(ArgumentError)
+      end
+
+      it "should add resource successfully" do
+        github.repos.add_collaborator user, repo, collaborator
+        a_put("/repos/#{user}/#{repo}/collaborators/#{collaborator}").should have_been_made
+      end
+    end
+
+    context "failed to add resource" do
+      before do
+        stub_put("/repos/#{user}/#{repo}/collaborators/#{collaborator}").
+          to_return(:body => '', :status => 404, :headers => {:content_type => "application/json; charset=utf-8"})
+      end
+
+      it "should fail to add resource" do
+        expect {
+          github.repos.add_collaborator user, repo, collaborator
+        }.to raise_error(Github::ResourceNotFound)
+      end
+    end
+  end # add_collaborator
+
+  describe "remove_collaborator" do
+
+    context "resouce added" do
+      before do
+        stub_delete("/repos/#{user}/#{repo}/collaborators/#{collaborator}").
+          to_return(:body => '', :status => 204, :headers => {:content_type => "application/json; charset=utf-8"})
+      end
+
+      it "should fail to add resource if 'collaborator' input is missing" do
+        expect {
+          github.repos.remove_collaborator user, repo, nil
+        }.to raise_error(ArgumentError)
+      end
+
+      it "should add resource successfully" do
+        github.repos.remove_collaborator user, repo, collaborator
+        a_delete("/repos/#{user}/#{repo}/collaborators/#{collaborator}").should have_been_made
+      end
+    end
+
+    context "failed to remove resource" do
+      before do
+        stub_delete("/repos/#{user}/#{repo}/collaborators/#{collaborator}").
+          to_return(:body => '', :status => 404, :headers => {:content_type => "application/json; charset=utf-8"})
+      end
+
+      it "should fail to remove resource" do
+        expect {
+          github.repos.remove_collaborator user, repo, collaborator
+        }.to raise_error(Github::ResourceNotFound)
+      end
+    end
+  end # remove_collaborator
+
+end # Github::Repos::Collaborators

data/spec/github/repos/commits_spec.rb

@@ -1,5 +1,424 @@
 require 'spec_helper'
 
 describe Github::Repos::Commits do
-  pending
-end
+
+  let(:github) { Github.new }
+  let(:user)   { 'peter-murach' }
+  let(:repo)   { 'github' }
+
+  describe "commits" do
+    context "resource found" do
+      before do
+        stub_get("/repos/#{user}/#{repo}/commits").
+          to_return(:body => fixture('repos/commits.json'), :status => 200, :headers => {:content_type => "application/json; charset=utf-8"})
+      end
+
+      it "should fail to get resource without username" do
+        github.user, github.repo = nil, nil
+        expect { github.repos.commits }.to raise_error(ArgumentError)
+      end
+
+      it "should get the resources" do
+        github.repos.commits user, repo
+        a_get("/repos/#{user}/#{repo}/commits").should have_been_made
+      end
+
+      it "should return array of resources" do
+        commits = github.repos.commits user, repo
+        commits.should be_an Array
+        commits.should have(1).items
+      end
+
+      it "should be a mash type" do
+        commits = github.repos.commits user, repo
+        commits.first.should be_a Hashie::Mash
+      end
+
+      it "should get commit information" do
+        commits = github.repos.commits user, repo
+        commits.first.author.name.should == 'Scott Chacon'
+      end
+
+      it "should yield to a block" do
+        github.repos.should_receive(:commits).with(user, repo).and_yield('web')
+        github.repos.commits(user, repo) { |param| 'web' }
+      end
+    end
+
+    context "resource not found" do
+      before do
+        stub_get("/repos/#{user}/#{repo}/commits").
+          to_return(:body => "", :status => [404, "Not Found"])
+      end
+
+      it "should return 404 with a message 'Not Found'" do
+        expect {
+          github.repos.commits user, repo
+        }.to raise_error(Github::ResourceNotFound)
+      end
+    end
+  end # commits
+
+  describe "commit" do
+    let(:sha) { '23432dfosfsufd' }
+
+    context "resource found" do
+      before do
+        stub_get("/repos/#{user}/#{repo}/commits/#{sha}").
+          to_return(:body => fixture('repos/commit.json'), :status => 200, :headers => {:content_type => "application/json; charset=utf-8"})
+      end
+
+      it "should fail to get resource without sha key" do
+        expect {
+          github.repos.commit(user, repo, nil)
+        }.to raise_error(ArgumentError)
+      end
+
+      it "should get the resource" do
+        github.repos.commit user, repo, sha
+        a_get("/repos/#{user}/#{repo}/commits/#{sha}").should have_been_made
+      end
+
+      it "should get commit information" do
+        commit = github.repos.commit user, repo, sha
+        commit.commit.author.name.should == 'Monalisa Octocat'
+      end
+
+      it "should return mash" do
+        commit = github.repos.commit user, repo, sha
+        commit.should be_a Hashie::Mash
+      end
+
+    end
+
+    context "resource not found" do
+      before do
+        stub_get("/repos/#{user}/#{repo}/commits/#{sha}").
+          to_return(:body => '', :status => 404, :headers => {:content_type => "application/json; charset=utf-8"})
+      end
+
+      it "should fail to retrive resource" do
+        expect {
+          github.repos.commit user, repo, sha
+        }.to raise_error(Github::ResourceNotFound)
+      end
+    end
+  end # commit
+
+  describe "commit comments" do
+    context "resource found" do
+      before do
+        stub_get("/repos/#{user}/#{repo}/comments").
+          to_return(:body => fixture('repos/repo_comments.json'), :status => 200, :headers => {:content_type => "application/json; charset=utf-8"})
+      end
+
+      it "should fail to get resource without username" do
+        github.user, github.repo = nil, nil
+        expect { github.repos.repo_comments }.to raise_error(ArgumentError)
+      end
+
+      it "should get the resources" do
+        github.repos.repo_comments user, repo
+        a_get("/repos/#{user}/#{repo}/comments").should have_been_made
+      end
+
+      it "should return array of resources" do
+        repo_comments = github.repos.repo_comments user, repo
+        repo_comments.should be_an Array
+        repo_comments.should have(1).items
+      end
+
+      it "should be a mash type" do
+        repo_comments = github.repos.repo_comments user, repo
+        repo_comments.first.should be_a Hashie::Mash
+      end
+
+      it "should get commit comment information" do
+        repo_comments = github.repos.repo_comments user, repo
+        repo_comments.first.user.login.should == 'octocat'
+      end
+
+      it "should yield to a block" do
+        github.repos.should_receive(:repo_comments).with(user, repo).and_yield('web')
+        github.repos.repo_comments(user, repo) { |param| 'web' }
+      end
+    end
+
+    context "resource not found" do
+      before do
+        stub_get("/repos/#{user}/#{repo}/comments").
+          to_return(:body => "", :status => [404, "Not Found"])
+      end
+
+      it "should return 404 with a message 'Not Found'" do
+        expect {
+          github.repos.repo_comments user, repo
+        }.to raise_error(Github::ResourceNotFound)
+      end
+    end
+  end # repo comments
+
+  describe "commit_comments" do
+    let(:sha) { '23432dfosfsufd' }
+
+    context "resource found" do
+      before do
+        stub_get("/repos/#{user}/#{repo}/commits/#{sha}/comments").
+          to_return(:body => fixture('repos/commit_comments.json'), :status => 200, :headers => {:content_type => "application/json; charset=utf-8"})
+      end
+
+      it "should fail to get resource without sha key" do
+        expect {
+          github.repos.commit_comments(user, repo, nil)
+        }.to raise_error(ArgumentError)
+      end
+
+      it "should get the resource" do
+        github.repos.commit_comments user, repo, sha
+        a_get("/repos/#{user}/#{repo}/commits/#{sha}/comments").should have_been_made
+      end
+
+      it "should return array of resources" do
+        commit_comments = github.repos.commit_comments user, repo, sha
+        commit_comments.should be_an Array
+        commit_comments.should have(1).items
+      end
+
+      it "should be a mash type" do
+        commit_comments = github.repos.commit_comments user, repo, sha
+        commit_comments.first.should be_a Hashie::Mash
+      end
+
+      it "should get commit comment information" do
+        commit_comments = github.repos.commit_comments user, repo, sha
+        commit_comments.first.user.login.should == 'octocat'
+      end
+
+      it "should yield to a block" do
+        github.repos.should_receive(:commit_comments).with(user, repo, sha).and_yield('web')
+        github.repos.commit_comments(user, repo, sha) { |param| 'web' }
+      end
+    end
+
+    context "resource not found" do
+      before do
+        stub_get("/repos/#{user}/#{repo}/commits/#{sha}/comments").
+          to_return(:body => '', :status => 404, :headers => {:content_type => "application/json; charset=utf-8"})
+      end
+
+      it "should fail to retrive resource" do
+        expect {
+          github.repos.commit_comments user, repo, sha
+        }.to raise_error(Github::ResourceNotFound)
+      end
+    end
+  end # commit_comments
+
+  describe "commit_comment" do
+    let(:comment_id) { 1 }
+
+    context "resource found" do
+      before do
+        stub_get("/repos/#{user}/#{repo}/comments/#{comment_id}").
+          to_return(:body => fixture('repos/commit_comment.json'), :status => 200, :headers => {:content_type => "application/json; charset=utf-8"})
+      end
+
+      it "should fail to get resource without comment id" do
+        expect {
+          github.repos.commit_comment user, repo, nil
+        }.to raise_error(ArgumentError)
+      end
+
+      it "should get the resource" do
+        github.repos.commit_comment user, repo, comment_id
+        a_get("/repos/#{user}/#{repo}/comments/#{comment_id}").should have_been_made
+      end
+
+      it "should get commit comment information" do
+        commit_comment = github.repos.commit_comment user, repo, comment_id
+        commit_comment.user.login.should == 'octocat'
+      end
+
+      it "should return mash" do
+        commit_comment = github.repos.commit_comment user, repo, comment_id
+        commit_comment.should be_a Hashie::Mash
+      end
+
+    end
+
+    context "resource not found" do
+      before do
+        stub_get("/repos/#{user}/#{repo}/comments/#{comment_id}").
+          to_return(:body => '', :status => 404, :headers => {:content_type => "application/json; charset=utf-8"})
+      end
+
+      it "should fail to retrive resource" do
+        expect {
+          github.repos.commit_comment user, repo, comment_id
+        }.to raise_error(Github::ResourceNotFound)
+      end
+    end
+  end # commit_comment
+
+  describe "create_comment" do
+    let(:sha) { '23432dfosfsufd' }
+    let(:inputs) { {'body'=> 'web', :commit_id => 1, :line => 1, :path => 'file1.txt', :position => 4 } }
+
+    context "resouce created" do
+      before do
+        stub_post("/repos/#{user}/#{repo}/commits/#{sha}/comments").with(inputs).
+          to_return(:body => fixture('repos/commit_comment.json'), :status => 201, :headers => {:content_type => "application/json; charset=utf-8"})
+
+      end
+
+      it "should fail to create resource if 'body' input is missing" do
+        expect {
+          github.repos.create_comment user, repo, sha, inputs.except('body')
+        }.to raise_error(ArgumentError)
+      end
+
+      it "should fail to create resource if 'commit_id' input is missing" do
+        expect {
+          github.repos.create_comment user, repo, sha, inputs.except(:commit_id)
+        }.to raise_error(ArgumentError)
+      end
+
+      it "should fail to create resource if 'line' input is missing" do
+        expect {
+          github.repos.create_comment user, repo, sha, inputs.except(:line)
+        }.to raise_error(ArgumentError)
+      end
+
+      it "should fail to create resource if 'path' input is missing" do
+        expect {
+          github.repos.create_comment user, repo, sha, inputs.except(:path)
+        }.to raise_error(ArgumentError)
+      end
+
+      it "should fail to create resource if 'position' input is missing" do
+        expect {
+          github.repos.create_comment user, repo, sha, inputs.except(:position)
+        }.to raise_error(ArgumentError)
+      end
+
+      it "should create resource successfully" do
+        github.repos.create_comment user, repo, sha, inputs
+        a_post("/repos/#{user}/#{repo}/commits/#{sha}/comments").with(inputs).should have_been_made
+      end
+
+      it "should return the resource" do
+        comment = github.repos.create_comment user, repo, sha, inputs
+        comment.should be_a Hashie::Mash
+      end
+
+      it "should get the commit comment information" do
+        comment = github.repos.create_comment user, repo, sha, inputs
+        comment.user.login.should == 'octocat'
+      end
+    end
+
+    context "failed to create resource" do
+      before do
+        stub_post("/repos/#{user}/#{repo}/commits/#{sha}/comments").with(inputs).
+          to_return(:body => fixture('repos/commit_comment.json'), :status => 404, :headers => {:content_type => "application/json; charset=utf-8"})
+
+      end
+
+      it "should fail to retrieve resource" do
+        expect {
+          github.repos.create_comment user, repo, sha, inputs
+        }.to raise_error(Github::ResourceNotFound)
+      end
+    end
+  end # create_comment
+
+  describe "delete_comment" do
+    let(:comment_id) { 1 }
+
+    context "resource deleted successfully" do
+      before do
+        stub_delete("/repos/#{user}/#{repo}/comments/#{comment_id}").
+          to_return(:body => '', :status => 204, :headers => { :content_type => "application/json; charset=utf-8"})
+      end
+
+      it "should fail to delete without 'user/repo' parameters" do
+        github.user, github.repo = nil, nil
+        expect { github.repos.delete_comment }.to raise_error(ArgumentError)
+      end
+
+      it "should fail to delete resource without 'comment_id'" do
+        expect {
+          github.repos.delete_comment user, repo, nil
+        }.to raise_error(ArgumentError)
+      end
+
+      it "should delete the resource" do
+        github.repos.delete_comment user, repo, comment_id
+        a_delete("/repos/#{user}/#{repo}/comments/#{comment_id}").should have_been_made
+      end
+    end
+
+    context "failed to delete resource" do
+      before do
+        stub_delete("/repos/#{user}/#{repo}/comments/#{comment_id}").
+          to_return(:body => '', :status => 404, :headers => { :content_type => "application/json; charset=utf-8"})
+
+      end
+
+      it "should fail to find resource" do
+        expect {
+          github.repos.delete_comment user, repo, comment_id
+        }.to raise_error(Github::ResourceNotFound)
+      end
+    end
+  end # delete_comment
+
+  describe "update_comment" do
+    let(:comment_id) { 1 }
+    let(:inputs) { {'body'=> 'web'} }
+
+    context "resouce created" do
+      before do
+        stub_patch("/repos/#{user}/#{repo}/comments/#{comment_id}").with(inputs).
+          to_return(:body => fixture('repos/commit_comment.json'), :status => 200, :headers => {:content_type => "application/json; charset=utf-8"})
+
+      end
+
+      it "should fail to create resource if 'body' input is missing" do
+        expect {
+          github.repos.update_comment user, repo, comment_id, inputs.except('body')
+        }.to raise_error(ArgumentError)
+      end
+
+      it "should create resource successfully" do
+        github.repos.update_comment user, repo, comment_id, inputs
+        a_patch("/repos/#{user}/#{repo}/comments/#{comment_id}").with(inputs).should have_been_made
+      end
+
+      it "should return the resource" do
+        comment = github.repos.update_comment user, repo, comment_id, inputs
+        comment.should be_a Hashie::Mash
+      end
+
+      it "should get the commit comment information" do
+        comment = github.repos.update_comment user, repo, comment_id, inputs
+        comment.user.login.should == 'octocat'
+      end
+    end
+
+    context "failed to update resource" do
+      before do
+        stub_patch("/repos/#{user}/#{repo}/comments/#{comment_id}").with(inputs).
+          to_return(:body => fixture('repos/commit_comment.json'), :status => 404, :headers => {:content_type => "application/json; charset=utf-8"})
+
+      end
+
+      it "should fail to retrieve resource" do
+        expect {
+          github.repos.update_comment user, repo, comment_id, inputs
+        }.to raise_error(Github::ResourceNotFound)
+      end
+    end
+  end # update_comment
+
+end # Github::Repos::Commits