knife-ionoscloud 6.0.0.alpha.1 → 6.1.0

data/spec/chef/knife/ionoscloud_firewall_update_spec.rb

@@ -0,0 +1,117 @@
+require 'spec_helper'
+require 'ionoscloud_firewall_update'
+
+Chef::Knife::IonoscloudFirewallUpdate.load_deps
+
+describe Chef::Knife::IonoscloudFirewallUpdate do
+  before :each do
+    subject { Chef::Knife::IonoscloudFirewallUpdate.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call FirewallRulesApi.datacenters_servers_nics_firewallrules_patch' do
+      firewall = firewall_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: 'datacenter_id',
+        server_id: 'server_id',
+        nic_id: 'nic_id',
+        firewall_id: firewall.id,
+        name: firewall.properties.name + '_edited',
+        protocol: 'TCP',
+        source_mac: '01:11:11:11:22:00',
+        source_ip: '1.2.3.4',
+        target_ip: '2.2.3.1',
+        port_range_start: 100,
+        port_range_end: 145,
+        icmp_type: 15,
+        icmp_code: 14,
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject).to receive(:puts).with("ID: #{firewall.id}")
+      expect(subject).to receive(:puts).with("Name: #{subject_config[:name]}")
+      expect(subject).to receive(:puts).with("Type: #{firewall.properties.type}")
+      expect(subject).to receive(:puts).with("Protocol: #{subject_config[:protocol]}")
+      expect(subject).to receive(:puts).with("Source MAC: #{subject_config[:source_mac]}")
+      expect(subject).to receive(:puts).with("Source IP: #{subject_config[:source_ip]}")
+      expect(subject).to receive(:puts).with("Target IP: #{subject_config[:target_ip]}")
+      expect(subject).to receive(:puts).with("Port Range Start: #{subject_config[:port_range_start]}")
+      expect(subject).to receive(:puts).with("Port Range End: #{subject_config[:port_range_end]}")
+      expect(subject).to receive(:puts).with("ICMP Type: #{subject_config[:icmp_type]}")
+      expect(subject).to receive(:puts).with("ICMP Code: #{subject_config[:icmp_code]}")
+
+      firewall.properties.name = subject_config[:name]
+      firewall.properties.protocol = subject_config[:protocol]
+      firewall.properties.source_mac = subject_config[:source_mac]
+      firewall.properties.source_ip = subject_config[:source_ip]
+      firewall.properties.target_ip = subject_config[:target_ip]
+      firewall.properties.port_range_start = subject_config[:port_range_start]
+      firewall.properties.port_range_end = subject_config[:port_range_end]
+      firewall.properties.icmp_type = subject_config[:icmp_type]
+      firewall.properties.icmp_code = subject_config[:icmp_code]
+
+      expected_body = {
+        name: subject_config[:name],
+        protocol: subject_config[:protocol],
+        sourceMac: subject_config[:source_mac],
+        sourceIp: subject_config[:source_ip],
+        targetIp: subject_config[:target_ip],
+        portRangeStart: subject_config[:port_range_start],
+        portRangeEnd: subject_config[:port_range_end],
+        icmpType: subject_config[:icmp_type],
+        icmpCode: subject_config[:icmp_code],
+      }
+
+      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]}/firewallrules/#{firewall.id}",
+            operation: :'FirewallRulesApi.datacenters_servers_nics_firewallrules_patch',
+            return_type: 'FirewallRule',
+            body: expected_body,
+            result: firewall,
+          },
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/servers/#{subject_config[:server_id]}/"\
+                  "nics/#{subject_config[:nic_id]}/firewallrules/#{firewall.id}",
+            operation: :'FirewallRulesApi.datacenters_servers_nics_firewallrules_find_by_id',
+            return_type: 'FirewallRule',
+            result: firewall,
+          },
+        ],
+      )
+
+      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_flowlog_create_spec.rb

@@ -148,7 +148,7 @@ describe Chef::Knife::IonoscloudFlowlogCreate do
         ionoscloud_password: 'password',
         datacenter_id: 'datacenter_id',
         network_loadbalancer_id: 'network_loadbalancer_id',
-        type: 'loadbalancer',
+        type: 'networkloadbalancer',
         name: flowlog.properties.name,
         action: flowlog.properties.action,
         direction: flowlog.properties.direction,
@@ -187,7 +187,7 @@ describe Chef::Knife::IonoscloudFlowlogCreate do
         ionoscloud_username: 'email',
         ionoscloud_password: 'password',
         datacenter_id: 'datacenter_id',
-        type: 'loadbalancer',
+        type: 'networkloadbalancer',
         name: flowlog.properties.name,
         action: flowlog.properties.action,
         direction: flowlog.properties.direction,
@@ -224,7 +224,7 @@ describe Chef::Knife::IonoscloudFlowlogCreate do
 
       subject_config.each { |key, value| subject.config[key] = value }
 
-      expect(subject.ui).to receive(:error).with("Flow log cannot belong to #{subject_config[:type]}. Value must be one of ['nic', 'natgateway', 'loadbalancer']")
+      expect(subject.ui).to receive(:error).with("Flow Log cannot belong to #{subject_config[:type]}. Value must be one of ['nic', 'natgateway', 'networkloadbalancer']")
 
       expect(subject.api_client).not_to receive(:call_api)
 

data/spec/chef/knife/ionoscloud_flowlog_delete_spec.rb

@@ -32,7 +32,7 @@ describe Chef::Knife::IonoscloudFlowlogDelete do
       expect(subject).to receive(:puts).with("Action: #{flowlog.properties.action}")
       expect(subject).to receive(:puts).with("Direction: #{flowlog.properties.direction}")
       expect(subject).to receive(:puts).with("Bucket: #{flowlog.properties.bucket}")
-      expect(subject.ui).to receive(:warn).with("Deleted flow log #{flowlog.id}. Request ID: ")
+      expect(subject.ui).to receive(:warn).with("Deleted Flow Log #{flowlog.id}. Request ID: ")
 
       expect(subject.api_client).not_to receive(:wait_for)
       expect(subject).to receive(:get_request_id).once
@@ -73,7 +73,7 @@ describe Chef::Knife::IonoscloudFlowlogDelete do
       subject_config.each { |key, value| subject.config[key] = value }
       subject.name_args = [flowlog_id]
 
-      expect(subject.ui).to receive(:error).with("Flow log ID #{flowlog_id} not found. Skipping.")
+      expect(subject.ui).to receive(:error).with("Flow Log ID #{flowlog_id} not found. Skipping.")
 
       expect(subject.api_client).not_to receive(:wait_for)
       mock_call_api(
@@ -133,7 +133,7 @@ describe Chef::Knife::IonoscloudFlowlogDelete do
       expect(subject).to receive(:puts).with("Action: #{flowlog.properties.action}")
       expect(subject).to receive(:puts).with("Direction: #{flowlog.properties.direction}")
       expect(subject).to receive(:puts).with("Bucket: #{flowlog.properties.bucket}")
-      expect(subject.ui).to receive(:warn).with("Deleted flow log #{flowlog.id}. Request ID: ")
+      expect(subject.ui).to receive(:warn).with("Deleted Flow Log #{flowlog.id}. Request ID: ")
 
       expect(subject.api_client).not_to receive(:wait_for)
       expect(subject).to receive(:get_request_id).once
@@ -171,7 +171,7 @@ describe Chef::Knife::IonoscloudFlowlogDelete do
       subject_config.each { |key, value| subject.config[key] = value }
       subject.name_args = [flowlog_id]
 
-      expect(subject.ui).to receive(:error).with("Flow log ID #{flowlog_id} not found. Skipping.")
+      expect(subject.ui).to receive(:error).with("Flow Log ID #{flowlog_id} not found. Skipping.")
 
       expect(subject.api_client).not_to receive(:wait_for)
       mock_call_api(
@@ -218,7 +218,7 @@ describe Chef::Knife::IonoscloudFlowlogDelete do
         ionoscloud_password: 'password',
         datacenter_id: 'datacenter_id',
         network_loadbalancer_id: 'network_loadbalancer_id',
-        type: 'loadbalancer',
+        type: 'networkloadbalancer',
         yes: true,
       }
 
@@ -230,7 +230,7 @@ describe Chef::Knife::IonoscloudFlowlogDelete do
       expect(subject).to receive(:puts).with("Action: #{flowlog.properties.action}")
       expect(subject).to receive(:puts).with("Direction: #{flowlog.properties.direction}")
       expect(subject).to receive(:puts).with("Bucket: #{flowlog.properties.bucket}")
-      expect(subject.ui).to receive(:warn).with("Deleted flow log #{flowlog.id}. Request ID: ")
+      expect(subject.ui).to receive(:warn).with("Deleted Flow Log #{flowlog.id}. Request ID: ")
 
       expect(subject.api_client).not_to receive(:wait_for)
       expect(subject).to receive(:get_request_id).once
@@ -264,13 +264,13 @@ describe Chef::Knife::IonoscloudFlowlogDelete do
         ionoscloud_password: 'password',
         datacenter_id: 'datacenter_id',
         network_loadbalancer_id: 'network_loadbalancer_id',
-        type: 'loadbalancer',
+        type: 'networkloadbalancer',
       }
 
       subject_config.each { |key, value| subject.config[key] = value }
       subject.name_args = [flowlog_id]
 
-      expect(subject.ui).to receive(:error).with("Flow log ID #{flowlog_id} not found. Skipping.")
+      expect(subject.ui).to receive(:error).with("Flow Log ID #{flowlog_id} not found. Skipping.")
 
       expect(subject.api_client).not_to receive(:wait_for)
       mock_call_api(
@@ -296,7 +296,7 @@ describe Chef::Knife::IonoscloudFlowlogDelete do
         ionoscloud_username: 'email',
         ionoscloud_password: 'password',
         datacenter_id: 'datacenter_id',
-        type: 'loadbalancer',
+        type: 'networkloadbalancer',
       }
 
       subject_config.each { |key, value| subject.config[key] = value }
@@ -330,7 +330,7 @@ describe Chef::Knife::IonoscloudFlowlogDelete do
 
       subject_config.each { |key, value| subject.config[key] = value }
 
-      expect(subject.ui).to receive(:error).with("Flow log cannot belong to #{subject_config[:type]}. Value must be one of ['nic', 'natgateway', 'loadbalancer']")
+      expect(subject.ui).to receive(:error).with("Flow Log cannot belong to #{subject_config[:type]}. Value must be one of ['nic', 'natgateway', 'networkloadbalancer']")
 
       expect(subject.api_client).not_to receive(:call_api)
 

data/spec/chef/knife/ionoscloud_flowlog_get_spec.rb

@@ -0,0 +1,171 @@
+require 'spec_helper'
+require 'ionoscloud_flowlog_get'
+
+Chef::Knife::IonoscloudFlowlogGet.load_deps
+
+describe Chef::Knife::IonoscloudFlowlogGet do
+  before :each do
+    subject { Chef::Knife::IonoscloudFlowlogGet.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call FlowLogsApi.datacenters_servers_nics_flowlogs_find_by_id' do
+      flowlog = flowlog_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        type: 'nic',
+        datacenter_id: 'datacenter_id',
+        server_id: 'server_id',
+        nic_id: 'nic_id',
+        flowlog_id: flowlog.id,
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject).to receive(:puts).with("ID: #{flowlog.id}")
+      expect(subject).to receive(:puts).with("Name: #{flowlog.properties.name}")
+      expect(subject).to receive(:puts).with("Action: #{flowlog.properties.action}")
+      expect(subject).to receive(:puts).with("Direction: #{flowlog.properties.direction}")
+      expect(subject).to receive(:puts).with("Bucket: #{flowlog.properties.bucket}")
+
+      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]}/flowlogs/#{flowlog.id}",
+            operation: :'FlowLogsApi.datacenters_servers_nics_flowlogs_find_by_id',
+            return_type: 'FlowLog',
+            result: flowlog,
+          },
+        ],
+      )
+
+      expect { subject.run }.not_to raise_error(Exception)
+    end
+
+    it 'should call NATGatewaysApi.datacenters_natgateways_flowlogs_find_by_flow_log_id' do
+      flowlog = flowlog_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        type: 'natgateway',
+        datacenter_id: 'datacenter_id',
+        natgateway_id: 'natgateway_id',
+        flowlog_id: flowlog.id,
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject).to receive(:puts).with("ID: #{flowlog.id}")
+      expect(subject).to receive(:puts).with("Name: #{flowlog.properties.name}")
+      expect(subject).to receive(:puts).with("Action: #{flowlog.properties.action}")
+      expect(subject).to receive(:puts).with("Direction: #{flowlog.properties.direction}")
+      expect(subject).to receive(:puts).with("Bucket: #{flowlog.properties.bucket}")
+
+      expect(subject.api_client).not_to receive(:wait_for)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/natgateways/#{subject_config[:natgateway_id]}/flowlogs/#{flowlog.id}",
+            operation: :'NATGatewaysApi.datacenters_natgateways_flowlogs_find_by_flow_log_id',
+            return_type: 'FlowLog',
+            result: flowlog,
+          },
+        ],
+      )
+
+      expect { subject.run }.not_to raise_error(Exception)
+    end
+
+    it 'should call NetworkLoadBalancersApi.datacenters_networkloadbalancers_flowlogs_find_by_flow_log_id' do
+      flowlog = flowlog_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        type: 'networkloadbalancer',
+        datacenter_id: 'datacenter_id',
+        network_loadbalancer_id: 'network_loadbalancer_id',
+        flowlog_id: flowlog.id,
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject).to receive(:puts).with("ID: #{flowlog.id}")
+      expect(subject).to receive(:puts).with("Name: #{flowlog.properties.name}")
+      expect(subject).to receive(:puts).with("Action: #{flowlog.properties.action}")
+      expect(subject).to receive(:puts).with("Direction: #{flowlog.properties.direction}")
+      expect(subject).to receive(:puts).with("Bucket: #{flowlog.properties.bucket}")
+
+      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]}/flowlogs/#{flowlog.id}",
+            operation: :'NetworkLoadBalancersApi.datacenters_networkloadbalancers_flowlogs_find_by_flow_log_id',
+            return_type: 'FlowLog',
+            result: flowlog,
+          },
+        ],
+      )
+
+      expect { subject.run }.not_to raise_error(Exception)
+    end
+
+    it 'should not call anything when the wrong type is given' do
+      flowlog = flowlog_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: 'datacenter_id',
+        server_id: 'server_id',
+        natgateway_id: 'natgateway_id',
+        network_loadbalancer_id: 'network_loadbalancer_id',
+        flowlog_id: 'flowlog_id',
+        nic_id: 'nic_id',
+        type: 'other',
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject.ui).to receive(:error).with("Flow Log cannot belong to #{subject_config[:type]}. Value must be one of ['nic', 'natgateway', 'networkloadbalancer']")
+
+      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
+    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_flowlog_list_spec.rb

@@ -155,7 +155,7 @@ describe Chef::Knife::IonoscloudFlowlogList do
         ionoscloud_password: 'password',
         datacenter_id: 'datacenter_id',
         network_loadbalancer_id: 'network_loadbalancer_id',
-        type: 'loadbalancer',
+        type: 'networkloadbalancer',
       }
 
       subject_config.each { |key, value| subject.config[key] = value }
@@ -200,7 +200,7 @@ describe Chef::Knife::IonoscloudFlowlogList do
         ionoscloud_username: 'email',
         ionoscloud_password: 'password',
         datacenter_id: 'datacenter_id',
-        type: 'loadbalancer',
+        type: 'networkloadbalancer',
       }
 
       subject_config.each { |key, value| subject.config[key] = value }
@@ -234,7 +234,7 @@ describe Chef::Knife::IonoscloudFlowlogList do
 
       subject_config.each { |key, value| subject.config[key] = value }
 
-      expect(subject.ui).to receive(:error).with("Flow log cannot belong to #{subject_config[:type]}. Value must be one of ['nic', 'natgateway', 'loadbalancer']")
+      expect(subject.ui).to receive(:error).with("Flow log cannot belong to #{subject_config[:type]}. Value must be one of ['nic', 'natgateway', 'networkloadbalancer']")
 
       expect(subject.api_client).not_to receive(:call_api)
 

data/spec/chef/knife/ionoscloud_flowlog_update_spec.rb

@@ -0,0 +1,244 @@
+require 'spec_helper'
+require 'ionoscloud_flowlog_update'
+
+Chef::Knife::IonoscloudFlowlogUpdate.load_deps
+
+describe Chef::Knife::IonoscloudFlowlogUpdate do
+  before :each do
+    subject { Chef::Knife::IonoscloudFlowlogUpdate.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call FlowLogsApi.datacenters_servers_nics_flowlogs_patch' do
+      flowlog = flowlog_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        type: 'nic',
+        datacenter_id: 'datacenter_id',
+        server_id: 'server_id',
+        nic_id: 'nic_id',
+        flowlog_id: flowlog.id,
+        name: flowlog.properties.name + '_edited',
+        action: 'REJECTED',
+        direction: 'BIDIRECTIONAL',
+        bucket: 'new_bucket_name',
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject).to receive(:puts).with("ID: #{flowlog.id}")
+      expect(subject).to receive(:puts).with("Name: #{subject_config[:name]}")
+      expect(subject).to receive(:puts).with("Action: #{subject_config[:action]}")
+      expect(subject).to receive(:puts).with("Direction: #{subject_config[:direction]}")
+      expect(subject).to receive(:puts).with("Bucket: #{subject_config[:bucket]}")
+
+      flowlog.properties.name = subject_config[:name]
+      flowlog.properties.action = subject_config[:action]
+      flowlog.properties.direction = subject_config[:direction]
+      flowlog.properties.bucket = subject_config[:bucket]
+
+      expected_body = {
+        name: subject_config[:name],
+        action: subject_config[:action],
+        direction: subject_config[:direction],
+        bucket: subject_config[:bucket],
+      }
+
+      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]}/flowlogs/#{flowlog.id}",
+            operation: :'FlowLogsApi.datacenters_servers_nics_flowlogs_patch',
+            return_type: 'FlowLog',
+            body: expected_body,
+            result: flowlog,
+          },
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/servers/#{subject_config[:server_id]}/"\
+                  "nics/#{subject_config[:nic_id]}/flowlogs/#{flowlog.id}",
+            operation: :'FlowLogsApi.datacenters_servers_nics_flowlogs_find_by_id',
+            return_type: 'FlowLog',
+            result: flowlog,
+          },
+        ],
+      )
+
+      expect { subject.run }.not_to raise_error(Exception)
+    end
+
+    it 'should call NATGatewaysApi.datacenters_natgateways_flowlogs_patch' do
+      flowlog = flowlog_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        type: 'natgateway',
+        datacenter_id: 'datacenter_id',
+        natgateway_id: 'natgateway_id',
+        flowlog_id: flowlog.id,
+        name: flowlog.properties.name + '_edited',
+        action: 'REJECTED',
+        direction: 'BIDIRECTIONAL',
+        bucket: 'new_bucket_name',
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject).to receive(:puts).with("ID: #{flowlog.id}")
+      expect(subject).to receive(:puts).with("Name: #{subject_config[:name]}")
+      expect(subject).to receive(:puts).with("Action: #{subject_config[:action]}")
+      expect(subject).to receive(:puts).with("Direction: #{subject_config[:direction]}")
+      expect(subject).to receive(:puts).with("Bucket: #{subject_config[:bucket]}")
+
+      flowlog.properties.name = subject_config[:name]
+      flowlog.properties.action = subject_config[:action]
+      flowlog.properties.direction = subject_config[:direction]
+      flowlog.properties.bucket = subject_config[:bucket]
+
+      expected_body = {
+        name: subject_config[:name],
+        action: subject_config[:action],
+        direction: subject_config[:direction],
+        bucket: subject_config[:bucket],
+      }
+
+      mock_wait_for(subject)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'PATCH',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/natgateways/#{subject_config[:natgateway_id]}/flowlogs/#{flowlog.id}",
+            operation: :'NATGatewaysApi.datacenters_natgateways_flowlogs_patch',
+            return_type: 'FlowLog',
+            body: expected_body,
+            result: flowlog,
+          },
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/natgateways/#{subject_config[:natgateway_id]}/flowlogs/#{flowlog.id}",
+            operation: :'NATGatewaysApi.datacenters_natgateways_flowlogs_find_by_flow_log_id',
+            return_type: 'FlowLog',
+            result: flowlog,
+          },
+        ],
+      )
+
+      expect { subject.run }.not_to raise_error(Exception)
+    end
+
+    it 'should call NetworkLoadBalancersApi.datacenters_networkloadbalancers_flowlogs_patch' do
+      flowlog = flowlog_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        type: 'networkloadbalancer',
+        datacenter_id: 'datacenter_id',
+        network_loadbalancer_id: 'network_loadbalancer_id',
+        flowlog_id: flowlog.id,
+        name: flowlog.properties.name + '_edited',
+        action: 'REJECTED',
+        direction: 'BIDIRECTIONAL',
+        bucket: 'new_bucket_name',
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject).to receive(:puts).with("ID: #{flowlog.id}")
+      expect(subject).to receive(:puts).with("Name: #{subject_config[:name]}")
+      expect(subject).to receive(:puts).with("Action: #{subject_config[:action]}")
+      expect(subject).to receive(:puts).with("Direction: #{subject_config[:direction]}")
+      expect(subject).to receive(:puts).with("Bucket: #{subject_config[:bucket]}")
+
+      flowlog.properties.name = subject_config[:name]
+      flowlog.properties.action = subject_config[:action]
+      flowlog.properties.direction = subject_config[:direction]
+      flowlog.properties.bucket = subject_config[:bucket]
+
+      expected_body = {
+        name: subject_config[:name],
+        action: subject_config[:action],
+        direction: subject_config[:direction],
+        bucket: subject_config[:bucket],
+      }
+
+      mock_wait_for(subject)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'PATCH',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/networkloadbalancers/#{subject_config[:network_loadbalancer_id]}/flowlogs/#{flowlog.id}",
+            operation: :'NetworkLoadBalancersApi.datacenters_networkloadbalancers_flowlogs_patch',
+            return_type: 'FlowLog',
+            body: expected_body,
+            result: flowlog,
+          },
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/networkloadbalancers/#{subject_config[:network_loadbalancer_id]}/flowlogs/#{flowlog.id}",
+            operation: :'NetworkLoadBalancersApi.datacenters_networkloadbalancers_flowlogs_find_by_flow_log_id',
+            return_type: 'FlowLog',
+            result: flowlog,
+          },
+        ],
+      )
+
+      expect { subject.run }.not_to raise_error(Exception)
+    end
+
+    it 'should not call anything when the wrong type is given' do
+      flowlog = flowlog_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: 'datacenter_id',
+        server_id: 'server_id',
+        natgateway_id: 'natgateway_id',
+        network_loadbalancer_id: 'network_loadbalancer_id',
+        flowlog_id: 'flowlog_id',
+        nic_id: 'nic_id',
+        type: 'other',
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject.ui).to receive(:error).with("Flow Log cannot belong to #{subject_config[:type]}. Value must be one of ['nic', 'natgateway', 'networkloadbalancer']")
+
+      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
+    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