diff options
author | Grzegorz Bizon <grzegorz@gitlab.com> | 2017-06-22 16:35:05 +0000 |
---|---|---|
committer | Grzegorz Bizon <grzegorz@gitlab.com> | 2017-06-22 16:35:05 +0000 |
commit | 8fe5602c1a4c3ca1815d31e6845dd3d3c2895ec9 (patch) | |
tree | ad77c8644ac9e237429193148deb380423be572a /spec/controllers/projects/prometheus_controller_spec.rb | |
parent | 3bbf0898c1aae90fdd030e41019b9c7b1dbb6215 (diff) | |
parent | 97c42df3b804a37e659c3cda6bd8a52570f31366 (diff) | |
download | gitlab-ce-8fe5602c1a4c3ca1815d31e6845dd3d3c2895ec9.tar.gz |
Merge branch '28717-additional-metrics-review-branch' into 'master'
Support additional prometheus metrics - review branch
Closes #28717
See merge request !11712
Diffstat (limited to 'spec/controllers/projects/prometheus_controller_spec.rb')
-rw-r--r-- | spec/controllers/projects/prometheus_controller_spec.rb | 59 |
1 files changed, 59 insertions, 0 deletions
diff --git a/spec/controllers/projects/prometheus_controller_spec.rb b/spec/controllers/projects/prometheus_controller_spec.rb new file mode 100644 index 00000000000..eddf7275975 --- /dev/null +++ b/spec/controllers/projects/prometheus_controller_spec.rb @@ -0,0 +1,59 @@ +require('spec_helper') + +describe Projects::PrometheusController do + let(:user) { create(:user) } + let!(:project) { create(:empty_project) } + + let(:prometheus_service) { double('prometheus_service') } + + before do + allow(controller).to receive(:project).and_return(project) + allow(project).to receive(:prometheus_service).and_return(prometheus_service) + + project.add_master(user) + sign_in(user) + end + + describe 'GET #active_metrics' do + context 'when prometheus metrics are enabled' do + context 'when data is not present' do + before do + allow(prometheus_service).to receive(:matched_metrics).and_return({}) + end + + it 'returns no content response' do + get :active_metrics, project_params(format: :json) + + expect(response).to have_http_status(204) + end + end + + context 'when data is available' do + let(:sample_response) { { some_data: 1 } } + + before do + allow(prometheus_service).to receive(:matched_metrics).and_return(sample_response) + end + + it 'returns no content response' do + get :active_metrics, project_params(format: :json) + + expect(response).to have_http_status(200) + expect(json_response).to eq(sample_response.deep_stringify_keys) + end + end + + context 'when requesting non json response' do + it 'returns not found response' do + get :active_metrics, project_params + + expect(response).to have_http_status(404) + end + end + end + end + + def project_params(opts = {}) + opts.reverse_merge(namespace_id: project.namespace, project_id: project) + end +end |