Don't merge font tags with different values

Closes #224

NEWS.md

@@ -1,6 +1,7 @@
 PENDING: Gaupol 1.15
 ====================
 
+* Don't merge font tags with different values (#224)
 * Drop dependency on chardet
 * Add dependency on charset-normalizer
 * Raise Python dependency to >= 3.5

aeidon/markups/subrip.py

@@ -50,7 +50,9 @@ def clean(self, text):
         # Remove tags that are immediately closed after opening.
         text = self._substitute(text, r"<([a-z]+)[^<]*?>( *)</\1>", r"\2")
         # Remove tags that are immediately opened after closing.
-        text = self._substitute(text, r"</([a-z]+)>( *)<\1[^<]*?>", r"\2")
+        # Don't apply to tags that have values, such as 'font',
+        # because then we'd need to check the value is the same too.
+        text = self._substitute(text, r"</([a-z]+)>( *)<\1>", r"\2")
         # Remove or relocate space right after an opening tag.
         text = self._substitute(text, r" ?(<(?!/)[^>]+?>) ", r" \1")
         # Remove or relocate space right before a closing tag.

aeidon/markups/test/test_subrip.py

@@ -38,6 +38,14 @@ def test_clean__closing(self):
             "All <i>things</i> weird are normal\n"
             "in <i>this</i> whore of cities.")
 
+    def test_clean__font(self):
+        # Don't merge font tags with different values.
+        # https://github.com/otsaloma/gaupol/issues/224
+        text = ('<font color="#00ff00">Yes, exactly.</font>'
+                '<font color="#00ffff">Oh, wow!</font>'
+                '<font color="#ffff00">Nice.</font>')
+        assert self.markup.clean(text) == text
+
     def test_clean__opening(self):
         text = ("All <i>things </i>weird are normal\n"
                 "in <i>this </i> whore of cities.")