diff options
author | Jose Ivan Vargas <jvargas@gitlab.com> | 2017-04-28 15:20:58 -0500 |
---|---|---|
committer | Jose Ivan Vargas <jvargas@gitlab.com> | 2017-05-04 13:56:26 -0500 |
commit | ad6ac17c5434f7eb87005dc3603b4ae9409c333f (patch) | |
tree | 9a2a976da09bece8ef6e4c418f55c4535d9334ae | |
parent | 91e8f7702c7781ee8ccbbe9338500e616606af29 (diff) | |
download | gitlab-ce-ad6ac17c5434f7eb87005dc3603b4ae9409c333f.tar.gz |
Added rescue block for the test method for the prometheus service
-rw-r--r-- | changelogs/unreleased/prometheus-integration-test-setting-fix.yml | 4 | ||||
-rw-r--r-- | lib/gitlab/prometheus.rb | 6 | ||||
-rw-r--r-- | spec/models/project_services/prometheus_service_spec.rb | 5 |
3 files changed, 12 insertions, 3 deletions
diff --git a/changelogs/unreleased/prometheus-integration-test-setting-fix.yml b/changelogs/unreleased/prometheus-integration-test-setting-fix.yml new file mode 100644 index 00000000000..c65c682a114 --- /dev/null +++ b/changelogs/unreleased/prometheus-integration-test-setting-fix.yml @@ -0,0 +1,4 @@ +--- +title: Added rescue block for the test method +merge_request: 10994 +author: diff --git a/lib/gitlab/prometheus.rb b/lib/gitlab/prometheus.rb index 62239779454..37afec5e7df 100644 --- a/lib/gitlab/prometheus.rb +++ b/lib/gitlab/prometheus.rb @@ -49,7 +49,11 @@ module Gitlab end def get(url) - handle_response(HTTParty.get(url)) + begin + handle_response(HTTParty.get(url)) + rescue SocketError + raise PrometheusError, "Can't connect to #{url}" + end end def handle_response(response) diff --git a/spec/models/project_services/prometheus_service_spec.rb b/spec/models/project_services/prometheus_service_spec.rb index d15079b686b..5ef1b53be15 100644 --- a/spec/models/project_services/prometheus_service_spec.rb +++ b/spec/models/project_services/prometheus_service_spec.rb @@ -93,11 +93,12 @@ describe PrometheusService, models: true, caching: true do [404, 500].each do |status| context "when Prometheus responds with #{status}" do + body_response = 'QUERY_FAILED' before do - stub_all_prometheus_requests(environment.slug, status: status, body: 'QUERY FAILED!') + stub_all_prometheus_requests(environment.slug, status: status, body: body_response) end - it { is_expected.to eq(success: false, result: %(#{status} - "QUERY FAILED!")) } + it { is_expected.to eq(success: false, result: %(#{status} - \"#{body_response}\")) } end end end |