Skip to content

Commit

Permalink
Merge pull request #2868 from arunagw/batches_test_pg_master
Browse files Browse the repository at this point in the history
Sort here to make sure the order is ok.
  • Loading branch information
jonleighton committed Sep 5, 2011
2 parents 7e16394 + 76e9401 commit 499ad11
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion activerecord/test/cases/batches_test.rb
Expand Up @@ -128,7 +128,7 @@ def test_find_in_batches_should_ignore_the_order_default_scope
end

def test_find_in_batches_should_not_ignore_the_default_scope_if_it_is_other_then_order
special_posts_ids = SpecialPostWithDefaultScope.all.map(&:id)
special_posts_ids = SpecialPostWithDefaultScope.all.map(&:id).sort
posts = []
SpecialPostWithDefaultScope.find_in_batches do |batch|
posts.concat(batch)
Expand Down

0 comments on commit 499ad11

Please sign in to comment.