Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Sort here to make sure the order is ok. #2868

Merged
merged 1 commit into from

2 participants

@arunagw
Collaborator

No description provided.

@jonleighton jonleighton merged commit 499ad11 into from
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Sep 5, 2011
  1. @arunagw
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  activerecord/test/cases/batches_test.rb
View
2  activerecord/test/cases/batches_test.rb
@@ -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)
Something went wrong with that request. Please try again.