From 44936096ce25f188edc69cbc39d8e2f47015262c Mon Sep 17 00:00:00 2001 From: Javier Cano Cano Date: Mon, 22 Apr 2024 11:18:57 +0200 Subject: [PATCH] test: add kubevirt_vm delete test case Adds a test to assert deletion options are the expected. Refactors python fixtures to reuse common fields in both create and delete tests. Signed-off-by: Javier Cano Cano --- .../unit/plugins/modules/test_kubevirt_vm.py | 25 +++++++++++++++++++ 1 file changed, 25 insertions(+) diff --git a/tests/unit/plugins/modules/test_kubevirt_vm.py b/tests/unit/plugins/modules/test_kubevirt_vm.py index c1d758e..361f04b 100644 --- a/tests/unit/plugins/modules/test_kubevirt_vm.py +++ b/tests/unit/plugins/modules/test_kubevirt_vm.py @@ -190,6 +190,16 @@ def module_params_stopped(module_params_default): } +@pytest.fixture(scope="module") +def module_params_delete(module_params_default): + return module_params_default | { + "name": "testvm", + "namespace": "default", + "state": "absent", + "wait": True, + } + + @pytest.fixture(scope="module") def k8s_module_params_create(module_params_create, vm_definition_create): return module_params_create | { @@ -217,6 +227,15 @@ def k8s_module_params_stopped(module_params_stopped, vm_definition_stopped): } +@pytest.fixture(scope="module") +def k8s_module_params_delete(module_params_delete, vm_definition_running): + return module_params_delete | { + "generate_name": None, + "resource_definition": dump(vm_definition_running, sort_keys=False), + "wait_condition": {"type": "Ready", "status": True}, + } + + def test_module_fails_when_required_args_missing(monkeypatch): monkeypatch.setattr(AnsibleModule, "fail_json", fail_json) with pytest.raises(AnsibleFailJson): @@ -245,6 +264,12 @@ def test_module_fails_when_required_args_missing(monkeypatch): "vm_definition_stopped", "update", ), + ( + "module_params_delete", + "k8s_module_params_delete", + "vm_definition_running", + "delete" + ), ], ) def test_module(