[bad-option-value] Use old_names to decrease message type to warning

Author: Pierre-Sassoulas

doc/data/messages/u/unknown-option-value/bad.py

@@ -0,0 +1 @@
+# disable=missnig-docstring  # [unknown-option-value]

doc/data/messages/u/unknown-option-value/good.py

@@ -0,0 +1 @@
+# disable=missing-docstring

doc/user_guide/messages/messages_overview.rst

@@ -58,7 +58,6 @@ All messages in the error category:
    error/bad-except-order
    error/bad-exception-context
    error/bad-format-character
-   error/bad-option-value
    error/bad-plugin-value
    error/bad-reversed-sequence
    error/bad-str-strip-call
@@ -178,6 +177,7 @@ All renamed messages in the error category:
    :titlesonly:
 
    error/bad-context-manager
+   error/bad-option-value
    error/maybe-no-member
    error/old-non-iterator-returned-2
    error/old-unbalanced-tuple-unpacking
@@ -306,6 +306,7 @@ All messages in the warning category:
    warning/try-except-raise
    warning/unbalanced-tuple-unpacking
    warning/undefined-loop-variable
+   warning/unknown-option-value
    warning/unnecessary-ellipsis
    warning/unnecessary-lambda
    warning/unnecessary-pass

doc/whatsnew/2/2.14/full.rst

@@ -31,8 +31,10 @@ Release date: TBA
 
   Closes #6624
 
-* Deleted messages that do not exist anymore in pylint now raise ``useless-option-value`` instead of ``bad-option-value``,
-  which permit to distinguish between genuine typos and old configuration that could be cleaned.
+* ``bad-option-value`` (E0012) is now a warning ``unknown-option-value`` (W0012). Deleted messages that do not exist
+  anymore in pylint now raise ``useless-option-value`` (I0012) instead of ``bad-option-value``. This permit to
+  distinguish between genuine typos and old configuration that could be cleaned.  Existing message control for
+  ``bad-option-value`` will still work on both new messages.
 
   Refs #6794
 

pylint/config/callback_actions.py

@@ -384,7 +384,7 @@ def _call(
                 ].append((option_string, str(e)))
             except exceptions.UnknownMessageError:
                 self.linter._stashed_messages[
-                    (self.linter.current_name, "bad-option-value")
+                    (self.linter.current_name, "unknown-option-value")
                 ].append((option_string, msgid))
 
     @abc.abstractmethod

pylint/lint/message_state_handler.py

@@ -419,7 +419,7 @@ def process_tokens(self, tokens: list[tokenize.TokenInfo]) -> None:
                             )
                         except exceptions.UnknownMessageError:
                             self.linter.add_message(
-                                "bad-option-value",
+                                "unknown-option-value",
                                 args=(pragma_repr.action, msgid),
                                 line=start[0],
                                 confidence=HIGH,

pylint/lint/pylinter.py

@@ -193,18 +193,24 @@ def _load_reporter_by_class(reporter_class: str) -> type[BaseReporter]:
         "Used when an unknown inline option is encountered.",
         {"scope": WarningScope.LINE},
     ),
-    "E0012": (
-        "Bad option value for '%s', expected a valid pylint message and got '%s'",
-        "bad-option-value",
-        "Used when a bad value for an option is encountered.",
-        {"scope": WarningScope.LINE},
+    "W0012": (
+        "Unknown option value for '%s', expected a valid pylint message and got '%s'",
+        "unknown-option-value",
+        "Used when an unknown value is encountered for an option.",
+        {
+            "scope": WarningScope.LINE,
+            "old_names": [("E0012", "bad-option-value")],
+        },
     ),
     "I0012": (
         "Useless option value for '%s', %s",
         "useless-option-value",
         "Used when a value for an option that is now deleted from pylint"
         " is encountered.",
-        {"scope": WarningScope.LINE},
+        {
+            "scope": WarningScope.LINE,
+            "old_names": [("E0012", "bad-option-value")],
+        },
     ),
     "E0013": (
         "Plugin '%s' is impossible to load, is it installed ? ('%s')",

tests/config/functional/ini/pylintrc_with_missing_comma.2.out

@@ -0,0 +1,3 @@
+************* Module {abspath}
+{relpath}:1:0: W0012: Unknown option value for '--disable', expected a valid pylint message and got 'logging-not-lazylogging-format-interpolation' (unknown-option-value)
+{relpath}:1:0: W0012: Unknown option value for '--enable', expected a valid pylint message and got 'locally-disabledsuppressed-message' (unknown-option-value)

tests/config/functional/ini/pylintrc_with_missing_comma.4.out

@@ -0,0 +1,2 @@
+************* Module {abspath}
+{relpath}:1:0: W0012: Unknown option value for '--enable', expected a valid pylint message and got 'useless-supression' (unknown-option-value)

tests/config/functional/setup_cfg/do_not_read_other_tools_configuration/setup.2.out

@@ -0,0 +1,3 @@
+************* Module {abspath}
+{relpath}:1:0: W0012: Unknown option value for '--disable', expected a valid pylint message and got 'logging-not-lazylogging-format-interpolation' (unknown-option-value)
+{relpath}:1:0: W0012: Unknown option value for '--enable', expected a valid pylint message and got 'locally-disabledsuppressed-message' (unknown-option-value)

tests/config/functional/setup_cfg/do_not_read_other_tools_configuration/setup.4.out

@@ -0,0 +1,3 @@
+************* Module {abspath}
+{relpath}:1:0: W0012: Unknown option value for '--disable', expected a valid pylint message and got 'logging-not-layzy' (unknown-option-value)
+{relpath}:1:0: W0012: Unknown option value for '--enable', expected a valid pylint message and got 'C00000' (unknown-option-value)

tests/config/functional/toml/issue_3122/toml_with_missing_comma.2.out

@@ -60,7 +60,7 @@ def test_unknown_message_id(capsys: CaptureFixture) -> None:
     """Check that we correctly raise a message on an unknown id."""
     Run([str(EMPTY_MODULE), "--disable=12345"], exit=False)
     output = capsys.readouterr()
-    assert "Command line:1:0: E0012: Bad option value for '--disable'" in output.out
+    assert "Command line:1:0: W0012: Unknown option value for '--disable'" in output.out
 
 
 def test_unknown_option_name(capsys: CaptureFixture) -> None:

tests/config/functional/toml/issue_3122/toml_with_missing_comma.4.out

@@ -1,20 +1,20 @@
 """Check unknown or deleted option. """
 
 # Standard disable with unknown value
-# pylint: disable=C05048  # [bad-option-value]
+# pylint: disable=C05048  # [unknown-option-value]
 # Standard disable with deleted symbol
 # pylint: disable=execfile-builtin  # [useless-option-value]
 # Standard disable with deleted msgid
 # pylint: disable=W1656  # [useless-option-value]
 # disable-next with unknown value
-# pylint: disable-next=R78948  # [bad-option-value]
+# pylint: disable-next=R78948  # [unknown-option-value]
 # disable-next with deleted symbol
 # pylint: disable-next=deprecated-types-field  # [useless-option-value]
 # disable-next with deleted msgid
 # pylint: disable-next=W1634  # [useless-option-value]
 
 # enable with unknown value
-# pylint:enable=W04044  # [bad-option-value]
+# pylint:enable=W04044  # [unknown-option-value]
 # enable with deleted symbol
 # pylint:enable=dict-values-not-iterating  # [useless-option-value]
 # enable with deleted msgid

tests/config/functional/toml/unknown_msgid/enable_unknown_msgid.2.out

@@ -1,10 +1,10 @@
-bad-option-value:4:0:None:None::Bad option value for 'disable', expected a valid pylint message and got 'C05048':HIGH
+unknown-option-value:4:0:None:None::Unknown option value for 'disable', expected a valid pylint message and got 'C05048':HIGH
 useless-option-value:6:0:None:None::"Useless option value for 'disable', 'execfile-builtin' was removed from pylint, see https://github.com/PyCQA/pylint/pull/4942.":HIGH
 useless-option-value:8:0:None:None::"Useless option value for 'disable', 'W1656' was removed from pylint, see https://github.com/PyCQA/pylint/pull/4942.":HIGH
-bad-option-value:10:0:None:None::Bad option value for 'disable-next', expected a valid pylint message and got 'R78948':HIGH
+unknown-option-value:10:0:None:None::Unknown option value for 'disable-next', expected a valid pylint message and got 'R78948':HIGH
 useless-option-value:12:0:None:None::"Useless option value for 'disable-next', 'deprecated-types-field' was removed from pylint, see https://github.com/PyCQA/pylint/pull/4942.":HIGH
 useless-option-value:14:0:None:None::"Useless option value for 'disable-next', 'W1634' was removed from pylint, see https://github.com/PyCQA/pylint/pull/4942.":HIGH
-bad-option-value:17:0:None:None::Bad option value for 'enable', expected a valid pylint message and got 'W04044':HIGH
+unknown-option-value:17:0:None:None::Unknown option value for 'enable', expected a valid pylint message and got 'W04044':HIGH
 useless-option-value:19:0:None:None::"Useless option value for 'enable', 'dict-values-not-iterating' was removed from pylint, see https://github.com/PyCQA/pylint/pull/4942.":HIGH
 useless-option-value:21:0:None:None::"Useless option value for 'enable', 'W1622' was removed from pylint, see https://github.com/PyCQA/pylint/pull/4942.":HIGH
 useless-option-value:24:0:None:None::"Useless option value for 'disable', 'no-space-after-operator' was removed from pylint, see https://github.com/PyCQA/pylint/pull/3577.":HIGH

tests/config/functional/toml/unknown_msgid/enable_unknown_msgid.4.out

@@ -7,8 +7,8 @@
 
 # pylint: enable=bad-option-value
 
-var = 1  # pylint: disable=a-removed-option # [bad-option-value]
+var = 1  # pylint: disable=a-removed-option # [unknown-option-value]
 
 # bad-option-value needs to be disabled before the bad option
-var = 1  # pylint: disable=a-removed-option, bad-option-value # [bad-option-value]
+var = 1  # pylint: disable=a-removed-option, bad-option-value # [unknown-option-value]
 var = 1  # pylint: disable=bad-option-value, a-removed-option

tests/config/test_config.py

@@ -1,2 +1,2 @@
-bad-option-value:10:0:None:None::Bad option value for 'disable', expected a valid pylint message and got 'a-removed-option':HIGH
-bad-option-value:13:0:None:None::Bad option value for 'disable', expected a valid pylint message and got 'a-removed-option':HIGH
+unknown-option-value:10:0:None:None::Unknown option value for 'disable', expected a valid pylint message and got 'a-removed-option':HIGH
+unknown-option-value:13:0:None:None::Unknown option value for 'disable', expected a valid pylint message and got 'a-removed-option':HIGH

tests/functional/b/bad_option_value.py

@@ -1,4 +1,4 @@
-# pylint: disable=C0111,R0903,T1234  # [bad-option-value,use-symbolic-message-instead,use-symbolic-message-instead]
+# pylint: disable=C0111,R0903,T1234  # [unknown-option-value,use-symbolic-message-instead,use-symbolic-message-instead]
 # pylint: enable=c0111,w0223   # [use-symbolic-message-instead,use-symbolic-message-instead]
 
 def my_function(arg):  # [missing-function-docstring]

tests/functional/b/bad_option_value.txt

@@ -1,4 +1,4 @@
-bad-option-value:1:0:None:None::Bad option value for 'disable', expected a valid pylint message and got 'T1234':HIGH
+unknown-option-value:1:0:None:None::Unknown option value for 'disable', expected a valid pylint message and got 'T1234':HIGH
 use-symbolic-message-instead:1:0:None:None::"'C0111' is cryptic: use '# pylint: disable=missing-docstring' instead":UNDEFINED
 use-symbolic-message-instead:1:0:None:None::"'R0903' is cryptic: use '# pylint: disable=too-few-public-methods' instead":UNDEFINED
 use-symbolic-message-instead:2:0:None:None::"'c0111' is cryptic: use '# pylint: enable=missing-docstring' instead":UNDEFINED