Move replace that's only for tests to test

It does not make sense to put test-specific code in the code that runs in production. Escaping ' as ' is perfectly fine and CommonMark compliant. (The official CommonMark test suite actually performs HTML normalization by default).
miyuchina · Feb 14, 2022 · ef9bd3a · ef9bd3a
1 parent 22ec204
commit ef9bd3a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/mistletoe/html_renderer.py b/mistletoe/html_renderer.py
@@ -205,7 +205,7 @@ def render_document(self, token: block_token.Document) -> str:
 
     @staticmethod
     def escape_html(raw: str) -> str:
-        return html.escape(raw).replace('&#x27;', "'")
+        return html.escape(raw)
 
     @staticmethod
     def escape_url(raw: str) -> str:

diff --git a/test/specification/commonmark.py b/test/specification/commonmark.py
@@ -29,7 +29,7 @@ def run_tests(test_entries, start=None, end=None,
 def run_test(test_entry, quiet=False):
     test_case = test_entry['markdown'].splitlines(keepends=True)
     try:
-        output = markdown(test_case)
+        output = markdown(test_case).replace('&#x27;', "'")
         success = test_entry['html'] == output
         if not success and not quiet:
             print_test_entry(test_entry, output)