knife-ionoscloud 5.1.1 → 5.1.2

data/spec/chef/knife/ionoscloud_lan_get_spec.rb

@@ -0,0 +1,68 @@
+require 'spec_helper'
+require 'ionoscloud_lan_get'
+
+Chef::Knife::IonoscloudLanGet.load_deps
+
+describe Chef::Knife::IonoscloudLanGet do
+  before :each do
+    subject { Chef::Knife::IonoscloudLanGet.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call LanApi.datacenters_lans_find_by_id' do
+      lan = lan_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: 'datacenter_id',
+        lan_id: lan.id,
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      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.api_client).not_to receive(:wait_for)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/lans/#{subject_config[:lan_id]}",
+            operation: :'LanApi.datacenters_lans_find_by_id',
+            return_type: 'Lan',
+            result: lan,
+          },
+        ],
+      )
+
+      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_list_spec.rb

@@ -27,6 +27,7 @@ describe Chef::Knife::IonoscloudLanList do
         subject.ui.color('Name', :bold),
         subject.ui.color('Public', :bold),
         subject.ui.color('PCC', :bold),
+        subject.ui.color('IP Failover Count', :bold),
       ]
 
       lans.items.each do |lan|
@@ -34,9 +35,10 @@ describe Chef::Knife::IonoscloudLanList do
         lan_list << lan.properties.name
         lan_list << lan.properties.public.to_s
         lan_list << lan.properties.pcc
+        lan_list << (lan.properties.ip_failover.nil? ? 0 : lan.properties.ip_failover.length)
       end
 
-      expect(subject.ui).to receive(:list).with(lan_list, :uneven_columns_across, 4)
+      expect(subject.ui).to receive(:list).with(lan_list, :uneven_columns_across, 5)
 
       mock_call_api(
         subject,

data/spec/chef/knife/ionoscloud_lan_update_spec.rb

@@ -0,0 +1,91 @@
+require 'spec_helper'
+require 'ionoscloud_lan_update'
+
+Chef::Knife::IonoscloudLanUpdate.load_deps
+
+describe Chef::Knife::IonoscloudLanUpdate do
+  before :each do
+    subject { Chef::Knife::IonoscloudLanUpdate.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call LanApi.datacenters_lans_patch' do
+      lan = lan_mock
+      uuid = SecureRandom.uuid
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: 'datacenter_id',
+        lan_id: lan.id,
+        name: lan.properties.name + '_edited',
+        public: (!lan.properties.public).to_s,
+        ip_failover: [{ 'ip' => '1.1.1.1', 'nic_uuid' => uuid }],
+        pcc: 'pcc_id',
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject).to receive(:puts).with("ID: #{lan.id}")
+      expect(subject).to receive(:puts).with("Name: #{subject_config[:name]}")
+      expect(subject).to receive(:puts).with("Public: #{subject_config[:public]}")
+      expect(subject).to receive(:puts).with("PCC: #{subject_config[:pcc]}")
+      expect(subject).to receive(:puts).with("IP Failover: #{[{ ip: '1.1.1.1', nicUuid: uuid }]}")
+
+      lan.properties.name = subject_config[:name]
+      lan.properties.public = subject_config[:public].to_s.downcase == 'true'
+      lan.properties.pcc = subject_config[:pcc]
+      lan.properties.ip_failover = [Ionoscloud::IPFailover.new(ip: '1.1.1.1', nic_uuid: uuid)]
+
+      mock_wait_for(subject)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'PATCH',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/lans/#{subject_config[:lan_id]}",
+            operation: :'LanApi.datacenters_lans_patch',
+            return_type: 'Lan',
+            body: {
+              name: subject_config[:name],
+              public: subject_config[:public].to_s.downcase == 'true',
+              pcc: subject_config[:pcc],
+              ipFailover: [{ ip: '1.1.1.1', nicUuid: uuid }],
+            },
+            result: lan,
+          },
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/lans/#{subject_config[:lan_id]}",
+            operation: :'LanApi.datacenters_lans_find_by_id',
+            return_type: 'Lan',
+            result: lan,
+          },
+        ],
+      )
+
+      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_loadbalancer_update_spec.rb

@@ -0,0 +1,89 @@
+require 'spec_helper'
+require 'ionoscloud_loadbalancer_update'
+
+Chef::Knife::IonoscloudLoadbalancerUpdate.load_deps
+
+describe Chef::Knife::IonoscloudLoadbalancerUpdate do
+  before :each do
+    subject { Chef::Knife::IonoscloudLoadbalancerUpdate.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call LoadBalancerApi.datacenters_loadbalancers_patch' do
+      load_balancer = load_balancer_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: 'datacenter_id',
+        loadbalancer_id: load_balancer.id,
+        name: load_balancer.properties.name + '_edited',
+        dhcp: (!load_balancer.properties.dhcp).to_s,
+        ip: '3.3.3.3',
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      nics = load_balancer.entities.balancednics.items.map { |nic| nic.id }
+
+      expect(subject).to receive(:puts).with("ID: #{load_balancer.id}")
+      expect(subject).to receive(:puts).with("Name: #{subject_config[:name]}")
+      expect(subject).to receive(:puts).with("IP address: #{subject_config[:ip]}")
+      expect(subject).to receive(:puts).with("DHCP: #{subject_config[:dhcp]}")
+      expect(subject).to receive(:puts).with("Balanced Nics: #{nics.to_s}")
+
+      load_balancer.properties.name = subject_config[:name]
+      load_balancer.properties.dhcp = subject_config[:dhcp].to_s.downcase == 'true'
+      load_balancer.properties.ip = subject_config[:ip]
+
+      mock_wait_for(subject)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'PATCH',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/loadbalancers/#{subject_config[:loadbalancer_id]}",
+            operation: :'LoadBalancerApi.datacenters_loadbalancers_patch',
+            return_type: 'Loadbalancer',
+            body: {
+              name: subject_config[:name],
+              dhcp: subject_config[:dhcp].to_s.downcase == 'true',
+              ip: subject_config[:ip],
+            },
+            result: load_balancer,
+          },
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/loadbalancers/#{subject_config[:loadbalancer_id]}",
+            operation: :'LoadBalancerApi.datacenters_loadbalancers_find_by_id',
+            return_type: 'Loadbalancer',
+            result: load_balancer,
+          },
+        ],
+      )
+
+      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_nic_get_spec.rb

@@ -0,0 +1,70 @@
+require 'spec_helper'
+require 'ionoscloud_nic_get'
+
+Chef::Knife::IonoscloudNicGet.load_deps
+
+describe Chef::Knife::IonoscloudNicGet do
+  before :each do
+    subject { Chef::Knife::IonoscloudNicGet.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call NicApi.datacenters_servers_nics_find_by_id' do
+      nic = nic_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: 'datacenter_id',
+        server_id: 'server_id',
+        nic_id: nic.id,
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject).to receive(:puts).with("ID: #{nic.id}")
+      expect(subject).to receive(:puts).with("Name: #{nic.properties.name}")
+      expect(subject).to receive(:puts).with("IPs: #{nic.properties.ips.to_s}")
+      expect(subject).to receive(:puts).with("DHCP: #{nic.properties.dhcp}")
+      expect(subject).to receive(:puts).with("LAN: #{nic.properties.lan}")
+      expect(subject).to receive(:puts).with("NAT: #{nic.properties.nat}")
+
+      expect(subject.api_client).not_to receive(:wait_for)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/servers/#{subject_config[:server_id]}/nics/#{subject_config[:nic_id]}",
+            operation: :'NicApi.datacenters_servers_nics_find_by_id',
+            return_type: 'Nic',
+            result: nic,
+          },
+        ],
+      )
+
+      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_nic_update_spec.rb

@@ -0,0 +1,92 @@
+require 'spec_helper'
+require 'ionoscloud_nic_update'
+
+Chef::Knife::IonoscloudNicUpdate.load_deps
+
+describe Chef::Knife::IonoscloudNicUpdate do
+  before :each do
+    subject { Chef::Knife::IonoscloudNicUpdate.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call NicApi.datacenters_servers_nics_patch' do
+      nic = nic_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: 'datacenter_id',
+        server_id: 'server_id',
+        nic_id: nic.id,
+        lan: 13,
+        name: nic.properties.name + '_edited',
+        dhcp: (!nic.properties.dhcp).to_s,
+        ips: (nic.properties.ips + ['3.3.3.3']).join(','),
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject).to receive(:puts).with("ID: #{nic.id}")
+      expect(subject).to receive(:puts).with("Name: #{subject_config[:name]}")
+      expect(subject).to receive(:puts).with("IPs: #{subject_config[:ips].split(',').to_s}")
+      expect(subject).to receive(:puts).with("DHCP: #{subject_config[:dhcp]}")
+      expect(subject).to receive(:puts).with("LAN: #{subject_config[:lan]}")
+      expect(subject).to receive(:puts).with("NAT: #{nic.properties.nat}")
+
+      nic.properties.name = subject_config[:name]
+      nic.properties.dhcp = subject_config[:dhcp].to_s.downcase == 'true'
+      nic.properties.ips = subject_config[:ips].split(',')
+      nic.properties.lan = subject_config[:lan]
+
+      mock_wait_for(subject)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'PATCH',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/servers/#{subject_config[:server_id]}/nics/#{subject_config[:nic_id]}",
+            operation: :'NicApi.datacenters_servers_nics_patch',
+            return_type: 'Nic',
+            body: {
+              name: subject_config[:name],
+              dhcp: subject_config[:dhcp].to_s.downcase == 'true',
+              ips: subject_config[:ips].split(','),
+              lan: subject_config[:lan],
+            },
+            result: nic,
+          },
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/servers/#{subject_config[:server_id]}/nics/#{subject_config[:nic_id]}",
+            operation: :'NicApi.datacenters_servers_nics_find_by_id',
+            return_type: 'Nic',
+            result: nic,
+          },
+        ],
+      )
+
+      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_node_get_spec.rb

@@ -0,0 +1,70 @@
+require 'spec_helper'
+require 'ionoscloud_node_get'
+
+Chef::Knife::IonoscloudNodeGet.load_deps
+
+describe Chef::Knife::IonoscloudNodeGet do
+  before :each do
+    subject { Chef::Knife::IonoscloudNodeGet.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call KubernetesApi.k8s_nodepools_nodes_find_by_id' do
+      k8s_node = k8s_node_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        cluster_id: 'cluster_id',
+        nodepool_id: 'nodepool_id',
+        node_id: k8s_node.id,
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject).to receive(:puts).with("ID: #{k8s_node.id}")
+      expect(subject).to receive(:puts).with("Name: #{k8s_node.properties.name}")
+      expect(subject).to receive(:puts).with("Public IP: #{k8s_node.properties.public_ip}")
+      expect(subject).to receive(:puts).with("Private IP: #{k8s_node.properties.private_ip}")
+      expect(subject).to receive(:puts).with("K8s Version: #{k8s_node.properties.k8s_version}")
+      expect(subject).to receive(:puts).with("State: #{k8s_node.metadata.state}")
+
+      expect(subject.api_client).not_to receive(:wait_for)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/k8s/#{subject_config[:cluster_id]}/nodepools/#{subject_config[:nodepool_id]}/nodes/#{subject_config[:node_id]}",
+            operation: :'KubernetesApi.k8s_nodepools_nodes_find_by_id',
+            return_type: 'KubernetesNode',
+            result: k8s_node,
+          },
+        ],
+      )
+
+      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_nodepool_create_spec.rb

@@ -20,7 +20,7 @@ describe Chef::Knife::IonoscloudNodepoolCreate do
         cluster_id: 'cluster_id',
         datacenter_id: nodepool.properties.datacenter_id,
         name: nodepool.properties.name,
-        version: nodepool.properties.k8s_version,
+        k8s_version: nodepool.properties.k8s_version,
         maintenance_day: nodepool.properties.maintenance_window.day_of_the_week,
         maintenance_time: nodepool.properties.maintenance_window.time,
         node_count: nodepool.properties.node_count,
@@ -40,10 +40,7 @@ describe Chef::Knife::IonoscloudNodepoolCreate do
 
       auto_scaling = "Min node count: #{nodepool.properties.auto_scaling.min_node_count}, Max node count:#{nodepool.properties.auto_scaling.max_node_count}"
       maintenance_window = "#{nodepool.properties.maintenance_window.day_of_the_week}, #{nodepool.properties.maintenance_window.time}"
-
-      expected_body = nodepool.properties.to_hash
-      expected_body[:lans].map! { |lan| lan.delete(:properties); lan[:id] = Integer(lan[:id]); lan }
-      expected_body.delete(:availableUpgradeVersions)
+      lans = nodepool.properties.lans.map { |lan| { id: lan.id } }
 
       expect(subject).to receive(:puts).with("ID: #{nodepool.id}")
       expect(subject).to receive(:puts).with("Name: #{nodepool.properties.name}")
@@ -55,11 +52,21 @@ describe Chef::Knife::IonoscloudNodepoolCreate do
       expect(subject).to receive(:puts).with("RAM: #{nodepool.properties.ram_size}")
       expect(subject).to receive(:puts).with("Storage Type: #{nodepool.properties.storage_type}")
       expect(subject).to receive(:puts).with("Storage Size: #{nodepool.properties.storage_size}")
+      expect(subject).to receive(:puts).with("Public IPs: #{nodepool.properties.public_ips}")
+      expect(subject).to receive(:puts).with("Labels: #{nodepool.properties.labels}")
+      expect(subject).to receive(:puts).with("Annotations: #{nodepool.properties.annotations}")
+      expect(subject).to receive(:puts).with("LANs: #{lans}")
       expect(subject).to receive(:puts).with("Availability Zone: #{nodepool.properties.availability_zone}")
       expect(subject).to receive(:puts).with("Auto Scaling: #{auto_scaling}")
       expect(subject).to receive(:puts).with("Maintenance Window: #{maintenance_window}")
       expect(subject).to receive(:puts).with("State: #{nodepool.metadata.state}")
 
+      expected_body = nodepool.properties.to_hash
+      expected_body[:lans].map! { |lan| lan.delete(:properties); lan[:id] = Integer(lan[:id]); lan }
+      expected_body.delete(:availableUpgradeVersions)
+      expected_body.delete(:labels)
+      expected_body.delete(:annotations)
+
       expect(subject.api_client).not_to receive(:wait_for)
       mock_call_api(
         subject,

data/spec/chef/knife/ionoscloud_nodepool_delete_spec.rb

@@ -13,7 +13,7 @@ describe Chef::Knife::IonoscloudNodepoolDelete do
 
   describe '#run' do
     it 'should call KubernetesApi.k8s_nodepools_delete when the ID is valid' do
-      k8s_nodepool = k8s_nodepool_mock
+      nodepool = k8s_nodepool_mock
       subject_config = {
         ionoscloud_username: 'email',
         ionoscloud_password: 'password',
@@ -22,15 +22,29 @@ describe Chef::Knife::IonoscloudNodepoolDelete do
       }
 
       subject_config.each { |key, value| subject.config[key] = value }
-      subject.name_args = [k8s_nodepool.id]
-
-      expect(subject).to receive(:puts).with("ID: #{k8s_nodepool.id}")
-      expect(subject).to receive(:puts).with("Name: #{k8s_nodepool.properties.name}")
-      expect(subject).to receive(:puts).with("K8s Version: #{k8s_nodepool.properties.k8s_version}")
-      expect(subject).to receive(:puts).with("Datacenter ID: #{k8s_nodepool.properties.datacenter_id}")
-      expect(subject).to receive(:puts).with("Node Count: #{k8s_nodepool.properties.node_count}")
-      expect(subject).to receive(:puts).with("State: #{k8s_nodepool.metadata.state}")
-      expect(subject.ui).to receive(:warn).with("Deleted K8s Nodepool #{k8s_nodepool.id}.")
+      subject.name_args = [nodepool.id]
+
+      auto_scaling = "Min node count: #{nodepool.properties.auto_scaling.min_node_count}, Max node count:#{nodepool.properties.auto_scaling.max_node_count}"
+      maintenance_window = "#{nodepool.properties.maintenance_window.day_of_the_week}, #{nodepool.properties.maintenance_window.time}"
+      lans = nodepool.properties.lans.map { |lan| { id: lan.id } }
+
+      expect(subject).to receive(:puts).with("ID: #{nodepool.id}")
+      expect(subject).to receive(:puts).with("Name: #{nodepool.properties.name}")
+      expect(subject).to receive(:puts).with("K8s Version: #{nodepool.properties.k8s_version}")
+      expect(subject).to receive(:puts).with("Datacenter ID: #{nodepool.properties.datacenter_id}")
+      expect(subject).to receive(:puts).with("Node Count: #{nodepool.properties.node_count}")
+      expect(subject).to receive(:puts).with("CPU Family: #{nodepool.properties.cpu_family}")
+      expect(subject).to receive(:puts).with("Cores Count: #{nodepool.properties.cores_count}")
+      expect(subject).to receive(:puts).with("RAM: #{nodepool.properties.ram_size}")
+      expect(subject).to receive(:puts).with("Storage Type: #{nodepool.properties.storage_type}")
+      expect(subject).to receive(:puts).with("Storage Size: #{nodepool.properties.storage_size}")
+      expect(subject).to receive(:puts).with("Public IPs: #{nodepool.properties.public_ips}")
+      expect(subject).to receive(:puts).with("LANs: #{lans}")
+      expect(subject).to receive(:puts).with("Availability Zone: #{nodepool.properties.availability_zone}")
+      expect(subject).to receive(:puts).with("Auto Scaling: #{auto_scaling}")
+      expect(subject).to receive(:puts).with("Maintenance Window: #{maintenance_window}")
+      expect(subject).to receive(:puts).with("State: #{nodepool.metadata.state}")
+      expect(subject.ui).to receive(:warn).with("Deleted K8s Nodepool #{nodepool.id}.")
 
       expect(subject.api_client).not_to receive(:wait_for)
       expect(subject).not_to receive(:get_request_id)
@@ -39,14 +53,14 @@ describe Chef::Knife::IonoscloudNodepoolDelete do
         [
           {
             method: 'GET',
-            path: "/k8s/#{subject_config[:cluster_id]}/nodepools/#{k8s_nodepool.id}",
+            path: "/k8s/#{subject_config[:cluster_id]}/nodepools/#{nodepool.id}",
             operation: :'KubernetesApi.k8s_nodepools_find_by_id',
             return_type: 'KubernetesNodePool',
-            result: k8s_nodepool,
+            result: nodepool,
           },
           {
             method: 'DELETE',
-            path: "/k8s/#{subject_config[:cluster_id]}/nodepools/#{k8s_nodepool.id}",
+            path: "/k8s/#{subject_config[:cluster_id]}/nodepools/#{nodepool.id}",
             operation: :'KubernetesApi.k8s_nodepools_delete',
           },
         ],