Project

General

Profile

« Previous | Next » 

Revision a9fffdb1

Added by Ori Rabin about 8 years ago

Fixes #14927 - correct override value requirment

override value should only be required when use_puppet_default is false

View differences:

test/functional/api/v2/override_values_controller_test.rb
end
assert_response :success
end
test "should create override value without when use_puppet_default is true" do
lookup_key = FactoryGirl.create(:puppetclass_lookup_key, :as_smart_class_param, :override => true, :puppetclass => puppetclasses(:two))
assert_difference('LookupValue.count', 1) do
post :create, {:smart_class_parameter_id => lookup_key.id, :override_value => { :match => 'os=string', :use_puppet_default => true}}
end
assert_response :success
end
test "should not create override value without when use_puppet_default is false" do
lookup_key = FactoryGirl.create(:puppetclass_lookup_key, :as_smart_class_param, :override => true, :puppetclass => puppetclasses(:two))
assert_difference('LookupValue.count', 0) do
post :create, {:smart_class_parameter_id => lookup_key.id, :override_value => { :match => 'os=string', :use_puppet_default => false}}
end
assert_response :error
end
end

Also available in: Unified diff