foreman_netbox 1.1.1 → 2.0.0

data/test/interactors/foreman_netbox/sync_host/sync_device/sync_site/find_test.rb

@@ -30,8 +30,8 @@ class FindSiteTest < ActiveSupport::TestCase
         body: {
           count: 1,
           results: [
-            { id: 1, name: host.location.netbox_site_name, slug: host.location.netbox_site_slug }
-          ]
+            { id: 1, name: host.location.netbox_site_name, slug: host.location.netbox_site_slug },
+          ],
         }.to_json
       )
 
@@ -48,7 +48,7 @@ class FindSiteTest < ActiveSupport::TestCase
         status: 200, headers: { 'Content-Type': 'application/json' },
         body: {
           count: 0,
-          results: []
+          results: [],
         }.to_json
       )
 

data/test/interactors/foreman_netbox/sync_host/sync_device/sync_site/update_test.rb

@@ -10,9 +10,9 @@ class UpdateSiteTest < ActiveSupport::TestCase
   end
 
   let(:host) { FactoryBot.build_stubbed(:host) }
-  # let(:tags) { [ForemanNetbox::API.client::Extras::Tag.new(id: 1, name: 'foreman', slug: 'foreman')] }
+  # let(:tags) { [ForemanNetbox::Api.client::Extras::Tag.new(id: 1, name: 'foreman', slug: 'foreman')] }
   let(:site) do
-    ForemanNetbox::API.client::DCIM::Site.new(id: 1).tap do |site|
+    ForemanNetbox::Api.client::DCIM::Site.new(id: 1).tap do |site|
       site.instance_variable_set(
         :@data,
         { 'id' => 1, 'tags' => site_tags }
@@ -30,7 +30,7 @@ class UpdateSiteTest < ActiveSupport::TestCase
     it 'updates site' do
       stub_patch = stub_request(:patch, "#{Setting[:netbox_url]}/api/dcim/sites/1.json").with(
         body: {
-          tags: default_tags.map(&:id)
+          tags: default_tags.map(&:id),
         }.to_json
       ).to_return(
         status: 200, headers: { 'Content-Type': 'application/json' },

data/test/interactors/foreman_netbox/sync_host/sync_device/update_test.rb

@@ -18,7 +18,7 @@ class UpdateDeviceTest < ActiveSupport::TestCase
   end
 
   let(:device) do
-    ForemanNetbox::API.client::DCIM::Device.new(id: 1).tap do |device|
+    ForemanNetbox::Api.client::DCIM::Device.new(id: 1).tap do |device|
       device.instance_variable_set(
         :@data,
         {
@@ -32,14 +32,14 @@ class UpdateDeviceTest < ActiveSupport::TestCase
           'primary_ip4' => {
             'id' => 1,
             'family' => 4,
-            'address' => '10.0.0.8/24'
+            'address' => '10.0.0.8/24',
           },
           'primary_ip6' => {
             'id' => 2,
             'family' => 6,
-            'address' => '1600:0:2d0:202::18/64'
+            'address' => '1600:0:2d0:202::18/64',
           },
-          'tags' => device_tags
+          'tags' => device_tags,
         }
       )
     end
@@ -68,7 +68,7 @@ class UpdateDeviceTest < ActiveSupport::TestCase
     )
   end
   let(:serialnumber) { device_data[:serial] }
-  let(:ip_addresses) { ForemanNetbox::API.client.ipam.ip_addresses.filter(device_id: device.id) }
+  let(:ip_addresses) { ForemanNetbox::Api.client.ipam.ip_addresses.filter(device_id: device.id) }
   let(:host) do
     FactoryBot.build_stubbed(
       :host,
@@ -90,8 +90,8 @@ class UpdateDeviceTest < ActiveSupport::TestCase
         count: 2,
         results: [
           { id: primary_ip4.id, address: primary_ip4.address.address },
-          { id: primary_ip6.id, address: primary_ip6.address.address }
-        ]
+          { id: primary_ip6.id, address: primary_ip6.address.address },
+        ],
       }.to_json
     )
   end
@@ -109,7 +109,7 @@ class UpdateDeviceTest < ActiveSupport::TestCase
             body: {
               id: t['id'],
               name: t['name'],
-              slug: t['slug']
+              slug: t['slug'],
             }.to_json
           )
       end
@@ -163,7 +163,7 @@ class UpdateDeviceTest < ActiveSupport::TestCase
           site: site.id,
           tenant: tenant.id,
           serial: serialnumber,
-          tags: default_tags.map(&:id)
+          tags: default_tags.map(&:id),
         }.to_json
       ).to_return(
         status: 200, headers: { 'Content-Type': 'application/json' },
@@ -187,7 +187,7 @@ class UpdateDeviceTest < ActiveSupport::TestCase
             primary_ip6: primary_ip6.id,
             site: site.id,
             tenant: tenant.id,
-            tags: default_tags.map(&:id)
+            tags: default_tags.map(&:id),
           }.to_json
         ).to_return(
           status: 200, headers: { 'Content-Type': 'application/json' },
@@ -219,7 +219,7 @@ class UpdateDeviceTest < ActiveSupport::TestCase
             body: {
               id: t['id'],
               name: t['name'],
-              slug: t['slug']
+              slug: t['slug'],
             }.to_json
           )
       end

data/test/interactors/foreman_netbox/sync_host/sync_tenant/create_test.rb

@@ -28,7 +28,7 @@ class CreateTenantTest < ActiveSupport::TestCase
         body: {
           name: host.owner.name,
           slug: host.owner.name.parameterize,
-          tags: default_tags.map(&:id)
+          tags: default_tags.map(&:id),
         }.to_json
       ).to_return(
         status: 201, headers: { 'Content-Type': 'application/json' },

data/test/interactors/foreman_netbox/sync_host/sync_tenant/find_test.rb

@@ -29,8 +29,8 @@ class FindTenantTest < ActiveSupport::TestCase
         body: {
           count: 1,
           results: [
-            { id: 1, name: host.owner.name, slug: host.owner.name.parameterize }
-          ]
+            { id: 1, name: host.owner.name, slug: host.owner.name.parameterize },
+          ],
         }.to_json
       )
 
@@ -47,7 +47,7 @@ class FindTenantTest < ActiveSupport::TestCase
         status: 200, headers: { 'Content-Type': 'application/json' },
         body: {
           count: 0,
-          results: []
+          results: [],
         }.to_json
       )
 

data/test/interactors/foreman_netbox/sync_host/sync_tenant/update_test.rb

@@ -10,7 +10,7 @@ class UpdateTenantTest < ActiveSupport::TestCase
   end
 
   let(:tenant) do
-    ForemanNetbox::API.client::Tenancy::Tenant.new(id: 1).tap do |tenant|
+    ForemanNetbox::Api.client::Tenancy::Tenant.new(id: 1).tap do |tenant|
       tenant.instance_variable_set(
         :@data,
         { 'id' => 1, 'tags' => tenant_tags }
@@ -34,7 +34,7 @@ class UpdateTenantTest < ActiveSupport::TestCase
     it 'updates tenant' do
       stub_patch = stub_request(:patch, "#{Setting[:netbox_url]}/api/tenancy/tenants/1.json").with(
         body: {
-          tags: default_tags.map(&:id)
+          tags: default_tags.map(&:id),
         }.to_json
       ).to_return(
         status: 200, headers: { 'Content-Type': 'application/json' },

data/test/interactors/foreman_netbox/sync_host/sync_virtual_machine/create_test.rb

@@ -26,7 +26,7 @@ class CreateVirtualMachineTest < ActiveSupport::TestCase
           cpus: 2,
           memory_mb: 512,
           volumes: [
-            OpenStruct.new(size_gb: 128)
+            OpenStruct.new(size_gb: 128),
           ]
         )
       )
@@ -53,7 +53,7 @@ class CreateVirtualMachineTest < ActiveSupport::TestCase
           name: netbox_virtual_machine_params[:name],
           cluster: cluster.id,
           tenant: tenant.id,
-          tags: default_tags.map(&:id)
+          tags: default_tags.map(&:id),
         }.to_json
       ).to_return(
         status: 201, headers: { 'Content-Type': 'application/json' },

data/test/interactors/foreman_netbox/sync_host/sync_virtual_machine/save_netbox_url_test.rb

@@ -2,9 +2,9 @@
 
 require 'test_plugin_helper'
 
-class SaveNetboxUrlVirtualMachineTest < ActiveSupport::TestCase
+class SaveNetboxURLVirtualMachineTest < ActiveSupport::TestCase
   subject do
-    ForemanNetbox::SyncHost::SyncVirtualMachine::SaveNetboxUrl.call(host: host, virtual_machine: virtual_machine)
+    ForemanNetbox::SyncHost::SyncVirtualMachine::SaveNetboxURL.call(host: host, virtual_machine: virtual_machine)
   end
 
   let(:virtual_machine) { OpenStruct.new(id: 1) }

data/test/interactors/foreman_netbox/sync_host/sync_virtual_machine/sync_cluster/create_test.rb

@@ -36,7 +36,7 @@ class CreateClusterTest < ActiveSupport::TestCase
         body: {
           name: host.netbox_facet.netbox_params.dig(:cluster, :name),
           type: cluster_type.id,
-          tags: default_tags.map(&:id)
+          tags: default_tags.map(&:id),
         }.to_json
       ).to_return(
         status: 201, headers: { 'Content-Type': 'application/json' },

data/test/interactors/foreman_netbox/sync_host/sync_virtual_machine/sync_cluster/find_test.rb

@@ -30,7 +30,7 @@ class FindClusterTest < ActiveSupport::TestCase
         status: 200, headers: { 'Content-Type': 'application/json' },
         body: {
           count: 1,
-          results: [{ id: 1, name: host.compute_object.cluster }]
+          results: [{ id: 1, name: host.compute_object.cluster }],
         }.to_json
       )
 
@@ -47,7 +47,7 @@ class FindClusterTest < ActiveSupport::TestCase
         status: 200, headers: { 'Content-Type': 'application/json' },
         body: {
           count: 0,
-          results: []
+          results: [],
         }.to_json
       )
 

data/test/interactors/foreman_netbox/sync_host/sync_virtual_machine/sync_cluster/sync_cluster_type/find_test.rb

@@ -35,9 +35,9 @@ class FindClusterTypeTest < ActiveSupport::TestCase
             {
               id: 1,
               name: cluster_type_params[:name],
-              slug: cluster_type_params[:slug]
-            }
-          ]
+              slug: cluster_type_params[:slug],
+            },
+          ],
         }.to_json
       )
 
@@ -54,7 +54,7 @@ class FindClusterTypeTest < ActiveSupport::TestCase
         status: 200, headers: { 'Content-Type': 'application/json' },
         body: {
           count: 0,
-          results: []
+          results: [],
         }.to_json
       )
 

data/test/interactors/foreman_netbox/sync_host/sync_virtual_machine/sync_cluster/update_test.rb

@@ -11,7 +11,7 @@ class UpdateClusterTest < ActiveSupport::TestCase
 
   let(:host) { FactoryBot.build_stubbed(:host).tap { |h| h.stubs(:compute?).returns(true) } }
   let(:cluster) do
-    ForemanNetbox::API.client::Virtualization::Cluster.new(id: 1).tap do |cluster|
+    ForemanNetbox::Api.client::Virtualization::Cluster.new(id: 1).tap do |cluster|
       cluster.instance_variable_set(
         :@data,
         { 'id' => 1, 'tags' => cluster_tags }
@@ -29,7 +29,7 @@ class UpdateClusterTest < ActiveSupport::TestCase
     it 'updates cluster' do
       stub_patch = stub_request(:patch, "#{Setting[:netbox_url]}/api/virtualization/clusters/1.json").with(
         body: {
-          tags: default_tags.map(&:id)
+          tags: default_tags.map(&:id),
         }.to_json
       ).to_return(
         status: 200, headers: { 'Content-Type': 'application/json' },

data/test/interactors/foreman_netbox/sync_host/sync_virtual_machine/sync_interfaces/create_test.rb

@@ -21,7 +21,7 @@ class CreateVirtualMachineInterfacesTest < ActiveSupport::TestCase
       interfaces: [
         FactoryBot.build_stubbed(:nic_base, mac: 'FE:13:C6:44:29:24', ip: '10.0.0.1', ip6: '1500:0:2d0:201::1'),
         FactoryBot.build_stubbed(:nic_base, mac: 'FE:13:C6:44:29:22', ip: '10.0.0.2', ip6: '1500:0:2d0:201::2'),
-        FactoryBot.build_stubbed(:nic_base, identifier: nil, mac: nil)
+        FactoryBot.build_stubbed(:nic_base, identifier: nil, mac: nil),
       ]
     )
   end
@@ -38,13 +38,13 @@ class CreateVirtualMachineInterfacesTest < ActiveSupport::TestCase
         name: host.interfaces.first.netbox_name,
         mac_address: host.interfaces.first.mac.upcase,
         virtual_machine: virtual_machine.id,
-        tags: default_tags.map(&:id)
+        tags: default_tags.map(&:id),
       }.to_json
     ).to_return(
       status: 201, headers: { 'Content-Type': 'application/json' },
       body: {
         id: 1,
-        name: host.interfaces.first.netbox_name
+        name: host.interfaces.first.netbox_name,
       }.to_json
     )
 

data/test/interactors/foreman_netbox/sync_host/sync_virtual_machine/sync_interfaces/delete_test.rb

@@ -10,7 +10,7 @@ class DeleteVirtualMachineInterfacesTest < ActiveSupport::TestCase
   end
 
   let(:interface_id) { 1 }
-  let(:interfaces) { ForemanNetbox::API.client::Virtualization::Interfaces.new }
+  let(:interfaces) { ForemanNetbox::Api.client::Virtualization::Interfaces.new }
   let(:host) { FactoryBot.build_stubbed(:host, interfaces: []) }
 
   setup do
@@ -21,7 +21,7 @@ class DeleteVirtualMachineInterfacesTest < ActiveSupport::TestCase
       status: 200, headers: { 'Content-Type': 'application/json' },
       body: {
         count: 1,
-        results: [{ id: interface_id, name: 'eth0' }]
+        results: [{ id: interface_id, name: 'eth0' }],
       }.to_json
     )
   end

data/test/interactors/foreman_netbox/sync_host/sync_virtual_machine/sync_interfaces/find_test.rb

@@ -24,8 +24,8 @@ class FindVirtualMachineInterfacesTest < ActiveSupport::TestCase
         body: {
           count: 1,
           results: [
-            { id: 1 }
-          ]
+            { id: 1 },
+          ],
         }.to_json
       )
 
@@ -42,11 +42,11 @@ class FindVirtualMachineInterfacesTest < ActiveSupport::TestCase
         status: 200, headers: { 'Content-Type': 'application/json' },
         body: {
           count: 0,
-          results: []
+          results: [],
         }.to_json
       )
 
-      assert_equal [], subject.interfaces.map(&:id)
+      assert_empty subject.interfaces.map(&:id)
       assert_requested(stub_get)
     end
   end

data/test/interactors/foreman_netbox/sync_host/sync_virtual_machine/sync_interfaces/sync_ip_addresses/create_test.rb

@@ -23,7 +23,7 @@ class CreateVirtualMachineIpAddressesTest < ActiveSupport::TestCase
           ip6: '1500:0:2d0:201::1',
           subnet: subnet,
           subnet6: subnet6
-        )
+        ),
       ]
     )
   end
@@ -36,7 +36,7 @@ class CreateVirtualMachineIpAddressesTest < ActiveSupport::TestCase
       status: 200, headers: { 'Content-Type': 'application/json' },
       body: {
         count: 0,
-        results: []
+        results: [],
       }.to_json
     )
     stub_request(:get, "#{Setting[:netbox_url]}/api/ipam/ip-addresses.json").with(
@@ -45,7 +45,7 @@ class CreateVirtualMachineIpAddressesTest < ActiveSupport::TestCase
       status: 200, headers: { 'Content-Type': 'application/json' },
       body: {
         count: 1,
-        results: [{ id: 2 }]
+        results: [{ id: 2 }],
       }.to_json
     )
   end
@@ -56,7 +56,7 @@ class CreateVirtualMachineIpAddressesTest < ActiveSupport::TestCase
         address: host.interfaces.first.netbox_ip,
         assigned_object_type: 'virtualization.vminterface',
         assigned_object_id: interfaces.first.id,
-        tags: default_tags.map(&:id)
+        tags: default_tags.map(&:id),
       }.to_json
     ).to_return(
       status: 201, headers: { 'Content-Type': 'application/json' },
@@ -68,7 +68,7 @@ class CreateVirtualMachineIpAddressesTest < ActiveSupport::TestCase
         address: host.interfaces.first.netbox_ip6,
         assigned_object_type: 'virtualization.vminterface',
         assigned_object_id: interfaces.first.id,
-        tags: default_tags.map(&:id)
+        tags: default_tags.map(&:id),
       }.to_json
     ).to_return(
       status: 201, headers: { 'Content-Type': 'application/json' },

data/test/interactors/foreman_netbox/sync_host/sync_virtual_machine/sync_interfaces/sync_ip_addresses/delete_test.rb

@@ -9,8 +9,8 @@ class DeleteVirtualMachineIpAddressesTest < ActiveSupport::TestCase
     )
   end
 
-  let(:interfaces) { ForemanNetbox::API.client.virtualization.interfaces.filter(virtual_machine_id: 1) }
-  let(:ip_addresses) { ForemanNetbox::API.client.ipam.ip_addresses.filter(virtual_machine_id: 1) }
+  let(:interfaces) { ForemanNetbox::Api.client.virtualization.interfaces.filter(virtual_machine_id: 1) }
+  let(:ip_addresses) { ForemanNetbox::Api.client.ipam.ip_addresses.filter(virtual_machine_id: 1) }
 
   let(:host) do
     FactoryBot.build_stubbed(
@@ -20,7 +20,7 @@ class DeleteVirtualMachineIpAddressesTest < ActiveSupport::TestCase
           :nic_base,
           ip: '10.0.0.1',
           subnet: FactoryBot.build_stubbed(:subnet_ipv4)
-        )
+        ),
       ]
     )
   end
@@ -40,8 +40,8 @@ class DeleteVirtualMachineIpAddressesTest < ActiveSupport::TestCase
       body: {
         count: 1,
         results: [
-          { id: interface_id, name: host.interfaces.first.netbox_name }
-        ]
+          { id: interface_id, name: host.interfaces.first.netbox_name },
+        ],
       }.to_json
     )
     stub_request(:get, "#{Setting[:netbox_url]}/api/ipam/ip-addresses.json").with(
@@ -52,8 +52,8 @@ class DeleteVirtualMachineIpAddressesTest < ActiveSupport::TestCase
         count: 2,
         results: [
           { id: ip_addresses_v4_id, address: ip_addresses_v4, assigned_object_type: 'virtualization.vminterface', assigned_object_id: interface_id },
-          { id: ip_addresses_v6_id, address: ip_addresses_v6, assigned_object_type: 'virtualization.vminterface', assigned_object_id: interface_id }
-        ]
+          { id: ip_addresses_v6_id, address: ip_addresses_v6, assigned_object_type: 'virtualization.vminterface', assigned_object_id: interface_id },
+        ],
       }.to_json
     )
   end

data/test/interactors/foreman_netbox/sync_host/sync_virtual_machine/sync_interfaces/sync_ip_addresses/find_test.rb

@@ -23,7 +23,7 @@ class FindVirtualMachineIpAddressesTest < ActiveSupport::TestCase
         status: 200, headers: { 'Content-Type': 'application/json' },
         body: {
           count: 1,
-          results: [{ id: 1 }]
+          results: [{ id: 1 }],
         }.to_json
       )
 
@@ -40,11 +40,11 @@ class FindVirtualMachineIpAddressesTest < ActiveSupport::TestCase
         status: 200, headers: { 'Content-Type': 'application/json' },
         body: {
           count: 0,
-          results: []
+          results: [],
         }.to_json
       )
 
-      assert_equal [], subject.ip_addresses.map(&:id)
+      assert_empty subject.ip_addresses.map(&:id)
       assert_requested(stub_get)
     end
   end

data/test/interactors/foreman_netbox/sync_host/sync_virtual_machine/sync_interfaces/sync_ip_addresses/update_test.rb

@@ -11,19 +11,19 @@ class UpdateVirtualMachineIpAddressesTest < ActiveSupport::TestCase
     )
   end
 
-  let(:ip_addresses) { ForemanNetbox::API.client.ipam.ip_addresses.filter(virtual_machine_id: 1) }
+  let(:ip_addresses) { ForemanNetbox::Api.client.ipam.ip_addresses.filter(virtual_machine_id: 1) }
   let(:ip_addresses_data) do
     [
       {
         id: 1,
         address: host.netbox_facet.netbox_params[:ip_addresses].first[:address],
-        tags: default_tags.map { |t| { id: t.id, name: t.name, slug: t.slug } }
+        tags: default_tags.map { |t| { id: t.id, name: t.name, slug: t.slug } },
       },
       {
         id: 2,
         address: host.netbox_facet.netbox_params[:ip_addresses].second[:address],
-        tags: []
-      }
+        tags: [],
+      },
     ]
   end
   let(:host) do
@@ -37,7 +37,7 @@ class UpdateVirtualMachineIpAddressesTest < ActiveSupport::TestCase
           ip6: '1600:0:2d0:202::17',
           subnet: FactoryBot.build_stubbed(:subnet_ipv4, organizations: [], locations: []),
           subnet6: FactoryBot.build_stubbed(:subnet_ipv6, organizations: [], locations: [])
-        )
+        ),
       ]
     )
   end
@@ -50,7 +50,7 @@ class UpdateVirtualMachineIpAddressesTest < ActiveSupport::TestCase
       status: 200, headers: { 'Content-Type': 'application/json' },
       body: {
         count: ip_addresses_data.count,
-        results: ip_addresses_data
+        results: ip_addresses_data,
       }.to_json
     )
   end
@@ -59,7 +59,7 @@ class UpdateVirtualMachineIpAddressesTest < ActiveSupport::TestCase
     stub_unexpected_patch = stub_request(:patch, "#{Setting[:netbox_url]}/api/ipam/ip-addresses/#{ip_addresses_data.first[:id]}.json")
     stub_expected_patch = stub_request(:patch, "#{Setting[:netbox_url]}/api/ipam/ip-addresses/#{ip_addresses_data.second[:id]}.json").with(
       body: {
-        tags: default_tags.map(&:id)
+        tags: default_tags.map(&:id),
       }.to_json
     ).to_return(
       status: 200, headers: { 'Content-Type': 'application/json' },

data/test/interactors/foreman_netbox/sync_host/sync_virtual_machine/sync_interfaces/update_test.rb

@@ -12,13 +12,13 @@ class UpdateVirtualMachineInterfacesTest < ActiveSupport::TestCase
     )
   end
 
-  let(:interfaces) { ForemanNetbox::API.client::Virtualization::Interfaces.new }
+  let(:interfaces) { ForemanNetbox::Api.client::Virtualization::Interfaces.new }
   let(:old_mac) { 'FE:13:C6:44:29:22' }
   let(:host) do
     FactoryBot.build_stubbed(
       :host,
       interfaces: [
-        FactoryBot.build_stubbed(:nic_base, mac: old_mac)
+        FactoryBot.build_stubbed(:nic_base, mac: old_mac),
       ]
     )
   end
@@ -42,15 +42,15 @@ class UpdateVirtualMachineInterfacesTest < ActiveSupport::TestCase
               id: 1,
               name: host.interfaces.first.netbox_name,
               mac_address: new_mac,
-              tags: []
-            }
-          ]
+              tags: [],
+            },
+          ],
         }.to_json
       )
       stub_patch = stub_request(:patch, "#{Setting[:netbox_url]}/api/virtualization/interfaces/1.json").with(
         body: {
           mac_address: host.interfaces.first.mac.upcase,
-          tags: default_tags.map(&:id)
+          tags: default_tags.map(&:id),
         }.to_json
       ).to_return(
         status: 200, headers: { 'Content-Type': 'application/json' },
@@ -78,9 +78,9 @@ class UpdateVirtualMachineInterfacesTest < ActiveSupport::TestCase
               mac_address: old_mac,
               tags: default_tags.map do |tag|
                 { id: tag.id }
-              end
-            }
-          ]
+              end,
+            },
+          ],
         }.to_json
       )
       stub_patch = stub_request(:patch, "#{Setting[:netbox_url]}/api/virtualization/interfaces/1.json")

data/test/interactors/foreman_netbox/sync_host/sync_virtual_machine/update_test.rb

@@ -16,7 +16,7 @@ class UpdateVirtualMachineTest < ActiveSupport::TestCase
   end
 
   let(:virtual_machine) do
-    ForemanNetbox::API.client::Virtualization::VirtualMachine.new(id: 1).tap do |virtual_machine|
+    ForemanNetbox::Api.client::Virtualization::VirtualMachine.new(id: 1).tap do |virtual_machine|
       virtual_machine.instance_variable_set(
         :@data,
         {
@@ -30,14 +30,14 @@ class UpdateVirtualMachineTest < ActiveSupport::TestCase
           'primary_ip4' => {
             'id' => 1,
             'family' => 4,
-            'address' => '10.0.0.1/24'
+            'address' => '10.0.0.1/24',
           },
           'primary_ip6' => {
             'id' => 2,
             'family' => 6,
-            'address' => '1600:0:2d0:201::18/64'
+            'address' => '1600:0:2d0:201::18/64',
           },
-          'tags' => virtual_machine_tags
+          'tags' => virtual_machine_tags,
         }
       )
     end
@@ -64,7 +64,7 @@ class UpdateVirtualMachineTest < ActiveSupport::TestCase
       )
     )
   end
-  let(:ip_addresses) { ForemanNetbox::API.client.ipam.ip_addresses.filter(virtual_machine_id: virtual_machine.id) }
+  let(:ip_addresses) { ForemanNetbox::Api.client.ipam.ip_addresses.filter(virtual_machine_id: virtual_machine.id) }
   let(:host) do
     FactoryBot.build_stubbed(
       :host,
@@ -78,7 +78,7 @@ class UpdateVirtualMachineTest < ActiveSupport::TestCase
           cpus: virtual_machine_data[:vcpus],
           memory_mb: virtual_machine_data[:memory],
           volumes: [
-            OpenStruct.new(size_gb: virtual_machine_data[:disk])
+            OpenStruct.new(size_gb: virtual_machine_data[:disk]),
           ]
         )
       )
@@ -95,8 +95,8 @@ class UpdateVirtualMachineTest < ActiveSupport::TestCase
         count: 2,
         results: [
           { id: primary_ip4.id, address: primary_ip4.address.address },
-          { id: primary_ip6.id, address: primary_ip6.address.address }
-        ]
+          { id: primary_ip6.id, address: primary_ip6.address.address },
+        ],
       }.to_json
     )
   end
@@ -142,7 +142,7 @@ class UpdateVirtualMachineTest < ActiveSupport::TestCase
             cpus: 4,
             memory_mb: 256,
             volumes: [
-              OpenStruct.new(size_gb: 1024)
+              OpenStruct.new(size_gb: 1024),
             ]
           )
         )
@@ -160,7 +160,7 @@ class UpdateVirtualMachineTest < ActiveSupport::TestCase
           primary_ip6: primary_ip6.id,
           tenant: tenant.id,
           vcpus: host.compute_object.cpus,
-          tags: virtual_machine_tags.map { |t| t['id'] } | default_tags.map(&:id)
+          tags: virtual_machine_tags.map { |t| t['id'] } | default_tags.map(&:id),
         }.to_json
       ).to_return(
         status: 200, headers: { 'Content-Type': 'application/json' },

data/test/models/foreman_netbox/netbox_facet_test.rb

@@ -152,7 +152,7 @@ module ForemanNetbox
                 memory_mb: memory,
                 volumes: [
                   OpenStruct.new(size_gb: disk / 2),
-                  OpenStruct.new(size_gb: disk / 2)
+                  OpenStruct.new(size_gb: disk / 2),
                 ]
               )
             )

data/test/models/foreman_netbox/nics/base_test.rb

@@ -73,7 +73,7 @@ module ForemanNetbox
         context 'without ip and ip6' do
           let(:nic) { FactoryBot.build_stubbed(:nic_base) }
 
-          it { assert_equal [], nic.netbox_ips }
+          it { assert_empty nic.netbox_ips }
         end
       end
     end