Adopt Black and isort (SC-700) (#1157)

Applied Black and isort, fixed any linting issues, updated tox.ini
and CI.

1 files changed, 87 insertions, 61 deletions

diff --git a/tests/unittests/config/test_cc_apk_configure.py b/tests/unittests/config/test_cc_apk_configure.py
index 70139451..6fbc3dec 100644
--- a/tests/unittests/config/test_cc_apk_configure.py
+++ b/tests/unittests/config/test_cc_apk_configure.py
@@ -8,20 +8,19 @@ import logging
 import os
 import textwrap
 
-from cloudinit import (cloud, helpers, util)
-
+from cloudinit import cloud, helpers, util
 from cloudinit.config import cc_apk_configure
-from tests.unittests.helpers import (FilesystemMockingTestCase, mock)
+from tests.unittests.helpers import FilesystemMockingTestCase, mock
 
 REPO_FILE = "/etc/apk/repositories"
 DEFAULT_MIRROR_URL = "https://alpine.global.ssl.fastly.net/alpine"
-CC_APK = 'cloudinit.config.cc_apk_configure'
+CC_APK = "cloudinit.config.cc_apk_configure"
 
 
 class TestNoConfig(FilesystemMockingTestCase):
     def setUp(self):
         super(TestNoConfig, self).setUp()
-        self.add_patch(CC_APK + '._write_repositories_file', 'm_write_repos')
+        self.add_patch(CC_APK + "._write_repositories_file", "m_write_repos")
         self.name = "apk-configure"
         self.cloud_init = None
         self.log = logging.getLogger("TestNoConfig")
@@ -34,8 +33,9 @@ class TestNoConfig(FilesystemMockingTestCase):
         """
         config = util.get_builtin_cfg()
 
-        cc_apk_configure.handle(self.name, config, self.cloud_init,
-                                self.log, self.args)
+        cc_apk_configure.handle(
+            self.name, config, self.cloud_init, self.log, self.args
+        )
 
         self.assertEqual(0, self.m_write_repos.call_count)
 
@@ -45,15 +45,15 @@ class TestConfig(FilesystemMockingTestCase):
         super(TestConfig, self).setUp()
         self.new_root = self.tmp_dir()
         self.new_root = self.reRoot(root=self.new_root)
-        for dirname in ['tmp', 'etc/apk']:
+        for dirname in ["tmp", "etc/apk"]:
             util.ensure_dir(os.path.join(self.new_root, dirname))
-        self.paths = helpers.Paths({'templates_dir': self.new_root})
+        self.paths = helpers.Paths({"templates_dir": self.new_root})
         self.name = "apk-configure"
         self.cloud = cloud.Cloud(None, self.paths, None, None, None)
         self.log = logging.getLogger("TestNoConfig")
         self.args = []
 
-    @mock.patch(CC_APK + '._write_repositories_file')
+    @mock.patch(CC_APK + "._write_repositories_file")
     def test_no_repo_settings(self, m_write_repos):
         """
         Test that nothing is written if the 'alpine-repo' key
@@ -61,20 +61,22 @@ class TestConfig(FilesystemMockingTestCase):
         """
         config = {"apk_repos": {}}
 
-        cc_apk_configure.handle(self.name, config, self.cloud, self.log,
-                                self.args)
+        cc_apk_configure.handle(
+            self.name, config, self.cloud, self.log, self.args
+        )
 
         self.assertEqual(0, m_write_repos.call_count)
 
-    @mock.patch(CC_APK + '._write_repositories_file')
+    @mock.patch(CC_APK + "._write_repositories_file")
     def test_empty_repo_settings(self, m_write_repos):
         """
         Test that nothing is written if 'alpine_repo' list is empty.
         """
         config = {"apk_repos": {"alpine_repo": []}}
 
-        cc_apk_configure.handle(self.name, config, self.cloud, self.log,
-                                self.args)
+        cc_apk_configure.handle(
+            self.name, config, self.cloud, self.log, self.args
+        )
 
         self.assertEqual(0, m_write_repos.call_count)
 
@@ -82,19 +84,15 @@ class TestConfig(FilesystemMockingTestCase):
         """
         Test when only details of main repo is written to file.
         """
-        alpine_version = 'v3.12'
-        config = {
-            "apk_repos": {
-                "alpine_repo": {
-                    "version": alpine_version
-                }
-            }
-        }
+        alpine_version = "v3.12"
+        config = {"apk_repos": {"alpine_repo": {"version": alpine_version}}}
 
-        cc_apk_configure.handle(self.name, config, self.cloud, self.log,
-                                self.args)
+        cc_apk_configure.handle(
+            self.name, config, self.cloud, self.log, self.args
+        )
 
-        expected_content = textwrap.dedent("""\
+        expected_content = textwrap.dedent(
+            """\
             #
             # Created by cloud-init
             #
@@ -103,7 +101,10 @@ class TestConfig(FilesystemMockingTestCase):
 
             {0}/{1}/main
 
-            """.format(DEFAULT_MIRROR_URL, alpine_version))
+            """.format(
+                DEFAULT_MIRROR_URL, alpine_version
+            )
+        )
 
         self.assertEqual(expected_content, util.load_file(REPO_FILE))
 
@@ -112,20 +113,22 @@ class TestConfig(FilesystemMockingTestCase):
         Test when only details of main and community repos are
         written to file.
         """
-        alpine_version = 'edge'
+        alpine_version = "edge"
         config = {
             "apk_repos": {
                 "alpine_repo": {
                     "version": alpine_version,
-                    "community_enabled": True
+                    "community_enabled": True,
                 }
             }
         }
 
-        cc_apk_configure.handle(self.name, config, self.cloud, self.log,
-                                self.args)
+        cc_apk_configure.handle(
+            self.name, config, self.cloud, self.log, self.args
+        )
 
-        expected_content = textwrap.dedent("""\
+        expected_content = textwrap.dedent(
+            """\
             #
             # Created by cloud-init
             #
@@ -135,7 +138,10 @@ class TestConfig(FilesystemMockingTestCase):
             {0}/{1}/main
             {0}/{1}/community
 
-            """.format(DEFAULT_MIRROR_URL, alpine_version))
+            """.format(
+                DEFAULT_MIRROR_URL, alpine_version
+            )
+        )
 
         self.assertEqual(expected_content, util.load_file(REPO_FILE))
 
@@ -144,21 +150,23 @@ class TestConfig(FilesystemMockingTestCase):
         Test when details of main, community and testing repos
         are written to file.
         """
-        alpine_version = 'v3.12'
+        alpine_version = "v3.12"
         config = {
             "apk_repos": {
                 "alpine_repo": {
                     "version": alpine_version,
                     "community_enabled": True,
-                    "testing_enabled": True
+                    "testing_enabled": True,
                 }
             }
         }
 
-        cc_apk_configure.handle(self.name, config, self.cloud, self.log,
-                                self.args)
+        cc_apk_configure.handle(
+            self.name, config, self.cloud, self.log, self.args
+        )
 
-        expected_content = textwrap.dedent("""\
+        expected_content = textwrap.dedent(
+            """\
             #
             # Created by cloud-init
             #
@@ -172,7 +180,10 @@ class TestConfig(FilesystemMockingTestCase):
             #
             {0}/edge/testing
 
-            """.format(DEFAULT_MIRROR_URL, alpine_version))
+            """.format(
+                DEFAULT_MIRROR_URL, alpine_version
+            )
+        )
 
         self.assertEqual(expected_content, util.load_file(REPO_FILE))
 
@@ -181,21 +192,23 @@ class TestConfig(FilesystemMockingTestCase):
         Test when details of main, community and testing repos
         for Edge version of Alpine are written to file.
         """
-        alpine_version = 'edge'
+        alpine_version = "edge"
         config = {
             "apk_repos": {
                 "alpine_repo": {
                     "version": alpine_version,
                     "community_enabled": True,
-                    "testing_enabled": True
+                    "testing_enabled": True,
                 }
             }
         }
 
-        cc_apk_configure.handle(self.name, config, self.cloud, self.log,
-                                self.args)
+        cc_apk_configure.handle(
+            self.name, config, self.cloud, self.log, self.args
+        )
 
-        expected_content = textwrap.dedent("""\
+        expected_content = textwrap.dedent(
+            """\
             #
             # Created by cloud-init
             #
@@ -206,7 +219,10 @@ class TestConfig(FilesystemMockingTestCase):
             {0}/{1}/community
             {0}/{1}/testing
 
-            """.format(DEFAULT_MIRROR_URL, alpine_version))
+            """.format(
+                DEFAULT_MIRROR_URL, alpine_version
+            )
+        )
 
         self.assertEqual(expected_content, util.load_file(REPO_FILE))
 
@@ -215,23 +231,25 @@ class TestConfig(FilesystemMockingTestCase):
         Test when details of main, community, testing and
         local repos are written to file.
         """
-        alpine_version = 'v3.12'
-        local_repo_url = 'http://some.mirror/whereever'
+        alpine_version = "v3.12"
+        local_repo_url = "http://some.mirror/whereever"
         config = {
             "apk_repos": {
                 "alpine_repo": {
                     "version": alpine_version,
                     "community_enabled": True,
-                    "testing_enabled": True
+                    "testing_enabled": True,
                 },
-                "local_repo_base_url": local_repo_url
+                "local_repo_base_url": local_repo_url,
             }
         }
 
-        cc_apk_configure.handle(self.name, config, self.cloud, self.log,
-                                self.args)
+        cc_apk_configure.handle(
+            self.name, config, self.cloud, self.log, self.args
+        )
 
-        expected_content = textwrap.dedent("""\
+        expected_content = textwrap.dedent(
+            """\
             #
             # Created by cloud-init
             #
@@ -250,7 +268,10 @@ class TestConfig(FilesystemMockingTestCase):
             #
             {2}/{1}
 
-            """.format(DEFAULT_MIRROR_URL, alpine_version, local_repo_url))
+            """.format(
+                DEFAULT_MIRROR_URL, alpine_version, local_repo_url
+            )
+        )
 
         self.assertEqual(expected_content, util.load_file(REPO_FILE))
 
@@ -259,23 +280,25 @@ class TestConfig(FilesystemMockingTestCase):
         Test when details of main, community, testing and local repos
         for Edge version of Alpine are written to file.
         """
-        alpine_version = 'edge'
-        local_repo_url = 'http://some.mirror/whereever'
+        alpine_version = "edge"
+        local_repo_url = "http://some.mirror/whereever"
         config = {
             "apk_repos": {
                 "alpine_repo": {
                     "version": alpine_version,
                     "community_enabled": True,
-                    "testing_enabled": True
+                    "testing_enabled": True,
                 },
-                "local_repo_base_url": local_repo_url
+                "local_repo_base_url": local_repo_url,
             }
         }
 
-        cc_apk_configure.handle(self.name, config, self.cloud, self.log,
-                                self.args)
+        cc_apk_configure.handle(
+            self.name, config, self.cloud, self.log, self.args
+        )
 
-        expected_content = textwrap.dedent("""\
+        expected_content = textwrap.dedent(
+            """\
             #
             # Created by cloud-init
             #
@@ -291,7 +314,10 @@ class TestConfig(FilesystemMockingTestCase):
             #
             {2}/{1}
 
-            """.format(DEFAULT_MIRROR_URL, alpine_version, local_repo_url))
+            """.format(
+                DEFAULT_MIRROR_URL, alpine_version, local_repo_url
+            )
+        )
 
         self.assertEqual(expected_content, util.load_file(REPO_FILE))