blob: 1f1b00e324e2ce6539acfe902f29a914801ef85b (
plain)
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, feature_category: :code_review_workflow 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
|