foreman_fog_proxmox 0.14.0 → 0.14.2

data/test/unit/foreman_fog_proxmox/proxmox_vm_commands_server_update_hard_disk_test.rb

@@ -24,7 +24,7 @@ require 'factories/foreman_fog_proxmox/proxmox_server_mock_factory'
 require 'active_support/core_ext/hash/indifferent_access'
 
 module ForemanFogProxmox
-  class ProxmoxVmCommandsServerUpdateTest < ActiveSupport::TestCase
+  class ProxmoxVmCommandsServerUpdateHardDiskTest < ActiveSupport::TestCase
     include ComputeResourceTestHelpers
     include ProxmoxNodeMockFactory
     include ProxmoxServerMockFactory
@@ -40,10 +40,12 @@ module ForemanFogProxmox
         config = mock('config')
         disks = mock('disks')
         disk = mock('disk')
-        disk.stubs(:size).returns(1_073_741_824)
+        disk.stubs(:size).returns('1')
+        disk.stubs(:hard_disk?).returns(true)
+        disk.stubs(:cdrom?).returns(false)
         disk.stubs(:storage).returns('local-lvm')
         disk.stubs(:id).returns('scsi0')
-        disk.stubs(:attributes).returns(id: 'scsi0', storage: 'local-lvm', size: 1_073_741_824)
+        disk.stubs(:attributes).returns(id: 'scsi0', storage: 'local-lvm', size: '1')
         disks.stubs(:get).returns
         config.stubs(:disks).returns(disks)
         config.stubs(:attributes).returns(:cores => '')
@@ -60,7 +62,7 @@ module ForemanFogProxmox
           'node_id' => 'proxmox',
           'config_attributes' => {
             'cores' => '1',
-            'cpulimit' => '1'
+            'cpulimit' => '1',
           },
           'volumes_attributes' => {
             '0' => {
@@ -70,14 +72,15 @@ module ForemanFogProxmox
               'controller' => 'scsi',
               'storage_type' => 'hard_disk',
               'storage' => 'local-lvm',
-              'size' => '2147483648',
-              'cache' => 'none'
-            }
-          }
+              'size' => '2',
+              'cache' => 'none',
+            },
+          },
         }.with_indifferent_access
-        @cr.stubs(:parse_server_vm).returns('vmid' => '100', 'node_id' => 'proxmox', 'type' => 'qemu', 'cores' => '1', 'cpulimit' => '1', 'onboot' => '0')
+        @cr.stubs(:parse_server_vm).returns('vmid' => '100', 'node_id' => 'proxmox', 'type' => 'qemu', 'cores' => '1',
+          'cpulimit' => '1', 'onboot' => '0')
         expected_config_attr = { :cores => '1', :cpulimit => '1' }
-        expected_volume_attr = { id: 'scsi0', storage: 'local:lvm', size: (2_147_483_648 / GIGA).to_s }
+        expected_volume_attr = { id: 'scsi0', storage: 'local:lvm', size: '2' }
         vm.expects(:attach, expected_volume_attr)
         vm.expects(:update, expected_config_attr)
         @cr.save_vm(uuid, new_attributes)
@@ -88,7 +91,9 @@ module ForemanFogProxmox
         config = mock('config')
         disks = mock('disks')
         disk = mock('disk')
-        disk.stubs(:size).returns(1_073_741_824)
+        disk.stubs(:size).returns('1')
+        disk.stubs(:hard_disk?).returns(true)
+        disk.stubs(:cdrom?).returns(false)
         disk.stubs(:storage).returns('local-lvm')
         disk.stubs(:id).returns('virtio0')
         disks.stubs(:get).returns(disk)
@@ -108,7 +113,7 @@ module ForemanFogProxmox
           'node_id' => 'proxmox',
           'config_attributes' => {
             'cores' => '1',
-            'cpulimit' => '1'
+            'cpulimit' => '1',
           },
           'volumes_attributes' => {
             '0' => {
@@ -119,12 +124,13 @@ module ForemanFogProxmox
               'device' => '0',
               'controller' => 'scsi',
               'storage' => 'local-lvm',
-              'size' => '2147483648',
-              'cache' => 'none'
-            }
-          }
+              'size' => '2',
+              'cache' => 'none',
+            },
+          },
         }.with_indifferent_access
-        @cr.stubs(:parse_server_vm).returns('vmid' => '100', 'node_id' => 'proxmox', 'type' => 'qemu', 'cores' => '1', 'cpulimit' => '1', 'config_attributes' => { 'onboot' => '0' })
+        @cr.stubs(:parse_server_vm).returns('vmid' => '100', 'node_id' => 'proxmox', 'type' => 'qemu', 'cores' => '1',
+          'cpulimit' => '1', 'config_attributes' => { 'onboot' => '0' })
         expected_config_attr = { :cores => '1', :cpulimit => '1' }
         expected_volume_attr = 'scsi0'
         vm.expects(:detach, expected_volume_attr)
@@ -138,10 +144,13 @@ module ForemanFogProxmox
         config = mock('config')
         disks = mock('disks')
         disk = mock('disk')
-        disk.stubs(:size).returns(1_073_741_824)
+        disk.stubs(:size).returns('1')
+        disk.stubs(:hard_disk?).returns(true)
+        disk.stubs(:cdrom?).returns(false)
         disk.stubs(:storage).returns('scsi0')
+        disk.stubs(:volid).returns('local-lvm:vm-100-disk-0')
         disk.stubs(:id).returns('local-lvm')
-        disk.stubs(:attributes).returns(id: 'scsi0', storage: 'local-lvm', size: 1_073_741_824)
+        disk.stubs(:attributes).returns(id: 'scsi0', storage: 'local-lvm', size: 1)
         disks.stubs(:get).returns(disk)
         config.stubs(:disks).returns(disks)
         config.stubs(:attributes).returns(:cores => '')
@@ -159,7 +168,7 @@ module ForemanFogProxmox
           'node_id' => 'proxmox',
           'config_attributes' => {
             'cores' => '1',
-            'cpulimit' => '1'
+            'cpulimit' => '1',
           },
           'volumes_attributes' => {
             '0' => {
@@ -170,12 +179,13 @@ module ForemanFogProxmox
               'device' => '0',
               'controller' => 'scsi',
               'storage' => 'local-lvm',
-              'size' => '2147483648',
-              'cache' => 'none'
-            }
-          }
+              'size' => '2',
+              'cache' => 'none',
+            },
+          },
         }.with_indifferent_access
-        @cr.stubs(:parse_server_vm).returns('vmid' => '100', 'node_id' => 'proxmox', 'type' => 'qemu', 'cores' => '1', 'cpulimit' => '1')
+        @cr.stubs(:parse_server_vm).returns('vmid' => '100', 'node_id' => 'proxmox', 'type' => 'qemu', 'cores' => '1',
+          'cpulimit' => '1')
         expected_config_attr = { :cores => '1', :cpulimit => '1' }
         expected_volume_attr = ['scsi0', '+1G']
         vm.expects(:extend, expected_volume_attr)
@@ -189,26 +199,30 @@ module ForemanFogProxmox
         disks = mock('disks')
         disk = mock('disk')
         disk.stubs(:id).returns('scsi0')
-        disk.stubs(:size).returns(1_073_741_824)
+        disk.stubs(:size).returns('2')
+        disk.stubs(:hard_disk?).returns(true)
+        disk.stubs(:cdrom?).returns(false)
         disk.stubs(:storage).returns('local-lvm')
-        disk.stubs(:attributes).returns(id: 'scsi0', storage: 'local-lvm', size: 1_073_741_824)
+        disk.stubs(:volid).returns('local-lvm:vm-100-disk-0')
+        disk.stubs(:attributes).returns(id: 'scsi0', storage: 'local-lvm', size: '2')
         disks.stubs(:get).returns(disk)
         config.stubs(:disks).returns(disks)
-        config.stubs(:attributes).returns(:cores => '')
+        config.stubs(:attributes).returns(:cores => '1')
         vm = mock('vm')
         vm.stubs(:identity).returns(uuid)
         vm.stubs(:attributes).returns('scsi0' => '')
         vm.stubs(:config).returns(config)
         vm.stubs(:container?).returns(false)
+        vm.stubs(:templated?).returns(false)
         vm.stubs(:type).returns('qemu')
-        vm.stubs(:node_id).returns('proxmox')
+        vm.stubs(:node_id).returns('pve')
         @cr.stubs(:find_vm_by_uuid).returns(vm)
         new_attributes = {
           'templated' => '0',
-          'node_id' => 'proxmox',
+          'node_id' => 'pve',
           'config_attributes' => {
             'cores' => '1',
-            'cpulimit' => '1'
+            'cpulimit' => '1',
           },
           'volumes_attributes' => {
             '0' => {
@@ -219,12 +233,12 @@ module ForemanFogProxmox
               'device' => '0',
               'controller' => 'scsi',
               'storage' => 'local-lvm',
-              'size' => '2',
-              'cache' => 'none'
-            }
-          }
+              'size' => '1',
+              'cache' => 'none',
+            },
+          },
         }.with_indifferent_access
-        @cr.stubs(:parse_server_vm).returns('vmid' => '100', 'node_id' => 'proxmox', 'type' => 'qemu', 'cores' => '1', 'cpulimit' => '1')
+        @cr.stubs(:parse_server_vm).returns('vmid' => '100', 'node_id' => 'pve', 'type' => 'qemu', 'cores' => '1', 'cpulimit' => '1')
         err = assert_raises Foreman::Exception do
           @cr.save_vm(uuid, new_attributes)
         end
@@ -237,9 +251,13 @@ module ForemanFogProxmox
         disks = mock('disks')
         disk = mock('disk')
         disk.stubs(:id).returns('scsi0')
-        disk.stubs(:size).returns(1_073_741_824)
+        disk.stubs(:size).returns('1')
+        disk.stubs(:hard_disk?).returns(true)
+        disk.stubs(:cdrom?).returns(false)
+        disk.stubs(:hard_disk?).returns(true)
+        disk.stubs(:cdrom?).returns(false)
         disk.stubs(:storage).returns('local-lvm')
-        disk.stubs(:attributes).returns(id: 'scsi0', storage: 'local-lvm', size: 1_073_741_824)
+        disk.stubs(:attributes).returns(id: 'scsi0', storage: 'local-lvm', size: '1', volid: 'local-lvm:vm-100-disk-0')
         disks.stubs(:get).returns(disk)
         config.stubs(:disks).returns(disks)
         config.stubs(:attributes).returns(:cores => '')
@@ -256,23 +274,24 @@ module ForemanFogProxmox
           'node_id' => 'proxmox',
           'config_attributes' => {
             'cores' => '1',
-            'cpulimit' => '1'
+            'cpulimit' => '1',
           },
           'volumes_attributes' => {
             '0' => {
               'id' => 'scsi0',
               '_delete' => '',
               'storage_type' => 'hard_disk',
-              'volid' => 'local-lvm:vm-100-disk-0',
+              'volid' => 'local-lvm2:vm-100-disk-0',
               'device' => '0',
               'controller' => 'scsi',
               'storage' => 'local-lvm2',
-              'size' => '1073741824',
-              'cache' => 'none'
-            }
-          }
+              'size' => '1',
+              'cache' => 'none',
+            },
+          },
         }.with_indifferent_access
-        @cr.stubs(:parse_server_vm).returns('vmid' => '100', 'node_id' => 'proxmox', 'type' => 'qemu', 'cores' => '1', 'cpulimit' => '1')
+        @cr.stubs(:parse_server_vm).returns('vmid' => '100', 'node_id' => 'proxmox', 'type' => 'qemu', 'cores' => '1',
+          'cpulimit' => '1')
         expected_config_attr = { :cores => '1', :cpulimit => '1' }
         expected_volume_attr = ['scsi0', 'local-lvm2']
         vm.expects(:move, expected_volume_attr)
@@ -285,11 +304,13 @@ module ForemanFogProxmox
         config = mock('config')
         disks = mock('disks')
         disk = mock('disk')
-        disk.stubs(:size).returns(1_073_741_824)
+        disk.stubs(:size).returns('1')
+        disk.stubs(:hard_disk?).returns(true)
+        disk.stubs(:cdrom?).returns(false)
         disk.stubs(:storage).returns('local-lvm')
         disk.stubs(:volid).returns('local-lvm:vm-100-disk-0')
         disk.stubs(:id).returns('scsi0')
-        disk.stubs(:attributes).returns(id: 'scsi0', storage: 'local-lvm', volid: 'local-lvm:vm-100-disk-0', size: 1_073_741_824)
+        disk.stubs(:attributes).returns(id: 'scsi0', storage: 'local-lvm', volid: 'local-lvm:vm-100-disk-0', size: '1')
         disks.stubs(:get).returns(disk)
         config.stubs(:disks).returns(disks)
         config.stubs(:attributes).returns(:cores => '')
@@ -306,7 +327,7 @@ module ForemanFogProxmox
           'node_id' => 'proxmox',
           'config_attributes' => {
             'cores' => '1',
-            'cpulimit' => '1'
+            'cpulimit' => '1',
           },
           'volumes_attributes' => {
             '0' => {
@@ -317,14 +338,15 @@ module ForemanFogProxmox
               'device' => '0',
               'controller' => 'scsi',
               'storage' => 'local-lvm',
-              'size' => '1073741824',
-              'cache' => 'directsync'
-            }
-          }
+              'size' => '1',
+              'cache' => 'directsync',
+            },
+          },
         }.with_indifferent_access
-        @cr.stubs(:parse_server_vm).returns('vmid' => '100', 'node_id' => 'proxmox', 'type' => 'qemu', 'cores' => '1', 'cpulimit' => '1')
+        @cr.stubs(:parse_server_vm).returns('vmid' => '100', 'node_id' => 'proxmox', 'type' => 'qemu', 'cores' => '1',
+          'cpulimit' => '1')
         expected_config_attr = { :cores => '1', :cpulimit => '1' }
-        expected_volume_attr = { :id => 'scsi0', :volid => 'local-lvm:vm-100-disk-0', :size => 1_073_741_824 }, { :cache => 'directsync' }
+        expected_volume_attr = { :id => 'scsi0', :volid => 'local-lvm:vm-100-disk-0', :size => 1 }, { :cache => 'directsync' }
         vm.expects(:attach, expected_volume_attr)
         vm.expects(:update, expected_config_attr)
         @cr.save_vm(uuid, new_attributes)

data/test/unit/foreman_fog_proxmox/proxmox_vm_commands_server_update_test.rb

@@ -62,8 +62,10 @@ module ForemanFogProxmox
         vm.stubs(:type).returns('qemu')
         vm.stubs(:node_id).returns('proxmox')
         @cr.stubs(:find_vm_by_uuid).returns(vm)
-        attr = { 'templated' => '0', 'node_id' => 'proxmox', 'config_attributes' => { 'cores' => '1', 'cpulimit' => '1', 'onboot' => '0' } }.with_indifferent_access
-        @cr.stubs(:parse_typed_vm).returns('vmid' => '100', 'node_id' => 'proxmox', 'type' => 'qemu', 'cores' => '1', 'cpulimit' => '1', 'onboot' => '0')
+        attr = { 'templated' => '0', 'node_id' => 'proxmox',
+                 'config_attributes' => { 'cores' => '1', 'cpulimit' => '1', 'onboot' => '0' } }.with_indifferent_access
+        @cr.stubs(:parse_typed_vm).returns('vmid' => '100', 'node_id' => 'proxmox', 'type' => 'qemu', 'cores' => '1',
+          'cpulimit' => '1', 'onboot' => '0')
         expected_attr = { :cores => '1', :cpulimit => '1' }.with_indifferent_access
         vm.expects(:update, expected_attr)
         @cr.save_vm(uuid, attr)
@@ -104,14 +106,14 @@ module ForemanFogProxmox
           'node_id' => 'proxmox',
           'config_attributes' => {
             'cores' => '1',
-            'cpulimit' => '1'
+            'cpulimit' => '1',
           },
           'interfaces_attributes' => {
             '0' => {
               '_delete' => '1',
-              'id' => 'net0'
-            }
-          }
+              'id' => 'net0',
+            },
+          },
         }.with_indifferent_access
         @cr.stubs(:parse_typed_vm).returns(
           'vmid' => '100',
@@ -154,9 +156,12 @@ module ForemanFogProxmox
         pools.expects(:get).with('pool2').returns(pool2)
         identity_client.stubs(:pools).returns(pools)
         @cr.stubs(:identity_client).returns(identity_client)
-        attr = { 'templated' => '0', 'node_id' => 'proxmox', 'pool' => 'pool2', 'config_attributes' => { 'cores' => '1', 'cpulimit' => '1', 'onboot' => '0' } }.with_indifferent_access
-        @cr.stubs(:parse_typed_vm).returns('vmid' => '100', 'node_id' => 'proxmox', 'type' => 'qemu', 'cores' => '1', 'cpulimit' => '1', 'onboot' => '0', 'pool' => 'pool2')
-        vm.expects(:update).with({ 'node_id' => 'proxmox', 'type' => 'qemu', 'cores' => '1', 'cpulimit' => '1', 'onboot' => '0' }.with_indifferent_access)
+        attr = { 'templated' => '0', 'node_id' => 'proxmox', 'pool' => 'pool2',
+                 'config_attributes' => { 'cores' => '1', 'cpulimit' => '1', 'onboot' => '0' } }.with_indifferent_access
+        @cr.stubs(:parse_typed_vm).returns('vmid' => '100', 'node_id' => 'proxmox', 'type' => 'qemu', 'cores' => '1',
+          'cpulimit' => '1', 'onboot' => '0', 'pool' => 'pool2')
+        vm.expects(:update).with({ 'node_id' => 'proxmox', 'type' => 'qemu', 'cores' => '1', 'cpulimit' => '1',
+                                   'onboot' => '0' }.with_indifferent_access)
         @cr.save_vm(uuid, attr)
       end
 
@@ -183,9 +188,12 @@ module ForemanFogProxmox
         pools.expects(:get).with('').returns(nil)
         identity_client.stubs(:pools).returns(pools)
         @cr.stubs(:identity_client).returns(identity_client)
-        attr = { 'templated' => '0', 'node_id' => 'proxmox', 'pool' => '', 'config_attributes' => { 'cores' => '1', 'cpulimit' => '1', 'onboot' => '0' } }.with_indifferent_access
-        @cr.stubs(:parse_typed_vm).returns('vmid' => '100', 'node_id' => 'proxmox', 'type' => 'qemu', 'cores' => '1', 'cpulimit' => '1', 'onboot' => '0', 'pool' => '')
-        vm.expects(:update).with({ 'node_id' => 'proxmox', 'type' => 'qemu', 'cores' => '1', 'cpulimit' => '1', 'onboot' => '0' }.with_indifferent_access)
+        attr = { 'templated' => '0', 'node_id' => 'proxmox', 'pool' => '',
+                 'config_attributes' => { 'cores' => '1', 'cpulimit' => '1', 'onboot' => '0' } }.with_indifferent_access
+        @cr.stubs(:parse_typed_vm).returns('vmid' => '100', 'node_id' => 'proxmox', 'type' => 'qemu', 'cores' => '1',
+          'cpulimit' => '1', 'onboot' => '0', 'pool' => '')
+        vm.expects(:update).with({ 'node_id' => 'proxmox', 'type' => 'qemu', 'cores' => '1', 'cpulimit' => '1',
+                                   'onboot' => '0' }.with_indifferent_access)
         @cr.save_vm(uuid, attr)
       end
 
@@ -212,9 +220,12 @@ module ForemanFogProxmox
         pools.expects(:get).with('').returns(nil)
         identity_client.stubs(:pools).returns(pools)
         @cr.stubs(:identity_client).returns(identity_client)
-        attr = { 'templated' => '0', 'node_id' => 'proxmox', 'pool' => 'pool2', 'config_attributes' => { 'cores' => '1', 'cpulimit' => '1', 'onboot' => '0' } }.with_indifferent_access
-        @cr.stubs(:parse_typed_vm).returns('vmid' => '100', 'node_id' => 'proxmox', 'type' => 'qemu', 'cores' => '1', 'cpulimit' => '1', 'onboot' => '0', 'pool' => 'pool2')
-        vm.expects(:update).with({ 'node_id' => 'proxmox', 'type' => 'qemu', 'cores' => '1', 'cpulimit' => '1', 'onboot' => '0' }.with_indifferent_access)
+        attr = { 'templated' => '0', 'node_id' => 'proxmox', 'pool' => 'pool2',
+                 'config_attributes' => { 'cores' => '1', 'cpulimit' => '1', 'onboot' => '0' } }.with_indifferent_access
+        @cr.stubs(:parse_typed_vm).returns('vmid' => '100', 'node_id' => 'proxmox', 'type' => 'qemu', 'cores' => '1',
+          'cpulimit' => '1', 'onboot' => '0', 'pool' => 'pool2')
+        vm.expects(:update).with({ 'node_id' => 'proxmox', 'type' => 'qemu', 'cores' => '1', 'cpulimit' => '1',
+                                   'onboot' => '0' }.with_indifferent_access)
         @cr.save_vm(uuid, attr)
       end
     end

data/test/unit/foreman_fog_proxmox/proxmox_vm_queries_test.rb

@@ -35,7 +35,7 @@ module ForemanFogProxmox
     describe 'find_vm_by_uuid' do
       it 'returns nil when the uuid does not match' do
         cr = mock_node_servers_containers(ForemanFogProxmox::Proxmox.new, empty_servers, empty_servers)
-        assert cr.find_vm_by_uuid('1_100').nil?
+        assert_nil cr.find_vm_by_uuid('1_100')
       end
 
       it 'raises RecordNotFound when the compute raises error' do

data/test/unit/foreman_fog_proxmox/semver_test.rb

@@ -77,7 +77,8 @@ module ForemanFogProxmox
         assert ForemanFogProxmox::Semver.to_semver('1.0.10') <= ForemanFogProxmox::Semver.to_semver('1.0.20')
       end
       it '#1.2.3-beta == 1.2.3-beta returns true' do
-        assert_equal ForemanFogProxmox::Semver.to_semver('1.2.3-beta'), ForemanFogProxmox::Semver.to_semver('1.2.3-beta')
+        assert_equal ForemanFogProxmox::Semver.to_semver('1.2.3-beta'),
+          ForemanFogProxmox::Semver.to_semver('1.2.3-beta')
       end
       it '#1.2.3-beta >= 1.-beta raises ArgumentError' do
         assert_raises ArgumentError do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: foreman_fog_proxmox
 version: !ruby/object:Gem::Version
-  version: 0.14.0
+  version: 0.14.2
 platform: ruby
 authors:
 - Tristan Robert
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-07-23 00:00:00.000000000 Z
+date: 2022-12-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: deface
@@ -31,14 +31,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.14'
+        version: '0.15'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.14'
+        version: '0.15'
 - !ruby/object:Gem::Dependency
   name: rdoc
   requirement: !ruby/object:Gem::Requirement
@@ -81,6 +81,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: theforeman-rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.1'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.1'
 description: Foreman plugin adds Proxmox VE compute resource using fog-proxmox. It
   is compatible with Foreman 1.22+
 email:
@@ -243,34 +257,34 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.22
+rubygems_version: 3.1.6
 signing_key: 
 specification_version: 4
 summary: Foreman plugin that adds Proxmox VE compute resource using fog-proxmox
 test_files:
-- test/factories/foreman_fog_proxmox/proxmox_container_mock_factory.rb
+- test/test_plugin_helper.rb
 - test/factories/foreman_fog_proxmox/proxmox_node_mock_factory.rb
 - test/factories/foreman_fog_proxmox/proxmox_server_mock_factory.rb
+- test/factories/foreman_fog_proxmox/proxmox_container_mock_factory.rb
 - test/factories/proxmox_factory.rb
-- test/functional/compute_resources_controller_test.rb
-- test/test_plugin_helper.rb
-- test/unit/foreman_fog_proxmox/helpers/proxmox_container_helper_test.rb
+- test/unit/foreman_fog_proxmox/proxmox_vm_commands_server_create_test.rb
+- test/unit/foreman_fog_proxmox/helpers/proxmox_vm_volumes_helper_test.rb
 - test/unit/foreman_fog_proxmox/helpers/proxmox_server_helper_test.rb
 - test/unit/foreman_fog_proxmox/helpers/proxmox_vm_helper_test.rb
 - test/unit/foreman_fog_proxmox/helpers/proxmox_vm_uuid_helper_test.rb
-- test/unit/foreman_fog_proxmox/helpers/proxmox_vm_volumes_helper_test.rb
-- test/unit/foreman_fog_proxmox/proxmox_compute_attributes_test.rb
-- test/unit/foreman_fog_proxmox/proxmox_images_test.rb
-- test/unit/foreman_fog_proxmox/proxmox_interfaces_test.rb
-- test/unit/foreman_fog_proxmox/proxmox_test.rb
+- test/unit/foreman_fog_proxmox/helpers/proxmox_container_helper_test.rb
 - test/unit/foreman_fog_proxmox/proxmox_version_test.rb
-- test/unit/foreman_fog_proxmox/proxmox_vm_commands_container_test.rb
-- test/unit/foreman_fog_proxmox/proxmox_vm_commands_server_create_test.rb
 - test/unit/foreman_fog_proxmox/proxmox_vm_commands_server_update_cdrom_test.rb
-- test/unit/foreman_fog_proxmox/proxmox_vm_commands_server_update_cloudinit_test.rb
+- test/unit/foreman_fog_proxmox/proxmox_vm_commands_container_test.rb
+- test/unit/foreman_fog_proxmox/proxmox_vm_commands_test.rb
+- test/unit/foreman_fog_proxmox/semver_test.rb
 - test/unit/foreman_fog_proxmox/proxmox_vm_commands_server_update_hard_disk_test.rb
+- test/unit/foreman_fog_proxmox/proxmox_vm_queries_test.rb
 - test/unit/foreman_fog_proxmox/proxmox_vm_commands_server_update_test.rb
-- test/unit/foreman_fog_proxmox/proxmox_vm_commands_test.rb
+- test/unit/foreman_fog_proxmox/proxmox_interfaces_test.rb
+- test/unit/foreman_fog_proxmox/proxmox_compute_attributes_test.rb
+- test/unit/foreman_fog_proxmox/proxmox_images_test.rb
+- test/unit/foreman_fog_proxmox/proxmox_vm_commands_server_update_cloudinit_test.rb
+- test/unit/foreman_fog_proxmox/proxmox_test.rb
 - test/unit/foreman_fog_proxmox/proxmox_vm_new_test.rb
-- test/unit/foreman_fog_proxmox/proxmox_vm_queries_test.rb
-- test/unit/foreman_fog_proxmox/semver_test.rb
+- test/functional/compute_resources_controller_test.rb