diff options
| author | Scott Willson <scott.willson@gmail.com> | 2012-10-16 19:04:40 -0700 |
|---|---|---|
| committer | Scott Willson <scott.willson@gmail.com> | 2012-10-16 19:04:40 -0700 |
| commit | 25262bc280e8c9c0e875315958f82230b67cbf35 (patch) | |
| tree | 48baee2cc9088c4eed11696e9c280718cf175211 /activerecord/test/cases | |
| parent | c432c74cd34d4230bfda6fa008f2af9dbc33e523 (diff) | |
| download | rails-25262bc280e8c9c0e875315958f82230b67cbf35.tar.gz rails-25262bc280e8c9c0e875315958f82230b67cbf35.tar.bz2 rails-25262bc280e8c9c0e875315958f82230b67cbf35.zip | |
Fix bug with presence validation of associations.
Would incorrectly add duplicated errors when the association was blank. Bug introduced in 1fab518c6a75dac5773654646eb724a59741bc13.
Diffstat (limited to 'activerecord/test/cases')
| -rw-r--r-- | activerecord/test/cases/validations/presence_validation_test.rb | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/activerecord/test/cases/validations/presence_validation_test.rb b/activerecord/test/cases/validations/presence_validation_test.rb index cd9175f454..1de8934406 100644 --- a/activerecord/test/cases/validations/presence_validation_test.rb +++ b/activerecord/test/cases/validations/presence_validation_test.rb @@ -18,6 +18,13 @@ class PresenceValidationTest < ActiveRecord::TestCase assert b.valid? end + def test_validates_presence_of_has_one + Boy.validates_presence_of(:face) + b = Boy.new + assert b.invalid?, "should not be valid if has_one association missing" + assert_equal 1, b.errors[:face].size, "validates_presence_of should only add one error" + end + def test_validates_presence_of_has_one_marked_for_destruction Boy.validates_presence_of(:face) b = Boy.new |