summaryrefslogtreecommitdiff
path: root/spec/workers/design_management/new_version_worker_spec.rb
blob: 3320d7a062dd1d29d6d1cde2b6fecad0c1beff2c (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
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
# frozen_string_literal: true

require 'spec_helper'

RSpec.describe DesignManagement::NewVersionWorker do
  describe '#perform' do
    let(:worker) { described_class.new }

    context 'the id is wrong or out-of-date' do
      let(:version_id) { -1 }

      it 'does not create system notes' do
        expect(SystemNoteService).not_to receive(:design_version_added)

        worker.perform(version_id)
      end

      it 'does not invoke GenerateImageVersionsService' do
        expect(DesignManagement::GenerateImageVersionsService).not_to receive(:new)

        worker.perform(version_id)
      end

      it 'logs the reason for this failure' do
        expect(Sidekiq.logger).to receive(:warn)
          .with(an_instance_of(ActiveRecord::RecordNotFound))

        worker.perform(version_id)
      end
    end

    context 'the version id is valid' do
      let_it_be(:version) { create(:design_version, :with_lfs_file, designs_count: 2) }

      it 'creates a system note' do
        expect { worker.perform(version.id) }.to change { Note.system.count }.by(1)
      end

      it 'does not create a system note if skip_system_notes is true' do
        expect { worker.perform(version.id, true) }.not_to change { Note.system.count }
      end

      it 'invokes GenerateImageVersionsService' do
        expect_next_instance_of(DesignManagement::GenerateImageVersionsService) do |service|
          expect(service).to receive(:execute)
        end

        worker.perform(version.id)
      end

      it 'does not log anything' do
        expect(Sidekiq.logger).not_to receive(:warn)

        worker.perform(version.id)
      end
    end

    context 'the version includes multiple types of action' do
      let_it_be(:version) do
        create(:design_version, :with_lfs_file,
               created_designs: create_list(:design, 1, :with_lfs_file),
               modified_designs: create_list(:design, 1))
      end

      it 'creates two system notes' do
        expect { worker.perform(version.id) }.to change { Note.system.count }.by(2)
      end

      it 'calls design_version_added' do
        expect(SystemNoteService).to receive(:design_version_added).with(version)

        worker.perform(version.id)
      end
    end
  end
end