Remove unnecessary test imports. (#583)

tests/unit/compat/builtins.py

@@ -13,7 +13,7 @@ __metaclass__ = type
 # One unittest needs to import builtins via __import__() so we need to have
 # the string that represents it
 try:
-    import __builtin__
+    import __builtin__  # noqa: F401, pylint: disable=unused-import
 except ImportError:
     BUILTINS = 'builtins'
 else:

tests/unit/compat/mock.py

@@ -9,7 +9,7 @@ __metaclass__ = type
 '''
 Compat module for Python3.x's unittest.mock module
 '''
-import sys
+import sys  # noqa: F401, pylint: disable=unused-import
 
 # Python 2.7
 
@@ -20,11 +20,11 @@ try:
     # Allow wildcard import because we really do want to import all of mock's
     # symbols into this compat shim
     # pylint: disable=wildcard-import,unused-wildcard-import
-    from unittest.mock import *
+    from unittest.mock import *  # noqa: F401, pylint: disable=unused-import
 except ImportError:
     # Python 2
     # pylint: disable=wildcard-import,unused-wildcard-import
     try:
-        from mock import *
+        from mock import *  # noqa: F401, pylint: disable=unused-import
     except ImportError:
         print('You need the mock library installed on python2.x to run tests')

tests/unit/plugins/connection/test_docker.py

@@ -7,13 +7,11 @@ from __future__ import (absolute_import, division, print_function)
 __metaclass__ = type
 
 from io import StringIO
-import pytest
 
 from ansible_collections.community.docker.tests.unit.compat import mock
 from ansible_collections.community.docker.tests.unit.compat import unittest
 from ansible.errors import AnsibleError
 from ansible.playbook.play_context import PlayContext
-from ansible_collections.community.docker.plugins.connection.docker import Connection as DockerConnection
 from ansible.plugins.loader import connection_loader
 
 

tests/unit/plugins/inventory/test_docker_containers.py

@@ -6,17 +6,9 @@ from __future__ import (absolute_import, division, print_function)
 __metaclass__ = type
 
 
-import json
-import textwrap
-
 import pytest
 
-from ansible_collections.community.docker.tests.unit.compat.mock import MagicMock
-
-from ansible import constants as C
-from ansible.errors import AnsibleError
 from ansible.inventory.data import InventoryData
-from ansible.inventory.manager import InventoryManager
 
 from ansible_collections.community.docker.plugins.inventory.docker_containers import InventoryModule
 

tests/unit/plugins/module_utils/_api/test_errors.py

@@ -20,11 +20,10 @@ if sys.version_info < (2, 7):
     pytestmark = pytest.mark.skip('Python 2.6 is not supported')
 
 from ansible_collections.community.docker.plugins.module_utils._api.errors import (
-    APIError, ContainerError, DockerException,
+    APIError, DockerException,
     create_unexpected_kwargs_error,
     create_api_error_from_http_exception,
 )
-from .fake_api import FAKE_CONTAINER_ID, FAKE_IMAGE_ID
 
 
 class APIErrorTest(unittest.TestCase):

tests/unit/plugins/module_utils/test__scramble.py

@@ -6,7 +6,6 @@ from __future__ import (absolute_import, division, print_function)
 __metaclass__ = type
 
 import pytest
-import tarfile
 
 from ansible_collections.community.docker.plugins.module_utils._scramble import (
     scramble,

tests/unit/plugins/module_utils/test_copy.py

@@ -6,7 +6,6 @@ from __future__ import (absolute_import, division, print_function)
 __metaclass__ = type
 
 import pytest
-import tarfile
 
 from ansible_collections.community.docker.plugins.module_utils.copy import (
     _stream_generator_to_fileobj,

tests/unit/plugins/modules/test_docker_image.py

@@ -9,10 +9,7 @@ import pytest
 
 from ansible_collections.community.docker.plugins.modules.docker_image import ImageManager
 
-from ansible_collections.community.docker.plugins.module_utils.image_archive import (
+from ansible_collections.community.docker.plugins.module_utils.image_archive import api_image_id
-    api_image_id,
-    ImageArchiveInvalidException
-)
 
 from ..test_support.docker_image_archive_stubbing import (
     write_imitation_archive,