spec/workers/delete_diff_files_worker_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

# frozen_string_literal: true

require 'spec_helper'

RSpec.describe DeleteDiffFilesWorker do
  describe '#perform' do
    let(:merge_request) { create(:merge_request) }
    let(:merge_request_diff) { merge_request.merge_request_diff }

    it 'deletes all merge request diff files' do
      expect { described_class.new.perform(merge_request_diff.id) }
        .to change { merge_request_diff.merge_request_diff_files.count }
        .from(20).to(0)
    end

    it 'updates state to without_files' do
      expect { described_class.new.perform(merge_request_diff.id) }
        .to change { merge_request_diff.reload.state }
        .from('collected').to('without_files')
    end

    it 'resets the files_count of the diff' do
      expect { described_class.new.perform(merge_request_diff.id) }
        .to change { merge_request_diff.reload.files_count }
        .from(20).to(0)
    end

    it 'does nothing if diff was already marked as "without_files"' do
      merge_request_diff.clean!

      expect_any_instance_of(MergeRequestDiff).not_to receive(:clean!)

      described_class.new.perform(merge_request_diff.id)
    end

    it 'rollsback if something goes wrong' do
      error = RuntimeError.new('something went wrong')

      expect(MergeRequestDiffFile).to receive_message_chain(:where, :delete_all)
        .and_raise(error)

      expect { described_class.new.perform(merge_request_diff.id) }
        .to raise_error(error)

      merge_request_diff.reload

      expect(merge_request_diff.state).to eq('collected')
      expect(merge_request_diff.merge_request_diff_files.count).to eq(20)
    end
  end
end