knife-ionoscloud 6.0.0.alpha.1 → 6.1.0

data/spec/chef/knife/ionoscloud_networkloadbalancer_rule_target_remove_spec.rb

@@ -14,7 +14,7 @@ describe Chef::Knife::IonoscloudNetworkloadbalancerRuleTargetRemove do
   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 = network_loadbalancer.entities.forwardingrules.items[0]
       subject_config = {
         ionoscloud_username: 'email',
         ionoscloud_password: 'password',
@@ -33,23 +33,29 @@ describe Chef::Knife::IonoscloudNetworkloadbalancerRuleTargetRemove do
 
       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|
+      fw_rules = network_loadbalancer.entities.forwardingrules.items.map do |rule|
         {
-          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,
+          id: rule.id,
+          name: rule.properties.name,
+          algorithm: rule.properties.algorithm,
+          protocol: rule.properties.protocol,
+          listener_ip: rule.properties.listener_ip,
+          listener_port: rule.properties.listener_port,
+          health_check: rule.properties.health_check.nil? ? nil : rule.properties.health_check.to_hash,
+          targets: rule.properties.targets.map { |target| target.to_hash },
         }
-      end}")
+      end
+
+      fw_rules[0][:targets] = remaining_targets.map { |target| target.to_hash }
+
+      expect(subject).to receive(:puts).with("ID: #{network_loadbalancer.id}")
+      expect(subject).to receive(:puts).with("Name: #{network_loadbalancer.properties.name}")
+      expect(subject).to receive(:puts).with("Listener LAN: #{network_loadbalancer.properties.listener_lan}")
+      expect(subject).to receive(:puts).with("IPS: #{network_loadbalancer.properties.ips}")
+      expect(subject).to receive(:puts).with("Target LAN: #{network_loadbalancer.properties.target_lan}")
+      expect(subject).to receive(:puts).with("Private IPS: #{network_loadbalancer.properties.lb_private_ips}")
+      expect(subject).to receive(:puts).with("Forwarding Rules: #{fw_rules}")
+      expect(subject).to receive(:puts).with("Flowlogs: #{network_loadbalancer.entities.flowlogs.items.map { |flowlog| flowlog.id }}")
 
       mock_wait_for(subject)
       mock_call_api(
@@ -73,11 +79,10 @@ describe Chef::Knife::IonoscloudNetworkloadbalancerRuleTargetRemove do
           },
           {
             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,
+            path: "/datacenters/#{subject_config[:datacenter_id]}/networkloadbalancers/#{subject_config[:network_loadbalancer_id]}",
+            operation: :'NetworkLoadBalancersApi.datacenters_networkloadbalancers_find_by_network_load_balancer_id',
+            return_type: 'NetworkLoadBalancer',
+            result: network_loadbalancer,
           },
         ],
       )
@@ -86,7 +91,6 @@ describe Chef::Knife::IonoscloudNetworkloadbalancerRuleTargetRemove do
     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(
@@ -106,6 +110,14 @@ describe Chef::Knife::IonoscloudNetworkloadbalancerRuleTargetRemove do
           ),
         ]
       )
+      network_loadbalancer = network_loadbalancer_mock(
+        rules: Ionoscloud::NetworkLoadBalancers.new(
+          id: 'network_loadbalancers',
+          type: 'collection',
+          items: [network_loadbalancer_rule, network_loadbalancer_rule_mock],
+        )
+      )
+
       subject_config = {
         ionoscloud_username: 'email',
         ionoscloud_password: 'password',
@@ -124,23 +136,29 @@ describe Chef::Knife::IonoscloudNetworkloadbalancerRuleTargetRemove do
 
       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|
+      fw_rules = network_loadbalancer.entities.forwardingrules.items.map do |rule|
         {
-          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,
+          id: rule.id,
+          name: rule.properties.name,
+          algorithm: rule.properties.algorithm,
+          protocol: rule.properties.protocol,
+          listener_ip: rule.properties.listener_ip,
+          listener_port: rule.properties.listener_port,
+          health_check: rule.properties.health_check.nil? ? nil : rule.properties.health_check.to_hash,
+          targets: rule.properties.targets.map { |target| target.to_hash },
         }
-      end}")
+      end
+
+      fw_rules[0][:targets] = remaining_targets.map { |target| target.to_hash }
+
+      expect(subject).to receive(:puts).with("ID: #{network_loadbalancer.id}")
+      expect(subject).to receive(:puts).with("Name: #{network_loadbalancer.properties.name}")
+      expect(subject).to receive(:puts).with("Listener LAN: #{network_loadbalancer.properties.listener_lan}")
+      expect(subject).to receive(:puts).with("IPS: #{network_loadbalancer.properties.ips}")
+      expect(subject).to receive(:puts).with("Target LAN: #{network_loadbalancer.properties.target_lan}")
+      expect(subject).to receive(:puts).with("Private IPS: #{network_loadbalancer.properties.lb_private_ips}")
+      expect(subject).to receive(:puts).with("Forwarding Rules: #{fw_rules}")
+      expect(subject).to receive(:puts).with("Flowlogs: #{network_loadbalancer.entities.flowlogs.items.map { |flowlog| flowlog.id }}")
 
       mock_wait_for(subject)
       mock_call_api(
@@ -164,11 +182,10 @@ describe Chef::Knife::IonoscloudNetworkloadbalancerRuleTargetRemove do
           },
           {
             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,
+            path: "/datacenters/#{subject_config[:datacenter_id]}/networkloadbalancers/#{subject_config[:network_loadbalancer_id]}",
+            operation: :'NetworkLoadBalancersApi.datacenters_networkloadbalancers_find_by_network_load_balancer_id',
+            return_type: 'NetworkLoadBalancer',
+            result: network_loadbalancer,
           },
         ],
       )
@@ -198,23 +215,25 @@ describe Chef::Knife::IonoscloudNetworkloadbalancerRuleTargetRemove do
 
       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|
+      expect(subject).to receive(:puts).with("ID: #{network_loadbalancer.id}")
+      expect(subject).to receive(:puts).with("Name: #{network_loadbalancer.properties.name}")
+      expect(subject).to receive(:puts).with("Listener LAN: #{network_loadbalancer.properties.listener_lan}")
+      expect(subject).to receive(:puts).with("IPS: #{network_loadbalancer.properties.ips}")
+      expect(subject).to receive(:puts).with("Target LAN: #{network_loadbalancer.properties.target_lan}")
+      expect(subject).to receive(:puts).with("Private IPS: #{network_loadbalancer.properties.lb_private_ips}")
+      expect(subject).to receive(:puts).with("Forwarding Rules: #{network_loadbalancer.entities.forwardingrules.items.map do |rule|
         {
-          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,
+          id: rule.id,
+          name: rule.properties.name,
+          algorithm: rule.properties.algorithm,
+          protocol: rule.properties.protocol,
+          listener_ip: rule.properties.listener_ip,
+          listener_port: rule.properties.listener_port,
+          health_check: rule.properties.health_check.nil? ? nil : rule.properties.health_check.to_hash,
+          targets: (rule.properties.targets.nil? ? [] : rule.properties.targets.map { |target| target.to_hash }),
         }
       end}")
+      expect(subject).to receive(:puts).with("Flowlogs: #{network_loadbalancer.entities.flowlogs.items.map { |flowlog| flowlog.id }}")
 
       expect(subject.api_client).not_to receive(:wait_for)
       mock_call_api(
@@ -230,11 +249,10 @@ describe Chef::Knife::IonoscloudNetworkloadbalancerRuleTargetRemove do
           },
           {
             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,
+            path: "/datacenters/#{subject_config[:datacenter_id]}/networkloadbalancers/#{subject_config[:network_loadbalancer_id]}",
+            operation: :'NetworkLoadBalancersApi.datacenters_networkloadbalancers_find_by_network_load_balancer_id',
+            return_type: 'NetworkLoadBalancer',
+            result: network_loadbalancer,
           },
         ],
       )

data/spec/chef/knife/ionoscloud_networkloadbalancer_rule_update_spec.rb

@@ -0,0 +1,163 @@
+require 'spec_helper'
+require 'ionoscloud_networkloadbalancer_rule_update'
+
+Chef::Knife::IonoscloudNetworkloadbalancerRuleUpdate.load_deps
+
+describe Chef::Knife::IonoscloudNetworkloadbalancerRuleUpdate do
+  before :each do
+    subject { Chef::Knife::IonoscloudNetworkloadbalancerRuleUpdate.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call NetworkLoadBalancersApi.datacenters_networkloadbalancers_forwardingrules_patch' do
+      network_loadbalancer = network_loadbalancer_mock
+      network_loadbalancer_rule = network_loadbalancer.entities.forwardingrules.items[0]
+
+      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,
+        name: network_loadbalancer_rule.properties.name + '_edited',
+        algorithm: 'LEAST_CONNECTION',
+        protocol: 'HTTP',
+        listener_ip: '1.1.1.1',
+        listener_port: network_loadbalancer_rule.properties.listener_port + 10,
+        client_timeout: network_loadbalancer_rule.properties.health_check.client_timeout + 100,
+        connect_timeout: network_loadbalancer_rule.properties.health_check.connect_timeout + 100,
+        target_timeout: network_loadbalancer_rule.properties.health_check.target_timeout + 100,
+        retries: network_loadbalancer_rule.properties.health_check.retries + 1,
+        targets: [
+          {
+            'ip' => '1.1.1.1',
+            'port' => 11,
+            'weight' => 15,
+            'health_check' => {
+              'check' => true,
+              'check_interval' => 1200,
+              'maintenance' => false,
+            },
+          },
+          {
+            'ip' => '1.1.1.1',
+            'port' => 14,
+            'weight' => 18,
+            'health_check' => {
+              'check' => false,
+              'check_interval' => 1500,
+              'maintenance' => true,
+            },
+          },
+        ],
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      parsed_targets = subject_config[:targets].map do
+        |target|
+        Ionoscloud::NetworkLoadBalancerForwardingRuleTarget.new(
+          ip: target['ip'],
+          port: target['port'],
+          weight: target['weight'],
+          health_check: Ionoscloud::NetworkLoadBalancerForwardingRuleTargetHealthCheck.new(
+            check: target['health_check']['check'],
+            check_interval: target['health_check']['check_interval'],
+            maintenance: target['health_check']['maintenance'],
+          ),
+        )
+      end
+
+      network_loadbalancer.entities.forwardingrules.items[0].properties.name = subject_config[:name]
+      network_loadbalancer.entities.forwardingrules.items[0].properties.algorithm = subject_config[:algorithm]
+      network_loadbalancer.entities.forwardingrules.items[0].properties.protocol = subject_config[:protocol]
+      network_loadbalancer.entities.forwardingrules.items[0].properties.listener_ip = subject_config[:listener_ip]
+      network_loadbalancer.entities.forwardingrules.items[0].properties.listener_port = subject_config[:algorithm]
+      network_loadbalancer.entities.forwardingrules.items[0].properties.health_check.client_timeout = subject_config[:client_timeout]
+      network_loadbalancer.entities.forwardingrules.items[0].properties.health_check.connect_timeout = subject_config[:connect_timeout]
+      network_loadbalancer.entities.forwardingrules.items[0].properties.health_check.target_timeout = subject_config[:target_timeout]
+      network_loadbalancer.entities.forwardingrules.items[0].properties.health_check.retries = subject_config[:retries]
+      network_loadbalancer.entities.forwardingrules.items[0].properties.targets = parsed_targets
+
+      expect(subject).to receive(:puts).with("ID: #{network_loadbalancer.id}")
+      expect(subject).to receive(:puts).with("Name: #{network_loadbalancer.properties.name}")
+      expect(subject).to receive(:puts).with("Listener LAN: #{network_loadbalancer.properties.listener_lan}")
+      expect(subject).to receive(:puts).with("IPS: #{network_loadbalancer.properties.ips}")
+      expect(subject).to receive(:puts).with("Target LAN: #{network_loadbalancer.properties.target_lan}")
+      expect(subject).to receive(:puts).with("Private IPS: #{network_loadbalancer.properties.lb_private_ips}")
+      expect(subject).to receive(:puts).with("Forwarding Rules: #{network_loadbalancer.entities.forwardingrules.items.map do |rule|
+        {
+          id: rule.id,
+          name: rule.properties.name,
+          algorithm: rule.properties.algorithm,
+          protocol: rule.properties.protocol,
+          listener_ip: rule.properties.listener_ip,
+          listener_port: rule.properties.listener_port,
+          health_check: rule.properties.health_check.nil? ? nil : rule.properties.health_check.to_hash,
+          targets: (rule.properties.targets.nil? ? [] : rule.properties.targets.map { |target| target.to_hash }),
+        }
+      end}")
+      expect(subject).to receive(:puts).with("Flowlogs: #{network_loadbalancer.entities.flowlogs.items.map { |flowlog| flowlog.id }}")
+
+      mock_wait_for(subject)
+      mock_call_api(
+        subject,
+        [
+          {
+            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',
+            return_type: 'NetworkLoadBalancerForwardingRule',
+            body: {
+              name: subject_config[:name],
+              algorithm: subject_config[:algorithm],
+              protocol: subject_config[:protocol],
+              listenerIp: subject_config[:listener_ip],
+              listenerPort: subject_config[:listener_port],
+              healthCheck: {
+                clientTimeout: subject_config[:client_timeout],
+                connectTimeout: subject_config[:connect_timeout],
+                targetTimeout: subject_config[:target_timeout],
+                retries: subject_config[:retries],
+              },
+              targets: parsed_targets.map(&:to_hash),
+            },
+            result: network_loadbalancer,
+          },
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/networkloadbalancers/#{subject_config[:network_loadbalancer_id]}",
+            operation: :'NetworkLoadBalancersApi.datacenters_networkloadbalancers_find_by_network_load_balancer_id',
+            return_type: 'NetworkLoadBalancer',
+            result: network_loadbalancer,
+          },
+        ],
+      )
+
+      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_update_spec.rb

@@ -0,0 +1,107 @@
+require 'spec_helper'
+require 'ionoscloud_networkloadbalancer_update'
+
+Chef::Knife::IonoscloudNetworkloadbalancerUpdate.load_deps
+
+describe Chef::Knife::IonoscloudNetworkloadbalancerUpdate do
+  before :each do
+    subject { Chef::Knife::IonoscloudNetworkloadbalancerUpdate.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call NetworkLoadBalancersApi.datacenters_networkloadbalancers_patch' do
+      network_loadbalancer = network_loadbalancer_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: 'datacenter_id',
+        network_loadbalancer_id: network_loadbalancer.id,
+        name: network_loadbalancer.properties.name + '_edited',
+        listener_lan: Integer(network_loadbalancer.properties.listener_lan) + 1,
+        target_lan: Integer(network_loadbalancer.properties.target_lan) + 1,
+        ips: (network_loadbalancer.properties.ips + ['3.3.3.3']).join(','),
+        lb_private_ips: (network_loadbalancer.properties.lb_private_ips + ['22.231.2.5/24']).join(','),
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject).to receive(:puts).with("ID: #{network_loadbalancer.id}")
+      expect(subject).to receive(:puts).with("Name: #{subject_config[:name]}")
+      expect(subject).to receive(:puts).with("Listener LAN: #{subject_config[:listener_lan]}")
+      expect(subject).to receive(:puts).with("IPS: #{subject_config[:ips].split(',')}")
+      expect(subject).to receive(:puts).with("Target LAN: #{subject_config[:target_lan]}")
+      expect(subject).to receive(:puts).with("Private IPS: #{subject_config[:lb_private_ips].split(',')}")
+      expect(subject).to receive(:puts).with("Forwarding Rules: #{network_loadbalancer.entities.forwardingrules.items.map do |rule|
+        {
+          id: rule.id,
+          name: rule.properties.name,
+          algorithm: rule.properties.algorithm,
+          protocol: rule.properties.protocol,
+          listener_ip: rule.properties.listener_ip,
+          listener_port: rule.properties.listener_port,
+          health_check: rule.properties.health_check.nil? ? nil : rule.properties.health_check.to_hash,
+          targets: (rule.properties.targets.nil? ? [] : rule.properties.targets.map { |target| target.to_hash }),
+        }
+      end}")
+      expect(subject).to receive(:puts).with("Flowlogs: #{network_loadbalancer.entities.flowlogs.items.map { |flowlog| flowlog.id }}")
+
+      network_loadbalancer.properties.name = subject_config[:name]
+      network_loadbalancer.properties.ips = subject_config[:ips].split(',')
+      network_loadbalancer.properties.lb_private_ips = subject_config[:lb_private_ips].split(',')
+      network_loadbalancer.properties.listener_lan = subject_config[:listener_lan]
+      network_loadbalancer.properties.target_lan = subject_config[:target_lan]
+
+      mock_wait_for(subject)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'PATCH',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/networkloadbalancers/#{subject_config[:network_loadbalancer_id]}",
+            operation: :'NetworkLoadBalancersApi.datacenters_networkloadbalancers_patch',
+            return_type: 'NetworkLoadBalancer',
+            body: {
+              name: subject_config[:name],
+              ips: subject_config[:ips].split(','),
+              lbPrivateIps: subject_config[:lb_private_ips].split(','),
+              listenerLan: subject_config[:listener_lan],
+              targetLan: subject_config[:target_lan],
+            },
+            result: network_loadbalancer,
+          },
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/networkloadbalancers/#{subject_config[:network_loadbalancer_id]}",
+            operation: :'NetworkLoadBalancersApi.datacenters_networkloadbalancers_find_by_network_load_balancer_id',
+            return_type: 'NetworkLoadBalancer',
+            result: network_loadbalancer,
+          },
+        ],
+      )
+
+      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,72 @@
+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 NetworkInterfacesApi.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("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}")
+
+      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: :'NetworkInterfacesApi.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,97 @@
+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 NetworkInterfacesApi.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(','),
+        firewall_type: 'EGRESS',
+        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(',')}")
+      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("Firewall Type: #{subject_config[: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}")
+
+      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]
+      nic.properties.firewall_type = subject_config[:firewall_type]
+
+      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: :'NetworkInterfacesApi.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],
+              firewallType: subject_config[:firewall_type],
+            },
+            result: nic,
+          },
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/servers/#{subject_config[:server_id]}/nics/#{subject_config[:nic_id]}",
+            operation: :'NetworkInterfacesApi.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