github_api 0.7.2 → 0.8.0

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

@@ -0,0 +1,37 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::Activity::Starring, '#unstar' do
+  let(:user) { 'peter-murach' }
+  let(:repo) { 'github' }
+  let(:request_path) { "/user/starred/#{user}/#{repo}" }
+
+  after { reset_authentication_for subject }
+
+  context "user authenticated" do
+    context "with correct information" do
+      before do
+        subject.oauth_token = OAUTH_TOKEN
+        stub_delete("/user/starred/#{user}/#{repo}").
+          with(:query => {:access_token => OAUTH_TOKEN}).
+          to_return(:body => "", :status => 204, :headers => {})
+      end
+
+      it "should successfully unstar a repo" do
+        subject.unstar user, repo
+        a_delete(request_path).with(:query => {:access_token => OAUTH_TOKEN}).
+          should have_been_made
+      end
+    end
+  end
+
+  context "user unauthenticated" do
+    it "should fail" do
+      stub_delete(request_path).to_return(:body => "", :status => 401, :headers => {})
+      expect {
+        subject.unstar user, repo
+      }.to raise_error(Github::Error::Unauthorized)
+    end
+  end
+end # unstar

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

@@ -0,0 +1,59 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::Activity::Watching, '#list' do
+  let(:user) { 'peter-murach' }
+  let(:repo) { 'github' }
+  let(:request_path) { "/repos/#{user}/#{repo}/subscribers" }
+  let(:body) { fixture("repos/watchers.json") }
+  let(:status) { 200 }
+
+  after { reset_authentication_for subject }
+
+  before {
+    stub_get(request_path).
+      to_return(:body => body, :status => status, :headers => {})
+  }
+
+  it "should fail to get resource without username" do
+    expect { subject.list }.to raise_error(ArgumentError)
+  end
+
+  it "should yield iterator if block given" do
+    subject.should_receive(:list).with(user, repo).and_yield('github')
+    subject.list(user, repo) { |param| 'github' }
+  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
+    watchers = subject.list user, repo
+    watchers.should be_an Array
+    watchers.should have(1).items
+  end
+
+  it "should return result of mash type" do
+    watchers = subject.list user, repo
+    watchers.first.should be_a Hashie::Mash
+  end
+
+  it "should get watcher information" do
+    watchers = subject.list user, repo
+    watchers.first.login.should == 'octocat'
+  end
+
+  context "fail to find resource" do
+    let(:body)   { '' }
+    let(:status) { 404 }
+
+    it "should return 404 not found message" do
+      expect {
+        subject.list user, repo
+      }.to raise_error(Github::Error::NotFound)
+    end
+  end
+end # list

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

@@ -0,0 +1,37 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::Activity::Watching, '#unwatch' do
+  let(:user) { 'peter-murach' }
+  let(:repo) { 'github' }
+  let(:request_path) { "/user/subscriptions/#{user}/#{repo}" }
+
+  after { reset_authentication_for subject }
+
+  context "user authenticated" do
+    context "with correct information" do
+      before do
+        subject.oauth_token = OAUTH_TOKEN
+        stub_delete(request_path).with(:query => {:access_token => OAUTH_TOKEN}).
+          to_return(:body => "", :status => 204, :headers => {})
+      end
+
+      it "should successfully watch a repo" do
+        subject.unwatch user, repo
+        a_delete(request_path).with(:query => {:access_token => OAUTH_TOKEN}).
+          should have_been_made
+      end
+    end
+  end
+
+  context "user unauthenticated" do
+    it "should fail" do
+      stub_delete(request_path).
+        to_return(:body => "", :status => 401, :headers => {})
+      expect {
+        subject.unwatch user, repo
+      }.to raise_error(Github::Error::Unauthorized)
+    end
+  end
+end # unwatch

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

@@ -0,0 +1,37 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::Activity::Watching, '#watch' do
+  let(:user) { 'peter-murach' }
+  let(:repo) { 'github' }
+  let(:request_path) { "/user/subscriptions/#{user}/#{repo}" }
+
+  after { reset_authentication_for subject }
+
+  context "user authenticated" do
+    context "with correct information" do
+      before do
+        subject.oauth_token = OAUTH_TOKEN
+        stub_put(request_path).
+          with(:query => {:access_token => OAUTH_TOKEN}).
+          to_return(:body => "", :status => 204, :headers => {})
+      end
+
+      it "should successfully watch a repo" do
+        subject.watch user, repo
+        a_put(request_path).with(:query => {:access_token => OAUTH_TOKEN}).
+          should have_been_made
+      end
+    end
+  end
+
+  context "user unauthenticated" do
+    it "should fail" do
+      stub_put(request_path).to_return(:body => "", :status => 401, :headers => {})
+      expect {
+        subject.watch user, repo
+      }.to raise_error(Github::Error::Unauthorized)
+    end
+  end
+end # watch

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

@@ -0,0 +1,53 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::Activity::Watching, '#watched' do
+  let(:user) { 'peter-murach' }
+
+  after { reset_authentication_for subject }
+
+  context "if user unauthenticated" do
+    it "should fail to get resource without username " do
+      stub_get("/user/subscriptions").
+        to_return(:body => '', :status => 401, :headers => {})
+      expect {
+        subject.watched
+      }.to raise_error(Github::Error::Unauthorized)
+    end
+
+    it "should get the resource with username" do
+      stub_get("/users/#{user}/subscriptions").
+        to_return(:body => fixture("repos/watched.json"), :status => 200, :headers => {})
+      subject.watched :user => user
+      a_get("/users/#{user}/subscriptions").should have_been_made
+    end
+  end
+
+  context "if user authenticated" do
+    before do
+      subject.oauth_token = OAUTH_TOKEN
+      stub_get("/user/subscriptions").
+        with(:query => {:access_token => OAUTH_TOKEN}).
+        to_return(:body => fixture("repos/watched.json"), :status => 200, :headers => {})
+    end
+
+    it "should get the resources" do
+      subject.watched
+      a_get("/user/subscriptions").with(:query => {:access_token => OAUTH_TOKEN}).
+        should have_been_made
+    end
+
+    it "should return array of resources" do
+      watched = subject.watched
+      watched.should be_an Array
+      watched.should have(1).items
+    end
+
+    it "should get watched information" do
+      watched = subject.watched
+      watched.first.name.should == 'Hello-World'
+    end
+  end
+end # watched
+

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

@@ -0,0 +1,41 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::Activity::Watching, '#list' do
+  let(:user) { 'peter-murach' }
+  let(:repo) { 'github' }
+  let(:request_path) { "/user/subscriptions/#{user}/#{repo}" }
+
+  after { reset_authentication_for subject }
+
+  context "with username ane reponame passed" do
+    context "this repo is being watched by the user"
+      before do
+        stub_get(request_path).
+          to_return(:body => "", :status => 404,
+                    :headers => {:user_agent => subject.user_agent})
+      end
+
+    it "should return false if resource not found" do
+      watching = subject.watching? user, repo
+      watching.should be_false
+    end
+
+    it "should return true if resoure found" do
+        stub_get(request_path).
+          to_return(:body => "", :status => 200,
+            :headers => {:user_agent => subject.user_agent})
+      watching = subject.watching? user, repo
+      watching.should be_true
+    end
+  end
+
+  context "without username and reponame passed" do
+    it "should fail validation " do
+      expect {
+        subject.watching?(nil, nil)
+      }.to raise_error(ArgumentError)
+    end
+  end
+end # watching?

data/spec/github/authorization_spec.rb

@@ -40,12 +40,12 @@ describe Github::Authorization do
 
     it "should throw an error if no client_id" do
       github.client_id = nil
-      expect { github.auth_code }.should raise_error(ArgumentError)
+      expect { github.auth_code }.to raise_error(ArgumentError)
     end
 
     it "should throw an error if no client_secret" do
       github.client_secret = nil
-      expect { github.auth_code }.should raise_error(ArgumentError)
+      expect { github.auth_code }.to raise_error(ArgumentError)
     end
 
     it "should return authentication token code" do

data/spec/github/issues_spec.rb

@@ -78,7 +78,7 @@ describe Github::Issues do
       it "should raise error if user-name empty" do
         expect {
           github.issues.list_repo nil, repo
-        }.should raise_error(ArgumentError)
+        }.to raise_error(ArgumentError)
       end
 
       it "should get the resources" do

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

@@ -14,6 +14,8 @@ describe Github::Repos::Comments, '#delete' do
         :headers => { :content_type => "application/json; charset=utf-8"})
   }
 
+  after { reset_authentication_for subject }
+
   context "resource deleted successfully" do
     let(:body) { '' }
     let(:status) { 204 }

data/spec/github/repos/delete_spec.rb

@@ -2,7 +2,7 @@
 
 require 'spec_helper'
 
-describe Github::Repos do
+describe Github::Repos, '#delete' do
   let(:user) { 'peter-murach' }
   let(:repo) { 'github' }
   let(:request_path) { "/repos/#{user}/#{repo}" }

data/spec/github/repos/get_spec.rb

@@ -8,8 +8,8 @@ describe Github::Repos, '#get' do
   let(:request_path) { "/repos/#{user}/#{repo}" }
 
   before {
-    stub_get(request_path).
-      to_return(:body => body, :status => status, :headers => {:content_type => "application/json; charset=utf-8"})
+    stub_get(request_path).to_return(:body => body, :status => status,
+      :headers => {:content_type => "application/json; charset=utf-8"})
   }
 
   after { reset_authentication_for subject }

data/spec/github/repos/starring_spec.rb

@@ -2,202 +2,3 @@
 
 require 'spec_helper'
 
-describe Github::Repos::Starring do
-  let(:github) { Github.new }
-  let(:user) { 'peter-murach' }
-  let(:repo) { 'github' }
-
-  after { github.user, github.repo, github.oauth_token = nil, nil, nil }
-
-  describe "#list" do
-    before do
-      stub_get("/repos/#{user}/#{repo}/stargazers").
-        to_return(:body => fixture("repos/stargazers.json"),
-                  :status => 200, :headers => {})
-    end
-
-    it "should fail to get resource without username" do
-      expect {
-        github.repos.starring.list
-      }.to raise_error(ArgumentError)
-    end
-
-    it "should yield iterator if block given" do
-      github.repos.starring.should_receive(:list).
-        with(user, repo).and_yield('github')
-      github.repos.starring.list(user, repo) { |param| 'github' }
-    end
-
-    it "should get the resources" do
-      github.repos.starring.list user, repo
-      a_get("/repos/#{user}/#{repo}/stargazers").should have_been_made
-    end
-
-    it "should return array of resources" do
-      stargazers = github.repos.starring.list user, repo
-      stargazers.should be_an Array
-      stargazers.should have(1).items
-    end
-
-    it "should return result of mash type" do
-      stargazers = github.repos.starring.list user, repo
-      stargazers.first.should be_a Hashie::Mash
-    end
-
-    it "should get watcher information" do
-      stargazers = github.repos.starring.list user, repo
-      stargazers.first.login.should == 'octocat'
-    end
-
-    context "fail to find resource" do
-      before do
-        stub_get("/repos/#{user}/#{repo}/stargazers").
-          to_return(:body => "", :status => 404)
-      end
-
-      it "should return 404 not found message" do
-        expect {
-          github.repos.starring.list user, repo
-        }.to raise_error(Github::Error::NotFound)
-      end
-    end
-  end
-
-  describe "#starred" do
-    context "if user unauthenticated" do
-      it "should fail to get resource without username " do
-        stub_get("/user/starred").
-          to_return(:body => '', :status => 401, :headers => {})
-        expect {
-          github.repos.starring.starred
-        }.to raise_error(Github::Error::Unauthorized)
-      end
-
-      it "should get the resource with username" do
-        stub_get("/users/#{user}/starred").
-          to_return(:body => fixture("repos/starred.json"), :status => 200, :headers => {})
-        github.repos.starring.starred :user => user
-        a_get("/users/#{user}/starred").should have_been_made
-      end
-    end
-
-    context "if user authenticated" do
-      before do
-        github.oauth_token = OAUTH_TOKEN
-        stub_get("/user/starred").
-          with(:query => {:access_token => OAUTH_TOKEN}).
-          to_return(:body => fixture("repos/starred.json"),
-            :status => 200, :headers => {})
-      end
-
-      it "should get the resources" do
-        github.repos.starring.starred
-        a_get("/user/starred").with(:query => {:access_token => OAUTH_TOKEN}).
-          should have_been_made
-      end
-
-      it "should return array of resources" do
-        starred = github.repos.starring.starred
-        starred.should be_an Array
-        starred.should have(1).items
-      end
-
-      it "should get starred information" do
-        starred = github.repos.starring.starred
-        starred.first.name.should == 'Hello-World'
-        starred.first.owner.login.should == 'octocat'
-      end
-    end
-  end # starred
-
-  describe "starring?" do
-    context "with username ane reponame passed" do
-      context "this repo is being watched by the user"
-        before do
-          stub_get("/user/starred/#{user}/#{repo}").
-            to_return(:body => "", :status => 404,
-                      :headers => {:user_agent => github.user_agent})
-        end
-
-      it "should return false if resource not found" do
-        starring = github.repos.starring.starring? user, repo
-        starring.should be_false
-      end
-
-      it "should return true if resoure found" do
-          stub_get("/user/starred/#{user}/#{repo}").
-            to_return(:body => "", :status => 200,
-            :headers => {:user_agent => github.user_agent})
-        starring = github.repos.starring.starring? user, repo
-        starring.should be_true
-      end
-    end
-
-    context "without username and reponame passed" do
-      it "should fail validation " do
-        expect {
-          github.repos.starring.starring?(nil, nil)
-        }.to raise_error(ArgumentError)
-      end
-    end
-  end # starring?
-
-  describe "#star" do
-    context "user authenticated" do
-      context "with correct information" do
-        before do
-          github.oauth_token = OAUTH_TOKEN
-          stub_put("/user/starred/#{user}/#{repo}").
-            with(:query => {:access_token => OAUTH_TOKEN}).
-            to_return(:body => "", :status => 204, :headers => {})
-        end
-
-        it "should successfully star a repo" do
-          github.repos.starring.star user, repo
-          a_put("/user/starred/#{user}/#{repo}").
-            with(:query => {:access_token => OAUTH_TOKEN}).
-            should have_been_made
-        end
-      end
-    end
-
-    context "user unauthenticated" do
-      it "should fail" do
-        stub_put("/user/starred/#{user}/#{repo}").
-          to_return(:body => "", :status => 401, :headers => {})
-        expect {
-          github.repos.starring.star user, repo
-        }.to raise_error(Github::Error::Unauthorized)
-      end
-    end
-  end # star
-
-  describe "#unstar" do
-    context "user authenticated" do
-      context "with correct information" do
-        before do
-          github.oauth_token = OAUTH_TOKEN
-          stub_delete("/user/starred/#{user}/#{repo}?access_token=#{OAUTH_TOKEN}").
-            to_return(:body => "", :status => 204, :headers => {})
-        end
-
-        it "should successfully unstar a repo" do
-          github.repos.starring.unstar user, repo
-          a_delete("/user/starred/#{user}/#{repo}?access_token=#{OAUTH_TOKEN}").
-            should have_been_made
-        end
-      end
-    end
-
-    context "user unauthenticated" do
-      it "should fail" do
-        stub_delete("/user/starred/#{user}/#{repo}").
-          to_return(:body => "", :status => 401, :headers => {})
-        expect {
-          github.repos.starring.unstar user, repo
-        }.to raise_error(Github::Error::Unauthorized)
-      end
-    end
-  end # stop_watching
-
-end # Github::Respos::Starring