BigW Consortium Gitlab

commit_range_spec.rb 4.87 KB
Newer Older
1 2
require 'spec_helper'

Douwe Maan committed
3
describe CommitRange, models: true do
4 5 6 7 8 9
  describe 'modules' do
    subject { described_class }

    it { is_expected.to include_module(Referable) }
  end

10 11 12
  let!(:project) { create(:project, :public) }
  let!(:commit1) { project.commit("HEAD~2") }
  let!(:commit2) { project.commit }
13

14 15 16 17 18 19 20 21
  let(:sha_from) { commit1.short_id }
  let(:sha_to)   { commit2.short_id }

  let(:full_sha_from) { commit1.id }
  let(:full_sha_to)   { commit2.id }

  let(:range)  { described_class.new("#{sha_from}...#{sha_to}", project) }
  let(:range2) { described_class.new("#{sha_from}..#{sha_to}", project) }
22 23

  it 'raises ArgumentError when given an invalid range string' do
24
    expect { described_class.new("Foo", project) }.to raise_error(ArgumentError)
25 26
  end

27 28 29 30 31 32 33 34 35 36
  describe '#initialize' do
    it 'does not modify strings in-place' do
      input = "#{sha_from}...#{sha_to}   "

      described_class.new(input, project)

      expect(input).to eq("#{sha_from}...#{sha_to}   ")
    end
  end

37 38
  describe '#to_s' do
    it 'is correct for three-dot syntax' do
39
      expect(range.to_s).to eq "#{full_sha_from}...#{full_sha_to}"
40
    end
41

42
    it 'is correct for two-dot syntax' do
43
      expect(range2.to_s).to eq "#{full_sha_from}..#{full_sha_to}"
44
    end
45 46 47
  end

  describe '#to_reference' do
48
    let(:cross) { create(:project) }
49

50
    it 'returns a String reference to the object' do
51
      expect(range.to_reference).to eq "#{full_sha_from}...#{full_sha_to}"
52 53 54
    end

    it 'returns a String reference to the object' do
55
      expect(range2.to_reference).to eq "#{full_sha_from}..#{full_sha_to}"
56 57 58
    end

    it 'supports a cross-project reference' do
59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75
      expect(range.to_reference(cross)).to eq "#{project.to_reference}@#{full_sha_from}...#{full_sha_to}"
    end
  end

  describe '#reference_link_text' do
    let(:cross) { create(:project) }

    it 'returns a String reference to the object' do
      expect(range.reference_link_text).to eq "#{sha_from}...#{sha_to}"
    end

    it 'returns a String reference to the object' do
      expect(range2.reference_link_text).to eq "#{sha_from}..#{sha_to}"
    end

    it 'supports a cross-project reference' do
      expect(range.reference_link_text(cross)).to eq "#{project.to_reference}@#{sha_from}...#{sha_to}"
76 77 78
    end
  end

79
  describe '#reference_title' do
80
    it 'returns the correct String for three-dot ranges' do
81
      expect(range.reference_title).to eq "Commits #{full_sha_from} through #{full_sha_to}"
82
    end
83 84

    it 'returns the correct String for two-dot ranges' do
85
      expect(range2.reference_title).to eq "Commits #{full_sha_from}^ through #{full_sha_to}"
86
    end
87 88 89 90 91 92 93 94
  end

  describe '#to_param' do
    it 'includes the correct keys' do
      expect(range.to_param.keys).to eq %i(from to)
    end

    it 'includes the correct values for a three-dot range' do
95
      expect(range.to_param).to eq({ from: full_sha_from, to: full_sha_to })
96 97 98
    end

    it 'includes the correct values for a two-dot range' do
99
      expect(range2.to_param).to eq({ from: full_sha_from + '^', to: full_sha_to })
100 101 102
    end
  end

103
  describe '#exclude_start?' do
104
    it 'is false for three-dot ranges' do
105
      expect(range.exclude_start?).to eq false
106 107 108
    end

    it 'is true for two-dot ranges' do
109
      expect(range2.exclude_start?).to eq true
110 111 112 113
    end
  end

  describe '#valid_commits?' do
114 115 116
    context 'with a valid repo' do
      before do
        expect(project).to receive(:valid_repo?).and_return(true)
117 118
      end

119 120 121
      it 'is false when `sha_from` is invalid' do
        expect(project).to receive(:commit).with(sha_from).and_return(nil)
        expect(project).to receive(:commit).with(sha_to).and_call_original
122

123 124
        expect(range).not_to be_valid_commits
      end
125

126 127 128
      it 'is false when `sha_to` is invalid' do
        expect(project).to receive(:commit).with(sha_from).and_call_original
        expect(project).to receive(:commit).with(sha_to).and_return(nil)
129

130 131
        expect(range).not_to be_valid_commits
      end
132

133 134
      it 'is true when both `sha_from` and `sha_to` are valid' do
        expect(range).to be_valid_commits
135
      end
136
    end
137

138 139 140 141
    context 'without a valid repo' do
      before do
        expect(project).to receive(:valid_repo?).and_return(false)
      end
142

143 144
      it 'returns false' do
        expect(range).not_to be_valid_commits
145 146 147
      end
    end
  end
Yorick Peterse committed
148 149 150 151 152 153

  describe '#has_been_reverted?' do
    it 'returns true if the commit has been reverted' do
      issue = create(:issue)

      create(:note_on_issue,
154
             noteable: issue,
Yorick Peterse committed
155
             system: true,
156 157
             note: commit1.revert_description,
             project: issue.project)
Yorick Peterse committed
158 159 160 161 162 163 164 165 166 167 168 169 170 171

      expect_any_instance_of(Commit).to receive(:reverts_commit?).
        with(commit1).
        and_return(true)

      expect(commit1.has_been_reverted?(nil, issue)).to eq(true)
    end

    it 'returns false a commit has not been reverted' do
      issue = create(:issue)

      expect(commit1.has_been_reverted?(nil, issue)).to eq(false)
    end
  end
172
end