Remove Auto DevOps checkboxes - backend

3 files changed, 46 insertions, 114 deletions

diff --git a/spec/controllers/projects/pipelines_settings_controller_spec.rb b/spec/controllers/projects/pipelines_settings_controller_spec.rb
index b2d83a02290..1cc488bef32 100644
--- a/spec/controllers/projects/pipelines_settings_controller_spec.rb
+++ b/spec/controllers/projects/pipelines_settings_controller_spec.rb
@@ -16,14 +16,13 @@ describe Projects::PipelinesSettingsController do
       patch :update,
         namespace_id: project.namespace.to_param,
         project_id: project,
-        project: { auto_devops_attributes: params,
-                   run_auto_devops_pipeline_implicit: 'false',
-                   run_auto_devops_pipeline_explicit: auto_devops_pipeline }
+        project: {
+          auto_devops_attributes: params
+        }
     end
 
     context 'when updating the auto_devops settings' do
       let(:params) { { enabled: '', domain: 'mepmep.md' } }
-      let(:auto_devops_pipeline) { 'false' }
 
       it 'redirects to the settings page' do
         subject
@@ -44,7 +43,9 @@ describe Projects::PipelinesSettingsController do
       end
 
       context 'when run_auto_devops_pipeline is true' do
-        let(:auto_devops_pipeline) { 'true' }
+        before do
+          expect_any_instance_of(Projects::UpdateService).to receive(:run_auto_devops_pipeline?).and_return(true)
+        end
 
         it 'queues a CreatePipelineWorker' do
           expect(CreatePipelineWorker).to receive(:perform_async).with(project.id, user.id, project.default_branch, :web, any_args)
@@ -54,7 +55,9 @@ describe Projects::PipelinesSettingsController do
       end
 
       context 'when run_auto_devops_pipeline is not true' do
-        let(:auto_devops_pipeline) { 'false' }
+        before do
+          expect_any_instance_of(Projects::UpdateService).to receive(:run_auto_devops_pipeline?).and_return(false)
+        end
 
         it 'does not queue a CreatePipelineWorker' do
           expect(CreatePipelineWorker).not_to receive(:perform_async).with(project.id, user.id, :web, any_args)
diff --git a/spec/helpers/auto_devops_helper_spec.rb b/spec/helpers/auto_devops_helper_spec.rb
index 7266e1b84d1..5e272af6073 100644
--- a/spec/helpers/auto_devops_helper_spec.rb
+++ b/spec/helpers/auto_devops_helper_spec.rb
@@ -82,104 +82,4 @@ describe AutoDevopsHelper do
       it { is_expected.to eq(false) }
     end
   end
-
-  describe '.show_run_auto_devops_pipeline_checkbox_for_instance_setting?' do
-    subject { helper.show_run_auto_devops_pipeline_checkbox_for_instance_setting?(project) }
-
-    context 'when master contains a .gitlab-ci.yml file' do
-      before do
-        allow(project.repository).to receive(:gitlab_ci_yml).and_return("script: ['test']")
-      end
-
-      it { is_expected.to eq(false) }
-    end
-
-    context 'when auto devops is explicitly enabled' do
-      before do
-        project.create_auto_devops!(enabled: true)
-      end
-
-      it { is_expected.to eq(false) }
-    end
-
-    context 'when auto devops is explicitly disabled' do
-      before do
-        project.create_auto_devops!(enabled: false)
-      end
-
-      context 'when auto devops is enabled system-wide' do
-        before do
-          stub_application_setting(auto_devops_enabled: true)
-        end
-
-        it { is_expected.to eq(true) }
-      end
-
-      context 'when auto devops is disabled system-wide' do
-        before do
-          stub_application_setting(auto_devops_enabled: false)
-        end
-
-        it { is_expected.to eq(false) }
-      end
-    end
-
-    context 'when auto devops is set to instance setting' do
-      before do
-        project.create_auto_devops!(enabled: nil)
-      end
-
-      it { is_expected.to eq(false) }
-    end
-  end
-
-  describe '.show_run_auto_devops_pipeline_checkbox_for_explicit_setting?' do
-    subject { helper.show_run_auto_devops_pipeline_checkbox_for_explicit_setting?(project) }
-
-    context 'when master contains a .gitlab-ci.yml file' do
-      before do
-        allow(project.repository).to receive(:gitlab_ci_yml).and_return("script: ['test']")
-      end
-
-      it { is_expected.to eq(false) }
-    end
-
-    context 'when auto devops is explicitly enabled' do
-      before do
-        project.create_auto_devops!(enabled: true)
-      end
-
-      it { is_expected.to eq(false) }
-    end
-
-    context 'when auto devops is explicitly disabled' do
-      before do
-        project.create_auto_devops!(enabled: false)
-      end
-
-      it { is_expected.to eq(true) }
-    end
-
-    context 'when auto devops is set to instance setting' do
-      before do
-        project.create_auto_devops!(enabled: nil)
-      end
-
-      context 'when auto devops is enabled system-wide' do
-        before do
-          stub_application_setting(auto_devops_enabled: true)
-        end
-
-        it { is_expected.to eq(false) }
-      end
-
-      context 'when auto devops is disabled system-wide' do
-        before do
-          stub_application_setting(auto_devops_enabled: false)
-        end
-
-        it { is_expected.to eq(true) }
-      end
-    end
-  end
 end
diff --git a/spec/services/projects/update_service_spec.rb b/spec/services/projects/update_service_spec.rb
index fcd71857af3..d887f70efae 100644
--- a/spec/services/projects/update_service_spec.rb
+++ b/spec/services/projects/update_service_spec.rb
@@ -199,24 +199,53 @@ describe Projects::UpdateService do
   end
 
   describe '#run_auto_devops_pipeline?' do
-    subject { described_class.new(project, user, params).run_auto_devops_pipeline? }
+    subject { described_class.new(project, user).run_auto_devops_pipeline? }
 
-    context 'when neither pipeline setting is true' do
-      let(:params) { {} }
+    context 'when master contains a .gitlab-ci.yml file' do
+      before do
+        allow(project.repository).to receive(:gitlab_ci_yml).and_return("script: ['test']")
+      end
 
       it { is_expected.to eq(false) }
     end
 
-    context 'when run_auto_devops_pipeline_explicit is true' do
-      let(:params) { { run_auto_devops_pipeline_explicit: 'true' } }
+    context 'when auto devops is explicitly enabled' do
+      before do
+        project.create_auto_devops!(enabled: true)
+      end
 
       it { is_expected.to eq(true) }
     end
 
-    context 'when run_auto_devops_pipeline_implicit is true' do
-      let(:params) { { run_auto_devops_pipeline_implicit: 'true' } }
+    context 'when auto devops is explicitly disabled' do
+      before do
+        project.create_auto_devops!(enabled: false)
+      end
 
-      it { is_expected.to eq(true) }
+      it { is_expected.to eq(false) }
+    end
+
+    context 'when auto devops is set to instance setting' do
+      before do
+        project.create_auto_devops!(enabled: nil)
+        allow(project.auto_devops).to receive(:previous_changes).and_return('enabled' => true)
+      end
+
+      context 'when auto devops is enabled system-wide' do
+        before do
+          stub_application_setting(auto_devops_enabled: true)
+        end
+
+        it { is_expected.to eq(true) }
+      end
+
+      context 'when auto devops is disabled system-wide' do
+        before do
+          stub_application_setting(auto_devops_enabled: false)
+        end
+
+        it { is_expected.to eq(false) }
+      end
     end
   end