github_api 0.9.7 → 0.10.0

data/spec/fixtures/repos/contribs.json

@@ -0,0 +1,20 @@
+[
+  {
+    "author": {
+      "login": "octocat",
+      "id": 1,
+      "avatar_url": "https://github.com/images/error/octocat_happy.gif",
+      "gravatar_id": "somehexcode",
+      "url": "https://api.github.com/users/octocat"
+    },
+    "total": 135,
+    "weeks": [
+      {
+        "w": "1367712000",
+        "a": 6898,
+        "d": 77,
+        "c": 10
+      }
+    ]
+  }
+]

data/spec/fixtures/repos/frequency.json

@@ -0,0 +1,7 @@
+[
+  [
+    1302998400,
+    1124,
+    -435
+  ]
+]

data/spec/fixtures/repos/participation.json

@@ -0,0 +1,110 @@
+{
+  "all": [
+    11,
+    21,
+    15,
+    2,
+    8,
+    1,
+    8,
+    23,
+    17,
+    21,
+    11,
+    10,
+    33,
+    91,
+    38,
+    34,
+    22,
+    23,
+    32,
+    3,
+    43,
+    87,
+    71,
+    18,
+    13,
+    5,
+    13,
+    16,
+    66,
+    27,
+    12,
+    45,
+    110,
+    117,
+    13,
+    8,
+    18,
+    9,
+    19,
+    26,
+    39,
+    12,
+    20,
+    31,
+    46,
+    91,
+    45,
+    10,
+    24,
+    9,
+    29,
+    7
+  ],
+  "owner": [
+    3,
+    2,
+    3,
+    0,
+    2,
+    0,
+    5,
+    14,
+    7,
+    9,
+    1,
+    5,
+    0,
+    48,
+    19,
+    2,
+    0,
+    1,
+    10,
+    2,
+    23,
+    40,
+    35,
+    8,
+    8,
+    2,
+    10,
+    6,
+    30,
+    0,
+    2,
+    9,
+    53,
+    104,
+    3,
+    3,
+    10,
+    4,
+    7,
+    11,
+    21,
+    4,
+    4,
+    22,
+    26,
+    63,
+    11,
+    2,
+    14,
+    1,
+    10,
+    3
+  ]
+}

data/spec/fixtures/repos/punch_card.json

@@ -0,0 +1,17 @@
+[
+  [
+    0,
+    0,
+    5
+  ],
+  [
+    0,
+    1,
+    43
+  ],
+  [
+    0,
+    2,
+    21
+  ]
+]

data/spec/github/arguments/parse_spec.rb

@@ -17,7 +17,7 @@ describe Github::Arguments, '#parse' do
 
     it { should == object }
 
-    its(:params) { should == params }
+    its(:params) { should == {"page" => 23} }
 
     context 'sets parameters' do
       it { subject.api.user.should == 'peter-murach' }
@@ -63,6 +63,6 @@ describe Github::Arguments, '#parse' do
     let(:required) { [] }
     let(:arguments) { [params] }
 
-    its(:params) { should == params }
+    its(:params) { should == {"page" => 23} }
   end
 end

data/spec/github/core_ext/deep_merge_spec.rb

@@ -0,0 +1,23 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Hash, 'deep_merge' do
+
+  let(:hash) { { :a => 'a', :b => "b", :c => {:c1 => "c1", :c2 => "c2", :c3 => {:d1 => "d1"} } } }
+  let(:other_hash) { { :a => 1, :b => "b", :c => {:c1 => 2, :c2 => "c2", :c3 => {:d1 => "d1", :d2 => "d2"} } } }
+
+  subject { hash.deep_merge(other_hash) }
+
+  it { expect(subject[:a]).to eql(1) }
+
+  it { expect(subject[:b]).to eql("b") }
+
+  it { expect(subject[:c][:c1]).to eql(2) }
+
+  it { expect(subject[:c][:c2]).to eql("c2") }
+
+  it { expect(subject[:c][:c3][:d1]).to eql("d1") }
+
+  it { expect(subject[:c][:c3][:d2]).to eql("d2") }
+end

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

@@ -49,6 +49,12 @@ describe Github::GitData::Trees, '#create' do
       tree_sha.should be_a Github::ResponseWrapper
     end
 
+    it "should not erase the tree data while evaluating params" do
+      original_tree = inputs['tree'].dup
+      tree_sha = subject.create user, repo, inputs
+      inputs['tree'].should == original_tree
+    end
+
     it "should get the tree information" do
       tree_sha = subject.create user, repo, inputs
       tree_sha.sha.should == sha

data/spec/github/gitignore/get_spec.rb

@@ -42,7 +42,7 @@ describe Github::Gitignore, '#get' do
     let(:accept) { 'application/vnd.github.raw' }
 
     it "should get the resource" do
-      subject.get template, 'mime' => 'application/vnd.github.raw'
+      subject.get template, 'accept' => 'application/vnd.github.raw'
       a_get(request_path).should have_been_made
     end
   end

data/spec/github/mime_type_spec.rb

@@ -2,69 +2,38 @@ require 'spec_helper'
 
 describe Github::MimeType do
 
-  let(:github) { Github.new }
+  let(:object) { Class.new.extend(described_class) }
 
-  it "should lookup mime type for :full" do
-    github.lookup_mime(:full).should == 'full+json'
-  end
-
-  it "should lookup mime type for :html" do
-    github.lookup_mime(:html).should == 'html+json'
-  end
-
-  it "should lookup mime type for :html" do
-    github.lookup_mime(:text).should == 'text+json'
-  end
+  context 'lookup' do
+    it 'retrieves media type' do
+      expect(object.lookup_media('text')).to eql('text+json')
+    end
 
-  it "should lookup mime type for :raw" do
-    github.lookup_mime(:raw).should == 'raw+json'
+    it 'raises error on unkonwn media type' do
+      expect { object.lookup_media('unknown') }.to raise_error(ArgumentError)
+    end
   end
 
-  it "should default to json if no parameters given" do
-    Github.should_receive(:parse).and_return 'application/json'
-    Github.parse.should == 'application/json'
-  end
+  context 'parse' do
+    it 'accepts text' do
+      expect(object.parse('text')).to eql('application/vnd.github.v3.text+json')
+    end
 
-  it "should accept header for 'issue' request" do
-    Github.should_receive(:parse).with(:issue, :full).
-      and_return 'application/vnd.github-issue.full+json'
-    Github.parse(:issue, :full).should == 'application/vnd.github-issue.full+json'
-  end
+    it 'accepts text+json' do
+      expect(object.parse('text+json')).to eql('application/vnd.github.v3.text+json')
+    end
 
-  it "should accept header for 'isssue comment' request" do
-    Github.should_receive(:parse).with(:issue_comment, :full).
-      and_return 'application/vnd.github-issuecomment.full+json'
-    Github.parse(:issue_comment, :full).should == 'application/vnd.github-issuecomment.full+json'
-  end
+    it 'accepts v3.text' do
+      expect(object.parse('v3.text')).to eql('application/vnd.github.v3.text+json')
+    end
 
-  it "should accept header for 'commit comment' request" do
-    Github.should_receive(:parse).with(:commit_comment, :full).
-      and_return 'application/vnd.github-commitcomment.full+json'
-    Github.parse(:commit_comment, :full).should == 'application/vnd.github-commitcomment.full+json'
-  end
-
-  it "should accept header for 'pull requst' request" do
-    Github.should_receive(:parse).with(:pull_request, :full).
-      and_return 'application/vnd.github-pull.full+json'
-    Github.parse(:pull_request, :full).should == 'application/vnd.github-pull.full+json'
-  end
-
-  it "should accept header for 'pull comment' request" do
-    Github.should_receive(:parse).with(:pull_comment, :full).
-      and_return 'application/vnd.github-pullcomment.full+json'
-    Github.parse(:pull_comment, :full).should == 'application/vnd.github-pullcomment.full+json'
-  end
-
-  it "should accept header for 'gist comment' request" do
-    Github.should_receive(:parse).with(:gist_comment, :full).
-      and_return 'application/vnd.github-gistcomment.full+json'
-    Github.parse(:gist_comment, :full).should == 'application/vnd.github-gistcomment.full+json'
-  end
+    it 'accepts v3.text+json' do
+      expect(object.parse('v3.text+json')).to eql('application/vnd.github.v3.text+json')
+    end
 
-  it "should accept header for 'blog' request" do
-    Github.should_receive(:parse).with(:blob, :blob).
-      and_return 'application/vnd.github-blob.raw'
-    Github.parse(:blob, :blob).should == 'application/vnd.github-blob.raw'
+    it 'accpets .v3.text' do
+      expect(object.parse('.v3.text')).to eql('application/vnd.github.v3.text+json')
+    end
   end
 
 end # Github::MimeType

data/spec/github/params_hash_spec.rb

@@ -0,0 +1,64 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::ParamsHash do
+  let(:object) { described_class }
+
+  subject { object.new(hash) }
+
+  context 'converts key to string' do
+    let(:hash) { {:foo => 123 }}
+
+    it { expect(subject).to be_a(Github::ParamsHash)}
+
+    it { expect(subject['foo']).to eql(123) }
+
+    it { expect(subject.data).to eql({"foo" => 123}) }
+  end
+
+  context 'media' do
+    let(:hash) { {:media => "raw"} }
+
+    it 'parses media key' do
+      expect(subject.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(subject.accept).to eql('application/json')
+    end
+  end
+
+  context 'without accept' do
+    let(:hash) { {} }
+
+    it 'overwrites media key' do
+      expect(subject.accept).to be_nil
+    end
+  end
+
+  context 'extract data' do
+    let(:hash) { {:data => 'foobar'} }
+
+    it { expect(subject.data).to eql('foobar') }
+  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(subject['homepage']).to eql("https://tty.github.io")
+      expect(subject['private']).to be_false
+    end
+  end
+
+end

data/spec/github/pull_requests/list_spec.rb

@@ -28,7 +28,7 @@ describe Github::PullRequests, '#list' do
 
     it "should get the resources" do
       subject.list user, repo, inputs
-      a_get(request_path).with(:query => inputs).should have_been_made
+      a_get(request_path).with(:query => inputs.except('unrelated')).should have_been_made
     end
 
     it_should_behave_like 'an array of resources' do

data/spec/github/repos/statistics/code_frequency_spec.rb

@@ -0,0 +1,25 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::Repos::Statistics, '#code_frequency' do
+  let(:user) { 'peter-murach' }
+  let(:repo) { 'github' }
+  let(:request_path) { "/repos/#{user}/#{repo}/stats/code_frequency" }
+
+  before {
+    stub_get(request_path).to_return(:body => body)
+  }
+
+  context "resource found" do
+    let(:body) { fixture('repos/frequency.json') }
+    let(:status) { 200 }
+
+    it { expect { subject.code_frequency }.to raise_error(ArgumentError) }
+
+    it "should get the resources" do
+      subject.code_frequency user, repo
+      a_get(request_path).should have_been_made
+    end
+  end
+end

data/spec/github/repos/statistics/commit_activity_spec.rb

@@ -0,0 +1,29 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::Repos::Statistics, '#commit_activity' do
+  let(:user) { 'peter-murach' }
+  let(:repo) { 'github' }
+  let(:request_path) { "/repos/#{user}/#{repo}/stats/commit_activity" }
+
+  before {
+    stub_get(request_path).to_return(:body => body)
+  }
+
+  context "resource found" do
+    let(:body) { fixture('repos/commit_activity.json') }
+    let(:status) { 200 }
+
+    it { expect { subject.commit_activity }.to raise_error(ArgumentError) }
+
+    it "should get the resources" do
+      subject.commit_activity user, repo
+      a_get(request_path).should have_been_made
+    end
+
+    it_should_behave_like 'an array of resources' do
+      let(:requestable) { subject.commit_activity user, repo }
+    end
+  end
+end