diff options
author | Alexis Bernard <alexis@obloh.com> | 2012-10-30 15:54:16 +0100 |
---|---|---|
committer | Alexis Bernard <alexis@obloh.com> | 2012-10-31 15:39:24 +0100 |
commit | a58cafeb3a86be46849de57481b6644094fb8165 (patch) | |
tree | 25e5f33cc12853e6bb522538951319470b484c01 /activerecord/test/cases | |
parent | 6a6909dc2940ac0a6d3536fdac96bdce55a18023 (diff) | |
download | rails-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.rb | 9 |
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 |