summaryrefslogtreecommitdiff
path: root/tests/cloud_tests
diff options
context:
space:
mode:
authorScott Moser <smoser@ubuntu.com>2018-03-15 11:45:12 -0400
committerScott Moser <smoser@brickies.net>2018-03-15 11:45:12 -0400
commitbef2f2c945fdee4a1141c3177b3e48b1537027e4 (patch)
tree2b85e97eff63df46075eab57dadbedbcc23de410 /tests/cloud_tests
parent133ad2cb327ad17b7b81319fac8f9f14577c04df (diff)
downloadvyos-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/cloud_tests')
-rw-r--r--tests/cloud_tests/platforms/ec2/__init__.py0
-rw-r--r--tests/cloud_tests/platforms/lxd/__init__.py0
-rw-r--r--tests/cloud_tests/platforms/lxd/platform.py4
-rw-r--r--tests/cloud_tests/platforms/nocloudkvm/__init__.py0
-rw-r--r--tests/cloud_tests/platforms/nocloudkvm/instance.py2
-rw-r--r--tests/cloud_tests/platforms/nocloudkvm/platform.py4
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.