knife-ionoscloud 5.1.2 → 6.0.0

data/spec/chef/knife/ionoscloud_nodepool_lan_add_spec.rb

@@ -0,0 +1,222 @@
+require 'spec_helper'
+require 'ionoscloud_nodepool_lan_add'
+
+Chef::Knife::IonoscloudNodepoolLanAdd.load_deps
+
+describe Chef::Knife::IonoscloudNodepoolLanAdd do
+  before :each do
+    subject { Chef::Knife::IonoscloudNodepoolLanAdd.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call KubernetesApi.k8s_nodepools_put to add the lan when the ID is valid' do
+      nodepool = k8s_nodepool_mock
+      nodepool_lan = nodepool_lan_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        cluster_id: 'cluster_id',
+        nodepool_id: nodepool.id,
+        lan_id: nodepool_lan.id,
+        no_dhcp: !nodepool_lan.dhcp,
+        routes: [nodepool_lan.routes.first.network, nodepool_lan.routes.first.gateway_ip].join(','),
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      auto_scaling = "Min node count: #{nodepool.properties.auto_scaling.min_node_count}, Max node count:#{nodepool.properties.auto_scaling.max_node_count}"
+      maintenance_window = "#{nodepool.properties.maintenance_window.day_of_the_week}, #{nodepool.properties.maintenance_window.time}"
+      lans = (nodepool.properties.lans + [nodepool_lan]).map { |lan| lan.to_hash }
+
+      expect(subject).to receive(:puts).with("ID: #{nodepool.id}")
+      expect(subject).to receive(:puts).with("Name: #{nodepool.properties.name}")
+      expect(subject).to receive(:puts).with("K8s Version: #{nodepool.properties.k8s_version}")
+      expect(subject).to receive(:puts).with("Datacenter ID: #{nodepool.properties.datacenter_id}")
+      expect(subject).to receive(:puts).with("Node Count: #{nodepool.properties.node_count}")
+      expect(subject).to receive(:puts).with("CPU Family: #{nodepool.properties.cpu_family}")
+      expect(subject).to receive(:puts).with("Cores Count: #{nodepool.properties.cores_count}")
+      expect(subject).to receive(:puts).with("RAM: #{nodepool.properties.ram_size}")
+      expect(subject).to receive(:puts).with("Storage Type: #{nodepool.properties.storage_type}")
+      expect(subject).to receive(:puts).with("Storage Size: #{nodepool.properties.storage_size}")
+      expect(subject).to receive(:puts).with("Public IPs: #{nodepool.properties.public_ips}")
+      expect(subject).to receive(:puts).with("LANs: #{lans}")
+      expect(subject).to receive(:puts).with("Availability Zone: #{nodepool.properties.availability_zone}")
+      expect(subject).to receive(:puts).with("Auto Scaling: #{auto_scaling}")
+      expect(subject).to receive(:puts).with("Maintenance Window: #{maintenance_window}")
+      expect(subject).to receive(:puts).with("State: #{nodepool.metadata.state}")
+      expect(subject.ui).to receive(:info).with("Adding Lan #{subject_config[:lan_id]} to the Nodepoool.")
+
+
+      expected_body = nodepool.properties.to_hash
+      expected_body[:lans] << nodepool_lan.to_hash
+      expected_body.delete(:name)
+      expected_body.delete(:datacenterId)
+      expected_body.delete(:ramSize)
+      expected_body.delete(:cpuFamily)
+      expected_body.delete(:coresCount)
+      expected_body.delete(:storageSize)
+      expected_body.delete(:storageType)
+      expected_body.delete(:availabilityZone)
+      expected_body.delete(:availableUpgradeVersions)
+      expected_body.delete(:labels)
+      expected_body.delete(:annotations)
+
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/k8s/#{subject_config[:cluster_id]}/nodepools/#{subject_config[:nodepool_id]}",
+            operation: :'KubernetesApi.k8s_nodepools_find_by_id',
+            return_type: 'KubernetesNodePool',
+            result: nodepool,
+          },
+          {
+            method: 'PUT',
+            path: "/k8s/#{subject_config[:cluster_id]}/nodepools/#{subject_config[:nodepool_id]}",
+            operation: :'KubernetesApi.k8s_nodepools_put',
+            body: { properties: expected_body },
+            return_type: 'KubernetesNodePool',
+            result: nodepool,
+          },
+        ],
+      )
+
+      expect { subject.run }.not_to raise_error(Exception)
+    end
+
+    it 'should call KubernetesApi.k8s_nodepools_put to update the lan when the ID is valid' do
+      new_gateway = '1.11.1.1'
+      nodepool = k8s_nodepool_mock()
+      nodepool_lan = nodepool_lan_mock(id: nodepool.properties.lans.first.id, gateway_ip: new_gateway)
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        cluster_id: 'cluster_id',
+        nodepool_id: nodepool.id,
+        lan_id: nodepool_lan.id,
+        no_dhcp: !nodepool_lan.dhcp,
+        routes: [nodepool_lan.routes.first.network, nodepool_lan.routes.first.gateway_ip].join(','),
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+
+      auto_scaling = "Min node count: #{nodepool.properties.auto_scaling.min_node_count}, Max node count:#{nodepool.properties.auto_scaling.max_node_count}"
+      maintenance_window = "#{nodepool.properties.maintenance_window.day_of_the_week}, #{nodepool.properties.maintenance_window.time}"
+      lans = [nodepool.properties.lans[1], nodepool_lan].map { |lan| lan.to_hash }
+
+      expect(subject).to receive(:puts).with("ID: #{nodepool.id}")
+      expect(subject).to receive(:puts).with("Name: #{nodepool.properties.name}")
+      expect(subject).to receive(:puts).with("K8s Version: #{nodepool.properties.k8s_version}")
+      expect(subject).to receive(:puts).with("Datacenter ID: #{nodepool.properties.datacenter_id}")
+      expect(subject).to receive(:puts).with("Node Count: #{nodepool.properties.node_count}")
+      expect(subject).to receive(:puts).with("CPU Family: #{nodepool.properties.cpu_family}")
+      expect(subject).to receive(:puts).with("Cores Count: #{nodepool.properties.cores_count}")
+      expect(subject).to receive(:puts).with("RAM: #{nodepool.properties.ram_size}")
+      expect(subject).to receive(:puts).with("Storage Type: #{nodepool.properties.storage_type}")
+      expect(subject).to receive(:puts).with("Storage Size: #{nodepool.properties.storage_size}")
+      expect(subject).to receive(:puts).with("Public IPs: #{nodepool.properties.public_ips}")
+      expect(subject).to receive(:puts).with("LANs: #{lans}")
+      expect(subject).to receive(:puts).with("Availability Zone: #{nodepool.properties.availability_zone}")
+      expect(subject).to receive(:puts).with("Auto Scaling: #{auto_scaling}")
+      expect(subject).to receive(:puts).with("Maintenance Window: #{maintenance_window}")
+      expect(subject).to receive(:puts).with("State: #{nodepool.metadata.state}")
+      expect(subject.ui).to receive(:info).with("Updating Lan #{subject_config[:lan_id]} in the Nodepoool.")
+
+      expected_body = nodepool.properties.to_hash
+      expected_body[:lans] = [expected_body[:lans][1], nodepool_lan.to_hash]
+      expected_body.delete(:name)
+      expected_body.delete(:datacenterId)
+      expected_body.delete(:ramSize)
+      expected_body.delete(:cpuFamily)
+      expected_body.delete(:coresCount)
+      expected_body.delete(:storageSize)
+      expected_body.delete(:storageType)
+      expected_body.delete(:availabilityZone)
+      expected_body.delete(:availableUpgradeVersions)
+      expected_body.delete(:labels)
+      expected_body.delete(:annotations)
+
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/k8s/#{subject_config[:cluster_id]}/nodepools/#{subject_config[:nodepool_id]}",
+            operation: :'KubernetesApi.k8s_nodepools_find_by_id',
+            return_type: 'KubernetesNodePool',
+            result: nodepool,
+          },
+          {
+            method: 'PUT',
+            path: "/k8s/#{subject_config[:cluster_id]}/nodepools/#{subject_config[:nodepool_id]}",
+            operation: :'KubernetesApi.k8s_nodepools_put',
+            body: { properties: expected_body },
+            return_type: 'KubernetesNodePool',
+            result: nodepool,
+          },
+        ],
+      )
+      expect { subject.run }.not_to raise_error(Exception)
+    end
+
+    it 'should not call LoadBalancersApi.datacenters_loadbalancers_delete when the ID is not valid' do
+      nodepool = k8s_nodepool_mock
+      nic_id = 'invalid_id'
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        cluster_id: 'cluster_id',
+        nodepool_id: 'nodepool_id',
+        lan_id: 'lan_id',
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject.ui).to receive(:error).with("Nodepool ID #{subject_config[:nodepool_id]} not found.")
+
+      expect(subject.api_client).not_to receive(:wait_for)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/k8s/#{subject_config[:cluster_id]}/nodepools/#{subject_config[:nodepool_id]}",
+            operation: :'KubernetesApi.k8s_nodepools_find_by_id',
+            return_type: 'KubernetesNodePool',
+            result: nodepool,
+            exception: Ionoscloud::ApiError.new(code: 404),
+          },
+        ],
+      )
+
+      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_nodepool_lan_remove.rb

@@ -0,0 +1,196 @@
+require 'spec_helper'
+require 'ionoscloud_nodepool_lan_remove'
+
+Chef::Knife::IonoscloudNodepoolLanRemove.load_deps
+
+describe Chef::Knife::IonoscloudNodepoolLanRemove do
+  before :each do
+    subject { Chef::Knife::IonoscloudNodepoolLanRemove.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call KubernetesApi.k8s_nodepools_put to remove the lan when the ID is valid and the lan exists on the nodepool' do
+      nodepool = k8s_nodepool_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        cluster_id: 'cluster_id',
+        nodepool_id: nodepool.id,
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+      subject.name_args = [nodepool.properties.lans.first.id]
+
+      auto_scaling = "Min node count: #{nodepool.properties.auto_scaling.min_node_count}, Max node count:#{nodepool.properties.auto_scaling.max_node_count}"
+      maintenance_window = "#{nodepool.properties.maintenance_window.day_of_the_week}, #{nodepool.properties.maintenance_window.time}"
+      lans = [nodepool.properties.lans[1]].map { |lan| lan.to_hash }
+
+      expect(subject).to receive(:puts).with("ID: #{nodepool.id}")
+      expect(subject).to receive(:puts).with("Name: #{nodepool.properties.name}")
+      expect(subject).to receive(:puts).with("K8s Version: #{nodepool.properties.k8s_version}")
+      expect(subject).to receive(:puts).with("Datacenter ID: #{nodepool.properties.datacenter_id}")
+      expect(subject).to receive(:puts).with("Node Count: #{nodepool.properties.node_count}")
+      expect(subject).to receive(:puts).with("CPU Family: #{nodepool.properties.cpu_family}")
+      expect(subject).to receive(:puts).with("Cores Count: #{nodepool.properties.cores_count}")
+      expect(subject).to receive(:puts).with("RAM: #{nodepool.properties.ram_size}")
+      expect(subject).to receive(:puts).with("Storage Type: #{nodepool.properties.storage_type}")
+      expect(subject).to receive(:puts).with("Storage Size: #{nodepool.properties.storage_size}")
+      expect(subject).to receive(:puts).with("Public IPs: #{nodepool.properties.public_ips}")
+      expect(subject).to receive(:puts).with("LANs: #{lans}")
+      expect(subject).to receive(:puts).with("Availability Zone: #{nodepool.properties.availability_zone}")
+      expect(subject).to receive(:puts).with("Auto Scaling: #{auto_scaling}")
+      expect(subject).to receive(:puts).with("Maintenance Window: #{maintenance_window}")
+      expect(subject).to receive(:puts).with("State: #{nodepool.metadata.state}")
+      expect(subject.ui).to receive(:info).with("Removing Lans #{subject.name_args} from the Nodepoool.")
+
+      expected_body = nodepool.properties.to_hash
+      expected_body[:lans] = [expected_body[:lans][1]]
+      expected_body.delete(:name)
+      expected_body.delete(:datacenterId)
+      expected_body.delete(:ramSize)
+      expected_body.delete(:cpuFamily)
+      expected_body.delete(:coresCount)
+      expected_body.delete(:storageSize)
+      expected_body.delete(:storageType)
+      expected_body.delete(:availabilityZone)
+      expected_body.delete(:availableUpgradeVersions)
+      expected_body.delete(:labels)
+      expected_body.delete(:annotations)
+
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/k8s/#{subject_config[:cluster_id]}/nodepools/#{subject_config[:nodepool_id]}",
+            operation: :'KubernetesApi.k8s_nodepools_find_by_id',
+            return_type: 'KubernetesNodePool',
+            result: nodepool,
+          },
+          {
+            method: 'PUT',
+            path: "/k8s/#{subject_config[:cluster_id]}/nodepools/#{subject_config[:nodepool_id]}",
+            operation: :'KubernetesApi.k8s_nodepools_put',
+            body: { properties: expected_body },
+            return_type: 'KubernetesNodePool',
+            result: nodepool,
+          },
+        ],
+      )
+
+      expect { subject.run }.not_to raise_error(Exception)
+    end
+
+    it 'should call KubernetesApi.k8s_nodepools_put to remove the lan when the ID is valid but the lan does not exist on the nodepool' do
+      nodepool = k8s_nodepool_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        cluster_id: 'cluster_id',
+        nodepool_id: nodepool.id,
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+      subject.name_args = [123]
+
+      auto_scaling = "Min node count: #{nodepool.properties.auto_scaling.min_node_count}, Max node count:#{nodepool.properties.auto_scaling.max_node_count}"
+      maintenance_window = "#{nodepool.properties.maintenance_window.day_of_the_week}, #{nodepool.properties.maintenance_window.time}"
+
+      expect(subject.ui).to receive(:info).with("None of the specified Lan IDs (#{subject.name_args}) are present on the Nodepoool.")
+      expect(subject).to receive(:puts).with("ID: #{nodepool.id}")
+      expect(subject).to receive(:puts).with("Name: #{nodepool.properties.name}")
+      expect(subject).to receive(:puts).with("K8s Version: #{nodepool.properties.k8s_version}")
+      expect(subject).to receive(:puts).with("Node Count: #{nodepool.properties.node_count}")
+      expect(subject).to receive(:puts).with("Lans: #{nodepool.properties.lans.map do
+        |lan|
+        {
+          id: lan.id,
+          dhcp: lan.dhcp,
+          routes: lan.routes ? lan.routes.map do
+            |route|
+            {
+              network: route.network,
+              gateway_ip: route.gateway_ip,
+            }
+          end : []
+        }
+      end}")
+      expect(subject).to receive(:puts).with("Auto Scaling: #{auto_scaling}")
+      expect(subject).to receive(:puts).with("Maintenance Window: #{maintenance_window}")
+      expect(subject).to receive(:puts).with("State: #{nodepool.metadata.state}")
+
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/k8s/#{subject_config[:cluster_id]}/nodepools/#{subject_config[:nodepool_id]}",
+            operation: :'KubernetesApi.k8s_nodepools_find_by_id',
+            return_type: 'KubernetesNodePool',
+            result: nodepool,
+          },
+        ],
+      )
+
+      expect { subject.run }.not_to raise_error(Exception)
+    end
+
+    it 'should not call LoadBalancersApi.datacenters_loadbalancers_delete when the ID is not valid' do
+      nodepool = k8s_nodepool_mock
+      nic_id = 'invalid_id'
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        cluster_id: 'cluster_id',
+        nodepool_id: 'nodepool_id',
+        lan_id: 'lan_id',
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject.ui).to receive(:error).with("Nodepool ID #{subject_config[:nodepool_id]} not found.")
+
+      expect(subject.api_client).not_to receive(:wait_for)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/k8s/#{subject_config[:cluster_id]}/nodepools/#{subject_config[:nodepool_id]}",
+            operation: :'KubernetesApi.k8s_nodepools_find_by_id',
+            return_type: 'KubernetesNodePool',
+            result: nodepool,
+            exception: Ionoscloud::ApiError.new(code: 404),
+          },
+        ],
+      )
+
+      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_nodepool_list_spec.rb

@@ -28,16 +28,18 @@ describe Chef::Knife::IonoscloudNodepoolList do
         subject.ui.color('K8s Version', :bold),
         subject.ui.color('Datacenter ID', :bold),
         subject.ui.color('Node Count', :bold),
+        subject.ui.color('Lan Count', :bold),
         subject.ui.color('State', :bold),
         k8s_nodepools.items.first.id,
         k8s_nodepools.items.first.properties.name,
         k8s_nodepools.items.first.properties.k8s_version,
         k8s_nodepools.items.first.properties.datacenter_id,
         k8s_nodepools.items.first.properties.node_count.to_s,
+        k8s_nodepools.items.first.properties.lans.length,
         k8s_nodepools.items.first.metadata.state,
       ]
 
-      expect(subject.ui).to receive(:list).with(nodepool_list, :uneven_columns_across, 6)
+      expect(subject.ui).to receive(:list).with(nodepool_list, :uneven_columns_across, 7)
 
       mock_call_api(
         subject,

data/spec/chef/knife/ionoscloud_nodepool_update_spec.rb

@@ -22,6 +22,7 @@ describe Chef::Knife::IonoscloudNodepoolUpdate do
         k8s_version: '18.18.18',
         node_count: nodepool.properties.node_count + 1,
         public_ips: '1.1.1.1,2.2.2.2,3.3.3.3',
+        labels: { "new_test": "new_test", "new_test2": "new_test2" },
         lans: '13',
         maintenance_day: 'Tuesday',
         maintenance_time: '03:48:30Z',
@@ -46,6 +47,8 @@ describe Chef::Knife::IonoscloudNodepoolUpdate do
       expect(subject).to receive(:puts).with("Storage Type: #{nodepool.properties.storage_type}")
       expect(subject).to receive(:puts).with("Storage Size: #{nodepool.properties.storage_size}")
       expect(subject).to receive(:puts).with("Public IPs: #{subject_config[:public_ips].split(',')}")
+      expect(subject).to receive(:puts).with("Labels: #{subject_config[:labels]}")
+      expect(subject).to receive(:puts).with("Annotations: #{nodepool.properties.annotations}")
       expect(subject).to receive(:puts).with("LANs: #{lans}")
       expect(subject).to receive(:puts).with("Availability Zone: #{nodepool.properties.availability_zone}")
       expect(subject).to receive(:puts).with("Auto Scaling: #{auto_scaling}")
@@ -55,6 +58,7 @@ describe Chef::Knife::IonoscloudNodepoolUpdate do
       nodepool.properties.k8s_version = subject_config[:k8s_version]
       nodepool.properties.node_count = subject_config[:node_count]
       nodepool.properties.public_ips = subject_config[:public_ips].split(',')
+      nodepool.properties.labels = subject_config[:labels]
       nodepool.properties.lans = subject_config[:lans].split(',').map { |lan| Ionoscloud::KubernetesNodePoolLan.new(id: lan) }
       nodepool.properties.maintenance_window.day_of_the_week = subject_config[:maintenance_day]
       nodepool.properties.maintenance_window.time = subject_config[:maintenance_time]
@@ -81,7 +85,7 @@ describe Chef::Knife::IonoscloudNodepoolUpdate do
                 k8sVersion: subject_config[:k8s_version],
                 nodeCount: subject_config[:node_count],
                 publicIps: subject_config[:public_ips].split(','),
-                labels: nodepool.properties.labels,
+                labels: subject_config[:labels],
                 annotations: nodepool.properties.annotations,
                 lans: subject_config[:lans].split(',').map { |lan| { id: Integer(lan) } },
                 maintenanceWindow: {

data/spec/chef/knife/ionoscloud_pcc_create_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudPccCreate do
   end
 
   describe '#run' do
-    it 'should call PrivateCrossConnectApi.pccs_post with the expected arguments and output based on what it receives' do
+    it 'should call PrivateCrossConnectsApi.pccs_post with the expected arguments and output based on what it receives' do
       pcc = pcc_mock
       datacenter_ids = pcc.properties.connectable_datacenters.map { |datacenter| datacenter.id }
       lan_ids = pcc.properties.peers.map { |peer| peer.id }
@@ -27,7 +27,7 @@ describe Chef::Knife::IonoscloudPccCreate do
       subject_config.each { |key, value| subject.config[key] = value }
 
       peers = pcc.properties.peers.map { |peer| peer.id }
-      datacenters = pcc.properties.connectable_datacenters.map { |datacenter| datacenter.id }
+      datacenters = pcc.properties.connectable_datacenters.map { |pcc| pcc.id }
 
       expect(subject).to receive(:puts).with("ID: #{pcc.id}")
       expect(subject).to receive(:puts).with("Name: #{pcc.properties.name}")
@@ -42,7 +42,7 @@ describe Chef::Knife::IonoscloudPccCreate do
           {
             method: 'POST',
             path: '/pccs',
-            operation: :'PrivateCrossConnectApi.pccs_post',
+            operation: :'PrivateCrossConnectsApi.pccs_post',
             return_type: 'PrivateCrossConnect',
             body: { properties: { name: pcc.properties.name, description: pcc.properties.description } },
             result: pcc,
@@ -50,14 +50,14 @@ describe Chef::Knife::IonoscloudPccCreate do
           {
             method: 'GET',
             path: "/pccs/#{pcc.id}",
-            operation: :'PrivateCrossConnectApi.pccs_find_by_id',
+            operation: :'PrivateCrossConnectsApi.pccs_find_by_id',
             return_type: 'PrivateCrossConnect',
             result: pcc,
           },
           {
             method: 'PATCH',
             path: "/datacenters/#{pcc.properties.connectable_datacenters[0].id}/lans/#{pcc.properties.peers[0].id}",
-            operation: :'LanApi.datacenters_lans_patch',
+            operation: :'LANsApi.datacenters_lans_patch',
             return_type: 'Lan',
             body: { pcc: pcc.id },
             result: pcc.properties.peers[0],
@@ -65,7 +65,7 @@ describe Chef::Knife::IonoscloudPccCreate do
           {
             method: 'PATCH',
             path: "/datacenters/#{pcc.properties.connectable_datacenters[1].id}/lans/#{pcc.properties.peers[1].id}",
-            operation: :'LanApi.datacenters_lans_patch',
+            operation: :'LANsApi.datacenters_lans_patch',
             return_type: 'Lan',
             body: { pcc: pcc.id },
             result: pcc.properties.peers[1],
@@ -73,7 +73,7 @@ describe Chef::Knife::IonoscloudPccCreate do
           {
             method: 'GET',
             path: "/pccs/#{pcc.id}",
-            operation: :'PrivateCrossConnectApi.pccs_find_by_id',
+            operation: :'PrivateCrossConnectsApi.pccs_find_by_id',
             return_type: 'PrivateCrossConnect',
             result: pcc,
           },

data/spec/chef/knife/ionoscloud_pcc_delete_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudPccDelete do
   end
 
   describe '#run' do
-    it 'should call PrivateCrossConnectApi.pccs_delete when the ID is valid' do
+    it 'should call PrivateCrossConnectsApi.pccs_delete when the ID is valid' do
       pcc = pcc_mock
       subject_config = {
         ionoscloud_username: 'email',
@@ -24,7 +24,7 @@ describe Chef::Knife::IonoscloudPccDelete do
       subject.name_args = [pcc.id]
 
       peers = pcc.properties.peers.map { |peer| peer.id }
-      datacenters = pcc.properties.connectable_datacenters.map { |datacenter| datacenter.id }
+      datacenters = pcc.properties.connectable_datacenters.map { |pcc| pcc.id }
 
       expect(subject).to receive(:puts).with("ID: #{pcc.id}")
       expect(subject).to receive(:puts).with("Name: #{pcc.properties.name}")
@@ -41,14 +41,14 @@ describe Chef::Knife::IonoscloudPccDelete do
           {
             method: 'GET',
             path: "/pccs/#{pcc.id}",
-            operation: :'PrivateCrossConnectApi.pccs_find_by_id',
+            operation: :'PrivateCrossConnectsApi.pccs_find_by_id',
             return_type: 'PrivateCrossConnect',
             result: pcc,
           },
           {
             method: 'DELETE',
             path: "/pccs/#{pcc.id}",
-            operation: :'PrivateCrossConnectApi.pccs_delete',
+            operation: :'PrivateCrossConnectsApi.pccs_delete',
           },
         ],
       )
@@ -56,7 +56,7 @@ describe Chef::Knife::IonoscloudPccDelete do
       expect { subject.run }.not_to raise_error(Exception)
     end
 
-    it 'should not call PrivateCrossConnectApi.pccs_delete when the ID is not valid' do
+    it 'should not call PrivateCrossConnectsApi.pccs_delete when the ID is not valid' do
       pcc_id = 'invalid_id'
       subject_config = {
         ionoscloud_username: 'email',
@@ -75,7 +75,7 @@ describe Chef::Knife::IonoscloudPccDelete do
           {
             method: 'GET',
             path: "/pccs/#{pcc_id}",
-            operation: :'PrivateCrossConnectApi.pccs_find_by_id',
+            operation: :'PrivateCrossConnectsApi.pccs_find_by_id',
             return_type: 'PrivateCrossConnect',
             exception: Ionoscloud::ApiError.new(code: 404),
           },

data/spec/chef/knife/ionoscloud_pcc_get_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudPccGet do
   end
 
   describe '#run' do
-    it 'should call PrivateCrossConnectApi.pccs_find_by_id' do
+    it 'should call PrivateCrossConnectsApi.pccs_find_by_id' do
       pcc = pcc_mock
       subject_config = {
         ionoscloud_username: 'email',
@@ -40,7 +40,7 @@ describe Chef::Knife::IonoscloudPccGet do
           {
             method: 'GET',
             path: "/pccs/#{pcc.id}",
-            operation: :'PrivateCrossConnectApi.pccs_find_by_id',
+            operation: :'PrivateCrossConnectsApi.pccs_find_by_id',
             return_type: 'PrivateCrossConnect',
             result: pcc,
           },

data/spec/chef/knife/ionoscloud_pcc_list_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudPccList do
   end
 
   describe '#run' do
-    it 'should call PrivateCrossConnectApi.pccs_get' do
+    it 'should call PrivateCrossConnectsApi.pccs_get' do
       pccs = pccs_mock
       subject_config = {
         ionoscloud_username: 'email',
@@ -38,7 +38,7 @@ describe Chef::Knife::IonoscloudPccList do
           {
             method: 'GET',
             path: '/pccs',
-            operation: :'PrivateCrossConnectApi.pccs_get',
+            operation: :'PrivateCrossConnectsApi.pccs_get',
             return_type: 'PrivateCrossConnects',
             result: pccs,
           },

data/spec/chef/knife/ionoscloud_pcc_update_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudPccUpdate do
   end
 
   describe '#run' do
-    it 'should call PrivateCrossConnectApi.pccs_patch' do
+    it 'should call PrivateCrossConnectsApi.pccs_patch' do
       pcc = pcc_mock
       subject_config = {
         ionoscloud_username: 'email',
@@ -44,7 +44,7 @@ describe Chef::Knife::IonoscloudPccUpdate do
           {
             method: 'PATCH',
             path: "/pccs/#{pcc.id}",
-            operation: :'PrivateCrossConnectApi.pccs_patch',
+            operation: :'PrivateCrossConnectsApi.pccs_patch',
             return_type: 'PrivateCrossConnect',
             body: { name: subject_config[:name], description: subject_config[:description] },
             result: pcc,
@@ -52,7 +52,7 @@ describe Chef::Knife::IonoscloudPccUpdate do
           {
             method: 'GET',
             path: "/pccs/#{pcc.id}",
-            operation: :'PrivateCrossConnectApi.pccs_find_by_id',
+            operation: :'PrivateCrossConnectsApi.pccs_find_by_id',
             return_type: 'PrivateCrossConnect',
             result: pcc,
           },

data/spec/chef/knife/ionoscloud_request_get_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudRequestGet do
   end
 
   describe '#run' do
-    it 'should call RequestApi.requests_find_by_id' do
+    it 'should call RequestsApi.requests_find_by_id' do
       request = request_mock
       subject_config = {
         ionoscloud_username: 'email',
@@ -38,7 +38,7 @@ describe Chef::Knife::IonoscloudRequestGet do
           {
             method: 'GET',
             path: "/requests/#{request.id}",
-            operation: :'RequestApi.requests_find_by_id',
+            operation: :'RequestsApi.requests_find_by_id',
             return_type: 'Request',
             result: request,
           },