Merge pull request #27355 from yukideluxe/fixtures-deleted-tables

fix fixtures loading for models with same table names in different databases

activerecord/lib/active_record/fixtures.rb

@@ -536,16 +536,16 @@ def self.create_fixtures(fixtures_directory, fixture_set_names, class_names = {}
           update_all_loaded_fixtures fixtures_map
 
           connection.transaction(requires_new: true) do
-            deleted_tables = Set.new
+            deleted_tables = Hash.new { |h, k| h[k] = Set.new }
             fixture_sets.each do |fs|
               conn = fs.model_class.respond_to?(:connection) ? fs.model_class.connection : connection
               table_rows = fs.table_rows
 
               table_rows.each_key do |table|
-                unless deleted_tables.include? table
+                unless deleted_tables[conn].include? table
                   conn.delete "DELETE FROM #{conn.quote_table_name(table)}", "Fixture Delete"
                 end
-                deleted_tables << table
+                deleted_tables[conn] << table
               end
 
               table_rows.each do |fixture_set_name, rows|

activerecord/test/cases/fixtures_test.rb

@@ -12,9 +12,11 @@
 require "models/computer"
 require "models/course"
 require "models/developer"
+require "models/dog"
 require "models/doubloon"
 require "models/joke"
 require "models/matey"
+require "models/other_dog"
 require "models/parrot"
 require "models/pirate"
 require "models/post"
@@ -1021,3 +1023,16 @@ def teardown
     assert_nil fixture_class_names["unregistered_identifier"]
   end
 end
+
+class SameNameDifferentDatabaseFixturesTest < ActiveRecord::TestCase
+  fixtures :dogs, :other_dogs
+
+  test "fixtures are properly loaded" do
+    # Force loading the fixtures again to reproduce issue
+    ActiveRecord::FixtureSet.reset_cache
+    create_fixtures("dogs", "other_dogs")
+
+    assert_kind_of Dog, dogs(:sophie)
+    assert_kind_of OtherDog, other_dogs(:lassie)
+  end
+end

activerecord/test/fixtures/other_dogs.yml

+lassie:
+  id: 1

activerecord/test/models/other_dog.rb

+require_dependency "models/arunit2_model"
+
+class OtherDog < ARUnit2Model
+  self.table_name = "dogs"
+end

activerecord/test/schema/schema.rb

@@ -1050,3 +1050,5 @@
   t.references :course
   t.references :professor
 end
+
+OtherDog.connection.create_table :dogs, force: true

activerecord/test/support/connection.rb

@@ -2,6 +2,7 @@
 require "models/college"
 require "models/course"
 require "models/professor"
+require "models/other_dog"
 
 module ARTest
   def self.connection_name