foreman_fog_proxmox 0.12.0

data/test/unit/foreman_fog_proxmox/proxmox_images_test.rb

@@ -0,0 +1,52 @@
+# frozen_string_literal: true
+
+# Copyright 2018 Tristan Robert
+
+# This file is part of ForemanFogProxmox.
+
+# ForemanFogProxmox is free software: you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation, either version 3 of the License, or
+# (at your option) any later version.
+
+# ForemanFogProxmox is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+
+# You should have received a copy of the GNU General Public License
+# along with ForemanFogProxmox. If not, see <http://www.gnu.org/licenses/>.
+
+require 'test_plugin_helper'
+require 'active_support/core_ext/hash/indifferent_access'
+
+module ForemanFogProxmox
+  class ProxmoxVmCommandsServerTest < ActiveSupport::TestCase
+    include ComputeResourceTestHelpers
+
+    describe 'clone_from_image' do
+      before do
+        @cr = FactoryBot.build_stubbed(:proxmox_cr)
+        @args = { :name => 'name' }
+        @image_id = 100
+        @vmid = 101
+        @image = mock('vm')
+        @image.expects(:clone)
+        @cr.stubs(:find_vm_by_uuid).with(@image_id).returns(@image)
+        @clone = mock('vm')
+      end
+      it 'clones server from image' do
+        @clone.expects(:update).with(:name => 'name')
+        @clone.stubs(:container?).returns(false)
+        @cr.stubs(:find_vm_by_uuid).with(@vmid).returns(@clone)
+        @cr.clone_from_image(@image_id, @args, @vmid)
+      end
+      it 'clones container from image' do
+        @clone.stubs(:container?).returns(true)
+        @clone.expects(:update).with(:hostname => 'name')
+        @cr.stubs(:find_vm_by_uuid).with(@vmid).returns(@clone)
+        @cr.clone_from_image(@image_id, @args, @vmid)
+      end
+    end
+  end
+end

data/test/unit/foreman_fog_proxmox/proxmox_interfaces_test.rb

@@ -0,0 +1,117 @@
+# frozen_string_literal: true
+
+# Copyright 2018 Tristan Robert
+
+# This file is part of ForemanFogProxmox.
+
+# ForemanFogProxmox is free software: you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation, either version 3 of the License, or
+# (at your option) any later version.
+
+# ForemanFogProxmox is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+
+# You should have received a copy of the GNU General Public License
+# along with ForemanFogProxmox. If not, see <http://www.gnu.org/licenses/>.
+
+require 'test_plugin_helper'
+require 'models/compute_resources/compute_resource_test_helpers'
+require 'factories/foreman_fog_proxmox/proxmox_node_mock_factory'
+require 'factories/foreman_fog_proxmox/proxmox_server_mock_factory'
+require 'factories/foreman_fog_proxmox/proxmox_container_mock_factory'
+require 'active_support/core_ext/hash/indifferent_access'
+
+module ForemanFogProxmox
+  class ProxmoxInterfacesTest < ActiveSupport::TestCase
+    include ComputeResourceTestHelpers
+    include ProxmoxNodeMockFactory
+    include ProxmoxServerMockFactory
+    include ProxmoxContainerMockFactory
+    include ProxmoxVmHelper
+
+    describe 'host_interfaces_attrs' do
+      before do
+        @cr = FactoryBot.build_stubbed(:proxmox_cr)
+      end
+
+      it 'raises Foreman::Exception when physical identifier does not match net[k] with k integer' do
+        physical_nic = FactoryBot.build(:nic_base_empty, :identifier => 'eth0')
+        host = FactoryBot.build(:host_empty, :interfaces => [physical_nic])
+        err = assert_raises Foreman::Exception do
+          @cr.host_interfaces_attrs(host)
+        end
+        assert err.message.end_with?('Invalid identifier interface[0]. Must be net[n] with n integer >= 0')
+      end
+
+      it 'sets server compute id with identifier, ip and ip6' do
+        ip = '192.168.56.100'
+        ip6 = Array.new(4) { format('%<x>s', x: rand(16**4)) }.join(':') + '::1'
+        physical_nic = FactoryBot.build(:nic_base_empty, :identifier => 'net0', :ip => ip, :ip6 => ip6)
+        host = FactoryBot.build(
+          :host_empty,
+          :interfaces => [physical_nic],
+          :compute_attributes => {
+            'type' => 'qemu',
+            'interfaces_attributes' => {
+              '0' => physical_nic
+            }
+          }
+        )
+        nic_attributes = @cr.host_interfaces_attrs(host).values.select(&:present?)
+        nic_attr = nic_attributes.first
+        assert_equal 'net0', nic_attr[:id]
+        assert_equal ip, nic_attr[:ip]
+        assert_equal ip6, nic_attr[:ip6]
+      end
+
+      it 'sets container compute id with identifier, ip/CIDR, gw and ip6' do
+        ip = '192.168.56.100'
+        cidr_suffix = '31'
+        ip6 = Array.new(4) { format('%<x>s', x: rand(16**4)) }.join(':') + '::1'
+        physical_nic = FactoryBot.build(:nic_base_empty, :identifier => 'net0', :ip => ip, :ip6 => ip6, :compute_attributes => { 'cidrv4_prefix' => cidr_suffix, 'gwv4' => ip, 'dhcpv6' => '1' })
+        host = FactoryBot.build(
+          :host_empty,
+          :interfaces => [physical_nic],
+          :compute_attributes => {
+            'type' => 'lxc',
+            'interfaces_attributes' => {
+              '0' => physical_nic
+            }
+          }
+        )
+        nic_attributes = @cr.host_interfaces_attrs(host).values.select(&:present?)
+        nic_attr = nic_attributes.first
+        assert_equal 'net0', nic_attr[:id]
+        assert_equal Fog::Proxmox::IpHelper.to_cidr(ip, cidr_suffix), nic_attr[:ip]
+        assert_equal ip, nic_attr[:gw]
+        assert_equal 'dhcp', nic_attr[:ip6]
+      end
+
+      it 'sets container compute id with identifier, ip DHCP, gw6 and ip6' do
+        ip = '192.168.56.100'
+        cidr6_suffix = '100'
+        ip6 = '2001:0:1234::c1c0:abcd:876'
+        physical_nic = FactoryBot.build(:nic_base_empty, :identifier => 'net0', :ip => ip, :ip6 => ip6, :compute_attributes => { 'cidrv6_prefix' => cidr6_suffix, 'dhcpv4' => '1', 'gwv6' => ip6 })
+        host = FactoryBot.build(
+          :host_empty,
+          :interfaces => [physical_nic],
+          :compute_attributes => {
+            'type' => 'lxc',
+            'interfaces_attributes' => {
+              '0' => physical_nic
+            }
+          }
+        )
+        nic_attributes = @cr.host_interfaces_attrs(host).values.select(&:present?)
+        nic_attr = nic_attributes.first
+        assert_equal 'net0', nic_attr[:id]
+        assert_equal 'dhcp', nic_attr[:ip]
+        assert_equal Fog::Proxmox::IpHelper.to_cidr6(ip6, cidr6_suffix), nic_attr[:ip6]
+        assert_equal ip6, nic_attr[:gw6]
+      end
+    end
+  end
+end

data/test/unit/foreman_fog_proxmox/proxmox_test.rb

@@ -0,0 +1,58 @@
+# frozen_string_literal: true
+
+# Copyright 2018 Tristan Robert
+
+# This file is part of ForemanFogProxmox.
+
+# ForemanFogProxmox is free software: you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation, either version 3 of the License, or
+# (at your option) any later version.
+
+# ForemanFogProxmox is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+
+# You should have received a copy of the GNU General Public License
+# along with ForemanFogProxmox. If not, see <http://www.gnu.org/licenses/>.
+
+require 'test_plugin_helper'
+require 'models/compute_resources/compute_resource_test_helpers'
+require 'factories/foreman_fog_proxmox/proxmox_node_mock_factory'
+require 'factories/foreman_fog_proxmox/proxmox_server_mock_factory'
+require 'factories/foreman_fog_proxmox/proxmox_container_mock_factory'
+require 'active_support/core_ext/hash/indifferent_access'
+
+module ForemanFogProxmox
+  class ProxmoxTest < ActiveSupport::TestCase
+    include ComputeResourceTestHelpers
+    include ProxmoxNodeMockFactory
+    include ProxmoxServerMockFactory
+    include ProxmoxContainerMockFactory
+    include ProxmoxVmHelper
+
+    should validate_presence_of(:url)
+    should validate_presence_of(:user)
+    should validate_presence_of(:password)
+    should allow_value('root@pam').for(:user)
+    should_not allow_value('root').for(:user)
+    should_not allow_value('a').for(:url)
+    should allow_values('http://foo.com', 'http://bar.com/baz').for(:url)
+
+    test '#associated_host matches any NIC' do
+      mac = 'ca:d0:e6:32:16:97'
+      host = FactoryBot.create(:host, :mac => mac)
+      cr = FactoryBot.build_stubbed(:proxmox_cr)
+      vm = mock('vm', :mac => mac)
+      assert_equal host, (as_admin { cr.associated_host(vm) })
+    end
+
+    test '#node' do
+      node = mock('node')
+      cr = FactoryBot.build_stubbed(:proxmox_cr)
+      cr.stubs(:node).returns(node)
+      assert_equal node, (as_admin { cr.node })
+    end
+  end
+end

data/test/unit/foreman_fog_proxmox/proxmox_version_test.rb

@@ -0,0 +1,94 @@
+# frozen_string_literal: true
+
+# Copyright 2018 Tristan Robert
+
+# This file is part of ForemanFogProxmox.
+
+# ForemanFogProxmox is free software: you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation, either version 3 of the License, or
+# (at your option) any later version.
+
+# ForemanFogProxmox is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+
+# You should have received a copy of the GNU General Public License
+# along with ForemanFogProxmox. If not, see <http://www.gnu.org/licenses/>.
+
+require 'test_plugin_helper'
+require 'models/compute_resources/compute_resource_test_helpers'
+require 'active_support/core_ext/hash/indifferent_access'
+
+module ForemanFogProxmox
+  class ProxmoxVersionTest < ActiveSupport::TestCase
+    include ComputeResourceTestHelpers
+    include ProxmoxVmHelper
+
+    wrong_version = { version: '5.a', release: '1' }.with_indifferent_access
+    supported_version = { version: '5.4', release: '3' }.with_indifferent_access
+    supported_6_version = { version: '6.0', release: '1' }.with_indifferent_access
+    supported_6_1_version = { version: '6.1', release: '3' }.with_indifferent_access
+    unsupported_version = { version: '5.2', release: '2' }.with_indifferent_access
+
+    describe 'version' do
+      before do
+        @cr = FactoryBot.build_stubbed(:proxmox_cr)
+        @identity_client = mock('identity_client')
+        @cr.stubs(:identity_client).returns(@identity_client)
+      end
+
+      it 'returns 5.a.1 with 5.a-1' do
+        @identity_client.stubs(:read_version).returns(wrong_version)
+        assert_equal '5.a.1', @cr.version
+      end
+
+      it 'returns 5.4.3 with 5.4-3' do
+        @identity_client.stubs(:read_version).returns(supported_version)
+        assert_equal '5.4.3', @cr.version
+      end
+
+      it 'returns 5.2.2 with 5.2-2' do
+        @identity_client.stubs(:read_version).returns(unsupported_version)
+        assert_equal '5.2.2', @cr.version
+      end
+    end
+
+    describe 'version_suitable?' do
+      before do
+        @cr = FactoryBot.build_stubbed(:proxmox_cr)
+        @identity_client = mock('identity_client')
+        @cr.stubs(:identity_client).returns(@identity_client)
+      end
+
+      it 'raises error with 5.a.1' do
+        @identity_client.stubs(:read_version).returns(wrong_version)
+        err = assert_raises Foreman::Exception do
+          @cr.version_suitable?
+        end
+        assert err.message.end_with?('Proxmox version 5.a.1 is not semver suitable')
+      end
+
+      it 'is true with 5.4-3' do
+        @identity_client.stubs(:read_version).returns(supported_version)
+        assert @cr.version_suitable?
+      end
+
+      it 'is true with 6.0-1' do
+        @identity_client.stubs(:read_version).returns(supported_6_version)
+        assert @cr.version_suitable?
+      end
+
+      it 'is true with 6.1-3' do
+        @identity_client.stubs(:read_version).returns(supported_6_1_version)
+        assert @cr.version_suitable?
+      end
+
+      it 'is false with 5.2-2' do
+        @identity_client.stubs(:read_version).returns(unsupported_version)
+        assert_not @cr.version_suitable?
+      end
+    end
+  end
+end

data/test/unit/foreman_fog_proxmox/proxmox_vm_commands_container_test.rb

@@ -0,0 +1,280 @@
+# frozen_string_literal: true
+
+# Copyright 2018 Tristan Robert
+
+# This file is part of ForemanFogProxmox.
+
+# ForemanFogProxmox is free software: you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation, either version 3 of the License, or
+# (at your option) any later version.
+
+# ForemanFogProxmox is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+
+# You should have received a copy of the GNU General Public License
+# along with ForemanFogProxmox. If not, see <http://www.gnu.org/licenses/>.
+
+require 'test_plugin_helper'
+require 'models/compute_resources/compute_resource_test_helpers'
+require 'factories/foreman_fog_proxmox/proxmox_node_mock_factory'
+require 'factories/foreman_fog_proxmox/proxmox_container_mock_factory'
+require 'active_support/core_ext/hash/indifferent_access'
+
+module ForemanFogProxmox
+  class ProxmoxVmCommandsContainerTest < ActiveSupport::TestCase
+    include ComputeResourceTestHelpers
+    include ProxmoxNodeMockFactory
+    include ProxmoxContainerMockFactory
+    include ProxmoxVmHelper
+
+    describe 'save_vm' do
+      before do
+        @cr = FactoryBot.build_stubbed(:proxmox_cr)
+      end
+
+      it 'migrates container from one node to another in the cluster' do
+        uuid = '100'
+        config = mock('config')
+        config.stubs(:attributes).returns(:cores => '')
+        vm = mock('vm')
+        vm.stubs(:config).returns(config)
+        vm.stubs(:container?).returns(true)
+        vm.stubs(:type).returns('lxc')
+        vm.stubs(:node_id).returns('pve')
+        @cr.stubs(:find_vm_by_uuid).returns(vm)
+        attr = { 'templated' => '0', 'node_id' => 'pve', 'config_attributes' => { 'cores' => '1', 'cpulimit' => '1', 'onboot' => '0' } }.with_indifferent_access
+        @cr.stubs(:parse_container_vm).returns('vmid' => '100', 'node_id' => 'pve2', 'type' => 'lxc', 'cores' => '1', 'cpulimit' => '1')
+        expected_attr = { :cores => '1', :cpulimit => '1' }
+        vm.expects(:update, expected_attr)
+        @cr.save_vm(uuid, attr)
+      end
+
+      it 'saves modified container config' do
+        uuid = '100'
+        config = mock('config')
+        config.stubs(:attributes).returns(:cores => '')
+        vm = mock('vm')
+        vm.stubs(:config).returns(config)
+        vm.stubs(:container?).returns(true)
+        vm.stubs(:type).returns('lxc')
+        vm.stubs(:node_id).returns('pve')
+        @cr.stubs(:find_vm_by_uuid).returns(vm)
+        attr = { 'templated' => '0', 'node_id' => 'pve', 'config_attributes' => { 'cores' => '1', 'cpulimit' => '1', 'onboot' => '0' } }.with_indifferent_access
+        @cr.stubs(:parse_container_vm).returns('vmid' => '100', 'node_id' => 'pve', 'type' => 'lxc', 'cores' => '1', 'cpulimit' => '1')
+        expected_attr = { :cores => '1', :cpulimit => '1' }
+        vm.expects(:update, expected_attr)
+        @cr.save_vm(uuid, attr)
+      end
+
+      it 'saves modified container config with added volumes' do
+        uuid = '100'
+        config = mock('config')
+        disks = mock('disks')
+        disk = mock('disk')
+        disk.stubs(:size).returns(1_073_741_824)
+        disk.stubs(:storage).returns('local-lvm')
+        disk.stubs(:id).returns('mp0')
+        disks.stubs(:get).returns
+        config.stubs(:disks).returns(disks)
+        config.stubs(:attributes).returns(:cores => '')
+        vm = mock('vm')
+        vm.stubs(:config).returns(config)
+        vm.stubs(:container?).returns(true)
+        vm.stubs(:type).returns('lxc')
+        vm.stubs(:node_id).returns('pve')
+        @cr.stubs(:find_vm_by_uuid).returns(vm)
+        new_attributes = {
+          'templated' => '0',
+          'node_id' => 'pve',
+          'config_attributes' => {
+            'cores' => '1',
+            'cpulimit' => '1'
+          },
+          'volumes_attributes' => {
+            '0' => {
+              'id' => 'mp0',
+              '_delete' => '',
+              'device' => '0',
+              'storage' => 'local-lvm',
+              'size' => '2147483648',
+              'cache' => 'none',
+              'mp' => '/opt/path'
+            }
+          }
+        }.with_indifferent_access
+        @cr.stubs(:parse_container_vm).returns('vmid' => '100', 'node_id' => 'pve', 'type' => 'lxc', 'cores' => '1', 'cpulimit' => '1')
+        expected_config_attr = { :cores => '1', :cpulimit => '1' }
+        expected_volume_attr =
+          [
+            {
+              id: 'mp0',
+              storage: 'local:lvm',
+              size: (2_147_483_648 / GIGA).to_s
+            },
+            {
+              mp: '/opt/path'
+            }
+          ]
+        vm.expects(:attach, expected_volume_attr)
+        vm.expects(:update, expected_config_attr)
+        @cr.save_vm(uuid, new_attributes)
+      end
+
+      it 'saves modified container config with resized volumes' do
+        uuid = '100'
+        config = mock('config')
+        disks = mock('disks')
+        disk = mock('disk')
+        disk.stubs(:size).returns(1_073_741_824)
+        disk.stubs(:storage).returns('local-lvm')
+        disk.stubs(:id).returns('rootfs')
+        disks.stubs(:get).returns(disk)
+        config.stubs(:disks).returns(disks)
+        config.stubs(:attributes).returns(:cores => '')
+        vm = mock('vm')
+        vm.stubs(:config).returns(config)
+        vm.stubs(:container?).returns(true)
+        vm.stubs(:type).returns('lxc')
+        vm.stubs(:node_id).returns('pve')
+        @cr.stubs(:find_vm_by_uuid).returns(vm)
+        new_attributes =
+          {
+            'templated' => '0',
+            'node_id' => 'pve',
+            'config_attributes' => {
+              'cores' => '1',
+              'cpulimit' => '1',
+              'onboot' => '0'
+            },
+            'volumes_attributes' => {
+              '0' => {
+                'id' => 'rootfs',
+                '_delete' => '',
+                'volid' => 'local-lvm:1073741824',
+                'device' => '0',
+                'storage' => 'local-lvm',
+                'size' => '2147483648',
+                'cache' => 'none'
+              }
+            }
+          }.with_indifferent_access
+        @cr.stubs(:parse_container_vm).returns(
+          'vmid' => '100',
+          'node_id' => 'pve',
+          'type' => 'lxc',
+          'cores' => '1',
+          'cpulimit' => '1'
+        )
+        expected_config_attr = { :cores => '1', :cpulimit => '1' }
+        expected_volume_attr = ['rootfs', '+1G']
+        vm.expects(:extend, expected_volume_attr)
+        vm.expects(:update, expected_config_attr)
+        @cr.save_vm(uuid, new_attributes)
+      end
+
+      it 'saves modified container config with modified volumes options' do
+        uuid = '100'
+        config = mock('config')
+        disks = mock('disks')
+        disk = mock('disk')
+        disk.stubs(:rootfs?).returns(false)
+        disk.stubs(:volid).returns('local-lvm:vm-100-disk-0')
+        disk.stubs(:id).returns('mp0')
+        disk.stubs(:size).returns(1_073_741_824)
+        disk.stubs(:storage).returns('local-lvm')
+        disks.stubs(:get).returns(disk)
+        config.stubs(:disks).returns(disks)
+        config.stubs(:attributes).returns(:cores => '')
+        vm = mock('vm')
+        vm.stubs(:config).returns(config)
+        vm.stubs(:container?).returns(true)
+        vm.stubs(:type).returns('lxc')
+        vm.stubs(:node_id).returns('pve')
+        @cr.stubs(:find_vm_by_uuid).returns(vm)
+        new_attributes = {
+          'templated' => '0',
+          'node_id' => 'pve',
+          'config_attributes' => {
+            'cores' => '1',
+            'cpulimit' => '1'
+          },
+          'volumes_attributes' => {
+            '0' => {
+              'id' => 'mp0',
+              '_delete' => '',
+              'volid' => 'local-lvm:vm-100-disk-0',
+              'device' => '0',
+              'controller' => 'mp',
+              'storage' => 'local-lvm',
+              'size' => '1073741824',
+              'mp' => '/opt/toto'
+            }
+          }
+        }.with_indifferent_access
+        @cr.stubs(:parse_server_vm).returns('vmid' => '100', 'node_id' => 'pve', 'type' => 'qemu', 'cores' => '1', 'cpulimit' => '1')
+        expected_config_attr = { :cores => '1', :cpulimit => '1' }
+        expected_volume_attr = { :id => 'mp0', :volid => 'local-lvm:vm-100-disk-0', :size => 1_073_741_824 }, { :mp => '/opt/toto' }
+        vm.expects(:attach, expected_volume_attr)
+        vm.expects(:update, expected_config_attr)
+        @cr.save_vm(uuid, new_attributes)
+      end
+    end
+
+    describe 'create_vm' do
+      it 'creates container without bootstart' do
+        args = { vmid: '100', type: 'lxc', node_id: 'pve', start_after_create: '0' }
+        servers = mock('servers')
+        servers.stubs(:id_valid?).returns(true)
+        containers = mock('containers')
+        containers.stubs(:create).with(vmid: 100, type: 'lxc', node_id: 'pve', start_after_create: '0')
+        cr = mock_node_servers_containers(ForemanFogProxmox::Proxmox.new, servers, containers)
+        cr.stubs(:convert_sizes).with(args)
+        cr.stubs(:parse_container_vm).with(args).returns(args)
+        vm = mock('vm')
+        cr.stubs(:find_vm_by_uuid).with((args[:vmid]).to_s).returns(vm)
+        cr.create_vm(args)
+      end
+
+      it 'creates container with bootstart' do
+        args = { vmid: '100', type: 'lxc', node_id: 'pve', start_after_create: '1' }
+        servers = mock('servers')
+        servers.stubs(:id_valid?).returns(true)
+        containers = mock('containers')
+        vm = mock('vm')
+        containers.stubs(:create).with(vmid: 100, type: 'lxc', node_id: 'pve', start_after_create: '1').returns(vm)
+        cr = mock_node_servers_containers(ForemanFogProxmox::Proxmox.new, servers, containers)
+        cr.stubs(:convert_sizes).with(args)
+        cr.stubs(:parse_container_vm).with(args).returns(args)
+        cr.stubs(:find_vm_by_uuid).with((args[:vmid]).to_s).returns(vm)
+        cr.stubs(:start_on_boot).with(vm, args).returns(vm)
+        cr.create_vm(args)
+      end
+
+      it 'creates container within pool' do
+        args = { vmid: '100', type: 'lxc', node_id: 'pve', start_after_create: '0', pool: 'pool1' }
+        servers = mock('servers')
+        servers.stubs(:id_valid?).returns(true)
+        containers = mock('containers')
+        vm = mock('vm')
+        containers.stubs(:create).with(vmid: 100, type: 'lxc', node_id: 'pve', start_after_create: '0', pool: 'pool1').returns(vm)
+        cr = mock_node_servers_containers(ForemanFogProxmox::Proxmox.new, servers, containers)
+        cr.stubs(:convert_sizes).with(args)
+        cr.stubs(:parse_container_vm).with(args).returns(args)
+        cr.stubs(:find_vm_by_uuid).with((args[:vmid]).to_s).returns(vm)
+        cr.create_vm(args)
+      end
+
+      it 'clones container' do
+        args = { vmid: '100', type: 'lxc', image_id: '999', name: 'name' }
+        servers = mock('servers')
+        servers.stubs(:id_valid?).returns(true)
+        cr = mock_node_servers(ForemanFogProxmox::Proxmox.new, servers)
+        cr.expects(:clone_from_image).with('999', args, 100)
+        cr.create_vm(args)
+      end
+    end
+  end
+end