diff --git a/tests/test_vm_provision_mixin.py b/tests/test_vm_provision_mixin.py index 0e5a487..ed9ad13 100644 --- a/tests/test_vm_provision_mixin.py +++ b/tests/test_vm_provision_mixin.py @@ -459,7 +459,13 @@ def test_destroy_vm_success(): def test_destroy_vm_already_stopped(): - """destroy_vm succeeds even if VM already stopped.""" + """destroy_vm succeeds even if VM already stopped. + + Proxmox refuses to stop a VM that is not running, so the stop call raises. + (A bare MagicMock as the stop task never reports "stopped": _wait_for_task + then spun for the full timeout with sleep patched out, and every recorded + mock call made the test take 60 s and several GB, enough for CI to kill it.) + """ mixin = ProxmoxVMProvisionMixin() mock_node = MagicMock() @@ -468,6 +474,7 @@ def test_destroy_vm_already_stopped(): # Mock VM operations mock_vm = MagicMock() mock_node.qemu.return_value = mock_vm + mock_vm.status.stop.post.side_effect = Exception("VM 101 not running") mock_vm.config.get.return_value = {} mock_vm.delete.return_value = None