deltacloud-core 1.0.1 → 1.0.2

data/tests/drivers/mock/storage_volumes_test.rb

@@ -1,122 +1,59 @@
-$:.unshift File.join(File.dirname(__FILE__), '..', '..', '..')
-require 'tests/drivers/mock/common'
+require 'minitest/autorun'
 
-describe 'Deltacloud API storage_volumes' do
-  include Deltacloud::Test
+require_relative File.join('..', '..', '..', 'lib', 'deltacloud', 'api.rb')
 
-  it 'must advertise have the storage_volumes collection in API entrypoint' do
-    get root_url
-    (xml_response/'api/link[@rel=storage_volumes]').wont_be_empty
-  end
-
-  it 'must require authentication to access the "storage_volume" collection' do
-    get collection_url(:storage_volumes)
-    last_response.status.must_equal 401
-  end
-
-  it 'should respond with HTTP_OK when accessing the :storage_volumes collection with authentication' do
-    authenticate
-    get collection_url(:storage_volumes)
-    last_response.status.must_equal 200
-  end
-
-  it 'should support the JSON media type' do
-    authenticate
-    header 'Accept', 'application/json'
-    get collection_url(:storage_volumes)
-    last_response.status.must_equal 200
-    last_response.headers['Content-Type'].must_equal 'application/json'
-  end
-
-  it 'must include the ETag in HTTP headers' do
-    authenticate
-    get collection_url(:storage_volumes)
-    last_response.headers['ETag'].wont_be_nil
-  end
-
-  it 'must have the "storage_volumes" element on top level' do
-    authenticate
-    get collection_url(:storage_volumes)
-    xml_response.root.name.must_equal 'storage_volumes'
-  end
-
-  it 'must have some "storage_volume" elements inside "storage_volumes"' do
-    authenticate
-    get collection_url(:storage_volumes)
-    (xml_response/'storage_volumes/storage_volume').wont_be_empty
-  end
-
-  it 'must provide the :id attribute for each storage_volume in collection' do
-    authenticate
-    get collection_url(:storage_volumes)
-    (xml_response/'storage_volumes/storage_volume').each do |r|
-      r[:id].wont_be_nil
-    end
-  end
-
-  it 'must include the :href attribute for each "storage_volume" element in collection' do
-    authenticate
-    get collection_url(:storage_volumes)
-    (xml_response/'storage_volumes/storage_volume').each do |r|
-      r[:href].wont_be_nil
-    end
-  end
+describe 'MockDriver StorageVolumes' do
 
-  it 'must use the absolute URL in each :href attribute' do
-    authenticate
-    get collection_url(:storage_volumes)
-    (xml_response/'storage_volumes/storage_volume').each do |r|
-      r[:href].must_match /^http/
-    end
+  before do
+    @driver = Deltacloud::new(:mock, :user => 'mockuser', :password => 'mockpassword')
   end
 
-  it 'must have the URL ending with the :id of the storage_volume' do
-    authenticate
-    get collection_url(:storage_volumes)
-    (xml_response/'storage_volumes/storage_volume').each do |r|
-      r[:href].must_match /#{r[:id]}$/
-    end
+  it 'must throw error when wrong credentials' do
+    Proc.new do
+      @driver.backend.storage_volumes(OpenStruct.new(:user => 'unknown', :password => 'wrong'))
+    end.must_raise Deltacloud::ExceptionHandler::AuthenticationFailure, 'Authentication Failure'
   end
 
-  it 'must return the list of valid parameters for the :index action' do
-    authenticate
-    options collection_url(:storage_volumes) + '/index'
-    last_response.headers['Allow'].wont_be_nil
+  it 'must return list of storage_volumes' do
+    @driver.storage_volumes.wont_be_empty
+    @driver.storage_volumes.first.must_be_kind_of StorageVolume
   end
 
-  it 'must have the "name" element defined for each storage_volume in collection' do
-    authenticate
-    get collection_url(:storage_volumes)
-    (xml_response/'storage_volumes/storage_volume').each do |r|
-      (r/'name').wont_be_empty
-    end
+  it 'must allow to filter storage_volumes' do
+    @driver.storage_volumes(:id => 'vol1').wont_be_empty
+    @driver.storage_volumes(:id => 'vol1').must_be_kind_of Array
+    @driver.storage_volumes(:id => 'vol1').size.must_equal 1
+    @driver.storage_volumes(:id => 'vol1').first.id.must_equal 'vol1'
+    @driver.storage_volumes(:id => 'unknown').must_be_empty
   end
 
-  it 'must have the "state" element defined for each storage_volume in collection' do
-    authenticate
-    get collection_url(:storage_volumes)
-    (xml_response/'storage_volumes/storage_volume').each do |r|
-      (r/'state').wont_be_empty
-    end
+  it 'must allow to retrieve single storage_volume' do
+    @driver.storage_volume(:id => 'vol1').wont_be_nil
+    @driver.storage_volume(:id => 'vol1').must_be_kind_of StorageVolume
+    @driver.storage_volume(:id => 'vol1').id.must_equal 'vol1'
+    @driver.storage_volume(:id => 'unknown').must_be_nil
   end
 
-  it 'must return the full "storage_volume" when following the URL in storage_volume element' do
-    authenticate
-    get collection_url(:storage_volumes)
-    (xml_response/'storage_volumes/storage_volume').each do |r|
-      get collection_url(:storage_volumes) + '/' + r[:id]
-      last_response.status.must_equal 200
-    end
+  it 'must allow to create and destroy the storage volume' do
+    volume = @driver.create_storage_volume(:name => 'Test Volume', :capacity => '100')
+    volume.must_be_kind_of StorageVolume
+    volume.name.must_equal 'Test Volume'
+    volume.capacity.must_equal '100'
+    @driver.destroy_storage_volume(:id => volume.id)
+    @driver.storage_volume(:id => volume.id).must_be_nil
   end
 
-  it 'must have the "name" element for the storage_volume and it should match with the one in collection' do
-    authenticate
-    get collection_url(:storage_volumes)
-    (xml_response/'storage_volumes/storage_volume').each do |r|
-      get collection_url(:storage_volumes) + '/' + r[:id]
-      (xml_response/'name').wont_be_empty
-      (xml_response/'name').first.text.must_equal((r/'name').first.text)
-    end
+  it 'must allow to attach and detach storage volume to instance' do
+    volume = @driver.create_storage_volume(:name => 'Test Volume', :capacity => '100')
+    volume.must_be_kind_of StorageVolume
+    @driver.attach_storage_volume(:id => volume.id, :device => '/dev/sda', :instance_id => 'inst1')
+    @driver.storage_volume(:id => volume.id).instance_id.must_equal 'inst1'
+    @driver.storage_volume(:id => volume.id).device.must_equal '/dev/sda'
+    @driver.detach_storage_volume(:id => volume.id, :instance_id => 'inst1')
+    @driver.storage_volume(:id => volume.id).instance_id.must_be_nil
+    @driver.storage_volume(:id => volume.id).device.must_be_nil
+    @driver.destroy_storage_volume(:id => volume.id)
+    @driver.storage_volume(:id => volume.id).must_be_nil
   end
 
 end

data/tests/drivers/models/address_test.rb

@@ -0,0 +1,18 @@
+require 'minitest/autorun'
+
+require_relative File.join('..', '..', '..', 'lib', 'deltacloud', 'models', 'base_model.rb')
+require_relative File.join('..', '..', '..', 'lib', 'deltacloud', 'models', 'address.rb')
+
+describe Address do
+
+  before do
+    @address = Address.new(:id => 'adr1', :instance_id => 'inst1')
+  end
+
+  it 'should tell if it is associated to instance' do
+    @address.associated?.must_equal true
+    @address.instance_id = nil
+    @address.associated?.must_equal false
+  end
+
+end

data/tests/drivers/models/base_test.rb

@@ -0,0 +1,33 @@
+require 'minitest/autorun'
+require_relative File.join('..', '..', '..', 'lib', 'deltacloud', 'models', 'base_model.rb')
+
+describe BaseModel do
+
+  before do
+    class CustomModel < BaseModel
+      attr_accessor :name
+      attr_accessor :custom
+    end
+    @model = CustomModel.new(:id => 'm1', :name => 'Model1', :custom => '1')
+  end
+
+  describe 'initialize' do
+
+    it 'should properly advertise given attributes' do
+      @model.must_respond_to :id
+      @model.must_respond_to :name
+      @model.must_respond_to :custom
+      @model.id.must_equal 'm1'
+      @model.name.must_equal 'Model1'
+      @model.custom.must_equal '1'
+    end
+
+    it 'should report all attributes' do
+      @model.attributes.must_include :id
+      @model.attributes.must_include :name
+      @model.attributes.must_include :custom
+    end
+
+  end
+
+end

data/tests/drivers/models/blob_test.rb

@@ -0,0 +1,18 @@
+require 'minitest/autorun'
+
+require_relative File.join('..', '..', '..', 'lib', 'deltacloud', 'models', 'base_model.rb')
+require_relative File.join('..', '..', '..', 'lib', 'deltacloud', 'models', 'address.rb')
+
+describe Address do
+
+  before do
+    @address = Address.new(:id => 'adr1', :instance_id => 'inst1')
+  end
+
+  it 'should tell if it is associated to instance' do
+    @address.associated?.must_equal true
+    @address.instance_id = nil
+    @address.associated?.must_equal false
+  end
+
+end

data/tests/drivers/models/hardware_profile_test.rb

@@ -0,0 +1,52 @@
+require 'minitest/autorun'
+require 'sinatra/base'
+require 'sinatra/rabbit'
+
+require_relative File.join('..', '..', '..', 'lib', 'deltacloud', 'models', 'base_model.rb')
+require_relative File.join('..', '..', '..', 'lib', 'deltacloud', 'models', 'hardware_profile.rb')
+
+describe Deltacloud::HardwareProfile do
+
+  before do
+    @profile1 = Deltacloud::HardwareProfile.new('p1') do |hwp|
+      cpu           1
+      memory        512
+      storage       100
+      architecture 'i386'
+    end
+    @profile2 = Deltacloud::HardwareProfile.new('p2') do |hwp|
+      cpu           1..10
+      memory        [512, 1024]
+    end
+  end
+
+  it 'should return proper unit for property' do
+    Deltacloud::HardwareProfile.unit(:cpu).must_equal 'count'
+    Deltacloud::HardwareProfile.unit(:storage).must_equal 'GB'
+    Deltacloud::HardwareProfile.unit(:memory).must_equal 'MB'
+    Deltacloud::HardwareProfile.unit(:architecture).must_equal 'label'
+  end
+
+  it 'should properly advertise the properties' do
+    @profile1.properties.wont_be_empty
+    @profile1.property(:cpu).must_be_kind_of Deltacloud::HardwareProfile::Property
+    @profile1.property(:cpu).name.must_equal :cpu
+  end
+
+  it 'should return the default value for given property' do
+    @profile1.default?(:cpu, '1').must_equal true
+    @profile1.default?(:cpu, '666').must_equal false
+  end
+
+  it 'should return if given value is within property range' do
+    @profile2.include?(:cpu, 5).must_equal true
+    @profile2.include?(:cpu, 100).must_equal false
+    @profile2.include?(:memory, 10).must_equal false
+    @profile2.include?(:memory, 1024).must_equal true
+  end
+
+  it 'should return query params' do
+    @profile1.params.wont_be_empty
+  end
+
+end

data/tests/drivers/models/instance_address_test.rb

@@ -0,0 +1,24 @@
+require 'minitest/autorun'
+
+require_relative File.join('..', '..', '..', 'lib', 'deltacloud', 'models', 'base_model.rb')
+require_relative File.join('..', '..', '..', 'lib', 'deltacloud', 'models', 'instance_address.rb')
+
+describe InstanceAddress do
+
+  before do
+    @address = InstanceAddress.new('192.168.0.1')
+  end
+
+  it 'should properly report address type' do
+    @address.address_type.must_equal :ipv4
+    @address.address.must_equal '192.168.0.1'
+    @address.is_ipv4?.must_equal true
+    InstanceAddress.new('01:23:45:67:89:ab', :type => :mac).address_type.must_equal :mac
+    InstanceAddress.new('01:23:45:67:89:ab', :type => :mac).is_mac?.must_equal true
+    InstanceAddress.new('test.local', :type => :hostname).is_hostname?.must_equal true
+    InstanceAddress.new('test.local', :port => '5000', :type => :vnc).is_vnc?.must_equal true
+    InstanceAddress.new('test.local', :port => '5000', :type => :vnc).port.must_equal '5000'
+    InstanceAddress.new('test.local', :port => '5000', :type => :vnc).to_s.must_equal 'VNC:test.local:5000'
+  end
+
+end

data/tests/drivers/models/instance_profile_test.rb

@@ -0,0 +1,21 @@
+require 'minitest/autorun'
+
+require_relative File.join('..', '..', '..', 'lib', 'deltacloud', 'models', 'base_model.rb')
+require_relative File.join('..', '..', '..', 'lib', 'deltacloud', 'models', 'instance_profile.rb')
+
+describe InstanceProfile do
+
+  before do
+    @instance = InstanceProfile.new(
+      'm1-small',
+      :hwp_memory => '512',
+      :hwp_cpu => '1'
+    )
+  end
+
+  it 'advertise the overrides' do
+    @instance.overrides.keys.must_include :cpu
+    @instance.overrides.keys.must_include :memory
+  end
+
+end

data/tests/drivers/models/instance_test.rb

@@ -0,0 +1,29 @@
+require 'minitest/autorun'
+
+require_relative File.join('..', '..', '..', 'lib', 'deltacloud', 'models', 'base_model.rb')
+require_relative File.join('..', '..', '..', 'lib', 'deltacloud', 'models', 'instance.rb')
+
+describe Instance do
+
+  before do
+    @instance = Instance.new(
+      :id => 'inst1',
+      :create_image => true,
+      :name => 'Instance',
+      :instance_profile => 'm1-small',
+      :state => 'RUNNING'
+    )
+  end
+
+  it 'advertise if can be used to create image' do
+    @instance.can_create_image?.must_equal true
+    @instance.create_image = false
+    @instance.can_create_image?.must_equal false
+  end
+
+  it 'advertise the current state using is_state?' do
+    @instance.is_running?.must_equal true
+    @instance.is_stopped?.must_equal false
+  end
+
+end

data/tests/drivers/models/keys_test.rb

@@ -0,0 +1,25 @@
+require 'minitest/autorun'
+
+require_relative File.join('..', '..', '..', 'lib', 'deltacloud', 'models', 'base_model.rb')
+require_relative File.join('..', '..', '..', 'lib', 'deltacloud', 'models', 'key.rb')
+
+describe Key do
+
+  before do
+    @key = Key.new(:credential_type => :key)
+  end
+
+  it 'advertise if it is password or key' do
+    @key.is_password?.must_equal false
+    @key.is_key?.must_equal true
+  end
+
+  it 'cat generate the mock fingerprint' do
+    Key.generate_mock_fingerprint.must_match /(\w{2}:?)/
+  end
+
+  it 'can generate the mock PEM key' do
+    Key.generate_mock_pem.must_include 'PRIVATE KEY'
+  end
+
+end

data/tests/drivers/models/metrics_test.rb

@@ -0,0 +1,20 @@
+require 'minitest/autorun'
+
+require_relative File.join('..', '..', '..', 'lib', 'deltacloud', 'models', 'base_model.rb')
+require_relative File.join('..', '..', '..', 'lib', 'deltacloud', 'models', 'metric.rb')
+
+describe Metric do
+
+  before do
+    @metric = Metric.new(:id => 'metric1', :entity => 'inst1')
+  end
+
+  it 'cat be extended by add_property' do
+    @metric.add_property :network, { :max => 100, :min => 10, :avg => 50 }
+    @metric.properties.wont_be_empty
+    @metric.properties.each do |p|
+      p.must_be_kind_of Metric::Property
+    end
+  end
+
+end

data/tests/drivers/rhevm/common.rb

@@ -1,21 +1,21 @@
-load File.join(File.dirname(__FILE__), '..', '..', 'common.rb')
-
-ENV['API_USER']     = 'vdcadmin@rhev.lab.eng.brq.redhat.com'
-ENV['API_PASSWORD'] = '123456'
-ENV['API_PROVIDER'] = 'https://rhev30-dc.lab.eng.brq.redhat.com:8443/rhevm-api;645e425e-66fe-4ac9-8874-537bd10ef08d'
-
 require 'vcr'
 
-DeltacloudTestCommon::record!
+# Credentials used to access RHEV-M server
+#
+# NOTE: If these are changed, the VCR fixtures need to be re-recorded
+#
+def credentials
+  {
+    :user => 'vdcadmin@rhev.lab.eng.brq.redhat.com',
+    :password => '123456',
+    :provider => 'https://rhev30-dc.lab.eng.brq.redhat.com:8443/api;645e425e-66fe-4ac9-8874-537bd10ef08d'
+  }
+end
 
-VCR.config do |c|
+VCR.configure do |c|
+  # NOTE: Empty this directory before re-recording
   c.cassette_library_dir = File.join(File.dirname(__FILE__), 'fixtures')
-  c.stub_with :webmock
+  c.hook_into :webmock
+  # Set this to :new_episodes when you want to 're-record'
   c.default_cassette_options = { :record => :none }
 end
-
-class WebMock::Config
-  def net_http_connect_on_start
-    true
-  end
-end