diff options
| author | Ryuta Kamizono <kamipo@gmail.com> | 2018-09-13 21:30:19 +0900 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-09-13 21:30:19 +0900 |
| commit | 823f9e0a89707561b54196bf4aabe20c5edb88c1 (patch) | |
| tree | 82b66bc3d46f5554f10e13e0a2bf598438885eb1 /activerecord/test | |
| parent | 7db3a6e89a05855d6f55864e15ee69986708c1dc (diff) | |
| parent | a50eacb03cf3df1d1cbc227ea0115901e0327f1c (diff) | |
| download | rails-823f9e0a89707561b54196bf4aabe20c5edb88c1.tar.gz rails-823f9e0a89707561b54196bf4aabe20c5edb88c1.tar.bz2 rails-823f9e0a89707561b54196bf4aabe20c5edb88c1.zip | |
Merge pull request #33844 from kamipo/too_many_eager_load_ids
Eager loading/preloading should be worked regardless of large number of records
Diffstat (limited to 'activerecord/test')
| -rw-r--r-- | activerecord/test/cases/associations/eager_test.rb | 13 | ||||
| -rw-r--r-- | activerecord/test/fixtures/citations.yml | 4 | ||||
| -rw-r--r-- | activerecord/test/models/citation.rb | 1 | ||||
| -rw-r--r-- | activerecord/test/schema/schema.rb | 1 |
4 files changed, 19 insertions, 0 deletions
diff --git a/activerecord/test/cases/associations/eager_test.rb b/activerecord/test/cases/associations/eager_test.rb index a1fba8dc66..ca902131f4 100644 --- a/activerecord/test/cases/associations/eager_test.rb +++ b/activerecord/test/cases/associations/eager_test.rb @@ -18,6 +18,7 @@ require "models/job" require "models/subscriber" require "models/subscription" require "models/book" +require "models/citation" require "models/developer" require "models/computer" require "models/project" @@ -29,6 +30,18 @@ require "models/sponsor" require "models/mentor" require "models/contract" +class EagerLoadingTooManyIdsTest < ActiveRecord::TestCase + fixtures :citations + + def test_preloading_too_many_ids + assert_equal Citation.count, Citation.preload(:citations).to_a.size + end + + def test_eager_loading_too_may_ids + assert_equal Citation.count, Citation.eager_load(:citations).offset(0).size + end +end + class EagerAssociationTest < ActiveRecord::TestCase fixtures :posts, :comments, :authors, :essays, :author_addresses, :categories, :categories_posts, :companies, :accounts, :tags, :taggings, :people, :readers, :categorizations, diff --git a/activerecord/test/fixtures/citations.yml b/activerecord/test/fixtures/citations.yml new file mode 100644 index 0000000000..d31cb8efa1 --- /dev/null +++ b/activerecord/test/fixtures/citations.yml @@ -0,0 +1,4 @@ +<% 65536.times do |i| %> +fixture_no_<%= i %>: + id: <%= i %> +<% end %> diff --git a/activerecord/test/models/citation.rb b/activerecord/test/models/citation.rb index 3d786f27eb..cee3d18173 100644 --- a/activerecord/test/models/citation.rb +++ b/activerecord/test/models/citation.rb @@ -2,4 +2,5 @@ class Citation < ActiveRecord::Base belongs_to :reference_of, class_name: "Book", foreign_key: :book2_id + has_many :citations end diff --git a/activerecord/test/schema/schema.rb b/activerecord/test/schema/schema.rb index 266e55f682..27c7348f5c 100644 --- a/activerecord/test/schema/schema.rb +++ b/activerecord/test/schema/schema.rb @@ -160,6 +160,7 @@ ActiveRecord::Schema.define do create_table :citations, force: true do |t| t.column :book1_id, :integer t.column :book2_id, :integer + t.references :citation end create_table :clubs, force: true do |t| |