foreman/test/unit/lookup_value_test.rb @ 80692275
| 6c492cce | Joseph Mitchell Magen | require 'test_helper'
|
|
| 832c0925 | Joseph Magen | class LookupValueTest < ActiveSupport::TestCase
|
|
| e14b5758 | Greg Sutcliffe | def setup
|
|
@host1 = FactoryGirl.create(:host)
|
|||
@host2 = FactoryGirl.create(:host)
|
|||
| 6c492cce | Joseph Mitchell Magen | end
|
|
| e14b5758 | Greg Sutcliffe | def valid_attrs1
|
|
{ :match => "fqdn=#{@host2.name}",
|
|||
| 6c492cce | Joseph Mitchell Magen | :value => "3001",
|
|
:lookup_key_id => lookup_keys(:one).id
|
|||
}
|
|||
end
|
|||
| e14b5758 | Greg Sutcliffe | def valid_attrs2
|
|
| 6c492cce | Joseph Mitchell Magen | { :match => "hostgroup=Common",
|
|
:value => "3001",
|
|||
:lookup_key_id => lookup_keys(:one).id
|
|||
}
|
|||
end
|
|||
test "create lookup value by admin" do
|
|||
as_admin do
|
|||
assert_difference('LookupValue.count') do
|
|||
| e14b5758 | Greg Sutcliffe | LookupValue.create!(valid_attrs1)
|
|
| 6c492cce | Joseph Mitchell Magen | end
|
|
end
|
|||
end
|
|||
test "update lookup value by admin" do
|
|||
| e14b5758 | Greg Sutcliffe | lookup_value = lookup_values(:hostgroupcommon)
|
|
| 6c492cce | Joseph Mitchell Magen | as_admin do
|
|
assert lookup_value.update_attributes!(:value => "9000")
|
|||
end
|
|||
end
|
|||
| fbbda128 | Marek Hulan | test "non-admin user can view only his hosts allowed by filters" do
|
|
| acfbc458 | Marek Hulan | # Host.authorized(:view_hosts, Host) returns only hosts(:one)
|
|
| 6c492cce | Joseph Mitchell Magen | user = users(:one)
|
|
| acfbc458 | Marek Hulan | role = FactoryGirl.create(:role, :name => 'user_view_host_by_ip')
|
|
| 43c4bd72 | Marek Hulan | FactoryGirl.create(:filter, :role => role, :permissions => [Permission.find_by_name(:view_hosts)], :search => "name = #{@host1.name}")
|
|
# Todo, restore the ip test variant once our scoped-search works with host.ip again
|
|||
# FactoryGirl.create(:filter, :role => role, :permissions => [Permission.find_by_name(:view_hosts)], :search => "ip = #{@host1.ip}")
|
|||
| acfbc458 | Marek Hulan | user.roles<< [ role ]
|
|
| 6c492cce | Joseph Mitchell Magen | as_user :one do
|
|
| e14b5758 | Greg Sutcliffe | assert Host.authorized(:view_hosts, Host).where(:name => @host1.name).exists?
|
|
refute Host.authorized(:view_hosts, Host).where(:name => @host2.name).exists?
|
|||
| 6c492cce | Joseph Mitchell Magen | end
|
|
end
|
|||
| bd48d5c9 | Joseph Magen | test "any user including admin cannot create lookup value if match fqdn= does not match existing host" do
|
|
as_admin do
|
|||
attrs = { :match => "fqdn=non.existing.com", :value => "123", :lookup_key_id => lookup_keys(:one).id }
|
|||
lookup_value = LookupValue.new(attrs)
|
|||
refute lookup_value.save
|
|||
assert_match /Match fqdn=non.existing.com does not match an existing host/, lookup_value.errors.full_messages.join("\n")
|
|||
end
|
|||
end
|
|||
test "any user including admin cannot create lookup value if match hostgroup= does not match existing hostgroup" do
|
|||
as_admin do
|
|||
attrs = { :match => "hostgroup=non_existing_group", :value => "123", :lookup_key_id => lookup_keys(:one).id }
|
|||
lookup_value = LookupValue.new(attrs)
|
|||
refute lookup_value.save
|
|||
assert_match /Match hostgroup=non_existing_group does not match an existing host group/, lookup_value.errors.full_messages.join("\n")
|
|||
end
|
|||
end
|
|||
| 6c492cce | Joseph Mitchell Magen | test "can create lookup value if user has matching hostgroup " do
|
|
as_user :one do
|
|||
| e14b5758 | Greg Sutcliffe | lookup_value = LookupValue.new(valid_attrs2)
|
|
| 6c492cce | Joseph Mitchell Magen | assert_difference('LookupValue.count') do
|
|
assert lookup_value.save
|
|||
end
|
|||
end
|
|||
end
|
|||
| cfc2bb66 | Stephen Benjamin | test "smart class parameter accepts valid data" do
|
|
as_admin do
|
|||
lk = LookupValue.new(:value => "---\nfoo:\n bar: baz", :match => "hostgroup=Common", :lookup_key => lookup_keys(:six))
|
|||
assert lk.valid?
|
|||
assert lk.save!
|
|||
end
|
|||
end
|
|||
test "smart class parameter validation detects invalid data" do
|
|||
as_admin do
|
|||
lk = LookupValue.new(:value => "---\n[[\n;", :match => "hostgroup=Common", :lookup_key => lookup_keys(:six))
|
|||
refute lk.valid?
|
|||
assert lk.errors.messages[:value].include? "is invalid yaml"
|
|||
end
|
|||
end
|
|||
| 801a90d1 | Stephen Benjamin | test "should cast and uncast string containing a Hash" do
|
|
lk1 = LookupValue.new(:value => "---\n foo: bar", :match => "hostgroup=Common", :lookup_key => lookup_keys(:six))
|
|||
assert lk1.save!
|
|||
assert lk1.value.is_a? Hash
|
|||
| f8a56f5b | Marek Hulan | assert_include lk1.value_before_type_cast, "foo: bar"
|
|
| 801a90d1 | Stephen Benjamin | ||
lk2 = LookupValue.new(:value => "{'foo': 'bar'}", :match => "environment=Production", :lookup_key => lookup_keys(:six))
|
|||
assert lk2.save!
|
|||
assert lk2.value.is_a? Hash
|
|||
| f8a56f5b | Marek Hulan | assert_include lk2.value_before_type_cast, "foo: bar"
|
|
| 801a90d1 | Stephen Benjamin | end
|
|
test "should cast and uncast string containing an Array" do
|
|||
lk = LookupValue.new(:value => "[{\"foo\":\"bar\"},{\"baz\":\"qux\"},\"baz\"]", :match => "hostgroup=Common", :lookup_key => lookup_keys(:seven))
|
|||
assert lk.save!
|
|||
assert lk.value.is_a? Array
|
|||
assert_equal lk.value_before_type_cast, "[{\"foo\":\"bar\"},{\"baz\":\"qux\"},\"baz\"]"
|
|||
end
|
|||
| 971fee79 | Dominic Cleal | ||
test "when created, an audit entry should be added" do
|
|||
env = FactoryGirl.create(:environment)
|
|||
pc = FactoryGirl.create(:puppetclass, :with_parameters, :environments => [env])
|
|||
key = pc.class_params.first
|
|||
lvalue = nil
|
|||
assert_difference('Audit.count') do
|
|||
lvalue = FactoryGirl.create :lookup_value, :lookup_key_id => key.id, :value => 'test', :match => 'foo=bar'
|
|||
end
|
|||
assert_equal "#{pc.name}::#{key.key}", lvalue.audits.last.associated_name
|
|||
end
|
|||
test "when changed, an audit entry should be added" do
|
|||
env = FactoryGirl.create(:environment)
|
|||
pc = FactoryGirl.create(:puppetclass, :environments => [env])
|
|||
key = FactoryGirl.create(:lookup_key, :as_smart_class_param, :with_override, :puppetclass => pc)
|
|||
lvalue = key.lookup_values.first
|
|||
assert_difference('Audit.count') do
|
|||
lvalue.value = 'new overridden value'
|
|||
lvalue.save!
|
|||
end
|
|||
assert_equal "#{pc.name}::#{key.key}", lvalue.audits.last.associated_name
|
|||
end
|
|||
| f8a56f5b | Marek Hulan | ||
test "shuld not cast string with erb" do
|
|||
key = FactoryGirl.create(:lookup_key, :as_smart_class_param,
|
|||
:override => true, :key_type => 'array', :merge_overrides => true, :avoid_duplicates => true,
|
|||
:default_value => [1,2,3], :puppetclass => puppetclasses(:one))
|
|||
lv = LookupValue.new(:value => "<%= [4,5,6] %>", :match => "hostgroup=Common", :lookup_key => key)
|
|||
# does not cast on save (validate_and_cast_value)
|
|||
assert lv.save!
|
|||
# does not cast on load (value_before_type_cast)
|
|||
assert_equal lv.value_before_type_cast, "<%= [4,5,6] %>"
|
|||
assert_equal lv.value, "<%= [4,5,6] %>"
|
|||
end
|
|||
| ee6fc204 | Tom Caspy | ||
test "boolean lookup value should allow for false value" do
|
|||
#boolean key
|
|||
key = lookup_keys(:three)
|
|||
value = LookupValue.new(:value => false, :match => "hostgroup=Common", :lookup_key_id => key.id)
|
|||
assert value.valid?
|
|||
end
|
|||
test "boolean lookup value should not allow for nil" do
|
|||
#boolean key
|
|||
key = lookup_keys(:three)
|
|||
value = LookupValue.new(:value => nil, :match => "hostgroup=Common", :lookup_key_id => key.id)
|
|||
refute value.valid?
|
|||
end
|
|||
| 92eb3d7c | Ori Rabin | ||
| 80692275 | Ori Rabin | test "boolean lookup value should allow nil value if use_puppet_default is true" do
|
|
#boolean key
|
|||
key = lookup_keys(:three)
|
|||
value = LookupValue.new(:value => nil, :match => "hostgroup=Common", :lookup_key_id => key.id, :use_puppet_default => true)
|
|||
assert_valid value
|
|||
end
|
|||
| 92eb3d7c | Ori Rabin | context "when key is a boolean and default_value is a string" do
|
|
def setup
|
|||
@key = FactoryGirl.create(:lookup_key, :as_smart_class_param,
|
|||
:override => true, :key_type => 'boolean',
|
|||
:default_value => 'whatever', :puppetclass => puppetclasses(:one), :use_puppet_default => true)
|
|||
@value = LookupValue.new(:value => 'abc', :match => "hostgroup=Common", :lookup_key_id => @key.id, :use_puppet_default => true)
|
|||
end
|
|||
test "value is not validated if use_puppet_default is true" do
|
|||
assert_valid @value
|
|||
end
|
|||
test "value is validated if use_puppet_default is false" do
|
|||
@value.use_puppet_default = false
|
|||
refute_valid @value
|
|||
end
|
|||
end
|
|||
| 6c492cce | Joseph Mitchell Magen | end
|