diff options
author | Ryuta Kamizono <kamipo@gmail.com> | 2018-09-26 23:02:14 +0900 |
---|---|---|
committer | Ryuta Kamizono <kamipo@gmail.com> | 2018-09-26 23:06:08 +0900 |
commit | 52e11e462f6114a4d12225c639c5f501f0ffec7a (patch) | |
tree | 6cf1ff6a3a1de741a9248be463f4125d0b581a29 /activerecord/test/cases | |
parent | 38fae1f2505179123ce59cf19a3f5571559ead35 (diff) | |
download | rails-52e11e462f6114a4d12225c639c5f501f0ffec7a.tar.gz rails-52e11e462f6114a4d12225c639c5f501f0ffec7a.tar.bz2 rails-52e11e462f6114a4d12225c639c5f501f0ffec7a.zip |
Revert "Remove `counter_cache_target` which is no longer called"
This reverts commit 376ffe0ea2e59dc51461122210729c05a10fb443.
Since 38fae1f, `association.increment_counters` is called without
inflated parent target if inverse_of is disabled.
In that case, that commit would cause extra queries to inflate parent.
Diffstat (limited to 'activerecord/test/cases')
-rw-r--r-- | activerecord/test/cases/associations/has_many_associations_test.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/activerecord/test/cases/associations/has_many_associations_test.rb b/activerecord/test/cases/associations/has_many_associations_test.rb index f8d27f1c56..0b44515e00 100644 --- a/activerecord/test/cases/associations/has_many_associations_test.rb +++ b/activerecord/test/cases/associations/has_many_associations_test.rb @@ -1205,7 +1205,7 @@ class HasManyAssociationsTest < ActiveRecord::TestCase reply1 = Reply.create!(title: "re: zoom", content: "speedy quick!") reply2 = Reply.create!(title: "re: zoom 2", content: "OMG lol!") - assert_queries(6) do + assert_queries(4) do topic.replies << [reply1, reply2] end |