github_api 0.5.1 → 0.5.2
Sign up to get free protection for your applications and to get access to all the features.
- data/features/cassettes/gists/gists/public_all.yml +41 -0
- data/features/cassettes/users/emails/add.yml +44 -0
- data/features/cassettes/users/emails/all.yml +44 -0
- data/features/cassettes/users/emails/delete.yml +40 -0
- data/features/gists.feature +17 -20
- data/features/gists/comments.feature +4 -4
- data/features/git_data/references.feature +6 -6
- data/features/pagination.feature +12 -12
- data/features/repos.feature +12 -12
- data/features/settings.yml +1 -0
- data/features/settings.yml.sample +1 -0
- data/features/step_definitions/common_steps.rb +12 -5
- data/features/support/settings.rb +1 -1
- data/features/support/vcr.rb +1 -0
- data/features/users/emails.feature +37 -0
- data/lib/github_api/connection.rb +2 -1
- data/lib/github_api/gists.rb +6 -7
- data/lib/github_api/issues.rb +6 -4
- data/lib/github_api/request.rb +3 -2
- data/lib/github_api/request/jsonize.rb +46 -0
- data/lib/github_api/request/oauth2.rb +19 -10
- data/lib/github_api/resource.rb +6 -0
- data/lib/github_api/users/emails.rb +3 -2
- data/lib/github_api/utils/url.rb +10 -0
- data/lib/github_api/version.rb +1 -1
- data/spec/github/gists_spec.rb +5 -5
- data/spec/github/issues_spec.rb +7 -6
- data/spec/github/request/jsonize_spec.rb +88 -0
- data/spec/github/request/oauth2_spec.rb +70 -0
- data/spec/github/users/emails_spec.rb +4 -5
- data/spec/github/utils/url_spec.rb +12 -0
- metadata +51 -43
|
@@ -0,0 +1,70 @@
|
|
|
1
|
+
require 'spec_helper'
|
|
2
|
+
|
|
3
|
+
describe Github::Request::OAuth2 do
|
|
4
|
+
include Github::Utils::Url
|
|
5
|
+
|
|
6
|
+
def auth_header(env)
|
|
7
|
+
env[:request_headers]['Authorization']
|
|
8
|
+
end
|
|
9
|
+
|
|
10
|
+
def middleware
|
|
11
|
+
described_class.new(lambda { |env| env }, options)
|
|
12
|
+
end
|
|
13
|
+
|
|
14
|
+
def process(params={}, headers={})
|
|
15
|
+
env = {
|
|
16
|
+
:url => URI('http://example.com/?' + build_query(params)),
|
|
17
|
+
:request_headers => headers
|
|
18
|
+
}
|
|
19
|
+
middleware.call(env)
|
|
20
|
+
end
|
|
21
|
+
|
|
22
|
+
context 'no token configured' do
|
|
23
|
+
let(:options) { nil }
|
|
24
|
+
|
|
25
|
+
it "doesn't add params" do
|
|
26
|
+
result = process(:q => 'query')
|
|
27
|
+
result[:url].query.should eql 'q=query'
|
|
28
|
+
end
|
|
29
|
+
|
|
30
|
+
it "doesn't add headers" do
|
|
31
|
+
auth_header(process).should be_nil
|
|
32
|
+
end
|
|
33
|
+
|
|
34
|
+
it "allows for ad hoc access token" do
|
|
35
|
+
result = process(:q => 'query', :access_token => 'abc123')
|
|
36
|
+
result[:url].query.should eql 'access_token=abc123&q=query'
|
|
37
|
+
end
|
|
38
|
+
|
|
39
|
+
it "creates header for ad hoc access token" do
|
|
40
|
+
result = process(:q => 'query', :access_token => 'abc123')
|
|
41
|
+
auth_header(result).should eql 'Token token="abc123"'
|
|
42
|
+
end
|
|
43
|
+
end
|
|
44
|
+
|
|
45
|
+
context 'default token configured' do
|
|
46
|
+
let(:options) { 'ABC' }
|
|
47
|
+
|
|
48
|
+
it "adds access token to params" do
|
|
49
|
+
result = process(:q => 'query')
|
|
50
|
+
result[:url].query.should eql 'access_token=ABC&q=query'
|
|
51
|
+
end
|
|
52
|
+
|
|
53
|
+
it "creates header for access token" do
|
|
54
|
+
auth_header(process).should eql 'Token token="ABC"'
|
|
55
|
+
end
|
|
56
|
+
|
|
57
|
+
it "overrides default with explicit token" do
|
|
58
|
+
result = process(:q => 'query', :access_token => 'abc123')
|
|
59
|
+
result[:url].query.should eql 'access_token=abc123&q=query'
|
|
60
|
+
auth_header(result).should eql 'Token token="abc123"'
|
|
61
|
+
end
|
|
62
|
+
|
|
63
|
+
it "clears default token with explicit one" do
|
|
64
|
+
result = process(:q => 'query', :access_token => nil)
|
|
65
|
+
result[:url].query.should eql 'q=query&access_token'
|
|
66
|
+
auth_header(result).should be_nil
|
|
67
|
+
end
|
|
68
|
+
end
|
|
69
|
+
|
|
70
|
+
end # Github::Request::OAuth2
|
|
@@ -88,22 +88,21 @@ describe Github::Users::Emails do
|
|
|
88
88
|
|
|
89
89
|
before do
|
|
90
90
|
stub_delete("/user/emails").
|
|
91
|
-
with(:query => { :access_token => "#{OAUTH_TOKEN}"
|
|
92
|
-
to_return(:body => fixture('users/emails.json'),
|
|
93
|
-
:status => 200,
|
|
91
|
+
with(:query => { :access_token => "#{OAUTH_TOKEN}"}).
|
|
92
|
+
to_return(:body => fixture('users/emails.json'), :status => 204,
|
|
94
93
|
:headers => {:content_type => "application/json; charset=utf-8"})
|
|
95
94
|
end
|
|
96
95
|
|
|
97
96
|
it 'extracts request parameters and email data' do
|
|
98
97
|
github.users.emails.should_receive(:delete_request).
|
|
99
|
-
with("/user/emails", { "per_page" => 21, "page" => 1,
|
|
98
|
+
with("/user/emails", { "per_page" => 21, "page" => 1, 'data' => [email] })
|
|
100
99
|
github.users.emails.delete email, params
|
|
101
100
|
end
|
|
102
101
|
|
|
103
102
|
it 'submits request successfully' do
|
|
104
103
|
github.users.emails.delete email
|
|
105
104
|
a_delete("/user/emails").
|
|
106
|
-
with(:query => { :access_token => "#{OAUTH_TOKEN}"
|
|
105
|
+
with(:query => { :access_token => "#{OAUTH_TOKEN}" } ).
|
|
107
106
|
should have_been_made
|
|
108
107
|
end
|
|
109
108
|
end # delete
|
|
@@ -2,6 +2,11 @@ require 'spec_helper'
|
|
|
2
2
|
|
|
3
3
|
describe Github::Utils::Url do
|
|
4
4
|
|
|
5
|
+
def eq_query_to(query)
|
|
6
|
+
parts = query.split('&')
|
|
7
|
+
lambda { |other| (parts & other.split('&')) == parts }
|
|
8
|
+
end
|
|
9
|
+
|
|
5
10
|
it 'escapes correctly' do
|
|
6
11
|
described_class.escape('<html>').should eql '%3Chtml%3E'
|
|
7
12
|
described_class.escape('a space').should eql 'a+space'
|
|
@@ -21,6 +26,13 @@ describe Github::Utils::Url do
|
|
|
21
26
|
it { described_class.parse_query("a+b=%28c%29").should eq 'a b' => '(c)' }
|
|
22
27
|
end
|
|
23
28
|
|
|
29
|
+
context 'builds query strings correctly' do
|
|
30
|
+
it { described_class.build_query("a" => "b").should eq "a=b" }
|
|
31
|
+
it { described_class.build_query("a" => ["b", "c"]).should eq "a=b&a=c" }
|
|
32
|
+
it { described_class.build_query("a" => ["b", "c"]).should eq "a=b&a=c" }
|
|
33
|
+
it { described_class.build_query("a" => 1, "b" => 2).should eq "a=1&b=2"}
|
|
34
|
+
end
|
|
35
|
+
|
|
24
36
|
context 'parse_query_for_param' do
|
|
25
37
|
it 'returns nil if cannot find parameter' do
|
|
26
38
|
described_class.parse_query_for_param("param1=a;param2=b", 'param3').
|
metadata
CHANGED
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
--- !ruby/object:Gem::Specification
|
|
2
2
|
name: github_api
|
|
3
3
|
version: !ruby/object:Gem::Version
|
|
4
|
-
version: 0.5.
|
|
4
|
+
version: 0.5.2
|
|
5
5
|
prerelease:
|
|
6
6
|
platform: ruby
|
|
7
7
|
authors:
|
|
@@ -9,11 +9,11 @@ authors:
|
|
|
9
9
|
autorequire:
|
|
10
10
|
bindir: bin
|
|
11
11
|
cert_chain: []
|
|
12
|
-
date: 2012-05-
|
|
12
|
+
date: 2012-05-20 00:00:00.000000000Z
|
|
13
13
|
dependencies:
|
|
14
14
|
- !ruby/object:Gem::Dependency
|
|
15
15
|
name: hashie
|
|
16
|
-
requirement: &
|
|
16
|
+
requirement: &2153991400 !ruby/object:Gem::Requirement
|
|
17
17
|
none: false
|
|
18
18
|
requirements:
|
|
19
19
|
- - ~>
|
|
@@ -21,21 +21,21 @@ dependencies:
|
|
|
21
21
|
version: 1.2.0
|
|
22
22
|
type: :runtime
|
|
23
23
|
prerelease: false
|
|
24
|
-
version_requirements: *
|
|
24
|
+
version_requirements: *2153991400
|
|
25
25
|
- !ruby/object:Gem::Dependency
|
|
26
26
|
name: faraday
|
|
27
|
-
requirement: &
|
|
27
|
+
requirement: &2153990900 !ruby/object:Gem::Requirement
|
|
28
28
|
none: false
|
|
29
29
|
requirements:
|
|
30
30
|
- - ~>
|
|
31
31
|
- !ruby/object:Gem::Version
|
|
32
|
-
version: 0.
|
|
32
|
+
version: 0.8.0
|
|
33
33
|
type: :runtime
|
|
34
34
|
prerelease: false
|
|
35
|
-
version_requirements: *
|
|
35
|
+
version_requirements: *2153990900
|
|
36
36
|
- !ruby/object:Gem::Dependency
|
|
37
37
|
name: multi_json
|
|
38
|
-
requirement: &
|
|
38
|
+
requirement: &2153990440 !ruby/object:Gem::Requirement
|
|
39
39
|
none: false
|
|
40
40
|
requirements:
|
|
41
41
|
- - ~>
|
|
@@ -43,21 +43,21 @@ dependencies:
|
|
|
43
43
|
version: '1.3'
|
|
44
44
|
type: :runtime
|
|
45
45
|
prerelease: false
|
|
46
|
-
version_requirements: *
|
|
46
|
+
version_requirements: *2153990440
|
|
47
47
|
- !ruby/object:Gem::Dependency
|
|
48
48
|
name: oauth2
|
|
49
|
-
requirement: &
|
|
49
|
+
requirement: &2153989980 !ruby/object:Gem::Requirement
|
|
50
50
|
none: false
|
|
51
51
|
requirements:
|
|
52
52
|
- - ~>
|
|
53
53
|
- !ruby/object:Gem::Version
|
|
54
|
-
version: 0.
|
|
54
|
+
version: '0.7'
|
|
55
55
|
type: :runtime
|
|
56
56
|
prerelease: false
|
|
57
|
-
version_requirements: *
|
|
57
|
+
version_requirements: *2153989980
|
|
58
58
|
- !ruby/object:Gem::Dependency
|
|
59
59
|
name: nokogiri
|
|
60
|
-
requirement: &
|
|
60
|
+
requirement: &2153989520 !ruby/object:Gem::Requirement
|
|
61
61
|
none: false
|
|
62
62
|
requirements:
|
|
63
63
|
- - ~>
|
|
@@ -65,10 +65,10 @@ dependencies:
|
|
|
65
65
|
version: 1.5.2
|
|
66
66
|
type: :runtime
|
|
67
67
|
prerelease: false
|
|
68
|
-
version_requirements: *
|
|
68
|
+
version_requirements: *2153989520
|
|
69
69
|
- !ruby/object:Gem::Dependency
|
|
70
70
|
name: rspec
|
|
71
|
-
requirement: &
|
|
71
|
+
requirement: &2153989060 !ruby/object:Gem::Requirement
|
|
72
72
|
none: false
|
|
73
73
|
requirements:
|
|
74
74
|
- - ! '>='
|
|
@@ -76,10 +76,10 @@ dependencies:
|
|
|
76
76
|
version: '0'
|
|
77
77
|
type: :development
|
|
78
78
|
prerelease: false
|
|
79
|
-
version_requirements: *
|
|
79
|
+
version_requirements: *2153989060
|
|
80
80
|
- !ruby/object:Gem::Dependency
|
|
81
81
|
name: cucumber
|
|
82
|
-
requirement: &
|
|
82
|
+
requirement: &2153988600 !ruby/object:Gem::Requirement
|
|
83
83
|
none: false
|
|
84
84
|
requirements:
|
|
85
85
|
- - ! '>='
|
|
@@ -87,10 +87,10 @@ dependencies:
|
|
|
87
87
|
version: '0'
|
|
88
88
|
type: :development
|
|
89
89
|
prerelease: false
|
|
90
|
-
version_requirements: *
|
|
90
|
+
version_requirements: *2153988600
|
|
91
91
|
- !ruby/object:Gem::Dependency
|
|
92
92
|
name: yajl-ruby
|
|
93
|
-
requirement: &
|
|
93
|
+
requirement: &2153988140 !ruby/object:Gem::Requirement
|
|
94
94
|
none: false
|
|
95
95
|
requirements:
|
|
96
96
|
- - ~>
|
|
@@ -98,10 +98,10 @@ dependencies:
|
|
|
98
98
|
version: 1.1.0
|
|
99
99
|
type: :development
|
|
100
100
|
prerelease: false
|
|
101
|
-
version_requirements: *
|
|
101
|
+
version_requirements: *2153988140
|
|
102
102
|
- !ruby/object:Gem::Dependency
|
|
103
103
|
name: webmock
|
|
104
|
-
requirement: &
|
|
104
|
+
requirement: &2153987680 !ruby/object:Gem::Requirement
|
|
105
105
|
none: false
|
|
106
106
|
requirements:
|
|
107
107
|
- - ~>
|
|
@@ -109,10 +109,10 @@ dependencies:
|
|
|
109
109
|
version: 1.8.0
|
|
110
110
|
type: :development
|
|
111
111
|
prerelease: false
|
|
112
|
-
version_requirements: *
|
|
112
|
+
version_requirements: *2153987680
|
|
113
113
|
- !ruby/object:Gem::Dependency
|
|
114
114
|
name: vcr
|
|
115
|
-
requirement: &
|
|
115
|
+
requirement: &2153987220 !ruby/object:Gem::Requirement
|
|
116
116
|
none: false
|
|
117
117
|
requirements:
|
|
118
118
|
- - ~>
|
|
@@ -120,10 +120,10 @@ dependencies:
|
|
|
120
120
|
version: 2.1.0
|
|
121
121
|
type: :development
|
|
122
122
|
prerelease: false
|
|
123
|
-
version_requirements: *
|
|
123
|
+
version_requirements: *2153987220
|
|
124
124
|
- !ruby/object:Gem::Dependency
|
|
125
125
|
name: simplecov
|
|
126
|
-
requirement: &
|
|
126
|
+
requirement: &2154003140 !ruby/object:Gem::Requirement
|
|
127
127
|
none: false
|
|
128
128
|
requirements:
|
|
129
129
|
- - ~>
|
|
@@ -131,43 +131,43 @@ dependencies:
|
|
|
131
131
|
version: 0.6.1
|
|
132
132
|
type: :development
|
|
133
133
|
prerelease: false
|
|
134
|
-
version_requirements: *
|
|
134
|
+
version_requirements: *2154003140
|
|
135
135
|
- !ruby/object:Gem::Dependency
|
|
136
136
|
name: guard
|
|
137
|
-
requirement: &
|
|
137
|
+
requirement: &2154002760 !ruby/object:Gem::Requirement
|
|
138
138
|
none: false
|
|
139
139
|
requirements:
|
|
140
|
-
- -
|
|
140
|
+
- - ! '>='
|
|
141
141
|
- !ruby/object:Gem::Version
|
|
142
|
-
version: 0
|
|
142
|
+
version: '0'
|
|
143
143
|
type: :development
|
|
144
144
|
prerelease: false
|
|
145
|
-
version_requirements: *
|
|
145
|
+
version_requirements: *2154002760
|
|
146
146
|
- !ruby/object:Gem::Dependency
|
|
147
147
|
name: guard-rspec
|
|
148
|
-
requirement: &
|
|
148
|
+
requirement: &2154002300 !ruby/object:Gem::Requirement
|
|
149
149
|
none: false
|
|
150
150
|
requirements:
|
|
151
|
-
- -
|
|
151
|
+
- - ! '>='
|
|
152
152
|
- !ruby/object:Gem::Version
|
|
153
|
-
version: 0
|
|
153
|
+
version: '0'
|
|
154
154
|
type: :development
|
|
155
155
|
prerelease: false
|
|
156
|
-
version_requirements: *
|
|
156
|
+
version_requirements: *2154002300
|
|
157
157
|
- !ruby/object:Gem::Dependency
|
|
158
158
|
name: guard-cucumber
|
|
159
|
-
requirement: &
|
|
159
|
+
requirement: &2154001880 !ruby/object:Gem::Requirement
|
|
160
160
|
none: false
|
|
161
161
|
requirements:
|
|
162
|
-
- -
|
|
162
|
+
- - ! '>='
|
|
163
163
|
- !ruby/object:Gem::Version
|
|
164
|
-
version: 0
|
|
164
|
+
version: '0'
|
|
165
165
|
type: :development
|
|
166
166
|
prerelease: false
|
|
167
|
-
version_requirements: *
|
|
167
|
+
version_requirements: *2154001880
|
|
168
168
|
- !ruby/object:Gem::Dependency
|
|
169
169
|
name: rake
|
|
170
|
-
requirement: &
|
|
170
|
+
requirement: &2154001460 !ruby/object:Gem::Requirement
|
|
171
171
|
none: false
|
|
172
172
|
requirements:
|
|
173
173
|
- - ! '>='
|
|
@@ -175,10 +175,10 @@ dependencies:
|
|
|
175
175
|
version: '0'
|
|
176
176
|
type: :development
|
|
177
177
|
prerelease: false
|
|
178
|
-
version_requirements: *
|
|
178
|
+
version_requirements: *2154001460
|
|
179
179
|
- !ruby/object:Gem::Dependency
|
|
180
180
|
name: bundler
|
|
181
|
-
requirement: &
|
|
181
|
+
requirement: &2154001040 !ruby/object:Gem::Requirement
|
|
182
182
|
none: false
|
|
183
183
|
requirements:
|
|
184
184
|
- - ! '>='
|
|
@@ -186,7 +186,7 @@ dependencies:
|
|
|
186
186
|
version: '0'
|
|
187
187
|
type: :development
|
|
188
188
|
prerelease: false
|
|
189
|
-
version_requirements: *
|
|
189
|
+
version_requirements: *2154001040
|
|
190
190
|
description: ! ' Ruby wrapper that supports all of the GitHub API v3 methods(nearly
|
|
191
191
|
200). It''s build in a modular way, that is, you can either instantiate the whole
|
|
192
192
|
api wrapper Github.new or use parts of it e.i. Github::Repos.new if working solely
|
|
@@ -224,6 +224,9 @@ files:
|
|
|
224
224
|
- features/cassettes/repos/list.yml
|
|
225
225
|
- features/cassettes/repos/tags.yml
|
|
226
226
|
- features/cassettes/repos/teams.yml
|
|
227
|
+
- features/cassettes/users/emails/add.yml
|
|
228
|
+
- features/cassettes/users/emails/all.yml
|
|
229
|
+
- features/cassettes/users/emails/delete.yml
|
|
227
230
|
- features/error_codes.feature
|
|
228
231
|
- features/gists/comments.feature
|
|
229
232
|
- features/gists.feature
|
|
@@ -241,6 +244,7 @@ files:
|
|
|
241
244
|
- features/support/helpers.rb
|
|
242
245
|
- features/support/settings.rb
|
|
243
246
|
- features/support/vcr.rb
|
|
247
|
+
- features/users/emails.feature
|
|
244
248
|
- lib/github_api/api/actions.rb
|
|
245
249
|
- lib/github_api/api.rb
|
|
246
250
|
- lib/github_api/api_factory.rb
|
|
@@ -304,8 +308,10 @@ files:
|
|
|
304
308
|
- lib/github_api/repos/watching.rb
|
|
305
309
|
- lib/github_api/repos.rb
|
|
306
310
|
- lib/github_api/request/basic_auth.rb
|
|
311
|
+
- lib/github_api/request/jsonize.rb
|
|
307
312
|
- lib/github_api/request/oauth2.rb
|
|
308
313
|
- lib/github_api/request.rb
|
|
314
|
+
- lib/github_api/resource.rb
|
|
309
315
|
- lib/github_api/response/helpers.rb
|
|
310
316
|
- lib/github_api/response/jsonize.rb
|
|
311
317
|
- lib/github_api/response/mashify.rb
|
|
@@ -438,6 +444,8 @@ files:
|
|
|
438
444
|
- spec/github/repos/pub_sub_hubbub_spec.rb
|
|
439
445
|
- spec/github/repos/watching_spec.rb
|
|
440
446
|
- spec/github/repos_spec.rb
|
|
447
|
+
- spec/github/request/jsonize_spec.rb
|
|
448
|
+
- spec/github/request/oauth2_spec.rb
|
|
441
449
|
- spec/github/request_spec.rb
|
|
442
450
|
- spec/github/response/helpers_spec.rb
|
|
443
451
|
- spec/github/result_spec.rb
|
|
@@ -463,7 +471,7 @@ files:
|
|
|
463
471
|
homepage: https://github.com/peter-murach/github
|
|
464
472
|
licenses: []
|
|
465
473
|
post_install_message: ! "\n--------------------------------------------------------------------------------\nThank
|
|
466
|
-
you for installing github_api-0.5.
|
|
474
|
+
you for installing github_api-0.5.2.\n\n*NOTE*: Version 0.5.0 introduces breaking
|
|
467
475
|
changes to the way github api is queried.\nThe interface has been rewritten to be
|
|
468
476
|
more consistent with REST verbs that\ninteract with GitHub hypermedia resources.
|
|
469
477
|
Thus, to list resources 'list' or 'all'\nverbs are used, to retrieve individual
|