knife-ionoscloud 6.1.0 → 6.2.0

data/spec/chef/knife/ionoscloud_nodepool_lan_add_spec.rb

@@ -90,7 +90,7 @@ describe Chef::Knife::IonoscloudNodepoolLanAdd do
     end
 
     it 'should call KubernetesApi.k8s_nodepools_put to update the lan when the ID is valid' do
-      new_gateway = '1.11.1.1'
+      new_gateway = '127.0.0.3'
       nodepool = k8s_nodepool_mock()
       nodepool_lan = nodepool_lan_mock(id: nodepool.properties.lans.first.id, gateway_ip: new_gateway)
       subject_config = {
@@ -168,7 +168,6 @@ describe Chef::Knife::IonoscloudNodepoolLanAdd do
 
     it 'should not call LoadBalancersApi.datacenters_loadbalancers_delete when the ID is not valid' do
       nodepool = k8s_nodepool_mock
-      nic_id = 'invalid_id'
       subject_config = {
         ionoscloud_username: 'email',
         ionoscloud_password: 'password',
@@ -204,18 +203,20 @@ describe Chef::Knife::IonoscloudNodepoolLanAdd do
     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|
+      if required_options.length > 0
+        arrays_without_one_element(required_options).each do |test_case|
+          subject.config[:ionoscloud_token] = 'token'
+          test_case[:array].each { |value| subject.config[value] = 'test' }
 
-        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).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
 
-        expect { subject.run }.to raise_error(SystemExit) do |error|
-          expect(error.status).to eq(1)
+          required_options.each { |value| subject.config[value] = nil }
         end
-
-        required_options.each { |value| subject.config[value] = nil }
       end
     end
   end

data/spec/chef/knife/{ionoscloud_nodepool_lan_remove.rb → ionoscloud_nodepool_lan_remove_spec.rb}

@@ -98,28 +98,25 @@ describe Chef::Knife::IonoscloudNodepoolLanRemove do
       subject_config.each { |key, value| subject.config[key] = value }
       subject.name_args = [123]
 
+      expect(subject.ui).to receive(:info).with("None of the specified Lan IDs (#{subject.name_args}) are present on the Nodepoool.")
+
       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| lan.to_hash }
 
-      expect(subject.ui).to receive(:info).with("None of the specified Lan IDs (#{subject.name_args}) are present on the Nodepoool.")
       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("Lans: #{nodepool.properties.lans.map do
-        |lan|
-        {
-          id: lan.id,
-          dhcp: lan.dhcp,
-          routes: lan.routes ? lan.routes.map do
-            |route|
-            {
-              network: route.network,
-              gateway_ip: route.gateway_ip,
-            }
-          end : []
-        }
-      end}")
+      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}")
@@ -142,7 +139,6 @@ describe Chef::Knife::IonoscloudNodepoolLanRemove do
 
     it 'should not call LoadBalancersApi.datacenters_loadbalancers_delete when the ID is not valid' do
       nodepool = k8s_nodepool_mock
-      nic_id = 'invalid_id'
       subject_config = {
         ionoscloud_username: 'email',
         ionoscloud_password: 'password',
@@ -178,18 +174,20 @@ describe Chef::Knife::IonoscloudNodepoolLanRemove do
     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|
+      if required_options.length > 0
+        arrays_without_one_element(required_options).each do |test_case|
+          subject.config[:ionoscloud_token] = 'token'
+          test_case[:array].each { |value| subject.config[value] = 'test' }
 
-        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).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
 
-        expect { subject.run }.to raise_error(SystemExit) do |error|
-          expect(error.status).to eq(1)
+          required_options.each { |value| subject.config[value] = nil }
         end
-
-        required_options.each { |value| subject.config[value] = nil }
       end
     end
   end

data/spec/chef/knife/ionoscloud_nodepool_list_spec.rb

@@ -60,18 +60,20 @@ describe Chef::Knife::IonoscloudNodepoolList do
     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|
+      if required_options.length > 0
+        arrays_without_one_element(required_options).each do |test_case|
+          subject.config[:ionoscloud_token] = 'token'
+          test_case[:array].each { |value| subject.config[value] = 'test' }
 
-        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).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
 
-        expect { subject.run }.to raise_error(SystemExit) do |error|
-          expect(error.status).to eq(1)
+          required_options.each { |value| subject.config[value] = nil }
         end
-
-        required_options.each { |value| subject.config[value] = nil }
       end
     end
   end

data/spec/chef/knife/ionoscloud_nodepool_update_spec.rb

@@ -21,8 +21,8 @@ describe Chef::Knife::IonoscloudNodepoolUpdate do
         nodepool_id: nodepool.id,
         k8s_version: '18.18.18',
         node_count: nodepool.properties.node_count + 1,
-        public_ips: '1.1.1.1,2.2.2.2,3.3.3.3',
-        labels: { "new_test": "new_test", "new_test2": "new_test2" },
+        public_ips: '127.0.0.3,127.0.0.4,127.0.0.5',
+        labels: { "new_test": 'new_test', "new_test2": 'new_test2' },
         lans: '13',
         maintenance_day: 'Tuesday',
         maintenance_time: '03:48:30Z',
@@ -116,18 +116,20 @@ describe Chef::Knife::IonoscloudNodepoolUpdate do
     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|
+      if required_options.length > 0
+        arrays_without_one_element(required_options).each do |test_case|
+          subject.config[:ionoscloud_token] = 'token'
+          test_case[:array].each { |value| subject.config[value] = 'test' }
 
-        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).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
 
-        expect { subject.run }.to raise_error(SystemExit) do |error|
-          expect(error.status).to eq(1)
+          required_options.each { |value| subject.config[value] = nil }
         end
-
-        required_options.each { |value| subject.config[value] = nil }
       end
     end
   end

data/spec/chef/knife/ionoscloud_pcc_create_spec.rb

@@ -86,18 +86,20 @@ describe Chef::Knife::IonoscloudPccCreate do
     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|
+      if required_options.length > 0
+        arrays_without_one_element(required_options).each do |test_case|
+          subject.config[:ionoscloud_token] = 'token'
+          test_case[:array].each { |value| subject.config[value] = 'test' }
 
-        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).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
 
-        expect { subject.run }.to raise_error(SystemExit) do |error|
-          expect(error.status).to eq(1)
+          required_options.each { |value| subject.config[value] = nil }
         end
-
-        required_options.each { |value| subject.config[value] = nil }
       end
     end
   end

data/spec/chef/knife/ionoscloud_pcc_delete_spec.rb

@@ -88,18 +88,20 @@ describe Chef::Knife::IonoscloudPccDelete do
     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|
+      if required_options.length > 0
+        arrays_without_one_element(required_options).each do |test_case|
+          subject.config[:ionoscloud_token] = 'token'
+          test_case[:array].each { |value| subject.config[value] = 'test' }
 
-        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).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
 
-        expect { subject.run }.to raise_error(SystemExit) do |error|
-          expect(error.status).to eq(1)
+          required_options.each { |value| subject.config[value] = nil }
         end
-
-        required_options.each { |value| subject.config[value] = nil }
       end
     end
   end

data/spec/chef/knife/ionoscloud_pcc_get_spec.rb

@@ -53,18 +53,20 @@ describe Chef::Knife::IonoscloudPccGet do
     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|
+      if required_options.length > 0
+        arrays_without_one_element(required_options).each do |test_case|
+          subject.config[:ionoscloud_token] = 'token'
+          test_case[:array].each { |value| subject.config[value] = 'test' }
 
-        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).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
 
-        expect { subject.run }.to raise_error(SystemExit) do |error|
-          expect(error.status).to eq(1)
+          required_options.each { |value| subject.config[value] = nil }
         end
-
-        required_options.each { |value| subject.config[value] = nil }
       end
     end
   end

data/spec/chef/knife/ionoscloud_pcc_list_spec.rb

@@ -51,18 +51,20 @@ describe Chef::Knife::IonoscloudPccList do
     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|
+      if required_options.length > 0
+        arrays_without_one_element(required_options).each do |test_case|
+          subject.config[:ionoscloud_token] = 'token'
+          test_case[:array].each { |value| subject.config[value] = 'test' }
 
-        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).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
 
-        expect { subject.run }.to raise_error(SystemExit) do |error|
-          expect(error.status).to eq(1)
+          required_options.each { |value| subject.config[value] = nil }
         end
-
-        required_options.each { |value| subject.config[value] = nil }
       end
     end
   end

data/spec/chef/knife/ionoscloud_pcc_update_spec.rb

@@ -65,18 +65,20 @@ describe Chef::Knife::IonoscloudPccUpdate do
     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|
+      if required_options.length > 0
+        arrays_without_one_element(required_options).each do |test_case|
+          subject.config[:ionoscloud_token] = 'token'
+          test_case[:array].each { |value| subject.config[value] = 'test' }
 
-        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).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
 
-        expect { subject.run }.to raise_error(SystemExit) do |error|
-          expect(error.status).to eq(1)
+          required_options.each { |value| subject.config[value] = nil }
         end
-
-        required_options.each { |value| subject.config[value] = nil }
       end
     end
   end

data/spec/chef/knife/ionoscloud_request_get_spec.rb

@@ -51,18 +51,20 @@ describe Chef::Knife::IonoscloudRequestGet do
     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|
+      if required_options.length > 0
+        arrays_without_one_element(required_options).each do |test_case|
+          subject.config[:ionoscloud_token] = 'token'
+          test_case[:array].each { |value| subject.config[value] = 'test' }
 
-        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).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
 
-        expect { subject.run }.to raise_error(SystemExit) do |error|
-          expect(error.status).to eq(1)
+          required_options.each { |value| subject.config[value] = nil }
         end
-
-        required_options.each { |value| subject.config[value] = nil }
       end
     end
   end

data/spec/chef/knife/ionoscloud_request_list_spec.rb

@@ -8,7 +8,7 @@ describe Chef::Knife::IonoscloudRequestList do
     subject { Chef::Knife::IonoscloudRequestList.new }
 
     @requests = requests_mock
-    @request_list = request_list = [
+    @request_list = [
       subject.ui.color('ID', :bold),
       subject.ui.color('Status', :bold),
       subject.ui.color('Method', :bold),
@@ -20,7 +20,6 @@ describe Chef::Knife::IonoscloudRequestList do
       @request_list << request.metadata.request_status.metadata.status
       @request_list << request.properties.method
       @request_list << request.metadata.request_status.metadata.targets.map { |target| [target.target.id, target.target.type] }.to_s
-
     end
 
     allow(subject).to receive(:puts)
@@ -266,18 +265,20 @@ describe Chef::Knife::IonoscloudRequestList do
     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|
+      if required_options.length > 0
+        arrays_without_one_element(required_options).each do |test_case|
+          subject.config[:ionoscloud_token] = 'token'
+          test_case[:array].each { |value| subject.config[value] = 'test' }
 
-        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).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
 
-        expect { subject.run }.to raise_error(SystemExit) do |error|
-          expect(error.status).to eq(1)
+          required_options.each { |value| subject.config[value] = nil }
         end
-
-        required_options.each { |value| subject.config[value] = nil }
       end
     end
   end

data/spec/chef/knife/ionoscloud_request_status_spec.rb

@@ -70,23 +70,22 @@ describe Chef::Knife::IonoscloudRequestStatus do
 
     it 'should not make any call if any required option is missing' do
       required_options = subject.instance_variable_get(:@required_options)
-      allow(subject).to receive(:puts)
-      allow(subject).to receive(:print)
 
-      arrays_without_one_element(required_options).each {
-        |test_case|
+      if required_options.length > 0
+        arrays_without_one_element(required_options).each do |test_case|
+          subject.config[:ionoscloud_token] = 'token'
+          test_case[:array].each { |value| subject.config[value] = 'test' }
 
-        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).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
 
-        expect { subject.run }.to raise_error(SystemExit) do |error|
-          expect(error.status).to eq(1)
+          required_options.each { |value| subject.config[value] = nil }
         end
-
-        required_options.each { |value| subject.config[value] = nil }
-      }
+      end
     end
   end
 end

data/spec/chef/knife/ionoscloud_request_wait_spec.rb

@@ -8,7 +8,6 @@ describe Chef::Knife::IonoscloudRequestWait do
 
   describe '#run' do
     it 'should call api_client.wait_for' do
-      request_status = request_status_mock
       subject_config = {
         ionoscloud_username: 'email',
         ionoscloud_password: 'password',
@@ -52,23 +51,22 @@ describe Chef::Knife::IonoscloudRequestWait do
 
     it 'should not make any call if any required option is missing' do
       required_options = subject.instance_variable_get(:@required_options)
-      allow(subject).to receive(:puts)
-      allow(subject).to receive(:print)
 
-      arrays_without_one_element(required_options).each {
-        |test_case|
+      if required_options.length > 0
+        arrays_without_one_element(required_options).each do |test_case|
+          subject.config[:ionoscloud_token] = 'token'
+          test_case[:array].each { |value| subject.config[value] = 'test' }
 
-        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).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
 
-        expect { subject.run }.to raise_error(SystemExit) do |error|
-          expect(error.status).to eq(1)
+          required_options.each { |value| subject.config[value] = nil }
         end
-
-        required_options.each { |value| subject.config[value] = nil }
-      }
+      end
     end
   end
 end

data/spec/chef/knife/ionoscloud_resource_list_spec.rb

@@ -5,7 +5,6 @@ require 'ionoscloud_resource_list'
 Chef::Knife::IonoscloudResourceList.load_deps
 
 describe Chef::Knife::IonoscloudResourceList do
-
   before :each do
     subject { Chef::Knife::IonoscloudResourceList.new }
 
@@ -110,7 +109,7 @@ describe Chef::Knife::IonoscloudResourceList do
       subject_config = {
         ionoscloud_username: 'email',
         ionoscloud_password: 'password',
-        resource_id: 'resource_id'
+        resource_id: 'resource_id',
       }
 
       subject_config.each { |key, value| subject.config[key] = value }
@@ -137,18 +136,20 @@ describe Chef::Knife::IonoscloudResourceList do
     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|
+      if required_options.length > 0
+        arrays_without_one_element(required_options).each do |test_case|
+          subject.config[:ionoscloud_token] = 'token'
+          test_case[:array].each { |value| subject.config[value] = 'test' }
 
-        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).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
 
-        expect { subject.run }.to raise_error(SystemExit) do |error|
-          expect(error.status).to eq(1)
+          required_options.each { |value| subject.config[value] = nil }
         end
-
-        required_options.each { |value| subject.config[value] = nil }
       end
     end
   end

data/spec/chef/knife/ionoscloud_s3key_create_spec.rb

@@ -46,18 +46,20 @@ describe Chef::Knife::IonoscloudS3keyCreate do
     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|
+      if required_options.length > 0
+        arrays_without_one_element(required_options).each do |test_case|
+          subject.config[:ionoscloud_token] = 'token'
+          test_case[:array].each { |value| subject.config[value] = 'test' }
 
-        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).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
 
-        expect { subject.run }.to raise_error(SystemExit) do |error|
-          expect(error.status).to eq(1)
+          required_options.each { |value| subject.config[value] = nil }
         end
-
-        required_options.each { |value| subject.config[value] = nil }
       end
     end
   end

data/spec/chef/knife/ionoscloud_s3key_delete_spec.rb

@@ -85,18 +85,20 @@ describe Chef::Knife::IonoscloudS3keyDelete do
     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|
+      if required_options.length > 0
+        arrays_without_one_element(required_options).each do |test_case|
+          subject.config[:ionoscloud_token] = 'token'
+          test_case[:array].each { |value| subject.config[value] = 'test' }
 
-        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).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
 
-        expect { subject.run }.to raise_error(SystemExit) do |error|
-          expect(error.status).to eq(1)
+          required_options.each { |value| subject.config[value] = nil }
         end
-
-        required_options.each { |value| subject.config[value] = nil }
       end
     end
   end