github_api 0.1.1 → 0.1.2

data/spec/fixtures/repos/tags.json

@@ -0,0 +1,11 @@
+[
+  {
+    "name": "v0.1",
+    "commit": {
+      "sha": "c5b97d5ae6c19d5c5df71a34c7fbeeda2479ccbc",
+      "url": "https://api.github.com/octocat/Hello-World/commits/c5b97d5ae6c19d5c5df71a34c7fbeeda2479ccbc"
+    },
+    "zipball_url": "https://github.com/octocat/Hello-World/zipball/v0.1",
+    "tarball_url": "https://github.com/octocat/Hello-World/tarball/v0.1"
+  }
+]

data/spec/fixtures/repos/teams.json

@@ -0,0 +1,7 @@
+[
+  {
+    "url": "https://api.github.com/teams/1",
+    "name": "Owners",
+    "id": 1
+  }
+]

data/spec/github/client_spec.rb

@@ -1,6 +1,50 @@
 require 'spec_helper'
 
 describe Github::Client do
-  
-  it "should have some tests"
+
+  let(:github) { Github.new }
+
+  it "should return Github::Gists instance" do
+    github.gists.should be_a Github::Gists
+  end
+
+  it "should return Github::GitData instance" do
+    github.git_data.should be_a Github::GitData
+  end
+
+  it "should return Github::Issues instance" do
+    github.issues.should be_a Github::Issues
+  end
+
+  it "should return Github::Orgs instance" do
+    github.orgs.should be_a Github::Orgs
+  end
+
+  it "should return Github::PullRequests instance" do
+    github.pull_requests.should be_a Github::PullRequests
+  end
+
+  it "should return Github::Repos instance" do
+    github.repos.should be_a Github::Repos
+  end
+
+  it "should return Github::Users instance" do
+    github.users.should be_a Github::Users
+  end
+
+  it "should respond to repos" do
+    github.should respond_to :repos
+  end
+
+  it "should respond to repositories" do
+    github.should respond_to :repositories
+  end
+
+  it "should respond to git_data" do
+    github.should respond_to :git_data
+  end
+
+  it "should respond to git" do
+    github.should respond_to :git
+  end
 end

data/spec/github/core_ext/hash_spec.rb

@@ -0,0 +1,34 @@
+require 'spec_helper'
+
+describe Hash do
+
+  before do
+    Github.new
+    @nested_hash = { 'a' => { 'b' => {'c' => 1 } } }
+    @symbols = { :a => { :b => { :c => 1 } } }
+  end
+
+  it "should respond to except" do
+    @nested_hash.should respond_to :except
+  end
+
+  it "should respond to except!" do
+    @nested_hash.should respond_to :except!
+  end
+
+  it "should respond to symbolize_keys!" do
+    @nested_hash.should respond_to :symbolize_keys!
+  end
+
+  it "should respond to symbolize_keys" do
+    @nested_hash.should respond_to :symbolize_keys
+  end
+
+  it "should remove key from the hash" do
+    @nested_hash.except('a').should be_empty
+  end
+
+  it "should convert nested keys to symbols" do
+    @nested_hash.symbolize_keys!.should == @symbols
+  end
+end

data/spec/github/repos/hooks_spec.rb

@@ -1,5 +1,307 @@
 require 'spec_helper'
 
 describe Github::Repos::Hooks do
-  pending
-end
+
+  let(:github) { Github.new }
+  let(:user)   { 'peter-murach' }
+  let(:repo)   { 'github' }
+
+  describe "hooks" do
+    context "resource found" do
+      before do
+        stub_get("/repos/#{user}/#{repo}/hooks").
+          to_return(:body => fixture('repos/hooks.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.hooks }.to raise_error(ArgumentError)
+      end
+
+      it "should get the resources" do
+        github.repos.hooks(user, repo)
+        a_get("/repos/#{user}/#{repo}/hooks").should have_been_made
+      end
+
+      it "should return array of resources" do
+        hooks = github.repos.hooks user, repo
+        hooks.should be_an Array
+        hooks.should have(1).items
+      end
+
+      it "should be a mash type" do
+        hooks = github.repos.hooks user, repo
+        hooks.first.should be_a Hashie::Mash
+      end
+
+      it "should get hook information" do
+        hooks = github.repos.hooks(user, repo)
+        hooks.first.name.should == 'web'
+      end
+
+      it "should yield to a block" do
+        github.repos.should_receive(:hooks).with(user, repo).and_yield('web')
+        github.repos.hooks(user, repo) { |param| 'web' }
+      end
+    end
+
+    context "resource not found" do
+      before do
+        stub_get("/repos/#{user}/#{repo}/hooks").
+          to_return(:body => "", :status => [404, "Not Found"])
+      end
+
+      it "should return 404 with a message 'Not Found'" do
+        expect {
+          github.repos.hooks user, repo
+        }.to raise_error(Github::ResourceNotFound)
+      end
+    end
+  end # hooks
+
+  describe "hook" do
+    let(:hook_id) { 1 }
+
+    context "resource found" do
+      before do
+        stub_get("/repos/#{user}/#{repo}/hooks/#{hook_id}").
+          to_return(:body => fixture('repos/hook.json'), :status => 200, :headers => {:content_type => "application/json; charset=utf-8"})
+      end
+
+      it "should fail to get resource without hook id" do
+        expect { github.repos.hook(user, repo, nil)}.to raise_error(ArgumentError)
+      end
+
+      it "should get the resource" do
+        github.repos.hook user, repo, hook_id
+        a_get("/repos/#{user}/#{repo}/hooks/#{hook_id}").should have_been_made
+      end
+
+      it "should get hook information" do
+        hook = github.repos.hook user, repo, hook_id
+        hook.id.should == hook_id
+        hook.name.should == 'web'
+      end
+
+      it "should return mash" do
+        hook = github.repos.hook user, repo, hook_id
+        hook.should be_a Hashie::Mash
+      end
+
+    end
+
+    context "resource not found" do
+      before do
+        stub_get("/repos/#{user}/#{repo}/hooks/#{hook_id}").
+          to_return(:body => fixture('repos/hook.json'), :status => 404, :headers => {:content_type => "application/json; charset=utf-8"})
+      end
+
+      it "should fail to retrive resource" do
+        expect {
+          github.repos.hook(user, repo, hook_id)
+        }.to raise_error(Github::ResourceNotFound)
+      end
+    end
+  end # hook
+
+  describe "create_hook" do
+    let(:inputs) { {:name => 'web', :config => {:url => "http://something.com/webhook"}, :active => true}}
+
+    context "resouce created" do
+      before do
+        stub_post("/repos/#{user}/#{repo}/hooks").with(inputs).
+          to_return(:body => fixture('repos/hook.json'), :status => 201, :headers => {:content_type => "application/json; charset=utf-8"})
+
+      end
+
+      it "should fail to create resource if 'name' input is missing" do
+        expect {
+          github.repos.create_hook(user, repo, inputs.except(:name) )
+        }.to raise_error(ArgumentError)
+      end
+
+      it "should failt to create resource if 'config' input is missing" do
+        expect {
+          github.repos.create_hook(user, repo, inputs.except(:config) )
+        }.to raise_error(ArgumentError)
+      end
+
+      it "should create resource successfully" do
+        github.repos.create_hook(user, repo, inputs)
+        a_post("/repos/#{user}/#{repo}/hooks").with(inputs).should have_been_made
+      end
+
+      it "should return the resource" do
+        hook = github.repos.create_hook user, repo, inputs
+        hook.should be_a Hashie::Mash
+      end
+
+      it "should get the hook information" do
+        hook = github.repos.create_hook(user, repo, inputs)
+        hook.name.should == 'web'
+      end
+    end
+
+    context "failed to create resource" do
+      before do
+        stub_post("/repos/#{user}/#{repo}/hooks").with(inputs).
+          to_return(:body => fixture('repos/hook.json'), :status => 404, :headers => {:content_type => "application/json; charset=utf-8"})
+
+      end
+
+      it "should faile to retrieve resource" do
+        expect {
+          github.repos.create_hook(user, repo, inputs)
+        }.to raise_error(Github::ResourceNotFound)
+      end
+    end
+  end # create_hook
+
+  describe "edit_hook" do
+    let(:hook_id) { 1 }
+    let(:inputs) { {:name => 'web', :config => {:url => "http://something.com/webhook"}, :active => true}}
+
+    context "resource edited successfully" do
+      before do
+        stub_patch("/repos/#{user}/#{repo}/hooks/#{hook_id}").with(inputs).
+          to_return(:body => fixture("repos/hook.json"), :status => 200, :headers => { :content_type => "application/json; charset=utf-8"})
+      end
+
+      it "should fail to edit without 'user/repo' parameters" do
+        github.user, github.repo = nil, nil
+        expect { github.repos.edit_hook }.to raise_error(ArgumentError)
+      end
+
+      it "should fail to edit resource without 'name' parameter" do
+        expect{
+          github.repos.edit_hook user, repo, inputs.except(:name)
+        }.to raise_error(ArgumentError)
+      end
+
+      it "should fail to edit resource without 'hook_id'" do
+        expect {
+          github.repos.edit_hook user, repo
+        }.to raise_error(ArgumentError)
+      end
+
+      it "should fail to edit resource without 'config' parameter" do
+        expect {
+          github.repos.edit_hook user, repo, hook_id, inputs.except(:config)
+        }.to raise_error(ArgumentError)
+      end
+
+      it "should edit the resource" do
+        github.repos.edit_hook user, repo, hook_id, inputs
+        a_patch("/repos/#{user}/#{repo}/hooks/#{hook_id}").with(inputs).should have_been_made
+      end
+
+      it "should return resource" do
+        hook = github.repos.edit_hook user, repo, hook_id, inputs
+        hook.should be_a Hashie::Mash
+      end
+
+      it "should be able to retrieve information" do
+        hook = github.repos.edit_hook user, repo, hook_id, inputs
+        hook.name.should == 'web'
+      end
+
+    end
+
+    context "failed to edit resource" do
+      before do
+        stub_patch("/repos/#{user}/#{repo}/hooks/#{hook_id}").with(inputs).
+          to_return(:body => fixture("repos/hook.json"), :status => 404, :headers => { :content_type => "application/json; charset=utf-8"})
+
+      end
+
+      it "should fail to find resource" do
+        expect {
+          github.repos.edit_hook user, repo, hook_id, inputs
+        }.to raise_error(Github::ResourceNotFound)
+      end
+    end
+  end # edit_hook
+
+  describe "delete_key" do
+    let(:hook_id) { 1 }
+
+    context "resource edited successfully" do
+      before do
+        stub_delete("/repos/#{user}/#{repo}/hooks/#{hook_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_hook }.to raise_error(ArgumentError)
+      end
+
+      it "should fail to delete resource without 'hook_id'" do
+        expect {
+          github.repos.edit_hook user, repo
+        }.to raise_error(ArgumentError)
+      end
+
+      it "should delete the resource" do
+        github.repos.delete_hook user, repo, hook_id
+        a_delete("/repos/#{user}/#{repo}/hooks/#{hook_id}").should have_been_made
+      end
+    end
+
+    context "failed to edit resource" do
+      before do
+        stub_delete("/repos/#{user}/#{repo}/hooks/#{hook_id}").
+          to_return(:body => fixture("repos/hook.json"), :status => 404, :headers => { :content_type => "application/json; charset=utf-8"})
+
+      end
+
+      it "should fail to find resource" do
+        expect {
+          github.repos.delete_hook user, repo, hook_id
+        }.to raise_error(Github::ResourceNotFound)
+      end
+    end
+  end # delete_hook
+
+  describe "test_hook" do
+    let(:hook_id) { 1 }
+
+    context "resource edited successfully" do
+      before do
+        stub_post("/repos/#{user}/#{repo}/hooks/#{hook_id}/test").
+          to_return(:body => '', :status => 204, :headers => { :content_type => "application/json; charset=utf-8"})
+      end
+
+      it "should fail to test without 'user/repo' parameters" do
+        github.user, github.repo = nil, nil
+        expect { github.repos.test_hook }.to raise_error(ArgumentError)
+      end
+
+      it "should fail to test resource without 'hook_id'" do
+        expect {
+          github.repos.test_hook user, repo
+        }.to raise_error(ArgumentError)
+      end
+
+      it "should trigger test for the resource" do
+        github.repos.test_hook user, repo, hook_id
+        a_post("/repos/#{user}/#{repo}/hooks/#{hook_id}/test").should have_been_made
+      end
+    end
+
+    context "failed to test resource" do
+      before do
+        stub_post("/repos/#{user}/#{repo}/hooks/#{hook_id}/test").
+          to_return(:body => '', :status => 404, :headers => { :content_type => "application/json; charset=utf-8"})
+
+      end
+
+      it "should fail to find resource" do
+        expect {
+          github.repos.test_hook user, repo, hook_id
+        }.to raise_error(Github::ResourceNotFound)
+      end
+    end
+  end # test_hook
+
+end # Github::Repos::Hooks

data/spec/github/repos/keys_spec.rb

@@ -96,7 +96,7 @@ describe Github::Repos::Keys do
 
     context "resource created" do
       before do
-        stub_post("/repos/#{user}/#{repo}/keys").
+        stub_post("/repos/#{user}/#{repo}/keys").with(inputs).
           to_return(:body => fixture("repos/key.json"), :status => 201)
       end
 
@@ -114,7 +114,7 @@ describe Github::Repos::Keys do
 
       it "should create the resource" do
         github.repos.create_key(user, repo, inputs)
-        a_post("/repos/#{user}/#{repo}/keys").should have_been_made
+        a_post("/repos/#{user}/#{repo}/keys").with(inputs).should have_been_made
       end
 
       it "should get the key information back" do
@@ -180,7 +180,18 @@ describe Github::Repos::Keys do
     context "resource found successfully" do
       before do
         stub_delete("/repos/#{user}/#{repo}/keys/#{key_id}").
-          to_return(:body => "", :status => 204)
+          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_key }.to raise_error(ArgumentError)
+      end
+
+      it "should fail to delete resource without key id" do
+        expect {
+          github.repos.delete_key user, repo, nil
+        }.to raise_error(ArgumentError)
       end
 
       it "should delete the resource" do