aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/test
diff options
context:
space:
mode:
authorEileen M. Uchitelle <eileencodes@users.noreply.github.com>2017-03-09 11:23:32 -0500
committerGitHub <noreply@github.com>2017-03-09 11:23:32 -0500
commitc77cbaff49218db7e5fe93cffc2f5333e63a99c9 (patch)
tree332a8c412548fa94594cc6e55384e86b8cc31f07 /activerecord/test
parent17c65342469f211f4a845073e47bf1dfa13ba259 (diff)
parentef3d6fc1d304dbfed7e49fdf501818891839ef65 (diff)
downloadrails-c77cbaff49218db7e5fe93cffc2f5333e63a99c9.tar.gz
rails-c77cbaff49218db7e5fe93cffc2f5333e63a99c9.tar.bz2
rails-c77cbaff49218db7e5fe93cffc2f5333e63a99c9.zip
Merge pull request #28354 from kamipo/fix_select_with_block_and_dirty_target
Fix select with block doesn't return newly built records in has_many association
Diffstat (limited to 'activerecord/test')
-rw-r--r--activerecord/test/cases/associations/has_many_associations_test.rb6
1 files changed, 6 insertions, 0 deletions
diff --git a/activerecord/test/cases/associations/has_many_associations_test.rb b/activerecord/test/cases/associations/has_many_associations_test.rb
index ede3a44090..14f515fa1c 100644
--- a/activerecord/test/cases/associations/has_many_associations_test.rb
+++ b/activerecord/test/cases/associations/has_many_associations_test.rb
@@ -783,6 +783,12 @@ class HasManyAssociationsTest < ActiveRecord::TestCase
assert_equal [1], posts(:welcome).comments.select { |c| c.id == 1 }.map(&:id)
end
+ def test_select_with_block_and_dirty_target
+ assert_equal 2, posts(:welcome).comments.select { true }.size
+ posts(:welcome).comments.build
+ assert_equal 3, posts(:welcome).comments.select { true }.size
+ end
+
def test_select_without_foreign_key
assert_equal companies(:first_firm).accounts.first.credit_limit, companies(:first_firm).accounts.select(:credit_limit).first.credit_limit
end