summaryrefslogtreecommitdiff
path: root/spec/lib/gitlab/markdown/commit_reference_filter_spec.rb
blob: d080efbf3d4fa011a7cea580601322c7120c2189 (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
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
require 'spec_helper'

module Gitlab::Markdown
  describe CommitReferenceFilter do
    include FilterSpecHelper

    let(:project) { create(:project, :public) }
    let(:commit)  { project.commit }

    it 'requires project context' do
      expect { described_class.call('') }.to raise_error(ArgumentError, /:project/)
    end

    %w(pre code a style).each do |elem|
      it "ignores valid references contained inside '#{elem}' element" do
        exp = act = "<#{elem}>Commit #{commit.id}</#{elem}>"
        expect(filter(act).to_html).to eq exp
      end
    end

    context 'internal reference' do
      let(:reference) { commit.id }

      # Let's test a variety of commit SHA sizes just to be paranoid
      [6, 8, 12, 18, 20, 32, 40].each do |size|
        it "links to a valid reference of #{size} characters" do
          doc = filter("See #{reference[0...size]}")

          expect(doc.css('a').first.text).to eq commit.short_id
          expect(doc.css('a').first.attr('href')).
            to eq urls.namespace_project_commit_url(project.namespace, project, reference)
        end
      end

      it 'always uses the short ID as the link text' do
        doc = filter("See #{commit.id}")
        expect(doc.text).to eq "See #{commit.short_id}"

        doc = filter("See #{commit.id[0...6]}")
        expect(doc.text).to eq "See #{commit.short_id}"
      end

      it 'links with adjacent text' do
        doc = filter("See (#{reference}.)")
        expect(doc.to_html).to match(/\(<a.+>#{commit.short_id}<\/a>\.\)/)
      end

      it 'ignores invalid commit IDs' do
        invalid = invalidate_reference(reference)
        exp = act = "See #{invalid}"

        expect(project).to receive(:valid_repo?).and_return(true)
        expect(project.repository).to receive(:commit).with(invalid)
        expect(filter(act).to_html).to eq exp
      end

      it 'includes a title attribute' do
        doc = filter("See #{reference}")
        expect(doc.css('a').first.attr('title')).to eq commit.link_title
      end

      it 'escapes the title attribute' do
        allow_any_instance_of(Commit).to receive(:title).and_return(%{"></a>whatever<a title="})

        doc = filter("See #{reference}")
        expect(doc.text).to eq "See #{commit.short_id}"
      end

      it 'includes default classes' do
        doc = filter("See #{reference}")
        expect(doc.css('a').first.attr('class')).to eq 'gfm gfm-commit'
      end

      it 'includes a data-project attribute' do
        doc = filter("See #{reference}")
        link = doc.css('a').first

        expect(link).to have_attribute('data-project')
        expect(link.attr('data-project')).to eq project.id.to_s
      end

      it 'includes a data-commit attribute' do
        doc = filter("See #{reference}")
        link = doc.css('a').first

        expect(link).to have_attribute('data-commit')
        expect(link.attr('data-commit')).to eq commit.id
      end

      it 'supports an :only_path context' do
        doc = filter("See #{reference}", only_path: true)
        link = doc.css('a').first.attr('href')

        expect(link).not_to match %r(https?://)
        expect(link).to eq urls.namespace_project_commit_url(project.namespace, project, reference, only_path: true)
      end

      it 'adds to the results hash' do
        result = reference_pipeline_result("See #{reference}")
        expect(result[:references][:commit]).not_to be_empty
      end
    end

    context 'cross-project reference' do
      let(:namespace) { create(:namespace, name: 'cross-reference') }
      let(:project2)  { create(:project, :public, namespace: namespace) }
      let(:commit)    { project2.commit }
      let(:reference) { commit.to_reference(project) }

      it 'links to a valid reference' do
        doc = filter("See #{reference}")

        expect(doc.css('a').first.attr('href')).
          to eq urls.namespace_project_commit_url(project2.namespace, project2, commit.id)
      end

      it 'links with adjacent text' do
        doc = filter("Fixed (#{reference}.)")

        exp = Regexp.escape(project2.to_reference)
        expect(doc.to_html).to match(/\(<a.+>#{exp}@#{commit.short_id}<\/a>\.\)/)
      end

      it 'ignores invalid commit IDs on the referenced project' do
        exp = act = "Committed #{invalidate_reference(reference)}"
        expect(filter(act).to_html).to eq exp
      end

      it 'adds to the results hash' do
        result = reference_pipeline_result("See #{reference}")
        expect(result[:references][:commit]).not_to be_empty
      end
    end
  end
end