diff options
author | Robert Speicher <robert@gitlab.com> | 2016-03-15 19:25:02 +0300 |
---|---|---|
committer | Robert Speicher <robert@gitlab.com> | 2016-03-15 19:25:02 +0300 |
commit | 178c80a561fa10a157bae6e5d4682b232ae727c7 (patch) | |
tree | d1c6e3116acc6d5f628ee1f8a37cdac5840dc24e /spec | |
parent | 09494ff413b342884f8df9d8385a71b6ec820688 (diff) | |
parent | c742760289e51117d3e76e27a626691bec631e1e (diff) |
Merge branch 'fix-activerecord-join-stupidity' into 'master'
Ignore eager loading in Project.search UNION
This fixes issues such as filtering groups by names on pages such as
https://gitlab.com/dashboard/groups.
See merge request !3229
Diffstat (limited to 'spec')
-rw-r--r-- | spec/models/project_spec.rb | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/spec/models/project_spec.rb b/spec/models/project_spec.rb index 59c5ffa6b9c..b8b9a455b83 100644 --- a/spec/models/project_spec.rb +++ b/spec/models/project_spec.rb @@ -634,6 +634,12 @@ describe Project, models: true do it 'returns projects with a matching namespace name regardless of the casing' do expect(described_class.search(project.namespace.name.upcase)).to eq([project]) end + + it 'returns projects when eager loading namespaces' do + relation = described_class.all.includes(:namespace) + + expect(relation.search(project.namespace.name)).to eq([project]) + end end describe '#rename_repo' do |