Commit ed079407 authored by Luke Duncalfe's avatar Luke Duncalfe

Fix broken master by updating expected params

https://gitlab.com/gitlab-org/gitlab/-/merge_requests/24296
introduced a spec that interacted badly with a schema change to the
`services` table in 8d4d9705.

`update_service_spec.rb` expected certain attributes, some of which
are based on default properties of a `PrometheusService`, which is a
model that reads from `services`.

8d4d9705 changed the schema of `services` which caused the test to
break.

https://gitlab.com/gitlab-org/gitlab/issues/205391
parent b6a3bd97
...@@ -304,7 +304,6 @@ describe Projects::Operations::UpdateService do ...@@ -304,7 +304,6 @@ describe Projects::Operations::UpdateService do
"title" => nil, "title" => nil,
"active" => true, "active" => true,
"properties" => { "api_url" => "http://example.prometheus.com", "manual_configuration" => "0" }, "properties" => { "api_url" => "http://example.prometheus.com", "manual_configuration" => "0" },
"instance" => false,
"push_events" => true, "push_events" => true,
"issues_events" => true, "issues_events" => true,
"merge_requests_events" => true, "merge_requests_events" => true,
...@@ -320,7 +319,8 @@ describe Projects::Operations::UpdateService do ...@@ -320,7 +319,8 @@ describe Projects::Operations::UpdateService do
"confidential_note_events" => true, "confidential_note_events" => true,
"deployment_events" => false, "deployment_events" => false,
"description" => nil, "description" => nil,
"comment_on_event_enabled" => true "comment_on_event_enabled" => true,
"template" => false
} }
end end
let(:params) do let(:params) do
......
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