foreman/test/unit/nic_test.rb @ 783fb4aa
8838eb42 | Ohad Levy | require 'test_helper'
|
|
class NicTest < ActiveSupport::TestCase
|
|||
def setup
|
|||
disable_orchestration
|
|||
e07f9a12 | Dominic Cleal | User.current = users :admin
|
|
43c4bd72 | Marek Hulan | ||
@nic = FactoryGirl.build(:nic_managed, :host => FactoryGirl.build(:host, :managed => true))
|
|||
8838eb42 | Ohad Levy | end
|
|
def teardown
|
|||
User.current = nil
|
|||
end
|
|||
test "should create simple interface" do
|
|||
e14b5758 | Greg Sutcliffe | i = Nic::Base.create! :mac => "cabbccddeeff", :host => FactoryGirl.create(:host)
|
|
8838eb42 | Ohad Levy | assert_equal "Nic::Base", i.class.to_s
|
|
end
|
|||
test "type casting should return the correct class" do
|
|||
43c4bd72 | Marek Hulan | i = Nic::Base.create! :ip => "127.2.3.8", :mac => "babbccddeeff", :host => FactoryGirl.create(:host),
|
|
:type => "Nic::Interface"
|
|||
8838eb42 | Ohad Levy | assert_equal "Nic::Interface", i.type
|
|
end
|
|||
test "should fail on invalid mac" do
|
|||
43c4bd72 | Marek Hulan | i = Nic::Base.new :mac => "abccddeeff", :host => FactoryGirl.create(:host, :managed)
|
|
8838eb42 | Ohad Levy | assert !i.valid?
|
|
assert i.errors.keys.include?(:mac)
|
|||
end
|
|||
e42aa400 | Trey Dockendorf | test "should be valid with 64-bit mac address" do
|
|
e14b5758 | Greg Sutcliffe | i = Nic::Base.new :mac => "babbccddeeff00112233445566778899aabbccdd", :host => FactoryGirl.create(:host)
|
|
e42aa400 | Trey Dockendorf | assert i.valid?
|
|
assert !i.errors.keys.include?(:mac)
|
|||
end
|
|||
4bae5ced | Stephen Benjamin | test "should fail on invalid dns name" do
|
|
e14b5758 | Greg Sutcliffe | i = Nic::Managed.new :mac => "dabbccddeeff", :host => FactoryGirl.create(:host), :name => "invalid_dns_name"
|
|
4bae5ced | Stephen Benjamin | assert !i.valid?
|
|
assert i.errors.keys.include?(:name)
|
|||
end
|
|||
8838eb42 | Ohad Levy | test "should fix mac address" do
|
|
e14b5758 | Greg Sutcliffe | interface = Nic::Base.create! :mac => "cabbccddeeff", :host => FactoryGirl.create(:host)
|
|
8838eb42 | Ohad Levy | assert_equal "ca:bb:cc:dd:ee:ff", interface.mac
|
|
end
|
|||
e42aa400 | Trey Dockendorf | test "should fix 64-bit mac address" do
|
|
e14b5758 | Greg Sutcliffe | interface = Nic::Base.create! :mac => "babbccddeeff00112233445566778899aabbccdd", :host => FactoryGirl.create(:host)
|
|
e42aa400 | Trey Dockendorf | assert_equal "ba:bb:cc:dd:ee:ff:00:11:22:33:44:55:66:77:88:99:aa:bb:cc:dd", interface.mac
|
|
end
|
|||
8838eb42 | Ohad Levy | test "should fix ip address if a leading zero is used" do
|
|
43c4bd72 | Marek Hulan | interface = Nic::Interface.create! :ip => "123.01.02.03", :mac => "dabbccddeeff", :host => FactoryGirl.create(:host)
|
|
8838eb42 | Ohad Levy | assert_equal "123.1.2.3", interface.ip
|
|
end
|
|||
test "should delegate subnet attributes" do
|
|||
8f695d94 | Shimon Shtein | subnet = subnets(:two)
|
|
8838eb42 | Ohad Levy | domain = (subnet.domains.any? ? subnet.domains : subnet.domains << Domain.first).first
|
|
e14b5758 | Greg Sutcliffe | interface = Nic::Managed.create! :ip => "2.3.4.127", :mac => "cabbccddeeff", :host => FactoryGirl.create(:host), :subnet => subnet, :name => "a" + FactoryGirl.create(:host).name, :domain => domain
|
|
8838eb42 | Ohad Levy | assert_equal subnet.network, interface.network
|
|
assert_equal subnet.vlanid, interface.vlanid
|
|||
end
|
|||
21e07e92 | Trey Dockendorf | ||
8f695d94 | Shimon Shtein | test "should reject subnet with mismatched taxonomy in host" do
|
|
taxonomy_to_test = [:organization, :location]
|
|||
taxonomy_to_test.each do |taxonomy|
|
|||
tax_object1 = FactoryGirl.build(taxonomy)
|
|||
tax_object2 = FactoryGirl.build(taxonomy)
|
|||
subnet = subnets(:one)
|
|||
host = FactoryGirl.build(:host)
|
|||
subnet_list = subnet.send("#{taxonomy.to_s.pluralize}")
|
|||
subnet_list << tax_object1
|
|||
host.send("#{taxonomy}=",tax_object2)
|
|||
i = Nic::Base.new :mac => "cabbccddeeff", :host => host
|
|||
i.subnet = subnet
|
|||
refute i.valid?
|
|||
end
|
|||
end
|
|||
d6a19253 | Dominic Cleal | test "should ignore subnet with mismatched taxonomy in host when settings disabled" do
|
|
disable_taxonomies do
|
|||
orgs = FactoryGirl.build_pair(:organization)
|
|||
locs = FactoryGirl.build_pair(:location)
|
|||
subn = FactoryGirl.build(:subnet, :locations => [locs.first], :organizations => [orgs.first])
|
|||
host = FactoryGirl.build(:host, :location => locs.last, :organization => orgs.last)
|
|||
nic = Nic::Base.new :mac => "cabbccddeeff", :host => host
|
|||
nic.subnet = subn
|
|||
assert_valid nic
|
|||
end
|
|||
end
|
|||
8f695d94 | Shimon Shtein | test "should accept subnets with aligned location and organization in host" do
|
|
location1 = FactoryGirl.build(:location)
|
|||
organization1 = FactoryGirl.build(:organization)
|
|||
subnet = subnets(:one)
|
|||
host = FactoryGirl.build(:host)
|
|||
subnet.locations << location1
|
|||
subnet.organizations << organization1
|
|||
host.location = location1
|
|||
host.organization = organization1
|
|||
i = Nic::Base.new :mac => "cabbccddeeff", :host => host
|
|||
i.subnet = subnet
|
|||
assert i.valid?
|
|||
end
|
|||
fc2ddea7 | David Swift | test "Nic::Managed#hostname should return blank for blank hostnames" do
|
|
e14b5758 | Greg Sutcliffe | i = Nic::Managed.new :mac => "babbccddeeff00112233445566778899aabbccdd", :host => FactoryGirl.create(:host), :subnet => subnets(:one), :domain => subnets(:one).domains.first, :name => ""
|
|
445dd7ab | Daniel Lobato | assert i.name.blank?
|
|
assert i.domain.present?
|
|||
assert i.hostname.blank?
|
|||
fc2ddea7 | David Swift | end
|
|
4f7a4d0b | David Davis | ||
43c4bd72 | Marek Hulan | test "Mac address uniqueness validation is skipped for virtual NICs and unmanaged hosts" do
|
|
host = FactoryGirl.create(:host, :managed)
|
|||
Nic::Base.create! :mac => "cabbccddeeff", :host => host # physical
|
|||
virtual = Nic::Base.new :mac => "cabbccddeeff", :host => host, :virtual => true
|
|||
d455f32c | Marek Hulan | assert virtual.valid?
|
|
assert virtual.save
|
|||
43c4bd72 | Marek Hulan | another_physical = Nic::Base.new :mac => "cabbccddeeff", :host => host
|
|
d455f32c | Marek Hulan | refute another_physical.save
|
|
43c4bd72 | Marek Hulan | another_physical_on_unmanaged = Nic::Base.new :mac => "cabbccddeeff", :host => FactoryGirl.create(:host)
|
|
assert another_physical_on_unmanaged.save
|
|||
d455f32c | Marek Hulan | end
|
|
57a32e98 | Daniel Lobato | ||
4feb1576 | Marek Hulan | test "VLAN requires identifier" do
|
|
nic = FactoryGirl.build(:nic_managed, :virtual => true, :attached_to => 'eth0', :tag => 5, :managed => true, :identifier => '')
|
|||
refute nic.valid?
|
|||
assert_includes nic.errors.keys, :identifier
|
|||
end
|
|||
test "Alias requires identifier" do
|
|||
nic = FactoryGirl.build(:nic_managed, :virtual => true, :attached_to => 'eth0', :managed => true, :identifier => '')
|
|||
refute nic.valid?
|
|||
assert_includes nic.errors.keys, :identifier
|
|||
end
|
|||
a45253c2 | Marek Hulan | test "#alias? detects alias based on virtual and identifier attributes" do
|
|
nic = FactoryGirl.build(:nic_managed, :virtual => true, :attached_to => 'eth0', :managed => true, :identifier => 'eth0')
|
|||
refute nic.alias?
|
|||
nic.identifier = 'eth0:0'
|
|||
assert nic.alias?
|
|||
nic.virtual = false
|
|||
refute nic.alias?
|
|||
end
|
|||
test "Alias subnet can only use static boot mode if it's managed" do
|
|||
nic = FactoryGirl.build(:nic_managed, :virtual => true, :attached_to => 'eth0', :managed => true, :identifier => 'eth0:0')
|
|||
7678cac5 | Daniel Lobato | nic.host = FactoryGirl.build(:host)
|
|
a45253c2 | Marek Hulan | nic.subnet = FactoryGirl.build(:subnet, :boot_mode => Subnet::BOOT_MODES[:dhcp])
|
|
refute nic.valid?
|
|||
assert_includes nic.errors.keys, :subnet_id
|
|||
nic.subnet.boot_mode = Subnet::BOOT_MODES[:static]
|
|||
nic.valid?
|
|||
refute_includes nic.errors.keys, :subnet_id
|
|||
nic.managed = false
|
|||
nic.subnet.boot_mode = Subnet::BOOT_MODES[:dhcp]
|
|||
nic.valid?
|
|||
refute_includes nic.errors.keys, :subnet_id
|
|||
end
|
|||
4feb1576 | Marek Hulan | test "BMC does not require identifier" do
|
|
nic = FactoryGirl.build(:nic_bmc, :managed => true, :identifier => '')
|
|||
nic.valid?
|
|||
refute_includes nic.errors.keys, :identifier
|
|||
end
|
|||
b3bdb092 | Marek Hulan | test "Bond requires identifier if managed" do
|
|
nic = FactoryGirl.build(:nic_bond, :attached_devices => 'eth0,eth1', :managed => true, :identifier => 'bond0')
|
|||
nic.valid?
|
|||
refute_includes nic.errors.keys, :identifier
|
|||
end
|
|||
test "Bond does not require identifier if not managed" do
|
|||
nic = FactoryGirl.build(:nic_bond, :attached_devices => 'eth0,eth1', :managed => false, :identifier => '')
|
|||
nic.valid?
|
|||
refute_includes nic.errors.keys, :identifier
|
|||
end
|
|||
6d05514a | Tomas Strachota | context 'physical?' do
|
|
test 'returns true for a physical interface' do
|
|||
nic = FactoryGirl.build(:nic_managed, :virtual => false)
|
|||
assert nic.physical?
|
|||
end
|
|||
test 'returns false for a virtual interface' do
|
|||
nic = FactoryGirl.build(:nic_managed, :virtual => true)
|
|||
refute nic.physical?
|
|||
end
|
|||
end
|
|||
57a32e98 | Daniel Lobato | context 'BMC' do
|
|
setup do
|
|||
43c4bd72 | Marek Hulan | disable_orchestration
|
|
@subnet = FactoryGirl.create(:subnet, :dhcp)
|
|||
@domain = FactoryGirl.create(:domain)
|
|||
@interface = FactoryGirl.create(:nic_bmc, :ip => @subnet.unused_ip,
|
|||
:host => FactoryGirl.create(:host),
|
|||
:subnet => @subnet, :domain => @domain, :name => 'bmc')
|
|||
57a32e98 | Daniel Lobato | end
|
|
test 'Nic::BMC should have hostname containing name and domain name' do
|
|||
43c4bd72 | Marek Hulan | assert_equal @interface.hostname, "#{@interface.shortname}.#{@domain.name}"
|
|
57a32e98 | Daniel Lobato | end
|
|
test 'Nic::BMC should have hostname containing name when domain nil' do
|
|||
@interface.domain = nil
|
|||
assert_equal @interface.name, @interface.hostname
|
|||
end
|
|||
test '.proxy uses any BMC SmartProxy if none is found in subnet' do
|
|||
assert @subnet.proxies.select { |proxy| proxy.features.map(&:name).include?('BMC') }
|
|||
assert_equal @interface.proxy.url, SmartProxy.with_features('BMC').first.url + '/bmc'
|
|||
end
|
|||
test '.proxy chooses BMC SmartProxy in Nic::BMC subnet if available' do
|
|||
@subnet.dhcp.features << Feature.find_by_name('BMC')
|
|||
assert_equal @interface.proxy.url, @subnet.dhcp.url + '/bmc'
|
|||
end
|
|||
test '.proxy raises exception if BMC SmartProxy cannot be found' do
|
|||
SmartProxy.with_features('BMC').map(&:destroy)
|
|||
assert_raise Foreman::Exception do
|
|||
@interface.proxy
|
|||
end
|
|||
end
|
|||
43c4bd72 | Marek Hulan | ||
6a2fce1f | Marek Hulan | test "bmc requires MAC address if managed" do
|
|
bmc = FactoryGirl.build(:nic_bmc, :managed => true, :mac => '')
|
|||
refute bmc.valid?
|
|||
assert_includes bmc.errors.keys, :mac
|
|||
end
|
|||
test "bmc does not require MAC address if unmanaged" do
|
|||
bmc = FactoryGirl.build(:nic_bmc, :managed => false, :mac => '')
|
|||
bmc.valid?
|
|||
refute_includes bmc.errors.keys, :mac
|
|||
end
|
|||
43c4bd72 | Marek Hulan | context "on managed host" do
|
|
setup do
|
|||
@host = FactoryGirl.create(:host, :managed, :ip => '127.0.0.1')
|
|||
end
|
|||
test "we can't destroy primary interface of managed host" do
|
|||
interface = @host.primary_interface
|
|||
refute interface.destroy
|
|||
assert_includes interface.errors.keys, :primary
|
|||
end
|
|||
test "we can destroy non primary interface of managed host" do
|
|||
interface = FactoryGirl.create(:nic_managed, :primary => false, :host => @host)
|
|||
assert interface.destroy
|
|||
end
|
|||
test "we can destroy primary interface when deleting the host" do
|
|||
interface = @host.primary_interface
|
|||
refute interface.destroy
|
|||
# we must reload the object (interface contains validation errors preventing deletion)
|
|||
@host.reload
|
|||
assert @host.destroy
|
|||
end
|
|||
test "we can't destroy provision interface of managed host" do
|
|||
interface = @host.provision_interface
|
|||
refute interface.destroy
|
|||
assert_includes interface.errors.keys, :provision
|
|||
end
|
|||
test "we can destroy non provision interface of managed host" do
|
|||
interface = FactoryGirl.create(:nic_managed, :provision => false, :host => @host)
|
|||
assert interface.destroy
|
|||
end
|
|||
test "we can destroy provision interface when deleting the host" do
|
|||
interface = @host.provision_interface
|
|||
refute interface.destroy
|
|||
# we must reload the object (interface contains validtion errors preventin deletion)
|
|||
@host.reload
|
|||
assert @host.destroy
|
|||
end
|
|||
end
|
|||
context "on unmanaged host" do
|
|||
setup do
|
|||
@host = FactoryGirl.create(:host)
|
|||
end
|
|||
test "we can destroy any interface of unmanaged host" do
|
|||
interface = @host.primary_interface
|
|||
assert interface.destroy
|
|||
end
|
|||
test "we can destroy any interface of unmanaged host" do
|
|||
interface = @host.provision_interface
|
|||
assert interface.destroy
|
|||
end
|
|||
test "host can have one primary interface at most" do
|
|||
# factory already created primary interface
|
|||
interface = FactoryGirl.build(:nic_managed, :primary => true, :host => @host)
|
|||
refute interface.save
|
|||
assert_includes interface.errors.keys, :primary
|
|||
interface.primary = false
|
|||
interface.name = ''
|
|||
assert interface.save
|
|||
end
|
|||
test "provision flag is set for primary interface automatically" do
|
|||
primary = FactoryGirl.build(:nic_managed, :primary => true, :provision => false,
|
|||
:domain => FactoryGirl.build(:domain))
|
|||
225cfa5b | Dominic Cleal | host = FactoryGirl.create(:host, :interfaces => [primary])
|
|
assert host.save!
|
|||
43c4bd72 | Marek Hulan | primary.reload
|
|
225cfa5b | Dominic Cleal | assert_equal primary, host.provision_interface
|
|
43c4bd72 | Marek Hulan | end
|
|
end
|
|||
57a32e98 | Daniel Lobato | end
|
|
cad1b13c | Tomas Strachota | ||
context "allowed type registration" do
|
|||
setup do
|
|||
class DefaultTestNic < Nic::Base
|
|||
end
|
|||
class HumanizedTestNic < Nic::Base
|
|||
def self.humanized_name
|
|||
"Custom"
|
|||
end
|
|||
end
|
|||
class DisallowedTestNic < Nic::Base
|
|||
end
|
|||
Nic::Base.register_type(DefaultTestNic)
|
|||
Nic::Base.register_type(HumanizedTestNic)
|
|||
end
|
|||
test "base registers allowed nic types" do
|
|||
expected_types = [DefaultTestNic, HumanizedTestNic]
|
|||
445dd7ab | Daniel Lobato | expected_types.map(&:name).each do |type|
|
|
assert Nic::Base.allowed_types.map(&:name).include? type
|
|||
end
|
|||
cad1b13c | Tomas Strachota | end
|
|
test "type_by_name returns nil for an unknown name" do
|
|||
assert_equal nil, Nic::Base.type_by_name("UNKNOWN_NAME")
|
|||
end
|
|||
test "type_by_name finds the class" do
|
|||
assert_equal HumanizedTestNic, Nic::Base.type_by_name("custom")
|
|||
end
|
|||
test "type_by_name returns nil for classes that aren't allowed" do
|
|||
assert_equal nil, Nic::Base.type_by_name("DisallowedTestNic")
|
|||
end
|
|||
43c4bd72 | Marek Hulan | test 'fqdn_changed? should be true if name changes' do
|
|
@nic.stubs(:name_changed?).returns(true)
|
|||
@nic.stubs(:domain_id_changed?).returns(false)
|
|||
assert @nic.fqdn_changed?
|
|||
end
|
|||
test 'fqdn_changed? should be true if domain changes' do
|
|||
@nic.stubs(:name_changed?).returns(false)
|
|||
@nic.stubs(:domain_id_changed?).returns(true)
|
|||
assert @nic.fqdn_changed?
|
|||
end
|
|||
test 'fqdn_changed? should be true if name and domain change' do
|
|||
@nic.stubs(:name_changed?).returns(true)
|
|||
@nic.stubs(:domain_id_changed?).returns(true)
|
|||
assert @nic.fqdn_changed?
|
|||
end
|
|||
cad1b13c | Tomas Strachota | end
|
|
cbe1391f | Shlomi Zadok | ||
test 'new nic name containing existing domain should set nic domain' do
|
|||
domain = FactoryGirl.create(:domain)
|
|||
host = FactoryGirl.create(:host)
|
|||
nic_name = [host.name, domain.name].join('.')
|
|||
interface = FactoryGirl.create(:nic_managed, :host => host, :name => nic_name)
|
|||
refute_nil(interface.domain)
|
|||
assert_equal(interface.domain, domain)
|
|||
end
|
|||
test 'new nic with non-existing domain should not set nic domain' do
|
|||
host = FactoryGirl.create(:host)
|
|||
nic_name = [host.name, 'domain.name'].join('.')
|
|||
interface = FactoryGirl.create(:nic_managed, :host => host, :name => nic_name)
|
|||
assert_nil(interface.domain)
|
|||
end
|
|||
test 'update nic domain should update nic name' do
|
|||
host = FactoryGirl.create(:host)
|
|||
existing_domain = Domain.first
|
|||
interface = FactoryGirl.create(:nic_managed, :host => host, :name => 'nick')
|
|||
# no domain
|
|||
assert_equal(interface.name, 'nick')
|
|||
interface.update_attributes(:domain_id => existing_domain.id)
|
|||
name_should_be = "nick.#{existing_domain.name}"
|
|||
assert_equal(name_should_be, interface.name)
|
|||
new_domain = FactoryGirl.create(:domain)
|
|||
interface.update_attributes(:domain_id => new_domain.id)
|
|||
name_should_change_to = "nick.#{new_domain.name}"
|
|||
assert_equal(name_should_change_to, interface.name)
|
|||
end
|
|||
4bae5ced | Stephen Benjamin | end
|