diff options
author | Scott Moser <smoser@ubuntu.com> | 2018-03-15 11:45:12 -0400 |
---|---|---|
committer | Scott Moser <smoser@brickies.net> | 2018-03-15 11:45:12 -0400 |
commit | bef2f2c945fdee4a1141c3177b3e48b1537027e4 (patch) | |
tree | 2b85e97eff63df46075eab57dadbedbcc23de410 /tests | |
parent | 133ad2cb327ad17b7b81319fac8f9f14577c04df (diff) | |
download | vyos-cloud-init-bef2f2c945fdee4a1141c3177b3e48b1537027e4.tar.gz vyos-cloud-init-bef2f2c945fdee4a1141c3177b3e48b1537027e4.zip |
tests: Fix some warnings in tests that popped up with newer python.
When running 'tox -e pylint' on a bionic system (python 3.6.4) I started
seeing errors today like:
tests/cloud_tests/platforms/__init__.py:5: [E0401(import-error), ]
Unable to import 'tests.cloud_tests.platforms.ec2'
The fix for those first errors was simply to create the __init__.py.
The second set of changes fixes fallout found from actually now having
pylint properly run on more of the cloud_tests.
Diffstat (limited to 'tests')
-rw-r--r-- | tests/cloud_tests/platforms/ec2/__init__.py | 0 | ||||
-rw-r--r-- | tests/cloud_tests/platforms/lxd/__init__.py | 0 | ||||
-rw-r--r-- | tests/cloud_tests/platforms/lxd/platform.py | 4 | ||||
-rw-r--r-- | tests/cloud_tests/platforms/nocloudkvm/__init__.py | 0 | ||||
-rw-r--r-- | tests/cloud_tests/platforms/nocloudkvm/instance.py | 2 | ||||
-rw-r--r-- | tests/cloud_tests/platforms/nocloudkvm/platform.py | 4 |
6 files changed, 1 insertions, 9 deletions
diff --git a/tests/cloud_tests/platforms/ec2/__init__.py b/tests/cloud_tests/platforms/ec2/__init__.py new file mode 100644 index 00000000..e69de29b --- /dev/null +++ b/tests/cloud_tests/platforms/ec2/__init__.py diff --git a/tests/cloud_tests/platforms/lxd/__init__.py b/tests/cloud_tests/platforms/lxd/__init__.py new file mode 100644 index 00000000..e69de29b --- /dev/null +++ b/tests/cloud_tests/platforms/lxd/__init__.py diff --git a/tests/cloud_tests/platforms/lxd/platform.py b/tests/cloud_tests/platforms/lxd/platform.py index 6a016929..f7251a07 100644 --- a/tests/cloud_tests/platforms/lxd/platform.py +++ b/tests/cloud_tests/platforms/lxd/platform.py @@ -101,8 +101,4 @@ class LXDPlatform(Platform): """ return self.client.images.get_by_alias(alias) - def destroy(self): - """Clean up platform data.""" - super(LXDPlatform, self).destroy() - # vi: ts=4 expandtab diff --git a/tests/cloud_tests/platforms/nocloudkvm/__init__.py b/tests/cloud_tests/platforms/nocloudkvm/__init__.py new file mode 100644 index 00000000..e69de29b --- /dev/null +++ b/tests/cloud_tests/platforms/nocloudkvm/__init__.py diff --git a/tests/cloud_tests/platforms/nocloudkvm/instance.py b/tests/cloud_tests/platforms/nocloudkvm/instance.py index 932dc0fa..33ff3f24 100644 --- a/tests/cloud_tests/platforms/nocloudkvm/instance.py +++ b/tests/cloud_tests/platforms/nocloudkvm/instance.py @@ -109,7 +109,7 @@ class NoCloudKVMInstance(Instance): if self.pid: try: c_util.subp(['kill', '-9', self.pid]) - except util.ProcessExectuionError: + except c_util.ProcessExecutionError: pass if self.pid_file: diff --git a/tests/cloud_tests/platforms/nocloudkvm/platform.py b/tests/cloud_tests/platforms/nocloudkvm/platform.py index a7e6f5de..85933463 100644 --- a/tests/cloud_tests/platforms/nocloudkvm/platform.py +++ b/tests/cloud_tests/platforms/nocloudkvm/platform.py @@ -21,10 +21,6 @@ class NoCloudKVMPlatform(Platform): platform_name = 'nocloud-kvm' - def __init__(self, config): - """Set up platform.""" - super(NoCloudKVMPlatform, self).__init__(config) - def get_image(self, img_conf): """Get image using specified image configuration. |