knife-ionoscloud 5.1.1 → 6.0.0.beta.2

data/spec/chef/knife/ionoscloud_datacenter_get_spec.rb

@@ -0,0 +1,69 @@
+require 'spec_helper'
+require 'ionoscloud_datacenter_get'
+
+Chef::Knife::IonoscloudDatacenterGet.load_deps
+
+describe Chef::Knife::IonoscloudDatacenterGet do
+  before :each do
+    subject { Chef::Knife::IonoscloudDatacenterGet.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call DataCentersApi.datacenters_find_by_id' do
+      datacenter = datacenter_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: datacenter.id,
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject).to receive(:puts).with("ID: #{datacenter.id}")
+      expect(subject).to receive(:puts).with("Name: #{datacenter.properties.name}")
+      expect(subject).to receive(:puts).with("Description: #{datacenter.properties.description}")
+      expect(subject).to receive(:puts).with("Location: #{datacenter.properties.location}")
+      expect(subject).to receive(:puts).with("Version: #{datacenter.properties.version}")
+      expect(subject).to receive(:puts).with("Features: #{datacenter.properties.features}")
+      expect(subject).to receive(:puts).with("Sec Auth Protection: #{datacenter.properties.sec_auth_protection}")
+
+      expect(subject.api_client).not_to receive(:wait_for)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/datacenters/#{datacenter.id}",
+            operation: :'DataCentersApi.datacenters_find_by_id',
+            return_type: 'Datacenter',
+            result: datacenter,
+          },
+        ],
+      )
+
+      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_datacenter_list_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudDatacenterList do
   end
 
   describe '#run' do
-    it 'should call DataCenterApi.datacenters_get' do
+    it 'should call DataCentersApi.datacenters_get' do
       datacenters = datacenters_mock
       subject_config = {
         ionoscloud_username: 'email',
@@ -26,6 +26,7 @@ describe Chef::Knife::IonoscloudDatacenterList do
         subject.ui.color('Name', :bold),
         subject.ui.color('Description', :bold),
         subject.ui.color('Location', :bold),
+        subject.ui.color('CPU Architectures', :bold),
         subject.ui.color('Version', :bold),
       ]
 
@@ -34,10 +35,11 @@ describe Chef::Knife::IonoscloudDatacenterList do
         datacenter_list << datacenter.properties.name
         datacenter_list << datacenter.properties.description
         datacenter_list << datacenter.properties.location
+        datacenter_list << datacenter.properties.cpu_architecture.map { |arch| arch.cpu_family }
         datacenter_list << datacenter.properties.version.to_s
       end
 
-      expect(subject.ui).to receive(:list).with(datacenter_list, :uneven_columns_across, 5)
+      expect(subject.ui).to receive(:list).with(datacenter_list, :uneven_columns_across, 6)
 
       mock_call_api(
         subject,
@@ -45,7 +47,7 @@ describe Chef::Knife::IonoscloudDatacenterList do
           {
             method: 'GET',
             path: '/datacenters',
-            operation: :'DataCenterApi.datacenters_get',
+            operation: :'DataCentersApi.datacenters_get',
             return_type: 'Datacenters',
             result: datacenters,
           },

data/spec/chef/knife/ionoscloud_datacenter_update_spec.rb

@@ -0,0 +1,82 @@
+require 'spec_helper'
+require 'ionoscloud_datacenter_update'
+
+Chef::Knife::IonoscloudDatacenterUpdate.load_deps
+
+describe Chef::Knife::IonoscloudDatacenterUpdate do
+  before :each do
+    subject { Chef::Knife::IonoscloudDatacenterUpdate.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call DataCentersApi.datacenters_patch' do
+      datacenter = datacenter_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: datacenter.id,
+        name: datacenter.properties.name + '_edited',
+        description: datacenter.properties.description + '_edited',
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject).to receive(:puts).with("ID: #{datacenter.id}")
+      expect(subject).to receive(:puts).with("Name: #{subject_config[:name]}")
+      expect(subject).to receive(:puts).with("Description: #{subject_config[:description]}")
+      expect(subject).to receive(:puts).with("Location: #{datacenter.properties.location}")
+      expect(subject).to receive(:puts).with("Version: #{datacenter.properties.version}")
+      expect(subject).to receive(:puts).with("Features: #{datacenter.properties.features}")
+      expect(subject).to receive(:puts).with("Sec Auth Protection: #{datacenter.properties.sec_auth_protection}")
+
+      datacenter.properties.name = subject_config[:name]
+      datacenter.properties.description = subject_config[:description]
+
+      mock_wait_for(subject)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'PATCH',
+            path: "/datacenters/#{datacenter.id}",
+            operation: :'DataCentersApi.datacenters_patch',
+            return_type: 'Datacenter',
+            body: { name: subject_config[:name], description: subject_config[:description] },
+            result: datacenter,
+          },
+          {
+            method: 'GET',
+            path: "/datacenters/#{datacenter.id}",
+            operation: :'DataCentersApi.datacenters_find_by_id',
+            return_type: 'Datacenter',
+            result: datacenter,
+          },
+        ],
+      )
+
+      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_firewall_create_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudFirewallCreate do
   end
 
   describe '#run' do
-    it 'should call NicApi.datacenters_servers_nics_firewallrules_post with the expected arguments and output based on what it receives' do
+    it 'should call FirewallRulesApi.datacenters_servers_nics_firewallrules_post with the expected arguments and output based on what it receives' do
       firewall = firewall_mock
       subject_config = {
         ionoscloud_username: 'email',
@@ -29,12 +29,14 @@ describe Chef::Knife::IonoscloudFirewallCreate do
         port_range_end: firewall.properties.port_range_end,
         icmp_type: firewall.properties.icmp_type,
         icmp_code: firewall.properties.icmp_code,
+        type: firewall.properties.type,
       }
 
       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: #{firewall.properties.name}")
+      expect(subject).to receive(:puts).with("Type: #{firewall.properties.type}")
       expect(subject).to receive(:puts).with("Protocol: #{firewall.properties.protocol}")
       expect(subject).to receive(:puts).with("Source MAC: #{firewall.properties.source_mac}")
       expect(subject).to receive(:puts).with("Source IP: #{firewall.properties.source_ip}")
@@ -51,7 +53,7 @@ describe Chef::Knife::IonoscloudFirewallCreate do
           {
             method: 'POST',
             path: "/datacenters/#{subject_config[:datacenter_id]}/servers/#{subject_config[:server_id]}/nics/#{subject_config[:nic_id]}/firewallrules",
-            operation: :'NicApi.datacenters_servers_nics_firewallrules_post',
+            operation: :'FirewallRulesApi.datacenters_servers_nics_firewallrules_post',
             return_type: 'FirewallRule',
             body: { properties: firewall.properties.to_hash },
             result: firewall,
@@ -59,7 +61,7 @@ describe Chef::Knife::IonoscloudFirewallCreate do
           {
             method: 'GET',
             path: "/datacenters/#{subject_config[:datacenter_id]}/servers/#{subject_config[:server_id]}/nics/#{subject_config[:nic_id]}/firewallrules/#{firewall.id}",
-            operation: :'NicApi.datacenters_servers_nics_firewallrules_find_by_id',
+            operation: :'FirewallRulesApi.datacenters_servers_nics_firewallrules_find_by_id',
             return_type: 'FirewallRule',
             result: firewall,
           },

data/spec/chef/knife/ionoscloud_firewall_delete_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudFirewallDelete do
   end
 
   describe '#run' do
-    it 'should call NicApi.datacenters_servers_nics_firewallrules_firewallrules_delete when the ID is valid' do
+    it 'should call FirewallRulesApi.datacenters_servers_nics_firewallrules_firewallrules_delete when the ID is valid' do
       firewall = firewall_mock
       subject_config = {
         ionoscloud_username: 'email',
@@ -28,6 +28,7 @@ describe Chef::Knife::IonoscloudFirewallDelete do
 
       expect(subject).to receive(:puts).with("ID: #{firewall.id}")
       expect(subject).to receive(:puts).with("Name: #{firewall.properties.name}")
+      expect(subject).to receive(:puts).with("Type: #{firewall.properties.type}")
       expect(subject).to receive(:puts).with("Protocol: #{firewall.properties.protocol}")
       expect(subject).to receive(:puts).with("Source MAC: #{firewall.properties.source_mac}")
       expect(subject).to receive(:puts).with("Source IP: #{firewall.properties.source_ip}")
@@ -47,7 +48,7 @@ describe Chef::Knife::IonoscloudFirewallDelete do
             method: 'GET',
             path: "/datacenters/#{subject_config[:datacenter_id]}/servers/#{subject_config[:server_id]}/"\
                   "nics/#{subject_config[:nic_id]}/firewallrules/#{firewall.id}",
-            operation: :'NicApi.datacenters_servers_nics_firewallrules_find_by_id',
+            operation: :'FirewallRulesApi.datacenters_servers_nics_firewallrules_find_by_id',
             return_type: 'FirewallRule',
             result: firewall,
           },
@@ -55,7 +56,7 @@ describe Chef::Knife::IonoscloudFirewallDelete do
             method: 'DELETE',
             path: "/datacenters/#{subject_config[:datacenter_id]}/servers/#{subject_config[:server_id]}/"\
                   "nics/#{subject_config[:nic_id]}/firewallrules/#{firewall.id}",
-            operation: :'NicApi.datacenters_servers_nics_firewallrules_delete',
+            operation: :'FirewallRulesApi.datacenters_servers_nics_firewallrules_delete',
           },
         ],
       )
@@ -63,7 +64,7 @@ describe Chef::Knife::IonoscloudFirewallDelete do
       expect { subject.run }.not_to raise_error(Exception)
     end
 
-    it 'should not call NicApi.datacenters_servers_nics_firewallrules_delete when the user ID is not valid' do
+    it 'should not call FirewallRulesApi.datacenters_servers_nics_firewallrules_delete when the user ID is not valid' do
       firewall_id = 'invalid_id'
       subject_config = {
         ionoscloud_username: 'email',
@@ -86,7 +87,7 @@ describe Chef::Knife::IonoscloudFirewallDelete do
             method: 'GET',
             path: "/datacenters/#{subject_config[:datacenter_id]}/servers/#{subject_config[:server_id]}/"\
                   "nics/#{subject_config[:nic_id]}/firewallrules/#{firewall_id}",
-            operation: :'NicApi.datacenters_servers_nics_firewallrules_find_by_id',
+            operation: :'FirewallRulesApi.datacenters_servers_nics_firewallrules_find_by_id',
             return_type: 'FirewallRule',
             exception: Ionoscloud::ApiError.new(code: 404),
           },

data/spec/chef/knife/ionoscloud_firewall_get_spec.rb

@@ -0,0 +1,77 @@
+require 'spec_helper'
+require 'ionoscloud_firewall_get'
+
+Chef::Knife::IonoscloudFirewallGet.load_deps
+
+describe Chef::Knife::IonoscloudFirewallGet do
+  before :each do
+    subject { Chef::Knife::IonoscloudFirewallGet.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call FirewallRulesApi.datacenters_servers_nics_firewallrules_find_by_id' 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,
+        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: #{firewall.properties.name}")
+      expect(subject).to receive(:puts).with("Type: #{firewall.properties.type}")
+      expect(subject).to receive(:puts).with("Protocol: #{firewall.properties.protocol}")
+      expect(subject).to receive(:puts).with("Source MAC: #{firewall.properties.source_mac}")
+      expect(subject).to receive(:puts).with("Source IP: #{firewall.properties.source_ip}")
+      expect(subject).to receive(:puts).with("Target IP: #{firewall.properties.target_ip}")
+      expect(subject).to receive(:puts).with("Port Range Start: #{firewall.properties.port_range_start}")
+      expect(subject).to receive(:puts).with("Port Range End: #{firewall.properties.port_range_end}")
+      expect(subject).to receive(:puts).with("ICMP Type: #{firewall.properties.icmp_type}")
+      expect(subject).to receive(:puts).with("ICMP Code: #{firewall.properties.icmp_code}")
+
+      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]}/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_firewall_list_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudFirewallList do
   end
 
   describe '#run' do
-    it 'should call NicApi.datacenters_servers_nics_firewallrules_get' do
+    it 'should call FirewallRulesApi.datacenters_servers_nics_firewallrules_get' do
       firewalls = firewalls_mock
       subject_config = {
         ionoscloud_username: 'email',
@@ -35,6 +35,7 @@ describe Chef::Knife::IonoscloudFirewallList do
         subject.ui.color('Port Range End', :bold),
         subject.ui.color('ICMP Type', :bold),
         subject.ui.color('ICMP CODE', :bold),
+        subject.ui.color('Type', :bold),
       ]
 
       firewalls.items.each do |firewall|
@@ -48,9 +49,10 @@ describe Chef::Knife::IonoscloudFirewallList do
         firewall_list << firewall.properties.port_range_end.to_s
         firewall_list << firewall.properties.icmp_type.to_s
         firewall_list << firewall.properties.icmp_code.to_s
+        firewall_list << firewall.properties.type.to_s
       end
 
-      expect(subject.ui).to receive(:list).with(firewall_list, :uneven_columns_across, 10)
+      expect(subject.ui).to receive(:list).with(firewall_list, :uneven_columns_across, 11)
 
       mock_call_api(
         subject,
@@ -59,7 +61,7 @@ describe Chef::Knife::IonoscloudFirewallList do
             method: 'GET',
             path: "/datacenters/#{subject_config[:datacenter_id]}/servers/#{subject_config[:server_id]}/"\
                   "nics/#{subject_config[:nic_id]}/firewallrules",
-            operation: :'NicApi.datacenters_servers_nics_firewallrules_get',
+            operation: :'FirewallRulesApi.datacenters_servers_nics_firewallrules_get',
             return_type: 'FirewallRules',
             result: firewalls,
           },

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