knife-ionoscloud 5.1.0 → 6.0.0.alpha.1

data/spec/chef/knife/ionoscloud_flowlog_list_spec.rb

@@ -0,0 +1,264 @@
+require 'spec_helper'
+require 'ionoscloud_flowlog_list'
+
+Chef::Knife::IonoscloudFlowlogList.load_deps
+
+describe Chef::Knife::IonoscloudFlowlogList do
+  before :each do
+    subject { Chef::Knife::IonoscloudFlowlogList.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call FlowLogsApi.datacenters_servers_nics_flowlogs_get' do
+      flowlogs = flowlogs_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: 'datacenter_id',
+        server_id: 'server_id',
+        nic_id: 'nic_id',
+        type: 'nic',
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      flowlog_list = [
+        subject.ui.color('ID', :bold),
+        subject.ui.color('Name', :bold),
+        subject.ui.color('Action', :bold),
+        subject.ui.color('Direction', :bold),
+        subject.ui.color('Bucket', :bold),
+      ]
+
+      flowlogs.items.each do |flowlog|
+        flowlog_list << flowlog.id
+        flowlog_list << flowlog.properties.name
+        flowlog_list << flowlog.properties.action.to_s
+        flowlog_list << flowlog.properties.direction.to_s
+        flowlog_list << flowlog.properties.bucket.to_s
+      end
+
+      expect(subject.ui).to receive(:list).with(flowlog_list, :uneven_columns_across, 5)
+
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/servers/#{subject_config[:server_id]}/"\
+                  "nics/#{subject_config[:nic_id]}/flowlogs",
+            operation: :'FlowLogsApi.datacenters_servers_nics_flowlogs_get',
+            return_type: 'FlowLogs',
+            result: flowlogs,
+          },
+        ],
+      )
+
+      expect { subject.run }.not_to raise_error(Exception)
+    end
+
+    it 'should not call FlowLogsApi.datacenters_servers_nics_flowlogs_get when server_id and nic_id are not given' do
+      flowlog_id = 'invalid_id'
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: 'datacenter_id',
+        type: 'nic',
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+      subject.name_args = [flowlog_id]
+
+      expect(subject).to receive(:puts).with("Missing required parameters #{[:server_id, :nic_id]}")
+
+      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 call NATGatewaysApi.datacenters_natgateways_flowlogs_get' do
+      flowlogs = flowlogs_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: 'datacenter_id',
+        natgateway_id: 'natgateway_id',
+        type: 'natgateway',
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      flowlog_list = [
+        subject.ui.color('ID', :bold),
+        subject.ui.color('Name', :bold),
+        subject.ui.color('Action', :bold),
+        subject.ui.color('Direction', :bold),
+        subject.ui.color('Bucket', :bold),
+      ]
+
+      flowlogs.items.each do |flowlog|
+        flowlog_list << flowlog.id
+        flowlog_list << flowlog.properties.name
+        flowlog_list << flowlog.properties.action.to_s
+        flowlog_list << flowlog.properties.direction.to_s
+        flowlog_list << flowlog.properties.bucket.to_s
+      end
+
+      expect(subject.ui).to receive(:list).with(flowlog_list, :uneven_columns_across, 5)
+
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/natgateways/#{subject_config[:natgateway_id]}/flowlogs",
+            operation: :'NATGatewaysApi.datacenters_natgateways_flowlogs_get',
+            return_type: 'FlowLogs',
+            result: flowlogs,
+          },
+        ],
+      )
+
+      expect { subject.run }.not_to raise_error(Exception)
+    end
+
+    it 'should not call NATGatewaysApi.datacenters_natgateways_flowlogs_get when natgateway_id is not given' do
+      flowlog_id = 'invalid_id'
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: 'datacenter_id',
+        type: 'natgateway',
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+      subject.name_args = [flowlog_id]
+
+      expect(subject).to receive(:puts).with("Missing required parameters #{[:natgateway_id]}")
+
+      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 call NetworkLoadBalancersApi.datacenters_servers_nics_flowlogs_get' do
+      flowlogs = flowlogs_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: 'datacenter_id',
+        network_loadbalancer_id: 'network_loadbalancer_id',
+        type: 'loadbalancer',
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      flowlog_list = [
+        subject.ui.color('ID', :bold),
+        subject.ui.color('Name', :bold),
+        subject.ui.color('Action', :bold),
+        subject.ui.color('Direction', :bold),
+        subject.ui.color('Bucket', :bold),
+      ]
+
+      flowlogs.items.each do |flowlog|
+        flowlog_list << flowlog.id
+        flowlog_list << flowlog.properties.name
+        flowlog_list << flowlog.properties.action.to_s
+        flowlog_list << flowlog.properties.direction.to_s
+        flowlog_list << flowlog.properties.bucket.to_s
+      end
+
+      expect(subject.ui).to receive(:list).with(flowlog_list, :uneven_columns_across, 5)
+
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/networkloadbalancers/#{subject_config[:network_loadbalancer_id]}/flowlogs",
+            operation: :'NetworkLoadBalancersApi.datacenters_networkloadbalancers_flowlogs_get',
+            return_type: 'FlowLogs',
+            result: flowlogs,
+          },
+        ],
+      )
+
+      expect { subject.run }.not_to raise_error(Exception)
+    end
+
+    it 'should not call NetworkLoadBalancersApi.datacenters_networkloadbalancers_flowlogs_get when network_loadbalancer_id is not given' do
+      flowlog_id = 'invalid_id'
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: 'datacenter_id',
+        type: 'loadbalancer',
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+      subject.name_args = [flowlog_id]
+
+      expect(subject).to receive(:puts).with("Missing required parameters #{[:network_loadbalancer_id]}")
+
+      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 the wrong type is given' do
+      flowlog = flowlog_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: 'datacenter_id',
+        server_id: 'server_id',
+        natgateway_id: 'natgateway_id',
+        network_loadbalancer_id: 'network_loadbalancer_id',
+        nic_id: 'nic_id',
+        type: 'other',
+        name: flowlog.properties.name,
+        action: flowlog.properties.action,
+        direction: flowlog.properties.direction,
+        bucket: flowlog.properties.bucket,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject.ui).to receive(:error).with("Flow log cannot belong to #{subject_config[:type]}. Value must be one of ['nic', 'natgateway', 'loadbalancer']")
+
+      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)
+
+      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_group_create_spec.rb

@@ -27,6 +27,9 @@ describe Chef::Knife::IonoscloudGroupCreate do
         create_k8s_cluster: group.properties.create_k8s_cluster,
         create_pcc: group.properties.create_pcc,
         create_internet_access: group.properties.create_internet_access,
+        create_flow_log: group.properties.create_flow_log,
+        access_and_manage_monitoring: group.properties.access_and_manage_monitoring,
+        access_and_manage_certificates: group.properties.access_and_manage_certificates,
       }
 
       subject_config.each { |key, value| subject.config[key] = value }
@@ -42,6 +45,9 @@ describe Chef::Knife::IonoscloudGroupCreate do
       expect(subject).to receive(:puts).with("Create K8s Clusters: #{group.properties.create_k8s_cluster.to_s}")
       expect(subject).to receive(:puts).with("Create PCC: #{group.properties.create_pcc.to_s}")
       expect(subject).to receive(:puts).with("Create Internet Acess: #{group.properties.create_internet_access.to_s}")
+      expect(subject).to receive(:puts).with("Create Flow Logs: #{group.properties.create_flow_log.to_s}")
+      expect(subject).to receive(:puts).with("Access and Manage Monitoring: #{group.properties.access_and_manage_monitoring.to_s}")
+      expect(subject).to receive(:puts).with("Access and Manage Certificates: #{group.properties.access_and_manage_certificates.to_s}")
 
       mock_wait_for(subject)
       mock_call_api(

data/spec/chef/knife/ionoscloud_group_delete_spec.rb

@@ -33,6 +33,12 @@ describe Chef::Knife::IonoscloudGroupDelete do
       expect(subject).to receive(:puts).with("Access Activity Log: #{group.properties.access_activity_log.to_s}")
       expect(subject).to receive(:puts).with("S3 Privilege: #{group.properties.s3_privilege.to_s}")
       expect(subject).to receive(:puts).with("Create Backup Unit: #{group.properties.create_backup_unit.to_s}")
+      expect(subject).to receive(:puts).with("Create K8s Clusters: #{group.properties.create_k8s_cluster.to_s}")
+      expect(subject).to receive(:puts).with("Create PCC: #{group.properties.create_pcc.to_s}")
+      expect(subject).to receive(:puts).with("Create Internet Acess: #{group.properties.create_internet_access.to_s}")
+      expect(subject).to receive(:puts).with("Create Flow Logs: #{group.properties.create_flow_log.to_s}")
+      expect(subject).to receive(:puts).with("Access and Manage Monitoring: #{group.properties.access_and_manage_monitoring.to_s}")
+      expect(subject).to receive(:puts).with("Access and Manage Certificates: #{group.properties.access_and_manage_certificates.to_s}")
       expect(subject).to receive(:puts).with("Users: #{users.to_s}")
       expect(subject.ui).to receive(:warn).with("Deleted Group #{group.id}. Request ID: ")
 

data/spec/chef/knife/ionoscloud_group_get_spec.rb

@@ -34,6 +34,12 @@ describe Chef::Knife::IonoscloudGroupGet do
       expect(subject).to receive(:puts).with("Access Activity Log: #{group.properties.access_activity_log.to_s}")
       expect(subject).to receive(:puts).with("S3 Privilege: #{group.properties.s3_privilege.to_s}")
       expect(subject).to receive(:puts).with("Create Backup Unit: #{group.properties.create_backup_unit.to_s}")
+      expect(subject).to receive(:puts).with("Create K8s Clusters: #{group.properties.create_k8s_cluster.to_s}")
+      expect(subject).to receive(:puts).with("Create PCC: #{group.properties.create_pcc.to_s}")
+      expect(subject).to receive(:puts).with("Create Internet Acess: #{group.properties.create_internet_access.to_s}")
+      expect(subject).to receive(:puts).with("Create Flow Logs: #{group.properties.create_flow_log.to_s}")
+      expect(subject).to receive(:puts).with("Access and Manage Monitoring: #{group.properties.access_and_manage_monitoring.to_s}")
+      expect(subject).to receive(:puts).with("Access and Manage Certificates: #{group.properties.access_and_manage_certificates.to_s}")
       expect(subject).to receive(:puts).with("Users: #{users.to_s}")
 
       mock_call_api(

data/spec/chef/knife/ionoscloud_group_list_spec.rb

@@ -11,12 +11,18 @@ describe Chef::Knife::IonoscloudGroupList do
     @group_list = group_list = [
       subject.ui.color('ID', :bold),
       subject.ui.color('Name', :bold),
-      subject.ui.color('Create Datacenter', :bold),
-      subject.ui.color('Create Snapshot', :bold),
-      subject.ui.color('Reserve IP', :bold),
-      subject.ui.color('Access Activity Log', :bold),
-      subject.ui.color('S3 Privilege', :bold),
-      subject.ui.color('Create Backup Unit', :bold),
+      subject.ui.color('Datacenter', :bold),
+      subject.ui.color('Snapshot', :bold),
+      subject.ui.color('IP', :bold),
+      subject.ui.color('Activity', :bold),
+      subject.ui.color('S3', :bold),
+      subject.ui.color('Backup', :bold),
+      subject.ui.color('K8s', :bold),
+      subject.ui.color('PCC', :bold),
+      subject.ui.color('Internet', :bold),
+      subject.ui.color('FlowLogs', :bold),
+      subject.ui.color('Monitoring', :bold),
+      subject.ui.color('Certificates', :bold),
       @groups.items.first.id,
       @groups.items.first.properties.name,
       @groups.items.first.properties.create_data_center.to_s,
@@ -25,6 +31,12 @@ describe Chef::Knife::IonoscloudGroupList do
       @groups.items.first.properties.access_activity_log.to_s,
       @groups.items.first.properties.s3_privilege.to_s,
       @groups.items.first.properties.create_backup_unit.to_s,
+      @groups.items.first.properties.create_k8s_cluster.to_s,
+      @groups.items.first.properties.create_pcc.to_s,
+      @groups.items.first.properties.create_internet_access.to_s,
+      @groups.items.first.properties.create_flow_log.to_s,
+      @groups.items.first.properties.access_and_manage_monitoring.to_s,
+      @groups.items.first.properties.access_and_manage_certificates.to_s,
       @groups.items[1].id,
       @groups.items[1].properties.name,
       @groups.items[1].properties.create_data_center.to_s,
@@ -33,6 +45,12 @@ describe Chef::Knife::IonoscloudGroupList do
       @groups.items[1].properties.access_activity_log.to_s,
       @groups.items[1].properties.s3_privilege.to_s,
       @groups.items[1].properties.create_backup_unit.to_s,
+      @groups.items[1].properties.create_k8s_cluster.to_s,
+      @groups.items[1].properties.create_pcc.to_s,
+      @groups.items[1].properties.create_internet_access.to_s,
+      @groups.items[1].properties.create_flow_log.to_s,
+      @groups.items[1].properties.access_and_manage_monitoring.to_s,
+      @groups.items[1].properties.access_and_manage_certificates.to_s,
     ]
 
     allow(subject).to receive(:puts)
@@ -48,7 +66,7 @@ describe Chef::Knife::IonoscloudGroupList do
 
       subject_config.each { |key, value| subject.config[key] = value }
 
-      expect(subject.ui).to receive(:list).with(@group_list, :uneven_columns_across, 8)
+      expect(subject.ui).to receive(:list).with(@group_list, :uneven_columns_across, 14)
 
       mock_call_api(
         subject,
@@ -75,7 +93,7 @@ describe Chef::Knife::IonoscloudGroupList do
 
       subject_config.each { |key, value| subject.config[key] = value }
 
-      expect(subject.ui).to receive(:list).with(@group_list, :uneven_columns_across, 8)
+      expect(subject.ui).to receive(:list).with(@group_list, :uneven_columns_across, 14)
 
       expect(subject.api_client).not_to receive(:wait_for)
       mock_call_api(

data/spec/chef/knife/ionoscloud_image_list_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudImageList do
   end
 
   describe '#run' do
-    it 'should call ImageApi.images_get' do
+    it 'should call ImagesApi.images_get' do
       images = images_mock
       subject_config = {
         ionoscloud_username: 'email',
@@ -44,7 +44,7 @@ describe Chef::Knife::IonoscloudImageList do
           {
             method: 'GET',
             path: '/images',
-            operation: :'ImageApi.images_get',
+            operation: :'ImagesApi.images_get',
             return_type: 'Images',
             result: images,
           },

data/spec/chef/knife/ionoscloud_ipfailover_add_spec.rb

@@ -38,21 +38,21 @@ describe Chef::Knife::IonoscloudIpfailoverAdd do
           {
             method: 'GET',
             path: "/datacenters/#{subject_config[:datacenter_id]}/lans/#{lan.id}",
-            operation: :'LanApi.datacenters_lans_find_by_id',
+            operation: :'LansApi.datacenters_lans_find_by_id',
             return_type: 'Lan',
             result: lan,
           },
           {
             method: 'PATCH',
             path: "/datacenters/#{subject_config[:datacenter_id]}/lans/#{lan.id}",
-            operation: :'LanApi.datacenters_lans_patch',
+            operation: :'LansApi.datacenters_lans_patch',
             body: Ionoscloud::LanProperties.new({ ip_failover: [({ ip: subject_config[:ip], nicUuid: subject_config[:nic_id] })] }).to_hash,
             return_type: 'Lan',
           },
           {
             method: 'GET',
             path: "/datacenters/#{subject_config[:datacenter_id]}/lans/#{lan.id}",
-            operation: :'LanApi.datacenters_lans_find_by_id',
+            operation: :'LansApi.datacenters_lans_find_by_id',
             return_type: 'Lan',
             result: lan,
           },

data/spec/chef/knife/ionoscloud_ipfailover_remove_spec.rb

@@ -41,21 +41,21 @@ describe Chef::Knife::IonoscloudIpfailoverRemove do
           {
             method: 'GET',
             path: "/datacenters/#{subject_config[:datacenter_id]}/lans/#{lan.id}",
-            operation: :'LanApi.datacenters_lans_find_by_id',
+            operation: :'LansApi.datacenters_lans_find_by_id',
             return_type: 'Lan',
             result: lan,
           },
           {
             method: 'PATCH',
             path: "/datacenters/#{subject_config[:datacenter_id]}/lans/#{lan.id}",
-            operation: :'LanApi.datacenters_lans_patch',
+            operation: :'LansApi.datacenters_lans_patch',
             body: Ionoscloud::LanProperties.new({ ip_failover: [] }).to_hash,
             return_type: 'Lan',
           },
           {
             method: 'GET',
             path: "/datacenters/#{subject_config[:datacenter_id]}/lans/#{lan.id}",
-            operation: :'LanApi.datacenters_lans_find_by_id',
+            operation: :'LansApi.datacenters_lans_find_by_id',
             return_type: 'Lan',
             result: lan_copy,
           },
@@ -91,7 +91,7 @@ describe Chef::Knife::IonoscloudIpfailoverRemove do
           {
               method: 'GET',
               path: "/datacenters/#{subject_config[:datacenter_id]}/lans/#{lan.id}",
-              operation: :'LanApi.datacenters_lans_find_by_id',
+              operation: :'LansApi.datacenters_lans_find_by_id',
               return_type: 'Lan',
               result: lan,
           },