deltacloud-core 0.5.0 → 1.0.0

data/tests/drivers/mock/storage_volumes_test.rb

@@ -0,0 +1,122 @@
+$:.unshift File.join(File.dirname(__FILE__), '..', '..', '..')
+require 'tests/drivers/mock/common'
+
+describe 'Deltacloud API storage_volumes' do
+  include Deltacloud::Test
+
+  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
+
+  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
+  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
+  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
+  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
+  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
+  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
+  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
+  end
+
+end

data/tests/drivers/openstack/api_test.rb

@@ -1,5 +1,5 @@
 $:.unshift File.join(File.dirname(__FILE__), '..', '..', '..')
-require 'tests/common'
+require 'tests/drivers/openstack/common'
 
 module OpenstackTest
 
@@ -7,7 +7,12 @@ module OpenstackTest
     include Rack::Test::Methods
 
     def app
-      Sinatra::Application
+      Rack::Builder.new {
+        map '/' do
+          use Rack::Static, :urls => ["/stylesheets", "/javascripts"], :root => "public"
+          run Rack::Cascade.new([Deltacloud::API])
+        end
+      }
     end
 
     def test_01_it_returns_entry_points
@@ -31,7 +36,7 @@ module OpenstackTest
       collections.include?('instance_states').should == true
       collections.include?('instances').should == true
       collections.include?('images').should == true
-      collections.include?('buckets').should == true
+      collections.include?('buckets').should == true # NOT YET IMPLEMENTED FOR V2 driver
       collections.include?('realms').should == true
       collections.include?('hardware_profiles').should == true
       collections.length.should == 7

data/tests/drivers/openstack/common.rb

@@ -0,0 +1,21 @@
+ENV['API_DRIVER']   = "openstack"
+ENV['API_USER']     = 'foo@bar.com+foo@bar.com-default-tenant'
+ENV['API_PASSWORD'] = 'Not_a_real_password!1'
+ENV['API_PROVIDER'] = 'https://region-a.geo-1.identity.hpcloudsvc.com:35357/v2.0/'
+
+load File.join(File.dirname(__FILE__), '..', '..', 'common.rb')
+require 'vcr'
+
+DeltacloudTestCommon::record!
+
+VCR.config do |c|
+  c.cassette_library_dir = File.join(File.dirname(__FILE__), 'fixtures')
+  c.stub_with :webmock
+  c.default_cassette_options = { :record => :none }
+end
+
+class WebMock::Config
+  def net_http_connect_on_start
+    true
+  end
+end

data/tests/drivers/openstack/hardware_profiles_test.rb

@@ -1,5 +1,5 @@
 $:.unshift File.join(File.dirname(__FILE__), '..', '..', '..')
-require 'tests/common'
+require 'tests/drivers/openstack/common'
 
 module OpenstackTest
 
@@ -7,12 +7,17 @@ module OpenstackTest
     include Rack::Test::Methods
 
     def app
-      Sinatra::Application
+      Rack::Builder.new {
+        map '/' do
+          use Rack::Static, :urls => ["/stylesheets", "/javascripts"], :root => "public"
+          run Rack::Cascade.new([Deltacloud::API])
+        end
+      }
     end
 
     def test_01_it_returns_hardware_profiles
       get_auth_url '/api;driver=openstack/hardware_profiles'
-      (last_xml_response/'hardware_profiles/hardware_profile').length.should == 5
+      (last_xml_response/'hardware_profiles/hardware_profile').length.should > 0
     end
 
     def test_02_each_hardware_profile_has_a_name
@@ -32,21 +37,27 @@ module OpenstackTest
         (profile/'property[@name="storage"]').first[:unit].should == 'GB'
         (profile/'property[@name="storage"]').first[:kind].should == 'fixed'
       end
+      hwp = ((last_xml_response/'hardware_profiles/hardware_profile').first)
+      @@hwp = { :id => hwp[:id],
+                :arch => (hwp/'property[@name="architecture"]').first[:value],
+                :mem => (hwp/'property[@name="memory"]').first[:value],
+                :disk => (hwp/'property[@name="storage"]').first[:value] }
+      @@no_of_profiles = (last_xml_response/'hardware_profiles/hardware_profile').length
     end
 
     def test_04_it_returns_single_hardware_profile
-      get_auth_url '/api;driver=openstack/hardware_profiles/1'
-      (last_xml_response/'hardware_profile/name').first.text.should == '1'
-      (last_xml_response/'hardware_profile/property[@name="architecture"]').first[:value].should == 'x86_64'
-      (last_xml_response/'hardware_profile/property[@name="memory"]').first[:value].should == '512'
-      (last_xml_response/'hardware_profile/property[@name="storage"]').first[:value].should == '0'
+      get_auth_url "/api;driver=openstack/hardware_profiles/#{@@hwp[:id]}"
+      (last_xml_response/'hardware_profile/name').first.text.should == @@hwp[:id]
+      (last_xml_response/'hardware_profile/property[@name="architecture"]').first[:value].should == @@hwp[:arch]
+      (last_xml_response/'hardware_profile/property[@name="memory"]').first[:value].should == @@hwp[:mem]
+      (last_xml_response/'hardware_profile/property[@name="storage"]').first[:value].should == @@hwp[:disk]
     end
 
     def test_05_it_filter_hardware_profiles
       get_auth_url '/api;driver=openstack/hardware_profiles?architecture=i386'
       (last_xml_response/'hardware_profiles/hardware_profile').length.should == 0
       get_auth_url '/api;driver=openstack/hardware_profiles?architecture=x86_64'
-      (last_xml_response/'hardware_profiles/hardware_profile').length.should == 5
+      (last_xml_response/'hardware_profiles/hardware_profile').length.should == @@no_of_profiles
     end
 
   end

data/tests/drivers/openstack/images_test.rb

@@ -1,5 +1,5 @@
 $:.unshift File.join(File.dirname(__FILE__), '..', '..', '..')
-require 'tests/common'
+require 'tests/drivers/openstack/common'
 
 module OpenstackTest
 
@@ -7,7 +7,12 @@ module OpenstackTest
     include Rack::Test::Methods
 
     def app
-      Sinatra::Application
+      Rack::Builder.new {
+        map '/' do
+          use Rack::Static, :urls => ["/stylesheets", "/javascripts"], :root => "public"
+          run Rack::Cascade.new([Deltacloud::API])
+        end
+      }
     end
 
     def test_01_it_returns_images
@@ -25,14 +30,15 @@ module OpenstackTest
         (image/'architecture').should_not == nil
         (image/'architecture').should_not == ''
         (image/'state').text.should == 'ACTIVE'
-        (image/'owner_id').text.should == ENV['API_USER']
-        (image/'actions/link').length.should == 1
+        ENV['API_USER'].include?((image/'owner_id').text).should == true
+        (image/'actions/link').length.should == 2
         (image/'actions/link').first[:rel].should == 'create_instance'
       end
+      @@image_id = ((last_xml_response/'images/image').first)[:id]
     end
 
     def test_03_it_returns_single_image
-      get_auth_url '/api;driver=openstack/images/1'
+      get_auth_url "/api;driver=openstack/images/#{@@image_id}"
       (last_xml_response/'image').length.should == 1
     end
 

data/tests/drivers/openstack/instances_test.rb

@@ -1,5 +1,5 @@
 $:.unshift File.join(File.dirname(__FILE__), '..', '..', '..')
-require 'tests/common'
+require 'tests/drivers/openstack/common'
 
 module OpenstackTest
 
@@ -7,39 +7,52 @@ module OpenstackTest
     include Rack::Test::Methods
 
     def app
-      Sinatra::Application
+      Rack::Builder.new {
+        map '/' do
+          use Rack::Static, :urls => ["/stylesheets", "/javascripts"], :root => "public"
+          run Rack::Cascade.new([Deltacloud::API])
+        end
+      }
     end
 
     def test_01_01_it_can_create_instance_without_hardware_profile
+      get_auth_url '/api;driver=openstack/images'
+      @@image_id = ((last_xml_response/'images/image').first)[:id]
       params = {
-        :image_id => '4',
+        :image_id => @@image_id,
         :'api[driver]' => 'openstack',
       }
-      post_url '/api/instances', params
+      uri = '/api/instances'
+      vcr_cassette = stable_vcr_cassette_name('post', uri, params)
+      post_url uri, params, {'vcr_cassette'=>vcr_cassette}
       last_response.status.should == 201 # Created
       @@instance = last_xml_response
       (@@instance/'instance').length.should > 0
       (@@instance/'instance/name').first.text.should_not == nil
       (@@instance/'instance/name').first.text.should_not == nil
       (@@instance/'instance/owner_id').first.text.should_not == ''
-      (@@instance/'instance/owner_id').first.text.should == ENV['API_USER']
+      ENV['API_USER'].include?((@@instance/'instance/owner_id').first.text).should == true
       (@@instance/'instance/state').first.text.should == 'PENDING'
     end
 
     def test_01_02_it_can_create_instance_with_hardware_profile
+      get_auth_url '/api;driver=openstack/hardware_profiles'
+      @@hwp_id = ((last_xml_response/'hardware_profiles/hardware_profile').first)[:id]
       params = {
-        :image_id => '4',
-        :hwp_id => '2',
+        :image_id => @@image_id,
+        :hwp_id => @@hwp_id,
         :'api[driver]' => 'openstack',
       }
-      post_url '/api/instances', params
+      uri = '/api/instances'
+      vcr_cassette = stable_vcr_cassette_name('post', uri, params)
+      post_url uri, params, {'vcr_cassette'=>vcr_cassette}
       last_response.status.should == 201 # Created
       @@instance2 = last_xml_response
       (@@instance2/'instance').length.should > 0
       (@@instance2/'instance/name').first.text.should_not == nil
       (@@instance2/'instance/name').first.text.should_not == nil
       (@@instance2/'instance/owner_id').first.text.should_not == ''
-      (@@instance2/'instance/owner_id').first.text.should == ENV['API_USER']
+      ENV['API_USER'].include?((@@instance2/'instance/owner_id').first.text).should == true
       (@@instance2/'instance/state').first.text.should == 'PENDING'
     end
 
@@ -58,6 +71,7 @@ module OpenstackTest
       (@@instance2/'instance/authentication/login/password').first.text.should_not == nil
       (@@instance2/'instance/authentication/login/password').first.text.should_not == ''
     end
+
 =begin
     TODO: Disabled since our testing setup doesn't return IP addresses yet ;-)
     def test_03_01_created_instance_has_correct_addresses
@@ -75,7 +89,7 @@ module OpenstackTest
 
     def test_03_02_created_instance_has_correct_hardware_profile
       (@@instance2/'instance/hardware_profile').length.should == 1
-      (@@instance2/'instance/hardware_profile').first[:id].should == "2"
+      (@@instance2/'instance/hardware_profile').first[:id].should == @@hwp_id
       (@@instance2/'instance/hardware_profile').first[:href].should_not == nil
     end
 
@@ -94,7 +108,6 @@ module OpenstackTest
       (last_xml_response/'instance/actions/link[@rel="reboot"]').first.should_not == nil
       (last_xml_response/'instance/actions/link[@rel="stop"]').first.should_not == nil
       (last_xml_response/'instance/actions/link[@rel="create_image"]').first.should_not == nil
-      (last_xml_response/'instance/actions/link[@rel="run"]').first.should_not == nil
     end
 
     def test_04_02_created_instance_goes_to_running_state
@@ -112,7 +125,6 @@ module OpenstackTest
       (last_xml_response/'instance/actions/link[@rel="reboot"]').first.should_not == nil
       (last_xml_response/'instance/actions/link[@rel="stop"]').first.should_not == nil
       (last_xml_response/'instance/actions/link[@rel="create_image"]').first.should_not == nil
-      (last_xml_response/'instance/actions/link[@rel="run"]').first.should_not == nil
     end
 
     def test_05_01_created_instance_can_be_rebooted
@@ -120,7 +132,7 @@ module OpenstackTest
         :'api[driver]' => 'openstack',
       }
       post_url "/api/instances/#{(@@instance/'instance').first[:id]}/reboot", params
-      last_response.status.should == 200
+      last_response.status.should == 202
       20.times do |tick|
         get_auth_url "/api;driver=openstack/instances/#{(@@instance/'instance').first[:id]}", { :tick => tick}
         last_response.status.should_not == 500
@@ -130,12 +142,36 @@ module OpenstackTest
       end
     end
 
+    def test_05_02_created_instance_can_be_rebooted
+      params = {
+        :'api[driver]' => 'openstack',
+      }
+      post_url "/api/instances/#{(@@instance2/'instance').first[:id]}/reboot", params
+      last_response.status.should == 202
+      20.times do |tick|
+        get_auth_url "/api;driver=openstack/instances/#{(@@instance2/'instance').first[:id]}", { :tick => tick}
+        last_response.status.should_not == 500
+        state = (last_xml_response/'instance/state').first.text
+        break if state=='RUNNING'
+        sleep(5)
+      end
+    end
+
     def test_06_01_created_instance_can_be_destroyed
+      #first make sure we recovered from the reboot
+      20.times do |tick|
+        get_auth_url "/api;driver=openstack/instances/#{(@@instance/'instance').first[:id]}", { :tick => tick}
+        last_response.status.should_not == 500
+        state = (last_xml_response/'instance/state').first.text
+        break if state=='RUNNING'
+        sleep(5)
+      end
+      #now destroy
       params = {
         :'api[driver]' => 'openstack',
       }
       post_url "/api/instances/#{(@@instance/'instance').first[:id]}/stop", params
-      last_response.status.should == 200
+      last_response.status.should == 202
       20.times do |tick|
         get_auth_url "/api;driver=openstack/instances/#{(@@instance/'instance').first[:id]}", { :tick => tick}
         last_response.status.should_not == 500
@@ -146,11 +182,20 @@ module OpenstackTest
     end
 
     def test_06_02_created_instance_can_be_destroyed
+      #first make sure we recovered from the reboot
+      20.times do |tick|
+        get_auth_url "/api;driver=openstack/instances/#{(@@instance2/'instance').first[:id]}", { :tick => tick}
+        last_response.status.should_not == 500
+        state = (last_xml_response/'instance/state').first.text
+        break if state=='RUNNING'
+        sleep(5)
+      end
+      #now destroy
       params = {
         :'api[driver]' => 'openstack',
       }
-      post_url "/api/instances/#{(@@instance2/'instance').first[:id]}/stop", params, authenticate
-      last_response.status.should == 200
+      post_url "/api/instances/#{(@@instance2/'instance').first[:id]}/stop", params
+      last_response.status.should == 202
       20.times do |tick|
         get_auth_url "/api;driver=openstack/instances/#{(@@instance2/'instance').first[:id]}", { :tick => tick}
         last_response.status.should_not == 500

data/tests/drivers/openstack/realms_test.rb

@@ -1,5 +1,5 @@
 $:.unshift File.join(File.dirname(__FILE__), '..', '..', '..')
-require 'tests/common'
+require 'tests/drivers/openstack/common'
 
 module OpenstackTest
 
@@ -7,12 +7,17 @@ module OpenstackTest
     include Rack::Test::Methods
 
     def app
-      Sinatra::Application
+      Rack::Builder.new {
+        map '/' do
+          use Rack::Static, :urls => ["/stylesheets", "/javascripts"], :root => "public"
+          run Rack::Cascade.new([Deltacloud::API])
+        end
+      }
     end
 
     def test_01_it_returns_realms
       get_auth_url '/api;driver=openstack/realms'
-      (last_xml_response/'realms/realm').length.should == 1
+      (last_xml_response/'realms/realm').length.should > 0
     end
 
     def test_02_each_realm_has_a_name
@@ -20,16 +25,15 @@ module OpenstackTest
       (last_xml_response/'realms/realm').each do |profile|
         (profile/'name').text.should_not == nil
         (profile/'name').text.should_not == ''
-        (profile/'name').text.should == 'United States'
+        (profile/'name').text.should == 'default'
       end
     end
 
     def test_03_it_returns_single_realm
       get_auth_url '/api;driver=openstack/realms/us'
-      (last_xml_response/'realm').first[:id].should == 'us'
-      (last_xml_response/'realm/name').first.text.should == 'United States'
+      (last_xml_response/'realm').first[:id].should == 'default'
+      (last_xml_response/'realm/name').first.text.should == 'default'
       (last_xml_response/'realm/state').first.text.should == 'AVAILABLE'
-      (last_xml_response/'realm/limit').first.text.should == ''
     end
 
   end