Make PyLinter subclass ArgumentsManager and add tests

pylint/lint/pylinter.py

@@ -1,6 +1,7 @@
 # Licensed under the GPL: https://www.gnu.org/licenses/old-licenses/gpl-2.0.html
 # For details: https://github.com/PyCQA/pylint/blob/main/LICENSE
 
+import argparse
 import collections
 import contextlib
 import functools
@@ -192,6 +193,7 @@ class PyLinter(
     config.OptionsManagerMixIn,
     reporters.ReportsHandlerMixIn,
     checkers.BaseTokenChecker,
+    config._ArgumentsManager,
 ):
     """lint Python modules using external checkers.
 
@@ -600,6 +602,7 @@ def __init__(
         self._by_id_managed_msgs: List[ManagedMessage] = []
 
         reporters.ReportsHandlerMixIn.__init__(self)
+        config._ArgumentsManager.__init__(self, argparse.Namespace())
         super().__init__(
             usage=__doc__,
             config_file=pylintrc or next(config.find_default_config_files(), None),

tests/config/test_argparse_config.py

@@ -0,0 +1,23 @@
+"""Test for the (new) implementation of option parsing with argparse"""
+
+import argparse
+
+from pylint import config
+from pylint.lint import PyLinter
+
+
+class TestArgumentsManager:
+    """Tests for the ArgumentsManager class"""
+
+    @staticmethod
+    def test_initialize_arguments_manager_class(linter: PyLinter) -> None:
+        """Tests that ArgumentsManager is correctly initialized on a PyLinter class"""
+        assert isinstance(linter, config._ArgumentsManager)
+        assert isinstance(linter.namespace, argparse.Namespace)
+
+    @staticmethod
+    def test_help_message(linter: PyLinter) -> None:
+        """Test the help_message attribute of the ArgumentsManager class"""
+        assert isinstance(linter.help_message, str)
+        lines = linter.help_message.splitlines()
+        assert lines[0] == "usage: pylint [-h]"

tests/test_regr.py

@@ -145,6 +145,7 @@ def test_pylint_config_attr() -> None:
         "BaseTokenChecker",
         "BaseChecker",
         "OptionsProviderMixIn",
+        "_ArgumentsManager",
     ]
     assert [c.name for c in pylinter.ancestors()] == expect
     assert list(astroid.Instance(pylinter).getattr("config"))