diff options
author | Yuya Tanaka <yuya.presto@gmail.com> | 2019-02-06 12:29:28 +0900 |
---|---|---|
committer | Yuya Tanaka <yuya.presto@gmail.com> | 2019-02-06 15:01:48 +0900 |
commit | 788eb51df3007d94b266965433b0be88a23b84c5 (patch) | |
tree | fab30aaec3eeef4679177885b6ba0a12bd49b27d /activerecord/test/cases/associations/has_many_through_associations_test.rb | |
parent | 3127f5783ef3f07a05476b4a036d069d96def603 (diff) | |
download | rails-788eb51df3007d94b266965433b0be88a23b84c5.tar.gz rails-788eb51df3007d94b266965433b0be88a23b84c5.tar.bz2 rails-788eb51df3007d94b266965433b0be88a23b84c5.zip |
Fix `CollectionProxy#concat` to return self by alias it to `#<<`
Formerly it was returning arguments (`records` array).
Diffstat (limited to 'activerecord/test/cases/associations/has_many_through_associations_test.rb')
-rw-r--r-- | activerecord/test/cases/associations/has_many_through_associations_test.rb | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/activerecord/test/cases/associations/has_many_through_associations_test.rb b/activerecord/test/cases/associations/has_many_through_associations_test.rb index 0133beccec..0ac56c6168 100644 --- a/activerecord/test/cases/associations/has_many_through_associations_test.rb +++ b/activerecord/test/cases/associations/has_many_through_associations_test.rb @@ -242,9 +242,10 @@ class HasManyThroughAssociationsTest < ActiveRecord::TestCase def test_concat person = people(:david) post = posts(:thinking) - post.people.concat [person] + result = post.people.concat [person] assert_equal 1, post.people.size assert_equal 1, post.people.reload.size + assert_equal post.people, result end def test_associate_existing_record_twice_should_add_to_target_twice |