spec/models/suggestion_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
66
67
68
69
70
71
72
73

# frozen_string_literal: true

require 'spec_helper'

describe Suggestion do
  let(:suggestion) { create(:suggestion) }

  describe 'associations' do
    it { is_expected.to belong_to(:note) }
  end

  describe 'validations' do
    it { is_expected.to validate_presence_of(:note) }

    context 'when suggestion is applied' do
      before do
        allow(subject).to receive(:applied?).and_return(true)
      end

      it { is_expected.to validate_presence_of(:commit_id) }
    end
  end

  describe '#diff_lines' do
    let(:suggestion) { create(:suggestion, :content_from_repo) }

    it 'returns parsed diff lines' do
      expected_diff_lines = Gitlab::Diff::SuggestionDiff.new(suggestion).diff_lines
      diff_lines = suggestion.diff_lines

      expect(diff_lines.size).to eq(expected_diff_lines.size)
      expect(diff_lines).to all(be_a(Gitlab::Diff::Line))

      expected_diff_lines.each_with_index do |expected_line, index|
        expect(diff_lines[index].to_hash).to eq(expected_line.to_hash)
      end
    end
  end

  describe '#appliable?' do
    context 'when note does not support suggestions' do
      it 'returns false' do
        expect_next_instance_of(DiffNote) do |note|
          allow(note).to receive(:supports_suggestion?) { false }
        end

        expect(suggestion).not_to be_appliable
      end
    end

    context 'when patch is already applied' do
      let(:suggestion) { create(:suggestion, :applied) }

      it 'returns false' do
        expect(suggestion).not_to be_appliable
      end
    end

    context 'when merge request is not opened' do
      let(:merge_request) { create(:merge_request, :merged) }
      let(:note) do
        create(:diff_note_on_merge_request, project: merge_request.project,
                                            noteable: merge_request)
      end

      let(:suggestion) { create(:suggestion, note: note) }

      it 'returns false' do
        expect(suggestion).not_to be_appliable
      end
    end
  end
end