aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/test/cases
diff options
context:
space:
mode:
authorAlexis Bernard <alexis@obloh.com>2012-10-30 15:54:16 +0100
committerAlexis Bernard <alexis@obloh.com>2012-10-31 15:39:24 +0100
commita58cafeb3a86be46849de57481b6644094fb8165 (patch)
tree25e5f33cc12853e6bb522538951319470b484c01 /activerecord/test/cases
parent6a6909dc2940ac0a6d3536fdac96bdce55a18023 (diff)
downloadrails-a58cafeb3a86be46849de57481b6644094fb8165.tar.gz
rails-a58cafeb3a86be46849de57481b6644094fb8165.tar.bz2
rails-a58cafeb3a86be46849de57481b6644094fb8165.zip
Fix find_in_batches against string IDs when start option is not specified.
Diffstat (limited to 'activerecord/test/cases')
-rw-r--r--activerecord/test/cases/batches_test.rb9
1 files changed, 9 insertions, 0 deletions
diff --git a/activerecord/test/cases/batches_test.rb b/activerecord/test/cases/batches_test.rb
index 12d5245fbd..70ceaec4f0 100644
--- a/activerecord/test/cases/batches_test.rb
+++ b/activerecord/test/cases/batches_test.rb
@@ -136,4 +136,13 @@ class EachTest < ActiveRecord::TestCase
assert_equal nick_order_subscribers[1..-1].map(&:id), subscribers.map(&:id)
end
+
+ def test_find_in_batches_should_use_any_column_as_primary_key_when_start_is_not_specified
+ Subscriber.count('nick') # preheat arel's table cache
+ assert_queries(Subscriber.count + 1) do
+ Subscriber.find_each(:batch_size => 1) do |subscriber|
+ assert_kind_of Subscriber, subscriber
+ end
+ end
+ end
end