Revert "Merge branch 'if-57131-external_auth_to_ce' into 'master'"

This reverts merge request !26823

1 files changed, 19 insertions, 31 deletions

diff --git a/spec/controllers/dashboard/groups_controller_spec.rb b/spec/controllers/dashboard/groups_controller_spec.rb
index 775b3ca40b2..c8d99f79277 100644
--- a/spec/controllers/dashboard/groups_controller_spec.rb
+++ b/spec/controllers/dashboard/groups_controller_spec.rb
@@ -1,8 +1,6 @@
 require 'spec_helper'
 
 describe Dashboard::GroupsController do
-  include ExternalAuthorizationServiceHelpers
-
   let(:user) { create(:user) }
 
   before do
@@ -13,43 +11,33 @@ describe Dashboard::GroupsController do
     expect(described_class).to include(GroupTree)
   end
 
-  describe '#index' do
-    it 'only includes projects the user is a member of' do
-      member_of_group = create(:group)
-      member_of_group.add_developer(user)
-      create(:group, :public)
-
-      get :index
-
-      expect(assigns(:groups)).to contain_exactly(member_of_group)
-    end
+  it 'only includes projects the user is a member of' do
+    member_of_group = create(:group)
+    member_of_group.add_developer(user)
+    create(:group, :public)
 
-    context 'when rendering an expanded hierarchy with public groups you are not a member of', :nested_groups do
-      let!(:top_level_result) { create(:group, name: 'chef-top') }
-      let!(:top_level_a) { create(:group, name: 'top-a') }
-      let!(:sub_level_result_a) { create(:group, name: 'chef-sub-a', parent: top_level_a) }
-      let!(:other_group) { create(:group, name: 'other') }
+    get :index
 
-      before do
-        top_level_result.add_maintainer(user)
-        top_level_a.add_maintainer(user)
-      end
+    expect(assigns(:groups)).to contain_exactly(member_of_group)
+  end
 
-      it 'renders only groups the user is a member of when searching hierarchy correctly' do
-        get :index, params: { filter: 'chef' }, format: :json
+  context 'when rendering an expanded hierarchy with public groups you are not a member of', :nested_groups do
+    let!(:top_level_result) { create(:group, name: 'chef-top') }
+    let!(:top_level_a) { create(:group, name: 'top-a') }
+    let!(:sub_level_result_a) { create(:group, name: 'chef-sub-a', parent: top_level_a) }
+    let!(:other_group) { create(:group, name: 'other') }
 
-        expect(response).to have_gitlab_http_status(200)
-        all_groups = [top_level_result, top_level_a, sub_level_result_a]
-        expect(assigns(:groups)).to contain_exactly(*all_groups)
-      end
+    before do
+      top_level_result.add_maintainer(user)
+      top_level_a.add_maintainer(user)
     end
 
-    it 'works when the external authorization service is enabled' do
-      enable_external_authorization_service_check
-
-      get :index
+    it 'renders only groups the user is a member of when searching hierarchy correctly' do
+      get :index, params: { filter: 'chef' }, format: :json
 
       expect(response).to have_gitlab_http_status(200)
+      all_groups = [top_level_result, top_level_a, sub_level_result_a]
+      expect(assigns(:groups)).to contain_exactly(*all_groups)
     end
   end
 end