Permalink
Browse files

Refactor test to avoid hackery

  • Loading branch information...
1 parent bbe0a50 commit e68a83c9ecdc5b9395e0dde687cafdf8330f9de0 @jonleighton jonleighton committed Apr 15, 2011
Showing with 4 additions and 4 deletions.
  1. +4 −4 activerecord/test/cases/associations/join_model_test.rb
@@ -516,10 +516,10 @@ def test_has_many_through_collection_size_doesnt_load_target_if_not_loaded
end
def test_has_many_through_collection_size_uses_counter_cache_if_it_exists
- author = authors(:david)
- author.stubs(:_read_attribute).with('comments_count').returns(100)
- assert_equal 100, author.comments.size
- assert !author.comments.loaded?
+ c = categories(:general)
+ c.categorizations_count = 100
+ assert_equal 100, c.categorizations.size
+ assert !c.categorizations.loaded?
end
def test_adding_junk_to_has_many_through_should_raise_type_mismatch

0 comments on commit e68a83c

Please sign in to comment.