foreman_fog_proxmox 0.8.0 → 0.8.2

data/test/unit/foreman_fog_proxmox/helpers/proxmox_server_helper_test.rb

@@ -20,132 +20,128 @@
 require 'test_plugin_helper'
 
 module ForemanFogProxmox
-class ProxmoxServerHelperTest < ActiveSupport::TestCase
-  include ProxmoxServerHelper
-  include ProxmoxVmHelper
-
-  describe 'parse' do
-
-    setup { Fog.mock! }
-    teardown { Fog.unmock! }
-
-    let(:host) do 
-      { 'vmid' => '100', 
-        'name' =>  'test', 
-        'node_id' => 'pve',
-        'type' => 'qemu',
-        'config_attributes' => { 
-          'memory' => '536870912', 
-          'min_memory' => '', 
-          'ballon' => '', 
-          'shares' => '', 
-          'cpu_type' => 'kvm64', 
-          'spectre' => '1', 
-          'pcid' => '0', 
-          'cores' => '1', 
-          'sockets' => '1'
-        },
-        'volumes_attributes' => {
-          '0'=> { 'controller' => 'scsi', 'device' => '0', 'storage' => 'local-lvm', 'size' => '1073741824', 'cache' => 'none' },
-          '1'=> { 'controller' => 'virtio', 'device' => '0', 'storage' => 'local-lvm', 'size' => '1073741824', 'cache' => 'none' }
-        }, 
-        'interfaces_attributes' => { 
-          '0' => { 'id' => 'net0', 'model' => 'virtio', 'bridge' => 'vmbr0', 'firewall' => '0', 'link_down' => '0', 'rate' => nil },
-          '1' => { 'id' => 'net1', 'model' => 'e1000', 'bridge' => 'vmbr0', 'firewall' => '0', 'link_down' => '0' } 
-        } 
-      }
+  class ProxmoxServerHelperTest < ActiveSupport::TestCase
+    include ProxmoxServerHelper
+    include ProxmoxVmHelper
+
+    describe 'parse' do
+      setup { Fog.mock! }
+      teardown { Fog.unmock! }
+
+      let(:host) do
+        { 'vmid' => '100',
+          'name' => 'test',
+          'node_id' => 'pve',
+          'type' => 'qemu',
+          'config_attributes' => {
+            'memory' => '536870912',
+            'min_memory' => '',
+            'ballon' => '',
+            'shares' => '',
+            'cpu_type' => 'kvm64',
+            'spectre' => '1',
+            'pcid' => '0',
+            'cores' => '1',
+            'sockets' => '1'
+          },
+          'volumes_attributes' => {
+            '0' => { 'controller' => 'scsi', 'device' => '0', 'storage' => 'local-lvm', 'size' => '1073741824', 'cache' => 'none' },
+            '1' => { 'controller' => 'virtio', 'device' => '0', 'storage' => 'local-lvm', 'size' => '1073741824', 'cache' => 'none' }
+          },
+          'interfaces_attributes' => {
+            '0' => { 'id' => 'net0', 'model' => 'virtio', 'bridge' => 'vmbr0', 'firewall' => '0', 'link_down' => '0', 'rate' => nil },
+            '1' => { 'id' => 'net1', 'model' => 'e1000', 'bridge' => 'vmbr0', 'firewall' => '0', 'link_down' => '0' }
+          } }
+      end
+
+      let(:host_delete) do
+        { 'vmid' => '100',
+          'name' => 'test',
+          'type' => 'qemu',
+          'cdrom' => 'image',
+          'cdrom_iso' => 'local-lvm:iso/debian-netinst.iso',
+          'volumes_attributes' => { '0' => { '_delete' => '1', 'controller' => 'scsi', 'device' => '0', 'storage' => 'local-lvm', 'size' => '1073741824' } },
+          'interfaces_attributes' => { '0' => { '_delete' => '1', 'id' => 'net0', 'model' => 'virtio' } } }
+      end
+
+      test '#memory' do
+        memory = parse_server_memory(host['config_attributes'])
+        assert memory.key?(:memory)
+        assert_equal 536_870_912, memory[:memory]
+      end
+
+      test '#cpu' do
+        cpu = parse_server_cpu(host['config_attributes'])
+        assert cpu.key?(:cpu)
+        assert_equal 'cputype=kvm64,flags=+spec-ctrl', cpu[:cpu]
+      end
+
+      test '#vm' do
+        vm = parse_server_vm(host)
+        assert_equal '1', vm['cores']
+        assert_equal '1', vm['sockets']
+        assert_equal 'cputype=kvm64,flags=+spec-ctrl', vm[:cpu]
+        assert_equal 536_870_912, vm[:memory]
+        assert_equal 'local-lvm:1073741824,cache=none', vm[:scsi0]
+        assert_equal 'model=virtio,bridge=vmbr0,firewall=0,link_down=0', vm[:net0]
+        assert_not vm.key?(:config)
+        assert_not vm.key?(:node)
+      end
+
+      test '#volume with scsi 1Gb' do
+        volumes = parse_server_volumes(host['volumes_attributes'])
+        assert_not volumes.empty?
+        assert volumes.size, 2
+        assert volume = volumes.first
+        assert volume.key?(:scsi0)
+        assert_equal 'local-lvm:1073741824,cache=none', volume[:scsi0]
+      end
+
+      test '#volume with virtio 1Gb' do
+        volumes = parse_server_volumes(host['volumes_attributes'])
+        assert_not volumes.empty?
+        assert volumes.size, 2
+        assert volume = volumes[1]
+        assert volume.key?(:virtio0)
+        assert_equal 'local-lvm:1073741824,cache=none', volume[:virtio0]
+      end
+
+      test '#interface with model virtio and bridge' do
+        interfaces_to_delete = []
+        interfaces_to_add = []
+        add_server_interface(host['interfaces_attributes']['0'], interfaces_to_delete, interfaces_to_add)
+        assert interfaces_to_delete.empty?
+        assert_equal 1, interfaces_to_add.length
+        assert interfaces_to_add[0].key?(:net0)
+        assert_equal 'model=virtio,bridge=vmbr0,firewall=0,link_down=0', interfaces_to_add[0][:net0]
+      end
+
+      test '#interface with model e1000 and bridge' do
+        interfaces_to_delete = []
+        interfaces_to_add = []
+        add_server_interface(host['interfaces_attributes']['1'], interfaces_to_delete, interfaces_to_add)
+        assert interfaces_to_delete.empty?
+        assert_equal 1, interfaces_to_add.length
+        assert interfaces_to_add[0].key?(:net1)
+        assert_equal 'model=e1000,bridge=vmbr0,firewall=0,link_down=0', interfaces_to_add[0][:net1]
+      end
+
+      test '#interface delete net0' do
+        interfaces_to_delete = []
+        interfaces_to_add = []
+        add_server_interface(host_delete['interfaces_attributes']['0'], interfaces_to_delete, interfaces_to_add)
+        assert interfaces_to_add.empty?
+        assert_equal 1, interfaces_to_delete.length
+        assert_equal 'net0', interfaces_to_delete[0]
+      end
+
+      test '#interfaces' do
+        interfaces_to_add, interfaces_to_delete = parse_server_interfaces(host['interfaces_attributes'])
+        assert interfaces_to_delete.empty?
+        assert_equal 2, interfaces_to_add.length
+        assert interfaces_to_add.include?(net0: 'model=virtio,bridge=vmbr0,firewall=0,link_down=0')
+        assert interfaces_to_add.include?(net1: 'model=e1000,bridge=vmbr0,firewall=0,link_down=0')
+      end
     end
-
-    let(:host_delete) do 
-      { 'vmid' => '100', 
-        'name' =>  'test', 
-        'type' => 'qemu',
-        'cdrom' => 'image',
-        'cdrom_iso' => 'local-lvm:iso/debian-netinst.iso',
-        'volumes_attributes' => { '0' => { '_delete' => '1', 'controller' => 'scsi', 'device' => '0', 'storage' => 'local-lvm', 'size' => '1073741824' }}, 
-        'interfaces_attributes' => { '0' => { '_delete' => '1', 'id' => 'net0', 'model' => 'virtio' } } 
-      }
-    end
-
-    test '#memory' do       
-      memory = parse_server_memory(host['config_attributes'])
-      assert memory.has_key?(:memory)
-      assert_equal 536870912, memory[:memory]
-    end   
-
-    test '#cpu' do       
-      cpu = parse_server_cpu(host['config_attributes'])
-      assert cpu.has_key?(:cpu)
-      assert_equal 'cputype=kvm64,flags=+spec-ctrl', cpu[:cpu]
-    end   
-
-    test '#vm' do       
-      vm = parse_server_vm(host)
-      assert_equal '1', vm['cores']
-      assert_equal '1', vm['sockets']
-      assert_equal 'cputype=kvm64,flags=+spec-ctrl', vm[:cpu]
-      assert_equal 536870912, vm[:memory]
-      assert_equal 'local-lvm:1073741824,cache=none', vm[:scsi0]
-      assert_equal 'model=virtio,bridge=vmbr0,firewall=0,link_down=0', vm[:net0]
-      assert !vm.has_key?(:config)
-      assert !vm.has_key?(:node)
-    end   
-
-    test '#volume with scsi 1Gb' do       
-      volumes = parse_server_volumes(host['volumes_attributes'])
-      assert !volumes.empty?
-      assert volumes.size, 2
-      assert volume = volumes.first
-      assert volume.has_key?(:scsi0)
-      assert_equal 'local-lvm:1073741824,cache=none', volume[:scsi0]
-    end    
-
-    test '#volume with virtio 1Gb' do       
-      volumes = parse_server_volumes(host['volumes_attributes'])
-      assert !volumes.empty?
-      assert volumes.size, 2
-      assert volume = volumes[1]
-      assert volume.has_key?(:virtio0)
-      assert_equal 'local-lvm:1073741824,cache=none', volume[:virtio0]
-    end  
-    
-    test '#interface with model virtio and bridge' do      
-      interfaces_to_delete = []
-      interfaces_to_add = []      
-      add_server_interface(host['interfaces_attributes']['0'],interfaces_to_delete,interfaces_to_add)
-      assert interfaces_to_delete.empty?
-      assert_equal 1, interfaces_to_add.length
-      assert interfaces_to_add[0].has_key?(:net0)
-      assert_equal 'model=virtio,bridge=vmbr0,firewall=0,link_down=0', interfaces_to_add[0][:net0]
-    end
-    
-    test '#interface with model e1000 and bridge' do    
-      interfaces_to_delete = []
-      interfaces_to_add = []   
-      interface = add_server_interface(host['interfaces_attributes']['1'],interfaces_to_delete,interfaces_to_add)
-      assert interfaces_to_delete.empty?
-      assert_equal 1, interfaces_to_add.length
-      assert interfaces_to_add[0].has_key?(:net1)
-      assert_equal 'model=e1000,bridge=vmbr0,firewall=0,link_down=0', interfaces_to_add[0][:net1]
-    end
-    
-    test '#interface delete net0' do       
-      interfaces_to_delete = []
-      interfaces_to_add = []
-      add_server_interface(host_delete['interfaces_attributes']['0'],interfaces_to_delete,interfaces_to_add)
-      assert interfaces_to_add.empty?
-      assert_equal 1, interfaces_to_delete.length
-      assert_equal 'net0', interfaces_to_delete[0]
-    end
-    
-    test '#interfaces' do       
-      interfaces_to_add, interfaces_to_delete = parse_server_interfaces(host['interfaces_attributes'])
-      assert interfaces_to_delete.empty?
-      assert_equal 2, interfaces_to_add.length
-      assert interfaces_to_add.include?({ net0: 'model=virtio,bridge=vmbr0,firewall=0,link_down=0'})
-      assert interfaces_to_add.include?({ net1: 'model=e1000,bridge=vmbr0,firewall=0,link_down=0'})
-    end
-
   end
 end
-end

data/test/unit/foreman_fog_proxmox/helpers/proxmox_vm_helper_test.rb

@@ -28,196 +28,189 @@ require 'fog/compute/proxmox/models/snapshots'
 require 'fog/compute/proxmox/models/tasks'
 
 module ForemanFogProxmox
-class ProxmoxVmHelperTest < ActiveSupport::TestCase
-  include ProxmoxVmHelper
-
-  let(:container) do 
-    service = mock('service')
-    service.stubs(:get_server_config).returns(nil)
-    service.stubs(:list_tasks).returns([])
-    Fog::Proxmox::Compute::Server.new(
-    { 'vmid' => '100', 
-      'hostname' =>  'test', 
-      :type =>  'lxc', 
-      :node_id => 'pve',
-      :service => service,
-      'templated' => '0', 
-      'memory' => '536870912', 
-      'swap' => '',
-      'cores' => '1',
-      'arch' => 'amd64',
-      'ostype' => 'debian',
-      'rootfs' => 'local-lvm:1073741824',
-      'mp0' => 'local-lvm:1073741824', 
-      'net0' => 'name=eth0,bridge=vmbr0,ip=dhcp,ip6=dhcp',
-      'net1' => 'name=eth1,bridge=vmbr0,ip=dhcp,ip6=dhcp'
-    })
-  end
-
-  let(:server) do 
-    service = mock('service')
-    service.stubs(:get_server_config).returns(nil)
-    service.stubs(:list_tasks).returns([])
-    Fog::Proxmox::Compute::Server.new(
-    { 'vmid' => '100', 
-      'name' =>  'test', 
-      :node_id => 'pve', 
-      :service => service,
-      :type => 'qemu',
-      'templated' => '0', 
-      'ide2' => 'local-lvm:iso/debian-netinst.iso,media=cdrom',
-      'memory' => '536870912', 
-      'min_memory' => '', 
-      'ballon' => '', 
-      'shares' => '', 
-      'cpu_type' => 'kvm64', 
-      'spectre' => '1', 
-      'pcid' => '0', 
-      'cores' => '1', 
-      'sockets' => '1',
-      'scsi0' => 'local-lvm:1073741824,cache=none', 
-      'net0' => 'model=virtio,bridge=vmbr0',
-      'net1'  => 'model=e1000,bridge=vmbr0'
-    })
-  end
-
-  let(:host_server) do 
-    { 'vmid' => '100', 
-      'name' =>  'test', 
-      'node_id' => 'pve',
-      'type' => 'qemu',
-      'config_attributes' => { 
-        'memory' => '536870912', 
-        'min_memory' => nil, 
-        'ballon' => '', 
-        'shares' => '', 
-        'cpu_type' => 'kvm64', 
-        'spectre' => '1', 
-        'pcid' => '0', 
-        'cores' => '1', 
-        'sockets' => '1'
-      },
-      'volumes_attributes' => {
-        '0'=> { 'controller' => 'scsi', 'device' => '0', 'storage' => 'local-lvm', 'size' => '1073741824', 'cache' => 'none' }
-      }, 
-      'interfaces_attributes' => { 
-        '0' => { 'id' => 'net0', 'model' => 'virtio', 'bridge' => 'vmbr0', 'firewall' => '0', 'disconnect' => '0' },
-        '1' => { 'id' => 'net1', 'model' => 'e1000', 'bridge' => 'vmbr0', 'firewall' => '0', 'disconnect' => '0' } 
-      } 
-    }
-  end    
-  
-  let(:host_container) do 
-    { 'vmid' => '100', 
-      'name' =>  'test', 
-      'type' =>  'lxc', 
-      'node_id' => 'pve',
-      'ostemplate_storage' => 'local',
-      'ostemplate_file' => 'local:vztmpl/alpine-3.7-default_20171211_amd64.tar.xz',
-      'password' => 'proxmox01',
-      'config_attributes' => { 
-        'onboot' => '0', 
-        'description' => '', 
-        'memory' => '536870912', 
+  class ProxmoxVmHelperTest < ActiveSupport::TestCase
+    include ProxmoxVmHelper
+
+    let(:container) do
+      service = mock('service')
+      service.stubs(:get_server_config).returns(nil)
+      service.stubs(:list_tasks).returns([])
+      Fog::Proxmox::Compute::Server.new(
+        'vmid' => '100',
+        'hostname' => 'test',
+        :type => 'lxc',
+        :node_id => 'pve',
+        :service => service,
+        'templated' => '0',
+        'memory' => '536870912',
         'swap' => '',
         'cores' => '1',
-        'cpulimit' => '',
-        'cpuunits' => '',
         'arch' => 'amd64',
         'ostype' => 'debian',
-        'hostname' => '',
-        'nameserver' => '',
-        'searchdomain' => '',
-        
-      },
-      'volumes_attributes' => {
-        '0'=> { 'id' => 'rootfs', 'storage' => 'local-lvm', 'size' => '1073741824' },
-        '1'=> { 'id' => 'mp0', 'storage' => 'local-lvm', 'size' => '1073741824' }
-      }, 
-      'interfaces_attributes' => { 
-        '0' => { 'id' => 'net0', 'name' => 'eth0', 'bridge' => 'vmbr0', 'ip' => 'dhcp', 'ip6' => 'dhcp' },
-        '1' => { 'id' => 'net1', 'name' => 'eth1', 'bridge' => 'vmbr0', 'ip' => 'dhcp', 'ip6' => 'dhcp' } 
-      } 
-    }
-  end
-
-  describe 'object_to_config_hash' do
-
-    setup { Fog.mock! }
-    teardown { Fog.unmock! }
-
-    it '#server qemu' do            
-      config_hash = object_to_config_hash(server,'qemu')
-      expected_config_hash = ActiveSupport::HashWithIndifferentAccess.new(server.config.attributes).reject { |key,_value| %w[templated ide2 scsi0 net0 net1].include? key }
-      assert_equal expected_config_hash, config_hash['config_attributes']
-    end  
-
-    it '#server lxc' do       
-      config_hash = object_to_config_hash(server,'lxc')
-      assert config_hash.has_key?('config_attributes')
-      expected_config_hash = ActiveSupport::HashWithIndifferentAccess.new(server.config.attributes).reject { |key,_value| %w[templated ide2 scsi0 net0 net1].include? key }
-      assert_equal expected_config_hash, config_hash['config_attributes']
-    end    
-
-    it '#container qemu' do       
-      config_hash = object_to_config_hash(container,'qemu')
-      assert config_hash.has_key?('config_attributes')
-      expected_config_hash = ActiveSupport::HashWithIndifferentAccess.new(container.config.attributes).reject { |key,_value| %w[templated rootfs mp0 net0 net1].include? key }
-      assert_equal expected_config_hash, config_hash['config_attributes']
-    end  
-
-    it '#container lxc' do       
-      config_hash = object_to_config_hash(container,'lxc')
-      assert config_hash.has_key?('config_attributes')
-      expected_config_hash = ActiveSupport::HashWithIndifferentAccess.new(container.config.attributes).reject { |key,_value| %w[templated rootfs mp0 net0 net1].include? key }
-      assert_equal expected_config_hash, config_hash['config_attributes']
-    end    
-  end
-
-
-  describe 'convert_sizes' do
-
-    setup { Fog.mock! }
-    teardown { Fog.unmock! }
-
-    it '#server' do       
-      convert_sizes(host_server)
-      assert_equal '512', host_server['config_attributes']['memory']
-      assert_equal '1', host_server['volumes_attributes']['0']['size']
-    end  
-
-    it '#container' do       
-      convert_sizes(host_container)
-      assert_equal '512', host_container['config_attributes']['memory']
-      assert_equal '1', host_container['volumes_attributes']['0']['size']
-    end  
-  end
-
-  describe 'convert_memory_size' do
-
-    setup { Fog.mock! }
-    teardown { Fog.unmock! }
-
-    it '#server memory' do       
-      convert_memory_size(host_server['config_attributes'],'memory')
-      assert_equal '512', host_server['config_attributes']['memory']
-    end  
-
-    it '#server min_memory nil' do       
-      convert_memory_size(host_server['config_attributes'],'min_memory')
-      assert_nil host_server['config_attributes']['min_memory']
-    end  
-
-    it '#container memory' do       
-      convert_memory_size(host_container['config_attributes'],'memory')
-      assert_equal '512', host_container['config_attributes']['memory']
-    end 
-
-    it '#container swap empty' do       
-      convert_memory_size(host_container['config_attributes'],'swap')
-      assert host_container['config_attributes']['swap'].empty?
-    end 
+        'rootfs' => 'local-lvm:1073741824',
+        'mp0' => 'local-lvm:1073741824',
+        'net0' => 'name=eth0,bridge=vmbr0,ip=dhcp,ip6=dhcp',
+        'net1' => 'name=eth1,bridge=vmbr0,ip=dhcp,ip6=dhcp'
+      )
+    end
+
+    let(:server) do
+      service = mock('service')
+      service.stubs(:get_server_config).returns(nil)
+      service.stubs(:list_tasks).returns([])
+      Fog::Proxmox::Compute::Server.new(
+        'vmid' => '100',
+        'name' => 'test',
+        :node_id => 'pve',
+        :service => service,
+        :type => 'qemu',
+        'templated' => '0',
+        'ide2' => 'local-lvm:iso/debian-netinst.iso,media=cdrom',
+        'memory' => '536870912',
+        'min_memory' => '',
+        'ballon' => '',
+        'shares' => '',
+        'cpu_type' => 'kvm64',
+        'spectre' => '1',
+        'pcid' => '0',
+        'cores' => '1',
+        'sockets' => '1',
+        'scsi0' => 'local-lvm:1073741824,cache=none',
+        'net0' => 'model=virtio,bridge=vmbr0',
+        'net1' => 'model=e1000,bridge=vmbr0'
+      )
+    end
+
+    let(:host_server) do
+      { 'vmid' => '100',
+        'name' => 'test',
+        'node_id' => 'pve',
+        'type' => 'qemu',
+        'config_attributes' => {
+          'memory' => '536870912',
+          'min_memory' => nil,
+          'ballon' => '',
+          'shares' => '',
+          'cpu_type' => 'kvm64',
+          'spectre' => '1',
+          'pcid' => '0',
+          'cores' => '1',
+          'sockets' => '1'
+        },
+        'volumes_attributes' => {
+          '0' => { 'controller' => 'scsi', 'device' => '0', 'storage' => 'local-lvm', 'size' => '1073741824', 'cache' => 'none' }
+        },
+        'interfaces_attributes' => {
+          '0' => { 'id' => 'net0', 'model' => 'virtio', 'bridge' => 'vmbr0', 'firewall' => '0', 'disconnect' => '0' },
+          '1' => { 'id' => 'net1', 'model' => 'e1000', 'bridge' => 'vmbr0', 'firewall' => '0', 'disconnect' => '0' }
+        } }
+    end
+
+    let(:host_container) do
+      { 'vmid' => '100',
+        'name' => 'test',
+        'type' => 'lxc',
+        'node_id' => 'pve',
+        'ostemplate_storage' => 'local',
+        'ostemplate_file' => 'local:vztmpl/alpine-3.7-default_20171211_amd64.tar.xz',
+        'password' => 'proxmox01',
+        'config_attributes' => {
+          'onboot' => '0',
+          'description' => '',
+          'memory' => '536870912',
+          'swap' => '',
+          'cores' => '1',
+          'cpulimit' => '',
+          'cpuunits' => '',
+          'arch' => 'amd64',
+          'ostype' => 'debian',
+          'hostname' => '',
+          'nameserver' => '',
+          'searchdomain' => ''
+
+        },
+        'volumes_attributes' => {
+          '0' => { 'id' => 'rootfs', 'storage' => 'local-lvm', 'size' => '1073741824' },
+          '1' => { 'id' => 'mp0', 'storage' => 'local-lvm', 'size' => '1073741824' }
+        },
+        'interfaces_attributes' => {
+          '0' => { 'id' => 'net0', 'name' => 'eth0', 'bridge' => 'vmbr0', 'ip' => 'dhcp', 'ip6' => 'dhcp' },
+          '1' => { 'id' => 'net1', 'name' => 'eth1', 'bridge' => 'vmbr0', 'ip' => 'dhcp', 'ip6' => 'dhcp' }
+        } }
+    end
+
+    describe 'object_to_config_hash' do
+      setup { Fog.mock! }
+      teardown { Fog.unmock! }
+
+      it '#server qemu' do
+        config_hash = object_to_config_hash(server)
+        expected_config_hash = ActiveSupport::HashWithIndifferentAccess.new(server.config.attributes).reject { |key, _value| ['templated', 'ide2', 'scsi0', 'net0', 'net1'].include? key }
+        assert_equal expected_config_hash, config_hash['config_attributes']
+      end
+
+      it '#server lxc' do
+        config_hash = object_to_config_hash(server)
+        assert config_hash.key?('config_attributes')
+        expected_config_hash = ActiveSupport::HashWithIndifferentAccess.new(server.config.attributes).reject { |key, _value| ['templated', 'ide2', 'scsi0', 'net0', 'net1'].include? key }
+        assert_equal expected_config_hash, config_hash['config_attributes']
+      end
+
+      it '#container qemu' do
+        config_hash = object_to_config_hash(container)
+        assert config_hash.key?('config_attributes')
+        expected_config_hash = ActiveSupport::HashWithIndifferentAccess.new(container.config.attributes).reject { |key, _value| ['templated', 'rootfs', 'mp0', 'net0', 'net1'].include? key }
+        assert_equal expected_config_hash, config_hash['config_attributes']
+      end
+
+      it '#container lxc' do
+        config_hash = object_to_config_hash(container)
+        assert config_hash.key?('config_attributes')
+        expected_config_hash = ActiveSupport::HashWithIndifferentAccess.new(container.config.attributes).reject { |key, _value| ['templated', 'rootfs', 'mp0', 'net0', 'net1'].include? key }
+        assert_equal expected_config_hash, config_hash['config_attributes']
+      end
+    end
+
+    describe 'convert_sizes' do
+      setup { Fog.mock! }
+      teardown { Fog.unmock! }
+
+      it '#server' do
+        convert_sizes(host_server)
+        assert_equal '512', host_server['config_attributes']['memory']
+        assert_equal '1', host_server['volumes_attributes']['0']['size']
+      end
+
+      it '#container' do
+        convert_sizes(host_container)
+        assert_equal '512', host_container['config_attributes']['memory']
+        assert_equal '1', host_container['volumes_attributes']['0']['size']
+      end
+    end
+
+    describe 'convert_memory_size' do
+      setup { Fog.mock! }
+      teardown { Fog.unmock! }
+
+      it '#server memory' do
+        convert_memory_size(host_server['config_attributes'], 'memory')
+        assert_equal '512', host_server['config_attributes']['memory']
+      end
+
+      it '#server min_memory nil' do
+        convert_memory_size(host_server['config_attributes'], 'min_memory')
+        assert_nil host_server['config_attributes']['min_memory']
+      end
+
+      it '#container memory' do
+        convert_memory_size(host_container['config_attributes'], 'memory')
+        assert_equal '512', host_container['config_attributes']['memory']
+      end
+
+      it '#container swap empty' do
+        convert_memory_size(host_container['config_attributes'], 'swap')
+        assert host_container['config_attributes']['swap'].empty?
+      end
+    end
   end
-
 end
-end