Skip to content
代码片段 群组 项目
提交 ce3edf33 编辑于 作者: Alex Kalderimis's avatar Alex Kalderimis
浏览文件

Merge branch 'bw-reference-project-caching' into 'master'

Check for N+1 in ProjectReferenceFilter

See merge request gitlab-org/gitlab!67362
No related branches found
No related tags found
无相关合并请求
......@@ -97,4 +97,34 @@ def get_reference(project)
expect(filter.send(:projects)).to eq([project.full_path])
end
end
context 'checking N+1' do
let_it_be(:normal_project) { create(:project, :public) }
let_it_be(:group) { create(:group) }
let_it_be(:group_project) { create(:project, group: group) }
let_it_be(:nested_group) { create(:group, :nested) }
let_it_be(:nested_project) { create(:project, group: nested_group) }
let_it_be(:normal_project_reference) { get_reference(normal_project) }
let_it_be(:group_project_reference) { get_reference(group_project) }
let_it_be(:nested_project_reference) { get_reference(nested_project) }
it 'does not have N+1 per multiple project references', :use_sql_query_cache do
markdown = "#{normal_project_reference}"
# warm up first
reference_filter(markdown)
max_count = ActiveRecord::QueryRecorder.new(skip_cached: false) do
reference_filter(markdown)
end.count
expect(max_count).to eq 1
markdown = "#{normal_project_reference} #{invalidate_reference(normal_project_reference)} #{group_project_reference} #{nested_project_reference}"
expect do
reference_filter(markdown)
end.not_to exceed_all_query_limit(max_count)
end
end
end
0% 加载中 .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册