Revision baa21c79
Added by Ori Rabin over 8 years ago
app/models/lookup_value.rb | ||
---|---|---|
scoped_search :in => :lookup_key, :on => :key, :rename => :lookup_key, :complete_value => true
|
||
|
||
def value_present?
|
||
self.errors.add(:value, :blank) if value.nil?
|
||
self.errors.add(:value, :blank) if value.nil? && !use_puppet_default
|
||
end
|
||
|
||
def value=(val)
|
test/unit/lookup_value_test.rb | ||
---|---|---|
refute value.valid?
|
||
end
|
||
|
||
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
|
||
|
||
test "lookup value should allow valid key" do
|
||
key = lookup_keys(:three)
|
||
value = LookupValue.new(:value => true, :match => "hostgroup=Common", :lookup_key_id => key.id)
|
Also available in: Unified diff
Fixes #11663 - Empty lookup value not validated when use_puppet_default is checked