diff options
author | Scott Moser <smoser@ubuntu.com> | 2012-08-09 13:42:55 -0400 |
---|---|---|
committer | Scott Moser <smoser@ubuntu.com> | 2012-08-09 13:42:55 -0400 |
commit | 0243c5d9c7948e247a85870a24835e2f5ce92cb0 (patch) | |
tree | b67c2911c07b013f893f1ac0c8428a99c9095628 /tests/unittests/test_handler | |
parent | 1bb67be5bd8c826717c9757f727406c73c7ef4e8 (diff) | |
parent | 0849ad447de8a1628e6b0158a5ce9b45d93f9bb8 (diff) | |
download | vyos-cloud-init-0243c5d9c7948e247a85870a24835e2f5ce92cb0.tar.gz vyos-cloud-init-0243c5d9c7948e247a85870a24835e2f5ce92cb0.zip |
fix pylint in all files used by ./tools/run-pylint
'make pylint' was not checking tests and tools.
This fixies a bunch of pylint/pep8 issues in that code.
It also enables 'make pylint' to check them.
Diffstat (limited to 'tests/unittests/test_handler')
-rw-r--r-- | tests/unittests/test_handler/test_handler_ca_certs.py | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/tests/unittests/test_handler/test_handler_ca_certs.py b/tests/unittests/test_handler/test_handler_ca_certs.py index 1f96e992..948de4c4 100644 --- a/tests/unittests/test_handler/test_handler_ca_certs.py +++ b/tests/unittests/test_handler/test_handler_ca_certs.py @@ -26,7 +26,8 @@ class TestNoConfig(MockerTestCase): self.mocker.replace(cc_ca_certs.update_ca_certs, passthrough=False) self.mocker.replay() - cc_ca_certs.handle(self.name, config, self.cloud_init, self.log, self.args) + cc_ca_certs.handle(self.name, config, self.cloud_init, self.log, + self.args) class TestConfig(MockerTestCase): @@ -39,11 +40,12 @@ class TestConfig(MockerTestCase): self.args = [] # Mock out the functions that actually modify the system - self.mock_add = self.mocker.replace(cc_ca_certs.add_ca_certs, passthrough=False) + self.mock_add = self.mocker.replace(cc_ca_certs.add_ca_certs, + passthrough=False) self.mock_update = self.mocker.replace(cc_ca_certs.update_ca_certs, passthrough=False) - self.mock_remove = self.mocker.replace(cc_ca_certs.remove_default_ca_certs, - passthrough=False) + self.mock_remove = self.mocker.replace( + cc_ca_certs.remove_default_ca_certs, passthrough=False) # Order must be correct self.mocker.order() @@ -183,8 +185,8 @@ class TestRemoveDefaultCaCerts(MockerTestCase): }) def test_commands(self): - mock_delete_dir_contents = self.mocker.replace(util.delete_dir_contents, - passthrough=False) + mock_delete_dir_contents = self.mocker.replace( + util.delete_dir_contents, passthrough=False) mock_write = self.mocker.replace(util.write_file, passthrough=False) mock_subp = self.mocker.replace(util.subp, passthrough=False) |