Merge branch '404901-add-license-check-to-summarize_diff' into 'master'

Put summarize_diff_quick_action behind license check

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/117937



Merged-by: Thong Kuah <tkuah@gitlab.com>
Approved-by: Thong Kuah <tkuah@gitlab.com>
Approved-by: Gary Holtz <gholtz@gitlab.com>
Co-authored-by: Kerri Miller <kerrizor@kerrizor.com>

ee/app/models/gitlab_subscriptions/features.rb

@@ -239,6 +239,7 @@ class Features
       security_orchestration_policies
       security_training
       ssh_key_expiration_policy
+      summarize_diff_quick_action
       summarize_notes
       stale_runner_cleanup_for_namespace
       status_page

ee/lib/ee/gitlab/quick_actions/merge_request_actions.rb

@@ -28,7 +28,8 @@ module MergeRequestActions
           types MergeRequest
           condition do
             ::Feature.enabled?(:openai_experimentation, current_user) &&
-              ::Feature.enabled?(:summarize_diff_quick_action, current_user)
+              ::Feature.enabled?(:summarize_diff_quick_action, current_user) &&
+              ::License.feature_available?(:summarize_diff_quick_action)
           end
           command :summarize_diff do
             ::MergeRequests::Llm::SummarizeMergeRequestWorker.new.perform(

ee/spec/services/quick_actions/interpret_service_spec.rb

@@ -453,6 +453,10 @@
     context "summarize_diff command" do
       let(:content) { "/summarize_diff" }
 
+      before do
+        stub_licensed_features(summarize_diff_quick_action: true)
+      end
+
       context "when :openai_experimentation feature flag is disabled" do
         before do
           stub_feature_flags(openai_experimentation: false)
@@ -482,6 +486,18 @@
           stub_feature_flags(openai_experimentation: true)
         end
 
+        context "when summarize_diff_quick_action is disabled" do
+          before do
+            stub_licensed_features(summarize_diff_quick_action: false)
+          end
+
+          it "doesn't apply /summarize_diff" do
+            _, _, msg = service.execute(content, merge_request)
+
+            expect(msg).to include("Could not apply summarize_diff command")
+          end
+        end
+
         it "applies /summarize_diff" do
           _, _, msg = service.execute(content, merge_request)