diff options
| author | Ryuta Kamizono <kamipo@gmail.com> | 2019-02-06 19:44:24 +0900 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2019-02-06 19:44:24 +0900 |
| commit | 9cc463ed7b7be098602b72a98f72220ea6466ba2 (patch) | |
| tree | 311386802febb8bd8ecac4e90a93f0e5dfbfa3be /activerecord/test/cases/associations/has_many_associations_test.rb | |
| parent | daa264fb5afc353cb044787993b9d9b98f774b26 (diff) | |
| parent | 788eb51df3007d94b266965433b0be88a23b84c5 (diff) | |
| download | rails-9cc463ed7b7be098602b72a98f72220ea6466ba2.tar.gz rails-9cc463ed7b7be098602b72a98f72220ea6466ba2.tar.bz2 rails-9cc463ed7b7be098602b72a98f72220ea6466ba2.zip | |
Merge pull request #35174 from ypresto/fix-concat-retval
Fix `CollectionProxy#concat` to return self by alias it to `#<<`
Diffstat (limited to 'activerecord/test/cases/associations/has_many_associations_test.rb')
| -rw-r--r-- | activerecord/test/cases/associations/has_many_associations_test.rb | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/activerecord/test/cases/associations/has_many_associations_test.rb b/activerecord/test/cases/associations/has_many_associations_test.rb index 4c9e4d0ad2..5fdc5a92fc 100644 --- a/activerecord/test/cases/associations/has_many_associations_test.rb +++ b/activerecord/test/cases/associations/has_many_associations_test.rb @@ -996,9 +996,10 @@ class HasManyAssociationsTest < ActiveRecord::TestCase assert_predicate companies(:first_firm).clients_of_firm, :loaded? - companies(:first_firm).clients_of_firm.concat([Client.new("name" => "Natural Company"), Client.new("name" => "Apple")]) + result = companies(:first_firm).clients_of_firm.concat([Client.new("name" => "Natural Company"), Client.new("name" => "Apple")]) assert_equal 4, companies(:first_firm).clients_of_firm.size assert_equal 4, companies(:first_firm).clients_of_firm.reload.size + assert_equal companies(:first_firm).clients_of_firm, result end def test_transactions_when_adding_to_persisted |