Commit 74caf547 authored by Grzegorz Bizon's avatar Grzegorz Bizon

Fix pipeline serializer queries count specs

parent a2f6aa96
...@@ -112,7 +112,7 @@ describe PipelineSerializer do ...@@ -112,7 +112,7 @@ describe PipelineSerializer do
# gitaly calls in this block # gitaly calls in this block
# Issue: https://gitlab.com/gitlab-org/gitlab-ce/issues/37772 # Issue: https://gitlab.com/gitlab-org/gitlab-ce/issues/37772
Gitlab::GitalyClient.allow_n_plus_1_calls do Gitlab::GitalyClient.allow_n_plus_1_calls do
Ci::Pipeline::AVAILABLE_STATUSES.each do |status| Ci::Pipeline::COMPLETED_STATUSES.each do |status|
create_pipeline(status) create_pipeline(status)
end end
end end
...@@ -125,7 +125,7 @@ describe PipelineSerializer do ...@@ -125,7 +125,7 @@ describe PipelineSerializer do
it 'verifies number of queries', :request_store do it 'verifies number of queries', :request_store do
recorded = ActiveRecord::QueryRecorder.new { subject } recorded = ActiveRecord::QueryRecorder.new { subject }
expect(recorded.count).to be_within(1).of(31) expect(recorded.count).to be_within(2).of(27)
expect(recorded.cached_count).to eq(0) expect(recorded.cached_count).to eq(0)
end end
end end
...@@ -144,7 +144,7 @@ describe PipelineSerializer do ...@@ -144,7 +144,7 @@ describe PipelineSerializer do
# pipeline. With the same ref this check is cached but if refs are # pipeline. With the same ref this check is cached but if refs are
# different then there is an extra query per ref # different then there is an extra query per ref
# https://gitlab.com/gitlab-org/gitlab-ce/issues/46368 # https://gitlab.com/gitlab-org/gitlab-ce/issues/46368
expect(recorded.count).to be_within(1).of(38) expect(recorded.count).to be_within(2).of(30)
expect(recorded.cached_count).to eq(0) expect(recorded.cached_count).to eq(0)
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