Skip to content

Commit f25d228

Browse files
committed
Catch Faraday::ConnectionFailed, too
1 parent deecc07 commit f25d228

2 files changed

Lines changed: 7 additions & 12 deletions

File tree

service_broker/github_service_helper.rb

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ def create_github_repo(name)
2828
# error due to unknown reason, pass the original error message upstream
2929
raise GithubServiceHelper::GithubError.new("GitHub returned an error - #{e.message}")
3030
end
31-
rescue Faraday::Error::TimeoutError
31+
rescue Faraday::Error::TimeoutError, Faraday::ConnectionFailed
3232
raise GithubServiceHelper::GithubUnreachableError
3333
end
3434

@@ -40,7 +40,7 @@ def delete_github_repo(name)
4040
octokit_client.delete_repository(full_repo_name(name))
4141
rescue Octokit::Error => e
4242
raise GithubServiceHelper::GithubError.new("GitHub returned an error - #{e.message}")
43-
rescue Faraday::Error::TimeoutError
43+
rescue Faraday::Error::TimeoutError, Faraday::ConnectionFailed
4444
raise GithubServiceHelper::GithubUnreachableError
4545
end
4646
end
@@ -88,7 +88,7 @@ def get_deploy_keys(full_repo_name)
8888
raise GithubServiceHelper::GithubResourceNotFoundError
8989
rescue Octokit::Error => e
9090
raise GithubServiceHelper::GithubError.new("GitHub returned an error - #{e.message}")
91-
rescue Faraday::Error::TimeoutError
91+
rescue Faraday::Error::TimeoutError, Faraday::ConnectionFailed
9292
raise GithubServiceHelper::GithubUnreachableError
9393
end
9494
end
@@ -98,7 +98,7 @@ def add_deploy_key(deploy_key_title, full_repo_name, public_key)
9898
octokit_client.add_deploy_key(full_repo_name, deploy_key_title, public_key)
9999
rescue Octokit::Error => e
100100
raise GithubServiceHelper::GithubError.new("GitHub returned an error - #{e.message}")
101-
rescue Faraday::Error::TimeoutError
101+
rescue Faraday::Error::TimeoutError, Faraday::ConnectionFailed
102102
raise GithubServiceHelper::GithubUnreachableError
103103
end
104104
end
@@ -108,7 +108,7 @@ def remove_deploy_key(full_repo_name, deploy_key_id)
108108
octokit_client.remove_deploy_key(full_repo_name, deploy_key_id)
109109
rescue Octokit::Error => e
110110
raise GithubServiceHelper::GithubError.new("GitHub returned an error - #{e.message}")
111-
rescue Faraday::Error::TimeoutError
111+
rescue Faraday::Error::TimeoutError, Faraday::ConnectionFailed
112112
raise GithubServiceHelper::GithubUnreachableError
113113
end
114114
end
@@ -129,4 +129,3 @@ def octokit_client
129129
::Octokit::Client.new(access_token: @access_token)
130130
end
131131
end
132-

service_broker/test/test_helper.rb

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,13 @@
1-
require 'codeclimate-test-reporter'
2-
CodeClimate::TestReporter.start
3-
41
ENV['RACK_ENV'] = 'test'
2+
53
require 'minitest/autorun'
64
require 'minitest/spec'
75
require 'rack/test'
86
require 'mocha/setup'
97
require 'webmock/minitest'
108
require 'pry'
119

12-
WebMock.disable_net_connect!(allow: 'codeclimate.com')
13-
1410
SETTINGS_FILENAME = "test/config/settings.yml"
1511

1612
require File.expand_path '../../service_broker_app.rb', __FILE__
17-
require File.expand_path '../../github_service_helper.rb', __FILE__
13+
require File.expand_path '../../github_service_helper.rb', __FILE__

0 commit comments

Comments
 (0)