Merge branch 'ccr/6274/add_weight_to_payload' into 'master'

Add weight to issue hook

Closes #6274

See merge request gitlab-org/gitlab-ee!6859

ee/changelogs/unreleased/ccr-6274-add_weight_to_payload.yml

+---
+title: Add weight to issue hook
+merge_request:
+author:
+type: fixed

ee/lib/ee/gitlab/hook_data/issue_builder.rb

+# frozen_string_literal: true
+module EE
+  module Gitlab
+    module HookData
+      module IssueBuilder
+        extend ActiveSupport::Concern
+
+        EE_SAFE_HOOK_ATTRIBUTES = %i[
+          weight
+        ].freeze
+
+        class_methods do
+          extend ::Gitlab::Utils::Override
+
+          override :safe_hook_attributes
+          def safe_hook_attributes
+            super + EE_SAFE_HOOK_ATTRIBUTES
+          end
+        end
+      end
+    end
+  end
+end

ee/spec/lib/ee/gitlab/hook_data/issue_builder_spec.rb

+# frozen_string_literal: true
+require 'spec_helper'
+
+describe Gitlab::HookData::IssueBuilder do
+  set(:issue) { create(:issue) }
+  let(:builder) { described_class.new(issue) }
+
+  describe '#build' do
+    let(:data) { builder.build }
+
+    it 'includes safe attribute' do
+      %w[
+        assignee_id
+        author_id
+        closed_at
+        confidential
+        created_at
+        description
+        due_date
+        id
+        iid
+        last_edited_at
+        last_edited_by_id
+        milestone_id
+        moved_to_id
+        project_id
+        relative_position
+        state
+        time_estimate
+        title
+        updated_at
+        updated_by_id
+        weight
+      ].each do |key|
+        expect(data).to include(key)
+      end
+    end
+
+    it 'includes additional attr' do
+      expect(data).to include(:weight)
+    end
+
+    context 'when the issue has an image in the description' do
+      let(:issue_with_description) { create(:issue, description: 'test![Issue_Image](/uploads/abc/Issue_Image.png)') }
+      let(:builder) { described_class.new(issue_with_description) }
+
+      it 'sets the image to use an absolute URL' do
+        expect(data[:description]).to eq("test![Issue_Image](#{Settings.gitlab.url}/uploads/abc/Issue_Image.png)")
+      end
+    end
+  end
+end

lib/gitlab/hook_data/issuable_builder.rb

@@ -28,7 +28,7 @@ module Gitlab
       end
 
       def safe_keys
-        issuable_builder::SAFE_HOOK_ATTRIBUTES + issuable_builder::SAFE_HOOK_RELATIONS
+        issuable_builder.safe_hook_attributes + issuable_builder::SAFE_HOOK_RELATIONS
       end
 
       private

lib/gitlab/hook_data/issue_builder.rb

 module Gitlab
   module HookData
     class IssueBuilder < BaseBuilder
-      SAFE_HOOK_ATTRIBUTES = %i[
+      prepend ::EE::Gitlab::HookData::IssueBuilder
+
+      SAFE_HOOK_RELATIONS = %i[
+        assignees
+        labels
+        total_time_spent
+      ].freeze
+
+      def self.safe_hook_attributes
+        %i[
           assignee_id
           author_id
           closed_at
@@ -23,12 +32,7 @@ module Gitlab
           updated_at
           updated_by_id
         ].freeze
-
-      SAFE_HOOK_RELATIONS = %i[
-        assignees
-        labels
-        total_time_spent
-      ].freeze
+      end
 
       alias_method :issue, :object
 
@@ -43,7 +47,7 @@ module Gitlab
           assignee_id: issue.assignee_ids.first # This key is deprecated
         }
 
-        issue.attributes.with_indifferent_access.slice(*SAFE_HOOK_ATTRIBUTES)
+        issue.attributes.with_indifferent_access.slice(*self.class.safe_hook_attributes)
           .merge!(attrs)
       end
     end

lib/gitlab/hook_data/merge_request_builder.rb

 module Gitlab
   module HookData
     class MergeRequestBuilder < BaseBuilder
-      SAFE_HOOK_ATTRIBUTES = %i[
+      def self.safe_hook_attributes
+        %i[
           assignee_id
           author_id
           created_at
@@ -28,6 +29,7 @@ module Gitlab
           updated_at
           updated_by_id
         ].freeze
+      end
 
       SAFE_HOOK_RELATIONS = %i[
         assignee
@@ -50,7 +52,7 @@ module Gitlab
           human_time_estimate: merge_request.human_time_estimate
         }
 
-        merge_request.attributes.with_indifferent_access.slice(*SAFE_HOOK_ATTRIBUTES)
+        merge_request.attributes.with_indifferent_access.slice(*self.class.safe_hook_attributes)
           .merge!(attrs)
       end
     end