knife-ionoscloud 5.1.0 → 6.0.0.alpha.1

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',
@@ -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_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_request_list_spec.rb

@@ -28,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,7 +44,7 @@ describe Chef::Knife::IonoscloudRequestList do
           {
             method: 'GET',
             path: '/requests',
-            operation: :'RequestApi.requests_get',
+            operation: :'RequestsApi.requests_get',
             options: { depth: 2, limit: 20, offset: 0 },
             return_type: 'Requests',
             result: @requests,
@@ -55,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,7 +73,7 @@ describe Chef::Knife::IonoscloudRequestList do
           {
             method: 'GET',
             path: '/requests',
-            operation: :'RequestApi.requests_get',
+            operation: :'RequestsApi.requests_get',
             options: { depth: 2, limit: subject_config[:limit], offset: subject_config[:offset] },
             return_type: 'Requests',
             result: @requests,
@@ -84,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,7 +103,7 @@ describe Chef::Knife::IonoscloudRequestList do
           {
             method: 'GET',
             path: '/requests',
-            operation: :'RequestApi.requests_get',
+            operation: :'RequestsApi.requests_get',
             options: { depth: 2, limit: 20, offset: subject_config[:offset] },
             return_type: 'Requests',
             result: @requests,
@@ -114,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,7 +133,7 @@ describe Chef::Knife::IonoscloudRequestList do
           {
             method: 'GET',
             path: '/requests',
-            operation: :'RequestApi.requests_get',
+            operation: :'RequestsApi.requests_get',
             options: { depth: 2, limit: subject_config[:limit], offset: 0 },
             return_type: 'Requests',
             result: @requests,
@@ -144,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,7 +162,7 @@ describe Chef::Knife::IonoscloudRequestList do
           {
             method: 'GET',
             path: '/requests',
-            operation: :'RequestApi.requests_get',
+            operation: :'RequestsApi.requests_get',
             options: { depth: 2, limit: subject_config[:limit], offset: 0, filter_request_status: subject_config[:status] },
             return_type: 'Requests',
             result: @requests,
@@ -173,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,7 +192,7 @@ describe Chef::Knife::IonoscloudRequestList do
           {
             method: 'GET',
             path: '/requests',
-            operation: :'RequestApi.requests_get',
+            operation: :'RequestsApi.requests_get',
             options: { depth: 2, limit: subject_config[:limit], offset: 0 },
             return_type: 'Requests',
             result: @requests,
@@ -204,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,7 +222,7 @@ describe Chef::Knife::IonoscloudRequestList do
           {
             method: 'GET',
             path: '/requests',
-            operation: :'RequestApi.requests_get',
+            operation: :'RequestsApi.requests_get',
             options: { depth: 2, limit: subject_config[:limit], offset: 0, filter_method: subject_config[:method] },
             return_type: 'Requests',
             result: @requests,
@@ -233,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,7 +252,7 @@ describe Chef::Knife::IonoscloudRequestList do
           {
             method: 'GET',
             path: '/requests',
-            operation: :'RequestApi.requests_get',
+            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_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',
@@ -45,7 +45,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 +53,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',
@@ -42,14 +42,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 +57,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 +77,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),
           },