diff options
Diffstat (limited to 'spec/requests/api/groups_spec.rb')
| -rw-r--r-- | spec/requests/api/groups_spec.rb | 20 |
1 files changed, 0 insertions, 20 deletions
diff --git a/spec/requests/api/groups_spec.rb b/spec/requests/api/groups_spec.rb index ffc5d353958..56f08249bdd 100644 --- a/spec/requests/api/groups_spec.rb +++ b/spec/requests/api/groups_spec.rb @@ -1177,7 +1177,6 @@ RSpec.describe API::Groups do it "only looks up root ancestor once and returns projects including those in subgroups" do expect(Namespace).to receive(:find_by).with(id: group1.id.to_s).once.and_call_original # For the group sent in the API call - expect(Namespace).to receive(:find_by).with(id: group1.traversal_ids.first).once.and_call_original # root_ancestor direct lookup expect(Namespace).to receive(:joins).with(start_with('INNER JOIN (SELECT id, traversal_ids[1]')).once.and_call_original # All-in-one root_ancestor query get api("/groups/#{group1.id}/projects", user1), params: { include_subgroups: true } @@ -1187,25 +1186,6 @@ RSpec.describe API::Groups do expect(json_response).to be_an(Array) expect(json_response.length).to eq(6) end - - context 'when group_projects_api_preload_groups feature is disabled' do - before do - stub_feature_flags(group_projects_api_preload_groups: false) - end - - it 'looks up the root ancestor multiple times' do - expect(Namespace).to receive(:find_by).with(id: group1.id.to_s).once.and_call_original - expect(Namespace).to receive(:find_by).with(id: group1.traversal_ids.first).at_least(:twice).and_call_original - expect(Namespace).not_to receive(:joins).with(start_with('INNER JOIN (SELECT id, traversal_ids[1]')) - - get api("/groups/#{group1.id}/projects", user1), params: { include_subgroups: true } - - expect(response).to have_gitlab_http_status(:ok) - expect(response).to include_pagination_headers - expect(json_response).to be_an(Array) - expect(json_response.length).to eq(6) - end - end end context 'when include_ancestor_groups is true' do |