github_api 0.2.1 → 0.2.2

data/spec/github/orgs_spec.rb

@@ -1,5 +1,160 @@
 require 'spec_helper'
 
 describe Github::Orgs do
-  pending
-end
+
+  let(:github) { Github.new }
+  let(:user)   { 'peter-murach' }
+  let(:org)   { 'github' }
+
+  describe "orgs" do
+    context "resource found for a user" do
+      before do
+        stub_get("/users/#{user}/orgs").
+          to_return(:body => fixture('orgs/orgs.json'), :status => 200, :headers => {:content_type => "application/json; charset=utf-8"})
+      end
+
+      it "should get the resources" do
+        github.orgs.orgs user
+        a_get("/users/#{user}/orgs").should have_been_made
+      end
+
+      it "should return array of resources" do
+        orgs = github.orgs.orgs user
+        orgs.should be_an Array
+        orgs.should have(1).items
+      end
+
+      it "should be a mash type" do
+        orgs = github.orgs.orgs user
+        orgs.first.should be_a Hashie::Mash
+      end
+
+      it "should get org information" do
+        orgs = github.orgs.orgs user
+        orgs.first.login.should == 'github'
+      end
+
+      it "should yield to a block" do
+        github.orgs.should_receive(:orgs).with(user).and_yield('web')
+        github.orgs.orgs(user) { |param| 'web' }
+      end
+    end
+
+    context "resource found for an au user" do
+      before do
+        github.user = nil
+        github.oauth_token = OAUTH_TOKEN
+        stub_get("/user/orgs?access_token=#{OAUTH_TOKEN}").
+          to_return(:body => fixture('orgs/orgs.json'), :status => 200, :headers => {:content_type => "application/json; charset=utf-8"})
+      end
+
+      after do
+        github.user, github.oauth_token = nil, nil
+      end
+
+      it "should get the resources" do
+        github.orgs.orgs
+        a_get("/user/orgs?access_token=#{OAUTH_TOKEN}").should have_been_made
+      end
+    end
+
+    context "resource not found for a user" do
+      before do
+        stub_get("/users/#{user}/orgs").
+          to_return(:body => "", :status => [404, "Not Found"])
+      end
+
+      it "should return 404 with a message 'Not Found'" do
+        expect {
+          github.orgs.orgs user
+        }.to raise_error(Github::ResourceNotFound)
+      end
+    end
+  end # orgs
+
+  describe "org" do
+    context "resource found" do
+      before do
+        stub_get("/orgs/#{org}").
+          to_return(:body => fixture('orgs/org.json'), :status => 200, :headers => {:content_type => "application/json; charset=utf-8"})
+      end
+
+      it "should fail to get resource without org name" do
+        expect { github.orgs.org }.to raise_error(ArgumentError)
+      end
+
+      it "should get the resource" do
+        github.orgs.org org
+        a_get("/orgs/#{org}").should have_been_made
+      end
+
+      it "should get org information" do
+        organisation = github.orgs.org org
+        organisation.id.should == 1
+        organisation.login.should == 'github'
+      end
+
+      it "should return mash" do
+        organisation = github.orgs.org org
+        organisation.should be_a Hashie::Mash
+      end
+    end
+
+    context "resource not found" do
+      before do
+        stub_get("/orgs/#{org}").
+          to_return(:body => fixture('orgs/org.json'), :status => 404, :headers => {:content_type => "application/json; charset=utf-8"})
+      end
+
+      it "should fail to retrive resource" do
+        expect {
+          github.orgs.org org
+        }.to raise_error(Github::ResourceNotFound)
+      end
+    end
+  end # org
+
+  describe "edit_org" do
+    let(:inputs) { { :billing_email => 'support@github.com', :blog => "https://github.com/blog", :company => "GitHub", :email => "support@github.com", :location => "San Francisco", :name => "github" } }
+
+    context "resource edited successfully" do
+      before do
+        stub_patch("/orgs/#{org}").with(inputs).
+          to_return(:body => fixture("orgs/org.json"), :status => 200, :headers => { :content_type => "application/json; charset=utf-8"})
+      end
+
+      it "should fail to edit without 'user/repo' parameters" do
+        expect { github.orgs.edit_org }.to raise_error(ArgumentError)
+      end
+
+      it "should edit the resource" do
+        github.orgs.edit_org org
+        a_patch("/orgs/#{org}").with(inputs).should have_been_made
+      end
+
+      it "should return resource" do
+        organisation = github.orgs.edit_org org
+        organisation.should be_a Hashie::Mash
+      end
+
+      it "should be able to retrieve information" do
+        organisation = github.orgs.edit_org org
+        organisation.name.should == 'github'
+      end
+    end
+
+    context "failed to edit resource" do
+      before do
+        stub_patch("/orgs/#{org}").with(inputs).
+          to_return(:body => fixture("orgs/org.json"), :status => 404, :headers => { :content_type => "application/json; charset=utf-8"})
+      end
+
+      it "should fail to find resource" do
+        expect {
+          github.orgs.edit_org org
+        }.to raise_error(Github::ResourceNotFound)
+      end
+    end
+  end # edit_org
+
+end # Github::Orgs

data/spec/github/repos/downloads_spec.rb

@@ -1,5 +1,236 @@
 require 'spec_helper'
 
 describe Github::Repos::Downloads do
-  pending
-end
+
+  let(:github) { Github.new }
+  let(:user)   { 'peter-murach' }
+  let(:repo)   { 'github' }
+
+  describe "downloads" do
+    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
+
+      it "should fail to get resource without username" do
+        github.user, github.repo = nil, nil
+        expect { github.repos.downloads }.to raise_error(ArgumentError)
+      end
+
+      it "should get the resources" do
+        github.repos.downloads  user, repo
+        a_get("/repos/#{user}/#{repo}/downloads").should have_been_made
+      end
+
+      it "should return array of resources" do
+        downloads = github.repos.downloads user, repo
+        downloads.should be_an Array
+        downloads.should have(1).items
+      end
+
+      it "should be a mash type" do
+        downloads = github.repos.downloads user, repo
+        downloads.first.should be_a Hashie::Mash
+      end
+
+      it "should get download information" do
+        downloads = github.repos.downloads user, repo
+        downloads.first.name.should == 'new_file.jpg'
+      end
+
+      it "should yield to a block" do
+        github.repos.should_receive(:downloads).with(user, repo).and_yield('web')
+        github.repos.downloads(user, repo) { |param| 'web' }
+      end
+    end
+
+    context "resource not found" do
+      before do
+        stub_get("/repos/#{user}/#{repo}/downloads").
+          to_return(:body => "", :status => [404, "Not Found"])
+      end
+
+      it "should return 404 with a message 'Not Found'" do
+        expect {
+          github.repos.downloads user, repo
+        }.to raise_error(Github::ResourceNotFound)
+      end
+    end
+  end # downloads
+
+  describe "download" do
+    let(:download_id) { 1 }
+
+    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
+
+      it "should fail to get resource without download id" do
+        expect { github.repos.download(user, repo, nil)}.to raise_error(ArgumentError)
+      end
+
+      it "should get the resource" do
+        github.repos.download user, repo, download_id
+        a_get("/repos/#{user}/#{repo}/downloads/#{download_id}").should have_been_made
+      end
+
+      it "should get download information" do
+        download = github.repos.download user, repo, download_id
+        download.id.should == download_id
+        download.name.should == 'new_file.jpg'
+      end
+
+      it "should return mash" do
+        download = github.repos.download user, repo, download_id
+        download.should be_a Hashie::Mash
+      end
+    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
+
+      it "should fail to retrive resource" do
+        expect {
+          github.repos.download user, repo, download_id
+        }.to raise_error(Github::ResourceNotFound)
+      end
+    end
+  end # download
+
+  describe "delete_download" 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
+
+      it "should fail to delete without 'user/repo' parameters" do
+        github.user, github.repo = nil, nil
+        expect { github.repos.delete_download }.to raise_error(ArgumentError)
+      end
+
+      it "should fail to delete resource without 'download_id'" do
+        expect {
+          github.repos.delete_download user, repo
+        }.to raise_error(ArgumentError)
+      end
+
+      it "should delete the resource" do
+        github.repos.delete_download user, repo, download_id
+        a_delete("/repos/#{user}/#{repo}/downloads/#{download_id}").should have_been_made
+      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
+
+      it "should fail to find resource" do
+        expect {
+          github.repos.delete_download user, repo, download_id
+        }.to raise_error(Github::ResourceNotFound)
+      end
+    end
+  end # delete_download
+
+  describe "create_download" 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"})
+
+      end
+
+      it "should fail to create resource if 'name' input is missing" do
+        expect {
+          github.repos.create_download user, repo, inputs.except(:name)
+        }.to raise_error(ArgumentError)
+      end
+
+      it "should failt to create resource if 'size' input is missing" do
+        expect {
+          github.repos.create_download user, repo, inputs.except(:size)
+        }.to raise_error(ArgumentError)
+      end
+
+      it "should create resource successfully" do
+        github.repos.create_download user, repo, inputs
+        a_post("/repos/#{user}/#{repo}/downloads").with(inputs).should have_been_made
+      end
+
+      it "should return the resource" do
+        download = github.repos.create_download user, repo, inputs
+        download.should be_a Hashie::Mash
+      end
+
+      it "should get the download information" do
+        download = github.repos.create_download user, repo, inputs
+        download.name.should == 'new_file.jpg'
+      end
+    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
+
+      it "should faile to retrieve resource" do
+        expect {
+          github.repos.create_download(user, repo, inputs)
+        }.to raise_error(Github::ResourceNotFound)
+      end
+    end
+  end # create_download
+
+  describe 'upload' do
+    let(:inputs) { {:name => 'new_file.jpg', :size => 114034, :description => "Latest release", :content_type => 'text/plain'} }
+    let(:resource) { Hashie::Mash.new ::JSON.parse(fixture('repos/download_s3.json').read) }
+    let(:file)     { 'filename' }
+
+    context 'resource uploaded' do
+      before do
+        stub_post('', 'https://github.s3.amazonaws.com/').with(resource).
+          to_return(:body => '', :status => 200, :headers => {})
+      end
+
+      it "should fail if resource is of incorrect type" do
+        expect { github.repos.upload file, file }.to raise_error(ArgumentError)
+      end
+
+      it "should upload resource successfully" do
+        github.repos.upload resource, file
+        a_post('', 'https://github.s3.amazonaws.com').with(resource).should have_been_made
+      end
+
+    end
+    context 'failed to upload resource' do
+      before do
+        stub_post('', 'https://github.s3.amazonaws.com/').with(resource).
+          to_return(:body => '', :status => 404, :headers => {})
+      end
+
+      it "should faile to retrieve resource" do
+        expect {
+          github.repos.upload resource, file
+        }.to raise_error(Github::ResourceNotFound)
+      end
+
+    end
+  end # upload
+
+end # Github::Repos::Downloads

data/spec/github/repos/hooks_spec.rb

@@ -19,7 +19,7 @@ describe Github::Repos::Hooks do
       end
 
       it "should get the resources" do
-        github.repos.hooks(user, repo)
+        github.repos.hooks user, repo
         a_get("/repos/#{user}/#{repo}/hooks").should have_been_made
       end
 
@@ -222,7 +222,7 @@ describe Github::Repos::Hooks do
     end
   end # edit_hook
 
-  describe "delete_key" do
+  describe "delete_hook" do
     let(:hook_id) { 1 }
 
     context "resource edited successfully" do
@@ -238,7 +238,7 @@ describe Github::Repos::Hooks do
 
       it "should fail to delete resource without 'hook_id'" do
         expect {
-          github.repos.edit_hook user, repo
+          github.repos.delete_hook user, repo
         }.to raise_error(ArgumentError)
       end
 

data/spec/spec_helper.rb

@@ -4,6 +4,7 @@ $LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
 $LOAD_PATH.unshift(File.dirname(__FILE__))
 
 require 'rspec'
+require 'json'
 require 'webmock/rspec'
 require 'github_api'
 

metadata

@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
   segments: 
   - 0
   - 2
-  - 1
-  version: 0.2.1
+  - 2
+  version: 0.2.2
 platform: ruby
 authors: 
 - Piotr Murach
@@ -14,7 +14,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-10-29 00:00:00 +01:00
+date: 2011-11-26 00:00:00 +00:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -198,6 +198,7 @@ files:
 - lib/github_api/authorization.rb
 - lib/github_api/cache_control.rb
 - lib/github_api/client.rb
+- lib/github_api/compatibility.rb
 - lib/github_api/configuration.rb
 - lib/github_api/connection.rb
 - lib/github_api/core_ext/array.rb
@@ -245,6 +246,12 @@ files:
 - lib/github_api/users.rb
 - lib/github_api/version.rb
 - lib/github_api.rb
+- spec/fixtures/orgs/members.json
+- spec/fixtures/orgs/org.json
+- spec/fixtures/orgs/orgs.json
+- spec/fixtures/orgs/team.json
+- spec/fixtures/orgs/team_repos.json
+- spec/fixtures/orgs/teams.json
 - spec/fixtures/repos/branches.json
 - spec/fixtures/repos/collaborators.json
 - spec/fixtures/repos/commit.json
@@ -252,6 +259,9 @@ files:
 - spec/fixtures/repos/commit_comments.json
 - spec/fixtures/repos/commits.json
 - spec/fixtures/repos/contributors.json
+- spec/fixtures/repos/download.json
+- spec/fixtures/repos/download_s3.json
+- spec/fixtures/repos/downloads.json
 - spec/fixtures/repos/fork.json
 - spec/fixtures/repos/forks.json
 - spec/fixtures/repos/hook.json