From d18c3d0acab0e7469c3284c897afcb61f9dd1fea Mon Sep 17 00:00:00 2001
From: Isaac Muse <faceless.shop@gmail.com>
Date: Wed, 17 Jan 2018 18:36:34 -0700
Subject: Flexible inline (#629)

Add new InlineProcessor class that handles inline processing much better and allows for more flexibility. This adds new InlineProcessors that no longer utilize unnecessary pretext and posttext captures. New class can accept the buffer that is being worked on and manually process the text without regex and return new replacement bounds. This helps us to handle links in a better way and handle nested brackets and logic that is too much for regular expression. The refactor also allows image links to have links/paths with spaces like links. Ref #551, #613, #590, #161.
---
 tests/test_syntax/inline/__init__.py |   0
 tests/test_syntax/inline/images.py   | 139 +++++++++++++++++++++++++++++++++++
 tests/test_syntax/inline/links.py    |  98 ++++++++++++++++++++++++
 3 files changed, 237 insertions(+)
 create mode 100644 tests/test_syntax/inline/__init__.py
 create mode 100644 tests/test_syntax/inline/images.py
 create mode 100644 tests/test_syntax/inline/links.py

(limited to 'tests/test_syntax')

diff --git a/tests/test_syntax/inline/__init__.py b/tests/test_syntax/inline/__init__.py
new file mode 100644
index 0000000..e69de29
diff --git a/tests/test_syntax/inline/images.py b/tests/test_syntax/inline/images.py
new file mode 100644
index 0000000..9c1dc34
--- /dev/null
+++ b/tests/test_syntax/inline/images.py
@@ -0,0 +1,139 @@
+from markdown.test_tools import TestCase
+
+
+class TestAdvancedImages(TestCase):
+
+    def test_nested_square_brackets(self):
+        self.assertMarkdownRenders(
+            """![Text[[[[[[[]]]]]]][]](http://link.com/image.png) more text""",
+            """<p><img alt="Text[[[[[[[]]]]]]][]" src="http://link.com/image.png" /> more text</p>"""
+        )
+
+    def test_nested_round_brackets(self):
+        self.assertMarkdownRenders(
+            """![Text](http://link.com/(((((((()))))))()).png) more text""",
+            """<p><img alt="Text" src="http://link.com/(((((((()))))))()).png" /> more text</p>"""
+        )
+
+    def test_uneven_brackets_with_titles1(self):
+        self.assertMarkdownRenders(
+            """![Text](http://link.com/(.png"title") more text""",
+            """<p><img alt="Text" src="http://link.com/(.png" title="title" /> more text</p>"""
+        )
+
+    def test_uneven_brackets_with_titles2(self):
+        self.assertMarkdownRenders(
+            """![Text](http://link.com/('.png"title") more text""",
+            """<p><img alt="Text" src="http://link.com/('.png" title="title" /> more text</p>"""
+        )
+
+    def test_uneven_brackets_with_titles3(self):
+        self.assertMarkdownRenders(
+            """![Text](http://link.com/(.png"title)") more text""",
+            """<p><img alt="Text" src="http://link.com/(.png" title="title)" /> more text</p>"""
+        )
+
+    def test_uneven_brackets_with_titles4(self):
+        self.assertMarkdownRenders(
+            """![Text](http://link.com/(.png "title") more text""",
+            """<p><img alt="Text" src="http://link.com/(.png" title="title" /> more text</p>"""
+        )
+
+    def test_uneven_brackets_with_titles5(self):
+        self.assertMarkdownRenders(
+            """![Text](http://link.com/(.png "title)") more text""",
+            """<p><img alt="Text" src="http://link.com/(.png" title="title)" /> more text</p>"""
+        )
+
+    def test_mixed_title_quotes1(self):
+        self.assertMarkdownRenders(
+            """![Text](http://link.com/'.png"title") more text""",
+            """<p><img alt="Text" src="http://link.com/'.png" title="title" /> more text</p>"""
+        )
+
+    def test_mixed_title_quotes2(self):
+        self.assertMarkdownRenders(
+            """![Text](http://link.com/".png'title') more text""",
+            """<p><img alt="Text" src="http://link.com/&quot;.png" title="title" /> more text</p>"""
+        )
+
+    def test_mixed_title_quotes3(self):
+        self.assertMarkdownRenders(
+            """![Text](http://link.com/with spaces.png'"and quotes" 'and title') more text""",
+            """<p><img alt="Text" src="http://link.com/with spaces.png" title="&quot;and quotes&quot; 'and title" />"""
+            """ more text</p>"""
+        )
+
+    def test_mixed_title_quotes4(self):
+        self.assertMarkdownRenders(
+            """![Text](http://link.com/with spaces'.png"and quotes" 'and title") more text""",
+            """<p><img alt="Text" src="http://link.com/with spaces'.png" title="and quotes&quot; 'and title" />"""
+            """ more text</p>"""
+        )
+
+    def test_mixed_title_quotes5(self):
+        self.assertMarkdownRenders(
+            """![Text](http://link.com/with spaces .png'"and quotes" 'and title') more text""",
+            """<p><img alt="Text" src="http://link.com/with spaces .png" title="&quot;and quotes&quot;"""
+            """ 'and title" /> more text</p>"""
+        )
+
+    def test_mixed_title_quotes6(self):
+        self.assertMarkdownRenders(
+            """![Text](http://link.com/with spaces "and quotes".png 'and title') more text""",
+            """<p><img alt="Text" src="http://link.com/with spaces &quot;and quotes&quot;.png" title="and title" />"""
+            """ more text</p>"""
+        )
+
+    def test_single_quote(self):
+        self.assertMarkdownRenders(
+            """![test](link"notitle.png)""",
+            """<p><img alt="test" src="link&quot;notitle.png" /></p>"""
+        )
+
+    def test_angle_with_mixed_title_quotes(self):
+        self.assertMarkdownRenders(
+            """![Text](<http://link.com/with spaces '"and quotes".png> 'and title') more text""",
+            """<p><img alt="Text" src="http://link.com/with spaces '&quot;and quotes&quot;.png" title="and title" />"""
+            """ more text</p>"""
+        )
+
+    def test_misc(self):
+        self.assertMarkdownRenders(
+            """![Poster](http://humane_man.jpg "The most humane man.")""",
+            """<p><img alt="Poster" src="http://humane_man.jpg" title="The most humane man." /></p>"""
+        )
+
+    def test_misc_ref(self):
+        self.assertMarkdownRenders(
+            self.dedent(
+                """
+                ![Poster][]
+
+                [Poster]:http://humane_man.jpg "The most humane man."
+                """
+            ),
+            self.dedent(
+                """
+                <p><img alt="Poster" src="http://humane_man.jpg" title="The most humane man." /></p>
+                """
+            )
+        )
+
+    def test_misc_blank(self):
+        self.assertMarkdownRenders(
+            """![Blank]()""",
+            """<p><img alt="Blank" src="" /></p>"""
+        )
+
+    def test_misc_img_title(self):
+        self.assertMarkdownRenders(
+            """![Image](http://humane man.jpg "The most humane man.")""",
+            """<p><img alt="Image" src="http://humane man.jpg" title="The most humane man." /></p>"""
+        )
+
+    def test_misc_img(self):
+        self.assertMarkdownRenders(
+            """![Image](http://humane man.jpg)""",
+            """<p><img alt="Image" src="http://humane man.jpg" /></p>"""
+        )
diff --git a/tests/test_syntax/inline/links.py b/tests/test_syntax/inline/links.py
new file mode 100644
index 0000000..fe58ada
--- /dev/null
+++ b/tests/test_syntax/inline/links.py
@@ -0,0 +1,98 @@
+from markdown.test_tools import TestCase
+
+
+class TestAdvancedLinks(TestCase):
+
+    def test_nested_square_brackets(self):
+        self.assertMarkdownRenders(
+            """[Text[[[[[[[]]]]]]][]](http://link.com) more text""",
+            """<p><a href="http://link.com">Text[[[[[[[]]]]]]][]</a> more text</p>"""
+        )
+
+    def test_nested_round_brackets(self):
+        self.assertMarkdownRenders(
+            """[Text](http://link.com/(((((((()))))))())) more text""",
+            """<p><a href="http://link.com/(((((((()))))))())">Text</a> more text</p>"""
+        )
+
+    def test_uneven_brackets_with_titles1(self):
+        self.assertMarkdownRenders(
+            """[Text](http://link.com/("title") more text""",
+            """<p><a href="http://link.com/(" title="title">Text</a> more text</p>"""
+        )
+
+    def test_uneven_brackets_with_titles2(self):
+        self.assertMarkdownRenders(
+            """[Text](http://link.com/('"title") more text""",
+            """<p><a href="http://link.com/('" title="title">Text</a> more text</p>"""
+        )
+
+    def test_uneven_brackets_with_titles3(self):
+        self.assertMarkdownRenders(
+            """[Text](http://link.com/("title)") more text""",
+            """<p><a href="http://link.com/(" title="title)">Text</a> more text</p>"""
+        )
+
+    def test_uneven_brackets_with_titles4(self):
+        self.assertMarkdownRenders(
+            """[Text](http://link.com/( "title") more text""",
+            """<p><a href="http://link.com/(" title="title">Text</a> more text</p>"""
+        )
+
+    def test_uneven_brackets_with_titles5(self):
+        self.assertMarkdownRenders(
+            """[Text](http://link.com/( "title)") more text""",
+            """<p><a href="http://link.com/(" title="title)">Text</a> more text</p>"""
+        )
+
+    def test_mixed_title_quotes1(self):
+        self.assertMarkdownRenders(
+            """[Text](http://link.com/'"title") more text""",
+            """<p><a href="http://link.com/'" title="title">Text</a> more text</p>"""
+        )
+
+    def test_mixed_title_quotes2(self):
+        self.assertMarkdownRenders(
+            """[Text](http://link.com/"'title') more text""",
+            """<p><a href="http://link.com/&quot;" title="title">Text</a> more text</p>"""
+        )
+
+    def test_mixed_title_quotes3(self):
+        self.assertMarkdownRenders(
+            """[Text](http://link.com/with spaces'"and quotes" 'and title') more text""",
+            """<p><a href="http://link.com/with spaces" title="&quot;and quotes&quot; 'and title">"""
+            """Text</a> more text</p>"""
+        )
+
+    def test_mixed_title_quotes4(self):
+        self.assertMarkdownRenders(
+            """[Text](http://link.com/with spaces'"and quotes" 'and title") more text""",
+            """<p><a href="http://link.com/with spaces'" title="and quotes&quot; 'and title">Text</a> more text</p>"""
+        )
+
+    def test_mixed_title_quotes5(self):
+        self.assertMarkdownRenders(
+            """[Text](http://link.com/with spaces '"and quotes" 'and title') more text""",
+            """<p><a href="http://link.com/with spaces" title="&quot;and quotes&quot; 'and title">"""
+            """Text</a> more text</p>"""
+        )
+
+    def test_mixed_title_quotes6(self):
+        self.assertMarkdownRenders(
+            """[Text](http://link.com/with spaces "and quotes" 'and title') more text""",
+            """<p><a href="http://link.com/with spaces &quot;and quotes&quot;" title="and title">"""
+            """Text</a> more text</p>"""
+        )
+
+    def test_single_quote(self):
+        self.assertMarkdownRenders(
+            """[test](link"notitle)""",
+            """<p><a href="link&quot;notitle">test</a></p>"""
+        )
+
+    def test_angle_with_mixed_title_quotes(self):
+        self.assertMarkdownRenders(
+            """[Text](<http://link.com/with spaces '"and quotes"> 'and title') more text""",
+            """<p><a href="http://link.com/with spaces '&quot;and quotes&quot;" title="and title">"""
+            """Text</a> more text</p>"""
+        )
-- 
cgit v1.2.3