[EE] Reduce the diff with CE in spec/helpers/auth_helper_spec.rb

Signed-off-by: Rémy Coutable <remy@rymai.me>

ee/spec/helpers/auth_helper_spec.rb

+# frozen_string_literal: true
+
+require "spec_helper"
+
+describe AuthHelper do
+  describe "button_based_providers" do
+    it 'excludes group_saml' do
+      allow(helper).to receive(:auth_providers) { [:group_saml] }
+      expect(helper.button_based_providers).to eq([])
+    end
+  end
+
+  describe "providers_for_base_controller" do
+    it 'excludes group_saml' do
+      allow(helper).to receive(:auth_providers) { [:group_saml] }
+      expect(helper.providers_for_base_controller).to eq([])
+    end
+  end
+
+  describe "form_based_providers" do
+    it 'includes kerberos provider' do
+      allow(helper).to receive(:auth_providers) { [:twitter, :kerberos] }
+      expect(helper.form_based_providers).to eq %i(kerberos)
+    end
+  end
+
+  describe 'form_based_auth_provider_has_active_class?' do
+    it 'selects main LDAP server' do
+      allow(helper).to receive(:auth_providers) { [:twitter, :ldapprimary, :ldapsecondary, :kerberos] }
+      expect(helper.form_based_auth_provider_has_active_class?(:twitter)).to be(false)
+      expect(helper.form_based_auth_provider_has_active_class?(:ldapprimary)).to be(true)
+      expect(helper.form_based_auth_provider_has_active_class?(:ldapsecondary)).to be(false)
+      expect(helper.form_based_auth_provider_has_active_class?(:kerberos)).to be(false)
+    end
+  end
+end

spec/helpers/auth_helper_spec.rb

+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe AuthHelper do
@@ -12,11 +14,6 @@ describe AuthHelper do
       expect(helper.button_based_providers).to include(:twitter)
     end
 
-    it 'excludes group_saml' do
-      allow(helper).to receive(:auth_providers) { [:group_saml] }
-      expect(helper.button_based_providers).to eq([])
-    end
-
     it 'returns empty array' do
       allow(helper).to receive(:auth_providers) { [] }
       expect(helper.button_based_providers).to eq([])
@@ -33,11 +30,6 @@ describe AuthHelper do
       allow(helper).to receive(:auth_providers) { [:twitter, :ldapmain] }
       expect(helper.providers_for_base_controller).not_to include(:ldapmain)
     end
-
-    it 'excludes group_saml' do
-      allow(helper).to receive(:auth_providers) { [:group_saml] }
-      expect(helper.providers_for_base_controller).to eq([])
-    end
   end
 
   describe "form_based_providers" do
@@ -50,21 +42,6 @@ describe AuthHelper do
       allow(helper).to receive(:auth_providers) { [:twitter, :crowd] }
       expect(helper.form_based_providers).to eq %i(crowd)
     end
-
-    it 'includes kerberos provider' do
-      allow(helper).to receive(:auth_providers) { [:twitter, :kerberos] }
-      expect(helper.form_based_providers).to eq %i(kerberos)
-    end
-  end
-
-  describe 'form_based_auth_provider_has_active_class?' do
-    it 'selects main LDAP server' do
-      allow(helper).to receive(:auth_providers) { [:twitter, :ldapprimary, :ldapsecondary, :kerberos] }
-      expect(helper.form_based_auth_provider_has_active_class?(:twitter)).to be(false)
-      expect(helper.form_based_auth_provider_has_active_class?(:ldapprimary)).to be(true)
-      expect(helper.form_based_auth_provider_has_active_class?(:ldapsecondary)).to be(false)
-      expect(helper.form_based_auth_provider_has_active_class?(:kerberos)).to be(false)
-    end
   end
 
   describe 'form_based_auth_provider_has_active_class?' do