Project

General

Profile

« Previous | Next » 

Revision baa21c79

Added by Ori Rabin over 8 years ago

Fixes #11663 - Empty lookup value not validated when use_puppet_default is checked

View differences:

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