Commit 99f91ccc authored by Jan Provaznik's avatar Jan Provaznik

Merge branch 'issue_329804' into 'master'

Improve GraphQL epics spec for N+1

See merge request gitlab-org/gitlab!66765
parents 1d4cc84a 8de4faca
...@@ -194,12 +194,10 @@ RSpec.describe 'Epics through GroupQuery' do ...@@ -194,12 +194,10 @@ RSpec.describe 'Epics through GroupQuery' do
epics_with_parent = create_list(:epic, 3, group: group) do |epic| epics_with_parent = create_list(:epic, 3, group: group) do |epic|
epic.update!(parent: create(:epic, group: group)) epic.update!(parent: create(:epic, group: group))
end end
group.reload
# Threshold of 3 due to an existing N+1 with licenses
expect do expect do
post_graphql(query({ iids: epics_with_parent.pluck(:iid) }), current_user: user) post_graphql(query({ iids: epics_with_parent.pluck(:iid) }), current_user: user)
end.not_to exceed_query_limit(control_count).with_threshold(3) end.not_to exceed_query_limit(control_count)
end end
end end
......
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