Commit 567b8a99 authored by Mike Greiling's avatar Mike Greiling

update gitlab flavored markdown tests to reflect namespaced ids

parent 94ae12f6
...@@ -2,7 +2,7 @@ module SharedMarkdown ...@@ -2,7 +2,7 @@ module SharedMarkdown
include Spinach::DSL include Spinach::DSL
def header_should_have_correct_id_and_link(level, text, id, parent = ".wiki") def header_should_have_correct_id_and_link(level, text, id, parent = ".wiki")
node = find("#{parent} h#{level} a##{id}") node = find("#{parent} h#{level} a#user-content_#{id}")
expect(node[:href]).to eq "##{id}" expect(node[:href]).to eq "##{id}"
# Work around a weird Capybara behavior where calling `parent` on a node # Work around a weird Capybara behavior where calling `parent` on a node
......
...@@ -22,7 +22,7 @@ describe Banzai::Filter::TableOfContentsFilter, lib: true do ...@@ -22,7 +22,7 @@ describe Banzai::Filter::TableOfContentsFilter, lib: true do
html = header(i, "Header #{i}") html = header(i, "Header #{i}")
doc = filter(html) doc = filter(html)
expect(doc.css("h#{i} a").first.attr('id')).to eq "header-#{i}" expect(doc.css("h#{i} a").first.attr('id')).to eq "user-content_header-#{i}"
end end
end end
...@@ -32,7 +32,12 @@ describe Banzai::Filter::TableOfContentsFilter, lib: true do ...@@ -32,7 +32,12 @@ describe Banzai::Filter::TableOfContentsFilter, lib: true do
expect(doc.css('h1 a').first.attr('class')).to eq 'anchor' expect(doc.css('h1 a').first.attr('class')).to eq 'anchor'
end end
it 'links to the id' do it 'has a namespaced id' do
doc = filter(header(1, 'Header'))
expect(doc.css('h1 a').first.attr('id')).to eq 'user-content_header'
end
it 'links to the non-namespaced id' do
doc = filter(header(1, 'Header')) doc = filter(header(1, 'Header'))
expect(doc.css('h1 a').first.attr('href')).to eq '#header' expect(doc.css('h1 a').first.attr('href')).to eq '#header'
end end
...@@ -40,29 +45,29 @@ describe Banzai::Filter::TableOfContentsFilter, lib: true do ...@@ -40,29 +45,29 @@ describe Banzai::Filter::TableOfContentsFilter, lib: true do
describe 'generated IDs' do describe 'generated IDs' do
it 'translates spaces to dashes' do it 'translates spaces to dashes' do
doc = filter(header(1, 'This header has spaces in it')) doc = filter(header(1, 'This header has spaces in it'))
expect(doc.css('h1 a').first.attr('id')).to eq 'this-header-has-spaces-in-it' expect(doc.css('h1 a').first.attr('href')).to eq '#this-header-has-spaces-in-it'
end end
it 'squeezes multiple spaces and dashes' do it 'squeezes multiple spaces and dashes' do
doc = filter(header(1, 'This---header is poorly-formatted')) doc = filter(header(1, 'This---header is poorly-formatted'))
expect(doc.css('h1 a').first.attr('id')).to eq 'this-header-is-poorly-formatted' expect(doc.css('h1 a').first.attr('href')).to eq '#this-header-is-poorly-formatted'
end end
it 'removes punctuation' do it 'removes punctuation' do
doc = filter(header(1, "This, header! is, filled. with @ punctuation?")) doc = filter(header(1, "This, header! is, filled. with @ punctuation?"))
expect(doc.css('h1 a').first.attr('id')).to eq 'this-header-is-filled-with-punctuation' expect(doc.css('h1 a').first.attr('href')).to eq '#this-header-is-filled-with-punctuation'
end end
it 'appends a unique number to duplicates' do it 'appends a unique number to duplicates' do
doc = filter(header(1, 'One') + header(2, 'One')) doc = filter(header(1, 'One') + header(2, 'One'))
expect(doc.css('h1 a').first.attr('id')).to eq 'one' expect(doc.css('h1 a').first.attr('href')).to eq '#one'
expect(doc.css('h2 a').first.attr('id')).to eq 'one-1' expect(doc.css('h2 a').first.attr('href')).to eq '#one-1'
end end
it 'supports Unicode' do it 'supports Unicode' do
doc = filter(header(1, '한글')) doc = filter(header(1, '한글'))
expect(doc.css('h1 a').first.attr('id')).to eq '한글' expect(doc.css('h1 a').first.attr('id')).to eq 'user-content_한글'
expect(doc.css('h1 a').first.attr('href')).to eq '#한글' expect(doc.css('h1 a').first.attr('href')).to eq '#한글'
end end
end end
......
...@@ -38,9 +38,9 @@ module MarkdownMatchers ...@@ -38,9 +38,9 @@ module MarkdownMatchers
set_default_markdown_messages set_default_markdown_messages
match do |actual| match do |actual|
expect(actual).to have_selector('h1 a#gitlab-markdown') expect(actual).to have_selector('h1 a#user-content_gitlab-markdown')
expect(actual).to have_selector('h2 a#markdown') expect(actual).to have_selector('h2 a#user-content_markdown')
expect(actual).to have_selector('h3 a#autolinkfilter') expect(actual).to have_selector('h3 a#user-content_autolinkfilter')
end end
end end
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment