diff options
author | Pratik Naik <pratiknaik@gmail.com> | 2008-10-04 20:03:42 +0100 |
---|---|---|
committer | Pratik Naik <pratiknaik@gmail.com> | 2008-10-04 20:13:44 +0100 |
commit | 9599948fbcd67c1c2c5fecc2dca148e998479e33 (patch) | |
tree | 919ce7796f1605c3ba6f7edc81afc2bcac4eaad5 /activerecord/test | |
parent | 1dfebd4f0d25ecf50ba4e320234523d9ee3f7523 (diff) | |
download | rails-9599948fbcd67c1c2c5fecc2dca148e998479e33.tar.gz rails-9599948fbcd67c1c2c5fecc2dca148e998479e33.tar.bz2 rails-9599948fbcd67c1c2c5fecc2dca148e998479e33.zip |
Ensure Model.sum and Model.avg typecast appropriately. [#1066 state:resolved]
Model.sum delegates typecasting to the column being summed. If that's not feasible, returns a string.
Model.avg always returns big decimal.
Diffstat (limited to 'activerecord/test')
-rw-r--r-- | activerecord/test/cases/calculations_test.rb | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/activerecord/test/cases/calculations_test.rb b/activerecord/test/cases/calculations_test.rb index 754fd58f35..0fa61500c0 100644 --- a/activerecord/test/cases/calculations_test.rb +++ b/activerecord/test/cases/calculations_test.rb @@ -18,8 +18,8 @@ class CalculationsTest < ActiveRecord::TestCase def test_should_average_field value = Account.average(:credit_limit) - assert_kind_of Float, value - assert_in_delta 53.0, value, 0.001 + assert_kind_of BigDecimal, value + assert_equal BigDecimal.new('53.0'), value end def test_should_return_nil_as_average @@ -273,7 +273,7 @@ class CalculationsTest < ActiveRecord::TestCase end def test_should_sum_expression - assert_equal 636, Account.sum("2 * credit_limit") + assert_equal '636', Account.sum("2 * credit_limit") end def test_count_with_from_option |