github_api 0.7.0 → 0.7.1

data/spec/github/repos/contributors_spec.rb

@@ -0,0 +1,62 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::Repos, '#contributors' do
+  let(:user) { 'peter-murach' }
+  let(:repo) { 'github' }
+
+  after { reset_authentication_for subject }
+
+  let(:request_path) { "/repos/#{user}/#{repo}/contributors" }
+
+  before do
+    stub_get(request_path).to_return(:body => body, :status => status,
+      :headers => {:content_type => "application/json; charset=utf-8"})
+  end
+
+  context "resource found" do
+    let(:body)   { fixture('repos/contributors.json') }
+    let(:status) { 200 }
+
+    it "should raise error when no user/repo parameters" do
+      expect { subject.contributors nil, repo }.to raise_error(ArgumentError)
+    end
+
+    it "should raise error when no repository" do
+      expect { subject.contributors user, nil }.to raise_error(ArgumentError)
+    end
+
+    it "should find resources" do
+      subject.contributors user, repo
+      a_get(request_path).should have_been_made
+    end
+
+    it "should return array of resources" do
+      contributors = subject.contributors user, repo
+      contributors.should be_an Array
+      contributors.should have(1).items
+    end
+
+    it "should get branch information" do
+      contributors = subject.contributors user, repo
+      contributors.first.login.should == 'octocat'
+    end
+
+    it "should yield to a block" do
+      subject.should_receive(:contributors).with(user, repo).and_yield('web')
+      subject.contributors(user, repo) { |param| 'web'}
+    end
+  end
+
+  context "resource not found" do
+    let(:body)   { '' }
+    let(:status) { 404 }
+
+    it "should fail to get resource" do
+      expect {
+        subject.contributors user, repo
+      }.to raise_error(Github::Error::NotFound)
+    end
+  end
+end # contributors

data/spec/github/repos/create_spec.rb

@@ -0,0 +1,77 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::Repos, '#create' do
+  let(:user) { 'peter-murach' }
+  let(:repo) { 'github' }
+  let(:inputs) { {
+    :name => 'web',
+    :description => "This is your first repo",
+    :homepage => "https://github.com",
+    :public => true,
+    :has_issues => true,
+    :has_wiki => true
+  } }
+
+  before {
+    subject.oauth_token = OAUTH_TOKEN
+    stub_post(request_path).with(inputs).
+      to_return(:body => body, :status => status,
+        :headers => {:content_type => "application/json; charset=utf-8"} )
+  }
+
+  after { reset_authentication_for subject }
+
+  context "resource created successfully" do
+    let(:body)   { fixture('repos/repo.json') }
+    let(:status) { 201 }
+
+    context "for the authenticated user" do
+      let(:request_path) { "/user/repos?access_token=#{OAUTH_TOKEN}" }
+
+      it "should faile to create resource if 'name' inputs is missing" do
+        expect {
+          subject.create inputs.except(:name)
+        }.to raise_error(Github::Error::RequiredParams)
+      end
+
+      it "should create resource" do
+        subject.create inputs
+        a_post(request_path).with(inputs).should have_been_made
+      end
+
+      it "should return the resource" do
+        repository = subject.create inputs
+        repository.name.should == 'Hello-World'
+      end
+
+      it "should return mash type" do
+        repository = subject.create inputs
+        repository.should be_a Hashie::Mash
+      end
+    end
+
+    context "for the authenticated user belonging to organization" do
+      let(:request_path) { "/orgs/#{org}/repos?access_token=#{OAUTH_TOKEN}" }
+      let(:org) { '37signals' }
+
+      it "should get the resource" do
+        subject.create inputs.merge(:org => org)
+        a_post(request_path).with(inputs).should have_been_made
+      end
+    end
+  end
+
+  context "failed to create" do
+    let(:request_path) { "/user/repos?access_token=#{OAUTH_TOKEN}" }
+    let(:body)   { '' }
+    let(:status) { 404 }
+
+    it "should faile to retrieve resource" do
+      expect {
+        subject.create inputs
+      }.to raise_error(Github::Error::NotFound)
+    end
+  end
+end # create

data/spec/github/repos/delete_spec.rb

@@ -0,0 +1,43 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::Repos do
+  let(:user) { 'peter-murach' }
+  let(:repo) { 'github' }
+  let(:request_path) { "/repos/#{user}/#{repo}" }
+  let(:body) { '' }
+  let(:status) { 204 }
+
+  before {
+    stub_delete(request_path).to_return(:body => body, :status => status,
+      :headers => { :content_type => "application/json; charset=utf-8"})
+  }
+
+  after { reset_authentication_for subject }
+
+  it { should respond_to :remove }
+
+  it "should delete the resource successfully" do
+    subject.delete user, repo
+    a_delete(request_path).should have_been_made
+  end
+
+  it "should fail to delete resource without 'user' parameter" do
+    expect { subject.delete nil, repo }.to raise_error(ArgumentError)
+  end
+
+  it "should fail to delete resource without 'repo' parameter" do
+    expect { subject.delete user, nil }.to raise_error(ArgumentError)
+  end
+
+  context 'failed to delete' do
+    let(:status) { 404 }
+
+    it "should fail to delete resource that is not found" do
+      expect {
+        subject.delete user, repo
+      }.to raise_error(Github::Error::NotFound)
+    end
+  end
+end # delete

data/spec/github/repos/downloads_spec.rb

@@ -8,20 +8,25 @@ describe Github::Repos::Downloads do
   let(:user) { 'peter-murach' }
   let(:repo) { 'github' }
 
-  after { github.user, github.repo, github.oauth_token = nil, nil, nil }
+  after { reset_authentication_for(github) }
 
   it { described_class::VALID_DOWNLOAD_PARAM_NAMES.should_not be_nil }
+
   it { described_class::REQUIRED_PARAMS.should_not be_nil }
 
   describe "#list" do
-    it { github.repos.downloads.should respond_to :list }
+
+    before do
+      stub_get("/repos/#{user}/#{repo}/downloads").
+        to_return(:body => body, :status => status,
+          :headers => {:content_type => "application/json; charset=utf-8"})
+    end
 
     context "resource found" do
-      before do
-        stub_get("/repos/#{user}/#{repo}/downloads").
-          to_return(:body => fixture('repos/downloads.json'), :status => 200,
-            :headers => {:content_type => "application/json; charset=utf-8"})
-      end
+      let(:body)   { fixture('repos/downloads.json') }
+      let(:status) { 200 }
+
+      it { should respond_to :list }
 
       it "should fail to get resource without username" do
         expect { github.repos.downloads.list }.to raise_error(ArgumentError)
@@ -56,10 +61,8 @@ describe Github::Repos::Downloads do
     end
 
     context "resource not found" do
-      before do
-        stub_get("/repos/#{user}/#{repo}/downloads").
-          to_return(:body => "", :status => [404, "Not Found"])
-      end
+      let(:body)   { "" }
+      let(:status) { [404, "Not Found"] }
 
       it "should return 404 with a message 'Not Found'" do
         expect {
@@ -72,14 +75,17 @@ describe Github::Repos::Downloads do
   describe "#get" do
     let(:download_id) { 1 }
 
-    it { github.repos.downloads.should respond_to :find }
+    before do
+      stub_get("/repos/#{user}/#{repo}/downloads/#{download_id}").
+        to_return(:body => body, :status => status,
+          :headers => {:content_type => "application/json; charset=utf-8"})
+    end
 
     context "resource found" do
-      before do
-        stub_get("/repos/#{user}/#{repo}/downloads/#{download_id}").
-          to_return(:body => fixture('repos/download.json'), :status => 200,
-            :headers => {:content_type => "application/json; charset=utf-8"})
-      end
+      let(:body)   { fixture('repos/download.json') }
+      let(:status) { 200 }
+
+      it { github.repos.downloads.should respond_to :find }
 
       it "should fail to get resource without download id" do
         expect {
@@ -105,10 +111,8 @@ describe Github::Repos::Downloads do
     end
 
     context "resource not found" do
-      before do
-        stub_get("/repos/#{user}/#{repo}/downloads/#{download_id}").
-          to_return(:body => fixture('repos/download.json'), :status => 404, :headers => {:content_type => "application/json; charset=utf-8"})
-      end
+      let(:body) { "" }
+      let(:status) { [404, "Not Found"] }
 
       it "should fail to retrive resource" do
         expect {
@@ -121,12 +125,15 @@ describe Github::Repos::Downloads do
   describe "#delete" do
     let(:download_id) { 1 }
 
-    context "resource edited successfully" do
-      before do
-        stub_delete("/repos/#{user}/#{repo}/downloads/#{download_id}").
-          to_return(:body => '', :status => 204,
-            :headers => { :content_type => "application/json; charset=utf-8"})
-      end
+    before do
+      stub_delete("/repos/#{user}/#{repo}/downloads/#{download_id}").
+        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 { github.repos.downloads.delete }.to raise_error(ArgumentError)
@@ -144,12 +151,9 @@ describe Github::Repos::Downloads do
       end
     end
 
-    context "failed to edit resource" do
-      before do
-        stub_delete("/repos/#{user}/#{repo}/downloads/#{download_id}").
-          to_return(:body => fixture("repos/download.json"), :status => 404,
-            :headers => { :content_type => "application/json; charset=utf-8"})
-      end
+    context "failed to delete resource" do
+      let(:body)   { "" }
+      let(:status) { [404, "Not Found"] }
 
       it "should fail to find resource" do
         expect {
@@ -162,12 +166,15 @@ describe Github::Repos::Downloads do
   describe "#create" do
     let(:inputs) { {:name => 'new_file.jpg', :size => 114034, :description => "Latest release", :content_type => 'text/plain'} }
 
-    context "resouce created" do
-      before do
-        stub_post("/repos/#{user}/#{repo}/downloads").with(inputs).
-          to_return(:body => fixture('repos/download_s3.json'), :status => 201, :headers => {:content_type => "application/json; charset=utf-8"})
+    before do
+      stub_post("/repos/#{user}/#{repo}/downloads").with(inputs).
+        to_return(:body => body, :status => status,
+          :headers => {:content_type => "application/json; charset=utf-8"})
+    end
 
-      end
+    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 {
@@ -198,11 +205,8 @@ describe Github::Repos::Downloads do
     end
 
     context "failed to create resource" do
-      before do
-        stub_post("/repos/#{user}/#{repo}/downloads").with(inputs).
-          to_return(:body => fixture('repos/download_s3.json'), :status => 404,
-            :headers => {:content_type => "application/json; charset=utf-8"})
-      end
+      let(:body)   { "" }
+      let(:status) { [404, "Not Found"] }
 
       it "should faile to retrieve resource" do
         expect {
@@ -215,7 +219,7 @@ describe Github::Repos::Downloads do
   describe 'upload' do
     let(:resource) { stub(:resource) }
     let(:filename) { 'filename' }
-    let(:res) { stub(:response, :body => 'success') }
+    let(:res)      { stub(:response, :body => 'success') }
     let(:uploader) { stub(:uploader, :send => res) }
 
     context 'resource uploaded' do
@@ -224,7 +228,9 @@ describe Github::Repos::Downloads do
       end
 
       it "should fail if resource is of incorrect type" do
-        expect { github.repos.downloads.upload resource, nil }.to raise_error(ArgumentError)
+        expect {
+          github.repos.downloads.upload resource, nil
+        }.to raise_error(ArgumentError)
       end
 
       it "should upload resource successfully" do

data/spec/github/repos/edit_spec.rb

@@ -0,0 +1,66 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::Repos, '#edit' do
+  let(:user) { 'peter-murach' }
+  let(:repo) { 'github' }
+  let(:request_path) { "/repos/#{user}/#{repo}" }
+  let(:inputs) do
+    { :name => 'web',
+      :description => "This is your first repo",
+      :homepage => "https://github.com",
+      :private => false,
+      :has_issues => true,
+      :has_wiki => true }
+  end
+
+  before {
+    stub_patch(request_path).with(inputs).
+      to_return(:body => body, :status => status,
+        :headers => { :content_type => "application/json; charset=utf-8"})
+  }
+
+  after { reset_authentication_for subject }
+
+  context "resource edited successfully" do
+    let(:body)  { fixture("repos/repo.json") }
+    let(:status) { 200 }
+
+    it "should fail to edit without 'user/repo' parameters" do
+      expect { subject.edit user, nil }.to raise_error(ArgumentError)
+    end
+
+    it "should fail to edit resource without 'name' parameter" do
+      expect{
+        subject.edit user, repo, inputs.except(:name)
+      }.to raise_error(Github::Error::RequiredParams)
+    end
+
+    it "should edit the resource" do
+      subject.edit user, repo, inputs
+      a_patch(request_path).with(inputs).should have_been_made
+    end
+
+    it "should return resource" do
+      repository = subject.edit user, repo, inputs
+      repository.should be_a Hashie::Mash
+    end
+
+    it "should be able to retrieve information" do
+      repository = subject.edit user, repo, inputs
+      repository.name.should == 'Hello-World'
+    end
+  end
+
+  context "failed to edit resource" do
+    let(:body)   { '' }
+    let(:status) { 404 }
+
+    it "should fail to find resource" do
+      expect {
+        subject.edit user, repo, inputs
+      }.to raise_error(Github::Error::NotFound)
+    end
+  end
+end # edit

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

@@ -0,0 +1,49 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::Repos::Forks, '#create' do
+  let(:user) { 'peter-murach' }
+  let(:repo) { 'github' }
+  let(:request_path) { "/repos/#{user}/#{repo}/forks" }
+  let(:inputs) { {:org => 'github'} }
+
+  before {
+    stub_post(request_path).with(inputs).
+      to_return(:body => body, :status => status,
+        :headers => {:content_type => "application/json; charset=utf-8"})
+  }
+
+  after { reset_authentication_for(subject) }
+
+  context "resouce created" do
+    let(:body)   { fixture('repos/fork.json') }
+    let(:status) { 202 }
+
+    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
+      fork = subject.create user, repo, inputs
+      fork.should be_a Hashie::Mash
+    end
+
+    it "should get the fork information" do
+      fork = subject.create user, repo, inputs
+      fork.name.should == 'Hello-World'
+    end
+  end
+
+  context "failed to create resource" do
+    let(:body)   { "" }
+    let(:status) { 404 }
+
+    it "should faile to retrieve resource" do
+      expect {
+        subject.create user, repo, inputs
+      }.to raise_error(Github::Error::NotFound)
+    end
+  end
+end # create