diff options
author | miloops <miloops@gmail.com> | 2008-09-15 13:23:50 -0300 |
---|---|---|
committer | Michael Koziarski <michael@koziarski.com> | 2008-09-15 18:32:05 +0200 |
commit | dc8bf7515de85f5bc28d17e96edf4a3e74a858da (patch) | |
tree | 3bcc577d8a233d6392b1b78c2b14afe928c2e514 /activerecord/test/cases | |
parent | 157141b2949b845e372ee703bfd6fba3ffb00415 (diff) | |
download | rails-dc8bf7515de85f5bc28d17e96edf4a3e74a858da.tar.gz rails-dc8bf7515de85f5bc28d17e96edf4a3e74a858da.tar.bz2 rails-dc8bf7515de85f5bc28d17e96edf4a3e74a858da.zip |
When counting grouped records the target should be loaded to return a valid groups count result.
Without this change count_records will group for the count in the query and return erroneous results.
Signed-off-by: Michael Koziarski <michael@koziarski.com>
[#937 state:committed]
Diffstat (limited to 'activerecord/test/cases')
-rw-r--r-- | activerecord/test/cases/associations/has_many_associations_test.rb | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/activerecord/test/cases/associations/has_many_associations_test.rb b/activerecord/test/cases/associations/has_many_associations_test.rb index 5bcbc5eb5b..ba750b266c 100644 --- a/activerecord/test/cases/associations/has_many_associations_test.rb +++ b/activerecord/test/cases/associations/has_many_associations_test.rb @@ -249,7 +249,9 @@ class HasManyAssociationsTest < ActiveRecord::TestCase end def test_find_scoped_grouped + assert_equal 1, companies(:first_firm).clients_grouped_by_firm_id.size assert_equal 1, companies(:first_firm).clients_grouped_by_firm_id.length + assert_equal 2, companies(:first_firm).clients_grouped_by_name.size assert_equal 2, companies(:first_firm).clients_grouped_by_name.length end |