@@ -57,7 +57,7 class CustomField < ActiveRecord::Base | |||
|
57 | 57 | end |
|
58 | 58 | end |
|
59 | 59 | |
|
60 |
|
|
|
60 | if default_value.present? && !valid_field_value?(default_value) | |
|
61 | 61 | errors.add(:default_value, :invalid) |
|
62 | 62 | end |
|
63 | 63 | end |
@@ -52,6 +52,11 class CustomFieldTest < ActiveSupport::TestCase | |||
|
52 | 52 | assert field.valid? |
|
53 | 53 | end |
|
54 | 54 | |
|
55 | def test_default_value_should_not_be_validated_when_blank | |
|
56 | field = CustomField.new(:name => 'Test', :field_format => 'list', :possible_values => ['a', 'b'], :is_required => true, :default_value => '') | |
|
57 | assert field.valid? | |
|
58 | end | |
|
59 | ||
|
55 | 60 | def test_possible_values_should_accept_an_array |
|
56 | 61 | field = CustomField.new |
|
57 | 62 | field.possible_values = ["One value", ""] |
General Comments 0
You need to be logged in to leave comments.
Login now