fix a slight issue with link text containing quotes, and bring coverage back to 100%.

Author: ikirudennis

tests/test_glyphs.py

@@ -26,7 +26,3 @@ def test_glyphs():
     result = t.glyphs("<p><cite>Cat's Cradle</cite> by Vonnegut</p>")
     expect = '<p><cite>Cat&#8217;s Cradle</cite> by Vonnegut</p>'
     assert result == expect
-
-    result = t.glyphs('test"')
-    expect = 'test&#8221; '
-    assert result == expect

tests/test_urls.py

@@ -59,3 +59,11 @@ def test_rel_attribute():
     result = t.parse('"$":http://domain.tld')
     expect = '\t<p><a href="http://domain.tld" rel="nofollow">domain.tld</a></p>'
     assert result == expect
+
+def test_quotes_in_link_text():
+    """quotes in link text are tricky."""
+    test = '""this is a quote in link text"":url'
+    t = Textile()
+    result = t.parse(test)
+    expect = '\t<p><a href="url">&#8220;this is a quote in link text&#8221;</a></p>'
+    assert result == expect

textile/core.py

@@ -577,10 +577,6 @@ def glyphs(self, text):
         So, for the first pass, we use the glyph_search_initial set of
         regexes.  For all remaining passes, we use glyph_search
         """
-        # fix: hackish
-        if text.endswith('"'):
-            text = '{0} '.format(text)
-
         text = text.rstrip('\n')
         result = []
         searchlist = self.glyph_search_initial
@@ -729,7 +725,7 @@ def markStartOfLinks(self, text):
 
                         try:
                             possibility = possible_start_quotes.pop()
-                        except IndexError:
+                        except IndexError: # pragma: no branch
                             # If out of possible starting segments we back the
                             # last one from the linkparts array
                             linkparts.pop()