fix(datasource_active_record): skip anonymous models when building collections

packages/forest_admin_agent/spec/lib/forest_admin_agent/audit_trail/store_spec.rb

@@ -274,6 +274,11 @@ def timeline(**query)
           .to eq(%w[main other])
       end
 
+      # The ActiveRecord datasource skips nameless models: a name would expose the audit table as a collection.
+      it 'keeps its model anonymous' do
+        expect(store.send(:model).name).to be_nil
+      end
+
       it 'lists only entries whose diff touched one of the given fields' do
         store.append(record(previous_values: { 'status' => 'open' }, new_values: { 'status' => 'closed' }))
         store.append(record(previous_values: { 'note' => 'a' }, new_values: { 'note' => 'b' }))

packages/forest_admin_datasource_active_record/lib/forest_admin_datasource_active_record/datasource.rb

@@ -76,6 +76,9 @@ def primary_key?(model)
     end
 
     def fetch_model(model)
+      # A collection is named after its model, so an anonymous one (such as the audit trail's) can't be one.
+      return if model.name.nil?
+
       if model.name.start_with?('HABTM_')
         build_habtm(model)
       else

packages/forest_admin_datasource_active_record/spec/lib/forest_admin_datasource_active_record/datasource_spec.rb

@@ -15,6 +15,21 @@ module ForestAdminDatasourceActiveRecord
       expect(collections).to all(be_a(String))
     end
 
+    describe 'with an anonymous model' do
+      let!(:anonymous_model) { Class.new(ActiveRecord::Base) { self.table_name = 'accounts' } }
+
+      it 'boots instead of failing on the nameless model' do
+        expect { described_class.new({ adapter: 'sqlite3', database: 'db/database.db' }) }.not_to raise_error
+      end
+
+      it 'leaves the nameless model out of the collections and keeps the named ones' do
+        datasource = described_class.new({ adapter: 'sqlite3', database: 'db/database.db' })
+
+        expect(datasource.collections.values.map(&:model)).not_to include(anonymous_model)
+        expect(datasource.collections.keys).to include('Account')
+      end
+    end
+
     describe '#init_orm' do
       it 'uses connection_pool.db_config instead of deprecated connection_db_config' do
         mock_db_config = instance_double(ActiveRecord::DatabaseConfigurations::HashConfig, env_name: 'test')