Skip to content
This repository was archived by the owner on Jul 19, 2025. It is now read-only.

Commit a4ce396

Browse files
committed
Refactor - making private methods private
1 parent 49af669 commit a4ce396

File tree

1 file changed

+27
-29
lines changed

1 file changed

+27
-29
lines changed

lib/cc/services/github_pull_requests.rb

Lines changed: 27 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,8 @@ def response
4343
# additional information (github-slug, PR number, etc) we can't test much
4444
# else.
4545
def receive_test
46+
setup_http
47+
4648
if config.update_status && config.add_comment
4749
ResponseAggregator.new(receive_test_status, receive_test_comment).response
4850
elsif config.update_status
@@ -52,35 +54,6 @@ def receive_test
5254
end
5355
end
5456

55-
def receive_test_status
56-
setup_http
57-
58-
http_post(base_status_url("0" * 40), "{}")
59-
60-
rescue HTTPError => ex
61-
if ex.status == 422 # response message: "No commit found for SHA"
62-
{ ok: true, message: "OAuth token is valid" }
63-
else ex.status == 401 # response message: "Bad credentials"
64-
{ ok: false, message: ex.message }
65-
end
66-
rescue => ex
67-
{ ok: false, message: ex.message }
68-
end
69-
70-
def receive_test_comment
71-
setup_http
72-
73-
response = http_get(user_url)
74-
if response_includes_repo_scope?(response)
75-
{ ok: true, message: "OAuth token is valid" }
76-
else
77-
{ ok: false, message: "OAuth token requires 'repo' scope to post comments." }
78-
end
79-
80-
rescue => ex
81-
{ ok: false, message: ex.message }
82-
end
83-
8457
def receive_pull_request
8558
setup_http
8659

@@ -118,6 +91,31 @@ def add_comment
11891
end
11992
end
12093

94+
def receive_test_status
95+
http_post(base_status_url("0" * 40), "{}")
96+
97+
rescue HTTPError => ex
98+
if ex.status == 422 # response message: "No commit found for SHA"
99+
{ ok: true, message: "OAuth token is valid" }
100+
else ex.status == 401 # response message: "Bad credentials"
101+
{ ok: false, message: ex.message }
102+
end
103+
rescue => ex
104+
{ ok: false, message: ex.message }
105+
end
106+
107+
def receive_test_comment
108+
response = http_get(user_url)
109+
if response_includes_repo_scope?(response)
110+
{ ok: true, message: "OAuth token is valid" }
111+
else
112+
{ ok: false, message: "OAuth token requires 'repo' scope to post comments." }
113+
end
114+
115+
rescue => ex
116+
{ ok: false, message: ex.message }
117+
end
118+
121119
def comment_present?
122120
response = http_get(comments_url)
123121
comments = JSON.parse(response.body)

0 commit comments

Comments
 (0)