diff options
author | Yves Senn <yves.senn@gmail.com> | 2013-06-22 03:11:19 -0700 |
---|---|---|
committer | Yves Senn <yves.senn@gmail.com> | 2013-06-22 03:11:19 -0700 |
commit | 6675d713186c270de1c4684bc1e56742ea2e19e1 (patch) | |
tree | f27bdc5797fdea14ce4040fae4f8c1ab02a26072 /activerecord/test/cases/associations | |
parent | 919d1a19d5e7871d50c1531351fc3f736bad5d07 (diff) | |
parent | c2377f72d2c7df1103eac73b0eb3781c02ef2e18 (diff) | |
download | rails-6675d713186c270de1c4684bc1e56742ea2e19e1.tar.gz rails-6675d713186c270de1c4684bc1e56742ea2e19e1.tar.bz2 rails-6675d713186c270de1c4684bc1e56742ea2e19e1.zip |
Merge pull request #10901 from armstrjare/fix_query_null_foreign_key_on_new_record_collection_ids_reader
Fixes CollectionAssociation#ids_reader returning incorrect ids for new records
Diffstat (limited to 'activerecord/test/cases/associations')
-rw-r--r-- | activerecord/test/cases/associations/has_many_associations_test.rb | 27 |
1 files changed, 27 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 a75d064ac0..55b57bc2f2 100644 --- a/activerecord/test/cases/associations/has_many_associations_test.rb +++ b/activerecord/test/cases/associations/has_many_associations_test.rb @@ -1311,6 +1311,33 @@ class HasManyAssociationsTest < ActiveRecord::TestCase assert !company.clients.loaded? end + def test_get_ids_for_association_on_new_record_does_not_try_to_find_records + Company.columns # Load schema information so we don't query below + Contract.columns # if running just this test. + + company = Company.new + assert_queries(0) do + company.contract_ids + end + + assert_equal [], company.contract_ids + end + + def test_set_ids_for_association_on_new_record_applies_association_correctly + contract_a = Contract.create! + contract_b = Contract.create! + another_contract = Contract.create! + company = Company.new(:name => "Some Company") + + company.contract_ids = [contract_a.id, contract_b.id] + assert_equal [contract_a.id, contract_b.id], company.contract_ids + assert_equal [contract_a, contract_b], company.contracts + + company.save! + assert_equal company, contract_a.reload.company + assert_equal company, contract_b.reload.company + end + def test_get_ids_ignores_include_option assert_equal [readers(:michael_welcome).id], posts(:welcome).readers_with_person_ids end |