From bae9b11da9ed7dd0b16fe5adeaf4774b7cc628cf Mon Sep 17 00:00:00 2001
From: James Falcon <james.falcon@canonical.com>
Date: Wed, 15 Dec 2021 20:16:38 -0600
Subject: Adopt Black and isort (SC-700) (#1157)

Applied Black and isort, fixed any linting issues, updated tox.ini
and CI.
---
 tests/unittests/distros/test_sysconfig.py | 62 +++++++++++++++++--------------
 1 file changed, 34 insertions(+), 28 deletions(-)

(limited to 'tests/unittests/distros/test_sysconfig.py')

diff --git a/tests/unittests/distros/test_sysconfig.py b/tests/unittests/distros/test_sysconfig.py
index 4368496d..d0979e17 100644
--- a/tests/unittests/distros/test_sysconfig.py
+++ b/tests/unittests/distros/test_sysconfig.py
@@ -3,22 +3,23 @@
 import re
 
 from cloudinit.distros.parsers.sys_conf import SysConf
-
 from tests.unittests.helpers import TestCase
 
-
 # Lots of good examples @
 # http://content.hccfl.edu/pollock/AUnix1/SysconfigFilesDesc.txt
 
+
 class TestSysConfHelper(TestCase):
     # This function was added in 2.7, make it work for 2.6
     def assertRegMatches(self, text, regexp):
         regexp = re.compile(regexp)
-        self.assertTrue(regexp.search(text),
-                        msg="%s must match %s!" % (text, regexp.pattern))
+        self.assertTrue(
+            regexp.search(text),
+            msg="%s must match %s!" % (text, regexp.pattern),
+        )
 
     def test_parse_no_change(self):
-        contents = '''# A comment
+        contents = """# A comment
 USESMBAUTH=no
 KEYTABLE=/usr/lib/kbd/keytables/us.map
 SHORTDATE=$(date +%y:%m:%d:%H:%M)
@@ -28,59 +29,64 @@ NETMASK0=255.255.255.0
 LIST=$LOGROOT/incremental-list
 IPV6TO4_ROUTING='eth0-:0004::1/64 eth1-:0005::1/64'
 ETHTOOL_OPTS="-K ${DEVICE} tso on; -G ${DEVICE} rx 256 tx 256"
-USEMD5=no'''
+USEMD5=no"""
         conf = SysConf(contents.splitlines())
-        self.assertEqual(conf['HOSTNAME'], 'blahblah')
-        self.assertEqual(conf['SHORTDATE'], '$(date +%y:%m:%d:%H:%M)')
+        self.assertEqual(conf["HOSTNAME"], "blahblah")
+        self.assertEqual(conf["SHORTDATE"], "$(date +%y:%m:%d:%H:%M)")
         # Should be unquoted
-        self.assertEqual(conf['ETHTOOL_OPTS'], ('-K ${DEVICE} tso on; '
-                                                '-G ${DEVICE} rx 256 tx 256'))
+        self.assertEqual(
+            conf["ETHTOOL_OPTS"],
+            "-K ${DEVICE} tso on; -G ${DEVICE} rx 256 tx 256",
+        )
         self.assertEqual(contents, str(conf))
 
     def test_parse_shell_vars(self):
-        contents = 'USESMBAUTH=$XYZ'
+        contents = "USESMBAUTH=$XYZ"
         conf = SysConf(contents.splitlines())
         self.assertEqual(contents, str(conf))
-        conf = SysConf('')
-        conf['B'] = '${ZZ}d apples'
+        conf = SysConf("")
+        conf["B"] = "${ZZ}d apples"
         # Should be quoted
         self.assertEqual('B="${ZZ}d apples"', str(conf))
-        conf = SysConf('')
-        conf['B'] = '$? d apples'
+        conf = SysConf("")
+        conf["B"] = "$? d apples"
         self.assertEqual('B="$? d apples"', str(conf))
         contents = 'IPMI_WATCHDOG_OPTIONS="timeout=60"'
         conf = SysConf(contents.splitlines())
-        self.assertEqual('IPMI_WATCHDOG_OPTIONS=timeout=60', str(conf))
+        self.assertEqual("IPMI_WATCHDOG_OPTIONS=timeout=60", str(conf))
 
     def test_parse_adjust(self):
         contents = 'IPV6TO4_ROUTING="eth0-:0004::1/64 eth1-:0005::1/64"'
         conf = SysConf(contents.splitlines())
         # Should be unquoted
-        self.assertEqual('eth0-:0004::1/64 eth1-:0005::1/64',
-                         conf['IPV6TO4_ROUTING'])
-        conf['IPV6TO4_ROUTING'] = "blah \tblah"
+        self.assertEqual(
+            "eth0-:0004::1/64 eth1-:0005::1/64", conf["IPV6TO4_ROUTING"]
+        )
+        conf["IPV6TO4_ROUTING"] = "blah \tblah"
         contents2 = str(conf).strip()
         # Should be requoted due to whitespace
-        self.assertRegMatches(contents2,
-                              r'IPV6TO4_ROUTING=[\']blah\s+blah[\']')
+        self.assertRegMatches(
+            contents2, r"IPV6TO4_ROUTING=[\']blah\s+blah[\']"
+        )
 
     def test_parse_no_adjust_shell(self):
-        conf = SysConf(''.splitlines())
-        conf['B'] = ' $(time)'
+        conf = SysConf("".splitlines())
+        conf["B"] = " $(time)"
         contents = str(conf)
-        self.assertEqual('B= $(time)', contents)
+        self.assertEqual("B= $(time)", contents)
 
     def test_parse_empty(self):
-        contents = ''
+        contents = ""
         conf = SysConf(contents.splitlines())
-        self.assertEqual('', str(conf).strip())
+        self.assertEqual("", str(conf).strip())
 
     def test_parse_add_new(self):
-        contents = 'BLAH=b'
+        contents = "BLAH=b"
         conf = SysConf(contents.splitlines())
-        conf['Z'] = 'd'
+        conf["Z"] = "d"
         contents = str(conf)
         self.assertIn("Z=d", contents)
         self.assertIn("BLAH=b", contents)
 
+
 # vi: ts=4 expandtab
-- 
cgit v1.2.3