diff options
author | Dieter Komendera <dieter@komendera.com> | 2011-07-04 18:39:25 +0200 |
---|---|---|
committer | Dieter Komendera <dieter@komendera.com> | 2011-07-04 18:43:09 +0200 |
commit | 7d3aa2462ce83abe5824c4abc1c326f0c52c1f03 (patch) | |
tree | d7a8eadb1e16f59798612512c2473e88455b99f0 /activerecord/test | |
parent | 718542bb182af38d225384b26adb820c7589ea9a (diff) | |
download | rails-7d3aa2462ce83abe5824c4abc1c326f0c52c1f03.tar.gz rails-7d3aa2462ce83abe5824c4abc1c326f0c52c1f03.tar.bz2 rails-7d3aa2462ce83abe5824c4abc1c326f0c52c1f03.zip |
Only call set_owner_attributes for has_one association if target exists.
Diffstat (limited to 'activerecord/test')
-rw-r--r-- | activerecord/test/cases/associations/has_one_associations_test.rb | 11 |
1 files changed, 11 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 a1d0dafa57..26931e3e85 100644 --- a/activerecord/test/cases/associations/has_one_associations_test.rb +++ b/activerecord/test/cases/associations/has_one_associations_test.rb @@ -345,6 +345,17 @@ class HasOneAssociationsTest < ActiveRecord::TestCase assert orig_ship.destroyed? end + def test_creation_failure_due_to_new_record_should_raise_error + pirate = pirates(:redbeard) + new_ship = Ship.new + + assert_raise(ActiveRecord::RecordNotSaved) do + pirate.ship = new_ship + end + assert_nil pirate.ship + assert_nil new_ship.pirate_id + end + def test_replacement_failure_due_to_existing_record_should_raise_error pirate = pirates(:blackbeard) pirate.ship.name = nil |