Merge branch 'pks-checks-skip-integrity-lfs-deletions' into 'master'

checks: Skip LFS checks when deleting refs

See merge request gitlab-org/gitlab!55609

changelogs/unreleased/pks-checks-skip-integrity-lfs-deletions.yml

+---
+title: 'checks: Skip LFS checks when deleting refs'
+merge_request: 55609
+author:
+type: fixed

lib/gitlab/checks/lfs_check.rb

@@ -13,6 +13,7 @@ module Gitlab
 
         return unless project.lfs_enabled?
         return if skip_lfs_integrity_check
+        return if deletion?
 
         logger.log_timed(LOG_MESSAGE) do
           lfs_check = Checks::LfsIntegrity.new(project, newrev, logger.time_left)

spec/lib/gitlab/checks/lfs_check_spec.rb

@@ -39,13 +39,26 @@ RSpec.describe Gitlab::Checks::LfsCheck do
         end
       end
 
-      context 'deletion' do
-        let(:changes) { { oldrev: oldrev, ref: ref } }
+      context 'with deletion' do
+        shared_examples 'a skipped integrity check' do
+          it 'skips integrity check' do
+            expect(project.repository).not_to receive(:new_objects)
+            expect_any_instance_of(Gitlab::Git::LfsChanges).not_to receive(:new_pointers)
+
+            subject.validate!
+          end
+        end
 
-        it 'skips integrity check' do
-          expect(project.repository).not_to receive(:new_objects)
+        context 'with missing newrev' do
+          it_behaves_like 'a skipped integrity check' do
+            let(:changes) { { oldrev: oldrev, ref: ref } }
+          end
+        end
 
-          subject.validate!
+        context 'with blank newrev' do
+          it_behaves_like 'a skipped integrity check' do
+            let(:changes) { { oldrev: oldrev, newrev: Gitlab::Git::BLANK_SHA, ref: ref } }
+          end
         end
       end