knife-ionoscloud 5.1.2 → 6.0.0

data/spec/chef/knife/ionoscloud_request_list_spec.rb

@@ -7,7 +7,6 @@ describe Chef::Knife::IonoscloudRequestList do
   before :each do
     subject { Chef::Knife::IonoscloudRequestList.new }
 
-    @request_path = '/requests'
     @requests = requests_mock
     @request_list = request_list = [
       subject.ui.color('ID', :bold),
@@ -29,7 +28,7 @@ describe Chef::Knife::IonoscloudRequestList do
   end
 
   describe '#run' do
-    it 'should call RequestApi.resources_get with default options when nothing is set' do
+    it 'should call RequestsApi.resources_get with default options when nothing is set' do
       subject_config = {
         ionoscloud_username: 'email',
         ionoscloud_password: 'password',
@@ -44,8 +43,8 @@ describe Chef::Knife::IonoscloudRequestList do
         [
           {
             method: 'GET',
-            path: @request_path,
-            operation: :'RequestApi.requests_get',
+            path: '/requests',
+            operation: :'RequestsApi.requests_get',
             options: { depth: 2, limit: 20, offset: 0 },
             return_type: 'Requests',
             result: @requests,
@@ -56,7 +55,7 @@ describe Chef::Knife::IonoscloudRequestList do
       expect { subject.run }.not_to raise_error(Exception)
     end
 
-    it 'should call RequestApi.resources_get with set options if present' do
+    it 'should call RequestsApi.resources_get with set options if present' do
       subject_config = {
         ionoscloud_username: 'email',
         ionoscloud_password: 'password',
@@ -73,8 +72,8 @@ describe Chef::Knife::IonoscloudRequestList do
         [
           {
             method: 'GET',
-            path: @request_path,
-            operation: :'RequestApi.requests_get',
+            path: '/requests',
+            operation: :'RequestsApi.requests_get',
             options: { depth: 2, limit: subject_config[:limit], offset: subject_config[:offset] },
             return_type: 'Requests',
             result: @requests,
@@ -85,7 +84,7 @@ describe Chef::Knife::IonoscloudRequestList do
       expect { subject.run }.not_to raise_error(Exception)
     end
 
-    it 'should call RequestApi.resources_get with default limit if it is not an Integer' do
+    it 'should call RequestsApi.resources_get with default limit if it is not an Integer' do
       subject_config = {
         ionoscloud_username: 'email',
         ionoscloud_password: 'password',
@@ -103,8 +102,8 @@ describe Chef::Knife::IonoscloudRequestList do
         [
           {
             method: 'GET',
-            path: @request_path,
-            operation: :'RequestApi.requests_get',
+            path: '/requests',
+            operation: :'RequestsApi.requests_get',
             options: { depth: 2, limit: 20, offset: subject_config[:offset] },
             return_type: 'Requests',
             result: @requests,
@@ -115,7 +114,7 @@ describe Chef::Knife::IonoscloudRequestList do
       expect { subject.run }.not_to raise_error(Exception)
     end
 
-    it 'should call RequestApi.resources_get with default offset if it is not an Integers' do
+    it 'should call RequestsApi.resources_get with default offset if it is not an Integers' do
       subject_config = {
         ionoscloud_username: 'email',
         ionoscloud_password: 'password',
@@ -133,8 +132,8 @@ describe Chef::Knife::IonoscloudRequestList do
         [
           {
             method: 'GET',
-            path: @request_path,
-            operation: :'RequestApi.requests_get',
+            path: '/requests',
+            operation: :'RequestsApi.requests_get',
             options: { depth: 2, limit: subject_config[:limit], offset: 0 },
             return_type: 'Requests',
             result: @requests,
@@ -145,7 +144,7 @@ describe Chef::Knife::IonoscloudRequestList do
       expect { subject.run }.not_to raise_error(Exception)
     end
 
-    it 'should call RequestApi.resources_get with the expected status when set' do
+    it 'should call RequestsApi.resources_get with the expected status when set' do
       subject_config = {
         ionoscloud_username: 'email',
         ionoscloud_password: 'password',
@@ -162,8 +161,8 @@ describe Chef::Knife::IonoscloudRequestList do
         [
           {
             method: 'GET',
-            path: @request_path,
-            operation: :'RequestApi.requests_get',
+            path: '/requests',
+            operation: :'RequestsApi.requests_get',
             options: { depth: 2, limit: subject_config[:limit], offset: 0, filter_request_status: subject_config[:status] },
             return_type: 'Requests',
             result: @requests,
@@ -174,7 +173,7 @@ describe Chef::Knife::IonoscloudRequestList do
       expect { subject.run }.not_to raise_error(Exception)
     end
 
-    it 'should call RequestApi.resources_get with no status when set wrong' do
+    it 'should call RequestsApi.resources_get with no status when set wrong' do
       subject_config = {
         ionoscloud_username: 'email',
         ionoscloud_password: 'password',
@@ -192,8 +191,8 @@ describe Chef::Knife::IonoscloudRequestList do
         [
           {
             method: 'GET',
-            path: @request_path,
-            operation: :'RequestApi.requests_get',
+            path: '/requests',
+            operation: :'RequestsApi.requests_get',
             options: { depth: 2, limit: subject_config[:limit], offset: 0 },
             return_type: 'Requests',
             result: @requests,
@@ -205,7 +204,7 @@ describe Chef::Knife::IonoscloudRequestList do
     end
 
 
-    it 'should call RequestApi.resources_get with the expected method when set' do
+    it 'should call RequestsApi.resources_get with the expected method when set' do
       subject_config = {
         ionoscloud_username: 'email',
         ionoscloud_password: 'password',
@@ -222,8 +221,8 @@ describe Chef::Knife::IonoscloudRequestList do
         [
           {
             method: 'GET',
-            path: @request_path,
-            operation: :'RequestApi.requests_get',
+            path: '/requests',
+            operation: :'RequestsApi.requests_get',
             options: { depth: 2, limit: subject_config[:limit], offset: 0, filter_method: subject_config[:method] },
             return_type: 'Requests',
             result: @requests,
@@ -234,7 +233,7 @@ describe Chef::Knife::IonoscloudRequestList do
       expect { subject.run }.not_to raise_error(Exception)
     end
 
-    it 'should call RequestApi.resources_get with no method when set wrong' do
+    it 'should call RequestsApi.resources_get with no method when set wrong' do
       subject_config = {
         ionoscloud_username: 'email',
         ionoscloud_password: 'password',
@@ -252,8 +251,8 @@ describe Chef::Knife::IonoscloudRequestList do
         [
           {
             method: 'GET',
-            path: @request_path,
-            operation: :'RequestApi.requests_get',
+            path: '/requests',
+            operation: :'RequestsApi.requests_get',
             options: { depth: 2, limit: subject_config[:limit], offset: 0 },
             return_type: 'Requests',
             result: @requests,

data/spec/chef/knife/ionoscloud_request_status_spec.rb

@@ -7,7 +7,7 @@ describe Chef::Knife::IonoscloudRequestStatus do
   subject { Chef::Knife::IonoscloudRequestStatus.new }
 
   describe '#run' do
-    it 'should call RequestApi.requests_status_get and output the received status when the request ID is valid' do
+    it 'should call RequestsApi.requests_status_get and output the received status when the request ID is valid' do
       request_status = request_status_mock
       subject_config = {
         ionoscloud_username: 'email',
@@ -26,7 +26,7 @@ describe Chef::Knife::IonoscloudRequestStatus do
           {
             method: 'GET',
             path: "/requests/#{subject_config[:request_id]}/status",
-            operation: :'RequestApi.requests_status_get',
+            operation: :'RequestsApi.requests_status_get',
             return_type: 'RequestStatus',
             result: request_status,
           },
@@ -58,7 +58,7 @@ describe Chef::Knife::IonoscloudRequestStatus do
           {
             method: 'GET',
             path: "/requests/#{request_id}/status",
-            operation: :'RequestApi.requests_status_get',
+            operation: :'RequestsApi.requests_status_get',
             return_type: 'RequestStatus',
             exception: Ionoscloud::ApiError.new(code: 404),
           },

data/spec/chef/knife/ionoscloud_s3key_create_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudS3keyCreate do
   end
 
   describe '#run' do
-    it 'should call UserManagementApi.um_users_s3keys_post with the expected arguments and output based on what it receives' do
+    it 'should call UserS3KeysApi.um_users_s3keys_post with the expected arguments and output based on what it receives' do
       s3_key = s3_key_mock
       subject_config = {
         ionoscloud_username: 'email',
@@ -33,7 +33,7 @@ describe Chef::Knife::IonoscloudS3keyCreate do
           {
             method: 'POST',
             path: "/um/users/#{subject_config[:user_id]}/s3keys",
-            operation: :'UserManagementApi.um_users_s3keys_post',
+            operation: :'UserS3KeysApi.um_users_s3keys_post',
             return_type: 'S3Key',
             result: s3_key,
           },

data/spec/chef/knife/ionoscloud_s3key_delete_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudS3keyDelete do
   end
 
   describe '#run' do
-    it 'should call UserManagementApi.um_users_s3keys_delete when the ID is valid' do
+    it 'should call UserS3KeysApi.um_users_s3keys_delete when the ID is valid' do
       s3_key = s3_key_mock
       subject_config = {
         ionoscloud_username: 'email',
@@ -37,14 +37,14 @@ describe Chef::Knife::IonoscloudS3keyDelete do
           {
             method: 'GET',
             path: "/um/users/#{subject_config[:user_id]}/s3keys/#{s3_key.id}",
-            operation: :'UserManagementApi.um_users_s3keys_find_by_key_id',
+            operation: :'UserS3KeysApi.um_users_s3keys_find_by_key_id',
             return_type: 'S3Key',
             result: s3_key,
           },
           {
             method: 'DELETE',
             path: "/um/users/#{subject_config[:user_id]}/s3keys/#{s3_key.id}",
-            operation: :'UserManagementApi.um_users_s3keys_delete',
+            operation: :'UserS3KeysApi.um_users_s3keys_delete',
           },
         ],
       )
@@ -52,7 +52,7 @@ describe Chef::Knife::IonoscloudS3keyDelete do
       expect { subject.run }.not_to raise_error(Exception)
     end
 
-    it 'should not call UserManagementApi.um_users_s3keys_delete when the ID is not valid' do
+    it 'should not call UserS3KeysApi.um_users_s3keys_delete when the ID is not valid' do
       s3_key_id = 'invalid_id'
       subject_config = {
         ionoscloud_username: 'email',
@@ -72,7 +72,7 @@ describe Chef::Knife::IonoscloudS3keyDelete do
           {
             method: 'GET',
             path: "/um/users/#{subject_config[:user_id]}/s3keys/#{s3_key_id}",
-            operation: :'UserManagementApi.um_users_s3keys_find_by_key_id',
+            operation: :'UserS3KeysApi.um_users_s3keys_find_by_key_id',
             return_type: 'S3Key',
             exception: Ionoscloud::ApiError.new(code: 404),
           },

data/spec/chef/knife/ionoscloud_s3key_get_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudS3keyGet do
   end
 
   describe '#run' do
-    it 'should call UserManagementApi.um_users_s3keys_find_by_key_id' do
+    it 'should call UserS3KeysApi.um_users_s3keys_find_by_key_id' do
       s3_key = s3_key_mock
       subject_config = {
         ionoscloud_username: 'email',
@@ -35,7 +35,7 @@ describe Chef::Knife::IonoscloudS3keyGet do
           {
             method: 'GET',
             path: "/um/users/#{subject_config[:user_id]}/s3keys/#{s3_key.id}",
-            operation: :'UserManagementApi.um_users_s3keys_find_by_key_id',
+            operation: :'UserS3KeysApi.um_users_s3keys_find_by_key_id',
             return_type: 'S3Key',
             result: s3_key,
           },

data/spec/chef/knife/ionoscloud_s3key_list_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudS3keyList do
   end
 
   describe '#run' do
-    it 'should call UserManagementApi.um_users_s3keys_get' do
+    it 'should call UserS3KeysApi.um_users_s3keys_get' do
       s3_keys = s3_keys_mock
       subject_config = {
         ionoscloud_username: 'email',
@@ -39,7 +39,7 @@ describe Chef::Knife::IonoscloudS3keyList do
           {
             method: 'GET',
             path: "/um/users/#{subject_config[:user_id]}/s3keys",
-            operation: :'UserManagementApi.um_users_s3keys_get',
+            operation: :'UserS3KeysApi.um_users_s3keys_get',
             return_type: 'S3Keys',
             result: s3_keys,
           },

data/spec/chef/knife/ionoscloud_server_console_spec.rb

@@ -0,0 +1,97 @@
+require 'spec_helper'
+require 'ionoscloud_server_console'
+
+Chef::Knife::IonoscloudServerConsole.load_deps
+
+describe Chef::Knife::IonoscloudServerConsole do
+  subject { Chef::Knife::IonoscloudServerConsole.new }
+
+  describe '#run' do
+    it 'should call ServersApi.datacenters_servers_remote_console_get and output the received url when the server ID is valid' do
+      datacenter = datacenter_mock
+      server = server_mock
+      console = console_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: datacenter.id,
+        server_id: server.id,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject).to receive(:puts).with(console.url)
+
+      expect(subject.api_client).not_to receive(:wait_for)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/datacenters/#{datacenter.id}/servers/#{server.id}/remoteconsole",
+            operation: :'ServersApi.datacenters_servers_remote_console_get',
+            return_type: 'RemoteConsoleUrl',
+            result: console,
+          },
+        ],
+      )
+
+      expect { subject.run }.not_to raise_error(Exception)
+    end
+
+    it 'should output an error when the server is not found' do
+      datacenter = datacenter_mock
+      server_id = 'invalid_id'
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: datacenter.id,
+        server_id: server_id,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      allow(subject).to receive(:puts)
+      allow(subject).to receive(:print)
+
+      expect(subject.ui).to receive(:error).with("Server ID #{server_id} not found.")
+
+      expect(subject.api_client).not_to receive(:wait_for)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/datacenters/#{datacenter.id}/servers/#{server_id}/remoteconsole",
+            operation: :'ServersApi.datacenters_servers_remote_console_get',
+            return_type: 'RemoteConsoleUrl',
+            exception: Ionoscloud::ApiError.new(code: 404),
+          },
+        ],
+      )
+
+      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)
+      allow(subject).to receive(:puts)
+      allow(subject).to receive(:print)
+
+      arrays_without_one_element(required_options).each {
+        |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

data/spec/chef/knife/ionoscloud_server_create_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudServerCreate do
   end
 
   describe '#run' do
-    it 'should call ServerApi.datacenters_servers_post with the expected arguments and output based on what it receives' do
+    it 'should call ServersApi.datacenters_servers_post with the expected arguments and output based on what it receives' do
       server = server_mock
       subject_config = {
         ionoscloud_username: 'email',
@@ -31,10 +31,13 @@ describe Chef::Knife::IonoscloudServerCreate do
 
       expect(subject).to receive(:puts).with("ID: #{server.id}")
       expect(subject).to receive(:puts).with("Name: #{server.properties.name}")
+      expect(subject).to receive(:puts).with("Type: #{server.properties.type}")
+      expect(subject).to receive(:puts).with("Template: #{server.properties.template_uuid}")
       expect(subject).to receive(:puts).with("Cores: #{server.properties.cores}")
       expect(subject).to receive(:puts).with("CPU Family: #{server.properties.cpu_family}")
       expect(subject).to receive(:puts).with("Ram: #{server.properties.ram}")
       expect(subject).to receive(:puts).with("Availability Zone: #{server.properties.availability_zone}")
+      expect(subject).to receive(:puts).with("VM State: #{server.properties.vm_state}")
       expect(subject).to receive(:puts).with("Boot Volume: #{server.properties.boot_volume.id}")
       expect(subject).to receive(:puts).with("Boot CDROM: #{server.properties.boot_cdrom.id}")
 
@@ -45,7 +48,7 @@ describe Chef::Knife::IonoscloudServerCreate do
           {
             method: 'POST',
             path: "/datacenters/#{subject_config[:datacenter_id]}/servers",
-            operation: :'ServerApi.datacenters_servers_post',
+            operation: :'ServersApi.datacenters_servers_post',
             return_type: 'Server',
             body: { properties: server.properties.to_hash },
             result: server,
@@ -53,7 +56,7 @@ describe Chef::Knife::IonoscloudServerCreate do
           {
             method: 'GET',
             path: "/datacenters/#{subject_config[:datacenter_id]}/servers/#{server.id}",
-            operation: :'ServerApi.datacenters_servers_find_by_id',
+            operation: :'ServersApi.datacenters_servers_find_by_id',
             return_type: 'Server',
             result: server,
           },

data/spec/chef/knife/ionoscloud_server_delete_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudServerDelete do
   end
 
   describe '#run' do
-    it 'should call ServerApi.datacenters_servers_delete when the ID is valid' do
+    it 'should call ServersApi.datacenters_servers_delete when the ID is valid' do
       server = server_mock
       subject_config = {
         ionoscloud_username: 'email',
@@ -26,10 +26,13 @@ describe Chef::Knife::IonoscloudServerDelete do
 
       expect(subject).to receive(:puts).with("ID: #{server.id}")
       expect(subject).to receive(:puts).with("Name: #{server.properties.name}")
+      expect(subject).to receive(:puts).with("Type: #{server.properties.type}")
+      expect(subject).to receive(:puts).with("Template: #{server.properties.template_uuid}")
       expect(subject).to receive(:puts).with("Cores: #{server.properties.cores}")
       expect(subject).to receive(:puts).with("CPU Family: #{server.properties.cpu_family}")
       expect(subject).to receive(:puts).with("Ram: #{server.properties.ram}")
       expect(subject).to receive(:puts).with("Availability Zone: #{server.properties.availability_zone}")
+      expect(subject).to receive(:puts).with("VM State: #{server.properties.vm_state}")
       expect(subject).to receive(:puts).with("Boot Volume: #{server.properties.boot_volume.id}")
       expect(subject).to receive(:puts).with("Boot CDROM: #{server.properties.boot_cdrom.id}")
       expect(subject.ui).to receive(:warn).with("Deleted Server #{server.id}. Request ID: ")
@@ -42,14 +45,14 @@ describe Chef::Knife::IonoscloudServerDelete do
           {
             method: 'GET',
             path: "/datacenters/#{subject_config[:datacenter_id]}/servers/#{server.id}",
-            operation: :'ServerApi.datacenters_servers_find_by_id',
+            operation: :'ServersApi.datacenters_servers_find_by_id',
             return_type: 'Server',
             result: server,
           },
           {
             method: 'DELETE',
             path: "/datacenters/#{subject_config[:datacenter_id]}/servers/#{server.id}",
-            operation: :'ServerApi.datacenters_servers_delete',
+            operation: :'ServersApi.datacenters_servers_delete',
           },
         ],
       )
@@ -57,7 +60,7 @@ describe Chef::Knife::IonoscloudServerDelete do
       expect { subject.run }.not_to raise_error(Exception)
     end
 
-    it 'should not call ServerApi.datacenters_servers_delete when the user ID is not valid' do
+    it 'should not call ServersApi.datacenters_servers_delete when the server ID is not valid' do
       server_id = 'invalid_id'
       subject_config = {
         ionoscloud_username: 'email',
@@ -77,7 +80,7 @@ describe Chef::Knife::IonoscloudServerDelete do
           {
             method: 'GET',
             path: "/datacenters/#{subject_config[:datacenter_id]}/servers/#{server_id}",
-            operation: :'ServerApi.datacenters_servers_find_by_id',
+            operation: :'ServersApi.datacenters_servers_find_by_id',
             return_type: 'Server',
             exception: Ionoscloud::ApiError.new(code: 404),
           },

data/spec/chef/knife/ionoscloud_server_get_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudServerGet do
   end
 
   describe '#run' do
-    it 'should call ServerApi.datacenters_servers_find_by_id' do
+    it 'should call ServersApi.datacenters_servers_find_by_id' do
       server = server_mock
       subject_config = {
         ionoscloud_username: 'email',
@@ -26,10 +26,13 @@ describe Chef::Knife::IonoscloudServerGet do
 
       expect(subject).to receive(:puts).with("ID: #{server.id}")
       expect(subject).to receive(:puts).with("Name: #{server.properties.name}")
+      expect(subject).to receive(:puts).with("Type: #{server.properties.type}")
+      expect(subject).to receive(:puts).with("Template: #{server.properties.template_uuid}")
       expect(subject).to receive(:puts).with("Cores: #{server.properties.cores}")
       expect(subject).to receive(:puts).with("CPU Family: #{server.properties.cpu_family}")
       expect(subject).to receive(:puts).with("Ram: #{server.properties.ram}")
       expect(subject).to receive(:puts).with("Availability Zone: #{server.properties.availability_zone}")
+      expect(subject).to receive(:puts).with("VM State: #{server.properties.vm_state}")
       expect(subject).to receive(:puts).with("Boot Volume: #{server.properties.boot_volume.id}")
       expect(subject).to receive(:puts).with("Boot CDROM: #{server.properties.boot_cdrom.id}")
 
@@ -40,7 +43,7 @@ describe Chef::Knife::IonoscloudServerGet do
           {
             method: 'GET',
             path: "/datacenters/#{subject_config[:datacenter_id]}/servers/#{subject_config[:server_id]}",
-            operation: :'ServerApi.datacenters_servers_find_by_id',
+            operation: :'ServersApi.datacenters_servers_find_by_id',
             return_type: 'Server',
             result: server,
           },

data/spec/chef/knife/ionoscloud_server_list_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudServerList do
   end
 
   describe '#run' do
-    it 'should call ServerApi.datacenters_servers_get' do
+    it 'should call ServersApi.datacenters_servers_get' do
       servers = servers_mock
       subject_config = {
         ionoscloud_username: 'email',
@@ -25,10 +25,11 @@ describe Chef::Knife::IonoscloudServerList do
       server_list = [
         subject.ui.color('ID', :bold),
         subject.ui.color('Name', :bold),
+        subject.ui.color('Type', :bold),
+        subject.ui.color('Template', :bold),
         subject.ui.color('Cores', :bold),
         subject.ui.color('CPU Family', :bold),
         subject.ui.color('RAM', :bold),
-        subject.ui.color('Availability Zone', :bold),
         subject.ui.color('VM State', :bold),
         subject.ui.color('Boot Volume', :bold),
         subject.ui.color('Boot CDROM', :bold),
@@ -37,16 +38,17 @@ describe Chef::Knife::IonoscloudServerList do
       servers.items.each do |server|
         server_list << server.id
         server_list << server.properties.name
+        server_list << server.properties.type
+        server_list << server.properties.template_uuid
         server_list << server.properties.cores.to_s
         server_list << server.properties.cpu_family
         server_list << server.properties.ram.to_s
-        server_list << server.properties.availability_zone
         server_list << server.properties.vm_state
         server_list << (server.properties.boot_volume.nil? ? '' : server.properties.boot_volume.id)
         server_list << (server.properties.boot_cdrom.nil? ? '' : server.properties.boot_cdrom.id)
       end
 
-      expect(subject.ui).to receive(:list).with(server_list, :uneven_columns_across, 9)
+      expect(subject.ui).to receive(:list).with(server_list, :uneven_columns_across, 10)
 
       mock_call_api(
         subject,
@@ -54,7 +56,63 @@ describe Chef::Knife::IonoscloudServerList do
           {
             method: 'GET',
             path: "/datacenters/#{subject_config[:datacenter_id]}/servers",
-            operation: :'ServerApi.datacenters_servers_get',
+            operation: :'ServersApi.datacenters_servers_get',
+            return_type: 'Servers',
+            result: servers,
+          },
+        ],
+      )
+
+      expect { subject.run }.not_to raise_error(Exception)
+    end
+
+    it 'should call ServersApi.datacenters_servers_get with upgradeNeeded option when it is received' do
+      servers = servers_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: 'datacenter_id',
+        upgrade_needed: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      server_list = [
+        subject.ui.color('ID', :bold),
+        subject.ui.color('Name', :bold),
+        subject.ui.color('Type', :bold),
+        subject.ui.color('Template', :bold),
+        subject.ui.color('Cores', :bold),
+        subject.ui.color('CPU Family', :bold),
+        subject.ui.color('RAM', :bold),
+        subject.ui.color('VM State', :bold),
+        subject.ui.color('Boot Volume', :bold),
+        subject.ui.color('Boot CDROM', :bold),
+      ]
+
+      servers.items.each do |server|
+        server_list << server.id
+        server_list << server.properties.name
+        server_list << server.properties.type
+        server_list << server.properties.template_uuid
+        server_list << server.properties.cores.to_s
+        server_list << server.properties.cpu_family
+        server_list << server.properties.ram.to_s
+        server_list << server.properties.vm_state
+        server_list << (server.properties.boot_volume.nil? ? '' : server.properties.boot_volume.id)
+        server_list << (server.properties.boot_cdrom.nil? ? '' : server.properties.boot_cdrom.id)
+      end
+
+      expect(subject.ui).to receive(:list).with(server_list, :uneven_columns_across, 10)
+
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/servers",
+            options: { upgrade_needed: true },
+            operation: :'ServersApi.datacenters_servers_get',
             return_type: 'Servers',
             result: servers,
           },

data/spec/chef/knife/ionoscloud_server_reboot_spec.rb

@@ -34,8 +34,7 @@ describe Chef::Knife::IonoscloudServerReboot do
           {
             method: 'POST',
             path: "/datacenters/#{subject_config[:datacenter_id]}/servers/#{server.id}/reboot",
-            operation: :'ServerApi.datacenters_servers_reboot_post',
-            return_type: 'Object',
+            operation: :'ServersApi.datacenters_servers_reboot_post',
             result: server,
           },
         ],
@@ -44,7 +43,7 @@ describe Chef::Knife::IonoscloudServerReboot do
       expect { subject.run }.not_to raise_error(Exception)
     end
 
-    it 'should output failure when the user ID is not valid' do
+    it 'should output failure when the server ID is not valid' do
       server_id = 'invalid_id'
       subject_config = {
         ionoscloud_username: 'email',
@@ -64,8 +63,7 @@ describe Chef::Knife::IonoscloudServerReboot do
           {
             method: 'POST',
             path: "/datacenters/#{subject_config[:datacenter_id]}/servers/#{server_id}/reboot",
-            operation: :'ServerApi.datacenters_servers_reboot_post',
-            return_type: 'Object',
+            operation: :'ServersApi.datacenters_servers_reboot_post',
             exception: Ionoscloud::ApiError.new(code: 404),
           },
         ],