knife-ionoscloud 5.1.1 → 6.0.0.beta.2

data/spec/chef/knife/ionoscloud_lan_delete_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudLanDelete do
   end
 
   describe '#run' do
-    it 'should call LanApi.datacenters_lans_delete when the ID is valid' do
+    it 'should call LansApi.datacenters_lans_delete when the ID is valid' do
       lan = lan_mock
       subject_config = {
         ionoscloud_username: 'email',
@@ -27,6 +27,8 @@ describe Chef::Knife::IonoscloudLanDelete do
       expect(subject).to receive(:puts).with("ID: #{lan.id}")
       expect(subject).to receive(:puts).with("Name: #{lan.properties.name}")
       expect(subject).to receive(:puts).with("Public: #{lan.properties.public.to_s}")
+      expect(subject).to receive(:puts).with("PCC: #{lan.properties.pcc}")
+      expect(subject).to receive(:puts).with("IP Failover: #{[]}")
       expect(subject.ui).to receive(:warn).with("Deleted Lan #{lan.id}. Request ID: ")
 
       expect(subject.api_client).not_to receive(:wait_for)
@@ -37,14 +39,14 @@ describe Chef::Knife::IonoscloudLanDelete do
           {
             method: 'GET',
             path: "/datacenters/#{subject_config[:datacenter_id]}/lans/#{lan.id}",
-            operation: :'LanApi.datacenters_lans_find_by_id',
+            operation: :'LansApi.datacenters_lans_find_by_id',
             return_type: 'Lan',
             result: lan,
           },
           {
             method: 'DELETE',
             path: "/datacenters/#{subject_config[:datacenter_id]}/lans/#{lan.id}",
-            operation: :'LanApi.datacenters_lans_delete',
+            operation: :'LansApi.datacenters_lans_delete',
           },
         ],
       )
@@ -52,7 +54,7 @@ describe Chef::Knife::IonoscloudLanDelete do
       expect { subject.run }.not_to raise_error(Exception)
     end
 
-    it 'should not call LanApi.datacenters_lans_delete when the user ID is not valid' do
+    it 'should not call LansApi.datacenters_lans_delete when the user ID is not valid' do
       lan_id = 'invalid_id'
       subject_config = {
         ionoscloud_username: 'email',
@@ -72,7 +74,7 @@ describe Chef::Knife::IonoscloudLanDelete do
           {
             method: 'GET',
             path: "/datacenters/#{subject_config[:datacenter_id]}/lans/#{lan_id}",
-            operation: :'LanApi.datacenters_lans_find_by_id',
+            operation: :'LansApi.datacenters_lans_find_by_id',
             return_type: 'Lan',
             exception: Ionoscloud::ApiError.new(code: 404),
           },

data/spec/chef/knife/ionoscloud_lan_get_spec.rb

@@ -0,0 +1,68 @@
+require 'spec_helper'
+require 'ionoscloud_lan_get'
+
+Chef::Knife::IonoscloudLanGet.load_deps
+
+describe Chef::Knife::IonoscloudLanGet do
+  before :each do
+    subject { Chef::Knife::IonoscloudLanGet.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call LansApi.datacenters_lans_find_by_id' do
+      lan = lan_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: 'datacenter_id',
+        lan_id: lan.id,
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject).to receive(:puts).with("ID: #{lan.id}")
+      expect(subject).to receive(:puts).with("Name: #{lan.properties.name}")
+      expect(subject).to receive(:puts).with("Public: #{lan.properties.public.to_s}")
+      expect(subject).to receive(:puts).with("PCC: #{lan.properties.pcc}")
+      expect(subject).to receive(:puts).with("IP Failover: #{[]}")
+
+      expect(subject.api_client).not_to receive(:wait_for)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/lans/#{subject_config[:lan_id]}",
+            operation: :'LansApi.datacenters_lans_find_by_id',
+            return_type: 'Lan',
+            result: lan,
+          },
+        ],
+      )
+
+      expect { subject.run }.not_to raise_error(Exception)
+    end
+
+    it 'should not make any call if any required option is missing' do
+      required_options = subject.instance_variable_get(:@required_options)
+
+      arrays_without_one_element(required_options).each do |test_case|
+
+        test_case[:array].each { |value| subject.config[value] = 'test' }
+
+        expect(subject).to receive(:puts).with("Missing required parameters #{test_case[:removed]}")
+        expect(subject.api_client).not_to receive(:call_api)
+
+        expect { subject.run }.to raise_error(SystemExit) do |error|
+          expect(error.status).to eq(1)
+        end
+
+        required_options.each { |value| subject.config[value] = nil }
+      end
+    end
+  end
+end

data/spec/chef/knife/ionoscloud_lan_list_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudLanList do
   end
 
   describe '#run' do
-    it 'should call LanApi.datacenters_lans_get' do
+    it 'should call LansApi.datacenters_lans_get' do
       lans = lans_mock
       subject_config = {
         ionoscloud_username: 'email',
@@ -44,7 +44,7 @@ describe Chef::Knife::IonoscloudLanList do
           {
             method: 'GET',
             path: "/datacenters/#{subject_config[:datacenter_id]}/lans",
-            operation: :'LanApi.datacenters_lans_get',
+            operation: :'LansApi.datacenters_lans_get',
             return_type: 'Lans',
             result: lans,
           },

data/spec/chef/knife/ionoscloud_lan_update_spec.rb

@@ -0,0 +1,87 @@
+require 'spec_helper'
+require 'ionoscloud_lan_update'
+
+Chef::Knife::IonoscloudLanUpdate.load_deps
+
+describe Chef::Knife::IonoscloudLanUpdate do
+  before :each do
+    subject { Chef::Knife::IonoscloudLanUpdate.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call LansApi.datacenters_lans_patch' do
+      lan = lan_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: 'datacenter_id',
+        lan_id: lan.id,
+        name: lan.properties.name + '_edited',
+        public: (!lan.properties.public).to_s,
+        pcc: 'pcc_id',
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject).to receive(:puts).with("ID: #{lan.id}")
+      expect(subject).to receive(:puts).with("Name: #{subject_config[:name]}")
+      expect(subject).to receive(:puts).with("Public: #{subject_config[:public]}")
+      expect(subject).to receive(:puts).with("PCC: #{subject_config[:pcc]}")
+      expect(subject).to receive(:puts).with("IP Failover: #{[]}")
+
+      lan.properties.name = subject_config[:name]
+      lan.properties.public = subject_config[:public].to_s.downcase == 'true'
+      lan.properties.pcc = subject_config[:pcc]
+
+      mock_wait_for(subject)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'PATCH',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/lans/#{subject_config[:lan_id]}",
+            operation: :'LansApi.datacenters_lans_patch',
+            return_type: 'Lan',
+            body: {
+              name: subject_config[:name],
+              public: subject_config[:public].to_s.downcase == 'true',
+              pcc: subject_config[:pcc],
+            },
+            result: lan,
+          },
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/lans/#{subject_config[:lan_id]}",
+            operation: :'LansApi.datacenters_lans_find_by_id',
+            return_type: 'Lan',
+            result: lan,
+          },
+        ],
+      )
+
+      expect { subject.run }.not_to raise_error(Exception)
+    end
+
+    it 'should not make any call if any required option is missing' do
+      required_options = subject.instance_variable_get(:@required_options)
+
+      arrays_without_one_element(required_options).each do |test_case|
+
+        test_case[:array].each { |value| subject.config[value] = 'test' }
+
+        expect(subject).to receive(:puts).with("Missing required parameters #{test_case[:removed]}")
+        expect(subject.api_client).not_to receive(:call_api)
+
+        expect { subject.run }.to raise_error(SystemExit) do |error|
+          expect(error.status).to eq(1)
+        end
+
+        required_options.each { |value| subject.config[value] = nil }
+      end
+    end
+  end
+end

data/spec/chef/knife/ionoscloud_loadbalancer_create_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudLoadbalancerCreate do
   end
 
   describe '#run' do
-    it 'should call KubernetesApi.k8s_nodepools_post with the expected arguments and output based on what it receives' do
+    it 'should call LoadBalancersApi.datacenters_loadbalancers_post with the expected arguments and output based on what it receives' do
       load_balancer = load_balancer_mock
       subject_config = {
         ionoscloud_username: 'email',
@@ -43,7 +43,7 @@ describe Chef::Knife::IonoscloudLoadbalancerCreate do
           {
             method: 'POST',
             path: "/datacenters/#{subject_config[:datacenter_id]}/loadbalancers",
-            operation: :'LoadBalancerApi.datacenters_loadbalancers_post',
+            operation: :'LoadBalancersApi.datacenters_loadbalancers_post',
             return_type: 'Loadbalancer',
             body: { properties: load_balancer.properties.to_hash, entities: expected_entities },
             result: load_balancer,
@@ -51,7 +51,7 @@ describe Chef::Knife::IonoscloudLoadbalancerCreate do
           {
             method: 'GET',
             path: "/datacenters/#{subject_config[:datacenter_id]}/loadbalancers/#{load_balancer.id}",
-            operation: :'LoadBalancerApi.datacenters_loadbalancers_find_by_id',
+            operation: :'LoadBalancersApi.datacenters_loadbalancers_find_by_id',
             return_type: 'Loadbalancer',
             result: load_balancer,
           },

data/spec/chef/knife/ionoscloud_loadbalancer_delete_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudLoadbalancerDelete do
   end
 
   describe '#run' do
-    it 'should call LoadBalancerApi.datacenters_loadbalancers_delete when the ID is valid' do
+    it 'should call LoadBalancersApi.datacenters_loadbalancers_delete when the ID is valid' do
       load_balancer = load_balancer_mock
       subject_config = {
         ionoscloud_username: 'email',
@@ -41,14 +41,14 @@ describe Chef::Knife::IonoscloudLoadbalancerDelete do
           {
             method: 'GET',
             path: "/datacenters/#{subject_config[:datacenter_id]}/loadbalancers/#{load_balancer.id}",
-            operation: :'LoadBalancerApi.datacenters_loadbalancers_find_by_id',
+            operation: :'LoadBalancersApi.datacenters_loadbalancers_find_by_id',
             return_type: 'Loadbalancer',
             result: load_balancer,
           },
           {
             method: 'DELETE',
             path: "/datacenters/#{subject_config[:datacenter_id]}/loadbalancers/#{load_balancer.id}",
-            operation: :'LoadBalancerApi.datacenters_loadbalancers_delete',
+            operation: :'LoadBalancersApi.datacenters_loadbalancers_delete',
           },
         ],
       )
@@ -56,7 +56,7 @@ describe Chef::Knife::IonoscloudLoadbalancerDelete do
       expect { subject.run }.not_to raise_error(Exception)
     end
 
-    it 'should not call LoadBalancerApi.datacenters_loadbalancers_delete when the ID is not valid' do
+    it 'should not call LoadBalancersApi.datacenters_loadbalancers_delete when the ID is not valid' do
       load_balancer_id = 'invalid_id'
       subject_config = {
         ionoscloud_username: 'email',
@@ -76,7 +76,7 @@ describe Chef::Knife::IonoscloudLoadbalancerDelete do
           {
             method: 'GET',
             path: "/datacenters/#{subject_config[:datacenter_id]}/loadbalancers/#{load_balancer_id}",
-            operation: :'LoadBalancerApi.datacenters_loadbalancers_find_by_id',
+            operation: :'LoadBalancersApi.datacenters_loadbalancers_find_by_id',
             return_type: 'Loadbalancer',
             exception: Ionoscloud::ApiError.new(code: 404),
           },

data/spec/chef/knife/ionoscloud_loadbalancer_get_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudLoadbalancerGet do
   end
 
   describe '#run' do
-    it 'should call LoadBalancerApi.datacenters_loadbalancers_get' do
+    it 'should call LoadBalancersApi.datacenters_loadbalancers_get' do
       load_balancer = load_balancer_mock
       subject_config = {
         ionoscloud_username: 'email',
@@ -37,7 +37,7 @@ describe Chef::Knife::IonoscloudLoadbalancerGet do
           {
             method: 'GET',
             path: "/datacenters/#{subject_config[:datacenter_id]}/loadbalancers/#{subject_config[:loadbalancer_id]}",
-            operation: :'LoadBalancerApi.datacenters_loadbalancers_find_by_id',
+            operation: :'LoadBalancersApi.datacenters_loadbalancers_find_by_id',
             return_type: 'Loadbalancer',
             result: load_balancer,
           },

data/spec/chef/knife/ionoscloud_loadbalancer_list_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudLoadbalancerList do
   end
 
   describe '#run' do
-    it 'should call LoadBalancerApi.datacenters_loadbalancers_get' do
+    it 'should call LoadBalancersApi.datacenters_loadbalancers_get' do
       load_balancers = load_balancers_mock
       subject_config = {
         ionoscloud_username: 'email',
@@ -43,7 +43,7 @@ describe Chef::Knife::IonoscloudLoadbalancerList do
           {
             method: 'GET',
             path: "/datacenters/#{subject_config[:datacenter_id]}/loadbalancers",
-            operation: :'LoadBalancerApi.datacenters_loadbalancers_get',
+            operation: :'LoadBalancersApi.datacenters_loadbalancers_get',
             return_type: 'Loadbalancers',
             result: load_balancers,
           },

data/spec/chef/knife/ionoscloud_loadbalancer_nic_add_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudLoadbalancerNicAdd do
   end
 
   describe '#run' do
-    it 'should call LoadBalancerApi.datacenters_loadbalancers_delete when the ID is valid' do
+    it 'should call LoadBalancersApi.datacenters_loadbalancers_delete when the ID is valid' do
       load_balancer = load_balancer_mock
       nic = nic_mock
       subject_config = {
@@ -43,14 +43,14 @@ describe Chef::Knife::IonoscloudLoadbalancerNicAdd do
           {
             method: 'POST',
             path: "/datacenters/#{subject_config[:datacenter_id]}/loadbalancers/#{load_balancer.id}/balancednics",
-            operation: :'LoadBalancerApi.datacenters_loadbalancers_balancednics_post',
+            operation: :'LoadBalancersApi.datacenters_loadbalancers_balancednics_post',
             body: { id: nic.id },
             return_type: 'Nic',
           },
           {
             method: 'GET',
             path: "/datacenters/#{subject_config[:datacenter_id]}/loadbalancers/#{load_balancer.id}",
-            operation: :'LoadBalancerApi.datacenters_loadbalancers_find_by_id',
+            operation: :'LoadBalancersApi.datacenters_loadbalancers_find_by_id',
             return_type: 'Loadbalancer',
             result: load_balancer,
           },
@@ -60,7 +60,7 @@ describe Chef::Knife::IonoscloudLoadbalancerNicAdd do
       expect { subject.run }.not_to raise_error(Exception)
     end
 
-    it 'should not call LoadBalancerApi.datacenters_loadbalancers_delete when the ID is not valid' do
+    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 = {
@@ -89,7 +89,7 @@ describe Chef::Knife::IonoscloudLoadbalancerNicAdd do
           {
             method: 'POST',
             path: "/datacenters/#{subject_config[:datacenter_id]}/loadbalancers/#{subject_config[:loadbalancer_id]}/balancednics",
-            operation: :'LoadBalancerApi.datacenters_loadbalancers_balancednics_post',
+            operation: :'LoadBalancersApi.datacenters_loadbalancers_balancednics_post',
             body: { id: nic_id },
             return_type: 'Nic',
             exception: Ionoscloud::ApiError.new(code: 404),
@@ -97,7 +97,7 @@ describe Chef::Knife::IonoscloudLoadbalancerNicAdd do
           {
             method: 'GET',
             path: "/datacenters/#{subject_config[:datacenter_id]}/loadbalancers/#{load_balancer.id}",
-            operation: :'LoadBalancerApi.datacenters_loadbalancers_find_by_id',
+            operation: :'LoadBalancersApi.datacenters_loadbalancers_find_by_id',
             return_type: 'Loadbalancer',
             result: load_balancer,
           },

data/spec/chef/knife/ionoscloud_loadbalancer_nic_remove_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudLoadbalancerNicRemove do
   end
 
   describe '#run' do
-    it 'should call LoadBalancerApi.datacenters_loadbalancers_delete when the ID is valid' do
+    it 'should call LoadBalancersApi.datacenters_loadbalancers_delete when the ID is valid' do
       load_balancer = load_balancer_mock
       nic = nic_mock
       subject_config = {
@@ -43,12 +43,12 @@ describe Chef::Knife::IonoscloudLoadbalancerNicRemove do
           {
             method: 'DELETE',
             path: "/datacenters/#{subject_config[:datacenter_id]}/loadbalancers/#{load_balancer.id}/balancednics/#{nic.id}",
-            operation: :'LoadBalancerApi.datacenters_loadbalancers_balancednics_delete',
+            operation: :'LoadBalancersApi.datacenters_loadbalancers_balancednics_delete',
           },
           {
             method: 'GET',
             path: "/datacenters/#{subject_config[:datacenter_id]}/loadbalancers/#{load_balancer.id}",
-            operation: :'LoadBalancerApi.datacenters_loadbalancers_find_by_id',
+            operation: :'LoadBalancersApi.datacenters_loadbalancers_find_by_id',
             return_type: 'Loadbalancer',
             result: load_balancer,
           },
@@ -58,7 +58,7 @@ describe Chef::Knife::IonoscloudLoadbalancerNicRemove do
       expect { subject.run }.not_to raise_error(Exception)
     end
 
-    it 'should not call LoadBalancerApi.datacenters_loadbalancers_delete when the ID is not valid' do
+    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 = {
@@ -87,13 +87,13 @@ describe Chef::Knife::IonoscloudLoadbalancerNicRemove do
           {
             method: 'DELETE',
             path: "/datacenters/#{subject_config[:datacenter_id]}/loadbalancers/#{subject_config[:loadbalancer_id]}/balancednics/#{nic_id}",
-            operation: :'LoadBalancerApi.datacenters_loadbalancers_balancednics_delete',
+            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: :'LoadBalancerApi.datacenters_loadbalancers_find_by_id',
+            operation: :'LoadBalancersApi.datacenters_loadbalancers_find_by_id',
             return_type: 'Loadbalancer',
             result: load_balancer,
           },

data/spec/chef/knife/ionoscloud_loadbalancer_update_spec.rb

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

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudLocationList do
   end
 
   describe '#run' do
-    it 'should call LocationApi.locations_get' do
+    it 'should call LocationsApi.locations_get' do
       locations = locations_mock
       subject_config = {
         ionoscloud_username: 'email',
@@ -24,11 +24,13 @@ describe Chef::Knife::IonoscloudLocationList do
       location_list = [
         subject.ui.color('ID', :bold),
         subject.ui.color('Name', :bold),
+        subject.ui.color('CPU Architectures', :bold),
         locations.items.first.id,
         locations.items.first.properties.name,
+        locations.items.first.properties.cpu_architecture.map { |arch| arch.cpu_family }
       ]
 
-      expect(subject.ui).to receive(:list).with(location_list, :uneven_columns_across, 2)
+      expect(subject.ui).to receive(:list).with(location_list, :uneven_columns_across, 3)
 
       mock_call_api(
         subject,
@@ -36,7 +38,7 @@ describe Chef::Knife::IonoscloudLocationList do
           {
             method: 'GET',
             path: '/locations',
-            operation: :'LocationApi.locations_get',
+            operation: :'LocationsApi.locations_get',
             return_type: 'Locations',
             result: locations,
           },

data/spec/chef/knife/ionoscloud_natgateway_create_spec.rb

@@ -0,0 +1,92 @@
+require 'spec_helper'
+require 'ionoscloud_natgateway_create'
+
+Chef::Knife::IonoscloudNatgatewayCreate.load_deps
+
+describe Chef::Knife::IonoscloudNatgatewayCreate do
+  before :each do
+    subject { Chef::Knife::IonoscloudNatgatewayCreate.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call NATGatewaysApi.datacenters_natgateways_post with the expected arguments and output based on what it receives' do
+      natgateway = natgateway_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: 'datacenter_id',
+        name: natgateway.properties.name,
+        ips: natgateway.properties.public_ips.join(','),
+      }.each do |key, value|
+        subject.config[key] = value
+      end
+
+      expect(subject).to receive(:puts).with("ID: #{natgateway.id}")
+      expect(subject).to receive(:puts).with("Name: #{natgateway.properties.name}")
+      expect(subject).to receive(:puts).with("IPS: #{natgateway.properties.public_ips}")
+      expect(subject).to receive(:puts).with("LANS: #{natgateway.properties.lans.map { |el| { id: el.id, gateway_ips: el.gateway_ips } }}")
+      expect(subject).to receive(:puts).with("Rules: #{natgateway.entities.rules.items.map do |el|
+        {
+          id: el.id,
+          name: el.properties.name,
+          type: el.properties.type,
+          protocol: el.properties.protocol,
+          public_ip: el.properties.public_ip,
+          source_subnet: el.properties.source_subnet,
+          target_subnet: el.properties.target_subnet,
+          target_port_range_start: el.properties.target_port_range ? el.properties.target_port_range.start : '',
+          target_port_range_end: el.properties.target_port_range ? el.properties.target_port_range._end : '',
+        }
+      end}")
+      expect(subject).to receive(:puts).with("Flowlogs: #{natgateway.entities.flowlogs.items.map { |flowlog| flowlog.id }}")
+
+      expected_properties = natgateway.properties.to_hash
+      expected_properties.delete(:lans)
+
+      mock_wait_for(subject)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'POST',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/natgateways",
+            operation: :'NATGatewaysApi.datacenters_natgateways_post',
+            return_type: 'NatGateway',
+            body: { properties: expected_properties },
+            result: natgateway,
+          },
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/natgateways/#{natgateway.id}",
+            operation: :'NATGatewaysApi.datacenters_natgateways_find_by_nat_gateway_id',
+            return_type: 'NatGateway',
+            result: natgateway,
+          },
+        ],
+      )
+
+      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