Don't HTML-escape JSON messages (#3467)

The JSON module already escapes special characters as needed.

This was already reported in #2769 and partially addressed by
6b1adc6, but that commit still quotes angle
brackets ('<' and '>').

Author: cpitclaudel

ChangeLog

@@ -7,6 +7,9 @@ What's New in Pylint 2.5.0?
 
 Release date: TBA
 
+* Remove HTML quoting of messages in JSON output.
+
+  Close #2769
 
 * Adjust the `invalid-name` rule to work with non-ASCII identifiers and add the `non-ascii-name` rule.
 

pylint/reporters/json_reporter.py

@@ -7,7 +7,6 @@
 # For details: https://github.com/PyCQA/pylint/blob/master/COPYING
 
 """JSON reporter"""
-import html
 import json
 import sys
 
@@ -37,7 +36,7 @@ def handle_message(self, msg):
                 "column": msg.column,
                 "path": msg.path,
                 "symbol": msg.symbol,
-                "message": html.escape(msg.msg or "", quote=False),
+                "message": msg.msg or "",
                 "message-id": msg.msg_id,
             }
         )

tests/test_self.py

@@ -347,6 +347,7 @@ def test_json_report_when_file_has_syntax_error(self):
             assert key in message
             assert message[key] == value
         assert "invalid syntax" in message["message"].lower()
+        assert "<unknown>" in message["message"].lower()
 
     def test_json_report_when_file_is_missing(self):
         out = StringIO()