summaryrefslogtreecommitdiff
path: root/spec
diff options
context:
space:
mode:
authorRobert Speicher <rspeicher@gmail.com>2015-05-06 17:21:19 -0400
committerRobert Speicher <rspeicher@gmail.com>2015-05-06 17:21:19 -0400
commitfdff815a4cddd9e956d1bf4cc8f01eeb3384b9ee (patch)
tree7c48d29b70d2a522641c332b4574348dffbf3f76 /spec
parent5b3c823fc9fe6788184a29dec7393158738ef584 (diff)
downloadgitlab-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.rb13
-rw-r--r--spec/fixtures/markdown.md.erb8
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 &lt; 3 &amp; 5 | 1 < 3 & 5 |
| `<foo>` | &lt;foo&gt; | <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: