diff options
author | Daniel Watkins <oddbloke@ubuntu.com> | 2020-07-15 10:26:12 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-07-15 10:26:12 -0400 |
commit | 4fe576516d65feda17ba78e9265a8e494a195e7b (patch) | |
tree | a4cdbde78f035bfca999c22ce5c1443ecb20c5ae /tests/cloud_tests | |
parent | 25289087e44c9c74543248519e37ca1f11b8a711 (diff) | |
download | vyos-cloud-init-4fe576516d65feda17ba78e9265a8e494a195e7b.tar.gz vyos-cloud-init-4fe576516d65feda17ba78e9265a8e494a195e7b.zip |
cloudinit: remove global disable of pylint W0107 and fix errors (#489)
* cloudinit: remove global disable of pylint W0107 and fix errors
This includes removing a test class which contained no tests but wasn't
detected as empty because of an errant pass statement.
* .pylintrc: update disable comment to match arguments
Diffstat (limited to 'tests/cloud_tests')
-rw-r--r-- | tests/cloud_tests/platforms/azurecloud/instance.py | 2 | ||||
-rw-r--r-- | tests/cloud_tests/platforms/images.py | 1 | ||||
-rw-r--r-- | tests/cloud_tests/platforms/snapshots.py | 1 | ||||
-rw-r--r-- | tests/cloud_tests/testcases/base.py | 1 |
4 files changed, 0 insertions, 5 deletions
diff --git a/tests/cloud_tests/platforms/azurecloud/instance.py b/tests/cloud_tests/platforms/azurecloud/instance.py index f1e28a96..a136cf0d 100644 --- a/tests/cloud_tests/platforms/azurecloud/instance.py +++ b/tests/cloud_tests/platforms/azurecloud/instance.py @@ -80,7 +80,6 @@ class AzureCloudInstance(Instance): except CloudError: LOG.debug(('image not found, launching instance with base image, ' 'image_id=%s'), self.image_id) - pass vm_params = { 'name': self.vm_name, @@ -169,7 +168,6 @@ class AzureCloudInstance(Instance): sleep(15) else: LOG.warning('Could not find console log: %s', e) - pass LOG.debug('stopping instance %s', self.image_id) vm_deallocate = \ diff --git a/tests/cloud_tests/platforms/images.py b/tests/cloud_tests/platforms/images.py index 557a5cf6..f047de2e 100644 --- a/tests/cloud_tests/platforms/images.py +++ b/tests/cloud_tests/platforms/images.py @@ -52,6 +52,5 @@ class Image(TargetBase): def destroy(self): """Clean up data associated with image.""" - pass # vi: ts=4 expandtab diff --git a/tests/cloud_tests/platforms/snapshots.py b/tests/cloud_tests/platforms/snapshots.py index 94328982..0f5f8bb6 100644 --- a/tests/cloud_tests/platforms/snapshots.py +++ b/tests/cloud_tests/platforms/snapshots.py @@ -40,6 +40,5 @@ class Snapshot(object): def destroy(self): """Clean up snapshot data.""" - pass # vi: ts=4 expandtab diff --git a/tests/cloud_tests/testcases/base.py b/tests/cloud_tests/testcases/base.py index 2e7c6686..4448e0b5 100644 --- a/tests/cloud_tests/testcases/base.py +++ b/tests/cloud_tests/testcases/base.py @@ -34,7 +34,6 @@ class CloudTestCase(unittest.TestCase): @classmethod def maybeSkipTest(cls): """Present to allow subclasses to override and raise a skipTest.""" - pass def assertPackageInstalled(self, name, version=None): """Check dpkg-query --show output for matching package name. |