github_api 0.7.1 → 0.7.2

data/spec/github/repos/comments/list_spec.rb

@@ -0,0 +1,120 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::Repos::Comments, '#list' do
+  let(:user) { 'peter-murach' }
+  let(:repo) { 'github' }
+
+  after { reset_authentication_for subject }
+
+  context 'without sha' do
+    let(:request_path) { "/repos/#{user}/#{repo}/comments" }
+
+    before {
+      stub_get(request_path).to_return(:body => body, :status => status,
+        :headers => {:content_type => "application/json; charset=utf-8"})
+    }
+
+    context "resource found" do
+      let(:body)   { fixture('repos/repo_comments.json') }
+      let(:status) { 200 }
+
+      it { should respond_to(:all) }
+
+      it "should fail to get resource without username" do
+        expect { subject.list }.to raise_error(ArgumentError)
+      end
+
+      it "should get the resources" do
+        subject.list user, repo
+        a_get(request_path).should have_been_made
+      end
+
+      it "should return array of resources" do
+        repo_comments = subject.list user, repo
+        repo_comments.should be_an Array
+        repo_comments.should have(1).items
+      end
+
+      it "should be a mash type" do
+        repo_comments = subject.list user, repo
+        repo_comments.first.should be_a Hashie::Mash
+      end
+
+      it "should get commit comment information" do
+        repo_comments = subject.list user, repo
+        repo_comments.first.user.login.should == 'octocat'
+      end
+
+      it "should yield to a block" do
+        subject.should_receive(:list).with(user, repo).and_yield('web')
+        subject.list(user, repo) { |param| 'web' }
+      end
+    end
+
+    context "resource not found" do
+      let(:body) { '' }
+      let(:status) { [404, "Not Found"] }
+
+      it "should return 404 with a message 'Not Found'" do
+        expect {
+          subject.list user, repo
+        }.to raise_error(Github::Error::NotFound)
+      end
+    end
+  end # without sha
+
+  context 'with sha' do
+    let(:sha) { '23432dfosfsufd' }
+    let(:request_path) { "/repos/#{user}/#{repo}/commits/#{sha}/comments" }
+
+    before {
+      stub_get(request_path).to_return(:body => body, :status => status,
+        :headers => {:content_type => "application/json; charset=utf-8"})
+    }
+
+    context "resource found" do
+      let(:body) { fixture('repos/commit_comments.json') }
+      let(:status) { 200 }
+
+      it "should get the resource" do
+        subject.list user, repo, :sha => sha
+        a_get(request_path).should have_been_made
+      end
+
+      it "should return array of resources" do
+        commit_comments = subject.list user, repo, :sha => sha
+        commit_comments.should be_an Array
+        commit_comments.should have(1).items
+      end
+
+      it "should be a mash type" do
+        commit_comments = subject.list user, repo, :sha => sha
+        commit_comments.first.should be_a Hashie::Mash
+      end
+
+      it "should get commit comment information" do
+        commit_comments = subject.list user, repo, :sha => sha
+        commit_comments.first.user.login.should == 'octocat'
+      end
+
+      it "should yield to a block" do
+        subject.should_receive(:list).with(user, repo, :sha=>sha).and_yield('web')
+        subject.list(user, repo, :sha => sha) { |param| 'web' }
+      end
+    end
+
+    context "resource not found" do
+      let(:body) { '' }
+      let(:status) { 404 }
+
+      it "should fail to retrive resource" do
+        expect {
+          subject.list user, repo, :sha => sha
+        }.to raise_error(Github::Error::NotFound)
+      end
+    end
+  end # with sha
+
+end # list

data/spec/github/repos/comments/update_spec.rb

@@ -0,0 +1,54 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::Repos::Comments, '#delete' do
+  let(:user) { 'peter-murach' }
+  let(:repo) { 'github' }
+  let(:comment_id) { 1 }
+  let(:inputs) { {'body'=> 'web'} }
+  let(:request_path) { "/repos/#{user}/#{repo}/comments/#{comment_id}" }
+
+  before {
+    stub_patch(request_path).with(inputs).
+      to_return(:body => body,:status => status,
+        :headers => {:content_type => "application/json; charset=utf-8"})
+  }
+
+  context "resouce created" do
+    let(:body)   { fixture('repos/commit_comment.json') }
+    let(:status) { 200 }
+
+    it "should fail to create resource if 'body' input is missing" do
+      expect {
+        subject.update user, repo, comment_id, inputs.except('body')
+      }.to raise_error(Github::Error::RequiredParams)
+    end
+
+    it "should create resource successfully" do
+      subject.update user, repo, comment_id, inputs
+      a_patch(request_path).with(inputs).should have_been_made
+    end
+
+    it "should return the resource" do
+      comment = subject.update user, repo, comment_id, inputs
+      comment.should be_a Hashie::Mash
+    end
+
+    it "should get the commit comment information" do
+      comment = subject.update user, repo, comment_id, inputs
+      comment.user.login.should == 'octocat'
+    end
+  end
+
+  context "failed to update resource" do
+    let(:body)   { '' }
+    let(:status) { 404 }
+
+    it "should fail to retrieve resource" do
+      expect {
+        subject.update user, repo, comment_id, inputs
+      }.to raise_error(Github::Error::NotFound)
+    end
+  end
+end # update_comment

data/spec/github/repos/commits/compare_spec.rb

@@ -0,0 +1,37 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::Repos::Commits, '#compare' do
+  let(:user) { 'peter-murach' }
+  let(:repo) { 'github' }
+  let(:base) { 'master' }
+  let(:head) { 'topic' }
+  let(:request_path) { "/repos/#{user}/#{repo}/compare/#{base}...#{head}" }
+
+  before {
+    stub_get(request_path).
+      to_return(:body => fixture('repos/commit_comparison.json'),
+        :status => 200,
+        :headers => {:content_type => "application/json; charset=utf-8"})
+  }
+
+  after { reset_authentication_for(subject) }
+
+  it "should fail to get resource without base" do
+    expect {
+      subject.compare user, repo, nil, head
+    }.to raise_error(ArgumentError)
+  end
+
+  it "should compare successfully" do
+    subject.compare user, repo, base, head
+    a_get(request_path).should have_been_made
+  end
+
+  it "should get comparison information" do
+    commit = subject.compare user, repo, base, head
+    commit.base_commit.commit.author.name.should == 'Monalisa Octocat'
+  end
+
+end # compare

data/spec/github/repos/commits/get_spec.rb

@@ -0,0 +1,56 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::Repos::Commits, '#get' do
+  let(:user) { 'peter-murach' }
+  let(:repo) { 'github' }
+  let(:sha) { '23432dfosfsufd' }
+  let(:request_path) { "/repos/#{user}/#{repo}/commits/#{sha}" }
+
+  before {
+    stub_get(request_path).to_return(:body => body, :status => status,
+      :headers => {:content_type => "application/json; charset=utf-8"})
+  }
+
+  after { reset_authentication_for(subject) }
+
+  context "resource found" do
+    let(:body)   { fixture('repos/commit.json') }
+    let(:status) { 200 }
+
+    it { should respond_to :find }
+
+    it "should fail to get resource without sha key" do
+      expect {
+        subject.get user, repo, nil
+      }.to raise_error(ArgumentError)
+    end
+
+    it "should get the resource" do
+      subject.get user, repo, sha
+      a_get(request_path).should have_been_made
+    end
+
+    it "should get commit information" do
+      commit = subject.get user, repo, sha
+      commit.commit.author.name.should == 'Monalisa Octocat'
+    end
+
+    it "should return mash" do
+      commit = subject.get user, repo, sha
+      commit.should be_a Hashie::Mash
+    end
+  end
+
+  context "resource not found" do
+    let(:body)   { '' }
+    let(:status) { 404 }
+
+    it "should fail to retrive resource" do
+      expect {
+        subject.get user, repo, sha
+      }.to raise_error(Github::Error::NotFound)
+    end
+  end
+end # get

data/spec/github/repos/commits/list_spec.rb

@@ -0,0 +1,63 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::Repos::Commits, '#list' do
+  let(:user) { 'peter-murach' }
+  let(:repo) { 'github' }
+  let(:request_path) { "/repos/#{user}/#{repo}/commits" }
+
+  before {
+    stub_get(request_path).to_return(:body => body, :status => status,
+      :headers => {:content_type => "application/json; charset=utf-8"})
+  }
+
+  after { reset_authentication_for(subject) }
+
+  context "resource found" do
+    let(:body)   { fixture('repos/commits.json') }
+    let(:status) { 200 }
+
+    it { should respond_to :all }
+    it "should fail to get resource without username" do
+      expect { subject.list }.to raise_error(ArgumentError)
+    end
+
+    it "should get the resources" do
+      subject.list user, repo
+      a_get(request_path).should have_been_made
+    end
+
+    it "should return array of resources" do
+      commits = subject.list user, repo
+      commits.should be_an Array
+      commits.should have(1).items
+    end
+
+    it "should be a mash type" do
+      commits = subject.list user, repo
+      commits.first.should be_a Hashie::Mash
+    end
+
+    it "should get commit information" do
+      commits = subject.list user, repo
+      commits.first.author.name.should == 'Scott Chacon'
+    end
+
+    it "should yield to a block" do
+      subject.should_receive(:list).with(user, repo).and_yield('web')
+      subject.list(user, repo) { |param| 'web' }
+    end
+  end
+
+  context "resource not found" do
+    let(:body) { '' }
+    let(:status) { [404, "Not Found"] }
+
+    it "should return 404 with a message 'Not Found'" do
+      expect {
+        subject.list user, repo
+      }.to raise_error(Github::Error::NotFound)
+    end
+  end
+end # list

data/spec/github/repos/downloads/create_spec.rb

@@ -0,0 +1,66 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::Repos::Downloads, '#create' do
+  let(:user) { 'peter-murach' }
+  let(:repo) { 'github' }
+  let(:request_path) { "/repos/#{user}/#{repo}/downloads" }
+  let(:inputs) { {
+    :name => 'new_file.jpg',
+    :size => 114034,
+    :description => "Latest release",
+    :content_type => 'text/plain'}
+  }
+
+  after { reset_authentication_for(subject) }
+
+  before {
+    stub_post(request_path).with(inputs).
+      to_return(:body => body, :status => status,
+        :headers => {:content_type => "application/json; charset=utf-8"})
+  }
+
+  context "resouce created" do
+    let(:body)   { fixture('repos/download_s3.json') }
+    let(:status) { 201}
+
+    it "should fail to create resource if 'name' input is missing" do
+      expect {
+        subject.create user, repo, inputs.except(:name)
+      }.to raise_error(Github::Error::RequiredParams)
+    end
+
+    it "should failt to create resource if 'size' input is missing" do
+      expect {
+        subject.create user, repo, inputs.except(:size)
+      }.to raise_error(Github::Error::RequiredParams)
+    end
+
+    it "should create resource successfully" do
+      subject.create user, repo, inputs
+      a_post(request_path).with(inputs).should have_been_made
+    end
+
+    it "should return the resource" do
+      download = subject.create user, repo, inputs
+      download.should be_a Hashie::Mash
+    end
+
+    it "should get the download information" do
+      download = subject.create user, repo, inputs
+      download.name.should == 'new_file.jpg'
+    end
+  end
+
+  context "failed to create resource" do
+    let(:body)   { "" }
+    let(:status) { [404, "Not Found"] }
+
+    it "should faile to retrieve resource" do
+      expect {
+        subject.create user, repo, inputs
+      }.to raise_error(Github::Error::NotFound)
+    end
+  end
+end # create

data/spec/github/repos/downloads/delete_spec.rb

@@ -0,0 +1,48 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::Repos::Downloads, '#delete' do
+  let(:github) { Github.new }
+  let(:user) { 'peter-murach' }
+  let(:repo) { 'github' }
+  let(:download_id) { 1 }
+  let(:request_path) { "/repos/#{user}/#{repo}/downloads/#{download_id}" }
+
+  after { reset_authentication_for(github) }
+
+  before do
+    stub_delete(request_path).
+      to_return(:body => body, :status => status,
+        :headers => { :content_type => "application/json; charset=utf-8"})
+  end
+
+  context "resource deleted successfully" do
+    let(:body)   { "" }
+    let(:status) { 204 }
+
+    it "should fail to delete without 'user/repo' parameters" do
+      expect { subject.delete }.to raise_error(ArgumentError)
+    end
+
+    it "should fail to delete resource without 'download_id'" do
+      expect { subject.delete user, repo }.to raise_error(ArgumentError)
+    end
+
+    it "should delete the resource" do
+      github.repos.downloads.delete user, repo, download_id
+      a_delete(request_path).should have_been_made
+    end
+  end
+
+  context "failed to delete resource" do
+    let(:body)   { "" }
+    let(:status) { [404, "Not Found"] }
+
+    it "should fail to find resource" do
+      expect {
+        subject.delete user, repo, download_id
+      }.to raise_error(Github::Error::NotFound)
+    end
+  end
+end # delete