diff options
| author | Matthew Draper <matthew@trebex.net> | 2016-08-31 02:13:03 +0930 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2016-08-31 02:13:03 +0930 |
| commit | e912915a006140350e411f1d0234fa2597491e37 (patch) | |
| tree | aa52e8a940978ad6e4768a3d36fa11a80c8f1592 /activerecord/test | |
| parent | f4418c9a1a2f44d49521cee9881617d163ad729d (diff) | |
| parent | a94fe2971b24d2a7827e205e7898e1b22771ed72 (diff) | |
| download | rails-e912915a006140350e411f1d0234fa2597491e37.tar.gz rails-e912915a006140350e411f1d0234fa2597491e37.tar.bz2 rails-e912915a006140350e411f1d0234fa2597491e37.zip | |
Merge pull request #23498 from jcoleman/remove-unnecessary-belongs-to-load
Don't unnecessarily load a belongs_to when saving.
Diffstat (limited to 'activerecord/test')
| -rw-r--r-- | activerecord/test/cases/associations/belongs_to_associations_test.rb | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/activerecord/test/cases/associations/belongs_to_associations_test.rb b/activerecord/test/cases/associations/belongs_to_associations_test.rb index 3f42cb9b9d..2418346d1b 100644 --- a/activerecord/test/cases/associations/belongs_to_associations_test.rb +++ b/activerecord/test/cases/associations/belongs_to_associations_test.rb @@ -626,6 +626,12 @@ class BelongsToAssociationsTest < ActiveRecord::TestCase assert_queries(0) { tagging.super_tag } end + def test_dont_find_target_when_saving_foreign_key_after_stale_association_loaded + client = Client.create!(name: "Test client", firm_with_basic_id: Firm.find(1)) + client.firm_id = Firm.create!(name: "Test firm").id + assert_queries(1) { client.save! } + end + def test_field_name_same_as_foreign_key computer = Computer.find(1) assert_not_nil computer.developer, ":foreign key == attribute didn't lock up" # ' |