diff options
author | Santiago Pastorino <santiago@wyeworks.com> | 2014-04-07 12:01:39 -0300 |
---|---|---|
committer | Santiago Pastorino <santiago@wyeworks.com> | 2014-04-07 12:01:39 -0300 |
commit | 83e9e193f86741401a463cae17163ea0cd6b2389 (patch) | |
tree | 56cc99b5d5258baf0b070344d3de386de2cf6695 /activerecord/test | |
parent | 013b7c19c81b95583cb537fd2b4330f6860c8502 (diff) | |
parent | bbad7523f08936f38938c7d4ff18f4084f008244 (diff) | |
download | rails-83e9e193f86741401a463cae17163ea0cd6b2389.tar.gz rails-83e9e193f86741401a463cae17163ea0cd6b2389.tar.bz2 rails-83e9e193f86741401a463cae17163ea0cd6b2389.zip |
Merge pull request #14631 from laurocaetano/fix_count_with_order
Ignore order when doing count.
Diffstat (limited to 'activerecord/test')
-rw-r--r-- | activerecord/test/cases/calculations_test.rb | 14 |
1 files changed, 14 insertions, 0 deletions
diff --git a/activerecord/test/cases/calculations_test.rb b/activerecord/test/cases/calculations_test.rb index db999f90ab..b8de78934e 100644 --- a/activerecord/test/cases/calculations_test.rb +++ b/activerecord/test/cases/calculations_test.rb @@ -387,6 +387,20 @@ class CalculationsTest < ActiveRecord::TestCase assert_raise(ArgumentError) { Account.count(1, 2, 3) } end + def test_count_with_order + assert_equal 6, Account.order(:credit_limit).count + end + + def test_count_with_reverse_order + assert_equal 6, Account.order(:credit_limit).reverse_order.count + end + + def test_count_with_where_and_order + assert_equal 1, Account.where(firm_name: '37signals').count + assert_equal 1, Account.where(firm_name: '37signals').order(:firm_name).count + assert_equal 1, Account.where(firm_name: '37signals').order(:firm_name).reverse_order.count + end + def test_should_sum_expression # Oracle adapter returns floating point value 636.0 after SUM if current_adapter?(:OracleAdapter) |