knife-ionoscloud 6.0.0.beta.1 → 6.0.0.beta.2

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_api = Ionoscloud::VolumesApi.new(api_client)
+
+        if @updatable_fields.map { |el| config[el] }.any?
+          print "#{ui.color('Updating Volume...', :magenta)}"
+
+          _, _, headers  = volume_api.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_api.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 = '6.0.0.beta.1'
+    VERSION = '6.0.0.beta.2'
     MAJOR, MINOR, TINY = VERSION.split('.')
   end
 end

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 BackupUnitsApi.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: :'BackupUnitsApi.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 BackupUnitsApi.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: :'BackupUnitsApi.backupunits_patch',
+            return_type: 'BackupUnit',
+            body: { email: subject_config[:email], password: subject_config[:password] },
+            result: backupunit,
+          },
+          {
+            method: 'GET',
+            path: "/backupunits/#{backupunit.id}",
+            operation: :'BackupUnitsApi.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)

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 DataCentersApi.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: :'DataCentersApi.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_update_spec.rb

@@ -0,0 +1,82 @@
+require 'spec_helper'
+require 'ionoscloud_datacenter_update'
+
+Chef::Knife::IonoscloudDatacenterUpdate.load_deps
+
+describe Chef::Knife::IonoscloudDatacenterUpdate do
+  before :each do
+    subject { Chef::Knife::IonoscloudDatacenterUpdate.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call DataCentersApi.datacenters_patch' do
+      datacenter = datacenter_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: datacenter.id,
+        name: datacenter.properties.name + '_edited',
+        description: datacenter.properties.description + '_edited',
+        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: #{subject_config[:name]}")
+      expect(subject).to receive(:puts).with("Description: #{subject_config[: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}")
+
+      datacenter.properties.name = subject_config[:name]
+      datacenter.properties.description = subject_config[:description]
+
+      mock_wait_for(subject)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'PATCH',
+            path: "/datacenters/#{datacenter.id}",
+            operation: :'DataCentersApi.datacenters_patch',
+            return_type: 'Datacenter',
+            body: { name: subject_config[:name], description: subject_config[:description] },
+            result: datacenter,
+          },
+          {
+            method: 'GET',
+            path: "/datacenters/#{datacenter.id}",
+            operation: :'DataCentersApi.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_firewall_create_spec.rb

@@ -36,6 +36,7 @@ describe Chef::Knife::IonoscloudFirewallCreate do
 
       expect(subject).to receive(:puts).with("ID: #{firewall.id}")
       expect(subject).to receive(:puts).with("Name: #{firewall.properties.name}")
+      expect(subject).to receive(:puts).with("Type: #{firewall.properties.type}")
       expect(subject).to receive(:puts).with("Protocol: #{firewall.properties.protocol}")
       expect(subject).to receive(:puts).with("Source MAC: #{firewall.properties.source_mac}")
       expect(subject).to receive(:puts).with("Source IP: #{firewall.properties.source_ip}")
@@ -44,7 +45,6 @@ describe Chef::Knife::IonoscloudFirewallCreate do
       expect(subject).to receive(:puts).with("Port Range End: #{firewall.properties.port_range_end}")
       expect(subject).to receive(:puts).with("ICMP Type: #{firewall.properties.icmp_type}")
       expect(subject).to receive(:puts).with("ICMP Code: #{firewall.properties.icmp_code}")
-      expect(subject).to receive(:puts).with("Type: #{firewall.properties.type}")
 
       mock_wait_for(subject)
       mock_call_api(

data/spec/chef/knife/ionoscloud_firewall_delete_spec.rb

@@ -28,6 +28,7 @@ describe Chef::Knife::IonoscloudFirewallDelete do
 
       expect(subject).to receive(:puts).with("ID: #{firewall.id}")
       expect(subject).to receive(:puts).with("Name: #{firewall.properties.name}")
+      expect(subject).to receive(:puts).with("Type: #{firewall.properties.type}")
       expect(subject).to receive(:puts).with("Protocol: #{firewall.properties.protocol}")
       expect(subject).to receive(:puts).with("Source MAC: #{firewall.properties.source_mac}")
       expect(subject).to receive(:puts).with("Source IP: #{firewall.properties.source_ip}")
@@ -36,7 +37,6 @@ describe Chef::Knife::IonoscloudFirewallDelete do
       expect(subject).to receive(:puts).with("Port Range End: #{firewall.properties.port_range_end}")
       expect(subject).to receive(:puts).with("ICMP Type: #{firewall.properties.icmp_type}")
       expect(subject).to receive(:puts).with("ICMP Code: #{firewall.properties.icmp_code}")
-      expect(subject).to receive(:puts).with("Type: #{firewall.properties.type}")
       expect(subject.ui).to receive(:warn).with("Deleted Firewall rule #{firewall.id}. Request ID: ")
 
       expect(subject.api_client).not_to receive(:wait_for)

data/spec/chef/knife/ionoscloud_firewall_get_spec.rb

@@ -0,0 +1,77 @@
+require 'spec_helper'
+require 'ionoscloud_firewall_get'
+
+Chef::Knife::IonoscloudFirewallGet.load_deps
+
+describe Chef::Knife::IonoscloudFirewallGet do
+  before :each do
+    subject { Chef::Knife::IonoscloudFirewallGet.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call FirewallRulesApi.datacenters_servers_nics_firewallrules_find_by_id' do
+      firewall = firewall_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: 'datacenter_id',
+        server_id: 'server_id',
+        nic_id: 'nic_id',
+        firewall_id: firewall.id,
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject).to receive(:puts).with("ID: #{firewall.id}")
+      expect(subject).to receive(:puts).with("Name: #{firewall.properties.name}")
+      expect(subject).to receive(:puts).with("Type: #{firewall.properties.type}")
+      expect(subject).to receive(:puts).with("Protocol: #{firewall.properties.protocol}")
+      expect(subject).to receive(:puts).with("Source MAC: #{firewall.properties.source_mac}")
+      expect(subject).to receive(:puts).with("Source IP: #{firewall.properties.source_ip}")
+      expect(subject).to receive(:puts).with("Target IP: #{firewall.properties.target_ip}")
+      expect(subject).to receive(:puts).with("Port Range Start: #{firewall.properties.port_range_start}")
+      expect(subject).to receive(:puts).with("Port Range End: #{firewall.properties.port_range_end}")
+      expect(subject).to receive(:puts).with("ICMP Type: #{firewall.properties.icmp_type}")
+      expect(subject).to receive(:puts).with("ICMP Code: #{firewall.properties.icmp_code}")
+
+      expect(subject.api_client).not_to receive(:wait_for)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/servers/#{subject_config[:server_id]}/"\
+                  "nics/#{subject_config[:nic_id]}/firewallrules/#{firewall.id}",
+            operation: :'FirewallRulesApi.datacenters_servers_nics_firewallrules_find_by_id',
+            return_type: 'FirewallRule',
+            result: firewall,
+          },
+        ],
+      )
+
+      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_firewall_update_spec.rb

@@ -0,0 +1,117 @@
+require 'spec_helper'
+require 'ionoscloud_firewall_update'
+
+Chef::Knife::IonoscloudFirewallUpdate.load_deps
+
+describe Chef::Knife::IonoscloudFirewallUpdate do
+  before :each do
+    subject { Chef::Knife::IonoscloudFirewallUpdate.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call FirewallRulesApi.datacenters_servers_nics_firewallrules_patch' do
+      firewall = firewall_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: 'datacenter_id',
+        server_id: 'server_id',
+        nic_id: 'nic_id',
+        firewall_id: firewall.id,
+        name: firewall.properties.name + '_edited',
+        protocol: 'TCP',
+        source_mac: '01:11:11:11:22:00',
+        source_ip: '1.2.3.4',
+        target_ip: '2.2.3.1',
+        port_range_start: 100,
+        port_range_end: 145,
+        icmp_type: 15,
+        icmp_code: 14,
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject).to receive(:puts).with("ID: #{firewall.id}")
+      expect(subject).to receive(:puts).with("Name: #{subject_config[:name]}")
+      expect(subject).to receive(:puts).with("Type: #{firewall.properties.type}")
+      expect(subject).to receive(:puts).with("Protocol: #{subject_config[:protocol]}")
+      expect(subject).to receive(:puts).with("Source MAC: #{subject_config[:source_mac]}")
+      expect(subject).to receive(:puts).with("Source IP: #{subject_config[:source_ip]}")
+      expect(subject).to receive(:puts).with("Target IP: #{subject_config[:target_ip]}")
+      expect(subject).to receive(:puts).with("Port Range Start: #{subject_config[:port_range_start]}")
+      expect(subject).to receive(:puts).with("Port Range End: #{subject_config[:port_range_end]}")
+      expect(subject).to receive(:puts).with("ICMP Type: #{subject_config[:icmp_type]}")
+      expect(subject).to receive(:puts).with("ICMP Code: #{subject_config[:icmp_code]}")
+
+      firewall.properties.name = subject_config[:name]
+      firewall.properties.protocol = subject_config[:protocol]
+      firewall.properties.source_mac = subject_config[:source_mac]
+      firewall.properties.source_ip = subject_config[:source_ip]
+      firewall.properties.target_ip = subject_config[:target_ip]
+      firewall.properties.port_range_start = subject_config[:port_range_start]
+      firewall.properties.port_range_end = subject_config[:port_range_end]
+      firewall.properties.icmp_type = subject_config[:icmp_type]
+      firewall.properties.icmp_code = subject_config[:icmp_code]
+
+      expected_body = {
+        name: subject_config[:name],
+        protocol: subject_config[:protocol],
+        sourceMac: subject_config[:source_mac],
+        sourceIp: subject_config[:source_ip],
+        targetIp: subject_config[:target_ip],
+        portRangeStart: subject_config[:port_range_start],
+        portRangeEnd: subject_config[:port_range_end],
+        icmpType: subject_config[:icmp_type],
+        icmpCode: subject_config[:icmp_code],
+      }
+
+      mock_wait_for(subject)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'PATCH',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/servers/#{subject_config[:server_id]}/"\
+            "nics/#{subject_config[:nic_id]}/firewallrules/#{firewall.id}",
+            operation: :'FirewallRulesApi.datacenters_servers_nics_firewallrules_patch',
+            return_type: 'FirewallRule',
+            body: expected_body,
+            result: firewall,
+          },
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/servers/#{subject_config[:server_id]}/"\
+                  "nics/#{subject_config[:nic_id]}/firewallrules/#{firewall.id}",
+            operation: :'FirewallRulesApi.datacenters_servers_nics_firewallrules_find_by_id',
+            return_type: 'FirewallRule',
+            result: firewall,
+          },
+        ],
+      )
+
+      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