Revision cd8f54c0
Added by Ivan Necas almost 7 years ago
| test/models/compute_resources/ovirt_test.rb | ||
|---|---|---|
|
assert_equal host, as_admin { cr.associated_host(vm) }
|
||
|
end
|
||
|
|
||
|
describe "destroy_vm" do
|
||
|
it "handles situation when vm is not present" do
|
||
|
cr = mock_cr_servers(Foreman::Model::Ovirt.new, empty_servers)
|
||
|
cr.expects(:find_vm_by_uuid).raises(ActiveRecord::RecordNotFound)
|
||
|
assert cr.destroy_vm('abc')
|
||
|
end
|
||
|
end
|
||
|
|
||
|
describe "find_vm_by_uuid" do
|
||
|
it "raises RecordNotFound when the vm does not exist" do
|
||
|
cr = mock_cr_servers(Foreman::Model::Ovirt.new, empty_servers)
|
||
Also available in: Unified diff
Fixes #19236 - handle situation when host is destroyed oVirt
The issue was introduced in 4269abbd, where we replaced the
`OVIRT::OvirtException` error by `ActiveRecord::RecordNotFound` in
`find_vm_by_uuid method`, but we haven't updated the corresponding
`rescue` block in `destroy_vm`.
(cherry picked from commit 736cb75f3d41d6a4d847556497574957342ca816)