Browse files

Ensure ordering to make the test pass with postgresql

  • Loading branch information...
1 parent 5e7919c commit a739340d3c9e66814429af6f3f410c01ff86810a @carlosantoniodasilva committed Nov 18, 2012
Showing with 5 additions and 4 deletions.
  1. +5 −4 activerecord/test/cases/relations_test.rb
View
9 activerecord/test/cases/relations_test.rb
@@ -157,19 +157,19 @@ def test_finding_with_arel_order
assert_equal 4, topics.to_a.size
assert_equal topics(:first).title, topics.first.title
end
-
+
def test_finding_with_assoc_order
topics = Topic.order(:id => :desc)
assert_equal 4, topics.to_a.size
assert_equal topics(:fourth).title, topics.first.title
end
-
+
def test_finding_with_reverted_assoc_order
topics = Topic.order(:id => :asc).reverse_order
assert_equal 4, topics.to_a.size
assert_equal topics(:fourth).title, topics.first.title
end
-
+
def test_raising_exception_on_invalid_hash_params
assert_raise(ArgumentError) { Topic.order(:name, "id DESC", :id => :DeSc) }
end
@@ -1440,7 +1440,8 @@ def test_presence
end
test 'group with select and includes' do
- authors_count = Post.select('author_id, COUNT(author_id) AS num_posts').group('author_id').includes(:author).to_a
+ authors_count = Post.select('author_id, COUNT(author_id) AS num_posts').
+ group('author_id').order('author_id').includes(:author).to_a
assert_no_queries do
result = authors_count.map do |post|

0 comments on commit a739340

Please sign in to comment.