Permalink
Browse files

Strengthening the test for nested_attribute

Loading the associate target in nested_attributes
should load most recent attributes for child
records marked for destruction

Signed-off-by: José Valim <jose.valim@gmail.com>
  • Loading branch information...
1 parent 0057d2d commit 96b2516c3cbcf900f2e84163baba3db7cb0e37d9 @subbarao subbarao committed with josevalim Jul 15, 2010
Showing with 20 additions and 1 deletion.
  1. +20 −1 activerecord/test/cases/associations_test.rb
@@ -17,7 +17,8 @@
require 'models/person'
require 'models/reader'
require 'models/parrot'
-require 'models/pirate'
+require 'models/ship_part'
+require 'models/ship'
require 'models/treasure'
require 'models/price_estimate'
require 'models/club'
@@ -29,6 +30,24 @@ class AssociationsTest < ActiveRecord::TestCase
fixtures :accounts, :companies, :developers, :projects, :developers_projects,
:computers, :people, :readers
+ def test_loading_the_association_target_should_keep_child_records_marked_for_destruction
+ ship = Ship.create!(:name => "The good ship Dollypop")
+ part = ship.parts.create!(:name => "Mast")
+ part.mark_for_destruction
+ ship.parts.send(:load_target)
+ assert ship.parts[0].marked_for_destruction?
+ end
+
+ def test_loading_the_association_target_should_load_most_recent_attributes_for_child_records_marked_for_destruction
+ ship = Ship.create!(:name => "The good ship Dollypop")
+ part = ship.parts.create!(:name => "Mast")
+ part.mark_for_destruction
+ ShipPart.find(part.id).update_attribute(:name, 'Deck')
+ ship.parts.send(:load_target)
+ assert_equal 'Deck', ship.parts[0].name
+ end
+
+
def test_include_with_order_works
assert_nothing_raised {Account.find(:first, :order => 'id', :include => :firm)}
assert_nothing_raised {Account.find(:first, :order => :id, :include => :firm)}

0 comments on commit 96b2516

Please sign in to comment.