foreman_netbox 1.0.2

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

@@ -0,0 +1,51 @@
+# frozen_string_literal: true
+
+require 'test_plugin_helper'
+
+class CreateSiteTest < ActiveSupport::TestCase
+  subject do
+    ForemanNetbox::SyncHost::SyncDevice::SyncSite::Create.call(
+      host: host, site: site, netbox_params: host.netbox_facet.netbox_params
+    )
+  end
+
+  let(:site_id) { 1 }
+  let(:host) do
+    FactoryBot.build_stubbed(
+      :host,
+      location: FactoryBot.build_stubbed(:location)
+    )
+  end
+
+  setup do
+    setup_default_netbox_settings
+  end
+
+  context 'when site is not assigned to the context' do
+    let(:site) { nil }
+
+    it 'assigns site to context' do
+      stub_post = stub_request(:post, "#{Setting[:netbox_url]}/api/dcim/sites/").with(
+        body: {
+          name: host.location.netbox_site_name,
+          slug: host.location.netbox_site_slug,
+          tags: ForemanNetbox::NetboxParameters::DEFAULT_TAGS
+        }.to_json
+      ).to_return(
+        status: 201, headers: { 'Content-Type': 'application/json' },
+        body: { id: site_id }.to_json
+      )
+
+      assert_equal site_id, subject.site.id
+      assert_requested(stub_post)
+    end
+  end
+
+  context 'when site is already assigned to the context' do
+    let(:site) { OpenStruct.new(id: site_id) }
+
+    it 'does not change site' do
+      assert_equal site_id, subject.site.id
+    end
+  end
+end

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

@@ -0,0 +1,59 @@
+# frozen_string_literal: true
+
+require 'test_plugin_helper'
+
+class FindSiteTest < ActiveSupport::TestCase
+  subject do
+    ForemanNetbox::SyncHost::SyncDevice::SyncSite::Find.call(
+      host: host,
+      netbox_params: host.netbox_facet.netbox_params
+    )
+  end
+
+  let(:host) do
+    FactoryBot.build_stubbed(
+      :host,
+      location: FactoryBot.build_stubbed(:location)
+    )
+  end
+
+  setup do
+    setup_default_netbox_settings
+  end
+
+  context 'when site exists in Netbox' do
+    it 'assigns site to context' do
+      stub_get = stub_request(:get, "#{Setting[:netbox_url]}/api/dcim/sites.json").with(
+        query: { limit: 50, slug: host.location.netbox_site_slug }
+      ).to_return(
+        status: 200, headers: { 'Content-Type': 'application/json' },
+        body: {
+          count: 1,
+          results: [
+            { id: 1, name: host.location.netbox_site_name, slug: host.location.netbox_site_slug }
+          ]
+        }.to_json
+      )
+
+      assert_equal 1, subject.site.id
+      assert_requested(stub_get)
+    end
+  end
+
+  context 'when site does not exist in NetBox' do
+    it 'does not assign site to context' do
+      stub_get = stub_request(:get, "#{Setting[:netbox_url]}/api/dcim/sites.json").with(
+        query: { limit: 50, slug: host.location.netbox_site_slug }
+      ).to_return(
+        status: 200, headers: { 'Content-Type': 'application/json' },
+        body: {
+          count: 0,
+          results: []
+        }.to_json
+      )
+
+      assert_nil subject.site
+      assert_requested(stub_get)
+    end
+  end
+end

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

@@ -0,0 +1,54 @@
+# frozen_string_literal: true
+
+require 'test_plugin_helper'
+
+class UpdateSiteTest < ActiveSupport::TestCase
+  subject do
+    ForemanNetbox::SyncHost::SyncDevice::SyncSite::Update.call(
+      host: host, site: site, netbox_params: host.netbox_facet.netbox_params
+    )
+  end
+
+  let(:host) { FactoryBot.build_stubbed(:host) }
+  let(:site) do
+    ForemanNetbox::API.client::DCIM::Site.new(id: 1).tap do |site|
+      site.instance_variable_set(
+        :@data,
+        { 'id' => 1, 'tags' => site_tags }
+      )
+    end
+  end
+
+  setup do
+    setup_default_netbox_settings
+  end
+
+  context 'when changed' do
+    let(:site_tags) { [] }
+
+    it 'updates site' do
+      stub_patch = stub_request(:patch, "#{Setting[:netbox_url]}/api/dcim/sites/1.json").with(
+        body: {
+          tags: host.netbox_facet.netbox_params.dig(:site, :tags)
+        }.to_json
+      ).to_return(
+        status: 200, headers: { 'Content-Type': 'application/json' },
+        body: { id: 1 }.to_json
+      )
+
+      assert subject.success?
+      assert_requested stub_patch
+    end
+  end
+
+  context 'when unchanged' do
+    let(:site_tags) { host.netbox_facet.netbox_params.dig(:site, :tags) }
+
+    it 'does not update site' do
+      stub_patch = stub_request(:patch, "#{Setting[:netbox_url]}/api/dcim/sites/1.json")
+
+      assert subject.success?
+      assert_not_requested stub_patch
+    end
+  end
+end

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

@@ -0,0 +1,188 @@
+# frozen_string_literal: true
+
+require 'test_plugin_helper'
+
+class UpdateDeviceTest < ActiveSupport::TestCase
+  subject do
+    ForemanNetbox::SyncHost::SyncDevice::Update.call(
+      device: device,
+      host: host,
+      netbox_params: host.netbox_facet.netbox_params,
+      device_role: device_role,
+      device_type: device_type,
+      site: site,
+      tenant: tenant,
+      ip_addresses: ip_addresses
+    )
+  end
+
+  let(:device) do
+    ForemanNetbox::API.client::DCIM::Device.new(id: 1).tap do |device|
+      device.instance_variable_set(
+        :@data,
+        {
+          'id' => 1,
+          'name' => device_name,
+          'device_role' => { 'id' => 1 },
+          'device_type' => { 'id' => 1 },
+          'site' => { 'id' => 1 },
+          'tenant' => { 'id' => 1 },
+          'serial' => 'old123',
+          'primary_ip4' => {
+            'id' => 1,
+            'family' => 4,
+            'address' => '10.0.0.8/24'
+          },
+          'primary_ip6' => {
+            'id' => 2,
+            'family' => 6,
+            'address' => '1600:0:2d0:202::18/64'
+          },
+          'tags' => device_tags
+        }
+      )
+    end
+  end
+  let(:device_name) { 'name.example.com' }
+  let(:device_tags) { ['tag'] }
+  let(:device_data) { device.instance_variable_get(:@data).deep_symbolize_keys }
+  let(:device_role) { OpenStruct.new(id: device_data.dig(:device_role, :id)) }
+  let(:device_type) { OpenStruct.new(id: device_data.dig(:device_type, :id)) }
+  let(:site) { OpenStruct.new(id: device_data.dig(:site, :id)) }
+  let(:tenant) { OpenStruct.new(id: device_data.dig(:tenant, :id)) }
+  let(:primary_ip4) do
+    OpenStruct.new(
+      id: device_data.dig(:primary_ip4, :id),
+      address: OpenStruct.new(
+        address: device_data.dig(:primary_ip4, :address).split('/')[0]
+      )
+    )
+  end
+  let(:primary_ip6) do
+    OpenStruct.new(
+      id: device_data.dig(:primary_ip6, :id),
+      address: OpenStruct.new(
+        address: device_data.dig(:primary_ip6, :address).split('/')[0]
+      )
+    )
+  end
+  let(:serialnumber) { device_data[:serial] }
+  let(:ip_addresses) { ForemanNetbox::API.client.ipam.ip_addresses.filter(device_id: device.id) }
+  let(:host) do
+    FactoryBot.build_stubbed(
+      :host,
+      hostname: device_name,
+      ip: primary_ip4.address.address,
+      ip6: primary_ip6.address.address
+    ).tap do |host|
+      host.stubs(:facts).returns({ 'serialnumber' => serialnumber })
+    end
+  end
+
+  setup do
+    setup_default_netbox_settings
+    stub_request(:get, "#{Setting[:netbox_url]}/api/ipam/ip-addresses.json").with(
+      query: { limit: 50, device_id: device.id }
+    ).to_return(
+      status: 200, headers: { 'Content-Type': 'application/json' },
+      body: {
+        count: 2,
+        results: [
+          { id: primary_ip4.id, address: primary_ip4.address.address },
+          { id: primary_ip6.id, address: primary_ip6.address.address }
+        ]
+      }.to_json
+    )
+  end
+
+  context 'if the host has not been updated since the last synchronization' do
+    let(:device_tags) { ForemanNetbox::NetboxParameters::DEFAULT_TAGS }
+
+    it 'does not update device' do
+      stub_patch = stub_request(:patch, "#{Setting[:netbox_url]}/api/dcim/devices/#{device.id}.json")
+
+      assert_equal device, subject.device
+      assert_not_requested(stub_patch)
+    end
+  end
+
+  context 'if the host has been updated since the last synchronization' do
+    let(:device_role) { OpenStruct.new(id: 2) }
+    let(:device_type) { OpenStruct.new(id: 2) }
+    let(:site) { OpenStruct.new(id: 2) }
+    let(:tenant) { OpenStruct.new(id: 2) }
+    let(:serialnumber) { 'new123' }
+    let(:primary_ip4) do
+      OpenStruct.new(
+        id: 3,
+        address: OpenStruct.new(
+          address: '10.0.0.2'
+        )
+      )
+    end
+    let(:primary_ip6) do
+      OpenStruct.new(
+        id: 4,
+        address: OpenStruct.new(
+          address: '1500:0:2d0:201::2'
+        )
+      )
+    end
+    let(:host) do
+      FactoryBot.build_stubbed(
+        :host,
+        ip: primary_ip4.address.address,
+        ip6: primary_ip6.address.address
+      ).tap do |host|
+        host.stubs(:facts).returns({ 'serialnumber' => serialnumber })
+      end
+    end
+
+    it 'updates device' do
+      stub_patch = stub_request(:patch, "#{Setting[:netbox_url]}/api/dcim/devices/#{device.id}.json").with(
+        body: {
+          name: host.name,
+          device_role: device_role.id,
+          device_type: device_type.id,
+          primary_ip4: primary_ip4.id,
+          primary_ip6: primary_ip6.id,
+          site: site.id,
+          tenant: tenant.id,
+          serial: serialnumber,
+          tags: device_tags | ForemanNetbox::NetboxParameters::DEFAULT_TAGS
+        }.to_json
+      ).to_return(
+        status: 200, headers: { 'Content-Type': 'application/json' },
+        body: { id: 1 }.to_json
+      )
+
+      subject
+      assert_requested(stub_patch)
+    end
+
+    context 'when serialnumber is empty' do
+      let(:serialnumber) { nil }
+
+      it 'updates device' do
+        stub_patch = stub_request(:patch, "#{Setting[:netbox_url]}/api/dcim/devices/#{device.id}.json").with(
+          body: {
+            name: host.name,
+            device_role: device_role.id,
+            device_type: device_type.id,
+            primary_ip4: primary_ip4.id,
+            primary_ip6: primary_ip6.id,
+            site: site.id,
+            tenant: tenant.id,
+            tags: device_tags | ForemanNetbox::NetboxParameters::DEFAULT_TAGS
+          }.to_json
+        ).to_return(
+          status: 200, headers: { 'Content-Type': 'application/json' },
+          body: { id: 1 }.to_json
+        )
+
+        subject
+        assert_requested(stub_patch)
+      end
+    end
+  end
+end

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

@@ -0,0 +1,50 @@
+# frozen_string_literal: true
+
+require 'test_plugin_helper'
+
+class CreateTenantTest < ActiveSupport::TestCase
+  subject do
+    ForemanNetbox::SyncHost::SyncTenant::Create.call(
+      host: host, tenant: tenant, netbox_params: host.netbox_facet.netbox_params
+    )
+  end
+
+  let(:host) do
+    FactoryBot.build_stubbed(
+      :host,
+      owner: FactoryBot.build_stubbed(:usergroup, name: 'Owner')
+    )
+  end
+
+  setup do
+    setup_default_netbox_settings
+  end
+
+  context 'when tenant is not assigned to the context' do
+    let(:tenant) { nil }
+
+    it 'assigns tenant to context' do
+      stub_post = stub_request(:post, "#{Setting[:netbox_url]}/api/tenancy/tenants/").with(
+        body: {
+          name: host.owner.name,
+          slug: host.owner.name.parameterize,
+          tags: ForemanNetbox::NetboxParameters::DEFAULT_TAGS
+        }.to_json
+      ).to_return(
+        status: 201, headers: { 'Content-Type': 'application/json' },
+        body: { id: 1 }.to_json
+      )
+
+      assert_equal 1, subject.tenant.id
+      assert_requested(stub_post)
+    end
+  end
+
+  context 'when tenant is already assigned to the context' do
+    let(:tenant) { OpenStruct.new }
+
+    it 'does not change tenant' do
+      assert_equal tenant, subject.tenant
+    end
+  end
+end

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

@@ -0,0 +1,58 @@
+# frozen_string_literal: true
+
+require 'test_plugin_helper'
+
+class FindTenantTest < ActiveSupport::TestCase
+  subject do
+    ForemanNetbox::SyncHost::SyncTenant::Find.call(
+      host: host, netbox_params: host.netbox_facet.netbox_params
+    )
+  end
+
+  let(:host) do
+    FactoryBot.build_stubbed(
+      :host,
+      owner: FactoryBot.build_stubbed(:usergroup, name: 'Owner')
+    )
+  end
+
+  setup do
+    setup_default_netbox_settings
+  end
+
+  context 'when tenant exists in Netbox' do
+    it 'assigns tenant to context' do
+      stub_get = stub_request(:get, "#{Setting[:netbox_url]}/api/tenancy/tenants.json").with(
+        query: { limit: 50, slug: host.owner.name.parameterize }
+      ).to_return(
+        status: 200, headers: { 'Content-Type': 'application/json' },
+        body: {
+          count: 1,
+          results: [
+            { id: 1, name: host.owner.name, slug: host.owner.name.parameterize }
+          ]
+        }.to_json
+      )
+
+      assert_equal 1, subject.tenant.id
+      assert_requested(stub_get)
+    end
+  end
+
+  context 'when tenant does not exist in NetBox' do
+    it 'does not assign tenant to context' do
+      stub_get = stub_request(:get, "#{Setting[:netbox_url]}/api/tenancy/tenants.json").with(
+        query: { limit: 50, slug: host.owner.name.parameterize }
+      ).to_return(
+        status: 200, headers: { 'Content-Type': 'application/json' },
+        body: {
+          count: 0,
+          results: []
+        }.to_json
+      )
+
+      assert_nil subject.tenant
+      assert_requested(stub_get)
+    end
+  end
+end