diff options
| author | Agis- <corestudiosinc@gmail.com> | 2014-10-13 17:50:32 +0300 |
|---|---|---|
| committer | Agis- <corestudiosinc@gmail.com> | 2014-10-13 17:50:32 +0300 |
| commit | 719d52db42a0eeebf0f1fa2f8ac3173544dd521e (patch) | |
| tree | 63b18091ff85b230f4708f0ebfad5b6e6f362562 /activerecord/test/cases | |
| parent | 51278579477eb7ee20fe2aba53b4b13203791b22 (diff) | |
| download | rails-719d52db42a0eeebf0f1fa2f8ac3173544dd521e.tar.gz rails-719d52db42a0eeebf0f1fa2f8ac3173544dd521e.tar.bz2 rails-719d52db42a0eeebf0f1fa2f8ac3173544dd521e.zip | |
Autosave callbacks shouldn't be `after_save`
068f092ced8483e557725542dd919ab7c516e567 registered autosave callbacks
as `after_save` callbacks. This caused the regression described in #17209.
Autosave callbacks should be registered as `after_update` and
`after_create` callbacks, just like before.
This is a partial revert of 068f092ced8483e557725542dd919ab7c516e567.
Fixes #17209.
Diffstat (limited to 'activerecord/test/cases')
| -rw-r--r-- | activerecord/test/cases/autosave_association_test.rb | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/activerecord/test/cases/autosave_association_test.rb b/activerecord/test/cases/autosave_association_test.rb index b2a7d3956d..734fd5fe18 100644 --- a/activerecord/test/cases/autosave_association_test.rb +++ b/activerecord/test/cases/autosave_association_test.rb @@ -1,5 +1,6 @@ require 'cases/helper' require 'models/bird' +require 'models/comment' require 'models/company' require 'models/customer' require 'models/developer' @@ -616,6 +617,14 @@ class TestDefaultAutosaveAssociationOnNewRecord < ActiveRecord::TestCase firm.save! assert !account.persisted? end + + def test_autosave_new_record_with_after_create_callback + post = PostWithAfterCreateCallback.new(title: 'Captain Murphy', body: 'is back') + post.comments.build(body: 'foo') + post.save! + + assert_not_nil post.author_id + end end class TestDestroyAsPartOfAutosaveAssociation < ActiveRecord::TestCase |