Permalink
Browse files

Fix for #5579 involved the code change for both has_one and has_many …

…relationships. The path included test only for has_one. This patch adds test for has_many relationship.

[#5706 state:resolved]

Signed-off-by: José Valim <jose.valim@gmail.com>
  • Loading branch information...
1 parent 7f74323 commit 4966b915fe96db73933d33176c1ea5cc53e58a22 Neeraj Singh committed with josevalim Sep 26, 2010
Showing with 9 additions and 1 deletion.
  1. +9 −1 activerecord/test/cases/nested_attributes_test.rb
@@ -115,14 +115,22 @@ def test_reject_if_with_indifferent_keys
assert_difference('Ship.count') { pirate.save! }
end
- def test_reject_if_with_a_proc_which_returns_true_always
+ def test_reject_if_with_a_proc_which_returns_true_always_for_has_one
Pirate.accepts_nested_attributes_for :ship, :reject_if => proc {|attributes| true }
pirate = Pirate.new(:catchphrase => "Stop wastin' me time")
ship = pirate.create_ship(:name => 's1')
pirate.update_attributes({:ship_attributes => { :name => 's2', :id => ship.id } })
assert_equal 's1', ship.reload.name
end
+ def test_reject_if_with_a_proc_which_returns_true_always_for_has_many
+ Man.accepts_nested_attributes_for :interests, :reject_if => proc {|attributes| true }
+ man = Man.create(:name => "John")
+ interest = man.interests.create(:topic => 'photography')
+ man.update_attributes({:interests_attributes => { :topic => 'gardening', :id => interest.id } })
+ assert_equal 'photography', interest.reload.topic
+ end
+
def test_has_many_association_updating_a_single_record
Man.accepts_nested_attributes_for(:interests)
man = Man.create(:name => 'John')

0 comments on commit 4966b91

Please sign in to comment.