2015-02-12 13:53:23 -05:00
|
|
|
require 'spec_helper'
|
2012-08-01 20:26:21 -04:00
|
|
|
|
2017-04-03 05:55:15 -04:00
|
|
|
describe MarkupHelper do
|
2017-01-25 17:10:32 -05:00
|
|
|
let!(:project) { create(:project, :repository) }
|
2012-09-02 02:13:13 -04:00
|
|
|
|
2012-12-06 22:40:42 -05:00
|
|
|
let(:user) { create(:user, username: 'gfm') }
|
2015-04-21 09:13:40 -04:00
|
|
|
let(:commit) { project.commit }
|
2012-09-02 02:13:13 -04:00
|
|
|
let(:issue) { create(:issue, project: project) }
|
2013-04-25 10:15:33 -04:00
|
|
|
let(:merge_request) { create(:merge_request, source_project: project, target_project: project) }
|
2013-03-25 03:20:14 -04:00
|
|
|
let(:snippet) { create(:project_snippet, project: project) }
|
2012-09-02 02:13:13 -04:00
|
|
|
|
2012-08-01 20:26:21 -04:00
|
|
|
before do
|
2015-09-01 18:09:20 -04:00
|
|
|
# Ensure the generated reference links aren't redacted
|
|
|
|
project.team << [user, :master]
|
|
|
|
|
2012-09-02 02:13:13 -04:00
|
|
|
# Helper expects a @project instance variable
|
2015-09-01 18:09:20 -04:00
|
|
|
helper.instance_variable_set(:@project, project)
|
|
|
|
|
|
|
|
# Stub the `current_user` helper
|
|
|
|
allow(helper).to receive(:current_user).and_return(user)
|
2012-08-01 20:26:21 -04:00
|
|
|
end
|
|
|
|
|
2015-08-27 16:09:01 -04:00
|
|
|
describe "#markdown" do
|
2012-09-02 02:13:13 -04:00
|
|
|
describe "referencing multiple objects" do
|
2015-05-21 16:35:15 -04:00
|
|
|
let(:actual) { "#{merge_request.to_reference} -> #{commit.to_reference} -> #{issue.to_reference}" }
|
2012-09-02 02:13:13 -04:00
|
|
|
|
2016-07-25 14:16:19 -04:00
|
|
|
it "links to the merge request" do
|
2017-06-29 13:06:35 -04:00
|
|
|
expected = project_merge_request_path(project, merge_request)
|
2015-09-01 18:09:20 -04:00
|
|
|
expect(helper.markdown(actual)).to match(expected)
|
2012-08-01 20:26:21 -04:00
|
|
|
end
|
|
|
|
|
2016-07-25 14:16:19 -04:00
|
|
|
it "links to the commit" do
|
2017-06-29 13:06:35 -04:00
|
|
|
expected = project_commit_path(project, commit)
|
2015-09-01 18:09:20 -04:00
|
|
|
expect(helper.markdown(actual)).to match(expected)
|
2012-08-01 20:26:21 -04:00
|
|
|
end
|
|
|
|
|
2016-07-25 14:16:19 -04:00
|
|
|
it "links to the issue" do
|
2017-06-29 13:06:35 -04:00
|
|
|
expected = project_issue_path(project, issue)
|
2015-09-01 18:09:20 -04:00
|
|
|
expect(helper.markdown(actual)).to match(expected)
|
2012-08-01 20:26:21 -04:00
|
|
|
end
|
|
|
|
end
|
2015-09-18 14:05:27 -04:00
|
|
|
|
|
|
|
describe "override default project" do
|
|
|
|
let(:actual) { issue.to_reference }
|
2017-08-02 15:55:11 -04:00
|
|
|
let(:second_project) { create(:project, :public) }
|
2015-09-18 14:05:27 -04:00
|
|
|
let(:second_issue) { create(:issue, project: second_project) }
|
|
|
|
|
2016-07-25 14:16:19 -04:00
|
|
|
it 'links to the issue' do
|
2017-06-29 13:06:35 -04:00
|
|
|
expected = project_issue_path(second_project, second_issue)
|
2015-09-18 14:05:27 -04:00
|
|
|
expect(markdown(actual, project: second_project)).to match(expected)
|
|
|
|
end
|
|
|
|
end
|
2012-09-02 02:13:13 -04:00
|
|
|
end
|
2012-08-01 20:26:21 -04:00
|
|
|
|
2017-08-23 12:53:29 -04:00
|
|
|
describe '#markdown_field' do
|
|
|
|
let(:attribute) { :title }
|
|
|
|
|
|
|
|
describe 'with already redacted attribute' do
|
|
|
|
it 'returns the redacted attribute' do
|
|
|
|
commit.redacted_title_html = 'commit title'
|
|
|
|
|
|
|
|
expect(Banzai).not_to receive(:render_field)
|
|
|
|
|
|
|
|
expect(helper.markdown_field(commit, attribute)).to eq('commit title')
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe 'without redacted attribute' do
|
|
|
|
it 'renders the markdown value' do
|
|
|
|
expect(Banzai).to receive(:render_field).with(commit, attribute).and_call_original
|
|
|
|
|
|
|
|
helper.markdown_field(commit, attribute)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe '#link_to_markdown_field' do
|
|
|
|
let(:link) { '/commits/0a1b2c3d' }
|
|
|
|
let(:issues) { create_list(:issue, 2, project: project) }
|
|
|
|
|
|
|
|
it 'handles references nested in links with all the text' do
|
|
|
|
allow(commit).to receive(:title).and_return("This should finally fix #{issues[0].to_reference} and #{issues[1].to_reference} for real")
|
|
|
|
|
|
|
|
actual = helper.link_to_markdown_field(commit, :title, link)
|
|
|
|
doc = Nokogiri::HTML.parse(actual)
|
|
|
|
|
|
|
|
# Make sure we didn't create invalid markup
|
|
|
|
expect(doc.errors).to be_empty
|
|
|
|
|
|
|
|
# Leading commit link
|
|
|
|
expect(doc.css('a')[0].attr('href')).to eq link
|
|
|
|
expect(doc.css('a')[0].text).to eq 'This should finally fix '
|
|
|
|
|
|
|
|
# First issue link
|
|
|
|
expect(doc.css('a')[1].attr('href'))
|
|
|
|
.to eq project_issue_path(project, issues[0])
|
|
|
|
expect(doc.css('a')[1].text).to eq issues[0].to_reference
|
|
|
|
|
|
|
|
# Internal commit link
|
|
|
|
expect(doc.css('a')[2].attr('href')).to eq link
|
|
|
|
expect(doc.css('a')[2].text).to eq ' and '
|
|
|
|
|
|
|
|
# Second issue link
|
|
|
|
expect(doc.css('a')[3].attr('href'))
|
|
|
|
.to eq project_issue_path(project, issues[1])
|
|
|
|
expect(doc.css('a')[3].text).to eq issues[1].to_reference
|
|
|
|
|
|
|
|
# Trailing commit link
|
|
|
|
expect(doc.css('a')[4].attr('href')).to eq link
|
|
|
|
expect(doc.css('a')[4].text).to eq ' for real'
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe '#link_to_markdown' do
|
2017-01-25 17:10:32 -05:00
|
|
|
let(:link) { '/commits/0a1b2c3d' }
|
2017-01-26 07:09:29 -05:00
|
|
|
let(:issues) { create_list(:issue, 2, project: project) }
|
2012-08-01 20:26:21 -04:00
|
|
|
|
2016-07-25 14:16:19 -04:00
|
|
|
it 'handles references nested in links with all the text' do
|
2017-08-23 12:53:29 -04:00
|
|
|
actual = helper.link_to_markdown("This should finally fix #{issues[0].to_reference} and #{issues[1].to_reference} for real", link)
|
2015-04-20 15:43:27 -04:00
|
|
|
doc = Nokogiri::HTML.parse(actual)
|
2012-08-01 20:26:21 -04:00
|
|
|
|
2015-04-21 13:12:19 -04:00
|
|
|
# Make sure we didn't create invalid markup
|
|
|
|
expect(doc.errors).to be_empty
|
|
|
|
|
2012-09-02 02:13:13 -04:00
|
|
|
# Leading commit link
|
2017-01-25 17:10:32 -05:00
|
|
|
expect(doc.css('a')[0].attr('href')).to eq link
|
2015-04-20 15:43:27 -04:00
|
|
|
expect(doc.css('a')[0].text).to eq 'This should finally fix '
|
2012-08-01 20:26:21 -04:00
|
|
|
|
2012-09-02 02:13:13 -04:00
|
|
|
# First issue link
|
2017-06-21 09:48:12 -04:00
|
|
|
expect(doc.css('a')[1].attr('href'))
|
2017-06-29 13:06:35 -04:00
|
|
|
.to eq project_issue_path(project, issues[0])
|
2015-05-21 16:35:15 -04:00
|
|
|
expect(doc.css('a')[1].text).to eq issues[0].to_reference
|
2012-08-01 20:28:50 -04:00
|
|
|
|
2012-09-02 02:13:13 -04:00
|
|
|
# Internal commit link
|
2017-01-25 17:10:32 -05:00
|
|
|
expect(doc.css('a')[2].attr('href')).to eq link
|
2015-04-20 15:43:27 -04:00
|
|
|
expect(doc.css('a')[2].text).to eq ' and '
|
2012-08-01 20:28:50 -04:00
|
|
|
|
2012-09-02 02:13:13 -04:00
|
|
|
# Second issue link
|
2017-06-21 09:48:12 -04:00
|
|
|
expect(doc.css('a')[3].attr('href'))
|
2017-06-29 13:06:35 -04:00
|
|
|
.to eq project_issue_path(project, issues[1])
|
2015-05-21 16:35:15 -04:00
|
|
|
expect(doc.css('a')[3].text).to eq issues[1].to_reference
|
2012-09-02 02:13:13 -04:00
|
|
|
|
|
|
|
# Trailing commit link
|
2017-01-25 17:10:32 -05:00
|
|
|
expect(doc.css('a')[4].attr('href')).to eq link
|
2015-04-20 15:43:27 -04:00
|
|
|
expect(doc.css('a')[4].text).to eq ' for real'
|
2012-08-01 20:28:50 -04:00
|
|
|
end
|
|
|
|
|
2016-07-25 14:16:19 -04:00
|
|
|
it 'forwards HTML options' do
|
2017-08-23 12:53:29 -04:00
|
|
|
actual = helper.link_to_markdown("Fixed in #{commit.id}", link, class: 'foo')
|
2015-08-27 17:28:45 -04:00
|
|
|
doc = Nokogiri::HTML.parse(actual)
|
|
|
|
|
|
|
|
expect(doc.css('a')).to satisfy do |v|
|
|
|
|
# 'foo' gets added to all links
|
|
|
|
v.all? { |a| a.attr('class').match(/foo$/) }
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2012-09-19 19:42:26 -04:00
|
|
|
it "escapes HTML passed in as the body" do
|
2015-05-21 16:35:15 -04:00
|
|
|
actual = "This is a <h1>test</h1> - see #{issues[0].to_reference}"
|
2017-08-23 12:53:29 -04:00
|
|
|
expect(helper.link_to_markdown(actual, link))
|
2017-06-21 09:48:12 -04:00
|
|
|
.to match('<h1>test</h1>')
|
2012-09-19 19:42:26 -04:00
|
|
|
end
|
2015-05-29 01:56:30 -04:00
|
|
|
|
|
|
|
it 'ignores reference links when they are the entire body' do
|
|
|
|
text = issues[0].to_reference
|
2017-08-23 12:53:29 -04:00
|
|
|
act = helper.link_to_markdown(text, '/foo')
|
2015-05-29 01:56:30 -04:00
|
|
|
expect(act).to eq %Q(<a href="/foo">#{issues[0].to_reference}</a>)
|
|
|
|
end
|
2015-10-01 00:09:07 -04:00
|
|
|
|
2016-07-25 14:16:19 -04:00
|
|
|
it 'replaces commit message with emoji to link' do
|
2017-08-23 12:53:29 -04:00
|
|
|
actual = link_to_markdown(':book: Book', '/foo')
|
2017-06-21 09:48:12 -04:00
|
|
|
expect(actual)
|
|
|
|
.to eq '<gl-emoji title="open book" data-name="book" data-unicode-version="6.0">📖</gl-emoji><a href="/foo"> Book</a>'
|
2015-10-01 00:09:07 -04:00
|
|
|
end
|
2012-08-01 20:28:50 -04:00
|
|
|
end
|
2012-08-01 20:29:15 -04:00
|
|
|
|
2017-08-23 12:53:29 -04:00
|
|
|
describe '#link_to_html' do
|
|
|
|
it 'wraps the rendered content in a link' do
|
|
|
|
link = '/commits/0a1b2c3d'
|
|
|
|
issue = create(:issue, project: project)
|
|
|
|
|
|
|
|
rendered = helper.markdown("This should finally fix #{issue.to_reference} for real", pipeline: :single_line)
|
|
|
|
doc = Nokogiri::HTML.parse(rendered)
|
|
|
|
|
|
|
|
expect(doc.css('a')[0].attr('href'))
|
|
|
|
.to eq project_issue_path(project, issue)
|
|
|
|
expect(doc.css('a')[0].text).to eq issue.to_reference
|
|
|
|
|
|
|
|
wrapped = helper.link_to_html(rendered, link)
|
|
|
|
doc = Nokogiri::HTML.parse(wrapped)
|
|
|
|
|
|
|
|
expect(doc.css('a')[0].attr('href')).to eq link
|
|
|
|
expect(doc.css('a')[0].text).to eq 'This should finally fix '
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-04-13 18:05:42 -04:00
|
|
|
describe '#render_wiki_content' do
|
2013-03-14 02:31:08 -04:00
|
|
|
before do
|
2013-12-14 08:43:48 -05:00
|
|
|
@wiki = double('WikiPage')
|
2015-02-12 13:17:35 -05:00
|
|
|
allow(@wiki).to receive(:content).and_return('wiki content')
|
2016-06-08 01:02:50 -04:00
|
|
|
allow(@wiki).to receive(:slug).and_return('nested/page')
|
2016-01-11 23:10:08 -05:00
|
|
|
helper.instance_variable_set(:@project_wiki, @wiki)
|
2013-03-14 02:31:08 -04:00
|
|
|
end
|
|
|
|
|
2016-07-25 14:16:19 -04:00
|
|
|
it "uses Wiki pipeline for markdown files" do
|
2015-02-12 13:17:35 -05:00
|
|
|
allow(@wiki).to receive(:format).and_return(:markdown)
|
2013-03-14 02:31:08 -04:00
|
|
|
|
2017-04-24 06:20:46 -04:00
|
|
|
expect(helper).to receive(:markdown_unsafe).with('wiki content', pipeline: :wiki, project: project, project_wiki: @wiki, page_slug: "nested/page")
|
2013-03-14 02:31:08 -04:00
|
|
|
|
|
|
|
helper.render_wiki_content(@wiki)
|
|
|
|
end
|
|
|
|
|
2016-07-25 14:16:19 -04:00
|
|
|
it "uses Asciidoctor for asciidoc files" do
|
2015-05-12 19:07:48 -04:00
|
|
|
allow(@wiki).to receive(:format).and_return(:asciidoc)
|
|
|
|
|
2017-04-21 11:17:19 -04:00
|
|
|
expect(helper).to receive(:asciidoc_unsafe).with('wiki content')
|
2015-05-12 19:07:48 -04:00
|
|
|
|
|
|
|
helper.render_wiki_content(@wiki)
|
|
|
|
end
|
|
|
|
|
2016-07-25 14:16:19 -04:00
|
|
|
it "uses the Gollum renderer for all other file types" do
|
2015-02-12 13:17:35 -05:00
|
|
|
allow(@wiki).to receive(:format).and_return(:rdoc)
|
2013-12-14 08:43:48 -05:00
|
|
|
formatted_content_stub = double('formatted_content')
|
2015-02-12 13:17:35 -05:00
|
|
|
expect(formatted_content_stub).to receive(:html_safe)
|
|
|
|
allow(@wiki).to receive(:formatted_content).and_return(formatted_content_stub)
|
2013-03-14 02:31:08 -04:00
|
|
|
|
|
|
|
helper.render_wiki_content(@wiki)
|
|
|
|
end
|
|
|
|
end
|
2015-06-23 21:00:10 -04:00
|
|
|
|
2017-04-21 11:17:19 -04:00
|
|
|
describe 'markup' do
|
2017-04-06 10:47:52 -04:00
|
|
|
let(:content) { 'Noël' }
|
|
|
|
|
|
|
|
it 'preserves encoding' do
|
|
|
|
expect(content.encoding.name).to eq('UTF-8')
|
2017-04-21 11:17:19 -04:00
|
|
|
expect(helper.markup('foo.rst', content).encoding.name).to eq('UTF-8')
|
2017-04-06 10:47:52 -04:00
|
|
|
end
|
|
|
|
|
2017-04-21 11:17:19 -04:00
|
|
|
it "delegates to #markdown_unsafe when file name corresponds to Markdown" do
|
2017-04-06 10:47:52 -04:00
|
|
|
expect(helper).to receive(:gitlab_markdown?).with('foo.md').and_return(true)
|
2017-04-21 11:17:19 -04:00
|
|
|
expect(helper).to receive(:markdown_unsafe).and_return('NOEL')
|
2017-04-06 10:47:52 -04:00
|
|
|
|
2017-04-21 11:17:19 -04:00
|
|
|
expect(helper.markup('foo.md', content)).to eq('NOEL')
|
2017-04-06 10:47:52 -04:00
|
|
|
end
|
|
|
|
|
2017-04-21 11:17:19 -04:00
|
|
|
it "delegates to #asciidoc_unsafe when file name corresponds to AsciiDoc" do
|
2017-04-06 10:47:52 -04:00
|
|
|
expect(helper).to receive(:asciidoc?).with('foo.adoc').and_return(true)
|
2017-04-21 11:17:19 -04:00
|
|
|
expect(helper).to receive(:asciidoc_unsafe).and_return('NOEL')
|
2017-04-06 10:47:52 -04:00
|
|
|
|
2017-04-21 11:17:19 -04:00
|
|
|
expect(helper.markup('foo.adoc', content)).to eq('NOEL')
|
2017-04-06 10:47:52 -04:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-09-23 00:56:27 -04:00
|
|
|
describe '#first_line_in_markdown' do
|
|
|
|
it 'truncates Markdown properly' do
|
2017-03-14 02:24:56 -04:00
|
|
|
text = "@#{user.username}, can you look at this?\nHello world\n"
|
2015-09-23 00:56:27 -04:00
|
|
|
actual = first_line_in_markdown(text, 100, project: project)
|
|
|
|
|
|
|
|
doc = Nokogiri::HTML.parse(actual)
|
|
|
|
|
|
|
|
# Make sure we didn't create invalid markup
|
|
|
|
expect(doc.errors).to be_empty
|
|
|
|
|
|
|
|
# Leading user link
|
|
|
|
expect(doc.css('a').length).to eq(1)
|
|
|
|
expect(doc.css('a')[0].attr('href')).to eq user_path(user)
|
|
|
|
expect(doc.css('a')[0].text).to eq "@#{user.username}"
|
|
|
|
|
|
|
|
expect(doc.content).to eq "@#{user.username}, can you look at this?..."
|
|
|
|
end
|
2017-03-14 02:24:56 -04:00
|
|
|
|
|
|
|
it 'truncates Markdown with emoji properly' do
|
|
|
|
text = "foo :wink:\nbar :grinning:"
|
|
|
|
actual = first_line_in_markdown(text, 100, project: project)
|
|
|
|
|
|
|
|
doc = Nokogiri::HTML.parse(actual)
|
|
|
|
|
|
|
|
# Make sure we didn't create invalid markup
|
|
|
|
# But also account for the 2 errors caused by the unknown `gl-emoji` elements
|
|
|
|
expect(doc.errors.length).to eq(2)
|
|
|
|
|
|
|
|
expect(doc.css('gl-emoji').length).to eq(2)
|
|
|
|
expect(doc.css('gl-emoji')[0].attr('data-name')).to eq 'wink'
|
|
|
|
expect(doc.css('gl-emoji')[1].attr('data-name')).to eq 'grinning'
|
|
|
|
|
|
|
|
expect(doc.content).to eq "foo 😉\nbar 😀"
|
|
|
|
end
|
2015-09-23 00:56:27 -04:00
|
|
|
end
|
2016-12-21 11:41:33 -05:00
|
|
|
|
|
|
|
describe '#cross_project_reference' do
|
|
|
|
it 'shows the full MR reference' do
|
2017-07-20 05:34:09 -04:00
|
|
|
expect(helper.cross_project_reference(project, merge_request)).to include(project.full_path)
|
2016-12-21 11:41:33 -05:00
|
|
|
end
|
|
|
|
|
|
|
|
it 'shows the full issue reference' do
|
2017-07-20 05:34:09 -04:00
|
|
|
expect(helper.cross_project_reference(project, issue)).to include(project.full_path)
|
2016-12-21 11:41:33 -05:00
|
|
|
end
|
|
|
|
end
|
2012-08-01 20:26:21 -04:00
|
|
|
end
|