Merge branch 'fix-udpate-head-pipeline-method' into 'master'

Fix unexpected exception by failure of finding an actual head pipeline

Closes #56113

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

Author: grzesiek

app/models/merge_request.rb

@@ -1108,9 +1108,10 @@ def all_pipelines(shas: all_commit_shas)
   end
 
   def update_head_pipeline
-    self.head_pipeline = find_actual_head_pipeline
-
-    update_column(:head_pipeline_id, head_pipeline.id) if head_pipeline_id_changed?
+    find_actual_head_pipeline.try do |pipeline|
+      self.head_pipeline = pipeline
+      update_column(:head_pipeline_id, head_pipeline.id) if head_pipeline_id_changed?
+    end
   end
 
   def merge_request_pipeline_exists?

changelogs/unreleased/fix-udpate-head-pipeline-method.yml

@@ -0,0 +1,5 @@
+---
+title: Fix unexpected exception by failure of finding an actual head pipeline
+merge_request: 24257
+author:
+type: fixed

spec/models/merge_request_spec.rb

@@ -1418,6 +1418,23 @@ def set_compare(merge_request)
           .to change { merge_request.reload.head_pipeline }
           .from(nil).to(pipeline)
       end
+
+      context 'when merge request has already had head pipeline' do
+        before do
+          merge_request.update!(head_pipeline: pipeline)
+        end
+
+        context 'when failed to find an actual head pipeline' do
+          before do
+            allow(merge_request).to receive(:find_actual_head_pipeline) { }
+          end
+
+          it 'does not update the current head pipeline' do
+            expect { subject }
+              .not_to change { merge_request.reload.head_pipeline }
+          end
+        end
+      end
     end
 
     context 'when there are no pipelines with the diff head sha' do