knife-ionoscloud 5.1.0 → 6.0.0.alpha.1

data/spec/chef/knife/ionoscloud_server_upgrade_spec.rb

@@ -0,0 +1,96 @@
+require 'spec_helper'
+require 'ionoscloud_server_upgrade'
+
+Chef::Knife::IonoscloudServerUpgrade.load_deps
+
+describe Chef::Knife::IonoscloudServerUpgrade do
+  before :each do
+    subject { Chef::Knife::IonoscloudServerUpgrade.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should output success when the ID is valid' do
+      server = server_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: 'datacenter_id',
+        server_id: server.id,
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+      subject.name_args = [server.id]
+
+      expect(subject.ui).to receive(:info).with("Server #{server.id} is being upgraded. Request ID: ")
+
+      expect(subject.api_client).not_to receive(:wait_for)
+      expect(subject).to receive(:get_request_id).once
+
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'POST',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/servers/#{server.id}/upgrade",
+            operation: :'ServersApi.datacenters_servers_upgrade_post',
+            result: server,
+          },
+        ],
+      )
+
+      expect { subject.run }.not_to raise_error(Exception)
+    end
+
+    it 'should output failure when the server ID is not valid' do
+      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 }
+      subject.name_args = [server_id]
+
+      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: 'POST',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/servers/#{server_id}/upgrade",
+            operation: :'ServersApi.datacenters_servers_upgrade_post',
+            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)
+
+      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_snapshot_create_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudSnapshotCreate do
   end
 
   describe '#run' do
-    it 'should call VolumeApi.datacenters_volumes_create_snapshot_post with the expected arguments and output based on what it receives' do
+    it 'should call VolumesApi.datacenters_volumes_create_snapshot_post with the expected arguments and output based on what it receives' do
       snapshot = snapshot_mock
       subject_config = {
         ionoscloud_username: 'email',
@@ -40,7 +40,7 @@ describe Chef::Knife::IonoscloudSnapshotCreate do
           {
             method: 'POST',
             path: "/datacenters/#{subject_config[:datacenter_id]}/volumes/#{subject_config[:volume_id]}/create-snapshot",
-            operation: :'VolumeApi.datacenters_volumes_create_snapshot_post',
+            operation: :'VolumesApi.datacenters_volumes_create_snapshot_post',
             return_type: 'Snapshot',
             result: snapshot,
             form_params: {
@@ -53,7 +53,7 @@ describe Chef::Knife::IonoscloudSnapshotCreate do
           {
             method: 'GET',
             path: "/snapshots/#{snapshot.id}",
-            operation: :'SnapshotApi.snapshots_find_by_id',
+            operation: :'SnapshotsApi.snapshots_find_by_id',
             return_type: 'Snapshot',
             result: snapshot,
           },

data/spec/chef/knife/ionoscloud_snapshot_delete_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudSnapshotDelete do
   end
 
   describe '#run' do
-    it 'should call SnapshotApi.snapshots_delete when the ID is valid' do
+    it 'should call SnapshotsApi.snapshots_delete when the ID is valid' do
       snapshot = snapshot_mock
       subject_config = {
         ionoscloud_username: 'email',
@@ -38,14 +38,14 @@ describe Chef::Knife::IonoscloudSnapshotDelete do
           {
             method: 'GET',
             path: "/snapshots/#{snapshot.id}",
-            operation: :'SnapshotApi.snapshots_find_by_id',
+            operation: :'SnapshotsApi.snapshots_find_by_id',
             return_type: 'Snapshot',
             result: snapshot,
           },
           {
             method: 'DELETE',
             path: "/snapshots/#{snapshot.id}",
-            operation: :'SnapshotApi.snapshots_delete',
+            operation: :'SnapshotsApi.snapshots_delete',
           },
         ],
       )
@@ -53,7 +53,7 @@ describe Chef::Knife::IonoscloudSnapshotDelete do
       expect { subject.run }.not_to raise_error(Exception)
     end
 
-    it 'should not call SnapshotApi.snapshots_delete when the user ID is not valid' do
+    it 'should not call SnapshotsApi.snapshots_delete when the user ID is not valid' do
       snapshot_id = 'invalid_id'
       subject_config = {
         ionoscloud_username: 'email',
@@ -72,7 +72,7 @@ describe Chef::Knife::IonoscloudSnapshotDelete do
           {
             method: 'GET',
             path: "/snapshots/#{snapshot_id}",
-            operation: :'SnapshotApi.snapshots_find_by_id',
+            operation: :'SnapshotsApi.snapshots_find_by_id',
             return_type: 'Snapshot',
             exception: Ionoscloud::ApiError.new(code: 404),
           },

data/spec/chef/knife/ionoscloud_snapshot_list_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudSnapshotList do
   end
 
   describe '#run' do
-    it 'should call SnapshotApi.snapshots_get' do
+    it 'should call SnapshotsApi.snapshots_get' do
       snapshots = snapshots_mock
       subject_config = {
         ionoscloud_username: 'email',
@@ -42,7 +42,7 @@ describe Chef::Knife::IonoscloudSnapshotList do
           {
             method: 'GET',
             path: '/snapshots',
-            operation: :'SnapshotApi.snapshots_get',
+            operation: :'SnapshotsApi.snapshots_get',
             return_type: 'Snapshots',
             result: snapshots,
           },

data/spec/chef/knife/ionoscloud_snapshot_restore_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudSnapshotRestore do
   end
 
   describe '#run' do
-    it 'should call VolumeApi.datacenters_volumes_create_snapshot_post with the expected arguments and output based on what it receives' do
+    it 'should call VolumesApi.datacenters_volumes_create_snapshot_post with the expected arguments and output based on what it receives' do
       snapshot = snapshot_mock
       volume = volume_mock
       subject_config = {
@@ -41,13 +41,13 @@ describe Chef::Knife::IonoscloudSnapshotRestore do
           {
             method: 'POST',
             path: "/datacenters/#{subject_config[:datacenter_id]}/volumes/#{volume.id}/restore-snapshot",
-            operation: :'VolumeApi.datacenters_volumes_restore_snapshot_post',
+            operation: :'VolumesApi.datacenters_volumes_restore_snapshot_post',
             form_params: { 'snapshotId' => snapshot.id },
           },
           {
             method: 'GET',
             path: "/datacenters/#{subject_config[:datacenter_id]}/volumes/#{volume.id}",
-            operation: :'VolumeApi.datacenters_volumes_find_by_id',
+            operation: :'VolumesApi.datacenters_volumes_find_by_id',
             return_type: 'Volume',
             result: volume,
           },

data/spec/chef/knife/ionoscloud_template_list_spec.rb

@@ -0,0 +1,78 @@
+require 'spec_helper'
+require 'ionoscloud_template_list'
+
+Chef::Knife::IonoscloudTemplateList.load_deps
+
+describe Chef::Knife::IonoscloudTemplateList do
+  before :each do
+    subject { Chef::Knife::IonoscloudTemplateList.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call TemplatesApi.templates_get' do
+      templates = templates_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      template_list = [
+        subject.ui.color('ID', :bold),
+        subject.ui.color('Name', :bold),
+        subject.ui.color('Cores', :bold),
+        subject.ui.color('RAM', :bold),
+        subject.ui.color('Storage Size', :bold),
+        templates.items[0].id,
+        templates.items[0].properties.name,
+        Integer(templates.items[0].properties.cores),
+        Integer(templates.items[0].properties.ram),
+        Integer(templates.items[0].properties.storage_size),
+        templates.items[1].id,
+        templates.items[1].properties.name,
+        Integer(templates.items[1].properties.cores),
+        Integer(templates.items[1].properties.ram),
+        Integer(templates.items[1].properties.storage_size),
+      ]
+
+      expect(subject.ui).to receive(:list).with(template_list, :uneven_columns_across, 5)
+
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: '/templates',
+            operation: :'TemplatesApi.templates_get',
+            return_type: 'Templates',
+            result: templates,
+          },
+        ],
+      )
+
+      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_user_ssourl_spec.rb

@@ -7,7 +7,7 @@ describe Chef::Knife::IonoscloudUserSsourl do
   subject { Chef::Knife::IonoscloudUserSsourl.new }
 
   describe '#run' do
-    it 'should call UserManagementApi.um_users_ssourl_get and output the received url when the user ID is valid' do
+    it 'should call UserS3KeysApi.um_users_ssourl_get and output the received url when the user ID is valid' do
       user = user_mock
       sso_url = sso_url_mock
       subject_config = {
@@ -27,7 +27,7 @@ describe Chef::Knife::IonoscloudUserSsourl do
           {
             method: 'GET',
             path: "/um/users/#{user.id}/s3ssourl",
-            operation: :'UserManagementApi.um_users_s3ssourl_get',
+            operation: :'UserS3KeysApi.um_users_s3ssourl_get',
             return_type: 'S3ObjectStorageSSO',
             result: sso_url,
           },
@@ -59,7 +59,7 @@ describe Chef::Knife::IonoscloudUserSsourl do
           {
             method: 'GET',
             path: "/um/users/#{user_id}/s3ssourl",
-            operation: :'UserManagementApi.um_users_s3ssourl_get',
+            operation: :'UserS3KeysApi.um_users_s3ssourl_get',
             return_type: 'S3ObjectStorageSSO',
             exception: Ionoscloud::ApiError.new(code: 404),
           },

data/spec/chef/knife/ionoscloud_volume_attach_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudVolumeAttach do
   end
 
   describe '#run' do
-    it 'should call ServerApi.datacenters_servers_volumes_post when the ID is valid' do
+    it 'should call ServersApi.datacenters_servers_volumes_post when the ID is valid' do
       volume = volume_mock
       subject_config = {
         ionoscloud_username: 'email',
@@ -35,7 +35,7 @@ describe Chef::Knife::IonoscloudVolumeAttach do
           {
             method: 'POST',
             path: "/datacenters/#{subject_config[:datacenter_id]}/servers/#{subject_config[:server_id]}/volumes",
-            operation: :'ServerApi.datacenters_servers_volumes_post',
+            operation: :'ServersApi.datacenters_servers_volumes_post',
             body: { id: volume.id },
             return_type: 'Volume',
             result: volume,
@@ -46,7 +46,7 @@ describe Chef::Knife::IonoscloudVolumeAttach do
       expect { subject.run }.not_to raise_error(Exception)
     end
 
-    it 'should not call ServerApi.datacenters_servers_volumes_post when the ID is not valid' do
+    it 'should not call ServersApi.datacenters_servers_volumes_post when the ID is not valid' do
       volume_id = 'invalid_id'
       subject_config = {
         ionoscloud_username: 'email',
@@ -67,7 +67,7 @@ describe Chef::Knife::IonoscloudVolumeAttach do
           {
             method: 'POST',
             path: "/datacenters/#{subject_config[:datacenter_id]}/servers/#{subject_config[:server_id]}/volumes",
-            operation: :'ServerApi.datacenters_servers_volumes_post',
+            operation: :'ServersApi.datacenters_servers_volumes_post',
             body: { id: volume_id },
             return_type: 'Volume',
             exception: Ionoscloud::ApiError.new(code: 404),

data/spec/chef/knife/ionoscloud_volume_create_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudVolumeCreate do
   end
 
   describe '#run' do
-    it 'should call VolumeApi.datacenters_volumes_post with the expected arguments and output based on what it receives' do
+    it 'should call VolumesApi.datacenters_volumes_post with the expected arguments and output based on what it receives' do
       volume = volume_mock
       subject_config = {
         ionoscloud_username: 'email',
@@ -23,17 +23,17 @@ describe Chef::Knife::IonoscloudVolumeCreate do
         type: volume.properties.type,
         bus: volume.properties.bus,
         availability_zone: volume.properties.availability_zone,
-        image_alias: 'debian:latest',
         image_password: 'K3tTj8G14a3EgKyNeeiY',
+        image: volume.properties.image,
+        backupunit_id: volume.properties.backupunit_id,
+        user_data: volume.properties.user_data,
       }
 
       subject_config.each { |key, value| subject.config[key] = value }
 
       expected_body = volume.properties.to_hash
-      expected_body.delete(:image)
       expected_body.delete(:licenceType)
 
-      expected_body[:imageAlias] = subject_config[:image_alias]
       expected_body[:imagePassword] = subject_config[:image_password]
 
       expect(subject).to receive(:puts).with("ID: #{volume.id}")
@@ -52,7 +52,7 @@ describe Chef::Knife::IonoscloudVolumeCreate do
           {
             method: 'POST',
             path: "/datacenters/#{subject_config[:datacenter_id]}/volumes",
-            operation: :'VolumeApi.datacenters_volumes_post',
+            operation: :'VolumesApi.datacenters_volumes_post',
             return_type: 'Volume',
             body: { properties: expected_body },
             result: volume,
@@ -60,7 +60,7 @@ describe Chef::Knife::IonoscloudVolumeCreate do
           {
             method: 'GET',
             path: "/datacenters/#{subject_config[:datacenter_id]}/volumes/#{volume.id}",
-            operation: :'VolumeApi.datacenters_volumes_find_by_id',
+            operation: :'VolumesApi.datacenters_volumes_find_by_id',
             return_type: 'Volume',
             result: volume,
           },
@@ -70,58 +70,6 @@ describe Chef::Knife::IonoscloudVolumeCreate do
       expect { subject.run }.not_to raise_error(Exception)
     end
 
-    it 'should not call anything when neither image not image-alias is provided' do
-      volume = volume_mock
-      subject_config = {
-        ionoscloud_username: 'email',
-        ionoscloud_password: 'password',
-        cluster_id: 'cluster_id',
-        datacenter_id: 'datacenter_id',
-        name: volume.properties.name,
-        size: volume.properties.size,
-        type: volume.properties.type,
-        bus: volume.properties.bus,
-        availability_zone: volume.properties.availability_zone,
-        image_password: 'K3tTj8G14a3EgKyNeeiY',
-      }
-
-      subject_config.each { |key, value| subject.config[key] = value }
-      expect(subject.ui).to receive(:error).with('Either \'--image\' or \'--image-alias\' parameter must be provided')
-
-      expect(subject.api_client).not_to receive(:wait_for)
-      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
-    end
-
-    it 'should not call anything when neither image not image-alias is provided' do
-      volume = volume_mock
-      subject_config = {
-        ionoscloud_username: 'email',
-        ionoscloud_password: 'password',
-        cluster_id: 'cluster_id',
-        datacenter_id: 'datacenter_id',
-        name: volume.properties.name,
-        size: volume.properties.size,
-        type: volume.properties.type,
-        bus: volume.properties.bus,
-        availability_zone: volume.properties.availability_zone,
-        image_alias: 'debian:latest',
-      }
-
-      subject_config.each { |key, value| subject.config[key] = value }
-      expect(subject.ui).to receive(:error).with('Either \'--image-password\' or \'--ssh-keys\' parameter must be provided')
-
-      expect(subject.api_client).not_to receive(:wait_for)
-      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
-    end
-
     it 'should not make any call if any required option is missing' do
       required_options = subject.instance_variable_get(:@required_options)