mirror of
https://github.com/fog/fog.git
synced 2022-11-09 13:51:43 -05:00
a65db7f594
Update `Fog::Compute::Vsphere` tests to reflect the way the `Fog::Compute::Vsphere#convert_vm_mob_ref_to_attr_hash` method currently converts a `RbVmomi::VIM::ManagedObject` object to a hash. Without this patch, tests related to converting an instance of `RbVmomi::VIM::ManagedObject` to a hash will raise an exception: NoMethodError: undefined method `collect!' for Hash; causing the test to fail. This patch solves the problem by mocking `RbVmomi::VIM::ManagedObject` with a new `MockManagedObject` class, which provides a `collect!` method, and the `_ref` and `parent` attributes. This patch renames fake_vm to fake_vm_mob_ref in order to provide a more descriptive name for what's actually being tested. The `Fog::Compute::Vshpere::Mock` class has been updated to require the rbvmomi library, which prevents an `NameError` exception from being raised due to the `Fog::Compute::Vsphere::Shared::RbVmomi` constant not being initialized. The `Fog::Compute::Vshpere::Mock` class has been updated with a `get_folder_path` method, which prevents a `NoMethodError` exception from being raised due to the `get_folder_path` method being undefined.
55 lines
1.6 KiB
Ruby
55 lines
1.6 KiB
Ruby
Shindo.tests('Fog::Compute[:vsphere]', ['vsphere']) do
|
|
|
|
compute = Fog::Compute[:vsphere]
|
|
|
|
tests("| convert_vm_mob_ref_to_attr_hash") do
|
|
# Mock the RbVmomi::VIM::ManagedObject class
|
|
class MockManagedObject
|
|
|
|
attr_reader :parent, :_ref
|
|
|
|
def initialize
|
|
@parent = @_ref = 'vm-123'
|
|
end
|
|
|
|
def collect! *pathSet
|
|
{ '_ref' => 'vm-123', 'name' => 'fakevm' }
|
|
end
|
|
end
|
|
|
|
fake_vm_mob_ref = MockManagedObject.new
|
|
|
|
tests("When converting an incomplete vm object") do
|
|
test("it should return a Hash") do
|
|
compute.convert_vm_mob_ref_to_attr_hash(fake_vm_mob_ref).kind_of? Hash
|
|
end
|
|
tests("The converted Hash should") do
|
|
attr_hash = compute.convert_vm_mob_ref_to_attr_hash(fake_vm_mob_ref)
|
|
test("have a name") { attr_hash['name'] == 'fakevm' }
|
|
test("have a mo_ref") {attr_hash['mo_ref'] == 'vm-123' }
|
|
test("have an id") { attr_hash['id'] == 'vm-123' }
|
|
test("not have a instance_uuid") { attr_hash['instance_uuid'].nil? }
|
|
end
|
|
end
|
|
|
|
tests("When passed a nil object") do
|
|
attr_hash = compute.convert_vm_mob_ref_to_attr_hash(nil)
|
|
test("it should return a nil object") do
|
|
attr_hash.nil?
|
|
end
|
|
end
|
|
end
|
|
|
|
tests("Compute attributes") do
|
|
%w{ vsphere_is_vcenter vsphere_rev vsphere_username vsphere_server }.each do |attr|
|
|
test("it should respond to #{attr}") { compute.respond_to? attr }
|
|
end
|
|
end
|
|
|
|
tests("Compute collections") do
|
|
%w{ servers }.each do |collection|
|
|
test("it should respond to #{collection}") { compute.respond_to? collection }
|
|
end
|
|
end
|
|
end
|
|
|