Merge branch '52560-fix-duplicate-tag-system-hooks' into 'master'

Only execute system hooks once when pushing tags

Closes #52560

See merge request gitlab-org/gitlab-ce!26888

1 files changed, 14 insertions, 0 deletions

diff --git a/spec/services/git/tag_push_service_spec.rb b/spec/services/git/tag_push_service_spec.rb
index e151db5827f..2d960fc9f08 100644
--- a/spec/services/git/tag_push_service_spec.rb
+++ b/spec/services/git/tag_push_service_spec.rb
@@ -31,6 +31,20 @@ describe Git::TagPushService do
     end
   end
 
+  describe 'System Hooks' do
+    let!(:push_data) { service.tap(&:execute).push_data }
+
+    it "executes system hooks after pushing a tag" do
+      expect_next_instance_of(SystemHooksService) do |system_hooks_service|
+        expect(system_hooks_service)
+          .to receive(:execute_hooks)
+          .with(push_data, :tag_push_hooks)
+      end
+
+      service.execute
+    end
+  end
+
   describe "Pipelines" do
     subject { service.execute }