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