knife-ionoscloud 5.0.0.beta.1 → 5.1.2

data/lib/chef/knife/ionoscloud_volume_create.rb

@@ -25,7 +25,8 @@ class Chef
       option :bus,
               short: '-b BUS',
               long: '--bus BUS',
-              description: 'The bus type of the volume (VIRTIO or IDE)'
+              description: 'The bus type of the volume (VIRTIO or IDE)',
+              default: 'VIRTIO'
 
       option :image,
               short: '-N ID',
@@ -59,8 +60,45 @@ class Chef
       option :availability_zone,
               short: '-Z AVAILABILITY_ZONE',
               long: '--availability-zone AVAILABILITY_ZONE',
-              description: 'The volume availability zone of the server',
-              required: false
+              description: 'The volume availability zone of the server'
+
+      option :backupunit_id,
+              short: '-B BACKUPUNIT_ID',
+              long: '--backupunit BACKUPUNIT_ID',
+              description: 'The uuid of the Backup Unit that user has access to. The property is immutable and is only allowed '\
+              'to be set on a new volume creation. It is mandatory to provide either \'public image\' or \'imageAlias\' in '\
+              'conjunction with this property.'
+
+      option :user_data,
+              short: '-u USER_DATA',
+              long: '--user-data USER_DATA',
+              description: 'The cloud-init configuration for the volume as base64 encoded string. The property is '\
+              'immutable and is only allowed to be set on a new volume creation. It is mandatory to provide either \'public image\' '\
+              'or \'imageAlias\' that has cloud-init compatibility in conjunction with this property.'
+
+      option :cpu_hot_plug,
+              long: '--cpu-hot-plug CPU_HOT_PLUG',
+              description: 'The licence type of the volume (LINUX, WINDOWS, UNKNOWN, OTHER)'
+
+      option :ram_hot_plug,
+              long: '--ram-hot-plug RAM_HOT_PLUG',
+              description: 'The licence type of the volume (LINUX, WINDOWS, UNKNOWN, OTHER)'
+
+      option :nic_hot_plug,
+              long: '--nic-hot-plug NIC_HOT_PLUG',
+              description: 'The licence type of the volume (LINUX, WINDOWS, UNKNOWN, OTHER)'
+
+      option :nic_hot_unplug,
+              long: '--nic-hot-unplug NIC_HOT_UNPLUG',
+              description: 'The licence type of the volume (LINUX, WINDOWS, UNKNOWN, OTHER)'
+
+      option :disc_virtio_hot_plug,
+              long: '--disc-virtio-hot_plug DISC_VIRTIO_HOT_PLUG',
+              description: 'The licence type of the volume (LINUX, WINDOWS, UNKNOWN, OTHER)'
+
+      option :disc_virtio_hot_unplug,
+              long: '--disc-virtio-hot_unplug DISC_VIRTIO_HOT_UNPLUG',
+              description: 'The licence type of the volume (LINUX, WINDOWS, UNKNOWN, OTHER)'
 
       attr_reader :description, :required_options
 
@@ -74,21 +112,10 @@ class Chef
 
       def run
         $stdout.sync = true
+        handle_extra_config
         validate_required_params(@required_options, config)
 
-        if !config[:image] && !config[:image_alias]
-          ui.error('Either \'--image\' or \'--image-alias\' parameter must be provided')
-          exit(1)
-        end
-
-        if !config[:ssh_keys] && !config[:image_password]
-          ui.error('Either \'--image-password\' or \'--ssh-keys\' parameter must be provided')
-          exit(1)
-        end
-
-        if config[:ssh_keys]
-          config[:ssh_keys] = config[:ssh_keys].split(',')
-        end
+        config[:ssh_keys] = config[:ssh_keys].split(',') if config[:ssh_keys] && config[:ssh_keys].instance_of?(String)
 
         print "#{ui.color('Creating volume...', :magenta)}"
 
@@ -96,37 +123,34 @@ class Chef
 
         volume, _, headers = volume_api.datacenters_volumes_post_with_http_info(
           config[:datacenter_id],
-          {
-            properties: {
+          Ionoscloud::Volume.new(
+            properties: Ionoscloud::VolumeProperties.new(
               name: config[:name],
               size: config[:size],
-              bus: config[:bus] || 'VIRTIO',
+              bus: config[:bus],
               type: config[:type],
-              licenceType: config[:licence_type],
+              licence_type: config[:licence_type],
               image: config[:image],
-              imageAlias: config[:image_alias],
-              sshKeys: config[:sshKeys],
-              imagePassword: config[:image_password],
-              availabilityZone: config[:availability_zone],
-            }.compact
-          },
+              image_alias: config[:image_alias],
+              ssh_keys: config[:ssh_keys],
+              image_password: config[:image_password],
+              availability_zone: config[:availability_zone],
+              backupunit_id: config[:backupunit_id],
+              user_data: config[:user_data],
+              cpu_hot_plug: (config.key?(:cpu_hot_plug) ? config[:cpu_hot_plug].to_s.downcase == 'true' : nil),
+              ram_hot_plug: (config.key?(:ram_hot_plug) ? config[:ram_hot_plug].to_s.downcase == 'true' : nil),
+              nic_hot_plug: (config.key?(:nic_hot_plug) ? config[:nic_hot_plug].to_s.downcase == 'true' : nil),
+              nic_hot_unplug: (config.key?(:nic_hot_unplug) ? config[:nic_hot_unplug].to_s.downcase == 'true' : nil),
+              disc_virtio_hot_plug: (config.key?(:disc_virtio_hot_plug) ? config[:disc_virtio_hot_plug].to_s.downcase == 'true' : nil),
+              disc_virtio_hot_unplug: (config.key?(:disc_virtio_hot_unplug) ? config[:disc_virtio_hot_unplug].to_s.downcase == 'true' : nil),
+            ),
+          ),
         )
 
         dot = ui.color('.', :magenta)
         api_client.wait_for(300) { print dot; is_done? get_request_id headers }
 
-        volume = volume_api.datacenters_volumes_find_by_id(config[:datacenter_id], volume.id)
-
-        puts "\n"
-        puts "#{ui.color('ID', :cyan)}: #{volume.id}"
-        puts "#{ui.color('Name', :cyan)}: #{volume.properties.name}"
-        puts "#{ui.color('Size', :cyan)}: #{volume.properties.size}"
-        puts "#{ui.color('Bus', :cyan)}: #{volume.properties.bus}"
-        puts "#{ui.color('Image', :cyan)}: #{volume.properties.image}"
-        puts "#{ui.color('Type', :cyan)}: #{volume.properties.type}"
-        puts "#{ui.color('Licence Type', :cyan)}: #{volume.properties.licence_type}"
-        puts "#{ui.color('Zone', :cyan)}: #{volume.properties.availability_zone}"
-        puts 'done'
+        print_volume(volume_api.datacenters_volumes_find_by_id(config[:datacenter_id], volume.id))
       end
     end
   end

data/lib/chef/knife/ionoscloud_volume_delete.rb

@@ -24,6 +24,7 @@ class Chef
 
       def run
         $stdout.sync = true
+        handle_extra_config
         validate_required_params(@required_options, config)
 
         volume_api = Ionoscloud::VolumeApi.new(api_client)
@@ -40,14 +41,7 @@ class Chef
             next
           end
 
-          msg_pair('ID', volume.id)
-          msg_pair('Name', volume.properties.name)
-          msg_pair('Size', volume.properties.size)
-          msg_pair('Bus', volume.properties.bus)
-          msg_pair('Image', volume.properties.image)
-          msg_pair('Type', volume.properties.type)
-          msg_pair('Licence Type', volume.properties.licence_type)
-          msg_pair('Zone', volume.properties.availability_zone)
+          print_volume(volume)
 
           begin
             confirm('Do you really want to delete this volume')

data/lib/chef/knife/ionoscloud_volume_detach.rb

@@ -31,6 +31,7 @@ class Chef
 
       def run
         $stdout.sync = true
+        handle_extra_config
         validate_required_params(@required_options, config)
 
         server_api = Ionoscloud::ServerApi.new(api_client)
@@ -48,14 +49,7 @@ class Chef
             next
           end
 
-          msg_pair('ID', volume.id)
-          msg_pair('Name', volume.properties.name)
-          msg_pair('Size', volume.properties.size)
-          msg_pair('Bus', volume.properties.bus)
-          msg_pair('Image', volume.properties.image)
-          msg_pair('Type', volume.properties.type)
-          msg_pair('Licence Type', volume.properties.licence_type)
-          msg_pair('Zone', volume.properties.availability_zone)
+          print_volume(volume)
 
           begin
             confirm('Do you really want to detach this volume')

data/lib/chef/knife/ionoscloud_volume_get.rb

@@ -0,0 +1,36 @@
+require_relative 'ionoscloud_base'
+
+class Chef
+  class Knife
+    class IonoscloudVolumeGet < Knife
+      include Knife::IonoscloudBase
+
+      banner 'knife ionoscloud volume get (options)'
+
+      option :datacenter_id,
+              short: '-D DATACENTER_ID',
+              long: '--datacenter-id DATACENTER_ID',
+              description: 'ID of the data center'
+
+      option :volume_id,
+              long: '--volume VOLUME_ID',
+              description: 'ID of the volume.'
+
+      attr_reader :description, :required_options
+
+      def initialize(args = [])
+        super(args)
+        @description =
+        'Retrieves the attributes of a given Volume.'
+        @required_options = [:datacenter_id, :volume_id, :ionoscloud_username, :ionoscloud_password]
+      end
+
+      def run
+        $stdout.sync = true
+        handle_extra_config
+        validate_required_params(@required_options, config)
+        print_volume(Ionoscloud::VolumeApi.new(api_client).datacenters_volumes_find_by_id(config[:datacenter_id], config[:volume_id]))
+      end
+    end
+  end
+end

data/lib/chef/knife/ionoscloud_volume_list.rb

@@ -29,6 +29,7 @@ class Chef
 
       def run
         $stdout.sync = true
+        handle_extra_config
         validate_required_params(@required_options, config)
 
         volume_list = [
@@ -39,7 +40,7 @@ class Chef
           ui.color('Image', :bold),
           ui.color('Type', :bold),
           ui.color('Zone', :bold),
-          ui.color('Device Number', :bold)
+          ui.color('Device Number', :bold),
         ]
 
         opts = { depth: 1 }

data/lib/chef/knife/ionoscloud_volume_update.rb

@@ -0,0 +1,106 @@
+require_relative 'ionoscloud_base'
+
+class Chef
+  class Knife
+    class IonoscloudVolumeUpdate < Knife
+      include Knife::IonoscloudBase
+
+      banner 'knife ionoscloud volume update (options)'
+
+      option :datacenter_id,
+              short: '-D DATACENTER_ID',
+              long: '--datacenter-id DATACENTER_ID',
+              description: 'Name of the data center'
+
+      option :volume_id,
+              long: '--volume VOLUME_ID',
+              description: 'ID of the Volume.'
+
+      option :name,
+              short: '-n NAME',
+              long: '--name NAME',
+              description: 'Name of the volume'
+
+      option :size,
+              short: '-S SIZE',
+              long: '--size SIZE',
+              description: 'The size of the volume in GB'
+
+      option :bus,
+              short: '-b BUS',
+              long: '--bus BUS',
+              description: 'The bus type of the volume (VIRTIO or IDE)'
+
+      option :cpu_hot_plug,
+              long: '--cpu-hot-plug CPU_HOT_PLUG',
+              description: 'The licence type of the volume (LINUX, WINDOWS, UNKNOWN, OTHER)'
+
+      option :ram_hot_plug,
+              long: '--ram-hot-plug RAM_HOT_PLUG',
+              description: 'The licence type of the volume (LINUX, WINDOWS, UNKNOWN, OTHER)'
+
+      option :nic_hot_plug,
+              long: '--nic-hot-plug NIC_HOT_PLUG',
+              description: 'The licence type of the volume (LINUX, WINDOWS, UNKNOWN, OTHER)'
+
+      option :nic_hot_unplug,
+              long: '--nic-hot-unplug NIC_HOT_UNPLUG',
+              description: 'The licence type of the volume (LINUX, WINDOWS, UNKNOWN, OTHER)'
+
+      option :disc_virtio_hot_plug,
+              long: '--disc-virtio-hot_plug DISC_VIRTIO_HOT_PLUG',
+              description: 'The licence type of the volume (LINUX, WINDOWS, UNKNOWN, OTHER)'
+
+      option :disc_virtio_hot_unplug,
+              long: '--disc-virtio-hot_unplug DISC_VIRTIO_HOT_UNPLUG',
+              description: 'The licence type of the volume (LINUX, WINDOWS, UNKNOWN, OTHER)'
+
+      attr_reader :description, :required_options
+
+      def initialize(args = [])
+        super(args)
+        @description =
+        'Updates information about a Ionoscloud Volume.'
+        @required_options = [:datacenter_id, :volume_id, :ionoscloud_username, :ionoscloud_password]
+        @updatable_fields = [
+          :name, :size, :bus, :cpu_hot_plug, :ram_hot_plug, :nic_hot_plug, :nic_hot_unplug, :disc_virtio_hot_plug, :disc_virtio_hot_unplug,
+        ]
+      end
+
+      def run
+        $stdout.sync = true
+        handle_extra_config
+        validate_required_params(@required_options, config)
+
+        volume = Ionoscloud::VolumeApi.new(api_client)
+
+        if @updatable_fields.map { |el| config[el] }.any?
+          print "#{ui.color('Updating Volume...', :magenta)}"
+
+          _, _, headers  = volume.datacenters_volumes_patch_with_http_info(
+            config[:datacenter_id],
+            config[:volume_id],
+            Ionoscloud::VolumeProperties.new(
+              name: config[:name],
+              size: config[:size],
+              bus: config[:bus],
+              cpu_hot_plug: (config.key?(:cpu_hot_plug) ? config[:cpu_hot_plug].to_s.downcase == 'true' : nil),
+              ram_hot_plug: (config.key?(:ram_hot_plug) ? config[:ram_hot_plug].to_s.downcase == 'true' : nil),
+              nic_hot_plug: (config.key?(:nic_hot_plug) ? config[:nic_hot_plug].to_s.downcase == 'true' : nil),
+              nic_hot_unplug: (config.key?(:nic_hot_unplug) ? config[:nic_hot_unplug].to_s.downcase == 'true' : nil),
+              disc_virtio_hot_plug: (config.key?(:disc_virtio_hot_plug) ? config[:disc_virtio_hot_plug].to_s.downcase == 'true' : nil),
+              disc_virtio_hot_unplug: (config.key?(:disc_virtio_hot_unplug) ? config[:disc_virtio_hot_unplug].to_s.downcase == 'true' : nil),
+            ),
+          )
+
+          dot = ui.color('.', :magenta)
+          api_client.wait_for { print dot; is_done? get_request_id headers }
+        else
+          ui.warn("Nothing to update, please set one of the attributes #{@updatable_fields}.")
+        end
+
+        print_volume(volume.datacenters_volumes_find_by_id(config[:datacenter_id], config[:volume_id]))
+      end
+    end
+  end
+end

data/lib/knife-ionoscloud/version.rb

@@ -1,6 +1,6 @@
 module Knife
   module Ionoscloud
-    VERSION = '5.0.0.beta.1'
+    VERSION = '5.1.2'
     MAJOR, MINOR, TINY = VERSION.split('.')
   end
 end

data/sonar-project.properties

@@ -0,0 +1,12 @@
+sonar.projectKey=knife-plugin
+sonar.organization=ionos-cloud
+
+# This is the name and version displayed in the SonarCloud UI.
+#sonar.projectName=knife-plugin
+#sonar.projectVersion=1.0
+
+# Path is relative to the sonar-project.properties file. Replace "\" by "/" on Windows.
+#sonar.sources=.
+
+# Encoding of the source code. Default is default system encoding
+#sonar.sourceEncoding=UTF-8

data/spec/chef/knife/ionoscloud_backupunit_get_spec.rb

@@ -0,0 +1,65 @@
+require 'spec_helper'
+require 'ionoscloud_backupunit_get'
+
+Chef::Knife::IonoscloudBackupunitGet.load_deps
+
+describe Chef::Knife::IonoscloudBackupunitGet do
+  before :each do
+    subject { Chef::Knife::IonoscloudBackupunitGet.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call BackupUnitApi.backupunits_find_by_id' do
+      backupunit = backupunit_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        backupunit_id: backupunit.id,
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject).to receive(:puts).with("ID: #{backupunit.id}")
+      expect(subject).to receive(:puts).with("Name: #{backupunit.properties.name}")
+      expect(subject).to receive(:puts).with("Email: #{backupunit.properties.email}")
+
+      expect(subject.api_client).not_to receive(:wait_for)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/backupunits/#{backupunit.id}",
+            operation: :'BackupUnitApi.backupunits_find_by_id',
+            return_type: 'BackupUnit',
+            result: backupunit,
+          },
+        ],
+      )
+
+      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_backupunit_update_spec.rb

@@ -0,0 +1,78 @@
+require 'spec_helper'
+require 'ionoscloud_backupunit_update'
+
+Chef::Knife::IonoscloudBackupunitUpdate.load_deps
+
+describe Chef::Knife::IonoscloudBackupunitUpdate do
+  before :each do
+    subject { Chef::Knife::IonoscloudBackupunitUpdate.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call BackupUnitApi.backupunits_patch' do
+      backupunit = backupunit_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        backupunit_id: backupunit.id,
+        email: 'edited' + backupunit.properties.email,
+        password: 'password_edited',
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+
+      expect(subject).to receive(:puts).with("ID: #{backupunit.id}")
+      expect(subject).to receive(:puts).with("Name: #{backupunit.properties.name}")
+      expect(subject).to receive(:puts).with("Email: #{subject_config[:email]}")
+
+      backupunit.properties.email = subject_config[:email]
+
+      mock_wait_for(subject)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'PATCH',
+            path: "/backupunits/#{backupunit.id}",
+            operation: :'BackupUnitApi.backupunits_patch',
+            return_type: 'BackupUnit',
+            body: { email: subject_config[:email], password: subject_config[:password] },
+            result: backupunit,
+          },
+          {
+            method: 'GET',
+            path: "/backupunits/#{backupunit.id}",
+            operation: :'BackupUnitApi.backupunits_find_by_id',
+            return_type: 'BackupUnit',
+            result: backupunit,
+          },
+        ],
+      )
+
+      expect { subject.run }.not_to raise_error(Exception)
+    end
+
+    it 'should not make any call if any required option is missing' do
+      required_options = subject.instance_variable_get(:@required_options)
+
+      arrays_without_one_element(required_options).each do |test_case|
+
+        test_case[:array].each { |value| subject.config[value] = 'test' }
+
+        expect(subject).to receive(:puts).with("Missing required parameters #{test_case[:removed]}")
+        expect(subject.api_client).not_to receive(:call_api)
+
+        expect { subject.run }.to raise_error(SystemExit) do |error|
+          expect(error.status).to eq(1)
+        end
+
+        required_options.each { |value| subject.config[value] = nil }
+      end
+    end
+  end
+end

data/spec/chef/knife/ionoscloud_datacenter_create_spec.rb

@@ -29,6 +29,9 @@ describe Chef::Knife::IonoscloudDatacenterCreate do
       expect(subject).to receive(:puts).with("Name: #{datacenter.properties.name}")
       expect(subject).to receive(:puts).with("Description: #{datacenter.properties.description}")
       expect(subject).to receive(:puts).with("Location: #{datacenter.properties.location}")
+      expect(subject).to receive(:puts).with("Version: #{datacenter.properties.version}")
+      expect(subject).to receive(:puts).with("Features: #{datacenter.properties.features}")
+      expect(subject).to receive(:puts).with("Sec Auth Protection: #{datacenter.properties.sec_auth_protection}")
 
       expected_body = datacenter.properties.to_hash
       expected_body.delete(:version)
@@ -45,6 +48,13 @@ describe Chef::Knife::IonoscloudDatacenterCreate do
             body: { properties: expected_body },
             result: datacenter,
           },
+          {
+            method: 'GET',
+            path: "/datacenters/#{datacenter.id}",
+            operation: :'DataCenterApi.datacenters_find_by_id',
+            return_type: 'Datacenter',
+            result: datacenter,
+          },
         ],
       )
 

data/spec/chef/knife/ionoscloud_datacenter_delete_spec.rb

@@ -27,6 +27,9 @@ describe Chef::Knife::IonoscloudDatacenterDelete do
       expect(subject).to receive(:puts).with("Name: #{datacenter.properties.name}")
       expect(subject).to receive(:puts).with("Description: #{datacenter.properties.description}")
       expect(subject).to receive(:puts).with("Location: #{datacenter.properties.location}")
+      expect(subject).to receive(:puts).with("Version: #{datacenter.properties.version}")
+      expect(subject).to receive(:puts).with("Features: #{datacenter.properties.features}")
+      expect(subject).to receive(:puts).with("Sec Auth Protection: #{datacenter.properties.sec_auth_protection}")
       expect(subject.ui).to receive(:warn).with("Deleted Data center #{datacenter.id}. Request ID: ")
 
       expect(subject.api_client).not_to receive(:wait_for)

data/spec/chef/knife/ionoscloud_datacenter_get_spec.rb

@@ -0,0 +1,69 @@
+require 'spec_helper'
+require 'ionoscloud_datacenter_get'
+
+Chef::Knife::IonoscloudDatacenterGet.load_deps
+
+describe Chef::Knife::IonoscloudDatacenterGet do
+  before :each do
+    subject { Chef::Knife::IonoscloudDatacenterGet.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call DataCenterApi.datacenters_find_by_id' do
+      datacenter = datacenter_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: datacenter.id,
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject).to receive(:puts).with("ID: #{datacenter.id}")
+      expect(subject).to receive(:puts).with("Name: #{datacenter.properties.name}")
+      expect(subject).to receive(:puts).with("Description: #{datacenter.properties.description}")
+      expect(subject).to receive(:puts).with("Location: #{datacenter.properties.location}")
+      expect(subject).to receive(:puts).with("Version: #{datacenter.properties.version}")
+      expect(subject).to receive(:puts).with("Features: #{datacenter.properties.features}")
+      expect(subject).to receive(:puts).with("Sec Auth Protection: #{datacenter.properties.sec_auth_protection}")
+
+      expect(subject.api_client).not_to receive(:wait_for)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/datacenters/#{datacenter.id}",
+            operation: :'DataCenterApi.datacenters_find_by_id',
+            return_type: 'Datacenter',
+            result: datacenter,
+          },
+        ],
+      )
+
+      expect { subject.run }.not_to raise_error(Exception)
+    end
+
+    it 'should not make any call if any required option is missing' do
+      required_options = subject.instance_variable_get(:@required_options)
+
+      arrays_without_one_element(required_options).each do |test_case|
+
+        test_case[:array].each { |value| subject.config[value] = 'test' }
+
+        expect(subject).to receive(:puts).with("Missing required parameters #{test_case[:removed]}")
+        expect(subject.api_client).not_to receive(:call_api)
+
+        expect { subject.run }.to raise_error(SystemExit) do |error|
+          expect(error.status).to eq(1)
+        end
+
+        required_options.each { |value| subject.config[value] = nil }
+      end
+    end
+  end
+end

data/spec/chef/knife/ionoscloud_datacenter_list_spec.rb

@@ -27,6 +27,7 @@ describe Chef::Knife::IonoscloudDatacenterList do
         subject.ui.color('Description', :bold),
         subject.ui.color('Location', :bold),
         subject.ui.color('Version', :bold),
+        subject.ui.color('Sec Auth Protection', :bold),
       ]
 
       datacenters.items.each do |datacenter|
@@ -35,9 +36,10 @@ describe Chef::Knife::IonoscloudDatacenterList do
         datacenter_list << datacenter.properties.description
         datacenter_list << datacenter.properties.location
         datacenter_list << datacenter.properties.version.to_s
+        datacenter_list << datacenter.properties.sec_auth_protection.to_s
       end
 
-      expect(subject.ui).to receive(:list).with(datacenter_list, :uneven_columns_across, 5)
+      expect(subject.ui).to receive(:list).with(datacenter_list, :uneven_columns_across, 6)
 
       mock_call_api(
         subject,