summaryrefslogtreecommitdiff
path: root/tests/unittests/test_handler/test_handler_install_hotplug.py
diff options
context:
space:
mode:
authorBrett Holman <bholman.devel@gmail.com>2021-12-03 13:11:46 -0700
committerGitHub <noreply@github.com>2021-12-03 13:11:46 -0700
commit039c40f9b3d88ee8158604bb18ca4bf2fb5d5e51 (patch)
tree5f1b09486ccaf98ee8159de58d9a2a1ef0af5dc1 /tests/unittests/test_handler/test_handler_install_hotplug.py
parentffa6fc88249aa080aa31811a45569a45e567418a (diff)
downloadvyos-cloud-init-039c40f9b3d88ee8158604bb18ca4bf2fb5d5e51.tar.gz
vyos-cloud-init-039c40f9b3d88ee8158604bb18ca4bf2fb5d5e51.zip
Reorganize unit test locations under tests/unittests (#1126)
This attempts to standardize unit test file location under test/unittests/ such that any source file located at cloudinit/path/to/file.py may have a corresponding unit test file at test/unittests/path/to/test_file.py. Noteworthy Comments: ==================== Four different duplicate test files existed: test_{gpg,util,cc_mounts,cc_resolv_conf}.py Each of these duplicate file pairs has been merged together. This is a break in git history for these files. The test suite appears to have a dependency on test order. Changing test order causes some tests to fail. This should be rectified, but for now some tests have been modified in tests/unittests/config/test_set_passwords.py. A helper class name starts with "Test" which causes pytest to try executing it as a test case, which then throws warnings "due to Class having __init__()". Silence by changing the name of the class. # helpers.py is imported in many test files, import paths change cloudinit/tests/helpers.py -> tests/unittests/helpers.py # Move directories: cloudinit/distros/tests -> tests/unittests/distros cloudinit/cmd/devel/tests -> tests/unittests/cmd/devel cloudinit/cmd/tests -> tests/unittests/cmd/ cloudinit/sources/helpers/tests -> tests/unittests/sources/helpers cloudinit/sources/tests -> tests/unittests/sources cloudinit/net/tests -> tests/unittests/net cloudinit/config/tests -> tests/unittests/config cloudinit/analyze/tests/ -> tests/unittests/analyze/ # Standardize tests already in tests/unittests/ test_datasource -> sources test_distros -> distros test_vmware -> sources/vmware test_handler -> config # this contains cloudconfig module tests test_runs -> runs
Diffstat (limited to 'tests/unittests/test_handler/test_handler_install_hotplug.py')
-rw-r--r--tests/unittests/test_handler/test_handler_install_hotplug.py113
1 files changed, 0 insertions, 113 deletions
diff --git a/tests/unittests/test_handler/test_handler_install_hotplug.py b/tests/unittests/test_handler/test_handler_install_hotplug.py
deleted file mode 100644
index 5d6b1e77..00000000
--- a/tests/unittests/test_handler/test_handler_install_hotplug.py
+++ /dev/null
@@ -1,113 +0,0 @@
-# This file is part of cloud-init. See LICENSE file for license information.
-from collections import namedtuple
-from unittest import mock
-
-import pytest
-
-from cloudinit.config.cc_install_hotplug import (
- handle,
- HOTPLUG_UDEV_PATH,
- HOTPLUG_UDEV_RULES_TEMPLATE,
-)
-from cloudinit.event import EventScope, EventType
-
-
-@pytest.yield_fixture()
-def mocks():
- m_update_enabled = mock.patch('cloudinit.stages.update_event_enabled')
- m_write = mock.patch('cloudinit.util.write_file', autospec=True)
- m_del = mock.patch('cloudinit.util.del_file', autospec=True)
- m_subp = mock.patch('cloudinit.subp.subp')
- m_which = mock.patch('cloudinit.subp.which', return_value=None)
- m_path_exists = mock.patch('os.path.exists', return_value=False)
-
- yield namedtuple(
- 'Mocks',
- 'm_update_enabled m_write m_del m_subp m_which m_path_exists'
- )(
- m_update_enabled.start(), m_write.start(), m_del.start(),
- m_subp.start(), m_which.start(), m_path_exists.start()
- )
-
- m_update_enabled.stop()
- m_write.stop()
- m_del.stop()
- m_subp.stop()
- m_which.stop()
- m_path_exists.stop()
-
-
-class TestInstallHotplug:
- @pytest.mark.parametrize('libexec_exists', [True, False])
- def test_rules_installed_when_supported_and_enabled(
- self, mocks, libexec_exists
- ):
- mocks.m_which.return_value = 'udevadm'
- mocks.m_update_enabled.return_value = True
- m_cloud = mock.MagicMock()
- m_cloud.datasource.get_supported_events.return_value = {
- EventScope.NETWORK: {EventType.HOTPLUG}
- }
-
- if libexec_exists:
- libexecdir = "/usr/libexec/cloud-init"
- else:
- libexecdir = "/usr/lib/cloud-init"
- with mock.patch('os.path.exists', return_value=libexec_exists):
- handle(None, {}, m_cloud, mock.Mock(), None)
- mocks.m_write.assert_called_once_with(
- filename=HOTPLUG_UDEV_PATH,
- content=HOTPLUG_UDEV_RULES_TEMPLATE.format(
- libexecdir=libexecdir),
- )
- assert mocks.m_subp.call_args_list == [mock.call([
- 'udevadm', 'control', '--reload-rules',
- ])]
- assert mocks.m_del.call_args_list == []
-
- def test_rules_not_installed_when_unsupported(self, mocks):
- mocks.m_update_enabled.return_value = True
- m_cloud = mock.MagicMock()
- m_cloud.datasource.get_supported_events.return_value = {}
-
- handle(None, {}, m_cloud, mock.Mock(), None)
- assert mocks.m_write.call_args_list == []
- assert mocks.m_del.call_args_list == []
- assert mocks.m_subp.call_args_list == []
-
- def test_rules_not_installed_when_disabled(self, mocks):
- mocks.m_update_enabled.return_value = False
- m_cloud = mock.MagicMock()
- m_cloud.datasource.get_supported_events.return_value = {
- EventScope.NETWORK: {EventType.HOTPLUG}
- }
-
- handle(None, {}, m_cloud, mock.Mock(), None)
- assert mocks.m_write.call_args_list == []
- assert mocks.m_del.call_args_list == []
- assert mocks.m_subp.call_args_list == []
-
- def test_rules_uninstalled_when_disabled(self, mocks):
- mocks.m_path_exists.return_value = True
- mocks.m_update_enabled.return_value = False
- m_cloud = mock.MagicMock()
- m_cloud.datasource.get_supported_events.return_value = {}
-
- handle(None, {}, m_cloud, mock.Mock(), None)
- mocks.m_del.assert_called_with(HOTPLUG_UDEV_PATH)
- assert mocks.m_subp.call_args_list == [mock.call([
- 'udevadm', 'control', '--reload-rules',
- ])]
- assert mocks.m_write.call_args_list == []
-
- def test_rules_not_installed_when_no_udevadm(self, mocks):
- mocks.m_update_enabled.return_value = True
- m_cloud = mock.MagicMock()
- m_cloud.datasource.get_supported_events.return_value = {
- EventScope.NETWORK: {EventType.HOTPLUG}
- }
-
- handle(None, {}, m_cloud, mock.Mock(), None)
- assert mocks.m_del.call_args_list == []
- assert mocks.m_write.call_args_list == []
- assert mocks.m_subp.call_args_list == []