github_api 0.8.1 → 0.8.2

data/lib/github_api/version.rb

@@ -4,7 +4,7 @@ module Github
   module VERSION
     MAJOR = 0
     MINOR = 8
-    PATCH = 1
+    PATCH = 2
     BUILD = nil
 
     STRING = [MAJOR, MINOR, PATCH, BUILD].compact.join('.');

data/spec/fixtures/gitignore/template.json

@@ -0,0 +1,4 @@
+{
+  "name": "C",
+  "source": "# Object files\n*.o\n\n# Libraries\n*.lib\n*.a\n\n# Shared objects (inc. Windows DLLs)\n*.dll\n*.so\n*.so.*\n*.dylib\n\n# Executables\n*.exe\n*.out\n*.app\n"
+}

data/spec/fixtures/gitignore/template_raw

@@ -0,0 +1 @@
+"*.gem\n*.rbc\n.bundle\n.config\ncoverage\nInstalledFiles\nlib/bundler/man\npkg\nrdoc\nspec/reports\ntest/tmp\ntest/version_tmp\ntmp\n\n# YARD artifacts\n.yardoc\n_yardoc\ndoc/\n"

data/spec/fixtures/gitignore/templates.json

@@ -0,0 +1,9 @@
+[
+  "Actionscript",
+  "Android",
+  "AppceleratorTitanium",
+  "Autotools",
+  "Bancha",
+  "C",
+  "C++"
+]

data/spec/fixtures/users/users.json

@@ -0,0 +1,9 @@
+[
+  {
+    "login": "octocat",
+    "id": 1,
+    "avatar_url": "https://github.com/images/error/octocat_happy.gif",
+    "gravatar_id": "somehexcode",
+    "url": "https://api.github.com/users/octocat"
+  }
+]

data/spec/github/activity/notifications/mark_spec.rb

@@ -3,7 +3,7 @@
 require 'spec_helper'
 
 describe Github::Activity::Notifications, '#mark' do
-  let(:body) { '' }
+  let(:body) { '[]' }
   let(:status) { 205 }
   let(:inputs) { {
     :read => true,

data/spec/github/activity/starring/starring_spec.rb

@@ -13,7 +13,7 @@ describe Github::Activity::Starring, '#list' do
     context "this repo is being watched by the user"
       before do
         stub_get(request_path).
-          to_return(:body => "", :status => 404,
+          to_return(:body => "[]", :status => 404,
                     :headers => {:user_agent => subject.user_agent})
       end
 
@@ -23,7 +23,7 @@ describe Github::Activity::Starring, '#list' do
     end
 
     it "should return true if resoure found" do
-        stub_get(request_path).to_return(:body => "", :status => 200,
+        stub_get(request_path).to_return(:body => "[]", :status => 200,
           :headers => {:user_agent => subject.user_agent})
       starring = subject.starring? user, repo
       starring.should be_true

data/spec/github/activity/watching/watching_spec.rb

@@ -13,7 +13,7 @@ describe Github::Activity::Watching, '#list' do
     context "this repo is being watched by the user"
       before do
         stub_get(request_path).
-          to_return(:body => "", :status => 404,
+          to_return(:body => "[]", :status => 404,
                     :headers => {:user_agent => subject.user_agent})
       end
 
@@ -24,7 +24,7 @@ describe Github::Activity::Watching, '#list' do
 
     it "should return true if resoure found" do
         stub_get(request_path).
-          to_return(:body => "", :status => 200,
+          to_return(:body => "[]", :status => 200,
             :headers => {:user_agent => subject.user_agent})
       watching = subject.watching? user, repo
       watching.should be_true

data/spec/github/git_data/commits/create_spec.rb

@@ -0,0 +1,75 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::GitData::Commits, '#get' do
+  let(:user) { 'peter-murach' }
+  let(:repo) { 'github' }
+  let(:request_path) { "/repos/#{user}/#{repo}/git/commits" }
+  let(:inputs) {
+    {
+      "message" =>  "my commit message",
+      "author" =>  {
+        "name" =>  "Scott Chacon",
+        "email" => "schacon@gmail.com",
+        "date" => "2008-07-09T16:13:30+12:00"
+        },
+      "parents" => [
+        "7d1b31e74ee336d15cbd21741bc88a537ed063a0"
+      ],
+      "tree" => "827efc6d56897b048c772eb4087f854f46256132",
+      'unrelated' => 'giberrish'
+    }
+  }
+
+  before {
+    stub_post(request_path).with(inputs.except('unrelated')).
+      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('git_data/commit.json') }
+    let(:status) { 201 }
+
+    it "should fail to create resource if 'message' input is missing" do
+      expect {
+        subject.create user, repo, inputs.except('message')
+      }.to raise_error(Github::Error::RequiredParams)
+    end
+
+    it "should fail to create resource if 'tree' input is missing" do
+      expect {
+        subject.create user, repo, inputs.except('tree')
+      }.to raise_error(Github::Error::RequiredParams)
+    end
+
+    it "should fail to create resource if 'parents' input is missing" do
+      expect {
+        subject.create user, repo, inputs.except('parents')
+      }.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
+      commit = subject.create user, repo, inputs
+      commit.should be_a Hashie::Mash
+    end
+
+    it "should get the commit information" do
+      commit = subject.create user, repo, inputs
+      commit.author.name.should eql "Scott Chacon"
+    end
+  end
+
+  it_should_behave_like 'request failure' do
+    let(:requestable) { subject.create user, repo, inputs }
+  end
+
+end # create

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

@@ -0,0 +1,50 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::GitData::Commits, '#get' do
+  let(:user) { 'peter-murach' }
+  let(:repo) { 'github' }
+  let(:sha) { "3a0f86fb8db8eea7ccbb9a95f325ddbedfb25e15" }
+  let(:request_path) { "/repos/#{user}/#{repo}/git/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('git_data/commit.json') }
+    let(:status) { 200 }
+
+    it { should respond_to :find }
+
+    it "should fail to get resource without sha" 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.author.name.should eql "Scott Chacon"
+    end
+
+    it "should return mash" do
+      commit = subject.get user, repo, sha
+      commit.should be_a Hashie::Mash
+    end
+  end
+
+  it_should_behave_like 'request failure' do
+    let(:requestable) { subject.get user, repo, sha }
+  end
+
+end # get

data/spec/github/git_data/commits_spec.rb

@@ -2,133 +2,8 @@
 
 require 'spec_helper'
 
-describe Github::GitData::Commits, :type => :base do
-  let(:github) { Github.new }
-  let(:user) { 'peter-murach' }
-  let(:repo) { 'github' }
-  let(:sha) { "3a0f86fb8db8eea7ccbb9a95f325ddbedfb25e15" }
-
-  after { github.user, github.repo, github.oauth_token = nil, nil, nil }
+describe Github::GitData::Commits do
 
   it { described_class::VALID_COMMIT_PARAM_NAMES.should_not be_nil }
 
-  describe "#get" do
-    it { github.git_data.commits.should respond_to :find }
-
-    context "resource found" do
-      before do
-        stub_get("/repos/#{user}/#{repo}/git/commits/#{sha}").
-          to_return(:body => fixture('git_data/commit.json'), :status => 200, :headers => {:content_type => "application/json; charset=utf-8"})
-      end
-
-      it "should fail to get resource without sha" do
-        expect {
-          github.git_data.commits.get user, repo, nil
-        }.to raise_error(ArgumentError)
-      end
-
-      it "should get the resource" do
-        github.git_data.commits.get user, repo, sha
-        a_get("/repos/#{user}/#{repo}/git/commits/#{sha}").should have_been_made
-      end
-
-      it "should get commit information" do
-        commit = github.git_data.commits.get user, repo, sha
-        commit.author.name.should eql "Scott Chacon"
-      end
-
-      it "should return mash" do
-        commit = github.git_data.commits.get user, repo, sha
-        commit.should be_a Hashie::Mash
-      end
-    end
-
-    context "resource not found" do
-      before do
-        stub_get("/repos/#{user}/#{repo}/git/commits/#{sha}").
-          to_return(:body => fixture('git_data/commit.json'),
-            :status => 404,
-            :headers => {:content_type => "application/json; charset=utf-8"})
-      end
-
-      it "should fail to retrive resource" do
-        expect {
-          github.git_data.commits.get user, repo, sha
-        }.to raise_error(Github::Error::NotFound)
-      end
-    end
-  end # get
-
-  describe "#create" do
-    let(:inputs) {
-      {
-        "message" =>  "my commit message",
-        "author" =>  {
-          "name" =>  "Scott Chacon",
-          "email" => "schacon@gmail.com",
-          "date" => "2008-07-09T16:13:30+12:00"
-         },
-        "parents" => [
-          "7d1b31e74ee336d15cbd21741bc88a537ed063a0"
-        ],
-        "tree" => "827efc6d56897b048c772eb4087f854f46256132",
-        'unrelated' => 'giberrish'
-      }
-    }
-
-    context "resouce created" do
-      before do
-        stub_post("/repos/#{user}/#{repo}/git/commits").
-          with(inputs.except('unrelated')).
-          to_return(:body => fixture('git_data/commit.json'), :status => 201, :headers => {:content_type => "application/json; charset=utf-8"})
-      end
-
-      it "should fail to create resource if 'message' input is missing" do
-        expect {
-          github.git_data.commits.create user, repo, inputs.except('message')
-        }.to raise_error(Github::Error::RequiredParams)
-      end
-
-      it "should fail to create resource if 'tree' input is missing" do
-        expect {
-          github.git_data.commits.create user, repo, inputs.except('tree')
-        }.to raise_error(Github::Error::RequiredParams)
-      end
-
-      it "should fail to create resource if 'parents' input is missing" do
-        expect {
-          github.git_data.commits.create user, repo, inputs.except('parents')
-        }.to raise_error(Github::Error::RequiredParams)
-      end
-
-      it "should create resource successfully" do
-        github.git_data.commits.create user, repo, inputs
-        a_post("/repos/#{user}/#{repo}/git/commits").with(inputs).should have_been_made
-      end
-
-      it "should return the resource" do
-        commit = github.git_data.commits.create user, repo, inputs
-        commit.should be_a Hashie::Mash
-      end
-
-      it "should get the commit information" do
-        commit = github.git_data.commits.create user, repo, inputs
-        commit.author.name.should eql "Scott Chacon"
-      end
-    end
-
-    context "failed to create resource" do
-      before do
-        stub_post("/repos/#{user}/#{repo}/git/commits").with(inputs).
-          to_return(:body => fixture('git_data/commit.json'), :status => 404, :headers => {:content_type => "application/json; charset=utf-8"})
-      end
-
-      it "should faile to retrieve resource" do
-        expect {
-          github.git_data.commits.create user, repo, inputs
-        }.to raise_error(Github::Error::NotFound)
-      end
-    end
-  end # create
-
 end # Github::GitData::Commits

data/spec/github/git_data/references/create_spec.rb

@@ -0,0 +1,67 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::GitData::References, '#create' do
+  let(:user) { 'peter-murach' }
+  let(:repo) { 'github' }
+  let(:request_path) { "/repos/#{user}/#{repo}/git/refs" }
+  let(:inputs) {
+    {
+      "ref" => "refs/heads/master",
+      "sha" => "827efc6d56897b048c772eb4087f854f46256132",
+      "unrelated" => 'giberrish'
+    }
+  }
+
+  before {
+    stub_post(request_path).with(inputs.except('unrelated')).
+      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('git_data/reference.json') }
+    let(:status) { 201 }
+
+    it "should fail to create resource if 'ref' input is missing" do
+      expect {
+        subject.create user, repo, inputs.except('ref')
+      }.to raise_error(ArgumentError)
+    end
+
+    it "should fail to create resource if 'sha' input is missing" do
+      expect {
+        subject.create user, repo, inputs.except('sha')
+      }.to raise_error(Github::Error::RequiredParams)
+    end
+
+    it "should fail to create resource if 'ref' is wrong" do
+      expect {
+        subject.create user, repo, :ref => '/heads/master', :sha => '13t2a1r3'
+      }.to raise_error(ArgumentError)
+    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
+      reference = subject.create user, repo, inputs
+      reference.first.should be_a Hashie::Mash
+    end
+
+    it "should get the reference information" do
+      reference = subject.create user, repo, inputs
+      reference.first.ref.should eql 'refs/heads/sc/featureA'
+    end
+  end
+
+  it_should_behave_like 'request failure' do
+    let(:requestable) { subject.create user, repo, inputs }
+  end
+
+end # create

data/spec/github/git_data/references/delete_spec.rb

@@ -0,0 +1,38 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::GitData::References, '#delete' do
+  let(:user) { 'peter-murach' }
+  let(:repo) { 'github' }
+  let(:ref) { "heads/master" }
+  let(:request_path) { "/repos/#{user}/#{repo}/git/refs/#{ref}" }
+
+  before {
+    stub_delete(request_path).to_return(:body => body, :status => status,
+      :headers => {:content_type => "application/json; charset=utf-8"})
+  }
+
+  after { reset_authentication_for(subject) }
+
+  context "resouce delete" do
+    let(:body) { '' }
+    let(:status) { 204 }
+
+    it { should respond_to :remove }
+
+    it "should fail to delete resource if 'ref' input is missing" do
+      expect { subject.delete user, repo, nil }.to raise_error(ArgumentError)
+    end
+
+    it "should delete resource successfully" do
+      subject.delete user, repo, ref
+      a_delete(request_path).should have_been_made
+    end
+  end
+
+  it_should_behave_like 'request failure' do
+    let(:requestable) { subject.delete user, repo, ref }
+  end
+
+end # delete