github_api 0.3.8 → 0.3.9

data/lib/github_api/api.rb

@@ -118,8 +118,8 @@ module Github
       return params
     end
 
+    # Removes any keys from nested hashes that don't match predefiend keys
     def _filter_params_keys(keys, params)  # :nodoc:
-      # params.reject! { |k,v| !keys.include? k }
       case params
       when Hash
         params.keys.each do |k, v|
@@ -156,6 +156,7 @@ module Github
       return hash
     end
 
+    # Ensures that hash values contain predefined values
     def _validate_params_values(options, params)  # :nodoc:
       params.each do |k, v|
         next unless options.keys.include?(k)

data/lib/github_api/git_data/commits.rb

@@ -15,6 +15,12 @@ module Github
         date
       ].freeze
 
+      REQUIRED_COMMIT_PARAMS = %w[
+        message
+        tree
+        parents
+      ].freeze
+
       # Get a commit
       #
       # = Examples
@@ -29,6 +35,7 @@ module Github
 
         get("/repos/#{user}/#{repo}/git/commits/#{sha}", params)
       end
+      alias :get_commit :commit
 
       # Create a commit
       #
@@ -62,13 +69,13 @@ module Github
       #    ],
       #    "tree": "827efc6d56897b048c772eb4087f854f46256132"]
       #
-      def create_commit(user_name=nil, repo_name=nil, params={})
+      def create_commit(user_name, repo_name, params={})
         _update_user_repo_params(user_name, repo_name)
         _validate_user_repo_params(user, repo) unless user? && repo?
         _normalize_params_keys(params)
         _filter_params_keys(VALID_COMMIT_PARAM_NAMES, params)
 
-        raise ArgumentError, "Required params are: message, tree, parents" unless _validate_inputs(%w[ message tree parents ], params)
+        raise ArgumentError, "Required params are: message, tree, parents" unless _validate_inputs(REQUIRED_COMMIT_PARAMS, params)
 
         post("/repos/#{user}/#{repo}/git/commits", params)
       end

data/lib/github_api/git_data/references.rb

@@ -10,26 +10,6 @@ module Github
         'ref' => %r{^refs\/\w+\/\w+(\/\w+)*} # test fully qualified reference
       }
 
-      # Get a reference
-      #
-      # The ref in the URL must be formatted as <tt>heads/branch</tt>,
-      # not just branch. For example, the call to get the data for a
-      # branch named <tt>sc/featureA</tt> would be formatted as
-      # <tt>heads/sc/featureA</tt>
-      #
-      # = Examples
-      #  @github = Github.new
-      #  @github.git_data.reference 'user-name', 'repo-name', 'reference'
-      #
-      def reference(user_name, repo_name, ref, params={})
-        _update_user_repo_params(user_name, repo_name)
-        _validate_user_repo_params(user, repo) unless user? && repo?
-        _validate_presence_of ref
-        _normalize_params_keys(params)
-
-        get("/repos/#{user}/#{repo}/git/refs/#{ref}", params)
-      end
-
       # Get all references
       #
       # This will return an array of all the references on the system,
@@ -49,6 +29,7 @@ module Github
         _normalize_params_keys(params)
 
         response = if ref
+          _validate_reference ref
           get("/repos/#{user}/#{repo}/git/refs/#{ref}", params)
         else
           get("/repos/#{user}/#{repo}/git/refs", params)
@@ -56,6 +37,32 @@ module Github
         return response unless block_given?
         response.each { |el| yield el }
       end
+      alias :list_references :references
+      alias :get_all_references :references
+
+      # Get a reference
+      #
+      # The ref in the URL must be formatted as <tt>heads/branch</tt>,
+      # not just branch. For example, the call to get the data for a
+      # branch named <tt>sc/featureA</tt> would be formatted as
+      # <tt>heads/sc/featureA</tt>
+      #
+      # = Examples
+      #  @github = Github.new
+      #  @github.git_data.reference 'user-name', 'repo-name', 'reference'
+      #
+      def reference(user_name, repo_name, ref, params={})
+        _update_user_repo_params(user_name, repo_name)
+        _validate_user_repo_params(user, repo) unless user? && repo?
+
+        _validate_presence_of ref
+        _validate_reference ref
+        _normalize_params_keys params
+
+        get("/repos/#{user}/#{repo}/git/refs/#{ref}", params)
+      end
+      alias :get_reference :reference
+
 
       # Create a reference
       #
@@ -72,12 +79,12 @@ module Github
       def create_reference(user_name, repo_name, params={})
         _update_user_repo_params(user_name, repo_name)
         _validate_user_repo_params(user, repo) unless user? && repo?
-        _normalize_params_keys(params)
 
-        raise ArgumentError, "Required params are: ref, sha" unless _validate_inputs(%w[ ref sha ], params)
+        _normalize_params_keys params
+        _filter_params_keys VALID_REF_PARAM_NAMES, params
+        _validate_reference params['ref']
 
-        _filter_params_keys(VALID_REF_PARAM_NAMES, params)
-        _validate_params_values(VALID_REF_PARAM_VALUES, params)
+        raise ArgumentError, "Required params are: ref, sha" unless _validate_inputs(%w[ ref sha ], params)
 
         post("/repos/#{user}/#{repo}/git/refs", params)
       end
@@ -97,17 +104,25 @@ module Github
       def update_reference(user_name, repo_name, ref, params={})
         _update_user_repo_params(user_name, repo_name)
         _validate_user_repo_params(user, repo) unless user? && repo?
+
         _validate_presence_of ref
+        _validate_reference ref
         _normalize_params_keys(params)
+        _filter_params_keys(VALID_REF_PARAM_NAMES, params)
 
         raise ArgumentError, "Required params are: sha" unless _validate_inputs(%w[ sha ], params)
 
-        _filter_params_keys(VALID_REF_PARAM_NAMES, params)
-        _validate_params_values(VALID_REF_PARAM_VALUES, params)
-
         patch("/repos/#{user}/#{repo}/git/refs/#{ref}", params)
       end
 
+    private
+
+      def _validate_reference ref
+        unless VALID_REF_PARAM_VALUES['ref'] =~ "refs/#{ref}"
+          raise ArgumentError, "Provided 'reference' is invalid"
+        end
+      end
+
     end # References
   end # GitData
 end # Github

data/lib/github_api/git_data/tags.rb

@@ -13,7 +13,10 @@ module Github
         name
         email
         date
-      ]
+        sha
+        url
+        tagger
+      ].freeze
 
       VALID_TAG_PARAM_VALUES = {
         'type' => %w[ blob tree commit ]
@@ -33,6 +36,7 @@ module Github
 
         get("/repos/#{user}/#{repo}/git/tags/#{sha}", params)
       end
+      alias :get_tag :tag
 
       # Create a tag object
       # Note that creating a tag object does not create the reference that

data/lib/github_api/version.rb

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

data/spec/fixtures/git_data/commit.json

@@ -0,0 +1,25 @@
+{
+  "sha": "7638417db6d59f3c431d3e1f261cc637155684cd",
+  "url": "https://api.github.com/repos/octocat/Hello-World/git/commits/7638417db6d59f3c431d3e1f261cc637155684cd",
+  "author": {
+    "date": "2010-04-10T14:10:01-07:00",
+    "name": "Scott Chacon",
+    "email": "schacon@gmail.com"
+  },
+  "committer": {
+    "date": "2010-04-10T14:10:01-07:00",
+    "name": "Scott Chacon",
+    "email": "schacon@gmail.com"
+  },
+  "message": "added readme, because im a good github citizen\n",
+  "tree": {
+    "url": "https://api.github.com/repos/octocat/Hello-World/git/trees/691272480426f78a0138979dd3ce63b77f706feb",
+    "sha": "691272480426f78a0138979dd3ce63b77f706feb"
+  },
+  "parents": [
+    {
+      "url": "https://api.github.com/repos/octocat/Hello-World/git/commits/1acc419d4d6a9ce985db7be48c6349a0475975b5",
+      "sha": "1acc419d4d6a9ce985db7be48c6349a0475975b5"
+    }
+  ]
+}

data/spec/fixtures/git_data/reference.json

@@ -0,0 +1,11 @@
+[
+  {
+    "ref": "refs/heads/sc/featureA",
+    "url": "https://api.github.com/repos/octocat/Hello-World/git/refs/heads/sc/featureA",
+    "object": {
+      "type": "commit",
+      "sha": "aa218f56b14c9653891f9e74264a383fa43fefbd",
+      "url": "https://api.github.com/repos/octocat/Hello-World/git/commits/aa218f56b14"
+    }
+  }
+]

data/spec/fixtures/git_data/references.json

@@ -0,0 +1,29 @@
+[
+  {
+    "ref": "refs/heads/master",
+    "url": "https://api.github.com/repos/octocat/Hello-World/git/refs/heads/master",
+    "object": {
+      "type": "commit",
+      "sha": "aa218f56b14c9653891f9e74264a383fa43fefbd",
+      "url": "https://api.github.com/repos/octocat/Hello-World/git/commits/aa218f56b14c9653891f9e74264a383fa"
+    }
+  },
+  {
+    "ref": "refs/heads/gh-pages",
+    "url": "https://api.github.com/repos/octocat/Hello-World/git/refs/heads/gh-pages",
+    "object": {
+      "type": "commit",
+      "sha": "612077ae6dffb4d2fbd8ce0cccaa58893b07b5ac",
+      "url": "https://api.github.com/repos/octocat/Hello-World/git/commits/612077ae6dffb4d2"
+    }
+  },
+  {
+    "ref": "refs/tags/v0.0.1",
+    "url": "https://api.github.com/repos/octocat/Hello-World/git/refs/tags/v0.0.1",
+    "object": {
+      "type": "tag",
+      "sha": "940bd336248efae0f9ee5bc7b2d5c985887b16ac",
+      "url": "https://api.github.com/repos/octocat/Hello-World/git/tags/940bd336248efae0f"
+    }
+  }
+]

data/spec/fixtures/git_data/tag.json

@@ -0,0 +1,16 @@
+{
+  "tag": "v0.0.1",
+  "sha": "940bd336248efae0f9ee5bc7b2d5c985887b16ac",
+  "url": "https://api.github.com/repos/octocat/Hello-World/git/tags/940bd336248efae0f9ee5bc7b2d5c985887b16ac",
+  "message": "initial version\n",
+  "tagger": {
+    "name": "Scott Chacon",
+    "email": "schacon@gmail.com",
+    "date": "2011-06-17T14:53:35-07:00"
+  },
+  "object": {
+    "type": "commit",
+    "sha": "c3d0be41ecbe669545ee3e94d31ed9a4bc91ee3c",
+    "url": "https://api.github.com/repos/octocat/Hello-World/git/commits/c3d0be41ecbe669545ee3e94d31ed9a4bc91ee3c"
+  }
+}

data/spec/github/gists/comments_spec.rb

@@ -1,5 +1,5 @@
 require 'spec_helper'
 
-describe Github::Gists::Comments do
+describe Github::Gists::Comments, :type => :base do
   pending
-end
+end # Github::Gists::Comments

data/spec/github/git_data/blobs_spec.rb

@@ -1,7 +1,6 @@
 require 'spec_helper'
 
-describe Github::GitData::Blobs do
-  include SpecHelpers::Base
+describe Github::GitData::Blobs, :type => :base do
 
   let(:sha) { "3a0f86fb8db8eea7ccbb9a95f325ddbedfb25e15" }
 
@@ -55,14 +54,15 @@ describe Github::GitData::Blobs do
     let(:inputs) {
       {
         "content" => "Content of the blob",
-        "encoding" =>  "utf-8"
+        "encoding" =>  "utf-8",
+        "unrelated" => 'giberrish'
       }
     }
 
     context "resouce created" do
       before do
         stub_post("/repos/#{user}/#{repo}/git/blobs").
-          with(:body => JSON.generate(inputs)).
+          with(:body => JSON.generate(inputs.except('unrelated'))).
           to_return(:body => fixture('git_data/blob_sha.json'), :status => 201, :headers => {:content_type => "application/json; charset=utf-8"})
       end
 

data/spec/github/git_data/commits_spec.rb

@@ -1,5 +1,126 @@
 require 'spec_helper'
 
-describe Github::GitData::Commits do
-  pending
-end
+describe Github::GitData::Commits, :type => :base do
+
+  let(:sha) { "3a0f86fb8db8eea7ccbb9a95f325ddbedfb25e15" }
+
+  it { described_class::VALID_COMMIT_PARAM_NAMES.should_not be_nil }
+
+  describe "commit" do
+    it { github.git_data.should respond_to :commit }
+    it { github.git_data.should respond_to :get_commit }
+
+    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.commit(user, repo, nil)}.to raise_error(ArgumentError)
+      end
+
+      it "should get the resource" do
+        github.git_data.commit 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.commit user, repo, sha
+        commit.author.name.should eql "Scott Chacon"
+      end
+
+      it "should return mash" do
+        commit = github.git_data.commit 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.commit user, repo, sha
+        }.to raise_error(Github::ResourceNotFound)
+      end
+    end
+  end # commit
+
+  describe "create_commit" 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(:body => JSON.generate(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.create_commit user, repo, inputs.except('message')
+        }.to raise_error(ArgumentError)
+      end
+
+      it "should fail to create resource if 'tree' input is missing" do
+        expect {
+          github.git_data.create_commit user, repo, inputs.except('tree')
+        }.to raise_error(ArgumentError)
+      end
+
+      it "should fail to create resource if 'parents' input is missing" do
+        expect {
+          github.git_data.create_commit user, repo, inputs.except('parents')
+        }.to raise_error(ArgumentError)
+      end
+
+      it "should create resource successfully" do
+        github.git_data.create_commit 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.create_commit user, repo, inputs
+        commit.should be_a Hashie::Mash
+      end
+
+      it "should get the commit information" do
+        commit = github.git_data.create_commit 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.create_commit user, repo, inputs
+        }.to raise_error(Github::ResourceNotFound)
+      end
+    end
+  end # create_commit
+
+end # Github::GitData::Commits

data/spec/github/git_data/references_spec.rb

@@ -1,5 +1,263 @@
 require 'spec_helper'
 
-describe Github::GitData::References do
-  pending
-end
+describe Github::GitData::References, :type => :base do
+
+  let(:ref) { "heads/master" }
+  let(:sha) { "3a0f86fb8db8eea7ccbb9a95f325ddbedfb25e15" }
+
+  it { described_class::VALID_REF_PARAM_NAMES.should_not be_nil }
+  it { described_class::VALID_REF_PARAM_VALUES.should_not be_nil }
+
+  describe "references" do
+
+    context 'check aliases' do
+      it { github.git_data.should respond_to :references }
+      it { github.git_data.should respond_to :list_references }
+      it { github.git_data.should respond_to :get_all_references }
+    end
+
+    context "get all the refernces based on sub-namespace" do
+      before do
+        stub_get("/repos/#{user}/#{repo}/git/refs/#{ref}").
+          to_return(:body => fixture('git_data/references.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.git_data.references }.to raise_error(ArgumentError)
+      end
+
+      it "should fail to call with invalid reference" do
+        expect {
+          github.git_data.references user, repo, 'branch'
+        }.to raise_error(ArgumentError)
+      end
+
+      it "should get the resources" do
+        github.git_data.references user, repo, ref
+        a_get("/repos/#{user}/#{repo}/git/refs/#{ref}").should have_been_made
+      end
+
+      it "should return array of resources" do
+        references = github.git_data.references user, repo, ref
+        references.should be_an Array
+        references.should have(3).items
+      end
+
+      it "should be a mash type" do
+        references = github.git_data.references user, repo, ref
+        references.first.should be_a Hashie::Mash
+      end
+
+      it "should get reference information" do
+        references = github.git_data.references user, repo, ref
+        references.first.ref.should eql 'refs/heads/master'
+      end
+
+      it "should yield to a block" do
+        github.git_data.should_receive(:references).with(user, repo, ref).and_yield('web')
+        github.git_data.references(user, repo, ref) { |param| 'web' }
+      end
+    end
+
+    context "get all the references on the system" do
+      before do
+        stub_get("/repos/#{user}/#{repo}/git/refs").
+          to_return(:body => fixture('git_data/references.json'), :status => 200, :headers => {:content_type => "application/json; charset=utf-8"})
+      end
+
+      it "should get the resources" do
+        github.git_data.references user, repo
+        a_get("/repos/#{user}/#{repo}/git/refs").should have_been_made
+      end
+    end
+
+    context "resource not found" do
+      before do
+        stub_get("/repos/#{user}/#{repo}/git/refs/#{ref}").
+          to_return(:body => "", :status => [404, "Not Found"])
+      end
+
+      it "should return 404 with a message 'Not Found'" do
+        expect {
+          github.git_data.references user, repo, ref
+        }.to raise_error(Github::ResourceNotFound)
+      end
+    end
+  end # references
+
+  describe "reference" do
+    it { github.git_data.should respond_to :reference }
+    it { github.git_data.should respond_to :get_reference }
+
+    context "resource found" do
+      before do
+        stub_get("/repos/#{user}/#{repo}/git/refs/#{ref}").
+          to_return(:body => fixture('git_data/reference.json'), :status => 200, :headers => {:content_type => "application/json; charset=utf-8"})
+      end
+
+      it "should fail to get resource without ref" do
+        expect { github.git_data.reference(user, repo, nil)}.to raise_error(ArgumentError)
+      end
+
+      it "should fail to get resource with wrong ref" do
+        expect {
+          github.git_data.reference user, repo, 'branch'
+        }.to raise_error(ArgumentError)
+      end
+
+      it "should get the resource" do
+        github.git_data.reference user, repo, ref
+        a_get("/repos/#{user}/#{repo}/git/refs/#{ref}").should have_been_made
+      end
+
+      it "should get reference information" do
+        reference = github.git_data.reference user, repo, ref
+        reference.first.ref.should eql "refs/heads/sc/featureA"
+      end
+
+      it "should return mash" do
+        reference = github.git_data.reference user, repo, ref
+        reference.first.should be_a Hashie::Mash
+      end
+    end
+
+    context "resource not found" do
+      before do
+        stub_get("/repos/#{user}/#{repo}/git/refs/#{ref}").
+          to_return(:body => fixture('git_data/reference.json'), :status => 404, :headers => {:content_type => "application/json; charset=utf-8"})
+      end
+
+      it "should fail to retrive resource" do
+        expect {
+          github.git_data.reference user, repo, ref
+        }.to raise_error(Github::ResourceNotFound)
+      end
+    end
+  end # reference
+
+  describe "create_reference" do
+    let(:inputs) {
+      {
+        "ref" => "refs/heads/master",
+        "sha" => "827efc6d56897b048c772eb4087f854f46256132",
+        "unrelated" => 'giberrish'
+      }
+    }
+
+    context "resouce created" do
+      before do
+        stub_post("/repos/#{user}/#{repo}/git/refs").
+          with(:body => JSON.generate(inputs.except('unrelated'))).
+          to_return(:body => fixture('git_data/reference.json'), :status => 201, :headers => {:content_type => "application/json; charset=utf-8"})
+      end
+
+      it "should fail to create resource if 'ref' input is missing" do
+        expect {
+          github.git_data.create_reference user, repo, inputs.except('ref')
+        }.to raise_error(ArgumentError)
+      end
+
+      it "should fail to create resource if 'sha' input is missing" do
+        expect {
+          github.git_data.create_reference user, repo, inputs.except('sha')
+        }.to raise_error(ArgumentError)
+      end
+
+      it "should fail to create resource if 'ref' is wrong" do
+        expect {
+          github.git_data.create_reference user, repo, 'branch'
+        }.to raise_error(ArgumentError)
+      end
+
+      it "should create resource successfully" do
+        github.git_data.create_reference user, repo, inputs
+        a_post("/repos/#{user}/#{repo}/git/refs").with(inputs).should have_been_made
+      end
+
+      it "should return the resource" do
+        reference = github.git_data.create_reference user, repo, inputs
+        reference.first.should be_a Hashie::Mash
+      end
+
+      it "should get the reference information" do
+        reference = github.git_data.create_reference user, repo, inputs
+        reference.first.ref.should eql 'refs/heads/sc/featureA'
+      end
+    end
+
+    context "failed to create resource" do
+      before do
+        stub_post("/repos/#{user}/#{repo}/git/refs").with(inputs).
+          to_return(:body => fixture('git_data/reference.json'), :status => 404, :headers => {:content_type => "application/json; charset=utf-8"})
+
+      end
+
+      it "should faile to retrieve resource" do
+        expect {
+          github.git_data.create_reference user, repo, inputs
+        }.to raise_error(Github::ResourceNotFound)
+      end
+    end
+  end # create_reference
+
+  describe "update_reference" do
+    let(:inputs) {
+      {
+        "sha" => "827efc6d56897b048c772eb4087f854f46256132",
+        "force" => true,
+        "unrelated" => 'giberrish'
+      }
+    }
+
+    context "resouce updated" do
+      before do
+        stub_patch("/repos/#{user}/#{repo}/git/refs/#{ref}").
+          with(:body => JSON.generate(inputs.except('unrelated'))).
+          to_return(:body => fixture('git_data/reference.json'), :status => 201, :headers => {:content_type => "application/json; charset=utf-8"})
+      end
+
+      it "should fail to create resource if 'sha' input is missing" do
+        expect {
+          github.git_data.update_reference user, repo, ref, inputs.except('sha')
+        }.to raise_error(ArgumentError)
+      end
+
+      it "should fail to create resource if 'ref' is wrong" do
+        expect {
+          github.git_data.update_reference user, repo, 'branch', inputs
+        }.to raise_error(ArgumentError)
+      end
+
+      it "should update resource successfully" do
+        github.git_data.update_reference user, repo, ref, inputs
+        a_patch("/repos/#{user}/#{repo}/git/refs/#{ref}").with(inputs).should have_been_made
+      end
+
+      it "should return the resource" do
+        reference = github.git_data.update_reference user, repo, ref, inputs
+        reference.first.should be_a Hashie::Mash
+      end
+
+      it "should get the reference information" do
+        reference = github.git_data.update_reference user, repo, ref, inputs
+        reference.first.ref.should eql 'refs/heads/sc/featureA'
+      end
+    end
+
+    context "failed to create resource" do
+      before do
+        stub_patch("/repos/#{user}/#{repo}/git/refs/#{ref}").with(inputs).
+          to_return(:body => fixture('git_data/reference.json'), :status => 404, :headers => {:content_type => "application/json; charset=utf-8"})
+
+      end
+
+      it "should faile to retrieve resource" do
+        expect {
+          github.git_data.update_reference user, repo, ref, inputs
+        }.to raise_error(Github::ResourceNotFound)
+      end
+    end
+  end # update_reference
+
+end # Github::GitData::References

data/spec/github/git_data/tags_spec.rb

@@ -1,5 +1,111 @@
 require 'spec_helper'
 
-describe Github::GitData::Tags do
-  pending
-end
+describe Github::GitData::Tags, :type => :base do
+
+  let(:sha) { "940bd336248efae0f9ee5bc7b2d5c985887b16ac" }
+
+  it { described_class::VALID_TAG_PARAM_NAMES.should_not be_nil }
+
+  describe "tag" do
+    it { github.git_data.should respond_to :tag }
+    it { github.git_data.should respond_to :get_tag }
+
+    context "resource found" do
+      before do
+        stub_get("/repos/#{user}/#{repo}/git/tags/#{sha}").
+          to_return(:body => fixture('git_data/tag.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.tag(user, repo, nil)}.to raise_error(ArgumentError)
+      end
+
+      it "should get the resource" do
+        github.git_data.tag user, repo, sha
+        a_get("/repos/#{user}/#{repo}/git/tags/#{sha}").should have_been_made
+      end
+
+      it "should get tag information" do
+        tag = github.git_data.tag user, repo, sha
+        tag.tag.should eql "v0.0.1"
+      end
+
+      it "should return mash" do
+        tag = github.git_data.tag user, repo, sha
+        tag.should be_a Hashie::Mash
+      end
+    end
+
+    context "resource not found" do
+      before do
+        stub_get("/repos/#{user}/#{repo}/git/tags/#{sha}").
+          to_return(:body => fixture('git_data/tag.json'), :status => 404, :headers => {:content_type => "application/json; charset=utf-8"})
+      end
+
+      it "should fail to retrive resource" do
+        expect {
+          github.git_data.tag user, repo, sha
+        }.to raise_error(Github::ResourceNotFound)
+      end
+    end
+  end # tag
+
+  describe "create_tag" do
+    let(:inputs) {
+      {
+        "tag" => "v0.0.1",
+        "message" => "initial version\n",
+         "object" => {
+           "type" => "commit",
+           "sha" => "c3d0be41ecbe669545ee3e94d31ed9a4bc91ee3c",
+           "url" => "https://api.github.com/repos/octocat/Hello-World/git/commits/c3d0be41ecbe669545ee3e94d31ed9a4bc91ee3c"
+         },
+        "tagger" => {
+          "name" => "Scott Chacon",
+          "email" => "schacon@gmail.com",
+          "date" => "2011-06-17T14:53:35-07:00"
+        },
+        'unrelated' => 'giberrish'
+      }
+    }
+
+    context "resouce created" do
+      before do
+        stub_post("/repos/#{user}/#{repo}/git/tags").
+          with(:body => JSON.generate(inputs.except('unrelated'))).
+          to_return(:body => fixture('git_data/tag.json'), :status => 201, :headers => {:content_type => "application/json; charset=utf-8"})
+      end
+
+      it "should create resource successfully" do
+        github.git_data.create_tag user, repo, inputs
+        a_post("/repos/#{user}/#{repo}/git/tags").with(inputs).should have_been_made
+      end
+
+      it "should return the resource" do
+        tag = github.git_data.create_tag user, repo, inputs
+        tag.should be_a Hashie::Mash
+      end
+
+      it "should get the tag information" do
+        tag = github.git_data.create_tag user, repo, inputs
+        tag.sha.should == sha
+      end
+    end
+
+    context "failed to create resource" do
+      before do
+        stub_post("/repos/#{user}/#{repo}/git/tags").
+          with(inputs).
+          to_return(:body => fixture('git_data/tag.json'), :status => 404, :headers => {:content_type => "application/json; charset=utf-8"})
+
+      end
+
+      it "should faile to retrieve resource" do
+        expect {
+          github.git_data.create_tag user, repo, inputs
+        }.to raise_error(Github::ResourceNotFound)
+      end
+    end
+  end # create_tag
+
+end # Github::GitData::Tags

data/spec/github/git_data/trees_spec.rb

@@ -1,7 +1,6 @@
 require 'spec_helper'
 
-describe Github::GitData::Trees do
-  include SpecHelpers::Base
+describe Github::GitData::Trees, :type => :base do
 
   let(:sha) { "9fb037999f264ba9a7fc6274d15fa3ae2ab98312" }
 

data/spec/github/issues/comments_spec.rb

@@ -1,11 +1,8 @@
 require 'spec_helper'
 
-describe Github::Issues::Comments do
-  include SpecHelpers::Base
+describe Github::Issues::Comments, :type => :base do
 
-  let(:comments_api) { Github::Issues::Comments }
-
-  it { comments_api::VALID_ISSUE_COMMENT_PARAM_NAME.should_not be_nil }
+  it { described_class::VALID_ISSUE_COMMENT_PARAM_NAME.should_not be_nil }
 
   describe 'comments' do
     let(:issue_id) { 1 }

data/spec/github/issues/events_spec.rb

@@ -1,7 +1,6 @@
 require 'spec_helper'
 
-describe Github::Issues::Events do
-  include SpecHelpers::Base
+describe Github::Issues::Events, :type => :base do
 
   describe 'events' do
     it { github.issues.should respond_to :events }

data/spec/github/issues/labels_spec.rb

@@ -1,7 +1,6 @@
 require 'spec_helper'
 
-describe Github::Issues::Labels do
-  include SpecHelpers::Base
+describe Github::Issues::Labels, :type => :base do
 
   it { described_class::VALID_LABEL_INPUTS.should_not be_nil }
 

data/spec/github/issues/milestones_spec.rb

@@ -1,7 +1,6 @@
 require 'spec_helper'
 
-describe Github::Issues::Milestones do
-  include SpecHelpers::Base
+describe Github::Issues::Milestones, :type => :base do
 
   it { described_class::VALID_MILESTONE_OPTIONS.should_not be_nil }
   it { described_class::VALID_MILESTONE_INPUTS.should_not be_nil }

data/spec/github/orgs/teams_spec.rb

@@ -1,13 +1,9 @@
 require 'spec_helper'
 
-describe Github::Orgs::Teams do
+describe Github::Orgs::Teams, :type => :base do
 
-  let(:github) { Github.new }
   let(:team)   { 'github' }
   let(:member) { 'github' }
-  let(:org)    { 'github' }
-  let(:user)   { 'peter-murach' }
-  let(:repo)   { 'github' }
 
   describe "teams" do
     context "resource found" do

data/spec/github/orgs_spec.rb

@@ -1,10 +1,6 @@
 require 'spec_helper'
 
-describe Github::Orgs do
-
-  let(:github) { Github.new }
-  let(:user)   { 'peter-murach' }
-  let(:org)   { 'github' }
+describe Github::Orgs, :type => :base do
 
   describe "orgs" do
     context "resource found for a user" do

data/spec/github/repos/collaborators_spec.rb

@@ -1,11 +1,6 @@
 require 'spec_helper'
 
-describe Github::Repos::Collaborators do
-
-  let(:github) { Github.new }
-  let(:user)   { 'peter-murach' }
-  let(:repo)   { 'github' }
-  let(:collaborator) { 'octocat' }
+describe Github::Repos::Collaborators, :type => :base do
 
   describe "collaborators" do
     context "resource found" do

data/spec/github/repos/commits_spec.rb

@@ -1,10 +1,6 @@
 require 'spec_helper'
 
-describe Github::Repos::Commits do
-
-  let(:github) { Github.new }
-  let(:user)   { 'peter-murach' }
-  let(:repo)   { 'github' }
+describe Github::Repos::Commits, :type => :base do
 
   describe "commits" do
     context "resource found" do

data/spec/github/repos/downloads_spec.rb

@@ -1,10 +1,6 @@
 require 'spec_helper'
 
-describe Github::Repos::Downloads do
-
-  let(:github) { Github.new }
-  let(:user)   { 'peter-murach' }
-  let(:repo)   { 'github' }
+describe Github::Repos::Downloads, :type => :base do
 
   it { described_class::VALID_DOWNLOAD_PARAM_NAMES.should_not be_nil }
   it { described_class::REQUIRED_PARAMS.should_not be_nil }

data/spec/github/repos/forks_spec.rb

@@ -1,10 +1,6 @@
 require 'spec_helper'
 
-describe Github::Repos::Forks do
-
-  let(:github) { Github.new }
-  let(:user)   { 'peter-murach' }
-  let(:repo)   { 'github' }
+describe Github::Repos::Forks, :type => :base do
 
   describe "forks" do
 

data/spec/github/repos/hooks_spec.rb

@@ -1,15 +1,10 @@
 require 'spec_helper'
 
-describe Github::Repos::Hooks do
+describe Github::Repos::Hooks, :type => :base do
 
-  let(:hooks_api) { Github::Repos::Hooks }
-  let(:github) { Github.new }
-  let(:user)   { 'peter-murach' }
-  let(:repo)   { 'github' }
-
-  it { hooks_api::VALID_HOOK_PARAM_NAMES.should_not be_nil }
-  it { hooks_api::VALID_HOOK_PARAM_VALUES.should_not be_nil }
-  it { hooks_api::REQUIRED_PARAMS.should_not be_nil }
+  it { described_class::VALID_HOOK_PARAM_NAMES.should_not be_nil }
+  it { described_class::VALID_HOOK_PARAM_VALUES.should_not be_nil }
+  it { described_class::REQUIRED_PARAMS.should_not be_nil }
 
   describe "hooks" do
     context 'check aliases' do

data/spec/github/repos/keys_spec.rb

@@ -1,10 +1,6 @@
 require 'spec_helper'
 
-describe Github::Repos::Keys do
-
-  let(:github) { Github.new }
-  let(:user)   { 'peter-murach'}
-  let(:repo)   { 'github' }
+describe Github::Repos::Keys, :type => :base do
 
   it { described_class::VALID_KEY_PARAM_NAMES.should_not be_nil }
 

data/spec/github/repos/watching_spec.rb

@@ -2,11 +2,7 @@
 
 require 'spec_helper'
 
-describe Github::Repos::Watching do
-
-  let(:github) { Github.new }
-  let(:user)   { 'peter-murach' }
-  let(:repo)   { 'github' }
+describe Github::Repos::Watching, :type => :base do
 
   describe "watchers" do
     before do

data/spec/github/repos_spec.rb

@@ -1,10 +1,6 @@
 require 'spec_helper'
 
-describe Github::Repos do
-
-  let(:github) { Github.new }
-  let(:user)   { 'peter-murach' }
-  let(:repo)   { 'github' }
+describe Github::Repos, :type => :base do
 
   describe "branches" do
     context "resource found" do

data/spec/github/users_spec.rb

@@ -1,9 +1,6 @@
 require 'spec_helper'
 
-describe Github::Users do
-
-  let(:github) { Github.new }
-  let(:user)   { 'peter-murach' }
+describe Github::Users, :type => :base do
 
   before do
     reset_authentication_for github

data/spec/support/base.rb

@@ -6,8 +6,14 @@ module SpecHelpers
         let(:github) { Github.new }
         let(:user)   { 'peter-murach' }
         let(:repo)   { 'github' }
+        let(:org)   { 'github' }
+        let(:collaborator) { 'octocat' }
       end
     end
 
   end
 end # SpecHelpers
+
+RSpec.configure do |conf|
+  conf.include SpecHelpers::Base, :type => :base
+end

metadata

@@ -2,7 +2,7 @@
 name: github_api
 version: !ruby/object:Gem::Version 
   prerelease: 
-  version: 0.3.8
+  version: 0.3.9
 platform: ruby
 authors: 
 - Piotr Murach
@@ -10,7 +10,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2012-01-01 00:00:00 +00:00
+date: 2012-01-03 00:00:00 +00:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -235,6 +235,10 @@ files:
 - spec/fixtures/events/events.json
 - spec/fixtures/git_data/blob.json
 - spec/fixtures/git_data/blob_sha.json
+- spec/fixtures/git_data/commit.json
+- spec/fixtures/git_data/reference.json
+- spec/fixtures/git_data/references.json
+- spec/fixtures/git_data/tag.json
 - spec/fixtures/git_data/tree.json
 - spec/fixtures/issues/comment.json
 - spec/fixtures/issues/comments.json