Clean up unused imports in integration tests (#79895)

test/integration/targets/ansible-doc/broken-docs/collections/ansible_collections/testns/testcol/plugins/inventory/statichost.py

@@ -20,7 +20,6 @@ DOCUMENTATION = '''
         required: True
 '''
 
-from ansible.errors import AnsibleParserError
 from ansible.plugins.inventory import BaseInventoryPlugin, Cacheable
 
 

test/integration/targets/ansible-doc/collections/ansible_collections/testns/testcol/plugins/inventory/statichost.py

@@ -19,7 +19,6 @@ DOCUMENTATION = '''
         required: True
 '''
 
-from ansible.errors import AnsibleParserError
 from ansible.plugins.inventory import BaseInventoryPlugin, Cacheable
 
 

test/integration/targets/ansible-runner/files/adhoc_example1.py

@@ -2,7 +2,6 @@ from __future__ import (absolute_import, division, print_function)
 __metaclass__ = type
 
 import json
-import os
 import sys
 import ansible_runner
 

test/integration/targets/ansible-test-sanity-import/ansible_collections/ns/col/plugins/lookup/vendor1.py

@@ -16,10 +16,10 @@ RETURN = '''#'''
 
 from ansible.plugins.lookup import LookupBase
 # noinspection PyUnresolvedReferences
-from ansible.plugins import loader  # import the loader to verify it works when the collection loader has already been loaded
+from ansible.plugins import loader  # import the loader to verify it works when the collection loader has already been loaded  # pylint: disable=unused-import
 
 try:
-    import demo
+    import demo  # pylint: disable=unused-import
 except ImportError:
     pass
 else:

test/integration/targets/ansible-test-sanity-import/ansible_collections/ns/col/plugins/lookup/vendor2.py

@@ -16,10 +16,10 @@ RETURN = '''#'''
 
 from ansible.plugins.lookup import LookupBase
 # noinspection PyUnresolvedReferences
-from ansible.plugins import loader  # import the loader to verify it works when the collection loader has already been loaded
+from ansible.plugins import loader  # import the loader to verify it works when the collection loader has already been loaded  # pylint: disable=unused-import
 
 try:
-    import demo
+    import demo  # pylint: disable=unused-import
 except ImportError:
     pass
 else:

test/integration/targets/ansible-test-sanity/ansible_collections/ns/col/plugins/lookup/bad.py

@@ -19,9 +19,9 @@ EXAMPLES = '''
 RETURN = ''' # '''
 
 from ansible.plugins.lookup import LookupBase
-from ansible import constants
+from ansible import constants  # pylint: disable=unused-import
 
-import lxml
+import lxml  # pylint: disable=unused-import
 
 
 class LookupModule(LookupBase):

test/integration/targets/ansible-test-sanity/ansible_collections/ns/col/plugins/lookup/world.py

@@ -19,7 +19,7 @@ EXAMPLES = '''
 RETURN = ''' # '''
 
 from ansible.plugins.lookup import LookupBase
-from ansible import constants
+from ansible import constants  # pylint: disable=unused-import
 
 
 class LookupModule(LookupBase):

test/integration/targets/ansible-test-sanity/ansible_collections/ns/col/plugins/modules/bad.py

@@ -19,7 +19,7 @@ EXAMPLES = '''
 RETURN = ''''''
 
 from ansible.module_utils.basic import AnsibleModule
-from ansible import constants  # intentionally trigger pylint ansible-bad-module-import error
+from ansible import constants  # intentionally trigger pylint ansible-bad-module-import error  # pylint: disable=unused-import
 
 
 def main():

test/integration/targets/ansible-test-sanity/ansible_collections/ns/col/plugins/plugin_utils/check_pylint.py

@@ -9,7 +9,7 @@ __metaclass__ = type
 #   syntax-error: Cannot import 'string' due to syntax error 'invalid syntax (<unknown>, line 109)'
 # Python 3.9 fails with astroid 2.2.5 but works on 2.3.3
 #   syntax-error: Cannot import 'string' due to syntax error 'invalid syntax (<unknown>, line 104)'
-import string
+import string  # pylint: disable=unused-import
 
 # Python 3.9 fails with pylint 2.3.1 or 2.4.4 with astroid 2.3.3 but works with pylint 2.5.0 and astroid 2.4.0
 #   'Call' object has no attribute 'value'

test/integration/targets/ansible-test-sanity/ansible_collections/ns/col/plugins/random_directory/bad.py

@@ -5,4 +5,4 @@ __metaclass__ = type
 
 # This is not an allowed import, but since this file is in a plugins/ subdirectory that is not checked,
 # the import sanity test will not complain.
-import lxml
+import lxml  # pylint: disable=unused-import

test/integration/targets/ansible-test-sanity/ansible_collections/ns/col/tests/integration/targets/hello/files/bad.py

@@ -4,12 +4,12 @@ __metaclass__ = type
 import tempfile
 
 try:
-    import urllib2  # intentionally trigger pylint ansible-bad-import error
+    import urllib2  # intentionally trigger pylint ansible-bad-import error  # pylint: disable=unused-import
 except ImportError:
     urllib2 = None
 
 try:
-    from urllib2 import Request  # intentionally trigger pylint ansible-bad-import-from error
+    from urllib2 import Request  # intentionally trigger pylint ansible-bad-import-from error  # pylint: disable=unused-import
 except ImportError:
     Request = None
 

test/integration/targets/collections/collection_root_user/ansible_collections/testns/testcoll/plugins/modules/uses_mu_missing.py

@@ -2,10 +2,7 @@
 from __future__ import (absolute_import, division, print_function)
 __metaclass__ = type
 
-import json
-import sys
-
-from ..module_utils import bogusmu  # pylint: disable=relative-beyond-top-level
+from ..module_utils import bogusmu  # pylint: disable=relative-beyond-top-level,unused-import
 
 
 def main():

test/integration/targets/collections/collection_root_user/ansible_collections/testns/testcoll/plugins/modules/uses_mu_missing_redirect_collection.py

@@ -2,10 +2,7 @@
 from __future__ import (absolute_import, division, print_function)
 __metaclass__ = type
 
-import json
-import sys
-
-from ..module_utils import missing_redirect_target_collection  # pylint: disable=relative-beyond-top-level
+from ..module_utils import missing_redirect_target_collection  # pylint: disable=relative-beyond-top-level,unused-import
 
 
 def main():

test/integration/targets/collections/collection_root_user/ansible_collections/testns/testcoll/plugins/modules/uses_mu_missing_redirect_module.py

@@ -2,10 +2,7 @@
 from __future__ import (absolute_import, division, print_function)
 __metaclass__ = type
 
-import json
-import sys
-
-from ..module_utils import missing_redirect_target_module  # pylint: disable=relative-beyond-top-level
+from ..module_utils import missing_redirect_target_module  # pylint: disable=relative-beyond-top-level,unused-import
 
 
 def main():

test/integration/targets/collections/collections/ansible_collections/testns/content_adj/plugins/inventory/statichost.py

@@ -19,7 +19,6 @@ DOCUMENTATION = '''
         required: True
 '''
 
-from ansible.errors import AnsibleParserError
 from ansible.plugins.inventory import BaseInventoryPlugin, Cacheable
 
 

test/integration/targets/collections/test_task_resolved_plugin/callback_plugins/display_resolved_action.py

@@ -14,7 +14,6 @@ DOCUMENTATION = '''
       - Enable in configuration.
 '''
 
-from ansible import constants as C
 from ansible.plugins.callback import CallbackBase
 
 

test/integration/targets/egg-info/lookup_plugins/import_pkg_resources.py

@@ -1,7 +1,7 @@
 from __future__ import (absolute_import, division, print_function)
 __metaclass__ = type
 
-import pkg_resources
+import pkg_resources  # pylint: disable=unused-import
 
 from ansible.plugins.lookup import LookupBase
 

test/integration/targets/error_from_connection/connection_plugins/dummy.py

@@ -11,7 +11,6 @@ DOCUMENTATION = """
     version_added: "2.0"
     options: {}
 """
-import ansible.constants as C
 from ansible.errors import AnsibleError
 from ansible.plugins.connection import ConnectionBase
 

test/integration/targets/gathering_facts/library/file_utils.py

@@ -1,9 +1,6 @@
 from __future__ import (absolute_import, division, print_function)
 __metaclass__ = type
 
-import json
-import sys
-
 from ansible.module_utils.basic import AnsibleModule
 from ansible.module_utils.facts.utils import (
     get_file_content,

test/integration/targets/missing_required_lib/library/missing_required_lib.py

@@ -8,7 +8,7 @@ __metaclass__ = type
 from ansible.module_utils.basic import AnsibleModule, missing_required_lib
 
 try:
-    import ansible_missing_lib
+    import ansible_missing_lib  # pylint: disable=unused-import
     HAS_LIB = True
 except ImportError as e:
     HAS_LIB = False

test/integration/targets/module_defaults/collections/ansible_collections/testns/testcoll/plugins/action/eos.py

@@ -5,7 +5,6 @@ from __future__ import (absolute_import, division, print_function)
 __metaclass__ = type
 
 from ansible.plugins.action.normal import ActionModule as ActionBase
-from ansible.utils.vars import merge_hash
 
 
 class ActionModule(ActionBase):

test/integration/targets/module_defaults/collections/ansible_collections/testns/testcoll/plugins/action/ios.py

@@ -5,7 +5,6 @@ from __future__ import (absolute_import, division, print_function)
 __metaclass__ = type
 
 from ansible.plugins.action.normal import ActionModule as ActionBase
-from ansible.utils.vars import merge_hash
 
 
 class ActionModule(ActionBase):

test/integration/targets/module_defaults/collections/ansible_collections/testns/testcoll/plugins/action/vyos.py

@@ -5,7 +5,6 @@ from __future__ import (absolute_import, division, print_function)
 __metaclass__ = type
 
 from ansible.plugins.action.normal import ActionModule as ActionBase
-from ansible.utils.vars import merge_hash
 
 
 class ActionModule(ActionBase):

test/integration/targets/old_style_cache_plugins/plugins/cache/configurable_redis.py

@@ -44,7 +44,6 @@ DOCUMENTATION = '''
 import time
 import json
 
-from ansible import constants as C
 from ansible.errors import AnsibleError
 from ansible.parsing.ajson import AnsibleJSONEncoder, AnsibleJSONDecoder
 from ansible.plugins.cache import BaseCacheModule

test/integration/targets/pkg_resources/lookup_plugins/check_pkg_resources.py

@@ -11,7 +11,7 @@ __metaclass__ = type
 
 # noinspection PyUnresolvedReferences
 try:
-    from pkg_resources import Requirement
+    from pkg_resources import Requirement  # pylint: disable=unused-import
 except ImportError:
     Requirement = None
 

test/integration/targets/service/files/ansible_test_service.py

@@ -9,7 +9,6 @@ __metaclass__ = type
 import os
 import resource
 import signal
-import sys
 import time
 
 UMASK = 0

test/integration/targets/var_precedence/ansible-var-precedence-check.py

@@ -14,7 +14,6 @@ import stat
 import subprocess
 import tempfile
 import yaml
-from pprint import pprint
 from optparse import OptionParser
 from jinja2 import Environment
 

test/integration/targets/yum/filter_plugins/filter_list_of_tuples_by_first_param.py

@@ -1,8 +1,6 @@
 from __future__ import (absolute_import, division, print_function)
 __metaclass__ = type
 
-from ansible.errors import AnsibleError, AnsibleFilterError
-
 
 def filter_list_of_tuples_by_first_param(lst, search, startswith=False):
     out = []