Merge branch 'refactor-verify-pages-domain-specs' into 'master'

Refactor verify_pages_domain_service specs

See merge request gitlab-org/gitlab-ce!26178

spec/services/verify_pages_domain_service_spec.rb

@@ -9,7 +9,6 @@ describe VerifyPagesDomainService do
   subject(:service) { described_class.new(domain) }
 
   describe '#execute' do
-    context 'verification code recognition (verified domain)' do
     where(:domain_sym, :code_sym) do
       :domain | :verification_code
       :domain | :keyed_verification_code
@@ -19,80 +18,123 @@ describe VerifyPagesDomainService do
     end
 
     with_them do
-        set(:domain) { create(:pages_domain) }
-
       let(:domain_name) { domain.send(domain_sym) }
       let(:verification_code) { domain.send(code_sym) }
 
+      shared_examples 'verifies and enables the domain' do
         it 'verifies and enables the domain' do
-          stub_resolver(domain_name => ['something else', verification_code])
-
           expect(service.execute).to eq(status: :success)
+
           expect(domain).to be_verified
           expect(domain).to be_enabled
         end
+      end
+
+      shared_examples 'successful enablement and verification' do
+        context 'when txt record contains verification code' do
+          before do
+            stub_resolver(domain_name => ['something else', verification_code])
+          end
+
+          include_examples 'verifies and enables the domain'
+        end
 
-        it 'verifies and enables when the code is contained partway through a TXT record' do
+        context 'when txt record contains verification code with other text' do
+          before do
             stub_resolver(domain_name => "something #{verification_code} else")
+          end
 
-          expect(service.execute).to eq(status: :success)
-          expect(domain).to be_verified
-          expect(domain).to be_enabled
+          include_examples 'verifies and enables the domain'
+        end
       end
 
-        it 'does not verify when the code is not present' do
-          stub_resolver(domain_name => 'something else')
+      context 'when domain is disabled(or new)' do
+        let(:domain) { create(:pages_domain, :disabled) }
+
+        include_examples 'successful enablement and verification'
 
+        shared_examples 'unverifies and disables domain' do
+          it 'unverifies and disables domain' do
             expect(service.execute).to eq(error_status)
 
             expect(domain).not_to be_verified
-          expect(domain).to be_enabled
+            expect(domain).not_to be_enabled
+          end
+        end
+
+        context 'when txt record does not contain verification code' do
+          before do
+            stub_resolver(domain_name => 'something else')
+          end
+
+          include_examples 'unverifies and disables domain'
+        end
+
+        context 'when no txt records are present' do
+          before do
+            stub_resolver
+          end
+
+          include_examples 'unverifies and disables domain'
         end
       end
 
-      context 'verified domain' do
-        set(:domain) { create(:pages_domain) }
+      context 'when domain is verified' do
+        let(:domain) { create(:pages_domain) }
 
-        it 'unverifies (but does not disable) when the right code is not present' do
-          stub_resolver(domain.domain => 'something else')
+        include_examples 'successful enablement and verification'
 
+        context 'when txt record does not contain verification code' do
+          before do
+            stub_resolver(domain_name => 'something else')
+          end
+
+          it 'unverifies but does not disable domain' do
             expect(service.execute).to eq(error_status)
             expect(domain).not_to be_verified
             expect(domain).to be_enabled
           end
+        end
 
-        it 'unverifies (but does not disable) when no records are present' do
+        context 'when no txt records are present' do
+          before do
             stub_resolver
+          end
 
+          it 'unverifies but does not disable domain' do
             expect(service.execute).to eq(error_status)
             expect(domain).not_to be_verified
             expect(domain).to be_enabled
           end
         end
+      end
 
-      context 'expired domain' do
-        set(:domain) { create(:pages_domain, :expired) }
+      context 'when domain is expired' do
+        let(:domain) { create(:pages_domain, :expired) }
 
-        it 'verifies and enables when the right code is present' do
-          stub_resolver(domain.domain => domain.keyed_verification_code)
+        context 'when the right code is present' do
+          before do
+            stub_resolver(domain_name => domain.keyed_verification_code)
+          end
 
-          expect(service.execute).to eq(status: :success)
+          include_examples 'verifies and enables the domain'
+        end
 
-          expect(domain).to be_verified
-          expect(domain).to be_enabled
+        context 'when the right code is not present' do
+          before do
+            stub_resolver
           end
 
-        it 'disables when the right code is not present' do
+          it 'disables domain' do
             error_status[:message] += '. It is now disabled.'
 
-          stub_resolver
-
             expect(service.execute).to eq(error_status)
 
             expect(domain).not_to be_verified
             expect(domain).not_to be_enabled
           end
         end
+      end
 
       context 'invalid domain' do
         let(:domain) { build(:pages_domain, :expired, :with_missing_chain) }