Commit e2cb152d authored by Peter Leitzen's avatar Peter Leitzen

Merge branch 'add-explicit-test-for-multiple-prometheus-alerts' into 'master'

Add test covering multiple prometheus alerts in single payload

See merge request gitlab-org/gitlab!16360
parents 7cebe9c5 cdc2a8a8
...@@ -292,6 +292,14 @@ describe Projects::Prometheus::Alerts::NotifyService do ...@@ -292,6 +292,14 @@ describe Projects::Prometheus::Alerts::NotifyService do
it_behaves_like 'processes incident issues', 2 it_behaves_like 'processes incident issues', 2
context 'multiple firing alerts' do
let(:payload_raw) do
payload_for(firing: [alert_firing, alert_firing], resolved: [])
end
it_behaves_like 'processes incident issues', 2
end
context 'without firing alerts' do context 'without firing alerts' do
let(:payload_raw) do let(:payload_raw) do
payload_for(firing: [], resolved: [alert_resolved]) payload_for(firing: [], resolved: [alert_resolved])
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment