test: fix all flake8 E241 (canonical#403)

Joshua Powers · web-flow · commit 5f7825e22241 · 2020-06-02T11:24:29.000-04:00
Remove extra spaces after a ','
diff --git a/cloudinit/cmd/tests/test_query.py b/cloudinit/cmd/tests/test_query.py
@@ -261,7 +261,7 @@ def test_handle_args_list_keys_errors_when_varname_is_not_a_dict(self):
         args = self.args(
             debug=False, dump_all=False, format=None,
             instance_data=self.instance_data, list_keys=True, user_data='ud',
-            vendor_data='vd',  varname='top')
+            vendor_data='vd', varname='top')
         with mock.patch('sys.stderr', new_callable=StringIO) as m_stderr:
             with mock.patch('sys.stdout', new_callable=StringIO) as m_stdout:
                 with mock.patch('os.getuid') as m_getuid:
diff --git a/cloudinit/config/cc_apt_configure.py b/cloudinit/config/cc_apt_configure.py
@@ -297,7 +297,7 @@
                 },
                 'conf': {
                     'type': 'string',
-                    'description':  dedent("""\
+                    'description': dedent("""\
                         Specify configuration for apt, such as proxy
                         configuration. This configuration is specified as a
                         string. For multiline apt configuration, make sure
diff --git a/cloudinit/net/tests/test_dhcp.py b/cloudinit/net/tests/test_dhcp.py
@@ -211,7 +211,7 @@ def test_parse_static_routes_logs_error_truncated(self):
             "class_b": "16,172,16,10",
             "class_a": "8,10,10",
             "gateway": "0,0",
-            "netlen":  "33,0",
+            "netlen": "33,0",
         }
         for rfc3442 in bad_rfc3442.values():
             self.assertEqual([], parse_static_routes(rfc3442))
diff --git a/cloudinit/sources/DataSourceGCE.py b/cloudinit/sources/DataSourceGCE.py
@@ -116,7 +116,7 @@ def _write_host_key_to_guest_attributes(key_type, key_value):
     resp = url_helper.readurl(url=url, data=key_value, headers=HEADERS,
                               request_method='PUT', check_status=False)
     if resp.ok():
-        LOG.debug('Wrote %s host key to guest attributes.',  key_type)
+        LOG.debug('Wrote %s host key to guest attributes.', key_type)
     else:
         LOG.debug('Unable to write %s host key to guest attributes.', key_type)
 
diff --git a/cloudinit/tests/test_url_helper.py b/cloudinit/tests/test_url_helper.py
@@ -85,7 +85,7 @@ def test_read_file_or_url_str_from_url_redacting_headers_from_logs(self):
         read_file_or_url(url, headers=headers, headers_redact=['sensitive'])
         logs = self.logs.getvalue()
         for k in headers.keys():
-            self.assertEqual(headers[k],  httpretty.last_request().headers[k])
+            self.assertEqual(headers[k], httpretty.last_request().headers[k])
         self.assertIn(REDACTED, logs)
         self.assertNotIn('sekret', logs)
 
diff --git a/tests/unittests/test_net.py b/tests/unittests/test_net.py
@@ -944,7 +944,7 @@
                         dhcp6: true
         """).rstrip(' '),
         'expected_sysconfig_opensuse': {
-            'ifcfg-iface0':  textwrap.dedent("""\
+            'ifcfg-iface0': textwrap.dedent("""\
                 BOOTPROTO=dhcp
                 DHCLIENT6_MODE=managed
                 STARTMODE=auto""")
@@ -1028,7 +1028,7 @@
     },
     'v6_and_v4': {
         'expected_sysconfig_opensuse': {
-            'ifcfg-iface0':  textwrap.dedent("""\
+            'ifcfg-iface0': textwrap.dedent("""\
                 BOOTPROTO=dhcp
                 DHCLIENT6_MODE=managed
                 STARTMODE=auto""")
diff --git a/tox.ini b/tox.ini
@@ -47,10 +47,9 @@ deps = -r{toxinidir}/test-requirements.txt
 # E123: closing bracket does not match indentation of opening bracket’s line
 # E126: continuation line over-indented for hanging indent
 # E226: missing whitespace around arithmetic operator
-# E241: multiple spaces after ‘,’
 # W503: line break before binary operator
 # W504: line break after binary operator
-ignore=E121,E123,E126,E226,E241,W503,W504
+ignore=E121,E123,E126,E226,W503,W504
 exclude = .venv,.tox,dist,doc,*egg,.git,build,tools
 per-file-ignores =
     cloudinit/cmd/main.py:E402

Original file line number	Diff line number	Diff line change
`@@ -211,7 +211,7 @@ def test_parse_static_routes_logs_error_truncated(self):`
`211`	`211`	`"class_b": "16,172,16,10",`
`212`	`212`	`"class_a": "8,10,10",`
`213`	`213`	`"gateway": "0,0",`
`214`		`- "netlen": "33,0",`
	`214`	`+ "netlen": "33,0",`
`215`	`215`	`}`
`216`	`216`	`for rfc3442 in bad_rfc3442.values():`
`217`	`217`	`self.assertEqual([], parse_static_routes(rfc3442))`