Merge pull request #21 from fog/add_specs

Added specs for collections

lib/fog/compute/xen_server.rb

@@ -18,22 +18,40 @@ class XenServer < Fog::Service
       collection :blobs
       model :bond
       collection :bonds
+      model :console
+      collection :consoles
       model :crash_dump
       collection :crash_dumps
       model :dr_task
       collection :dr_tasks
       model :gpu_group
       collection :gpu_groups
+      model :guest_metrics
+      collection :guests_metrics
+      model :host
+      collection :hosts
+      model :host_cpu
+      collection :host_cpus
       model :host_crash_dump
       collection :host_crash_dumps
+      model :host_metrics
+      collection :hosts_metrics
       model :host_patch
       collection :host_patchs
+      model :network
+      collection :networks
+      model :pbd
+      collection :pbds
       model :pci
       collection :pcis
       model :pgpu
       collection :pgpus
+      model :pif
+      collection :pifs
       model :pif_metrics
       collection :pifs_metrics
+      model :pool
+      collection :pools
       model :pool_patch
       collection :pool_patchs
       model :role
@@ -42,48 +60,30 @@ class XenServer < Fog::Service
       collection :servers
       model :server_appliance
       collection :server_appliances
+      model :server_metrics
+      collection :servers_metrics
       model :storage_manager
       collection :storage_managers
+      model :storage_repository
+      collection :storage_repositories
       model :tunnel
       collection :tunnels
-      model :vmpp
-      collection :vmpps
-      model :vtpm
-      collection :vtpms
-      model :host
-      collection :hosts
-      collection :vifs
-      model :vif
-      collection :storage_repositories
-      model :storage_repository
-      collection :pools
-      model :pool
-      collection :vbds
       model :vbd
-      collection :vdis
-      model :vdi
-      collection :networks
-      model :network
-      collection :pifs
-      model :pif
-      collection :pbds
-      model :pbd
-      model :guest_metrics
-      collection :guests_metrics
-      model :server_metrics
-      collection :servers_metrics
+      collection :vbds
       model :vbd_metrics
       collection :vbds_metrics
+      model :vdi
+      collection :vdis
+      model :vif
+      collection :vifs
       model :vif_metrics
       collection :vifs_metrics
-      model :host_metrics
-      collection :hosts_metrics
-      model :host_cpu
-      collection :host_cpus
       model :vlan
       collection :vlans
-      model :console
-      collection :consoles
+      model :vmpp
+      collection :vmpps
+      model :vtpm
+      collection :vtpms
 
       request_path 'fog/compute/xen_server/requests'
       request :builtin_templates

lib/fog/compute/xen_server/models.rb

@@ -15,6 +15,7 @@ module Models
         autoload :GpuGroup, 'fog/compute/xen_server/models/gpu_group'
         autoload :GpuGroups, 'fog/compute/xen_server/models/gpu_groups'
         autoload :GuestMetrics, 'fog/compute/xen_server/models/guest_metrics'
+        autoload :GuestsMetrics, 'fog/compute/xen_server/models/guests_metrics'
         autoload :Host, 'fog/compute/xen_server/models/host'
         autoload :HostCpu, 'fog/compute/xen_server/models/host_cpu'
         autoload :HostCpus, 'fog/compute/xen_server/models/host_cpus'

lib/fog/compute/xen_server/models/pifs_metrics.rb

@@ -1,8 +1,10 @@
 module Fog
   module Compute
     class XenServer
-      class PifsMetrics < Fog::Collection
-        model Fog::Compute::XenServer::Models::PifMetrics
+      module Models
+        class PifsMetrics < Fog::Collection
+          model Fog::Compute::XenServer::Models::PifMetrics
+        end
       end
     end
   end

lib/fog/compute/xen_server/models/roles.rb

@@ -1,7 +1,7 @@
 module Fog
   module Compute
     class XenServer
-      module Model
+      module Models
         class Roles < Fog::Collection
           model Fog::Compute::XenServer::Models::Role
         end

spec/fog/compute/xen_server/models/blob_spec.rb

@@ -6,16 +6,12 @@ class Fog::Compute::XenServer::Models::Blob
       def self.read_identity
         instance_variable_get('@identity')
       end
-
-      def self.read_provider_class
-        instance_variable_get('@provider_class')
-      end
     end
     Fog::Compute::XenServer::Models::Blob
   end
 
   it 'should associate to a provider class' do
-    blob_class.read_provider_class.must_equal('blob')
+    blob_class.provider_class.must_equal('blob')
   end
 
   it 'should have an unique id' do

spec/fog/compute/xen_server/models/blobs_spec.rb

@@ -0,0 +1,9 @@
+require 'minitest_helper'
+
+describe Fog::Compute::XenServer::Models::Blobs do
+  let(:blobs_class) { Fog::Compute::XenServer::Models::Blobs }
+
+  it 'should be a collection of Blobs' do
+    blobs_class.model.must_equal(Fog::Compute::XenServer::Models::Blob)
+  end
+end

spec/fog/compute/xen_server/models/bond_spec.rb

@@ -6,16 +6,12 @@ class Fog::Compute::XenServer::Models::Bond
       def self.read_identity
         instance_variable_get('@identity')
       end
-
-      def self.read_provider_class
-        instance_variable_get('@provider_class')
-      end
     end
     Fog::Compute::XenServer::Models::Bond
   end
 
   it 'should associate to a provider class' do
-    bond_class.read_provider_class.must_equal('Bond')
+    bond_class.provider_class.must_equal('Bond')
   end
 
   it 'should have an unique id' do

spec/fog/compute/xen_server/models/bonds_spec.rb

@@ -0,0 +1,9 @@
+require 'minitest_helper'
+
+describe Fog::Compute::XenServer::Models::Bonds do
+  let(:bonds_class) { Fog::Compute::XenServer::Models::Bonds }
+
+  it 'should be a collection of Bonds' do
+    bonds_class.model.must_equal(Fog::Compute::XenServer::Models::Bond)
+  end
+end

spec/fog/compute/xen_server/models/console_spec.rb

@@ -6,16 +6,12 @@ class Fog::Compute::XenServer::Models::Console
       def self.read_identity
         instance_variable_get('@identity')
       end
-
-      def self.read_provider_class
-        instance_variable_get('@provider_class')
-      end
     end
     Fog::Compute::XenServer::Models::Console
   end
 
   it 'should associate to a provider class' do
-    console_class.read_provider_class.must_equal('console')
+    console_class.provider_class.must_equal('console')
   end
 
   it 'should have an unique id' do

spec/fog/compute/xen_server/models/consoles_spec.rb

@@ -0,0 +1,9 @@
+require 'minitest_helper'
+
+describe Fog::Compute::XenServer::Models::Consoles do
+  let(:consoles_class) { Fog::Compute::XenServer::Models::Consoles }
+
+  it 'should be a collection of Consoles' do
+    consoles_class.model.must_equal(Fog::Compute::XenServer::Models::Console)
+  end
+end

spec/fog/compute/xen_server/models/crash_dump_spec.rb

@@ -6,16 +6,12 @@ class Fog::Compute::XenServer::Models::CrashDump
       def self.read_identity
         instance_variable_get('@identity')
       end
-
-      def self.read_provider_class
-        instance_variable_get('@provider_class')
-      end
     end
     Fog::Compute::XenServer::Models::CrashDump
   end
 
   it 'should associate to a provider class' do
-    crash_dump_class.read_provider_class.must_equal('crashdump')
+    crash_dump_class.provider_class.must_equal('crashdump')
   end
 
   it 'should have an unique id' do

spec/fog/compute/xen_server/models/crash_dumps_spec.rb

@@ -0,0 +1,9 @@
+require 'minitest_helper'
+
+describe Fog::Compute::XenServer::Models::CrashDumps do
+  let(:crash_dumps_class) { Fog::Compute::XenServer::Models::CrashDumps }
+
+  it 'should be a collection of CrashDumps' do
+    crash_dumps_class.model.must_equal(Fog::Compute::XenServer::Models::CrashDump)
+  end
+end

spec/fog/compute/xen_server/models/dr_task_spec.rb

@@ -6,16 +6,12 @@ class Fog::Compute::XenServer::Models::DrTask
       def self.read_identity
         instance_variable_get('@identity')
       end
-
-      def self.read_provider_class
-        instance_variable_get('@provider_class')
-      end
     end
     Fog::Compute::XenServer::Models::DrTask
   end
 
   it 'should associate to a provider class' do
-    dr_task_class.read_provider_class.must_equal('DR_task')
+    dr_task_class.provider_class.must_equal('DR_task')
   end
 
   it 'should have an unique id' do

spec/fog/compute/xen_server/models/dr_tasks_spec.rb

@@ -0,0 +1,9 @@
+require 'minitest_helper'
+
+describe Fog::Compute::XenServer::Models::DrTasks do
+  let(:dr_tasks_class) { Fog::Compute::XenServer::Models::DrTasks }
+
+  it 'should be a collection of DrTasks' do
+    dr_tasks_class.model.must_equal(Fog::Compute::XenServer::Models::DrTask)
+  end
+end

spec/fog/compute/xen_server/models/gpu_group_spec.rb

@@ -6,16 +6,12 @@ class Fog::Compute::XenServer::Models::GpuGroup
       def self.read_identity
         instance_variable_get('@identity')
       end
-
-      def self.read_provider_class
-        instance_variable_get('@provider_class')
-      end
     end
     Fog::Compute::XenServer::Models::GpuGroup
   end
 
   it 'should associate to a provider class' do
-    gpu_group_class.read_provider_class.must_equal('GPU_group')
+    gpu_group_class.provider_class.must_equal('GPU_group')
   end
 
   it 'should have an unique id' do

spec/fog/compute/xen_server/models/gpu_groups_spec.rb

@@ -0,0 +1,9 @@
+require 'minitest_helper'
+
+describe Fog::Compute::XenServer::Models::GpuGroups do
+  let(:gpu_groups_class) { Fog::Compute::XenServer::Models::GpuGroups }
+
+  it 'should be a collection of GpuGroups' do
+    gpu_groups_class.model.must_equal(Fog::Compute::XenServer::Models::GpuGroup)
+  end
+end

spec/fog/compute/xen_server/models/guest_metrics_spec.rb

@@ -6,16 +6,12 @@ class Fog::Compute::XenServer::Models::GuestMetrics
       def self.read_identity
         instance_variable_get('@identity')
       end
-
-      def self.read_provider_class
-        instance_variable_get('@provider_class')
-      end
     end
     Fog::Compute::XenServer::Models::GuestMetrics
   end
 
   it 'should associate to a provider class' do
-    guest_metrics_class.read_provider_class.must_equal('VM_guest_metrics')
+    guest_metrics_class.provider_class.must_equal('VM_guest_metrics')
   end
 
   it 'should have an unique id' do

spec/fog/compute/xen_server/models/guests_metrics_spec.rb

@@ -0,0 +1,9 @@
+require 'minitest_helper'
+
+describe Fog::Compute::XenServer::Models::GuestsMetrics do
+  let(:guests_metrics_class) { Fog::Compute::XenServer::Models::GuestsMetrics }
+
+  it 'should be a collection of GuestsMetrics' do
+    guests_metrics_class.model.must_equal(Fog::Compute::XenServer::Models::GuestMetrics)
+  end
+end

spec/fog/compute/xen_server/models/host_cpu_spec.rb

@@ -6,16 +6,12 @@ class Fog::Compute::XenServer::Models::HostCpu
       def self.read_identity
         instance_variable_get('@identity')
       end
-
-      def self.read_provider_class
-        instance_variable_get('@provider_class')
-      end
     end
     Fog::Compute::XenServer::Models::HostCpu
   end
 
   it 'should associate to a provider class' do
-    host_cpu_class.read_provider_class.must_equal('host_cpu')
+    host_cpu_class.provider_class.must_equal('host_cpu')
   end
 
   it 'should have an unique id' do

spec/fog/compute/xen_server/models/host_cpus_spec.rb

@@ -0,0 +1,9 @@
+require 'minitest_helper'
+
+describe Fog::Compute::XenServer::Models::HostCpus do
+  let(:host_cpus_class) { Fog::Compute::XenServer::Models::HostCpus }
+
+  it 'should be a collection of HostsCpus' do
+    host_cpus_class.model.must_equal(Fog::Compute::XenServer::Models::HostCpu)
+  end
+end

spec/fog/compute/xen_server/models/host_crash_dump_spec.rb

@@ -6,16 +6,12 @@ class Fog::Compute::XenServer::Models::HostCrashDump
       def self.read_identity
         instance_variable_get('@identity')
       end
-
-      def self.read_provider_class
-        instance_variable_get('@provider_class')
-      end
     end
     Fog::Compute::XenServer::Models::HostCrashDump
   end
 
   it 'should associate to a provider class' do
-    host_crash_dump_class.read_provider_class.must_equal('host_crashdump')
+    host_crash_dump_class.provider_class.must_equal('host_crashdump')
   end
 
   it 'should have an unique id' do

spec/fog/compute/xen_server/models/host_crash_dumps_spec.rb

@@ -0,0 +1,9 @@
+require 'minitest_helper'
+
+describe Fog::Compute::XenServer::Models::HostCrashDumps do
+  let(:host_crash_dumps_class) { Fog::Compute::XenServer::Models::HostCrashDumps }
+
+  it 'should be a collection of HostCrashDumps' do
+    host_crash_dumps_class.model.must_equal(Fog::Compute::XenServer::Models::HostCrashDump)
+  end
+end