diff options
author | Will Bryant <will.bryant@gmail.com> | 2009-03-06 22:29:36 +0000 |
---|---|---|
committer | Pratik Naik <pratiknaik@gmail.com> | 2009-03-06 22:29:53 +0000 |
commit | 7fb7b48a1f771e50896a82d749a70922a18afae7 (patch) | |
tree | 6e583256462d9da5431098ddd351e432723b5df3 /activerecord/test/cases/calculations_test.rb | |
parent | 3ca5a0f9fd7b7921bca970859da8637011b22dd1 (diff) | |
download | rails-7fb7b48a1f771e50896a82d749a70922a18afae7.tar.gz rails-7fb7b48a1f771e50896a82d749a70922a18afae7.tar.bz2 rails-7fb7b48a1f771e50896a82d749a70922a18afae7.zip |
Allow :having conditions to be sanitized like regular :condition. [#2158 state:resolved]
Signed-off-by: Pratik Naik <pratiknaik@gmail.com>
Diffstat (limited to 'activerecord/test/cases/calculations_test.rb')
-rw-r--r-- | activerecord/test/cases/calculations_test.rb | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/activerecord/test/cases/calculations_test.rb b/activerecord/test/cases/calculations_test.rb index 5a4ed429b6..c158706645 100644 --- a/activerecord/test/cases/calculations_test.rb +++ b/activerecord/test/cases/calculations_test.rb @@ -92,6 +92,14 @@ class CalculationsTest < ActiveRecord::TestCase assert_equal 60, c[2] end + def test_should_group_by_summed_field_having_sanitized_condition + c = Account.sum(:credit_limit, :group => :firm_id, + :having => ['sum(credit_limit) > ?', 50]) + assert_nil c[1] + assert_equal 105, c[6] + assert_equal 60, c[2] + end + def test_should_group_by_summed_association c = Account.sum(:credit_limit, :group => :firm) assert_equal 50, c[companies(:first_firm)] |