spec/services/users/migrate_records_to_ghost_user_in_batches_service_spec.rb


1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65

# frozen_string_literal: true

require 'spec_helper'

RSpec.describe Users::MigrateRecordsToGhostUserInBatchesService, feature_category: :user_management do
  let(:service) { described_class.new }

  let_it_be(:ghost_user_migration) { create(:ghost_user_migration) }

  describe '#execute' do
    it 'stops when execution time limit reached' do
      expect_next_instance_of(::Gitlab::Utils::ExecutionTracker) do |tracker|
        expect(tracker).to receive(:over_limit?).and_return(true)
      end

      expect(Users::MigrateRecordsToGhostUserService).not_to receive(:new)

      service.execute
    end

    it 'calls Users::MigrateRecordsToGhostUserService' do
      expect_next_instance_of(Users::MigrateRecordsToGhostUserService) do |service|
        expect(service).to(
          receive(:execute)
            .with(hard_delete: ghost_user_migration.hard_delete))
      end

      service.execute
    end

    it 'process jobs ordered by the consume_after timestamp' do
      older_ghost_user_migration = create(
        :ghost_user_migration,
        user: create(:user),
        consume_after: 5.minutes.ago
      )

      # setup execution tracker to only allow a single job to be processed
      allow_next_instance_of(::Gitlab::Utils::ExecutionTracker) do |tracker|
        allow(tracker).to receive(:over_limit?).and_return(false, true)
      end

      expect(Users::MigrateRecordsToGhostUserService).to(
        receive(:new).with(
          older_ghost_user_migration.user,
          older_ghost_user_migration.initiator_user,
          any_args
        )
      ).and_call_original

      service.execute
    end

    it 'reschedules job in case of an error', :freeze_time do
      expect_next_instance_of(Users::MigrateRecordsToGhostUserService) do |service|
        expect(service).to(receive(:execute)).and_raise(ActiveRecord::QueryCanceled)
      end
      expect(Gitlab::ErrorTracking).to receive(:track_exception)

      expect { service.execute }.to(
        change { ghost_user_migration.reload.consume_after }
          .to(30.minutes.from_now))
    end
  end
end