mirror of
https://github.com/ansible-collections/community.general.git
synced 2026-04-26 13:42:49 +00:00
Reformat everything.
This commit is contained in:
Felix Fontein
parent
3f2213791a
commit
340ff8586d
1008 changed files with 61301 additions and 58309 deletions
|
|
@ -7,7 +7,11 @@ from __future__ import annotations
|
|||
from unittest.mock import call, patch
|
||||
from ansible_collections.community.general.plugins.modules import rhsm_release
|
||||
from ansible_collections.community.internal_test_tools.tests.unit.plugins.modules.utils import (
|
||||
AnsibleExitJson, AnsibleFailJson, ModuleTestCase, set_module_args)
|
||||
AnsibleExitJson,
|
||||
AnsibleFailJson,
|
||||
ModuleTestCase,
|
||||
set_module_args,
|
||||
)
|
||||
|
||||
|
||||
class RhsmRepositoryReleaseModuleTestCase(ModuleTestCase):
|
||||
|
|
@ -21,19 +25,20 @@ class RhsmRepositoryReleaseModuleTestCase(ModuleTestCase):
|
|||
# Mainly interested that the subscription-manager calls are right
|
||||
# based on the module args, so patch out run_command in the module.
|
||||
# returns (rc, out, err) structure
|
||||
self.mock_run_command = patch('ansible_collections.community.general.plugins.modules.rhsm_release.'
|
||||
'AnsibleModule.run_command')
|
||||
self.mock_run_command = patch(
|
||||
"ansible_collections.community.general.plugins.modules.rhsm_release.AnsibleModule.run_command"
|
||||
)
|
||||
self.module_main_command = self.mock_run_command.start()
|
||||
|
||||
# Module does a get_bin_path check before every run_command call
|
||||
self.mock_get_bin_path = patch('ansible_collections.community.general.plugins.modules.rhsm_release.'
|
||||
'AnsibleModule.get_bin_path')
|
||||
self.mock_get_bin_path = patch(
|
||||
"ansible_collections.community.general.plugins.modules.rhsm_release.AnsibleModule.get_bin_path"
|
||||
)
|
||||
self.get_bin_path = self.mock_get_bin_path.start()
|
||||
self.get_bin_path.return_value = '/testbin/subscription-manager'
|
||||
self.get_bin_path.return_value = "/testbin/subscription-manager"
|
||||
|
||||
# subscription-manager needs to be run as root
|
||||
self.mock_os_getuid = patch('ansible_collections.community.general.plugins.modules.rhsm_release.'
|
||||
'os.getuid')
|
||||
self.mock_os_getuid = patch("ansible_collections.community.general.plugins.modules.rhsm_release.os.getuid")
|
||||
self.os_getuid = self.mock_os_getuid.start()
|
||||
self.os_getuid.return_value = 0
|
||||
|
||||
|
|
@ -51,97 +56,105 @@ class RhsmRepositoryReleaseModuleTestCase(ModuleTestCase):
|
|||
def test_release_set(self):
|
||||
# test that the module attempts to change the release when the current
|
||||
# release is not the same as the user-specific target release
|
||||
with set_module_args({'release': '7.5'}):
|
||||
with set_module_args({"release": "7.5"}):
|
||||
self.module_main_command.side_effect = [
|
||||
# first call, get_release: returns different version so set_release is called
|
||||
(0, '7.4', ''),
|
||||
(0, "7.4", ""),
|
||||
# second call, set_release: just needs to exit with 0 rc
|
||||
(0, '', ''),
|
||||
(0, "", ""),
|
||||
]
|
||||
|
||||
result = self.module_main(AnsibleExitJson)
|
||||
|
||||
self.assertTrue(result['changed'])
|
||||
self.assertEqual('7.5', result['current_release'])
|
||||
self.module_main_command.assert_has_calls([
|
||||
call(['/testbin/subscription-manager', 'release', '--show'], **self.SUBMAN_KWARGS),
|
||||
call(['/testbin/subscription-manager', 'release', '--set', '7.5'], **self.SUBMAN_KWARGS),
|
||||
])
|
||||
self.assertTrue(result["changed"])
|
||||
self.assertEqual("7.5", result["current_release"])
|
||||
self.module_main_command.assert_has_calls(
|
||||
[
|
||||
call(["/testbin/subscription-manager", "release", "--show"], **self.SUBMAN_KWARGS),
|
||||
call(["/testbin/subscription-manager", "release", "--set", "7.5"], **self.SUBMAN_KWARGS),
|
||||
]
|
||||
)
|
||||
|
||||
def test_release_set_idempotent(self):
|
||||
# test that the module does not attempt to change the release when
|
||||
# the current release matches the user-specified target release
|
||||
with set_module_args({'release': '7.5'}):
|
||||
with set_module_args({"release": "7.5"}):
|
||||
self.module_main_command.side_effect = [
|
||||
# first call, get_release: returns same version, set_release is not called
|
||||
(0, '7.5', ''),
|
||||
(0, "7.5", ""),
|
||||
]
|
||||
|
||||
result = self.module_main(AnsibleExitJson)
|
||||
|
||||
self.assertFalse(result['changed'])
|
||||
self.assertEqual('7.5', result['current_release'])
|
||||
self.module_main_command.assert_has_calls([
|
||||
call(['/testbin/subscription-manager', 'release', '--show'], **self.SUBMAN_KWARGS),
|
||||
])
|
||||
self.assertFalse(result["changed"])
|
||||
self.assertEqual("7.5", result["current_release"])
|
||||
self.module_main_command.assert_has_calls(
|
||||
[
|
||||
call(["/testbin/subscription-manager", "release", "--show"], **self.SUBMAN_KWARGS),
|
||||
]
|
||||
)
|
||||
|
||||
def test_release_unset(self):
|
||||
# test that the module attempts to change the release when the current
|
||||
# release is not the same as the user-specific target release
|
||||
with set_module_args({'release': None}):
|
||||
with set_module_args({"release": None}):
|
||||
self.module_main_command.side_effect = [
|
||||
# first call, get_release: returns version so set_release is called
|
||||
(0, '7.5', ''),
|
||||
(0, "7.5", ""),
|
||||
# second call, set_release: just needs to exit with 0 rc
|
||||
(0, '', ''),
|
||||
(0, "", ""),
|
||||
]
|
||||
|
||||
result = self.module_main(AnsibleExitJson)
|
||||
|
||||
self.assertTrue(result['changed'])
|
||||
self.assertIsNone(result['current_release'])
|
||||
self.module_main_command.assert_has_calls([
|
||||
call(['/testbin/subscription-manager', 'release', '--show'], **self.SUBMAN_KWARGS),
|
||||
call(['/testbin/subscription-manager', 'release', '--unset'], **self.SUBMAN_KWARGS),
|
||||
])
|
||||
self.assertTrue(result["changed"])
|
||||
self.assertIsNone(result["current_release"])
|
||||
self.module_main_command.assert_has_calls(
|
||||
[
|
||||
call(["/testbin/subscription-manager", "release", "--show"], **self.SUBMAN_KWARGS),
|
||||
call(["/testbin/subscription-manager", "release", "--unset"], **self.SUBMAN_KWARGS),
|
||||
]
|
||||
)
|
||||
|
||||
def test_release_unset_idempotent(self):
|
||||
# test that the module attempts to change the release when the current
|
||||
# release is not the same as the user-specific target release
|
||||
with set_module_args({'release': None}):
|
||||
with set_module_args({"release": None}):
|
||||
self.module_main_command.side_effect = [
|
||||
# first call, get_release: returns no version, set_release is not called
|
||||
(0, 'Release not set', ''),
|
||||
(0, "Release not set", ""),
|
||||
]
|
||||
|
||||
result = self.module_main(AnsibleExitJson)
|
||||
|
||||
self.assertFalse(result['changed'])
|
||||
self.assertIsNone(result['current_release'])
|
||||
self.module_main_command.assert_has_calls([
|
||||
call(['/testbin/subscription-manager', 'release', '--show'], **self.SUBMAN_KWARGS),
|
||||
])
|
||||
self.assertFalse(result["changed"])
|
||||
self.assertIsNone(result["current_release"])
|
||||
self.module_main_command.assert_has_calls(
|
||||
[
|
||||
call(["/testbin/subscription-manager", "release", "--show"], **self.SUBMAN_KWARGS),
|
||||
]
|
||||
)
|
||||
|
||||
def test_release_insane(self):
|
||||
# test that insane values for release trigger fail_json
|
||||
insane_value = 'this is an insane release value'
|
||||
with set_module_args({'release': insane_value}):
|
||||
insane_value = "this is an insane release value"
|
||||
with set_module_args({"release": insane_value}):
|
||||
result = self.module_main(AnsibleFailJson)
|
||||
|
||||
# also ensure that the fail msg includes the insane value
|
||||
self.assertIn(insane_value, result['msg'])
|
||||
self.assertIn(insane_value, result["msg"])
|
||||
|
||||
def test_release_matcher(self):
|
||||
# throw a few values at the release matcher -- only sane_values should match
|
||||
sane_values = ['1Server', '1Client', '10Server', '1.10', '10.0', '9']
|
||||
sane_values = ["1Server", "1Client", "10Server", "1.10", "10.0", "9"]
|
||||
insane_values = [
|
||||
'6server', # lowercase 's'
|
||||
'100Server', # excessively long 'x' component
|
||||
'100.100', # excessively long 'x' and 'y' components
|
||||
'+.-', # illegal characters
|
||||
"6server", # lowercase 's'
|
||||
"100Server", # excessively long 'x' component
|
||||
"100.100", # excessively long 'x' and 'y' components
|
||||
"+.-", # illegal characters
|
||||
]
|
||||
|
||||
matches = self.module.release_matcher.findall(' '.join(sane_values + insane_values))
|
||||
matches = self.module.release_matcher.findall(" ".join(sane_values + insane_values))
|
||||
|
||||
# matches should be returned in the same order they were parsed,
|
||||
# so sorting shouldn't be necessary here
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue