mirror of
https://github.com/rails/rails.git
synced 2022-11-09 12:12:34 -05:00
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>
This commit is contained in:
parent
7f743233c4
commit
4966b915fe
1 changed files with 9 additions and 1 deletions
|
@ -115,7 +115,7 @@ class TestNestedAttributesInGeneral < ActiveRecord::TestCase
|
|||
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')
|
||||
|
@ -123,6 +123,14 @@ class TestNestedAttributesInGeneral < ActiveRecord::TestCase
|
|||
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')
|
||||
|
|
Loading…
Reference in a new issue