knife-ionoscloud 6.0.0.alpha.1 → 6.1.0

data/spec/chef/knife/ionoscloud_snapshot_delete_spec.rb

@@ -28,6 +28,18 @@ describe Chef::Knife::IonoscloudSnapshotDelete do
       expect(subject).to receive(:puts).with("Description: #{snapshot.properties.description}")
       expect(subject).to receive(:puts).with("Location: #{snapshot.properties.location}")
       expect(subject).to receive(:puts).with("Size: #{snapshot.properties.size.to_s}")
+      expect(subject).to receive(:puts).with("Sec Auth Protection: #{snapshot.properties.sec_auth_protection}")
+      expect(subject).to receive(:puts).with("License Type: #{snapshot.properties.licence_type}")
+      expect(subject).to receive(:puts).with("CPU Hot Plug: #{snapshot.properties.cpu_hot_plug}")
+      expect(subject).to receive(:puts).with("CPU Hot Unplug: #{snapshot.properties.cpu_hot_unplug}")
+      expect(subject).to receive(:puts).with("RAM Hot Plug: #{snapshot.properties.ram_hot_plug}")
+      expect(subject).to receive(:puts).with("RAM Hot Unplug: #{snapshot.properties.ram_hot_unplug}")
+      expect(subject).to receive(:puts).with("NIC Hot Plug: #{snapshot.properties.nic_hot_plug}")
+      expect(subject).to receive(:puts).with("NIC Hot Unplug: #{snapshot.properties.nic_hot_unplug}")
+      expect(subject).to receive(:puts).with("Disc Virtio Hot Plug: #{snapshot.properties.disc_virtio_hot_plug}")
+      expect(subject).to receive(:puts).with("Disc Virtio Hot Unplug: #{snapshot.properties.disc_virtio_hot_unplug}")
+      expect(subject).to receive(:puts).with("Disc Scsi Hot Plug: #{snapshot.properties.disc_scsi_hot_plug}")
+      expect(subject).to receive(:puts).with("Disc Scsi Hot Unplug: #{snapshot.properties.disc_scsi_hot_unplug}")
       expect(subject.ui).to receive(:warn).with("Deleted Snapshot #{snapshot.id}. Request ID: ")
 
       expect(subject.api_client).not_to receive(:wait_for)

data/spec/chef/knife/ionoscloud_snapshot_get_spec.rb

@@ -0,0 +1,79 @@
+require 'spec_helper'
+require 'ionoscloud_snapshot_get'
+
+Chef::Knife::IonoscloudSnapshotGet.load_deps
+
+describe Chef::Knife::IonoscloudSnapshotGet do
+  before :each do
+    subject { Chef::Knife::IonoscloudSnapshotGet.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call SnapshotsApi.snapshots_find_by_id' do
+      snapshot = snapshot_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        snapshot_id: snapshot.id,
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject).to receive(:puts).with("ID: #{snapshot.id}")
+      expect(subject).to receive(:puts).with("Name: #{snapshot.properties.name}")
+      expect(subject).to receive(:puts).with("Description: #{snapshot.properties.description}")
+      expect(subject).to receive(:puts).with("Location: #{snapshot.properties.location}")
+      expect(subject).to receive(:puts).with("Size: #{snapshot.properties.size.to_s}")
+      expect(subject).to receive(:puts).with("Sec Auth Protection: #{snapshot.properties.sec_auth_protection}")
+      expect(subject).to receive(:puts).with("License Type: #{snapshot.properties.licence_type}")
+      expect(subject).to receive(:puts).with("CPU Hot Plug: #{snapshot.properties.cpu_hot_plug}")
+      expect(subject).to receive(:puts).with("CPU Hot Unplug: #{snapshot.properties.cpu_hot_unplug}")
+      expect(subject).to receive(:puts).with("RAM Hot Plug: #{snapshot.properties.ram_hot_plug}")
+      expect(subject).to receive(:puts).with("RAM Hot Unplug: #{snapshot.properties.ram_hot_unplug}")
+      expect(subject).to receive(:puts).with("NIC Hot Plug: #{snapshot.properties.nic_hot_plug}")
+      expect(subject).to receive(:puts).with("NIC Hot Unplug: #{snapshot.properties.nic_hot_unplug}")
+      expect(subject).to receive(:puts).with("Disc Virtio Hot Plug: #{snapshot.properties.disc_virtio_hot_plug}")
+      expect(subject).to receive(:puts).with("Disc Virtio Hot Unplug: #{snapshot.properties.disc_virtio_hot_unplug}")
+      expect(subject).to receive(:puts).with("Disc Scsi Hot Plug: #{snapshot.properties.disc_scsi_hot_plug}")
+      expect(subject).to receive(:puts).with("Disc Scsi Hot Unplug: #{snapshot.properties.disc_scsi_hot_unplug}")
+
+      expect(subject.api_client).not_to receive(:wait_for)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/snapshots/#{subject_config[:snapshot_id]}",
+            operation: :'SnapshotsApi.snapshots_find_by_id',
+            return_type: 'Snapshot',
+            result: snapshot,
+          },
+        ],
+      )
+
+      expect { subject.run }.not_to raise_error(Exception)
+    end
+
+    it 'should not make any call if any required option is missing' do
+      required_options = subject.instance_variable_get(:@required_options)
+
+      arrays_without_one_element(required_options).each do |test_case|
+
+        test_case[:array].each { |value| subject.config[value] = 'test' }
+
+        expect(subject).to receive(:puts).with("Missing required parameters #{test_case[:removed]}")
+        expect(subject.api_client).not_to receive(:call_api)
+
+        expect { subject.run }.to raise_error(SystemExit) do |error|
+          expect(error.status).to eq(1)
+        end
+
+        required_options.each { |value| subject.config[value] = nil }
+      end
+    end
+  end
+end

data/spec/chef/knife/ionoscloud_snapshot_restore_spec.rb

@@ -32,7 +32,16 @@ describe Chef::Knife::IonoscloudSnapshotRestore do
       expect(subject).to receive(:puts).with("Image: #{volume.properties.image}")
       expect(subject).to receive(:puts).with("Type: #{volume.properties.type}")
       expect(subject).to receive(:puts).with("Licence Type: #{volume.properties.licence_type}")
+      expect(subject).to receive(:puts).with("Backupunit ID: #{volume.properties.backupunit_id}")
+      expect(subject).to receive(:puts).with("User Data: #{volume.properties.user_data}")
       expect(subject).to receive(:puts).with("Zone: #{volume.properties.availability_zone}")
+      expect(subject).to receive(:puts).with("CPU Hot Plug: #{volume.properties.cpu_hot_plug}")
+      expect(subject).to receive(:puts).with("RAM Hot Plug: #{volume.properties.ram_hot_plug}")
+      expect(subject).to receive(:puts).with("NIC Hot Plug: #{volume.properties.nic_hot_plug}")
+      expect(subject).to receive(:puts).with("NIC Hot Unplug: #{volume.properties.nic_hot_unplug}")
+      expect(subject).to receive(:puts).with("Disc Virtio Hot Plug: #{volume.properties.disc_virtio_hot_plug}")
+      expect(subject).to receive(:puts).with("Disc Virtio Hot Unplug: #{volume.properties.disc_virtio_hot_unplug}")
+      expect(subject).to receive(:puts).with("Device number: #{volume.properties.device_number}")
 
       mock_wait_for(subject)
       mock_call_api(

data/spec/chef/knife/ionoscloud_snapshot_update_spec.rb

@@ -0,0 +1,113 @@
+require 'spec_helper'
+require 'ionoscloud_snapshot_update'
+
+Chef::Knife::IonoscloudSnapshotUpdate.load_deps
+
+describe Chef::Knife::IonoscloudSnapshotUpdate do
+  before :each do
+    subject { Chef::Knife::IonoscloudSnapshotUpdate.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call SnapshotsApi.datacenters_snapshots_patch' do
+      snapshot = snapshot_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        snapshot_id: snapshot.id,
+        name: snapshot.properties.name + '_edited',
+        description: snapshot.properties.description + '_edited',
+        licence_type: 'WINDOWS',
+        cpu_hot_plug: (!snapshot.properties.cpu_hot_plug).to_s,
+        ram_hot_plug: (!snapshot.properties.ram_hot_plug).to_s,
+        disc_virtio_hot_plug: (!snapshot.properties.disc_virtio_hot_plug).to_s,
+        disc_virtio_hot_unplug: (!snapshot.properties.disc_virtio_hot_unplug).to_s,
+        disc_scsi_hot_plug: (!snapshot.properties.disc_scsi_hot_plug).to_s,
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject).to receive(:puts).with("ID: #{snapshot.id}")
+      expect(subject).to receive(:puts).with("Name: #{subject_config[:name]}")
+      expect(subject).to receive(:puts).with("Description: #{subject_config[:description]}")
+      expect(subject).to receive(:puts).with("Location: #{snapshot.properties.location}")
+      expect(subject).to receive(:puts).with("Size: #{snapshot.properties.size.to_s}")
+      expect(subject).to receive(:puts).with("Sec Auth Protection: #{snapshot.properties.sec_auth_protection}")
+      expect(subject).to receive(:puts).with("License Type: #{subject_config[:licence_type]}")
+      expect(subject).to receive(:puts).with("CPU Hot Plug: #{subject_config[:cpu_hot_plug]}")
+      expect(subject).to receive(:puts).with("CPU Hot Unplug: #{snapshot.properties.cpu_hot_unplug}")
+      expect(subject).to receive(:puts).with("RAM Hot Plug: #{subject_config[:ram_hot_plug]}")
+      expect(subject).to receive(:puts).with("RAM Hot Unplug: #{snapshot.properties.ram_hot_unplug}")
+      expect(subject).to receive(:puts).with("NIC Hot Plug: #{snapshot.properties.nic_hot_plug}")
+      expect(subject).to receive(:puts).with("NIC Hot Unplug: #{snapshot.properties.nic_hot_unplug}")
+      expect(subject).to receive(:puts).with("Disc Virtio Hot Plug: #{subject_config[:disc_virtio_hot_plug]}")
+      expect(subject).to receive(:puts).with("Disc Virtio Hot Unplug: #{subject_config[:disc_virtio_hot_unplug]}")
+      expect(subject).to receive(:puts).with("Disc Scsi Hot Plug: #{subject_config[:disc_scsi_hot_plug]}")
+      expect(subject).to receive(:puts).with("Disc Scsi Hot Unplug: #{snapshot.properties.disc_scsi_hot_unplug}")
+
+      snapshot.properties.name = subject_config[:name]
+      snapshot.properties.description = subject_config[:description]
+      snapshot.properties.licence_type = subject_config[:licence_type]
+      snapshot.properties.cpu_hot_plug = subject_config[:cpu_hot_plug].to_s.downcase == 'true'
+      snapshot.properties.ram_hot_plug = subject_config[:ram_hot_plug].to_s.downcase == 'true'
+      snapshot.properties.disc_virtio_hot_plug = subject_config[:disc_virtio_hot_plug].to_s.downcase == 'true'
+      snapshot.properties.disc_virtio_hot_unplug = subject_config[:disc_virtio_hot_unplug].to_s.downcase == 'true'
+      snapshot.properties.disc_scsi_hot_plug = subject_config[:disc_scsi_hot_plug].to_s.downcase == 'true'
+
+      mock_wait_for(subject)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'PATCH',
+            path: "/snapshots/#{subject_config[:snapshot_id]}",
+            operation: :'SnapshotsApi.snapshots_patch',
+            return_type: 'Snapshot',
+            body: {
+              name: subject_config[:name],
+              description: subject_config[:description],
+              licenceType: subject_config[:licence_type],
+              cpuHotPlug: subject_config[:cpu_hot_plug].to_s.downcase == 'true',
+              ramHotPlug: subject_config[:ram_hot_plug].to_s.downcase == 'true',
+              discVirtioHotPlug: subject_config[:disc_virtio_hot_plug].to_s.downcase == 'true',
+              discVirtioHotUnplug: subject_config[:disc_virtio_hot_unplug].to_s.downcase == 'true',
+              discScsiHotPlug: subject_config[:disc_scsi_hot_plug].to_s.downcase == 'true',
+            },
+            result: snapshot,
+          },
+          {
+            method: 'GET',
+            path: "/snapshots/#{subject_config[:snapshot_id]}",
+            operation: :'SnapshotsApi.snapshots_find_by_id',
+            return_type: 'Snapshot',
+            result: snapshot,
+          },
+        ],
+      )
+
+      expect { subject.run }.not_to raise_error(Exception)
+    end
+
+    it 'should not make any call if any required option is missing' do
+      required_options = subject.instance_variable_get(:@required_options)
+
+      arrays_without_one_element(required_options).each do |test_case|
+
+        test_case[:array].each { |value| subject.config[value] = 'test' }
+
+        expect(subject).to receive(:puts).with("Missing required parameters #{test_case[:removed]}")
+        expect(subject.api_client).not_to receive(:call_api)
+
+        expect { subject.run }.to raise_error(SystemExit) do |error|
+          expect(error.status).to eq(1)
+        end
+
+        required_options.each { |value| subject.config[value] = nil }
+      end
+    end
+  end
+end

data/spec/chef/knife/ionoscloud_user_create_spec.rb

@@ -32,7 +32,9 @@ describe Chef::Knife::IonoscloudUserCreate do
       expect(subject).to receive(:puts).with("Lastname: #{user.properties.lastname}")
       expect(subject).to receive(:puts).with("Email: #{user.properties.email}")
       expect(subject).to receive(:puts).with("Administrator: #{user.properties.administrator}")
-      expect(subject).to receive(:puts).with("2-Factor Auth: #{user.properties.force_sec_auth}")
+      expect(subject).to receive(:puts).with("Force 2-Factor Auth: #{user.properties.force_sec_auth}")
+      expect(subject).to receive(:puts).with("2-Factor Auth Active: #{user.properties.sec_auth_active}")
+      expect(subject).to receive(:puts).with("Active: #{user.properties.active}")
 
       mock_wait_for(subject)
       mock_call_api(

data/spec/chef/knife/ionoscloud_user_delete_spec.rb

@@ -27,8 +27,10 @@ describe Chef::Knife::IonoscloudUserDelete do
       expect(subject).to receive(:puts).with("Firstname: #{user.properties.firstname}")
       expect(subject).to receive(:puts).with("Lastname: #{user.properties.lastname}")
       expect(subject).to receive(:puts).with("Email: #{user.properties.email}")
-      expect(subject).to receive(:puts).with("Administrator: #{user.properties.administrator.to_s}")
-      expect(subject).to receive(:puts).with("2-Factor Auth: #{user.properties.force_sec_auth.to_s}")
+      expect(subject).to receive(:puts).with("Administrator: #{user.properties.administrator}")
+      expect(subject).to receive(:puts).with("Force 2-Factor Auth: #{user.properties.force_sec_auth}")
+      expect(subject).to receive(:puts).with("2-Factor Auth Active: #{user.properties.sec_auth_active}")
+      expect(subject).to receive(:puts).with("Active: #{user.properties.active}")
       expect(subject.ui).to receive(:warn).with("Deleted User #{user.id}. Request ID: ")
 
       expect(subject.api_client).not_to receive(:wait_for)

data/spec/chef/knife/ionoscloud_user_get_spec.rb

@@ -0,0 +1,70 @@
+require 'spec_helper'
+require 'ionoscloud_user_get'
+
+Chef::Knife::IonoscloudUserGet.load_deps
+
+describe Chef::Knife::IonoscloudUserGet do
+  before :each do
+    subject { Chef::Knife::IonoscloudUserGet.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call UserApi.users_find_by_id' do
+      user = user_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        user_id: user.id,
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject).to receive(:puts).with("ID: #{user.id}")
+      expect(subject).to receive(:puts).with("Firstname: #{user.properties.firstname}")
+      expect(subject).to receive(:puts).with("Lastname: #{user.properties.lastname}")
+      expect(subject).to receive(:puts).with("Email: #{user.properties.email}")
+      expect(subject).to receive(:puts).with("Administrator: #{user.properties.administrator}")
+      expect(subject).to receive(:puts).with("Force 2-Factor Auth: #{user.properties.force_sec_auth}")
+      expect(subject).to receive(:puts).with("2-Factor Auth Active: #{user.properties.sec_auth_active}")
+      expect(subject).to receive(:puts).with("Active: #{user.properties.active}")
+
+      expect(subject.api_client).not_to receive(:wait_for)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/um/users/#{subject_config[:user_id]}",
+            operation: :'UserManagementApi.um_users_find_by_id',
+            return_type: 'User',
+            result: user,
+          },
+        ],
+      )
+
+      expect { subject.run }.not_to raise_error(Exception)
+    end
+
+    it 'should not make any call if any required option is missing' do
+      required_options = subject.instance_variable_get(:@required_options)
+
+      arrays_without_one_element(required_options).each do |test_case|
+
+        test_case[:array].each { |value| subject.config[value] = 'test' }
+
+        expect(subject).to receive(:puts).with("Missing required parameters #{test_case[:removed]}")
+        expect(subject.api_client).not_to receive(:call_api)
+
+        expect { subject.run }.to raise_error(SystemExit) do |error|
+          expect(error.status).to eq(1)
+        end
+
+        required_options.each { |value| subject.config[value] = nil }
+      end
+    end
+  end
+end

data/spec/chef/knife/ionoscloud_user_update_spec.rb

@@ -0,0 +1,110 @@
+require 'spec_helper'
+require 'ionoscloud_user_update'
+
+Chef::Knife::IonoscloudUserUpdate.load_deps
+
+describe Chef::Knife::IonoscloudUserUpdate do
+  before :each do
+    subject { Chef::Knife::IonoscloudUserUpdate.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call UserManagementApi.um_groups_users_put' do
+      user = user_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        user_id: user.id,
+        firstname: user.properties.firstname + '_edited',
+        lastname: user.properties.lastname + '_edited',
+        email: user.properties.email + '_edited',
+        administrator: (!user.properties.administrator).to_s,
+        force_sec_auth: (!user.properties.force_sec_auth).to_s,
+        sec_auth_active: (!user.properties.sec_auth_active).to_s,
+        active: (!user.properties.active).to_s,
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject).to receive(:puts).with("ID: #{user.id}")
+      expect(subject).to receive(:puts).with("Firstname: #{subject_config[:firstname]}")
+      expect(subject).to receive(:puts).with("Lastname: #{subject_config[:lastname]}")
+      expect(subject).to receive(:puts).with("Email: #{subject_config[:email]}")
+      expect(subject).to receive(:puts).with("Administrator: #{subject_config[:administrator]}")
+      expect(subject).to receive(:puts).with("Force 2-Factor Auth: #{subject_config[:force_sec_auth]}")
+      expect(subject).to receive(:puts).with("2-Factor Auth Active: #{subject_config[:sec_auth_active]}")
+      expect(subject).to receive(:puts).with("Active: #{subject_config[:active]}")
+
+      user.properties.firstname = subject_config[:firstname]
+      user.properties.lastname = subject_config[:lastname]
+      user.properties.email = subject_config[:email]
+      user.properties.administrator = subject_config[:administrator].to_s.downcase == 'true'
+      user.properties.force_sec_auth = subject_config[:force_sec_auth].to_s.downcase == 'true'
+      user.properties.sec_auth_active = subject_config[:sec_auth_active].to_s.downcase == 'true'
+      user.properties.active = subject_config[:active].to_s.downcase == 'true'
+
+      mock_wait_for(subject)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/um/users/#{subject_config[:user_id]}",
+            operation: :'UserManagementApi.um_users_find_by_id',
+            return_type: 'User',
+            result: user,
+          },
+          {
+            method: 'PUT',
+            path: "/um/users/#{subject_config[:user_id]}",
+            operation: :'UserManagementApi.um_users_put',
+            return_type: 'User',
+            body: {
+                properties: {
+                  firstname: subject_config[:firstname],
+                  lastname: subject_config[:lastname],
+                  email: subject_config[:email],
+                  administrator: subject_config[:administrator].to_s.downcase == 'true',
+                  forceSecAuth: subject_config[:force_sec_auth].to_s.downcase == 'true',
+                  secAuthActive: subject_config[:sec_auth_active].to_s.downcase == 'true',
+                  active: subject_config[:active].to_s.downcase == 'true',
+              },
+            },
+            result: user,
+          },
+          {
+            method: 'GET',
+            path: "/um/users/#{subject_config[:user_id]}",
+            operation: :'UserManagementApi.um_users_find_by_id',
+            return_type: 'User',
+            result: user,
+          },
+        ],
+      )
+
+      expect { subject.run }.not_to raise_error(Exception)
+    end
+
+    it 'should not make any call if any required option is missing' do
+      required_options = subject.instance_variable_get(:@required_options)
+
+      arrays_without_one_element(required_options).each do |test_case|
+
+        test_case[:array].each { |value| subject.config[value] = 'test' }
+
+        expect(subject).to receive(:puts).with("Missing required parameters #{test_case[:removed]}")
+        expect(subject.api_client).not_to receive(:call_api)
+
+        expect { subject.run }.to raise_error(SystemExit) do |error|
+          expect(error.status).to eq(1)
+        end
+
+        required_options.each { |value| subject.config[value] = nil }
+      end
+    end
+  end
+end

data/spec/chef/knife/ionoscloud_volume_create_spec.rb

@@ -37,13 +37,23 @@ describe Chef::Knife::IonoscloudVolumeCreate do
       expected_body[:imagePassword] = subject_config[:image_password]
 
       expect(subject).to receive(:puts).with("ID: #{volume.id}")
-      expect(subject).to receive(:puts).with("Name: #{subject_config[:name]}")
-      expect(subject).to receive(:puts).with("Size: #{subject_config[:size]}")
-      expect(subject).to receive(:puts).with("Bus: #{subject_config[:bus]}")
+      expect(subject).to receive(:puts).with("Name: #{volume.properties.name}")
+      expect(subject).to receive(:puts).with("Size: #{volume.properties.size}")
+      expect(subject).to receive(:puts).with("Bus: #{volume.properties.bus}")
       expect(subject).to receive(:puts).with("Image: #{volume.properties.image}")
-      expect(subject).to receive(:puts).with("Type: #{subject_config[:type]}")
+      expect(subject).to receive(:puts).with("Type: #{volume.properties.type}")
       expect(subject).to receive(:puts).with("Licence Type: #{volume.properties.licence_type}")
-      expect(subject).to receive(:puts).with("Zone: #{subject_config[:availability_zone]}")
+      expect(subject).to receive(:puts).with("Backupunit ID: #{volume.properties.backupunit_id}")
+      expect(subject).to receive(:puts).with("User Data: #{volume.properties.user_data}")
+      expect(subject).to receive(:puts).with("Zone: #{volume.properties.availability_zone}")
+      expect(subject).to receive(:puts).with("CPU Hot Plug: #{volume.properties.cpu_hot_plug}")
+      expect(subject).to receive(:puts).with("RAM Hot Plug: #{volume.properties.ram_hot_plug}")
+      expect(subject).to receive(:puts).with("NIC Hot Plug: #{volume.properties.nic_hot_plug}")
+      expect(subject).to receive(:puts).with("NIC Hot Unplug: #{volume.properties.nic_hot_unplug}")
+      expect(subject).to receive(:puts).with("Disc Virtio Hot Plug: #{volume.properties.disc_virtio_hot_plug}")
+      expect(subject).to receive(:puts).with("Disc Virtio Hot Unplug: #{volume.properties.disc_virtio_hot_unplug}")
+      expect(subject).to receive(:puts).with("Device number: #{volume.properties.device_number}")
+      expect(subject).to receive(:puts).with("PCI Slot: #{volume.properties.pci_slot}")
 
       mock_wait_for(subject)
       mock_call_api(

data/spec/chef/knife/ionoscloud_volume_delete_spec.rb

@@ -31,7 +31,17 @@ describe Chef::Knife::IonoscloudVolumeDelete do
       expect(subject).to receive(:puts).with("Image: #{volume.properties.image}")
       expect(subject).to receive(:puts).with("Type: #{volume.properties.type}")
       expect(subject).to receive(:puts).with("Licence Type: #{volume.properties.licence_type}")
+      expect(subject).to receive(:puts).with("Backupunit ID: #{volume.properties.backupunit_id}")
+      expect(subject).to receive(:puts).with("User Data: #{volume.properties.user_data}")
       expect(subject).to receive(:puts).with("Zone: #{volume.properties.availability_zone}")
+      expect(subject).to receive(:puts).with("CPU Hot Plug: #{volume.properties.cpu_hot_plug}")
+      expect(subject).to receive(:puts).with("RAM Hot Plug: #{volume.properties.ram_hot_plug}")
+      expect(subject).to receive(:puts).with("NIC Hot Plug: #{volume.properties.nic_hot_plug}")
+      expect(subject).to receive(:puts).with("NIC Hot Unplug: #{volume.properties.nic_hot_unplug}")
+      expect(subject).to receive(:puts).with("Disc Virtio Hot Plug: #{volume.properties.disc_virtio_hot_plug}")
+      expect(subject).to receive(:puts).with("Disc Virtio Hot Unplug: #{volume.properties.disc_virtio_hot_unplug}")
+      expect(subject).to receive(:puts).with("Device number: #{volume.properties.device_number}")
+      expect(subject).to receive(:puts).with("PCI Slot: #{volume.properties.pci_slot}")
       expect(subject.ui).to receive(:warn).with("Deleted Volume #{volume.id}. Request ID: ")
 
       expect(subject).not_to receive(:wait_for)

data/spec/chef/knife/ionoscloud_volume_get_spec.rb

@@ -0,0 +1,81 @@
+require 'spec_helper'
+require 'ionoscloud_volume_get'
+
+Chef::Knife::IonoscloudVolumeGet.load_deps
+
+describe Chef::Knife::IonoscloudVolumeGet do
+  before :each do
+    subject { Chef::Knife::IonoscloudVolumeGet.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call VolumesApi.datacenters_volumes_find_by_id' do
+      volume = volume_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: 'datacenter_id',
+        volume_id: volume.id,
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject).to receive(:puts).with("ID: #{volume.id}")
+      expect(subject).to receive(:puts).with("Name: #{volume.properties.name}")
+      expect(subject).to receive(:puts).with("Size: #{volume.properties.size}")
+      expect(subject).to receive(:puts).with("Bus: #{volume.properties.bus}")
+      expect(subject).to receive(:puts).with("Image: #{volume.properties.image}")
+      expect(subject).to receive(:puts).with("Type: #{volume.properties.type}")
+      expect(subject).to receive(:puts).with("Licence Type: #{volume.properties.licence_type}")
+      expect(subject).to receive(:puts).with("Backupunit ID: #{volume.properties.backupunit_id}")
+      expect(subject).to receive(:puts).with("User Data: #{volume.properties.user_data}")
+      expect(subject).to receive(:puts).with("Zone: #{volume.properties.availability_zone}")
+      expect(subject).to receive(:puts).with("CPU Hot Plug: #{volume.properties.cpu_hot_plug}")
+      expect(subject).to receive(:puts).with("RAM Hot Plug: #{volume.properties.ram_hot_plug}")
+      expect(subject).to receive(:puts).with("NIC Hot Plug: #{volume.properties.nic_hot_plug}")
+      expect(subject).to receive(:puts).with("NIC Hot Unplug: #{volume.properties.nic_hot_unplug}")
+      expect(subject).to receive(:puts).with("Disc Virtio Hot Plug: #{volume.properties.disc_virtio_hot_plug}")
+      expect(subject).to receive(:puts).with("Disc Virtio Hot Unplug: #{volume.properties.disc_virtio_hot_unplug}")
+      expect(subject).to receive(:puts).with("Device number: #{volume.properties.device_number}")
+      expect(subject).to receive(:puts).with("PCI Slot: #{volume.properties.pci_slot}")
+
+      expect(subject.api_client).not_to receive(:wait_for)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/volumes/#{subject_config[:volume_id]}",
+            operation: :'VolumesApi.datacenters_volumes_find_by_id',
+            return_type: 'Volume',
+            result: volume,
+          },
+        ],
+      )
+
+      expect { subject.run }.not_to raise_error(Exception)
+    end
+
+    it 'should not make any call if any required option is missing' do
+      required_options = subject.instance_variable_get(:@required_options)
+
+      arrays_without_one_element(required_options).each do |test_case|
+
+        test_case[:array].each { |value| subject.config[value] = 'test' }
+
+        expect(subject).to receive(:puts).with("Missing required parameters #{test_case[:removed]}")
+        expect(subject.api_client).not_to receive(:call_api)
+
+        expect { subject.run }.to raise_error(SystemExit) do |error|
+          expect(error.status).to eq(1)
+        end
+
+        required_options.each { |value| subject.config[value] = nil }
+      end
+    end
+  end
+end