diff options
| author | Jon Leighton <j@jonathanleighton.com> | 2011-01-08 19:39:40 +0000 |
|---|---|---|
| committer | Aaron Patterson <aaron.patterson@gmail.com> | 2011-01-11 13:45:06 -0800 |
| commit | 1bc71ed9607ba88c21c14b670a4308c8549f3941 (patch) | |
| tree | 89acbe329f1c54cddc192b42e2115ed6abec3e15 /activerecord/test/cases | |
| parent | c47c54140246f4e5b49376ae9c408c85968ed6c3 (diff) | |
| download | rails-1bc71ed9607ba88c21c14b670a4308c8549f3941.tar.gz rails-1bc71ed9607ba88c21c14b670a4308c8549f3941.tar.bz2 rails-1bc71ed9607ba88c21c14b670a4308c8549f3941.zip | |
When assigning a has_one, if the existing record fails to be removed from the association, raise an error
Diffstat (limited to 'activerecord/test/cases')
| -rw-r--r-- | activerecord/test/cases/associations/has_one_associations_test.rb | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/activerecord/test/cases/associations/has_one_associations_test.rb b/activerecord/test/cases/associations/has_one_associations_test.rb index f004d46c98..46459df7c5 100644 --- a/activerecord/test/cases/associations/has_one_associations_test.rb +++ b/activerecord/test/cases/associations/has_one_associations_test.rb @@ -308,4 +308,16 @@ class HasOneAssociationsTest < ActiveRecord::TestCase assert new_ship.new_record? assert orig_ship.destroyed? end + + def test_replacement_failure_due_to_existing_record_should_raise_error + pirate = pirates(:blackbeard) + pirate.ship.name = nil + + assert !pirate.ship.valid? + assert_raise(ActiveRecord::RecordNotSaved) do + pirate.ship = ships(:interceptor) + end + assert_equal ships(:black_pearl), pirate.ship + assert_equal pirate.id, pirate.ship.pirate_id + end end |