aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/test
diff options
context:
space:
mode:
authorYves Senn <yves.senn@gmail.com>2013-07-15 11:17:25 -0700
committerRafael Mendonça França <rafaelmfranca@gmail.com>2013-07-15 15:49:42 -0300
commit1c2545a45537661c3b1e896dd93b87377a65eb31 (patch)
treee34bbcdf7432a23b7a388519d2cd9f72e8ac5fb7 /activerecord/test
parent714cb5a436e72d00ca0e5e6efa4b789c3e6f5b04 (diff)
downloadrails-1c2545a45537661c3b1e896dd93b87377a65eb31.tar.gz
rails-1c2545a45537661c3b1e896dd93b87377a65eb31.tar.bz2
rails-1c2545a45537661c3b1e896dd93b87377a65eb31.zip
Merge pull request #11451 from jetthoughts/11450_do_not_resave_destroyed_association
Do not re-save destroyed association on saving parent object Conflicts: activerecord/lib/active_record/autosave_association.rb Conflicts: activerecord/CHANGELOG.md
Diffstat (limited to 'activerecord/test')
-rw-r--r--activerecord/test/cases/autosave_association_test.rb7
1 files changed, 7 insertions, 0 deletions
diff --git a/activerecord/test/cases/autosave_association_test.rb b/activerecord/test/cases/autosave_association_test.rb
index 6dbc419eb5..5556ab8fcc 100644
--- a/activerecord/test/cases/autosave_association_test.rb
+++ b/activerecord/test/cases/autosave_association_test.rb
@@ -734,6 +734,13 @@ class TestDestroyAsPartOfAutosaveAssociation < ActiveRecord::TestCase
ids.each { |id| assert_nil klass.find_by_id(id) }
end
+ def test_should_not_resave_destroyed_association
+ @pirate.birds.create!(name: :parrot)
+ @pirate.birds.first.destroy
+ @pirate.save!
+ assert @pirate.reload.birds.empty?
+ end
+
def test_should_skip_validation_on_has_many_if_marked_for_destruction
2.times { |i| @pirate.birds.create!(:name => "birds_#{i}") }