{Pylint} Fix useless-object-inheritance (#30331)

Author: atombrella

scripts/ci/automation_full_test.py

@@ -496,7 +496,7 @@ def save_pipeline_result(pipeline_result):
     logger.info(f"save pipeline result to file: {filename}")
 
 
-class AutomaticScheduling(object):
+class AutomaticScheduling:
 
     def __init__(self):
         """

src/azure-cli-core/azure/cli/core/command_recommender.py

@@ -83,7 +83,7 @@ def get_error_type(error_msg):
     return error_type.value
 
 
-class CommandRecommender():  # pylint: disable=too-few-public-methods
+class CommandRecommender:  # pylint: disable=too-few-public-methods
     """Recommend a command for user when user's command fails.
     It combines Aladdin recommendations and examples in help files."""
 

src/azure-cli-core/azure/cli/core/extension/tests/latest/test_index_get.py

@@ -12,7 +12,7 @@
                                              ERR_UNABLE_TO_GET_EXTENSIONS)
 
 
-class MockResponse(object):
+class MockResponse:
     def __init__(self, status_code, data):
         self.status_code = status_code
         self.data = data

src/azure-cli-core/azure/cli/core/tests/test_generic_update.py

@@ -15,19 +15,19 @@
 from knack.util import CLIError
 
 
-class ListTestObject(object):
+class ListTestObject:
 
     def __init__(self, val):
         self.list_value = list(val)
 
 
-class DictTestObject(object):
+class DictTestObject:
 
     def __init__(self, val):
         self.dict_value = dict(val)
 
 
-class ObjectTestObject(object):
+class ObjectTestObject:
 
     def __init__(self, str_val, int_val, bool_val):
         self.my_string = str(str_val)

src/azure-cli-core/azure/cli/core/tests/test_help_loaders.py

@@ -52,7 +52,7 @@ def dummy_handler(arg1, arg2=None, arg3=None, arg4=None):
 # region Test Help Loader
 
 
-class JsonLoaderMixin(object):
+class JsonLoaderMixin:
     """A class containing helper methods for Json Loaders."""
 
     # get the list of json help file names for the command or group

src/azure-cli-core/azure/cli/core/tests/test_parser.py

@@ -274,7 +274,7 @@ def mock_add_extension(*args, **kwargs):
             self.assertIn("Extension another-ext-name installed. Please rerun your command.", logger_msgs[6])
 
 
-class VerifyError(object):  # pylint: disable=too-few-public-methods
+class VerifyError:  # pylint: disable=too-few-public-methods
 
     def __init__(self, test, substr=None):
         self.test = test

src/azure-cli-core/azure/cli/core/tests/test_profile.py

@@ -1493,7 +1493,7 @@ def test_get_auth_info_for_newly_created_service_principal(self):
         self.assertEqual('very_secret', extended_info['clientSecret'])
 
 
-class FileHandleStub(object):  # pylint: disable=too-few-public-methods
+class FileHandleStub:  # pylint: disable=too-few-public-methods
 
     def write(self, content):
         pass
@@ -1538,7 +1538,7 @@ def __init__(self, tenant_id):  # pylint: disable=redefined-builtin
         self.tenant_id = tenant_id
 
 
-class TenantStub(object):  # pylint: disable=too-few-public-methods
+class TenantStub:  # pylint: disable=too-few-public-methods
 
     def __init__(self, tenant_id, display_name=MOCK_TENANT_DISPLAY_NAME, default_domain=MOCK_TENANT_DEFAULT_DOMAIN):
         self.tenant_id = tenant_id

src/azure-cli-testsdk/azure/cli/testsdk/base.py

@@ -34,7 +34,7 @@
 COVERAGE_FILE = 'az_command_coverage.txt'
 
 
-class CheckerMixin(object):
+class CheckerMixin:
 
     def _apply_kwargs(self, val):
         try:
@@ -235,7 +235,7 @@ def get_subscription_id(self):
         return self.cmd('account list --query "[?isDefault].id" -o tsv').output.strip()
 
 
-class ExecutionResult(object):
+class ExecutionResult:
     def __init__(self, cli_ctx, command, expect_failure=False):
         self.output = ''
         self.applog = ''

src/azure-cli-testsdk/azure/cli/testsdk/checkers.py

@@ -9,7 +9,7 @@
 from .exceptions import JMESPathCheckAssertionError
 
 
-class JMESPathCheck(object):  # pylint: disable=too-few-public-methods
+class JMESPathCheck:  # pylint: disable=too-few-public-methods
     def __init__(self, query, expected_result, case_sensitive=True):
         self._query = query
         self._expected_result = expected_result
@@ -37,7 +37,7 @@ def __call__(self, execution_result):
                                               execution_result.output)
 
 
-class JMESPathCheckExists(object):  # pylint: disable=too-few-public-methods
+class JMESPathCheckExists:  # pylint: disable=too-few-public-methods
     def __init__(self, query):
         self._query = query
 
@@ -50,7 +50,7 @@ def __call__(self, execution_result):
                                               execution_result.output)
 
 
-class JMESPathCheckNotExists(object):  # pylint: disable=too-few-public-methods
+class JMESPathCheckNotExists:  # pylint: disable=too-few-public-methods
     def __init__(self, query):
         self._query = query
 
@@ -63,7 +63,7 @@ def __call__(self, execution_result):
                                               execution_result.output)
 
 
-class JMESPathCheckGreaterThan(object):  # pylint: disable=too-few-public-methods
+class JMESPathCheckGreaterThan:  # pylint: disable=too-few-public-methods
     def __init__(self, query, expected_result):
         self._query = query
         self._expected_result = expected_result
@@ -82,7 +82,7 @@ def __call__(self, execution_result):
                                               execution_result.output)
 
 
-class JMESPathPatternCheck(object):  # pylint: disable=too-few-public-methods
+class JMESPathPatternCheck:  # pylint: disable=too-few-public-methods
     def __init__(self, query, expected_result):
         self._query = query
         self._expected_result = expected_result
@@ -96,7 +96,7 @@ def __call__(self, execution_result):
                                               execution_result.output)
 
 
-class NoneCheck(object):  # pylint: disable=too-few-public-methods
+class NoneCheck:  # pylint: disable=too-few-public-methods
     def __call__(self, execution_result):  # pylint: disable=no-self-use
         none_strings = ['[]', '{}', 'false']
         try:
@@ -107,7 +107,7 @@ def __call__(self, execution_result):  # pylint: disable=no-self-use
                                  "string in {}".format(data, none_strings))
 
 
-class StringCheck(object):  # pylint: disable=too-few-public-methods
+class StringCheck:  # pylint: disable=too-few-public-methods
     def __init__(self, expected_result):
         self.expected_result = expected_result
 
@@ -120,7 +120,7 @@ def __call__(self, execution_result):
                 "Actual value '{}' != Expected value {}".format(result, self.expected_result))
 
 
-class StringContainCheck(object):  # pylint: disable=too-few-public-methods
+class StringContainCheck:  # pylint: disable=too-few-public-methods
     def __init__(self, expected_result):
         self.expected_result = expected_result
 
@@ -134,7 +134,7 @@ def __call__(self, execution_result):
                                                                              self.expected_result))
 
 
-class StringContainCheckIgnoreCase(object):  # pylint: disable=too-few-public-methods
+class StringContainCheckIgnoreCase:  # pylint: disable=too-few-public-methods
     def __init__(self, expected_result):
         self.expected_result = expected_result.lower()
 

src/azure-cli-testsdk/azure/cli/testsdk/scenario_tests/config.py

@@ -8,7 +8,7 @@
 from .const import ENV_LIVE_TEST
 
 
-class TestConfig(object):  # pylint: disable=too-few-public-methods
+class TestConfig:  # pylint: disable=too-few-public-methods
     def __init__(self, parent_parsers=None, config_file=None):
         parent_parsers = parent_parsers or []
         self.parser = configargparse.ArgumentParser(parents=parent_parsers)