Skip to content

Commit

Permalink
Fix false negative for use-symbolic-message
Browse files Browse the repository at this point in the history
Closes #5000
  • Loading branch information
Pierre-Sassoulas committed Sep 14, 2021
1 parent cb7bba4 commit bbaa7bc
Show file tree
Hide file tree
Showing 4 changed files with 20 additions and 15 deletions.
4 changes: 4 additions & 0 deletions ChangeLog
Expand Up @@ -93,6 +93,10 @@ Release date: TBA

Closes #4616

* Non symbolic messages with the wrong capitalisation now correctly trigger ``use-symbolic-message-instead``

Closes #5000


What's New in Pylint 2.10.3?
============================
Expand Down
2 changes: 1 addition & 1 deletion pylint/message/message_id_store.py
Expand Up @@ -26,7 +26,7 @@ def __repr__(self):

def get_symbol(self, msgid: str) -> str:
try:
return self.__msgid_to_symbol[msgid]
return self.__msgid_to_symbol[msgid.upper()]
except KeyError as e:
msg = f"'{msgid}' is not stored in the message store."
raise UnknownMessageError(msg) from e
Expand Down
2 changes: 1 addition & 1 deletion tests/functional/u/use/use_symbolic_message_instead.py
@@ -1,5 +1,5 @@
# pylint: disable=C0111,R0903,T1234 # [bad-option-value,use-symbolic-message-instead,use-symbolic-message-instead]
# pylint: enable=C0111 # [use-symbolic-message-instead]
# pylint: enable=c0111,w0223 # [use-symbolic-message-instead,use-symbolic-message-instead]

def my_function(arg): # [missing-function-docstring]
return arg or True
Expand Down
27 changes: 14 additions & 13 deletions tests/functional/u/use/use_symbolic_message_instead.txt
@@ -1,13 +1,14 @@
bad-option-value:1:0::Bad option value 'T1234'
use-symbolic-message-instead:1:0::"'C0111' is cryptic: use '# pylint: disable=missing-docstring' instead"
use-symbolic-message-instead:1:0::"'R0903' is cryptic: use '# pylint: disable=too-few-public-methods' instead"
use-symbolic-message-instead:2:0::"'C0111' is cryptic: use '# pylint: enable=missing-docstring' instead"
missing-function-docstring:4:0:my_function:Missing function or method docstring
use-symbolic-message-instead:7:0::"'C0111' is cryptic: use '# pylint: disable=missing-docstring' instead"
use-symbolic-message-instead:8:0::"'R0903' is cryptic: use '# pylint: enable=too-few-public-methods' instead"
use-symbolic-message-instead:9:0::"'R0903' is cryptic: use '# pylint: disable=too-few-public-methods' instead"
use-symbolic-message-instead:12:0::"'C0102' is cryptic: use '# pylint: disable=blacklisted-name' instead"
use-symbolic-message-instead:16:0::"'C0102' is cryptic: use '# pylint: disable=blacklisted-name' instead"
use-symbolic-message-instead:16:0::"'R1711' is cryptic: use '# pylint: disable=useless-return' instead"
missing-function-docstring:20:0:test_enabled_by_id_msg:Missing function or method docstring
use-symbolic-message-instead:20:0::"'C0111' is cryptic: use '# pylint: enable=missing-docstring' instead"
bad-option-value:1:0::Bad option value 'T1234':HIGH
use-symbolic-message-instead:1:0::"'C0111' is cryptic: use '# pylint: disable=missing-docstring' instead":HIGH
use-symbolic-message-instead:1:0::"'R0903' is cryptic: use '# pylint: disable=too-few-public-methods' instead":HIGH
use-symbolic-message-instead:2:0::"'c0111' is cryptic: use '# pylint: enable=missing-docstring' instead":HIGH
use-symbolic-message-instead:2:0::"'w0223' is cryptic: use '# pylint: enable=abstract-method' instead":HIGH
missing-function-docstring:4:0:my_function:Missing function or method docstring:HIGH
use-symbolic-message-instead:7:0::"'C0111' is cryptic: use '# pylint: disable=missing-docstring' instead":HIGH
use-symbolic-message-instead:8:0::"'R0903' is cryptic: use '# pylint: enable=too-few-public-methods' instead":HIGH
use-symbolic-message-instead:9:0::"'R0903' is cryptic: use '# pylint: disable=too-few-public-methods' instead":HIGH
use-symbolic-message-instead:12:0::"'C0102' is cryptic: use '# pylint: disable=blacklisted-name' instead":HIGH
use-symbolic-message-instead:16:0::"'C0102' is cryptic: use '# pylint: disable=blacklisted-name' instead":HIGH
use-symbolic-message-instead:16:0::"'R1711' is cryptic: use '# pylint: disable=useless-return' instead":HIGH
missing-function-docstring:20:0:test_enabled_by_id_msg:Missing function or method docstring:HIGH
use-symbolic-message-instead:20:0::"'C0111' is cryptic: use '# pylint: enable=missing-docstring' instead":HIGH

0 comments on commit bbaa7bc

Please sign in to comment.