Merge pull request rails#46688 from rails/revert-46613-strict-loading-build

Revert "Fix: Strict Loading error after initializing association owner with primary key"

Author: rafaelfranca

activerecord/lib/active_record/associations/collection_association.rb

@@ -119,7 +119,7 @@ def build(attributes = nil, &block)
       def concat(*records)
         records = records.flatten
         if owner.new_record?
-          loaded!
+          load_target
           concat_records(records)
         else
           transaction { concat_records(records) }

activerecord/test/cases/associations/join_model_test.rb

@@ -478,7 +478,7 @@ def test_associating_unsaved_records_with_has_many_through
     new_tag = Tag.new(name: "new")
 
     saved_post.tags << new_tag
-    assert_predicate new_tag, :persisted? # consistent with habtm!
+    assert new_tag.persisted? # consistent with habtm!
     assert_predicate saved_post, :persisted?
     assert_includes saved_post.tags, new_tag
 

activerecord/test/cases/associations_test.rb

@@ -157,20 +157,16 @@ def test_push_followed_by_save_does_not_load_target
 
   def test_push_does_not_lose_additions_to_new_record
     josh = Author.new(name: "Josh")
-    new_post = Post.new(title: "New on Edge", body: "More cool stuff!")
-    josh.posts << new_post
+    josh.posts << Post.new(title: "New on Edge", body: "More cool stuff!")
     assert_predicate josh.posts, :loaded?
     assert_equal 1, josh.posts.size
-    assert_includes josh.posts, new_post
   end
 
   def test_append_behaves_like_push
     josh = Author.new(name: "Josh")
-    new_post = Post.new(title: "New on Edge", body: "More cool stuff!")
-    josh.posts.append new_post
+    josh.posts.append Post.new(title: "New on Edge", body: "More cool stuff!")
     assert_predicate josh.posts, :loaded?
     assert_equal 1, josh.posts.size
-    assert_includes josh.posts, new_post
   end
 
   def test_prepend_is_not_defined

activerecord/test/cases/strict_loading_test.rb

@@ -162,33 +162,6 @@ def test_strict_loading_with_reflection_is_ignored_in_validation_context
     end
   end
 
-  def test_strict_loading_on_concat_is_ignored
-    developer = Developer.first
-    developer.strict_loading!
-
-    assert_nothing_raised do
-      developer.audit_logs << AuditLog.new(message: "message")
-    end
-  end
-
-  def test_strict_loading_with_new_record_on_concat_is_ignored
-    developer = Developer.new(id: Developer.first.id)
-    developer.strict_loading!
-
-    assert_nothing_raised do
-      developer.audit_logs << AuditLog.new(message: "message")
-    end
-  end
-
-  def test_strict_loading_with_new_record_on_build_is_ignored
-    developer = Developer.new(id: Developer.first.id)
-    developer.strict_loading!
-
-    assert_nothing_raised do
-      developer.audit_logs.build(message: "message")
-    end
-  end
-
   def test_strict_loading_has_one_reload
     with_strict_loading_by_default(Developer) do
       ship = Ship.create!(developer: Developer.first, name: "The Great Ship")