Commit b91a220b authored by Jarka Košanová's avatar Jarka Košanová

Merge branch 'fix_epics_finder_spec_query_count' into 'master'

Lower expected query counts in EpicsFinder spec

See merge request gitlab-org/gitlab!33465
parents a663b754 d156b7d3
...@@ -57,8 +57,8 @@ RSpec.describe EpicsFinder do ...@@ -57,8 +57,8 @@ RSpec.describe EpicsFinder do
expect(epics).to contain_exactly(epic1, epic2, epic3) expect(epics).to contain_exactly(epic1, epic2, epic3)
end end
it 'does not execute more than 9 SQL queries' do it 'does not execute more than 8 SQL queries' do
expect { epics.to_a }.not_to exceed_all_query_limit(9) expect { epics.to_a }.not_to exceed_all_query_limit(8)
end end
context 'sorting' do context 'sorting' do
...@@ -169,18 +169,18 @@ RSpec.describe EpicsFinder do ...@@ -169,18 +169,18 @@ RSpec.describe EpicsFinder do
end end
end end
it 'does not execute more than 14 SQL queries' do it 'does not execute more than 12 SQL queries' do
expect { epics.to_a }.not_to exceed_all_query_limit(14) expect { epics.to_a }.not_to exceed_all_query_limit(12)
end end
it 'does not execute more than 15 SQL queries when checking namespace plans' do it 'does not execute more than 13 SQL queries when checking namespace plans' do
allow(Gitlab::CurrentSettings) allow(Gitlab::CurrentSettings)
.to receive(:should_check_namespace_plan?) .to receive(:should_check_namespace_plan?)
.and_return(true) .and_return(true)
create(:gitlab_subscription, :gold, namespace: group) create(:gitlab_subscription, :gold, namespace: group)
expect { epics.to_a }.not_to exceed_all_query_limit(15) expect { epics.to_a }.not_to exceed_all_query_limit(13)
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