Merge branch 'ban-ai-never-on-set-experiments' into 'master'

Set Experiments Checkbox to 'false' when 'never_on' is selected in Duo Availability

See merge request !163799



Merged-by: Dmitry Gruzd <dgruzd@gitlab.com>
Approved-by: Jeff Park <jpark@gitlab.com>
Approved-by: Dmitry Gruzd <dgruzd@gitlab.com>
Co-authored-by: Lindsey Shelton <lshelton@gitlab.com>

ee/app/models/ee/application_setting.rb

@@ -537,6 +537,7 @@ def duo_availability=(value)
       else
         self.duo_features_enabled = false
         self.lock_duo_features_enabled = true
+        self.instance_level_ai_beta_features_enabled = false
       end
     end
 

ee/app/models/ee/namespace_setting.rb

@@ -119,6 +119,7 @@ def duo_availability=(value)
         else
           self.duo_features_enabled = false
           self.lock_duo_features_enabled = true
+          self.experiment_features_enabled = false
         end
       end
 

ee/spec/models/application_setting_spec.rb

@@ -202,6 +202,14 @@
           expect(setting.lock_duo_features_enabled).to be lock_duo_features_enabled_expectation
         end
       end
+
+      context 'when value is "never_on"' do
+        it 'disables instance level AI and beta features' do
+          setting.duo_availability = "never_on"
+
+          expect(setting.instance_level_ai_beta_features_enabled).to be false
+        end
+      end
     end
 
     context 'when validating geo_node_allowed_ips', feature_category: :geo_replication do

ee/spec/models/namespace_setting_spec.rb

@@ -333,6 +333,14 @@
         expect(setting.duo_availability).to eq(expectation)
       end
     end
+
+    context 'when value is "never_on"' do
+      it 'sets experiment_features_enabled to false' do
+        setting.duo_availability = "never_on"
+
+        expect(setting.experiment_features_enabled).to be false
+      end
+    end
   end
 
   describe '#duo_availability=' do