diff options
Diffstat (limited to 'spec/requests/api/graphql/project/alert_management')
3 files changed, 63 insertions, 9 deletions
diff --git a/spec/requests/api/graphql/project/alert_management/alert/todos_spec.rb b/spec/requests/api/graphql/project/alert_management/alert/todos_spec.rb new file mode 100644 index 00000000000..3a9077061ad --- /dev/null +++ b/spec/requests/api/graphql/project/alert_management/alert/todos_spec.rb @@ -0,0 +1,51 @@ +# frozen_string_literal: true + +require 'spec_helper' + +RSpec.describe 'getting Alert Management Alert Assignees' do + include GraphqlHelpers + + let_it_be(:project) { create(:project) } + let_it_be(:current_user) { create(:user) } + let_it_be(:alert) { create(:alert_management_alert, project: project) } + let_it_be(:other_alert) { create(:alert_management_alert, project: project) } + let_it_be(:todo) { create(:todo, :pending, target: alert, user: current_user, project: project) } + let_it_be(:other_todo) { create(:todo, :pending, target: other_alert, user: current_user, project: project) } + + let(:fields) do + <<~QUERY + nodes { + iid + todos { + nodes { + id + } + } + } + QUERY + end + + let(:graphql_query) do + graphql_query_for( + 'project', + { 'fullPath' => project.full_path }, + query_graphql_field('alertManagementAlerts', {}, fields) + ) + end + + let(:gql_alerts) { graphql_data.dig('project', 'alertManagementAlerts', 'nodes') } + let(:gql_todos) { gql_alerts.map { |gql_alert| [gql_alert['iid'], gql_alert['todos']['nodes']] }.to_h } + let(:gql_alert_todo) { gql_todos[alert.iid.to_s].first } + let(:gql_other_alert_todo) { gql_todos[other_alert.iid.to_s].first } + + before do + project.add_developer(current_user) + end + + it 'includes the correct metrics dashboard url' do + post_graphql(graphql_query, current_user: current_user) + + expect(gql_alert_todo['id']).to eq(todo.to_global_id.to_s) + expect(gql_other_alert_todo['id']).to eq(other_todo.to_global_id.to_s) + end +end diff --git a/spec/requests/api/graphql/project/alert_management/alert_status_counts_spec.rb b/spec/requests/api/graphql/project/alert_management/alert_status_counts_spec.rb index b62215f43fb..9fbf5aaa41f 100644 --- a/spec/requests/api/graphql/project/alert_management/alert_status_counts_spec.rb +++ b/spec/requests/api/graphql/project/alert_management/alert_status_counts_spec.rb @@ -6,8 +6,8 @@ RSpec.describe 'getting Alert Management Alert counts by status' do let_it_be(:project) { create(:project, :repository) } let_it_be(:current_user) { create(:user) } - let_it_be(:alert_1) { create(:alert_management_alert, :resolved, project: project) } - let_it_be(:alert_2) { create(:alert_management_alert, project: project) } + let_it_be(:alert_resolved) { create(:alert_management_alert, :resolved, project: project) } + let_it_be(:alert_triggered) { create(:alert_management_alert, project: project) } let_it_be(:other_project_alert) { create(:alert_management_alert) } let(:params) { {} } @@ -58,7 +58,7 @@ RSpec.describe 'getting Alert Management Alert counts by status' do end context 'with search criteria' do - let(:params) { { search: alert_1.title } } + let(:params) { { search: alert_resolved.title } } it_behaves_like 'a working graphql query' it 'returns the correct counts for each status' do diff --git a/spec/requests/api/graphql/project/alert_management/alerts_spec.rb b/spec/requests/api/graphql/project/alert_management/alerts_spec.rb index f050c6873f3..d3a2e6a1deb 100644 --- a/spec/requests/api/graphql/project/alert_management/alerts_spec.rb +++ b/spec/requests/api/graphql/project/alert_management/alerts_spec.rb @@ -4,12 +4,12 @@ require 'spec_helper' RSpec.describe 'getting Alert Management Alerts' do include GraphqlHelpers - let_it_be(:payload) { { 'custom' => { 'alert' => 'payload' } } } + let_it_be(:payload) { { 'custom' => { 'alert' => 'payload' }, 'runbook' => 'runbook' } } let_it_be(:project) { create(:project, :repository) } let_it_be(:current_user) { create(:user) } - let_it_be(:resolved_alert) { create(:alert_management_alert, :all_fields, :resolved, project: project, issue: nil, severity: :low) } - let_it_be(:triggered_alert) { create(:alert_management_alert, :all_fields, project: project, severity: :critical, payload: payload) } - let_it_be(:other_project_alert) { create(:alert_management_alert, :all_fields) } + let_it_be(:resolved_alert) { create(:alert_management_alert, :all_fields, :resolved, project: project, issue: nil, severity: :low).present } + let_it_be(:triggered_alert) { create(:alert_management_alert, :all_fields, project: project, severity: :critical, payload: payload).present } + let_it_be(:other_project_alert) { create(:alert_management_alert, :all_fields).present } let(:params) { {} } @@ -71,10 +71,13 @@ RSpec.describe 'getting Alert Management Alerts' do 'eventCount' => triggered_alert.events, 'startedAt' => triggered_alert.started_at.strftime('%Y-%m-%dT%H:%M:%SZ'), 'endedAt' => nil, - 'details' => { 'custom.alert' => 'payload' }, + 'details' => { 'custom.alert' => 'payload', 'runbook' => 'runbook' }, 'createdAt' => triggered_alert.created_at.strftime('%Y-%m-%dT%H:%M:%SZ'), 'updatedAt' => triggered_alert.updated_at.strftime('%Y-%m-%dT%H:%M:%SZ'), - 'metricsDashboardUrl' => nil + 'metricsDashboardUrl' => nil, + 'detailsUrl' => triggered_alert.details_url, + 'prometheusAlert' => nil, + 'runbook' => 'runbook' ) expect(second_alert).to include( |