Rename AutoDevopsHelper helper methods

app/helpers/auto_devops_helper.rb

@@ -9,28 +9,28 @@ module AutoDevopsHelper
   end
 
   def auto_devops_warning_message(project)
-    if missing_service?(project)
+    if missing_auto_devops_service?(project)
       params = {
         kubernetes: link_to('Kubernetes cluster', project_clusters_path(project))
       }
 
-      if missing_domain?(project)
+      if missing_auto_devops_domain?(project)
         _('Auto Review Apps and Auto Deploy need a domain name and a %{kubernetes} to work correctly.') % params
       else
         _('Auto Review Apps and Auto Deploy need a %{kubernetes} to work correctly.') % params
       end
-    elsif missing_domain?(project)
+    elsif missing_auto_devops_domain?(project)
       _('Auto Review Apps and Auto Deploy need a domain name to work correctly.')
     end
   end
 
   private
 
-  def missing_domain?(project)
+  def missing_auto_devops_domain?(project)
     !(project.auto_devops&.has_domain? || current_application_settings.auto_devops_domain.present?)
   end
 
-  def missing_service?(project)
+  def missing_auto_devops_service?(project)
     !project.deployment_platform&.active?
   end
 end

spec/helpers/auto_devops_helper_spec.rb

@@ -88,12 +88,12 @@ describe AutoDevopsHelper do
 
     context 'when the service is missing' do
       before do
-        allow(helper).to receive(:missing_service?).and_return(true)
+        allow(helper).to receive(:missing_auto_devops_service?).and_return(true)
       end
 
       context 'when the domain is missing' do
         before do
-          allow(helper).to receive(:missing_domain?).and_return(true)
+          allow(helper).to receive(:missing_auto_devops_domain?).and_return(true)
         end
 
         it { is_expected.to match(/Auto Review Apps and Auto Deploy need a domain name and a .* to work correctly./) }
@@ -101,7 +101,7 @@ describe AutoDevopsHelper do
 
       context 'when the domain is not missing' do
         before do
-          allow(helper).to receive(:missing_domain?).and_return(false)
+          allow(helper).to receive(:missing_auto_devops_domain?).and_return(false)
         end
 
         it { is_expected.to match(/Auto Review Apps and Auto Deploy need a .* to work correctly./) }
@@ -110,8 +110,8 @@ describe AutoDevopsHelper do
 
     context 'when the domain is missing' do
       before do
-        allow(helper).to receive(:missing_service?).and_return(false)
-        allow(helper).to receive(:missing_domain?).and_return(true)
+        allow(helper).to receive(:missing_auto_devops_service?).and_return(false)
+        allow(helper).to receive(:missing_auto_devops_domain?).and_return(true)
       end
 
       it { is_expected.to eq('Auto Review Apps and Auto Deploy need a domain name to work correctly.') }