diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2020-10-21 10:08:36 +0300 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2020-10-21 10:08:36 +0300 |
commit | 48aff82709769b098321c738f3444b9bdaa694c6 (patch) | |
tree | e00c7c43e2d9b603a5a6af576b1685e400410dee /spec/models/preloaders | |
parent | 879f5329ee916a948223f8f43d77fba4da6cd028 (diff) |
Add latest changes from gitlab-org/gitlab@13-5-stable-eev13.5.0-rc42
Diffstat (limited to 'spec/models/preloaders')
-rw-r--r-- | spec/models/preloaders/merge_request_diff_preloader_spec.rb | 29 |
1 files changed, 29 insertions, 0 deletions
diff --git a/spec/models/preloaders/merge_request_diff_preloader_spec.rb b/spec/models/preloaders/merge_request_diff_preloader_spec.rb new file mode 100644 index 00000000000..9a76d42e73f --- /dev/null +++ b/spec/models/preloaders/merge_request_diff_preloader_spec.rb @@ -0,0 +1,29 @@ +# frozen_string_literal: true + +require 'spec_helper' + +RSpec.describe Preloaders::MergeRequestDiffPreloader do + let_it_be(:merge_request_1) { create(:merge_request) } + let_it_be(:merge_request_2) { create(:merge_request) } + let_it_be(:merge_request_3) { create(:merge_request_without_merge_request_diff) } + + let(:merge_requests) { [merge_request_1, merge_request_2, merge_request_3] } + + def trigger(merge_requests) + Array(merge_requests).each(&:merge_request_diff) + end + + def merge_requests_with_preloaded_diff + described_class.new(MergeRequest.where(id: merge_requests.map(&:id)).to_a).preload_all + end + + it 'does not trigger N+1 queries' do + # warmup + trigger(merge_requests_with_preloaded_diff) + + first_merge_request = merge_requests_with_preloaded_diff.first + clean_merge_requests = merge_requests_with_preloaded_diff + + expect { trigger(clean_merge_requests) }.to issue_same_number_of_queries_as { trigger(first_merge_request) } + end +end |