From 3d6176fded424a2195e399d031e343bbcd0edcad Mon Sep 17 00:00:00 2001 From: Hendy Tanata Date: Tue, 28 Mar 2017 18:53:58 -0700 Subject: Load only needed records on ActiveRecord::Relation#inspect Instead of loading all records and returning only a subset of those, just load the records as needed. Fixes #25537. --- activerecord/test/cases/relations_test.rb | 6 ++++++ 1 file changed, 6 insertions(+) (limited to 'activerecord/test/cases') diff --git a/activerecord/test/cases/relations_test.rb b/activerecord/test/cases/relations_test.rb index 8c06b1537a..856469c710 100644 --- a/activerecord/test/cases/relations_test.rb +++ b/activerecord/test/cases/relations_test.rb @@ -1901,6 +1901,12 @@ class RelationTest < ActiveRecord::TestCase assert_equal "#", relation.inspect end + test "relations don't load all records in #inspect" do + assert_sql(/LIMIT/) do + Post.all.inspect + end + end + test "already-loaded relations don't perform a new query in #inspect" do relation = Post.limit(2) relation.to_a -- cgit v1.2.3