2012-08-02 02:26:21 +02:00
|
|
|
require "spec_helper"
|
|
|
|
|
2012-08-08 10:52:09 +02:00
|
|
|
describe GitlabMarkdownHelper do
|
2012-09-02 08:13:13 +02:00
|
|
|
let!(:project) { create(:project) }
|
|
|
|
|
2012-12-07 04:40:42 +01:00
|
|
|
let(:user) { create(:user, username: 'gfm') }
|
2012-09-02 08:13:13 +02:00
|
|
|
let(:commit) { CommitDecorator.decorate(project.commit) }
|
|
|
|
let(:issue) { create(:issue, project: project) }
|
|
|
|
let(:merge_request) { create(:merge_request, project: project) }
|
|
|
|
let(:snippet) { create(:snippet, project: project) }
|
|
|
|
let(:member) { project.users_projects.where(user_id: user).first }
|
|
|
|
|
2012-08-02 02:26:21 +02:00
|
|
|
before do
|
2012-09-02 08:13:13 +02:00
|
|
|
# Helper expects a @project instance variable
|
|
|
|
@project = project
|
2012-08-02 02:26:21 +02:00
|
|
|
end
|
|
|
|
|
|
|
|
describe "#gfm" do
|
2012-09-02 08:13:13 +02:00
|
|
|
it "should return unaltered text if project is nil" do
|
|
|
|
actual = "Testing references: ##{issue.id}"
|
|
|
|
|
|
|
|
gfm(actual).should_not == actual
|
|
|
|
|
2012-08-06 02:32:17 +02:00
|
|
|
@project = nil
|
2012-09-02 08:13:13 +02:00
|
|
|
gfm(actual).should == actual
|
|
|
|
end
|
2012-08-06 02:32:17 +02:00
|
|
|
|
2012-09-02 08:13:13 +02:00
|
|
|
it "should not alter non-references" do
|
|
|
|
actual = expected = "_Please_ *stop* 'helping' and all the other b*$#%' you do."
|
|
|
|
gfm(actual).should == expected
|
|
|
|
end
|
|
|
|
|
|
|
|
it "should not touch HTML entities" do
|
2012-09-13 21:20:00 +02:00
|
|
|
@project.issues.stub(:where).with(id: '39').and_return([issue])
|
2012-09-02 08:13:13 +02:00
|
|
|
actual = expected = "We'll accept good pull requests."
|
|
|
|
gfm(actual).should == expected
|
|
|
|
end
|
|
|
|
|
|
|
|
it "should forward HTML options to links" do
|
|
|
|
gfm("Fixed in #{commit.id}", class: "foo").should have_selector("a.gfm.foo")
|
2012-08-06 02:32:17 +02:00
|
|
|
end
|
|
|
|
|
2012-08-02 02:26:21 +02:00
|
|
|
describe "referencing a commit" do
|
2012-09-02 08:13:13 +02:00
|
|
|
let(:expected) { project_commit_path(project, commit) }
|
|
|
|
|
2012-08-02 02:26:21 +02:00
|
|
|
it "should link using a full id" do
|
2012-09-02 08:13:13 +02:00
|
|
|
actual = "Reverts #{commit.id}"
|
|
|
|
gfm(actual).should match(expected)
|
2012-08-02 02:26:21 +02:00
|
|
|
end
|
|
|
|
|
|
|
|
it "should link using a short id" do
|
2012-09-02 08:13:13 +02:00
|
|
|
actual = "Backported from #{commit.short_id(6)}"
|
|
|
|
gfm(actual).should match(expected)
|
|
|
|
end
|
|
|
|
|
|
|
|
it "should link with adjacent text" do
|
|
|
|
actual = "Reverted (see #{commit.id})"
|
|
|
|
gfm(actual).should match(expected)
|
2012-08-02 02:26:21 +02:00
|
|
|
end
|
|
|
|
|
2012-09-02 08:13:13 +02:00
|
|
|
it "should keep whitespace intact" do
|
|
|
|
actual = "Changes #{commit.id} dramatically"
|
|
|
|
expected = /Changes <a.+>#{commit.id}<\/a> dramatically/
|
|
|
|
gfm(actual).should match(expected)
|
2012-08-02 02:26:21 +02:00
|
|
|
end
|
|
|
|
|
|
|
|
it "should not link with an invalid id" do
|
2012-09-02 08:13:13 +02:00
|
|
|
actual = expected = "What happened in #{commit.id.reverse}"
|
|
|
|
gfm(actual).should == expected
|
|
|
|
end
|
|
|
|
|
|
|
|
it "should include a title attribute" do
|
|
|
|
actual = "Reverts #{commit.id}"
|
|
|
|
gfm(actual).should match(/title="#{commit.link_title}"/)
|
|
|
|
end
|
|
|
|
|
|
|
|
it "should include standard gfm classes" do
|
|
|
|
actual = "Reverts #{commit.id}"
|
|
|
|
gfm(actual).should match(/class="\s?gfm gfm-commit\s?"/)
|
2012-08-02 02:26:21 +02:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe "referencing a team member" do
|
2012-12-07 04:40:42 +01:00
|
|
|
let(:actual) { "@#{user.username} you are right." }
|
2012-09-02 08:13:13 +02:00
|
|
|
let(:expected) { project_team_member_path(project, member) }
|
2012-08-02 02:26:21 +02:00
|
|
|
|
2012-09-02 08:13:13 +02:00
|
|
|
before do
|
|
|
|
project.users << user
|
2012-08-02 02:26:21 +02:00
|
|
|
end
|
|
|
|
|
2012-09-02 08:13:13 +02:00
|
|
|
it "should link using a simple name" do
|
|
|
|
gfm(actual).should match(expected)
|
|
|
|
end
|
2012-08-02 02:26:21 +02:00
|
|
|
|
2012-09-02 08:13:13 +02:00
|
|
|
it "should link using a name with dots" do
|
|
|
|
user.update_attributes(name: "alphA.Beta")
|
|
|
|
gfm(actual).should match(expected)
|
2012-08-02 02:26:21 +02:00
|
|
|
end
|
|
|
|
|
|
|
|
it "should link using name with underscores" do
|
2012-09-02 08:13:13 +02:00
|
|
|
user.update_attributes(name: "ping_pong_king")
|
|
|
|
gfm(actual).should match(expected)
|
2012-08-02 02:26:21 +02:00
|
|
|
end
|
|
|
|
|
2012-09-02 08:13:13 +02:00
|
|
|
it "should link with adjacent text" do
|
2012-12-07 04:40:42 +01:00
|
|
|
actual = "Mail the admin (@#{user.username})"
|
2012-09-02 08:13:13 +02:00
|
|
|
gfm(actual).should match(expected)
|
|
|
|
end
|
2012-08-02 02:26:21 +02:00
|
|
|
|
2012-09-02 08:13:13 +02:00
|
|
|
it "should keep whitespace intact" do
|
2012-12-07 04:40:42 +01:00
|
|
|
actual = "Yes, @#{user.username} is right."
|
|
|
|
expected = /Yes, <a.+>@#{user.username}<\/a> is right/
|
2012-09-02 08:13:13 +02:00
|
|
|
gfm(actual).should match(expected)
|
2012-08-02 02:26:21 +02:00
|
|
|
end
|
|
|
|
|
2012-09-02 08:13:13 +02:00
|
|
|
it "should not link with an invalid id" do
|
2012-12-07 04:40:42 +01:00
|
|
|
actual = expected = "@#{user.username.reverse} you are right."
|
2012-09-02 08:13:13 +02:00
|
|
|
gfm(actual).should == expected
|
2012-08-02 02:26:21 +02:00
|
|
|
end
|
|
|
|
|
2012-09-02 08:13:13 +02:00
|
|
|
it "should include standard gfm classes" do
|
|
|
|
gfm(actual).should match(/class="\s?gfm gfm-team_member\s?"/)
|
2012-08-02 02:26:21 +02:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2012-09-02 08:13:13 +02:00
|
|
|
# Shared examples for referencing an object
|
|
|
|
#
|
|
|
|
# Expects the following attributes to be available in the example group:
|
|
|
|
#
|
|
|
|
# - object - The object itself
|
|
|
|
# - reference - The object reference string (e.g., #1234, $1234, !1234)
|
|
|
|
#
|
|
|
|
# Currently limited to Snippets, Issues and MergeRequests
|
|
|
|
shared_examples 'referenced object' do
|
|
|
|
let(:actual) { "Reference to #{reference}" }
|
|
|
|
let(:expected) { polymorphic_path([project, object]) }
|
|
|
|
|
|
|
|
it "should link using a valid id" do
|
|
|
|
gfm(actual).should match(expected)
|
2012-08-02 02:26:21 +02:00
|
|
|
end
|
|
|
|
|
2012-09-02 08:13:13 +02:00
|
|
|
it "should link with adjacent text" do
|
|
|
|
# Wrap the reference in parenthesis
|
|
|
|
gfm(actual.gsub(reference, "(#{reference})")).should match(expected)
|
2012-08-02 02:26:21 +02:00
|
|
|
|
2012-09-02 08:13:13 +02:00
|
|
|
# Append some text to the end of the reference
|
|
|
|
gfm(actual.gsub(reference, "#{reference}, right?")).should match(expected)
|
2012-08-02 02:26:21 +02:00
|
|
|
end
|
|
|
|
|
2012-09-02 08:13:13 +02:00
|
|
|
it "should keep whitespace intact" do
|
|
|
|
actual = "Referenced #{reference} already."
|
|
|
|
expected = /Referenced <a.+>[^\s]+<\/a> already/
|
|
|
|
gfm(actual).should match(expected)
|
2012-08-02 02:26:21 +02:00
|
|
|
end
|
|
|
|
|
2012-09-02 08:13:13 +02:00
|
|
|
it "should not link with an invalid id" do
|
|
|
|
# Modify the reference string so it's still parsed, but is invalid
|
|
|
|
reference.gsub!(/^(.)(\d+)$/, '\1' + ('\2' * 2))
|
|
|
|
gfm(actual).should == actual
|
2012-08-02 02:26:21 +02:00
|
|
|
end
|
|
|
|
|
2012-09-02 08:13:13 +02:00
|
|
|
it "should include a title attribute" do
|
|
|
|
title = "#{object.class.to_s.titlecase}: #{object.title}"
|
|
|
|
gfm(actual).should match(/title="#{title}"/)
|
2012-08-02 02:26:21 +02:00
|
|
|
end
|
|
|
|
|
2012-09-02 08:13:13 +02:00
|
|
|
it "should include standard gfm classes" do
|
|
|
|
css = object.class.to_s.underscore
|
|
|
|
gfm(actual).should match(/class="\s?gfm gfm-#{css}\s?"/)
|
2012-08-02 02:26:21 +02:00
|
|
|
end
|
2012-09-02 08:13:13 +02:00
|
|
|
end
|
2012-08-02 02:26:21 +02:00
|
|
|
|
2012-09-02 08:13:13 +02:00
|
|
|
describe "referencing an issue" do
|
|
|
|
let(:object) { issue }
|
|
|
|
let(:reference) { "##{issue.id}" }
|
2012-08-02 02:26:21 +02:00
|
|
|
|
2012-09-02 08:13:13 +02:00
|
|
|
include_examples 'referenced object'
|
|
|
|
end
|
2012-08-02 02:26:21 +02:00
|
|
|
|
2012-09-02 08:13:13 +02:00
|
|
|
describe "referencing a merge request" do
|
|
|
|
let(:object) { merge_request }
|
|
|
|
let(:reference) { "!#{merge_request.id}" }
|
|
|
|
|
|
|
|
include_examples 'referenced object'
|
2012-08-02 02:26:21 +02:00
|
|
|
end
|
|
|
|
|
|
|
|
describe "referencing a snippet" do
|
2012-09-02 08:13:13 +02:00
|
|
|
let(:object) { snippet }
|
|
|
|
let(:reference) { "$#{snippet.id}" }
|
2012-08-02 02:26:21 +02:00
|
|
|
|
2012-09-02 08:13:13 +02:00
|
|
|
include_examples 'referenced object'
|
|
|
|
end
|
2012-08-02 02:26:21 +02:00
|
|
|
|
2012-09-02 08:13:13 +02:00
|
|
|
describe "referencing multiple objects" do
|
|
|
|
let(:actual) { "!#{merge_request.id} -> #{commit.id} -> ##{issue.id}" }
|
|
|
|
|
|
|
|
it "should link to the merge request" do
|
|
|
|
expected = project_merge_request_path(project, merge_request)
|
|
|
|
gfm(actual).should match(expected)
|
2012-08-02 02:26:21 +02:00
|
|
|
end
|
|
|
|
|
2012-09-02 08:13:13 +02:00
|
|
|
it "should link to the commit" do
|
|
|
|
expected = project_commit_path(project, commit)
|
|
|
|
gfm(actual).should match(expected)
|
2012-08-02 02:26:21 +02:00
|
|
|
end
|
|
|
|
|
2012-09-02 08:13:13 +02:00
|
|
|
it "should link to the issue" do
|
|
|
|
expected = project_issue_path(project, issue)
|
|
|
|
gfm(actual).should match(expected)
|
2012-08-02 02:26:21 +02:00
|
|
|
end
|
|
|
|
end
|
2012-09-05 22:05:20 +02:00
|
|
|
|
|
|
|
describe "emoji" do
|
|
|
|
it "matches at the start of a string" do
|
|
|
|
gfm(":+1:").should match(/<img/)
|
|
|
|
end
|
|
|
|
|
|
|
|
it "matches at the end of a string" do
|
|
|
|
gfm("This gets a :-1:").should match(/<img/)
|
|
|
|
end
|
|
|
|
|
|
|
|
it "matches with adjacent text" do
|
|
|
|
gfm("+1 (:+1:)").should match(/<img/)
|
|
|
|
end
|
|
|
|
|
|
|
|
it "has a title attribute" do
|
|
|
|
gfm(":-1:").should match(/title=":-1:"/)
|
|
|
|
end
|
|
|
|
|
|
|
|
it "has an alt attribute" do
|
|
|
|
gfm(":-1:").should match(/alt=":-1:"/)
|
|
|
|
end
|
|
|
|
|
|
|
|
it "has an emoji class" do
|
|
|
|
gfm(":+1:").should match('class="emoji"')
|
|
|
|
end
|
|
|
|
|
2012-09-05 22:14:54 +02:00
|
|
|
it "sets height and width" do
|
|
|
|
actual = gfm(":+1:")
|
|
|
|
actual.should match(/width="20"/)
|
|
|
|
actual.should match(/height="20"/)
|
|
|
|
end
|
|
|
|
|
2012-09-05 22:05:20 +02:00
|
|
|
it "keeps whitespace intact" do
|
|
|
|
gfm("This deserves a :+1: big time.").should match(/deserves a <img.+\/> big time/)
|
|
|
|
end
|
|
|
|
|
|
|
|
it "ignores invalid emoji" do
|
|
|
|
gfm(":invalid-emoji:").should_not match(/<img/)
|
|
|
|
end
|
2012-09-08 00:49:17 +02:00
|
|
|
|
|
|
|
it "should work independet of reference links (i.e. without @project being set)" do
|
|
|
|
@project = nil
|
|
|
|
gfm(":+1:").should match(/<img/)
|
|
|
|
end
|
2012-09-05 22:05:20 +02:00
|
|
|
end
|
2012-09-02 08:13:13 +02:00
|
|
|
end
|
2012-08-02 02:26:21 +02:00
|
|
|
|
2012-09-02 08:13:13 +02:00
|
|
|
describe "#link_to_gfm" do
|
|
|
|
let(:commit_path) { project_commit_path(project, commit) }
|
|
|
|
let(:issues) { create_list(:issue, 2, project: project) }
|
2012-08-02 02:26:21 +02:00
|
|
|
|
2012-09-02 08:13:13 +02:00
|
|
|
it "should handle references nested in links with all the text" do
|
|
|
|
actual = link_to_gfm("This should finally fix ##{issues[0].id} and ##{issues[1].id} for real", commit_path)
|
2012-08-02 02:26:21 +02:00
|
|
|
|
2012-09-02 08:13:13 +02:00
|
|
|
# Break the result into groups of links with their content, without
|
|
|
|
# closing tags
|
|
|
|
groups = actual.split("</a>")
|
2012-08-02 02:26:21 +02:00
|
|
|
|
2012-09-02 08:13:13 +02:00
|
|
|
# Leading commit link
|
|
|
|
groups[0].should match(/href="#{commit_path}"/)
|
|
|
|
groups[0].should match(/This should finally fix $/)
|
2012-08-02 02:26:21 +02:00
|
|
|
|
2012-09-02 08:13:13 +02:00
|
|
|
# First issue link
|
|
|
|
groups[1].should match(/href="#{project_issue_path(project, issues[0])}"/)
|
|
|
|
groups[1].should match(/##{issues[0].id}$/)
|
2012-08-02 02:28:50 +02:00
|
|
|
|
2012-09-02 08:13:13 +02:00
|
|
|
# Internal commit link
|
|
|
|
groups[2].should match(/href="#{commit_path}"/)
|
|
|
|
groups[2].should match(/ and /)
|
2012-08-02 02:28:50 +02:00
|
|
|
|
2012-09-02 08:13:13 +02:00
|
|
|
# Second issue link
|
|
|
|
groups[3].should match(/href="#{project_issue_path(project, issues[1])}"/)
|
|
|
|
groups[3].should match(/##{issues[1].id}$/)
|
|
|
|
|
|
|
|
# Trailing commit link
|
|
|
|
groups[4].should match(/href="#{commit_path}"/)
|
|
|
|
groups[4].should match(/ for real$/)
|
2012-08-02 02:28:50 +02:00
|
|
|
end
|
|
|
|
|
|
|
|
it "should forward HTML options" do
|
2012-09-02 08:13:13 +02:00
|
|
|
actual = link_to_gfm("Fixed in #{commit.id}", commit_path, class: 'foo')
|
|
|
|
actual.should have_selector 'a.gfm.gfm-commit.foo'
|
2012-08-02 02:28:50 +02:00
|
|
|
end
|
2012-09-20 01:42:26 +02:00
|
|
|
|
|
|
|
it "escapes HTML passed in as the body" do
|
|
|
|
actual = "This is a <h1>test</h1> - see ##{issues[0].id}"
|
|
|
|
link_to_gfm(actual, commit_path).should match('<h1>test</h1>')
|
|
|
|
end
|
2012-08-02 02:28:50 +02:00
|
|
|
end
|
2012-08-02 02:29:15 +02:00
|
|
|
|
|
|
|
describe "#markdown" do
|
|
|
|
it "should handle references in paragraphs" do
|
2012-09-20 01:42:26 +02:00
|
|
|
actual = "\n\nLorem ipsum dolor sit amet. #{commit.id} Nam pulvinar sapien eget.\n"
|
|
|
|
expected = project_commit_path(project, commit)
|
|
|
|
markdown(actual).should match(expected)
|
2012-08-02 02:29:15 +02:00
|
|
|
end
|
|
|
|
|
|
|
|
it "should handle references in headers" do
|
2012-09-02 08:13:13 +02:00
|
|
|
actual = "\n# Working around ##{issue.id}\n## Apply !#{merge_request.id}"
|
|
|
|
|
|
|
|
markdown(actual).should match(%r{<h1[^<]*>Working around <a.+>##{issue.id}</a></h1>})
|
|
|
|
markdown(actual).should match(%r{<h2[^<]*>Apply <a.+>!#{merge_request.id}</a></h2>})
|
2012-08-02 02:29:15 +02:00
|
|
|
end
|
|
|
|
|
|
|
|
it "should handle references in lists" do
|
2012-09-02 08:13:13 +02:00
|
|
|
project.users << user
|
|
|
|
|
2012-12-07 04:40:42 +01:00
|
|
|
actual = "\n* dark: ##{issue.id}\n* light by @#{member.user.username}"
|
2012-09-02 08:13:13 +02:00
|
|
|
|
|
|
|
markdown(actual).should match(%r{<li>dark: <a.+>##{issue.id}</a></li>})
|
2012-12-07 04:40:42 +01:00
|
|
|
markdown(actual).should match(%r{<li>light by <a.+>@#{member.user.username}</a></li>})
|
2012-08-02 02:29:15 +02:00
|
|
|
end
|
|
|
|
|
|
|
|
it "should handle references in <em>" do
|
2012-09-02 08:13:13 +02:00
|
|
|
actual = "Apply _!#{merge_request.id}_ ASAP"
|
|
|
|
|
|
|
|
markdown(actual).should match(%r{Apply <em><a.+>!#{merge_request.id}</a></em>})
|
2012-08-02 02:29:15 +02:00
|
|
|
end
|
|
|
|
|
|
|
|
it "should leave code blocks untouched" do
|
2012-11-28 16:05:11 +01:00
|
|
|
helper.stub(:user_color_scheme_class).and_return(:white)
|
2012-08-02 02:29:15 +02:00
|
|
|
|
2012-11-28 16:05:11 +01:00
|
|
|
helper.markdown("\n some code from $#{snippet.id}\n here too\n").should == "<div class=\"white\"><div class=\"highlight\"><pre><span class=\"n\">some</span> <span class=\"n\">code</span> <span class=\"n\">from</span> $#{snippet.id}\n<span class=\"n\">here</span> <span class=\"n\">too</span>\n</pre></div></div>"
|
|
|
|
|
|
|
|
helper.markdown("\n```\nsome code from $#{snippet.id}\nhere too\n```\n").should == "<div class=\"white\"><div class=\"highlight\"><pre><span class=\"n\">some</span> <span class=\"n\">code</span> <span class=\"n\">from</span> $#{snippet.id}\n<span class=\"n\">here</span> <span class=\"n\">too</span>\n</pre></div></div>"
|
2012-08-02 02:29:15 +02:00
|
|
|
end
|
|
|
|
|
|
|
|
it "should leave inline code untouched" do
|
2012-09-02 08:13:13 +02:00
|
|
|
markdown("\nDon't use `$#{snippet.id}` here.\n").should == "<p>Don't use <code>$#{snippet.id}</code> here.</p>\n"
|
2012-08-02 02:29:15 +02:00
|
|
|
end
|
|
|
|
end
|
2012-08-02 02:26:21 +02:00
|
|
|
end
|