aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/test/cases
diff options
context:
space:
mode:
authorlulalala <mark@goodlife.tw>2012-10-02 22:30:06 +0800
committerlulalala <mark@goodlife.tw>2012-10-02 23:49:32 +0800
commit6e56a03f834202af4e57c50a71cc74ea08303bfd (patch)
treee3d60012629b06078cbda72e5f1d0130a01900bc /activerecord/test/cases
parent8ea88c3b74a20b56d39ab820244e0b3806a84169 (diff)
downloadrails-6e56a03f834202af4e57c50a71cc74ea08303bfd.tar.gz
rails-6e56a03f834202af4e57c50a71cc74ea08303bfd.tar.bz2
rails-6e56a03f834202af4e57c50a71cc74ea08303bfd.zip
Fix reset_counters() crashing on has_many :through associations.
The counter column name in the intermediate model need to be access via the through reflection.
Diffstat (limited to 'activerecord/test/cases')
-rw-r--r--activerecord/test/cases/counter_cache_test.rb15
1 files changed, 14 insertions, 1 deletions
diff --git a/activerecord/test/cases/counter_cache_test.rb b/activerecord/test/cases/counter_cache_test.rb
index ee443741ca..fc46a249c8 100644
--- a/activerecord/test/cases/counter_cache_test.rb
+++ b/activerecord/test/cases/counter_cache_test.rb
@@ -10,9 +10,12 @@ require 'models/dog'
require 'models/dog_lover'
require 'models/person'
require 'models/friendship'
+require 'models/subscriber'
+require 'models/subscription'
+require 'models/book'
class CounterCacheTest < ActiveRecord::TestCase
- fixtures :topics, :categories, :categorizations, :cars, :dogs, :dog_lovers, :people, :friendships
+ fixtures :topics, :categories, :categorizations, :cars, :dogs, :dog_lovers, :people, :friendships, :subscribers, :subscriptions, :books
class ::SpecialTopic < ::Topic
has_many :special_replies, :foreign_key => 'parent_id'
@@ -118,4 +121,14 @@ class CounterCacheTest < ActiveRecord::TestCase
Person.reset_counters(michael.id, :followers)
end
end
+
+ test "reset counter of has_many :through association" do
+ subscriber = subscribers('second')
+ Subscriber.reset_counters(subscriber.id, 'books')
+ Subscriber.increment_counter('books_count', subscriber.id)
+
+ assert_difference 'subscriber.reload.books_count', -1 do
+ Subscriber.reset_counters(subscriber.id, 'books')
+ end
+ end
end