chore: rename testing classes as new versions became reserved (#477)

* chore: rename testing classes as new versions became reserved

* rename the sqliterecord

* chore: lock minitest version

Author: seuros

.tool-versions

@@ -1 +1 @@
-ruby 3.4.7
+ruby 3.4.8

closure_tree.gemspec

@@ -30,7 +30,7 @@ Gem::Specification.new do |gem|
   gem.add_dependency 'zeitwerk', '~> 2.7'
 
   gem.add_development_dependency 'database_cleaner'
-  gem.add_development_dependency 'minitest'
+  gem.add_development_dependency 'minitest', '~> 5.0'
   gem.add_development_dependency 'minitest-reporters'
   gem.add_development_dependency 'parallel'
   gem.add_development_dependency 'simplecov'

lib/closure_tree/support.rb

@@ -42,7 +42,7 @@ def abstract_base_class
       klass = model_class
       while klass.superclass != ActiveRecord::Base
         parent = klass.superclass
-        # Stop at abstract class (ApplicationRecord, MysqlRecord, etc.)
+        # Stop at abstract class (ApplicationRecord, SecondaryRecord, etc.)
         return parent if parent.abstract_class?
         # Stop at connection boundary (handles non-abstract parents with custom connections)
         return parent if parent.connection_specification_name != parent.superclass.connection_specification_name

test/closure_tree/adopt_test.rb

@@ -255,7 +255,7 @@ def run_adopt_tests_for(model_class)
 run_adopt_tests_for(AdoptableTag) if postgresql?(ApplicationRecord.connection)
 
 # Test with MySQL
-run_adopt_tests_for(MysqlAdoptableTag) if mysql?(MysqlRecord.connection)
+run_adopt_tests_for(SecondaryAdoptableTag) if mysql?(SecondaryRecord.connection)
 
 # Test with SQLite
-run_adopt_tests_for(MemoryAdoptableTag) if sqlite?(SqliteRecord.connection)
+run_adopt_tests_for(MemoryAdoptableTag) if sqlite?(LiteRecord.connection)

test/closure_tree/multi_database_test.rb

@@ -6,27 +6,27 @@ class MultiDatabaseTest < ActiveSupport::TestCase
   def setup
     super
     # Create memory tables - always recreate for clean state
-    SqliteRecord.connection.create_table :memory_tags, force: true do |t|
+    LiteRecord.connection.create_table :memory_tags, force: true do |t|
       t.string :name
       t.integer :parent_id
       t.timestamps
     end
 
-    SqliteRecord.connection.create_table :memory_tag_hierarchies, id: false, force: true do |t|
+    LiteRecord.connection.create_table :memory_tag_hierarchies, id: false, force: true do |t|
       t.integer :ancestor_id, null: false
       t.integer :descendant_id, null: false
       t.integer :generations, null: false
     end
 
-    SqliteRecord.connection.add_index :memory_tag_hierarchies, %i[ancestor_id descendant_id generations],
+    LiteRecord.connection.add_index :memory_tag_hierarchies, %i[ancestor_id descendant_id generations],
                                       unique: true, name: 'memory_tag_anc_desc_idx'
-    SqliteRecord.connection.add_index :memory_tag_hierarchies, [:descendant_id], name: 'memory_tag_desc_idx'
+    LiteRecord.connection.add_index :memory_tag_hierarchies, [:descendant_id], name: 'memory_tag_desc_idx'
   end
 
   def teardown
     # Clean up SQLite tables after each test
-    SqliteRecord.connection.drop_table :memory_tag_hierarchies, if_exists: true
-    SqliteRecord.connection.drop_table :memory_tags, if_exists: true
+    LiteRecord.connection.drop_table :memory_tag_hierarchies, if_exists: true
+    LiteRecord.connection.drop_table :memory_tags, if_exists: true
     super
   end
 
@@ -46,13 +46,13 @@ def test_postgresql_with_advisory_lock
   end
 
   def test_mysql_with_advisory_lock
-    skip 'MySQL not configured' unless mysql?(MysqlRecord.connection)
+    skip 'MySQL not configured' unless mysql?(SecondaryRecord.connection)
 
-    tag = MysqlTag.create!(name: 'MySQL Root')
+    tag = SecondaryTag.create!(name: 'MySQL Root')
     child = nil
 
     # Advisory locks should work on MySQL
-    MysqlTag.with_advisory_lock('test_lock') do
+    SecondaryTag.with_advisory_lock('test_lock') do
       child = tag.children.create!(name: 'MySQL Child')
     end
 
@@ -80,21 +80,21 @@ def test_concurrent_operations_different_databases
       pg_tag.children.create!(name: 'PG Child 1')
     end
 
-    mysql_tag = MysqlTag.create!(name: 'MySQL Root')
+    mysql_tag = SecondaryTag.create!(name: 'MySQL Root')
     sqlite_tag = MemoryTag.create!(name: 'SQLite Root')
 
     # Test concurrent operations only for MySQL and SQLite
     threads = []
 
     threads << Thread.new do
-      MysqlRecord.connection_pool.with_connection do
-        tag = MysqlTag.find(mysql_tag.id)
+      SecondaryRecord.connection_pool.with_connection do
+        tag = SecondaryTag.find(mysql_tag.id)
         tag.children.create!(name: 'MySQL Child 1')
       end
     end
 
     threads << Thread.new do
-      SqliteRecord.connection_pool.with_connection do
+      LiteRecord.connection_pool.with_connection do
         tag = MemoryTag.find(sqlite_tag.id)
         tag.children.create!(name: 'SQLite Child 1')
       end

test/closure_tree/parallel_test.rb

@@ -67,9 +67,9 @@ def work
     # Clean up SQLite database file if it exists
     db_file = 'test/dummy/db/test.sqlite3'
     if File.exist?(db_file)
-      SqliteRecord.connection.disconnect!
+      LiteRecord.connection.disconnect!
       File.delete(db_file)
-      SqliteRecord.connection.reconnect!
+      LiteRecord.connection.reconnect!
     end
     Tag.delete_all
     Tag.hierarchy_class.delete_all

test/dummy/app/models/lite_record.rb

@@ -0,0 +1,6 @@
+# frozen_string_literal: true
+
+class LiteRecord < ApplicationRecord
+  self.abstract_class = true
+  connects_to database: { writing: :lite, reading: :lite }
+end

test/dummy/app/models/memory_adoptable_tag.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
-class MemoryAdoptableTag < SqliteRecord
+class MemoryAdoptableTag < LiteRecord
   has_closure_tree dependent: :adopt, name_column: 'name'
 end

test/dummy/app/models/memory_tag.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
-class MemoryTag < SqliteRecord
+class MemoryTag < LiteRecord
   has_closure_tree
 end

…/dummy/app/models/mysql_adoptable_tag.rb …my/app/models/secondary_adoptable_tag.rbtest/dummy/app/models/mysql_adoptable_tag.rb renamed to test/dummy/app/models/secondary_adoptable_tag.rb test/dummy/app/models/mysql_adoptable_tag.rb renamed to test/dummy/app/models/secondary_adoptable_tag.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
-class MysqlAdoptableTag < MysqlRecord
+class SecondaryAdoptableTag < SecondaryRecord
   has_closure_tree dependent: :adopt, name_column: 'name'
 end