diff options
author | Robert Speicher <rspeicher@gmail.com> | 2015-05-06 17:21:19 -0400 |
---|---|---|
committer | Robert Speicher <rspeicher@gmail.com> | 2015-05-06 17:21:19 -0400 |
commit | fdff815a4cddd9e956d1bf4cc8f01eeb3384b9ee (patch) | |
tree | 7c48d29b70d2a522641c332b4574348dffbf3f76 /spec | |
parent | 5b3c823fc9fe6788184a29dec7393158738ef584 (diff) | |
download | gitlab-ce-fdff815a4cddd9e956d1bf4cc8f01eeb3384b9ee.tar.gz |
Add Markdown feature specs for markup inside linksrs-issue-1348
It was already working, just want to make sure we don't break it in the
future.
Diffstat (limited to 'spec')
-rw-r--r-- | spec/features/markdown_spec.rb | 13 | ||||
-rw-r--r-- | spec/fixtures/markdown.md.erb | 8 |
2 files changed, 21 insertions, 0 deletions
diff --git a/spec/features/markdown_spec.rb b/spec/features/markdown_spec.rb index 3528200e12b..3fe593c47b4 100644 --- a/spec/features/markdown_spec.rb +++ b/spec/features/markdown_spec.rb @@ -161,6 +161,19 @@ describe 'GitLab Markdown' do end end + describe 'Edge Cases' do + it 'allows markup inside link elements' do + expect(@doc.at_css('a[href="#link-emphasis"]').to_html). + to eq %{<a href="#link-emphasis"><em>text</em></a>} + + expect(@doc.at_css('a[href="#link-strong"]').to_html). + to eq %{<a href="#link-strong"><strong>text</strong></a>} + + expect(@doc.at_css('a[href="#link-code"]').to_html). + to eq %{<a href="#link-code"><code>text</code></a>} + end + end + describe 'EmojiFilter' do it 'parses Emoji' do expect(@doc).to have_selector('img.emoji', count: 10) diff --git a/spec/fixtures/markdown.md.erb b/spec/fixtures/markdown.md.erb index 0c140758557..6b79aad8f86 100644 --- a/spec/fixtures/markdown.md.erb +++ b/spec/fixtures/markdown.md.erb @@ -94,6 +94,14 @@ The problem with SanitizationFilter is that it can be too aggressive. | `1 < 3 & 5` | 1 < 3 & 5 | 1 < 3 & 5 | | `<foo>` | <foo> | <foo> | +### Edge Cases + +Markdown should be usable inside a link. Let's try! + +- [_text_](#link-emphasis) +- [**text**](#link-strong) +- [`text`](#link-code) + ### EmojiFilter Because life would be :zzz: without Emoji, right? :rocket: |