knife-ionoscloud 5.1.1 → 5.1.2

data/spec/chef/knife/ionoscloud_group_update_spec.rb

@@ -0,0 +1,109 @@
+require 'spec_helper'
+require 'ionoscloud_group_update'
+
+Chef::Knife::IonoscloudGroupUpdate.load_deps
+
+describe Chef::Knife::IonoscloudGroupUpdate do
+  before :each do
+    subject { Chef::Knife::IonoscloudGroupUpdate.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call UserManagementApi.um_groups_put' do
+      group = group_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        group_id: group.id,
+        name: group.properties.name + '_edited',
+        create_snapshot: !group.properties.create_snapshot,
+        reserve_ip: group.properties.reserve_ip,
+        access_activity_log: !group.properties.access_activity_log,
+        create_backup_unit: !group.properties.create_backup_unit,
+        create_pcc: !group.properties.create_pcc,
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject).to receive(:puts).with("ID: #{group.id}")
+      expect(subject).to receive(:puts).with("Name: #{subject_config[:name]}")
+      expect(subject).to receive(:puts).with("Create Datacenter: #{group.properties.create_data_center.to_s}")
+      expect(subject).to receive(:puts).with("Create Snapshot: #{subject_config[:create_snapshot].to_s}")
+      expect(subject).to receive(:puts).with("Reserve IP: #{group.properties.reserve_ip.to_s}")
+      expect(subject).to receive(:puts).with("Access Activity Log: #{subject_config[:access_activity_log].to_s}")
+      expect(subject).to receive(:puts).with("S3 Privilege: #{group.properties.s3_privilege.to_s}")
+      expect(subject).to receive(:puts).with("Create Backup Unit: #{subject_config[:create_backup_unit].to_s}")
+      expect(subject).to receive(:puts).with("Create K8s Clusters: #{group.properties.create_k8s_cluster.to_s}")
+      expect(subject).to receive(:puts).with("Create PCC: #{subject_config[:create_pcc].to_s}")
+      expect(subject).to receive(:puts).with("Create Internet Acess: #{group.properties.create_internet_access.to_s}")
+
+      group.properties.name = subject_config[:name]
+      group.properties.create_snapshot = subject_config[:create_snapshot]
+      group.properties.access_activity_log = subject_config[:access_activity_log]
+      group.properties.create_backup_unit = subject_config[:create_backup_unit]
+      group.properties.create_pcc = subject_config[:create_pcc]
+
+      expected_body = {
+        name: subject_config[:name],
+        createSnapshot: subject_config[:create_snapshot],
+        reserveIp: subject_config[:reserve_ip],
+        accessActivityLog: subject_config[:access_activity_log],
+        createBackupUnit: subject_config[:create_backup_unit],
+        createPcc: subject_config[:create_pcc],
+      }.merge(group.properties.to_hash)
+
+      mock_wait_for(subject)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/um/groups/#{group.id}",
+            operation: :'UserManagementApi.um_groups_find_by_id',
+            return_type: 'Group',
+            result: group,
+          },
+          {
+            method: 'PUT',
+            path: "/um/groups/#{group.id}",
+            operation: :'UserManagementApi.um_groups_put',
+            return_type: 'Group',
+            body: { properties: expected_body },
+            result: group,
+          },
+          {
+            method: 'GET',
+            path: "/um/groups/#{group.id}",
+            operation: :'UserManagementApi.um_groups_find_by_id',
+            return_type: 'Group',
+            result: group,
+          },
+        ],
+      )
+
+      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_group_user_add_spec.rb

@@ -35,6 +35,9 @@ describe Chef::Knife::IonoscloudGroupUserAdd do
       expect(subject).to receive(:puts).with("Access Activity Log: #{group.properties.access_activity_log.to_s}")
       expect(subject).to receive(:puts).with("S3 Privilege: #{group.properties.s3_privilege.to_s}")
       expect(subject).to receive(:puts).with("Create Backup Unit: #{group.properties.create_backup_unit.to_s}")
+      expect(subject).to receive(:puts).with("Create K8s Clusters: #{group.properties.create_k8s_cluster.to_s}")
+      expect(subject).to receive(:puts).with("Create PCC: #{group.properties.create_pcc.to_s}")
+      expect(subject).to receive(:puts).with("Create Internet Acess: #{group.properties.create_internet_access.to_s}")
       expect(subject).to receive(:puts).with("Users: #{users.to_s}")
       expect(subject.ui).to receive(:info).with("Added User #{user.id} to the Group #{group.id}. Request ID: .")
 

data/spec/chef/knife/ionoscloud_group_user_remove_spec.rb

@@ -35,6 +35,9 @@ describe Chef::Knife::IonoscloudGroupUserRemove do
       expect(subject).to receive(:puts).with("Access Activity Log: #{group.properties.access_activity_log.to_s}")
       expect(subject).to receive(:puts).with("S3 Privilege: #{group.properties.s3_privilege.to_s}")
       expect(subject).to receive(:puts).with("Create Backup Unit: #{group.properties.create_backup_unit.to_s}")
+      expect(subject).to receive(:puts).with("Create K8s Clusters: #{group.properties.create_k8s_cluster.to_s}")
+      expect(subject).to receive(:puts).with("Create PCC: #{group.properties.create_pcc.to_s}")
+      expect(subject).to receive(:puts).with("Create Internet Acess: #{group.properties.create_internet_access.to_s}")
       expect(subject).to receive(:puts).with("Users: #{users.to_s}")
       expect(subject.ui).to receive(:warn).with("Removed User #{user.id} from the Group #{group.id}. Request ID: .")
 

data/spec/chef/knife/ionoscloud_ipblock_delete_spec.rb

@@ -23,10 +23,12 @@ describe Chef::Knife::IonoscloudIpblockDelete do
       subject_config.each { |key, value| subject.config[key] = value }
       subject.name_args = [ipblock.id]
 
+      ip_consumers = (ipblock.properties.ip_consumers.nil? ? [] : ipblock.properties.ip_consumers.map { |el| el.to_hash })
       expect(subject).to receive(:puts).with("ID: #{ipblock.id}")
       expect(subject).to receive(:puts).with("Name: #{ipblock.properties.name}")
       expect(subject).to receive(:puts).with("IP Addresses: #{ipblock.properties.ips.to_s}")
       expect(subject).to receive(:puts).with("Location: #{ipblock.properties.location}")
+      expect(subject).to receive(:puts).with("IP Consumers: #{ip_consumers}")
       expect(subject.ui).to receive(:warn).with("Released IP block #{ipblock.id}. Request ID: ")
 
       expect(subject.api_client).not_to receive(:wait_for)

data/spec/chef/knife/ionoscloud_ipblock_get_spec.rb

@@ -0,0 +1,66 @@
+require 'spec_helper'
+require 'ionoscloud_ipblock_get'
+
+Chef::Knife::IonoscloudIpblockGet.load_deps
+
+describe Chef::Knife::IonoscloudIpblockGet do
+  before :each do
+    subject { Chef::Knife::IonoscloudIpblockGet.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call IPBlocksApi.ipblocks_find_by_id' do
+      ipblock = ipblock_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        ipblock_id: ipblock.id,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      ip_consumers = (ipblock.properties.ip_consumers.nil? ? [] : ipblock.properties.ip_consumers.map { |el| el.to_hash })
+      expect(subject).to receive(:puts).with("ID: #{ipblock.id}")
+      expect(subject).to receive(:puts).with("Name: #{ipblock.properties.name}")
+      expect(subject).to receive(:puts).with("IP Addresses: #{ipblock.properties.ips.to_s}")
+      expect(subject).to receive(:puts).with("Location: #{ipblock.properties.location}")
+      expect(subject).to receive(:puts).with("IP Consumers: #{ip_consumers}")
+
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/ipblocks/#{ipblock.id}",
+            operation: :'IPBlocksApi.ipblocks_find_by_id',
+            return_type: 'IpBlock',
+            result: ipblock,
+          },
+        ],
+      )
+
+      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_ipblock_list_spec.rb

@@ -34,7 +34,7 @@ describe Chef::Knife::IonoscloudIpblockList do
         ipblock_list << ipblock.properties.name
         ipblock_list << ipblock.properties.location
         ipblock_list << ipblock.properties.ips.join(', ')
-        ipblock_list << ipblock.properties.ip_consumers.nil? ? 0 : ipblock.properties.ip_consumers.length
+        ipblock_list << (ipblock.properties.ip_consumers.nil? ? 0 : ipblock.properties.ip_consumers.length)
       end
 
       expect(subject.ui).to receive(:list).with(ipblock_list, :uneven_columns_across, 5)

data/spec/chef/knife/ionoscloud_ipblock_update_spec.rb

@@ -0,0 +1,79 @@
+require 'spec_helper'
+require 'ionoscloud_ipblock_update'
+
+Chef::Knife::IonoscloudIpblockUpdate.load_deps
+
+describe Chef::Knife::IonoscloudIpblockUpdate do
+  before :each do
+    subject { Chef::Knife::IonoscloudIpblockUpdate.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call DataCenterApi.datacenters_patch' do
+      ipblock = ipblock_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        ipblock_id: ipblock.id,
+        name: ipblock.properties.name + '_edited',
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      ip_consumers = (ipblock.properties.ip_consumers.nil? ? [] : ipblock.properties.ip_consumers.map { |el| el.to_hash })
+      expect(subject).to receive(:puts).with("ID: #{ipblock.id}")
+      expect(subject).to receive(:puts).with("Name: #{subject_config[:name]}")
+      expect(subject).to receive(:puts).with("IP Addresses: #{ipblock.properties.ips.to_s}")
+      expect(subject).to receive(:puts).with("Location: #{ipblock.properties.location}")
+      expect(subject).to receive(:puts).with("IP Consumers: #{ip_consumers}")
+
+      ipblock.properties.name = subject_config[:name]
+
+      mock_wait_for(subject)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'PATCH',
+            path: "/ipblocks/#{ipblock.id}",
+            operation: :'IPBlocksApi.ipblocks_patch',
+            return_type: 'IpBlock',
+            body: { name: subject_config[:name] },
+            result: ipblock,
+          },
+          {
+            method: 'GET',
+            path: "/ipblocks/#{ipblock.id}",
+            operation: :'IPBlocksApi.ipblocks_find_by_id',
+            return_type: 'IpBlock',
+            result: ipblock,
+          },
+        ],
+      )
+
+      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_ipfailover_add_spec.rb

@@ -20,7 +20,7 @@ describe Chef::Knife::IonoscloudIpfailoverAdd do
         datacenter_id: 'datacenter_id',
         lan_id: lan.id,
         nic_id: 'nic_id',
-        ip: '1.1.1.1',
+        ip: '127.1.1.1',
         yes: true,
       }
 
@@ -29,6 +29,7 @@ describe Chef::Knife::IonoscloudIpfailoverAdd do
       expect(subject).to receive(:puts).with("ID: #{lan.id}")
       expect(subject).to receive(:puts).with("Name: #{lan.properties.name}")
       expect(subject).to receive(:puts).with("Public: #{lan.properties.public.to_s}")
+      expect(subject).to receive(:puts).with("PCC: #{lan.properties.pcc}")
       expect(subject).to receive(:puts).with("IP Failover: #{[{ ip: subject_config[:ip], nicUuid: subject_config[:nic_id] }]}")
 
       mock_wait_for(subject)

data/spec/chef/knife/ionoscloud_ipfailover_remove_spec.rb

@@ -13,7 +13,7 @@ describe Chef::Knife::IonoscloudIpfailoverRemove do
 
   describe '#run' do
     it 'should call Lan.datacenters_lans_patch when the ip failvoer rule exists' do
-      lan = lan_mock({ ip_failover: [ Ionoscloud::IPFailover.new(ip: '1.1.1.2', nic_uuid: 'nic_id2')] })
+      lan = lan_mock({ ip_failover: [ Ionoscloud::IPFailover.new(ip: '127.1.1.2', nic_uuid: 'nic_id2')] })
       subject_config = {
         ionoscloud_username: 'email',
         ionoscloud_password: 'password',
@@ -29,6 +29,7 @@ describe Chef::Knife::IonoscloudIpfailoverRemove do
       expect(subject).to receive(:puts).with("ID: #{lan.id}")
       expect(subject).to receive(:puts).with("Name: #{lan.properties.name}")
       expect(subject).to receive(:puts).with("Public: #{lan.properties.public.to_s}")
+      expect(subject).to receive(:puts).with("PCC: #{lan.properties.pcc}")
       expect(subject).to receive(:puts).with("IP Failover: #{[]}")
 
       lan_copy = Marshal.load(Marshal.dump(lan))
@@ -66,14 +67,14 @@ describe Chef::Knife::IonoscloudIpfailoverRemove do
     end
 
     it 'should not call Lan.datacenters_lans_patch when the ip failvoer rule does not exist' do
-      lan = lan_mock({ ip_failover: [ Ionoscloud::IPFailover.new(ip: '1.1.1.2', nic_uuid: 'nic_id2')] })
+      lan = lan_mock({ ip_failover: [ Ionoscloud::IPFailover.new(ip: '127.1.1.2', nic_uuid: 'nic_id2')] })
       subject_config = {
         ionoscloud_username: 'email',
         ionoscloud_password: 'password',
         datacenter_id: 'datacenter_id',
         lan_id: lan.id,
         nic_id: 'nic_id',
-        ip: '1.1.1.1',
+        ip: '127.1.1.1',
         yes: true,
       }
 

data/spec/chef/knife/ionoscloud_k8s_create_spec.rb

@@ -21,17 +21,25 @@ describe Chef::Knife::IonoscloudK8sCreate do
         version: cluster.properties.k8s_version,
         maintenance_day: cluster.properties.maintenance_window.day_of_the_week,
         maintenance_time: cluster.properties.maintenance_window.time,
+        s3_buckets: cluster.properties.s3_buckets.map { |el| el.name }.join(','),
+        api_subnet_allow_list: cluster.properties.api_subnet_allow_list.join(','),
       }
 
       subject_config.each { |key, value| subject.config[key] = value }
 
       maintenance_window = "#{cluster.properties.maintenance_window.day_of_the_week}, #{cluster.properties.maintenance_window.time}"
+      s3_buckets = (cluster.properties.s3_buckets.nil? ? [] : cluster.properties.s3_buckets.map { |el| el.name })
 
       expect(subject).to receive(:puts).with("ID: #{cluster.id}")
       expect(subject).to receive(:puts).with("Name: #{cluster.properties.name}")
+      expect(subject).to receive(:puts).with("Public: #{cluster.properties.public}")
       expect(subject).to receive(:puts).with("k8s Version: #{cluster.properties.k8s_version}")
       expect(subject).to receive(:puts).with("Maintenance Window: #{maintenance_window}")
       expect(subject).to receive(:puts).with("State: #{cluster.metadata.state}")
+      expect(subject).to receive(:puts).with("Api Subnet Allow List: #{cluster.properties.api_subnet_allow_list}")
+      expect(subject).to receive(:puts).with("S3 Buckets: #{s3_buckets}")
+      expect(subject).to receive(:puts).with("Available Upgrade Versions: #{cluster.properties.available_upgrade_versions}")
+      expect(subject).to receive(:puts).with("Viable NodePool Versions: #{cluster.properties.viable_node_pool_versions}")
 
       expected_body = cluster.properties.to_hash
       expected_body.delete(:viableNodePoolVersions)

data/spec/chef/knife/ionoscloud_k8s_delete_spec.rb

@@ -26,12 +26,18 @@ describe Chef::Knife::IonoscloudK8sDelete do
       subject.name_args = [k8s_cluster.id]
 
       maintenance_window = "#{k8s_cluster.properties.maintenance_window.day_of_the_week}, #{k8s_cluster.properties.maintenance_window.time}"
+      s3_buckets = (k8s_cluster.properties.s3_buckets.nil? ? [] : k8s_cluster.properties.s3_buckets.map { |el| el.name })
 
       expect(subject).to receive(:puts).with("ID: #{k8s_cluster.id}")
       expect(subject).to receive(:puts).with("Name: #{k8s_cluster.properties.name}")
-      expect(subject).to receive(:puts).with("Version: #{k8s_cluster.properties.k8s_version}")
+      expect(subject).to receive(:puts).with("Public: #{k8s_cluster.properties.public}")
+      expect(subject).to receive(:puts).with("k8s Version: #{k8s_cluster.properties.k8s_version}")
       expect(subject).to receive(:puts).with("Maintenance Window: #{maintenance_window}")
       expect(subject).to receive(:puts).with("State: #{k8s_cluster.metadata.state}")
+      expect(subject).to receive(:puts).with("Api Subnet Allow List: #{k8s_cluster.properties.api_subnet_allow_list}")
+      expect(subject).to receive(:puts).with("S3 Buckets: #{s3_buckets}")
+      expect(subject).to receive(:puts).with("Available Upgrade Versions: #{k8s_cluster.properties.available_upgrade_versions}")
+      expect(subject).to receive(:puts).with("Viable NodePool Versions: #{k8s_cluster.properties.viable_node_pool_versions}")
       expect(subject.ui).to receive(:warn).with("Deleted K8s Cluster #{k8s_cluster.id}. Request ID: ")
 
       expect(subject).not_to receive(:wait_for)

data/spec/chef/knife/ionoscloud_k8s_get_spec.rb

@@ -0,0 +1,73 @@
+require 'spec_helper'
+require 'ionoscloud_k8s_get'
+
+Chef::Knife::IonoscloudK8sGet.load_deps
+
+describe Chef::Knife::IonoscloudK8sGet do
+  before :each do
+    subject { Chef::Knife::IonoscloudK8sGet.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call KubernetesApi.k8s_find_by_cluster_id' do
+      cluster = k8s_cluster_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        cluster_id: cluster.id,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      maintenance_window = "#{cluster.properties.maintenance_window.day_of_the_week}, #{cluster.properties.maintenance_window.time}"
+      s3_buckets = (cluster.properties.s3_buckets.nil? ? [] : cluster.properties.s3_buckets.map { |el| el.name })
+
+      expect(subject).to receive(:puts).with("ID: #{cluster.id}")
+      expect(subject).to receive(:puts).with("Name: #{cluster.properties.name}")
+      expect(subject).to receive(:puts).with("Public: #{cluster.properties.public}")
+      expect(subject).to receive(:puts).with("k8s Version: #{cluster.properties.k8s_version}")
+      expect(subject).to receive(:puts).with("Maintenance Window: #{maintenance_window}")
+      expect(subject).to receive(:puts).with("State: #{cluster.metadata.state}")
+      expect(subject).to receive(:puts).with("Api Subnet Allow List: #{cluster.properties.api_subnet_allow_list}")
+      expect(subject).to receive(:puts).with("S3 Buckets: #{s3_buckets}")
+      expect(subject).to receive(:puts).with("Available Upgrade Versions: #{cluster.properties.available_upgrade_versions}")
+      expect(subject).to receive(:puts).with("Viable NodePool Versions: #{cluster.properties.viable_node_pool_versions}")
+
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/k8s/#{cluster.id}",
+            operation: :'KubernetesApi.k8s_find_by_cluster_id',
+            return_type: 'KubernetesCluster',
+            result: cluster,
+          },
+        ],
+      )
+
+      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_k8s_update_spec.rb

@@ -0,0 +1,115 @@
+require 'spec_helper'
+require 'ionoscloud_k8s_update'
+
+Chef::Knife::IonoscloudK8sUpdate.load_deps
+
+describe Chef::Knife::IonoscloudK8sUpdate do
+  before :each do
+    subject { Chef::Knife::IonoscloudK8sUpdate.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call NicApi.datacenters_servers_nics_clusterrules_patch' do
+      cluster = k8s_cluster_mock
+
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        cluster_id: cluster.id,
+        name: cluster.properties.name + '_edited',
+        version: '19.9.9',
+        maintenance_day: 'Monday',
+        maintenance_time: '13:03:19Z',
+        api_subnet_allow_list: (cluster.properties.api_subnet_allow_list + ['1.1.1.1']).join(','),
+        s3_buckets: (cluster.properties.s3_buckets.map { |el| el.name } + ['new_bucket']).join(','),
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      maintenance_window = "#{subject_config[:maintenance_day]}, #{subject_config[:maintenance_time]}"
+      s3_buckets = (cluster.properties.s3_buckets.map { |el| el.name } + ['new_bucket'])
+
+      expect(subject).to receive(:puts).with("ID: #{cluster.id}")
+      expect(subject).to receive(:puts).with("Name: #{subject_config[:name]}")
+      expect(subject).to receive(:puts).with("Public: #{cluster.properties.public}")
+      expect(subject).to receive(:puts).with("k8s Version: #{subject_config[:version]}")
+      expect(subject).to receive(:puts).with("Maintenance Window: #{maintenance_window}")
+      expect(subject).to receive(:puts).with("State: #{cluster.metadata.state}")
+      expect(subject).to receive(:puts).with("Api Subnet Allow List: #{cluster.properties.api_subnet_allow_list + ['1.1.1.1']}")
+      expect(subject).to receive(:puts).with("S3 Buckets: #{s3_buckets}")
+      expect(subject).to receive(:puts).with("Available Upgrade Versions: #{cluster.properties.available_upgrade_versions}")
+      expect(subject).to receive(:puts).with("Viable NodePool Versions: #{cluster.properties.viable_node_pool_versions}")
+
+      cluster.properties.name = subject_config[:name]
+      cluster.properties.k8s_version = subject_config[:version]
+      cluster.properties.maintenance_window.day_of_the_week = subject_config[:maintenance_day]
+      cluster.properties.maintenance_window.time = subject_config[:maintenance_time]
+      cluster.properties.api_subnet_allow_list = subject_config[:api_subnet_allow_list].split(',')
+      cluster.properties.s3_buckets = subject_config[:s3_buckets].split(',').map { |el| Ionoscloud::S3Bucket.new(name: el) }
+
+      expected_body = {
+        name: subject_config[:name],
+        k8sVersion: subject_config[:version],
+        maintenanceWindow: {
+          dayOfTheWeek: subject_config[:maintenance_day],
+          time: subject_config[:maintenance_time],
+        },
+        apiSubnetAllowList: subject_config[:api_subnet_allow_list].split(','),
+        s3Buckets: subject_config[:s3_buckets].split(',').map { |el| { name: el } },
+      }
+
+      expect(subject.api_client).not_to receive(:wait_for)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/k8s/#{cluster.id}",
+            operation: :'KubernetesApi.k8s_find_by_cluster_id',
+            return_type: 'KubernetesCluster',
+            result: cluster,
+          },
+          {
+            method: 'PUT',
+            path: "/k8s/#{cluster.id}",
+            operation: :'KubernetesApi.k8s_put',
+            return_type: 'KubernetesCluster',
+            body: { properties: expected_body },
+            result: cluster,
+          },
+          {
+            method: 'GET',
+            path: "/k8s/#{cluster.id}",
+            operation: :'KubernetesApi.k8s_find_by_cluster_id',
+            return_type: 'KubernetesCluster',
+            result: cluster,
+          },
+        ],
+      )
+
+      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_lan_create_spec.rb

@@ -29,6 +29,7 @@ describe Chef::Knife::IonoscloudLanCreate do
       expect(subject).to receive(:puts).with("Name: #{lan.properties.name}")
       expect(subject).to receive(:puts).with("Public: #{lan.properties.public.to_s}")
       expect(subject).to receive(:puts).with("PCC: #{lan.properties.pcc}")
+      expect(subject).to receive(:puts).with("IP Failover: #{[]}")
 
       expected_body = lan.properties.to_hash
       expected_body.delete(:ipFailover)

data/spec/chef/knife/ionoscloud_lan_delete_spec.rb

@@ -27,6 +27,8 @@ describe Chef::Knife::IonoscloudLanDelete do
       expect(subject).to receive(:puts).with("ID: #{lan.id}")
       expect(subject).to receive(:puts).with("Name: #{lan.properties.name}")
       expect(subject).to receive(:puts).with("Public: #{lan.properties.public.to_s}")
+      expect(subject).to receive(:puts).with("PCC: #{lan.properties.pcc}")
+      expect(subject).to receive(:puts).with("IP Failover: #{[]}")
       expect(subject.ui).to receive(:warn).with("Deleted Lan #{lan.id}. Request ID: ")
 
       expect(subject.api_client).not_to receive(:wait_for)