Skip to content
代码片段 群组 项目
未验证 提交 b0d407ea 编辑于 作者: Ravi Kumar's avatar Ravi Kumar 提交者: GitLab
浏览文件

Merge branch '517048-move-specs-away-from-issues' into 'master'

Use other scopes in tests as we are moving to WorkItem from Issue class

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/183700



Merged-by: default avatarRavi Kumar <rkumar@gitlab.com>
Approved-by: default avatarArturo Herrero <arturo.herrero@gmail.com>
Approved-by: default avatarRavi Kumar <rkumar@gitlab.com>
Co-authored-by: default avatarSiddharth Dungarwal <sdungarwal@gitlab.com>
No related branches found
No related tags found
2 合并请求!3031Merge per-main-jh to main-jh by luzhiyuan,!3030Merge per-main-jh to main-jh
...@@ -2,7 +2,7 @@ ...@@ -2,7 +2,7 @@
require 'spec_helper' require 'spec_helper'
RSpec.describe Elasticsearch::Model::Adapter::ActiveRecord::Records, :elastic do RSpec.describe Elasticsearch::Model::Adapter::ActiveRecord::Records, :elastic, feature_category: :global_search do
before do before do
stub_ee_application_setting(elasticsearch_indexing: true) stub_ee_application_setting(elasticsearch_indexing: true)
end end
...@@ -10,16 +10,16 @@ ...@@ -10,16 +10,16 @@
describe '#records' do describe '#records' do
let(:user) { create(:user) } let(:user) { create(:user) }
let(:search_options) { { options: { search_level: 'global', current_user: user, project_ids: :any, order_by: 'created_at', sort: 'desc' } } } let(:search_options) { { options: { search_level: 'global', current_user: user, project_ids: :any, order_by: 'created_at', sort: 'desc' } } }
let(:results) { Issue.elastic_search('*', **search_options).records.to_a } let(:results) { MergeRequest.elastic_search('*', **search_options).records.to_a }
let!(:new_issue) { create(:issue) } let!(:new_merge_request) { create(:merge_request) }
let!(:recent_issue) { create(:issue, created_at: 1.hour.ago) } let!(:recent_merge_request) { create(:merge_request, created_at: 1.hour.ago) }
let!(:old_issue) { create(:issue, created_at: 7.days.ago) } let!(:old_merge_request) { create(:merge_request, created_at: 7.days.ago) }
it 'returns results in the same sorted order as they come back from Elasticsearch' do it 'returns results in the same sorted order as they come back from Elasticsearch' do
ensure_elasticsearch_index! ensure_elasticsearch_index!
expect(results).to eq([new_issue, recent_issue, old_issue]) expect(results).to eq([new_merge_request, recent_merge_request, old_merge_request])
end end
end end
end end
...@@ -10,15 +10,15 @@ ...@@ -10,15 +10,15 @@
describe '#records' do describe '#records' do
let(:user) { create(:user) } let(:user) { create(:user) }
let(:search_options) { { options: { current_user: user, project_ids: :any } } } let(:search_options) { { options: { current_user: user, project_ids: :any } } }
let(:records) { Elasticsearch::Model.search('*', [Issue, MergeRequest]).records.to_a } let(:records) { Elasticsearch::Model.search('*', [MergeRequest, Note]).records.to_a }
let!(:issue) { create(:issue) } let!(:note) { create(:note) }
let!(:merge_request) { create(:merge_request) } let!(:merge_request) { create(:merge_request) }
it 'returns results from both classes in different Elasticsearch indexes' do it 'returns results from both classes in different Elasticsearch indexes' do
ensure_elasticsearch_index! ensure_elasticsearch_index!
expect(records).to match_array([issue, merge_request]) expect(records).to match_array([merge_request, note])
end end
end end
end end
0% 加载中 .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册