diff options
author | Ben Woosley <ben.woosley@gmail.com> | 2013-06-18 02:21:52 -0700 |
---|---|---|
committer | Ben Woosley <ben.woosley@gmail.com> | 2013-06-19 04:04:13 -0700 |
commit | 25042359b388a73bae798e023276df59e53c74e2 (patch) | |
tree | 8128c43c74df54dc60a1c8e4d755af179a830ab7 /activerecord/test/cases | |
parent | 7324624f748b9717a5f9757ca89d791563ac5f95 (diff) | |
download | rails-25042359b388a73bae798e023276df59e53c74e2.tar.gz rails-25042359b388a73bae798e023276df59e53c74e2.tar.bz2 rails-25042359b388a73bae798e023276df59e53c74e2.zip |
When .find_each is called without a block, return an Enumerator.
This lets us do things like call: .find_each.with_index
Diffstat (limited to 'activerecord/test/cases')
-rw-r--r-- | activerecord/test/cases/batches_test.rb | 18 |
1 files changed, 18 insertions, 0 deletions
diff --git a/activerecord/test/cases/batches_test.rb b/activerecord/test/cases/batches_test.rb index e09fa95756..1d6676da1c 100644 --- a/activerecord/test/cases/batches_test.rb +++ b/activerecord/test/cases/batches_test.rb @@ -26,6 +26,24 @@ class EachTest < ActiveRecord::TestCase end end + def test_each_should_return_an_enumerator_if_no_block_is_present + assert_queries(1) do + Post.find_each(:batch_size => 100000).with_index do |post, index| + assert_kind_of Post, post + assert_kind_of Integer, index + end + end + end + + def test_each_enumerator_should_execute_one_query_per_batch + assert_queries(@total + 1) do + Post.find_each(:batch_size => 1).with_index do |post, index| + assert_kind_of Post, post + assert_kind_of Integer, index + end + end + end + def test_each_should_raise_if_select_is_set_without_id assert_raise(RuntimeError) do Post.select(:title).find_each(:batch_size => 1) { |post| post } |