knife-ionoscloud 5.1.2 → 6.0.0.alpha.1

data/spec/chef/knife/ionoscloud_networkloadbalancer_rule_remove_spec.rb

@@ -0,0 +1,124 @@
+require 'spec_helper'
+require 'ionoscloud_loadbalancer_nic_remove'
+
+Chef::Knife::IonoscloudLoadbalancerNicRemove.load_deps
+
+describe Chef::Knife::IonoscloudLoadbalancerNicRemove do
+  before :each do
+    subject { Chef::Knife::IonoscloudLoadbalancerNicRemove.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call LoadBalancersApi.datacenters_loadbalancers_delete when the ID is valid' do
+      load_balancer = load_balancer_mock
+      nic = nic_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: 'datacenter_id',
+        loadbalancer_id: load_balancer.id,
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+      subject.name_args = [nic.id]
+
+      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: #{load_balancer.properties.name}")
+      expect(subject).to receive(:puts).with("IP address: #{load_balancer.properties.ip}")
+      expect(subject).to receive(:puts).with("DHCP: #{load_balancer.properties.dhcp}")
+      expect(subject).to receive(:puts).with("Balanced Nics: #{nics.to_s}")
+      expect(subject.ui).to receive(:warn).with("Removed NIC #{nic.id} from the Load balancer #{load_balancer.id}. Request ID: .")
+
+      expect(subject).to receive(:get_request_id).once
+      mock_wait_for(subject)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'DELETE',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/loadbalancers/#{load_balancer.id}/balancednics/#{nic.id}",
+            operation: :'LoadBalancersApi.datacenters_loadbalancers_balancednics_delete',
+          },
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/loadbalancers/#{load_balancer.id}",
+            operation: :'LoadBalancersApi.datacenters_loadbalancers_find_by_id',
+            return_type: 'Loadbalancer',
+            result: load_balancer,
+          },
+        ],
+      )
+
+      expect { subject.run }.not_to raise_error(Exception)
+    end
+
+    it 'should not call LoadBalancersApi.datacenters_loadbalancers_delete when the ID is not valid' do
+      load_balancer = load_balancer_mock
+      nic_id = 'invalid_id'
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: 'datacenter_id',
+        loadbalancer_id: load_balancer.id,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+      subject.name_args = [nic_id]
+
+      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: #{load_balancer.properties.name}")
+      expect(subject).to receive(:puts).with("IP address: #{load_balancer.properties.ip}")
+      expect(subject).to receive(:puts).with("DHCP: #{load_balancer.properties.dhcp}")
+      expect(subject).to receive(:puts).with("Balanced Nics: #{nics.to_s}")
+      expect(subject.ui).to receive(:error).with("NIC ID #{nic_id} not found. Skipping.")
+
+      expect(subject.api_client).not_to receive(:wait_for)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'DELETE',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/loadbalancers/#{subject_config[:loadbalancer_id]}/balancednics/#{nic_id}",
+            operation: :'LoadBalancersApi.datacenters_loadbalancers_balancednics_delete',
+            exception: Ionoscloud::ApiError.new(code: 404),
+          },
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/loadbalancers/#{load_balancer.id}",
+            operation: :'LoadBalancersApi.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_networkloadbalancer_rule_target_add_spec.rb

@@ -0,0 +1,109 @@
+require 'spec_helper'
+require 'ionoscloud_networkloadbalancer_rule_target_add'
+
+Chef::Knife::IonoscloudNetworkloadbalancerRuleTargetAdd.load_deps
+
+describe Chef::Knife::IonoscloudNetworkloadbalancerRuleTargetAdd do
+  before :each do
+    subject { Chef::Knife::IonoscloudNetworkloadbalancerRuleTargetAdd.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call NetworkLoadBalancersApi.datacenters_networkloadbalancers_forwardingrules_patch_with_http_info' do
+      network_loadbalancer = network_loadbalancer_mock
+      network_loadbalancer_rule = network_loadbalancer_rule_mock
+      network_loadbalancer_rule_target = network_loadbalancer_rule_target_mock(
+        port: 123, ip: '1.1.1.1', check_interval: 2345, maintenance: true, check: false, weight: 13,
+      )
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: 'datacenter_id',
+        network_loadbalancer_id: network_loadbalancer.id,
+        forwarding_rule_id: network_loadbalancer_rule.id,
+        ip: network_loadbalancer_rule_target.ip,
+        port: network_loadbalancer_rule_target.port,
+        weight: network_loadbalancer_rule_target.weight,
+        check: network_loadbalancer_rule_target.health_check.check,
+        check_interval: network_loadbalancer_rule_target.health_check.check_interval,
+        maintenance: network_loadbalancer_rule_target.health_check.maintenance,
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject).to receive(:puts).with("ID: #{network_loadbalancer_rule.id}")
+      expect(subject).to receive(:puts).with("Name: #{network_loadbalancer_rule.properties.name}")
+      expect(subject).to receive(:puts).with("Algorithm: #{network_loadbalancer_rule.properties.algorithm}")
+      expect(subject).to receive(:puts).with("Protocol: #{network_loadbalancer_rule.properties.protocol}")
+      expect(subject).to receive(:puts).with("Listener IP: #{network_loadbalancer_rule.properties.listener_ip}")
+      expect(subject).to receive(:puts).with("Listener Port: #{network_loadbalancer_rule.properties.listener_port}")
+      expect(subject).to receive(:puts).with("Health Check: #{network_loadbalancer_rule.properties.health_check}")
+      expect(subject).to receive(:puts).with("Targets: #{
+        (network_loadbalancer_rule.properties.targets + [network_loadbalancer_rule_target]).map do |target|
+        {
+          ip: target.ip,
+          port: target.port,
+          weight: target.weight,
+          check: target.health_check.check,
+          check_interval: target.health_check.check_interval,
+          maintenance: target.health_check.maintenance,
+        }
+      end}")
+
+      mock_wait_for(subject)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/networkloadbalancers/#{subject_config[:network_loadbalancer_id]}"\
+            "/forwardingrules/#{subject_config[:forwarding_rule_id]}",
+            operation: :'NetworkLoadBalancersApi.datacenters_networkloadbalancers_forwardingrules_find_by_forwarding_rule_id',
+            return_type: 'NetworkLoadBalancerForwardingRule',
+            result: network_loadbalancer_rule,
+          },
+          {
+            method: 'PATCH',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/networkloadbalancers/#{subject_config[:network_loadbalancer_id]}"\
+            "/forwardingrules/#{subject_config[:forwarding_rule_id]}",
+            operation: :'NetworkLoadBalancersApi.datacenters_networkloadbalancers_forwardingrules_patch',
+            body: { targets: (network_loadbalancer_rule.properties.targets + [network_loadbalancer_rule_target]).map { |target| target.to_hash } },
+            return_type: 'NetworkLoadBalancerForwardingRule',
+          },
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/networkloadbalancers/#{subject_config[:network_loadbalancer_id]}"\
+            "/forwardingrules/#{subject_config[:forwarding_rule_id]}",
+            operation: :'NetworkLoadBalancersApi.datacenters_networkloadbalancers_forwardingrules_find_by_forwarding_rule_id',
+            return_type: 'NetworkLoadBalancerForwardingRule',
+            result: network_loadbalancer_rule,
+          },
+        ],
+      )
+
+      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_networkloadbalancer_rule_target_list_spec.rb

@@ -0,0 +1,85 @@
+require 'spec_helper'
+require 'ionoscloud_networkloadbalancer_rule_target_list'
+
+Chef::Knife::IonoscloudNetworkloadbalancerRuleTargetList.load_deps
+
+describe Chef::Knife::IonoscloudNetworkloadbalancerRuleTargetList do
+  before :each do
+    subject { Chef::Knife::IonoscloudNetworkloadbalancerRuleTargetList.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call NetworkLoadBalancersApi.datacenters_networkloadbalancers_forwardingrules_get' do
+      network_loadbalancer_rule = network_loadbalancer_rule_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: 'datacenter_id',
+        network_loadbalancer_id: 'network_loadbalancer_id',
+        forwarding_rule_id: network_loadbalancer_rule.id,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      network_loadbalancer_list = [
+        subject.ui.color('IP', :bold),
+        subject.ui.color('Port', :bold),
+        subject.ui.color('Weight', :bold),
+        subject.ui.color('Check', :bold),
+        subject.ui.color('Check interval', :bold),
+        subject.ui.color('Maintenance', :bold),
+        network_loadbalancer_rule.properties.targets.first.ip,
+        network_loadbalancer_rule.properties.targets.first.port,
+        network_loadbalancer_rule.properties.targets.first.weight,
+        network_loadbalancer_rule.properties.targets.first.health_check.check,
+        network_loadbalancer_rule.properties.targets.first.health_check.check_interval,
+        network_loadbalancer_rule.properties.targets.first.health_check.maintenance,
+        network_loadbalancer_rule.properties.targets[1].ip,
+        network_loadbalancer_rule.properties.targets[1].port,
+        network_loadbalancer_rule.properties.targets[1].weight,
+        network_loadbalancer_rule.properties.targets[1].health_check.check,
+        network_loadbalancer_rule.properties.targets[1].health_check.check_interval,
+        network_loadbalancer_rule.properties.targets[1].health_check.maintenance,
+      ]
+
+      expect(subject.ui).to receive(:list).with(network_loadbalancer_list, :uneven_columns_across, 6)
+
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/networkloadbalancers/#{subject_config[:network_loadbalancer_id]}"\
+            "/forwardingrules/#{subject_config[:forwarding_rule_id]}",
+            operation: :'NetworkLoadBalancersApi.datacenters_networkloadbalancers_forwardingrules_find_by_forwarding_rule_id',
+            return_type: 'NetworkLoadBalancerForwardingRule',
+            result: network_loadbalancer_rule,
+          },
+        ],
+      )
+
+      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_networkloadbalancer_rule_target_remove_spec.rb

@@ -0,0 +1,263 @@
+require 'spec_helper'
+require 'ionoscloud_networkloadbalancer_rule_target_remove'
+
+Chef::Knife::IonoscloudNetworkloadbalancerRuleTargetRemove.load_deps
+
+describe Chef::Knife::IonoscloudNetworkloadbalancerRuleTargetRemove do
+  before :each do
+    subject { Chef::Knife::IonoscloudNetworkloadbalancerRuleTargetRemove.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call NetworkLoadBalancersApi.datacenters_networkloadbalancers_forwardingrules_patch_with_http_info' do
+      network_loadbalancer = network_loadbalancer_mock
+      network_loadbalancer_rule = network_loadbalancer_rule_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: 'datacenter_id',
+        network_loadbalancer_id: network_loadbalancer.id,
+        forwarding_rule_id: network_loadbalancer_rule.id,
+        ip: network_loadbalancer_rule.properties.targets.first.ip,
+        port: network_loadbalancer_rule.properties.targets.first.port,
+        yes: true,
+      }
+
+      remaining_targets = network_loadbalancer_rule.properties.targets.reject do
+        |target|
+        target.ip == subject_config[:ip] && target.port == subject_config[:port]
+      end
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject).to receive(:puts).with("ID: #{network_loadbalancer_rule.id}")
+      expect(subject).to receive(:puts).with("Name: #{network_loadbalancer_rule.properties.name}")
+      expect(subject).to receive(:puts).with("Algorithm: #{network_loadbalancer_rule.properties.algorithm}")
+      expect(subject).to receive(:puts).with("Protocol: #{network_loadbalancer_rule.properties.protocol}")
+      expect(subject).to receive(:puts).with("Listener IP: #{network_loadbalancer_rule.properties.listener_ip}")
+      expect(subject).to receive(:puts).with("Listener Port: #{network_loadbalancer_rule.properties.listener_port}")
+      expect(subject).to receive(:puts).with("Health Check: #{network_loadbalancer_rule.properties.health_check}")
+      expect(subject).to receive(:puts).with("Targets: #{remaining_targets.map do |target|
+        {
+          ip: target.ip,
+          port: target.port,
+          weight: target.weight,
+          check: target.health_check.check,
+          check_interval: target.health_check.check_interval,
+          maintenance: target.health_check.maintenance,
+        }
+      end}")
+
+      mock_wait_for(subject)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/networkloadbalancers/#{subject_config[:network_loadbalancer_id]}"\
+            "/forwardingrules/#{subject_config[:forwarding_rule_id]}",
+            operation: :'NetworkLoadBalancersApi.datacenters_networkloadbalancers_forwardingrules_find_by_forwarding_rule_id',
+            return_type: 'NetworkLoadBalancerForwardingRule',
+            result: network_loadbalancer_rule,
+          },
+          {
+            method: 'PATCH',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/networkloadbalancers/#{subject_config[:network_loadbalancer_id]}"\
+            "/forwardingrules/#{subject_config[:forwarding_rule_id]}",
+            operation: :'NetworkLoadBalancersApi.datacenters_networkloadbalancers_forwardingrules_patch',
+            body: { targets: remaining_targets.map { |target| target.to_hash } },
+            return_type: 'NetworkLoadBalancerForwardingRule',
+          },
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/networkloadbalancers/#{subject_config[:network_loadbalancer_id]}"\
+            "/forwardingrules/#{subject_config[:forwarding_rule_id]}",
+            operation: :'NetworkLoadBalancersApi.datacenters_networkloadbalancers_forwardingrules_find_by_forwarding_rule_id',
+            return_type: 'NetworkLoadBalancerForwardingRule',
+            result: network_loadbalancer_rule,
+          },
+        ],
+      )
+
+      expect { subject.run }.not_to raise_error(Exception)
+    end
+
+    it 'should call NetworkLoadBalancersApi.datacenters_networkloadbalancers_forwardingrules_patch_with_http_info 2' do
+      network_loadbalancer = network_loadbalancer_mock
+      network_loadbalancer_rule = network_loadbalancer_rule_mock(
+        targets: [
+          network_loadbalancer_rule_target_mock(
+            port: 123, ip: '1.1.1.1', check_interval: 2345, maintenance: true, check: true, weight: 13,
+          ),
+          network_loadbalancer_rule_target_mock(
+            port: 1234, ip: '1.1.122.1', check_interval: 1231, maintenance: true, check: false, weight: 456,
+          ),
+          network_loadbalancer_rule_target_mock(
+            port: 112, ip: '1.133.1.1', check_interval: 2525, maintenance: true, check: true, weight: 425,
+          ),
+          network_loadbalancer_rule_target_mock(
+            port: 123, ip: '1.133.1.1', check_interval: 36346, maintenance: false, check: false, weight: 4,
+          ),
+          network_loadbalancer_rule_target_mock(
+            port: 164, ip: '1.1.1.1', check_interval: 243, maintenance: true, check: false, weight: 123,
+          ),
+        ]
+      )
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: 'datacenter_id',
+        network_loadbalancer_id: network_loadbalancer.id,
+        forwarding_rule_id: network_loadbalancer_rule.id,
+        ip: network_loadbalancer_rule.properties.targets.first.ip,
+        port: network_loadbalancer_rule.properties.targets.first.port,
+        yes: true,
+      }
+
+      remaining_targets = network_loadbalancer_rule.properties.targets.reject do
+        |target|
+        target.ip == subject_config[:ip] && target.port == subject_config[:port]
+      end
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject).to receive(:puts).with("ID: #{network_loadbalancer_rule.id}")
+      expect(subject).to receive(:puts).with("Name: #{network_loadbalancer_rule.properties.name}")
+      expect(subject).to receive(:puts).with("Algorithm: #{network_loadbalancer_rule.properties.algorithm}")
+      expect(subject).to receive(:puts).with("Protocol: #{network_loadbalancer_rule.properties.protocol}")
+      expect(subject).to receive(:puts).with("Listener IP: #{network_loadbalancer_rule.properties.listener_ip}")
+      expect(subject).to receive(:puts).with("Listener Port: #{network_loadbalancer_rule.properties.listener_port}")
+      expect(subject).to receive(:puts).with("Health Check: #{network_loadbalancer_rule.properties.health_check}")
+      expect(subject).to receive(:puts).with("Targets: #{remaining_targets.map do |target|
+        {
+          ip: target.ip,
+          port: target.port,
+          weight: target.weight,
+          check: target.health_check.check,
+          check_interval: target.health_check.check_interval,
+          maintenance: target.health_check.maintenance,
+        }
+      end}")
+
+      mock_wait_for(subject)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/networkloadbalancers/#{subject_config[:network_loadbalancer_id]}"\
+            "/forwardingrules/#{subject_config[:forwarding_rule_id]}",
+            operation: :'NetworkLoadBalancersApi.datacenters_networkloadbalancers_forwardingrules_find_by_forwarding_rule_id',
+            return_type: 'NetworkLoadBalancerForwardingRule',
+            result: network_loadbalancer_rule,
+          },
+          {
+            method: 'PATCH',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/networkloadbalancers/#{subject_config[:network_loadbalancer_id]}"\
+            "/forwardingrules/#{subject_config[:forwarding_rule_id]}",
+            operation: :'NetworkLoadBalancersApi.datacenters_networkloadbalancers_forwardingrules_patch',
+            body: { targets: remaining_targets.map { |target| target.to_hash } },
+            return_type: 'NetworkLoadBalancerForwardingRule',
+          },
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/networkloadbalancers/#{subject_config[:network_loadbalancer_id]}"\
+            "/forwardingrules/#{subject_config[:forwarding_rule_id]}",
+            operation: :'NetworkLoadBalancersApi.datacenters_networkloadbalancers_forwardingrules_find_by_forwarding_rule_id',
+            return_type: 'NetworkLoadBalancerForwardingRule',
+            result: network_loadbalancer_rule,
+          },
+        ],
+      )
+
+      expect { subject.run }.not_to raise_error(Exception)
+    end
+
+    it 'should not call NetworkLoadBalancersApi.datacenters_networkloadbalancers_forwardingrules_patch_with_http_info when no target exists' do
+      network_loadbalancer = network_loadbalancer_mock
+      network_loadbalancer_rule = network_loadbalancer_rule_mock(
+        targets: [
+          network_loadbalancer_rule_target_mock(
+            port: 123, ip: '1.1.1.1', check_interval: 2345, maintenance: true, check: true, weight: 13,
+          ),
+        ]
+      )
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: 'datacenter_id',
+        network_loadbalancer_id: network_loadbalancer.id,
+        forwarding_rule_id: network_loadbalancer_rule.id,
+        ip: '1.1.1.2',
+        port: 123,
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject).to receive(:puts).with("ID: #{network_loadbalancer_rule.id}")
+      expect(subject).to receive(:puts).with("Name: #{network_loadbalancer_rule.properties.name}")
+      expect(subject).to receive(:puts).with("Algorithm: #{network_loadbalancer_rule.properties.algorithm}")
+      expect(subject).to receive(:puts).with("Protocol: #{network_loadbalancer_rule.properties.protocol}")
+      expect(subject).to receive(:puts).with("Listener IP: #{network_loadbalancer_rule.properties.listener_ip}")
+      expect(subject).to receive(:puts).with("Listener Port: #{network_loadbalancer_rule.properties.listener_port}")
+      expect(subject).to receive(:puts).with("Health Check: #{network_loadbalancer_rule.properties.health_check}")
+      expect(subject).to receive(:puts).with("Targets: #{network_loadbalancer_rule.properties.targets.map do |target|
+        {
+          ip: target.ip,
+          port: target.port,
+          weight: target.weight,
+          check: target.health_check.check,
+          check_interval: target.health_check.check_interval,
+          maintenance: target.health_check.maintenance,
+        }
+      end}")
+
+      expect(subject.api_client).not_to receive(:wait_for)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/networkloadbalancers/#{subject_config[:network_loadbalancer_id]}"\
+            "/forwardingrules/#{subject_config[:forwarding_rule_id]}",
+            operation: :'NetworkLoadBalancersApi.datacenters_networkloadbalancers_forwardingrules_find_by_forwarding_rule_id',
+            return_type: 'NetworkLoadBalancerForwardingRule',
+            result: network_loadbalancer_rule,
+          },
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/networkloadbalancers/#{subject_config[:network_loadbalancer_id]}"\
+            "/forwardingrules/#{subject_config[:forwarding_rule_id]}",
+            operation: :'NetworkLoadBalancersApi.datacenters_networkloadbalancers_forwardingrules_find_by_forwarding_rule_id',
+            return_type: 'NetworkLoadBalancerForwardingRule',
+            result: network_loadbalancer_rule,
+          },
+        ],
+      )
+
+      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_create_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudNicCreate do
   end
 
   describe '#run' do
-    it 'should call NicApi.datacenters_servers_nics_post with the expected arguments and output based on what it receives' do
+    it 'should call NetworkInterfacesApi.datacenters_servers_nics_post with the expected arguments and output based on what it receives' do
       nic = nic_mock
       subject_config = {
         ionoscloud_username: 'email',
@@ -23,7 +23,7 @@ describe Chef::Knife::IonoscloudNicCreate do
         name: nic.properties.name,
         dhcp: nic.properties.dhcp,
         ips: nic.properties.ips.join(','),
-        nat: nic.properties.nat,
+        firewall_type: 'INGRESS',
       }
 
       subject_config.each { |key, value| subject.config[key] = value }
@@ -31,13 +31,17 @@ describe Chef::Knife::IonoscloudNicCreate do
       expected_body = nic.properties.to_hash
       expected_body.delete(:firewallActive)
       expected_body.delete(:mac)
+      expected_body.delete(:deviceNumber)
+      expected_body.delete(:pciSlot)
 
       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).to receive(:puts).with("Firewall Type: #{nic.properties.firewall_type}")
+      expect(subject).to receive(:puts).with("Device Number: #{nic.properties.device_number}")
+      expect(subject).to receive(:puts).with("PCI Slot: #{nic.properties.pci_slot}")
 
       mock_wait_for(subject)
       mock_call_api(
@@ -46,7 +50,7 @@ describe Chef::Knife::IonoscloudNicCreate do
           {
             method: 'POST',
             path: "/datacenters/#{subject_config[:datacenter_id]}/servers/#{subject_config[:server_id]}/nics",
-            operation: :'NicApi.datacenters_servers_nics_post',
+            operation: :'NetworkInterfacesApi.datacenters_servers_nics_post',
             return_type: 'Nic',
             body: { properties: expected_body },
             result: nic,
@@ -54,7 +58,7 @@ describe Chef::Knife::IonoscloudNicCreate do
           {
             method: 'GET',
             path: "/datacenters/#{subject_config[:datacenter_id]}/servers/#{subject_config[:server_id]}/nics/#{nic.id}",
-            operation: :'NicApi.datacenters_servers_nics_find_by_id',
+            operation: :'NetworkInterfacesApi.datacenters_servers_nics_find_by_id',
             return_type: 'Nic',
             result: nic,
           },