danger-lgtm 1.0.1 → 1.0.2
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +4 -4
- data/Gemfile.lock +2 -2
- data/lib/lgtm/gem_version.rb +1 -1
- data/lib/lgtm/plugin.rb +13 -3
- data/spec/lgtm_spec.rb +13 -2
- metadata +3 -3
checksums.yaml
CHANGED
@@ -1,7 +1,7 @@
|
|
1
1
|
---
|
2
2
|
SHA1:
|
3
|
-
metadata.gz:
|
4
|
-
data.tar.gz:
|
3
|
+
metadata.gz: fc895c5b5010b4c4a5f4d7447c0f1e8ae6c3e739
|
4
|
+
data.tar.gz: 7e65307505e57e652024c53b27aa9e155f4115f3
|
5
5
|
SHA512:
|
6
|
-
metadata.gz:
|
7
|
-
data.tar.gz:
|
6
|
+
metadata.gz: feabb54a4e5d374387f0d64542565991115236d90a1cd098e54cb68250662368f12694f745f19900b1974a65f9abbec1aeb18836ca13f2251fa44324bfe6c606
|
7
|
+
data.tar.gz: 66d8d3a8f931cad7346f4e3ef563da528b14c561cec252e4c11c4d0b556a3b7f4cb7b6bc3dd4f3317bdbb755f6278f14022c7c1fc2939f802f7be66cb1f2471f
|
data/Gemfile.lock
CHANGED
@@ -1,7 +1,7 @@
|
|
1
1
|
PATH
|
2
2
|
remote: .
|
3
3
|
specs:
|
4
|
-
danger-lgtm (1.0.
|
4
|
+
danger-lgtm (1.0.2)
|
5
5
|
danger-plugin-api (~> 1.0)
|
6
6
|
|
7
7
|
GEM
|
@@ -19,7 +19,7 @@ GEM
|
|
19
19
|
colored2 (3.1.2)
|
20
20
|
cork (0.3.0)
|
21
21
|
colored2 (~> 3.1)
|
22
|
-
danger (5.6.
|
22
|
+
danger (5.6.4)
|
23
23
|
claide (~> 1.0)
|
24
24
|
claide-plugins (>= 0.9.2)
|
25
25
|
colored2 (~> 3.1)
|
data/lib/lgtm/gem_version.rb
CHANGED
data/lib/lgtm/plugin.rb
CHANGED
@@ -39,9 +39,9 @@ module Danger
|
|
39
39
|
|
40
40
|
private
|
41
41
|
|
42
|
+
# returns "<h1 align="center">LGTM</h1>" when ServiceTemporarilyUnavailable.
|
42
43
|
def fetch_image_url(https_image_only: false)
|
43
|
-
|
44
|
-
|
44
|
+
return unless lgtm_post_url
|
45
45
|
lgtm_post_response = process_request(lgtm_post_url) do |req|
|
46
46
|
req['Accept'] = 'application/json'
|
47
47
|
end
|
@@ -67,8 +67,18 @@ module Danger
|
|
67
67
|
end
|
68
68
|
end
|
69
69
|
|
70
|
+
def lgtm_post_url
|
71
|
+
lgtm_post_req = process_request(RANDOM_LGTM_POST_URL)
|
72
|
+
return if lgtm_post_req.code == '503'
|
73
|
+
lgtm_post_req['location']
|
74
|
+
end
|
75
|
+
|
70
76
|
def markdown_template(image_url)
|
71
|
-
|
77
|
+
if image_url.nil?
|
78
|
+
"<h1 align='center'>LGTM</h1>"
|
79
|
+
else
|
80
|
+
"<p align='center'><img src='#{image_url}' alt='LGTM' /></p>"
|
81
|
+
end
|
72
82
|
end
|
73
83
|
end
|
74
84
|
end
|
data/spec/lgtm_spec.rb
CHANGED
@@ -25,13 +25,24 @@ module Danger
|
|
25
25
|
expect(@dangerfile.status_report[:markdowns].length).to eq(1)
|
26
26
|
end
|
27
27
|
|
28
|
+
it 'lgtm with default url is OverQuota' do
|
29
|
+
allow(Net::HTTP).to receive(:start).and_return(mock(code: '503'))
|
30
|
+
|
31
|
+
@lgtm.check_lgtm
|
32
|
+
|
33
|
+
expect(@dangerfile.status_report[:markdowns][0].message)
|
34
|
+
.to eq("<h1 align='center'>LGTM</h1>")
|
35
|
+
end
|
36
|
+
|
28
37
|
def mock(request_url: 'https://lgtm.in/p/sSuI4hm0q',
|
29
|
-
actual_image_url: 'https://example.com/image.jpg'
|
38
|
+
actual_image_url: 'https://example.com/image.jpg',
|
39
|
+
code: '302')
|
30
40
|
double(
|
31
41
|
:[] => request_url,
|
32
42
|
body: JSON.generate(
|
33
43
|
actualImageUrl: actual_image_url
|
34
|
-
)
|
44
|
+
),
|
45
|
+
code: code
|
35
46
|
)
|
36
47
|
end
|
37
48
|
|
metadata
CHANGED
@@ -1,14 +1,14 @@
|
|
1
1
|
--- !ruby/object:Gem::Specification
|
2
2
|
name: danger-lgtm
|
3
3
|
version: !ruby/object:Gem::Version
|
4
|
-
version: 1.0.
|
4
|
+
version: 1.0.2
|
5
5
|
platform: ruby
|
6
6
|
authors:
|
7
7
|
- leonhartX
|
8
8
|
autorequire:
|
9
9
|
bindir: bin
|
10
10
|
cert_chain: []
|
11
|
-
date: 2018-
|
11
|
+
date: 2018-08-01 00:00:00.000000000 Z
|
12
12
|
dependencies:
|
13
13
|
- !ruby/object:Gem::Dependency
|
14
14
|
name: danger-plugin-api
|
@@ -193,7 +193,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
|
|
193
193
|
version: '0'
|
194
194
|
requirements: []
|
195
195
|
rubyforge_project:
|
196
|
-
rubygems_version: 2.5.2
|
196
|
+
rubygems_version: 2.5.2.3
|
197
197
|
signing_key:
|
198
198
|
specification_version: 4
|
199
199
|
summary: Danger Plugin used to post LGTM iamge when there is no violations
|