summaryrefslogtreecommitdiff
path: root/tests/unittests
diff options
context:
space:
mode:
authorDaniel Watkins <oddbloke@ubuntu.com>2020-03-19 12:01:15 -0400
committerGitHub <noreply@github.com>2020-03-19 12:01:15 -0400
commitade47866aa2f81ab0f3baabbb1fc1e484abdb741 (patch)
tree0c28ae27861335a73b035ad68d506cc4ab3bcdb5 /tests/unittests
parent9db87379b5c04f4c9662eaf81390e702ea63b8b3 (diff)
downloadvyos-cloud-init-ade47866aa2f81ab0f3baabbb1fc1e484abdb741.tar.gz
vyos-cloud-init-ade47866aa2f81ab0f3baabbb1fc1e484abdb741.zip
cloudinit/tests: remove unneeded with_logs configuration (#263)
These classes don't use `self.logs` anywhere in their body, so we can remove the `with_logs = True` setting from them. These instances were found using astpath[0], with the following invocation: astpath "//Name[@id='with_logs' and not(ancestor::ClassDef//Attribute[@attr='logs'])]" [0] https://github.com/hchasestevens/astpath
Diffstat (limited to 'tests/unittests')
-rw-r--r--tests/unittests/test_datasource/test_azure.py2
-rw-r--r--tests/unittests/test_datasource/test_maas.py1
-rw-r--r--tests/unittests/test_handler/test_handler_locale.py2
-rw-r--r--tests/unittests/test_handler/test_handler_puppet.py2
-rw-r--r--tests/unittests/test_util.py1
5 files changed, 0 insertions, 8 deletions
diff --git a/tests/unittests/test_datasource/test_azure.py b/tests/unittests/test_datasource/test_azure.py
index a809fd87..2c6aa44d 100644
--- a/tests/unittests/test_datasource/test_azure.py
+++ b/tests/unittests/test_datasource/test_azure.py
@@ -383,8 +383,6 @@ class TestGetMetadataFromIMDS(HttprettyTestCase):
class TestAzureDataSource(CiTestCase):
- with_logs = True
-
def setUp(self):
super(TestAzureDataSource, self).setUp()
self.tmp = self.tmp_dir()
diff --git a/tests/unittests/test_datasource/test_maas.py b/tests/unittests/test_datasource/test_maas.py
index 2a81d3f5..41b6c27b 100644
--- a/tests/unittests/test_datasource/test_maas.py
+++ b/tests/unittests/test_datasource/test_maas.py
@@ -158,7 +158,6 @@ class TestMAASDataSource(CiTestCase):
@mock.patch("cloudinit.sources.DataSourceMAAS.url_helper.OauthUrlHelper")
class TestGetOauthHelper(CiTestCase):
- with_logs = True
base_cfg = {'consumer_key': 'FAKE_CONSUMER_KEY',
'token_key': 'FAKE_TOKEN_KEY',
'token_secret': 'FAKE_TOKEN_SECRET',
diff --git a/tests/unittests/test_handler/test_handler_locale.py b/tests/unittests/test_handler/test_handler_locale.py
index 2b22559f..407aa6c4 100644
--- a/tests/unittests/test_handler/test_handler_locale.py
+++ b/tests/unittests/test_handler/test_handler_locale.py
@@ -29,8 +29,6 @@ LOG = logging.getLogger(__name__)
class TestLocale(t_help.FilesystemMockingTestCase):
- with_logs = True
-
def setUp(self):
super(TestLocale, self).setUp()
self.new_root = tempfile.mkdtemp()
diff --git a/tests/unittests/test_handler/test_handler_puppet.py b/tests/unittests/test_handler/test_handler_puppet.py
index 1494177d..04aa7d03 100644
--- a/tests/unittests/test_handler/test_handler_puppet.py
+++ b/tests/unittests/test_handler/test_handler_puppet.py
@@ -16,8 +16,6 @@ LOG = logging.getLogger(__name__)
@mock.patch('cloudinit.config.cc_puppet.os')
class TestAutostartPuppet(CiTestCase):
- with_logs = True
-
def test_wb_autostart_puppet_updates_puppet_default(self, m_os, m_util):
"""Update /etc/default/puppet to autostart if it exists."""
diff --git a/tests/unittests/test_util.py b/tests/unittests/test_util.py
index 9ff17f52..5b2eaa69 100644
--- a/tests/unittests/test_util.py
+++ b/tests/unittests/test_util.py
@@ -737,7 +737,6 @@ class TestReadSeeded(helpers.TestCase):
class TestSubp(helpers.CiTestCase):
- with_logs = True
allowed_subp = [BASH, 'cat', helpers.CiTestCase.SUBP_SHELL_TRUE,
BOGUS_COMMAND, sys.executable]