github_api 0.12.3 → 0.12.4

data/spec/github/normalizer_spec.rb

@@ -1,34 +0,0 @@
-# encoding: utf-8
-
-require 'spec_helper'
-require 'github_api/core_ext/hash'
-
-describe Github::Normalizer, '#normalize!' do
-  let(:hash) { { 'a' => { :b => { 'c' => 1 }, 'd' => [ 'a', { :e => 2 }] } } }
-
-  let(:klass) {
-    Class.new do
-      include Github::Normalizer
-    end
-  }
-
-  subject(:instance) { klass.new }
-
-  context '#normalize!' do
-    it 'converts hash keys to string' do
-      ['a', 'b', 'c'].each do |key|
-        expect(subject.normalize!(hash).deep_key?(key)).to be_true
-      end
-      [:a, :b, :c].each do |key|
-        expect(subject.normalize!(hash).deep_key?(key)).to be_false
-      end
-    end
-
-    it "should stringify all the keys inside nested hash" do
-      actual = subject.normalize! hash
-      expected = { 'a' => { 'b'=> { 'c' => 1 }, 'd' => [ 'a', { 'e'=> 2 }] } }
-      actual.should be_eql expected
-    end
-  end
-
-end # Github::Normalizer

data/spec/github/page_links_spec.rb

@@ -1,35 +0,0 @@
-# encoding: utf-8
-
-require 'spec_helper'
-
-describe Github::PageLinks do
-  let(:link) {
-    "<https://api.github.com/users/wycats/repos?page=4&per_page=20>; rel=\"next\", <https://api.github.com/users/wycats/repos?page=6&per_page=20>; rel=\"last\", <https://api.github.com/users/wycats/repos?page=1&per_page=20>; rel=\"first\", <https://api.github.com/users/wycats/repos?page=2&per_page=20>; rel=\"prev\""
-  }
-  let(:response_headers)  { { 'Link' => link } }
-
-  let(:first) { "https://api.github.com/users/wycats/repos?page=1&per_page=20" }
-  let(:last)  { "https://api.github.com/users/wycats/repos?page=6&per_page=20" }
-  let(:prev)  { "https://api.github.com/users/wycats/repos?page=2&per_page=20" }
-  let(:nexxt)  { "https://api.github.com/users/wycats/repos?page=4&per_page=20" }
-
-  context 'build page links instance' do
-
-    it 'parses first link successfully' do
-      Github::PageLinks.new(response_headers).first.should eql first
-    end
-
-    it 'parses last link successfully' do
-      Github::PageLinks.new(response_headers).last.should eql last
-    end
-
-    it 'parses next link successfully' do
-      Github::PageLinks.new(response_headers).next.should eql nexxt
-    end
-
-    it 'parses prev link successfully' do
-      Github::PageLinks.new(response_headers).prev.should eql prev
-    end
-  end
-
-end # Github::PageLinks

data/spec/github/paged_request_spec.rb

@@ -1,45 +0,0 @@
-# encoding: utf-8
-
-require 'spec_helper'
-
-describe Github::PagedRequest, '#page_request' do
-  let(:current_api) { Github::Client::Repos.new }
-  let(:path) { "/repositories/"}
-  let(:klass) {
-    klass = Class.new do
-      include Github::PagedRequest
-    end
-  }
-
-  subject(:instance) { klass.new }
-
-  before {
-    instance.stub(:current_api).and_return current_api
-  }
-
-  it { klass.constants.should include :FIRST_PAGE }
-
-  it { klass.constants.should include :PER_PAGE }
-
-  it { klass.constants.should include :NOT_FOUND }
-
-  it { should respond_to(:page_request) }
-
-  it 'calls get_request on api current instance' do
-    current_api.should_receive(:get_request).with(path, {})
-    instance.page_request(path)
-  end
-
-  it 'sets default per_page when only custom page passed' do
-    current_api.should_receive(:get_request).
-      with(path, {'page' => 3, 'per_page' => 30})
-    instance.page_request(path, {'page' => 3, 'per_page' => -1})
-  end
-
-  it 'sets default page when only custom per_page passed' do
-    current_api.should_receive(:get_request).
-      with(path, {'page' => 1, 'per_page' => 33})
-    instance.page_request(path, {'page' => -1, 'per_page' => 33})
-  end
-
-end # Github::PagedRequest

data/spec/github/pagination/iterator/number_spec.rb

@@ -1,123 +0,0 @@
-# encoding: utf-8
-
-require 'spec_helper'
-
-describe Github::PageIterator, 'when number' do
-  let(:header) { {
-    "Link" => "<https://api.github.com/users/wycats/repos?page=4&per_page=20>; rel=\"next\", <https://api.github.com/users/wycats/repos?page=6&per_page=20>; rel=\"last\", <https://api.github.com/users/wycats/repos?page=1&per_page=20>; rel=\"first\", <https://api.github.com/users/wycats/repos?page=2&per_page=20>; rel=\"prev\""
-  } }
-
-  let(:first_link) { "https://api.github.com/users/wycats/repos?page=1&per_page=20" }
-  let(:next_link) { "https://api.github.com/users/wycats/repos?page=4&per_page=20" }
-  let(:prev_link) { "https://api.github.com/users/wycats/repos?page=2&per_page=20" }
-  let(:last_link) { "https://api.github.com/users/wycats/repos?page=6&per_page=20" }
-
-  let(:links)       { Github::PageLinks.new(header) }
-  let(:current_api) { Github::Client::Repos.new }
-  let(:user)        { 'wycats' }
-  let(:response)    { double(:response).as_null_object }
-
-  subject(:iterator) { described_class.new(links, current_api) }
-
-  before {
-    allow(iterator).to receive(:next?).and_return(true)
-    stub_get("/users/#{user}/repos").
-      with(query: { 'page' => '4', 'per_page' => '20'}).
-      to_return(:body => '', :status => 200, :headers => header)
-  }
-
-  it { described_class::ATTRIBUTES.should_not be_nil }
-
-  it { expect(iterator.first_page).to eq(1) }
-
-  it { expect(iterator.first_page_uri).to eq(first_link) }
-
-  it { expect(iterator.next_page).to eq(4)}
-
-  it { expect(iterator.next_page_uri).to eq(next_link) }
-
-  it { expect(iterator.prev_page).to eq(2)}
-
-  it { expect(iterator.prev_page_uri).to eq(prev_link) }
-
-  it { expect(iterator.last_page).to eq(6)}
-
-  it { expect(iterator.last_page_uri).to eq(last_link) }
-
-  context 'next?' do
-    it "returns true when next_page_uri is present" do
-      expect(iterator.next?).to eq(true)
-    end
-  end
-
-  context 'first page request' do
-    it 'returns nil if there are no more pages' do
-      allow(iterator).to receive(:first_page_uri).and_return(false)
-      expect(iterator.first).to be_nil
-    end
-
-    it 'performs request' do
-      expect(iterator).to receive(:page_request).
-        with("/users/#{user}/repos", 'per_page' => 20, 'page' => 1).
-        and_return(response)
-      iterator.first
-    end
-  end
-
-  context 'next page request' do
-    it 'returns nil if there are no more pages' do
-      allow(iterator).to receive(:next?).and_return(false)
-      expect(iterator.next).to be_nil
-    end
-
-    it 'performs request' do
-      expect(iterator).to receive(:page_request).
-        with("/users/#{user}/repos", 'page' => 4,'per_page' => 20).
-        and_return(response)
-      iterator.next
-    end
-  end
-
-  context 'prev page request' do
-    it 'returns nil if there are no more pages' do
-      allow(iterator).to receive(:prev_page_uri).and_return(false)
-      expect(iterator.prev).to be_nil
-    end
-
-    it 'performs request' do
-      expect(iterator).to receive(:page_request).
-        with("/users/#{user}/repos", 'page' => 2,'per_page' => 20).
-        and_return(response)
-      iterator.prev
-    end
-  end
-
-  context 'last page request' do
-    it 'returns nil if there are no more pages' do
-      allow(iterator).to receive(:last_page_uri).and_return(false)
-      expect(iterator.last).to be_nil
-    end
-
-    it 'performs request' do
-      expect(iterator).to receive(:page_request).
-        with("/users/#{user}/repos", 'page' => 6,'per_page' => 20).
-        and_return(response)
-      iterator.last
-    end
-  end
-
-  context 'get_page request' do
-    it 'returns nil if there are no more pages' do
-      allow(iterator).to receive(:first_page_uri).and_return(nil)
-      allow(iterator).to receive(:last_page_uri).and_return(nil)
-      expect(iterator.get_page(5)).to be_nil
-    end
-
-    it 'performs request' do
-      expect(iterator).to receive(:page_request).
-        with("/users/#{user}/repos", 'page' => 2,'per_page' => 20).
-        and_return(response)
-      iterator.get_page(2)
-    end
-  end
-end

data/spec/github/pagination/iterator/sha_spec.rb

@@ -1,66 +0,0 @@
-# encoding: utf-8
-
-require 'spec_helper'
-
-describe Github::PageIterator, 'when sha' do
-  let(:header) { {"Link" => "<https://api.github.com/repos/peter-murach/github/commits?last_sha=d1e503c02fa770859895dd0d12aedefa28b95723&per_page=30&sha=801d80dfd59bf1d2cb30a243799953ab683a3abd&top=801d80dfd59bf1d2cb30a243799953ab683a3abd>; rel=\"next\", <https://api.github.com/repos/peter-murach/github/commits?per_page=30&sha=801d80dfd59bf1d2cb30a243799953ab683a3abd>; rel=\"first\"" }
-  }
-
-  let(:first_link) { "https://api.github.com/repos/peter-murach/github/commits?per_page=30&sha=801d80dfd59bf1d2cb30a243799953ab683a3abd" }
-  let(:next_link)  { "https://api.github.com/repos/peter-murach/github/commits?last_sha=d1e503c02fa770859895dd0d12aedefa28b95723&per_page=30&sha=801d80dfd59bf1d2cb30a243799953ab683a3abd&top=801d80dfd59bf1d2cb30a243799953ab683a3abd" }
-
-  let(:top_sha)  { '801d80dfd59bf1d2cb30a243799953ab683a3abd' }
-  let(:sha)      { '801d80dfd59bf1d2cb30a243799953ab683a3abd' }
-  let(:last_sha) { "d1e503c02fa770859895dd0d12aedefa28b95723"}
-
-  let(:links)       { Github::PageLinks.new(header) }
-  let(:current_api) { Github::Client::Repos.new }
-  let(:user)     { 'wycats' }
-  let(:response) { double(:response).as_null_object }
-
-  subject(:iterator) { described_class.new(links, current_api) }
-
-  before {
-    allow(iterator).to receive(:next?).and_return(true)
-    allow(iterator).to receive(:next_page).and_return(-1)
-    stub_get("/repos/peter-murach/github/commits").
-      to_return(body: '', status: 200, headers: header)
-  }
-
-  it { expect(described_class::ATTRIBUTES).to_not be_nil }
-
-  it { expect(iterator.first_page).to eq(-1) }
-
-  it { expect(iterator.first_page_uri).to eq(first_link) }
-
-  it { expect(iterator.next_page).to eq(-1) }
-
-  it { expect(iterator.next_page_uri).to eq(next_link) }
-
-  it { expect(iterator.prev_page).to eq(-1) }
-
-  it { expect(iterator.prev_page_uri).to be_nil }
-
-  it { expect(iterator.last_page).to eq(-1) }
-
-  it { expect(iterator.last_page_uri).to be_nil }
-
-  context 'first page request' do
-    it 'performs request' do
-      expect(iterator).to receive(:page_request).
-        with("/repos/peter-murach/github/commits",
-          'sha' => 'master', 'per_page' => 30).and_return(response)
-      iterator.first
-    end
-  end
-
-  context 'next page request' do
-    it 'performs request' do
-      expect(iterator).to receive(:page_request).
-        with("/repos/peter-murach/github/commits", 'last_sha' => last_sha,
-          'sha' => last_sha, 'per_page' => 30, 'top' => top_sha).
-        and_return(response)
-      iterator.next
-    end
-  end
-end

data/spec/github/parameter_filter_spec.rb

@@ -1,34 +0,0 @@
-# encoding: utf-8
-
-require 'spec_helper'
-require 'github_api/core_ext/hash'
-
-describe Github::ParameterFilter, '#filter!' do
-  let(:hash) {  { :a => { :b => { :c => 1 } } }  }
-
-  let(:klass) {
-    Class.new do
-      include Github::ParameterFilter
-    end
-  }
-
-  subject(:instance) { klass.new }
-
-  it 'removes unwanted keys from hash' do
-    instance.filter!([:a], hash)
-    expect(hash.deep_key?(:a)).to be_true
-    expect(hash.deep_key?(:b)).to be_false
-    expect(hash.deep_key?(:c)).to be_false
-  end
-
-  it 'recursively filters inputs tree' do
-    instance.filter!([:a, :b], hash)
-    expect(hash.deep_key?(:c)).to be_false
-  end
-
-  it 'filters inputs tree only on top level' do
-    instance.filter!([:a, :b], hash, :recursive => false)
-    expect(hash.deep_key?(:c)).to be_true
-  end
-
-end # Github::ParameterFilter

data/spec/github/params_hash_spec.rb

@@ -1,97 +0,0 @@
-# encoding: utf-8
-
-require 'spec_helper'
-
-RSpec.describe Github::ParamsHash do
-  let(:object) { described_class }
-
-  subject(:params) { object.new(hash) }
-
-  context 'when empty' do
-    let(:hash) { {} }
-
-    it { expect(params.options).to eq({:raw => false}) }
-
-    it { expect(params.data).to eq({}) }
-
-    it { expect(params.accept).to eq(nil) }
-  end
-
-  context 'converts key to string' do
-    let(:hash) { {foo: 123 } }
-
-    it { expect(params['foo']).to eql(123) }
-
-    it { expect(params.data).to eql({"foo" => 123}) }
-  end
-
-  context 'media' do
-    let(:hash) { {media: "raw"} }
-
-    it 'parses media key' do
-      expect(params.media).to eql('application/vnd.github.v3.raw+json')
-    end
-  end
-
-  context 'with accept' do
-    let(:hash) { {accept: "application/json"} }
-
-    it 'overwrites media key' do
-      expect(params.accept).to eql('application/json')
-      expect(params.to_hash).to eq({'accept' => 'application/json'})
-    end
-  end
-
-  context 'without accept' do
-    let(:hash) { {} }
-
-    it 'overwrites media key' do
-      expect(params.accept).to be_nil
-    end
-  end
-
-  context 'when data' do
-    let(:hash) { {data: 'foobar'} }
-
-    it 'extracts data key' do
-      expect(params.data).to eql('foobar')
-      expect(params.to_hash).to eql({'data' => 'foobar'})
-    end
-  end
-
-  context 'when content_type' do
-    let(:hash) { {content_type: 'application/octet-stream'} }
-
-    it 'extracts content_type key' do
-      expect(params.options[:headers]).to eql(hash)
-    end
-  end
-
-  context 'when header' do
-    let(:hash) { {headers: {"X-GitHub-OTP" => "<2FA token>"}} }
-
-    it "sets headers" do
-      expect(params.options[:headers]).to eql({"X-GitHub-OTP" => "<2FA token>" })
-    end
-  end
-
-  context 'merges defaults' do
-    let(:hash) { { :homepage => "https://tty.github.io" }}
-    let(:defaults) {
-      { "homepage" => "https://github.com",
-        "private"  => false}
-    }
-
-    it 'correctly updates values' do
-      subject.merge_default(defaults)
-      expect(params['homepage']).to eql("https://tty.github.io")
-      expect(params['private']).to be_false
-    end
-  end
-
-  context 'strict encode' do
-    let(:hash) { {content: "puts 'hello ruby'"} }
-
-    it { expect(params.strict_encode64('content')).to eql('cHV0cyAnaGVsbG8gcnVieSc=')  }
-  end
-end

data/spec/github/request/endpoint_spec.rb

@@ -1,22 +0,0 @@
-# encoding: utf-8
-
-require 'spec_helper'
-
-describe Github::API, 'endpoint' do
-  let(:endpoint) { "https://my-company/api/v3/" }
-  let(:options) { {:endpoint => endpoint } }
-  let(:path)  { "/repos/GitHub/issues-dev/issues" }
-
-  subject(:api) { described_class.new(options) }
-
-  before {
-    stub_get(path, 'https://my-company/api/v3').
-      to_return(:body => "[]", :status => 200, :headers =>{})
-  }
-
-  its(:endpoint) { should == endpoint }
-
-  it "doesn't truncate endpoint" do
-    expect { api.get_request(path) }.not_to raise_error()
-  end
-end