github_api 0.10.2 → 0.11.0

data/spec/github/repos/releases/list_spec.rb

@@ -0,0 +1,54 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::Repos::Releases, '#list' do
+  let(:owner) { 'peter-murach' }
+  let(:repo)  { 'github' }
+  let(:path)  { "/repos/#{owner}/#{repo}/releases" }
+
+  before {
+    stub_get(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("repos/releases.json") }
+    let(:status) { 200 }
+
+    it { should respond_to :all }
+
+    it { expect { subject.list }.to raise_error(ArgumentError) }
+
+    it "should fail to get resource without repository" do
+      expect { subject.list owner }.to raise_error(ArgumentError)
+    end
+
+    it "should get the resources" do
+      subject.list owner, repo
+      expect(a_get(path)).to have_been_made
+    end
+
+    it_should_behave_like 'an array of resources' do
+      let(:requestable) { subject.list owner, repo }
+    end
+
+    it "should get key information" do
+      keys = subject.list owner, repo
+      expect(keys.first.tag_name).to eq 'v1.0.0'
+    end
+
+    it "should yield to a block" do
+      yielded = []
+      result = subject.list(owner, repo) { |obj| yielded << obj }
+      expect(yielded).to eq result
+    end
+  end
+
+  it_should_behave_like 'request failure' do
+    let(:requestable) { subject.list owner, repo }
+  end
+
+end # list

data/spec/github/request/endpoint_spec.rb

@@ -17,8 +17,6 @@ describe Github::API, 'endpoint' do
   its(:endpoint) { should == endpoint }
 
   it "doesn't truncate endpoint" do
-    expect {
-      api.get_request(path)
-    }.not_to raise_error(WebMock::NetConnectNotAllowedError)
+    expect { api.get_request(path) }.not_to raise_error()
   end
 end

data/spec/github/request/request_spec.rb

@@ -3,10 +3,9 @@
 require 'spec_helper'
 
 describe Github::API, '#request' do
-  let(:token) { "2fdsfdo23fsdf3omkhen34n2jh" }
-  let(:per_page) { 100 }
-  let(:path)  { "/api/v3/repos/GitHub/issues-dev/issues" }
-
+  let(:token)      { "2fdsfdo23fsdf3omkhen34n2jh" }
+  let(:per_page)   { 100 }
+  let(:path)       { "/api/v3/repos/GitHub/issues-dev/issues" }
   let(:url_prefix) { "https://my-company/api/v3/repos/GitHub/issues-dev/issues?access_token=#{token}&page=2&per_page=#{per_page}" }
 
   let(:conn) { Faraday::Connection.new }
@@ -30,9 +29,6 @@ describe Github::API, '#request' do
 
   it 'handles enterprise uri correctly' do
     subject.stub(:connection).and_return conn
-
-    expect {
-      subject.get_request(path)
-    }.not_to raise_error(WebMock::NetConnectNotAllowedError)
+    expect { subject.get_request(path) }.not_to raise_error()
   end
 end

data/spec/github/request_spec.rb

@@ -3,9 +3,9 @@
 require 'spec_helper'
 
 describe Github::Request do
-  let(:github) { Github::API.new  }
-  let(:path) { 'github.api/repos/users' }
-  let(:params) { {} }
+  let(:github)   { Github::API.new  }
+  let(:path)     { 'github.api/repos/users' }
+  let(:params)   { {} }
   let(:response) { double('response').as_null_object }
 
   it "knows how to make get request" do

data/spec/github/response_wrapper_spec.rb

@@ -53,9 +53,9 @@ describe Github::ResponseWrapper do
   end
 
   context "pagination methods" do
-    let(:links) { Github::PageLinks.new({}) }
-    let(:current_api) { stub(:api).as_null_object }
-    let(:iterator) { Github::PageIterator.new(links, current_api) }
+    let(:links)       { Github::PageLinks.new({}) }
+    let(:current_api) { double(:api).as_null_object }
+    let(:iterator)    { Github::PageIterator.new(links, current_api) }
 
     before do
       described_class.stub(:page_iterator).and_return iterator

data/spec/github/s3_uploader_spec.rb

@@ -4,9 +4,8 @@ require 'spec_helper'
 require 'github_api/s3_uploader'
 
 describe Github::S3Uploader do
-
   let(:result) {
-    stub(:resource,
+    double(:resource,
       'path' => 'downloads/octokit/github/droid',
       'acl'  => 'public-read',
       'name' => 'droid',
@@ -31,7 +30,7 @@ describe Github::S3Uploader do
   end
 
   it 'checks for missing parameters' do
-    resource = stub(:resource)
+    resource = double(:resource)
     uploader = Github::S3Uploader.new resource, filename
     expect {
       uploader.send

data/spec/github/search/code_spec.rb

@@ -0,0 +1,38 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::Search, '#code' do
+  let(:query)        { 'tty' }
+  let(:request_path) { '/search/code' }
+
+  before {
+    stub_get(request_path).with(query: {q: query}).
+      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('search/code.json') }
+    let(:status) { 200 }
+
+    it { should respond_to :code }
+
+    it "should get the resources" do
+      subject.code q: query
+      a_get(request_path).with(query: {q: query}).should have_been_made
+    end
+
+    it "should be a response wrapper" do
+      code = subject.code q: query
+      code.should be_a Github::ResponseWrapper
+    end
+
+    it "should get information" do
+      code = subject.code q: query
+      code.total_count.should == 104
+    end
+  end
+end

data/spec/github/search/issues_spec.rb

@@ -0,0 +1,38 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::Search, '#issues' do
+  let(:query)        { 'tty' }
+  let(:request_path) { "/search/issues" }
+
+  before {
+    stub_get(request_path).with(query: {q: query}).
+      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('search/issues.json') }
+    let(:status) { 200 }
+
+    it { should respond_to :issues }
+
+    it "should get the resources" do
+      subject.issues q: query
+      a_get(request_path).with(query: {q: query}).should have_been_made
+    end
+
+    it "should be a response wrapper" do
+      code = subject.issues q: query
+      code.should be_a Github::ResponseWrapper
+    end
+
+    it "should get information" do
+      code = subject.issues q: query
+      code.total_count.should == 280
+    end
+  end
+end

data/spec/github/search/legacy/email_spec.rb

@@ -0,0 +1,34 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::Search::Legacy, '#email' do
+  let(:keyword)      { 'api' }
+  let(:request_path) { "/legacy/user/email/#{keyword}" }
+
+  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('search/email.json') }
+    let(:status) { 200 }
+
+    it "should get the resources" do
+      subject.email keyword
+      a_get(request_path).should have_been_made
+    end
+
+    it "should get the resource through params hash" do
+      subject.email email: keyword
+      a_get(request_path).should have_been_made
+    end
+
+    it "should get user information" do
+      user = subject.email email: keyword
+      user.user.username.should == 'techno'
+    end
+  end
+end

data/spec/github/search/legacy/issues_spec.rb

@@ -0,0 +1,39 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::Search::Legacy, '#issues' do
+  let(:keyword)      { 'api' }
+  let(:request_path) {"/legacy/issues/search/#{owner}/#{repo}/#{state}/#{keyword}"}
+  let(:owner)        { 'peter-murach' }
+  let(:repo)         { 'github' }
+  let(:state)        { 'closed' }
+
+  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('search/issues_legacy.json') }
+    let(:status) { 200 }
+
+    it { expect { subject.issues }.to raise_error(ArgumentError) }
+
+    it "should get the resources" do
+      subject.issues owner, repo, state, keyword
+      a_get(request_path).should have_been_made
+    end
+
+    it "should get the resource through params hash" do
+      subject.issues owner: owner, repo: repo, state: state, keyword: keyword
+      a_get(request_path).should have_been_made
+    end
+
+    it "should get repository information" do
+      issues = subject.issues owner: owner, repo: repo, state: state, keyword: keyword
+      issues.issues.first.user.should == 'ckarbass'
+    end
+  end
+end

data/spec/github/search/legacy/repos_spec.rb

@@ -0,0 +1,36 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::Search::Legacy, '#repos' do
+  let(:keyword)      { 'api' }
+  let(:request_path) { "/legacy/repos/search/#{keyword}" }
+
+  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('search/repos_legacy.json') }
+    let(:status) { 200 }
+
+    it { expect { subject.repos }.to raise_error(ArgumentError) }
+
+    it "should get the resources" do
+      subject.repos keyword
+      a_get(request_path).should have_been_made
+    end
+
+    it "should get the resource through params hash" do
+      subject.repos keyword: keyword
+      a_get(request_path).should have_been_made
+    end
+
+    it "should get repository information" do
+      repos = subject.repos keyword: keyword
+      repos.repositories.first.username.should == 'mathiasbynens'
+    end
+  end
+end

data/spec/github/search/legacy/users_spec.rb

@@ -0,0 +1,36 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::Search::Legacy, '#users' do
+  let(:keyword)      { 'api' }
+  let(:request_path) { "/legacy/user/search/#{keyword}" }
+
+  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('search/users_legacy.json') }
+    let(:status) { 200 }
+
+    it { expect { subject.users }.to raise_error(ArgumentError) }
+
+    it "should get the resources" do
+      subject.users keyword
+      a_get(request_path).should have_been_made
+    end
+
+    it "should get the resource through params hash" do
+      subject.users keyword: keyword
+      a_get(request_path).should have_been_made
+    end
+
+    it "should get repository information" do
+      users = subject.users keyword: keyword
+      users.users.first.username.should == 'techno'
+    end
+  end
+end

data/spec/github/search/repos_spec.rb

@@ -0,0 +1,38 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::Search, '#repos' do
+  let(:query)        { 'tty' }
+  let(:request_path) { "/search/repositories" }
+
+  before {
+    stub_get(request_path).with(query: {q: query}).
+      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('search/repos.json') }
+    let(:status) { 200 }
+
+    it { should respond_to :repos }
+
+    it "should get the resources" do
+      subject.repos q: query
+      a_get(request_path).with(query: {q: query}).should have_been_made
+    end
+
+    it "should be a response wrapper" do
+      code = subject.repos q: query
+      code.should be_a Github::ResponseWrapper
+    end
+
+    it "should get information" do
+      code = subject.repos q: query
+      code.total_count.should == 40
+    end
+  end
+end

data/spec/github/search/users_spec.rb

@@ -0,0 +1,38 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::Search, '#users' do
+  let(:query)        { 'tty' }
+  let(:request_path) { "/search/users" }
+
+  before {
+    stub_get(request_path).with(query: {q: query}).
+      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('search/users.json') }
+    let(:status) { 200 }
+
+    it { should respond_to :users }
+
+    it "should get the resources" do
+      subject.users q: query
+      a_get(request_path).with(query: {q: query}).should have_been_made
+    end
+
+    it "should be a response wrapper" do
+      code = subject.users q: query
+      code.should be_a Github::ResponseWrapper
+    end
+
+    it "should get information" do
+      code = subject.users q: query
+      code.total_count.should == 12
+    end
+  end
+end