Step 1 (PR-C): Add addAllDomainClasses helper to GrailsDataTckManager

grails-data-hibernate5/core/src/test/groovy/grails/gorm/hibernate/mapping/HibernateOptimisticLockingStyleMappingSpec.groovy

@@ -28,7 +28,7 @@ import org.hibernate.mapping.PersistentClass
 class HibernateOptimisticLockingStyleMappingSpec extends GrailsDataTckSpec<GrailsDataHibernate5TckManager> {
 
     void setupSpec() {
-        manager.domainClasses.addAll([HibernateOptLockingStyleVersioned, HibernateOptLockingStyleNotVersioned])
+        manager.addAllDomainClasses([HibernateOptLockingStyleVersioned, HibernateOptLockingStyleNotVersioned])
     }
 
     void testEvaluateHibernateOptimisticLockStyleIsDefined() {

grails-data-hibernate5/core/src/test/groovy/grails/gorm/tests/AutoTimestampSpec.groovy

@@ -25,7 +25,7 @@ import org.apache.grails.data.testing.tck.base.GrailsDataTckSpec
 
 class AutoTimestampSpec extends GrailsDataTckSpec<GrailsDataHibernate5TckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([DateCreatedTestA, DateCreatedTestB])
+        manager.addAllDomainClasses([DateCreatedTestA, DateCreatedTestB])
     }
 
     void "autoTimestamp should prevent custom changes to dateCreated and lastUpdated if turned on"() {

grails-data-hibernate5/core/src/test/groovy/grails/gorm/tests/CascadeToBidirectionalAsssociationSpec.groovy

@@ -29,7 +29,7 @@ import spock.lang.Issue
 @Issue('https://github.com/apache/grails-core/issues/9290')
 class CascadeToBidirectionalAsssociationSpec extends GrailsDataTckSpec<GrailsDataHibernate5TckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([Club, Team, Player, Contract])
+        manager.addAllDomainClasses([Club, Team, Player, Contract])
     }
 
     /**

grails-data-hibernate5/core/src/test/groovy/grails/gorm/tests/CountByWithEmbeddedSpec.groovy

@@ -29,7 +29,7 @@ import spock.lang.Issue
  */
 class CountByWithEmbeddedSpec extends GrailsDataTckSpec<GrailsDataHibernate5TckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([CountByPerson])
+        manager.addAllDomainClasses([CountByPerson])
     }
 
     @Issue('https://github.com/apache/grails-core/issues/9846')

grails-data-hibernate5/core/src/test/groovy/grails/gorm/tests/DeleteAllWhereSpec.groovy

@@ -28,7 +28,7 @@ import spock.lang.Issue
  */
 class DeleteAllWhereSpec extends GrailsDataTckSpec<GrailsDataHibernate5TckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([Club])
+        manager.addAllDomainClasses([Club])
     }
 
     @Issue('https://github.com/apache/grails-data-mapping/issues/969')

grails-data-hibernate5/core/src/test/groovy/grails/gorm/tests/DetachCriteriaSubquerySpec.groovy

@@ -27,7 +27,7 @@ import org.apache.grails.data.testing.tck.base.GrailsDataTckSpec
 @SuppressWarnings("GrMethodMayBeStatic")
 class DetachCriteriaSubquerySpec extends GrailsDataTckSpec<GrailsDataHibernate5TckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([User, Group, GroupAssignment, Organisation])
+        manager.addAllDomainClasses([User, Group, GroupAssignment, Organisation])
     }
 
     void "test detached associated criteria in subquery"() {

grails-data-hibernate5/core/src/test/groovy/grails/gorm/tests/DetachedCriteriaJoinSpec.groovy

@@ -29,7 +29,7 @@ import jakarta.persistence.criteria.JoinType
 
 class DetachedCriteriaJoinSpec extends GrailsDataTckSpec<GrailsDataHibernate5TckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([Team, Club, Player, Contract])
+        manager.addAllDomainClasses([Team, Club, Player, Contract])
     }
 
     def "check if count works as expected"() {

grails-data-hibernate5/core/src/test/groovy/grails/gorm/tests/DomainGetterSpec.groovy

@@ -27,7 +27,7 @@ import org.apache.grails.data.testing.tck.base.GrailsDataTckSpec
  */
 class DomainGetterSpec extends GrailsDataTckSpec<GrailsDataHibernate5TckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([DomainOne, DomainWithGetter])
+        manager.addAllDomainClasses([DomainOne, DomainWithGetter])
     }
 
     void "test a domain with a getter"() {

grails-data-hibernate5/core/src/test/groovy/grails/gorm/tests/EnumMappingSpec.groovy

@@ -29,7 +29,7 @@ import java.sql.ResultSet
  */
 class EnumMappingSpec extends GrailsDataTckSpec<GrailsDataHibernate5TckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([Recipe])
+        manager.addAllDomainClasses([Recipe])
     }
 
     void "Test enum mapping"() {

grails-data-hibernate5/core/src/test/groovy/grails/gorm/tests/HibernateValidationSpec.groovy

@@ -32,8 +32,8 @@ import org.springframework.transaction.support.TransactionSynchronizationManager
  */
 class HibernateValidationSpec extends GrailsDataTckSpec<GrailsDataHibernate5TckManager> {
     void setupSpec() {
-        manager.domainClasses += [ClassWithListArgBeforeValidate, ClassWithNoArgBeforeValidate,
-                                 ClassWithOverloadedBeforeValidate]
+        manager.addAllDomainClasses([ClassWithListArgBeforeValidate, ClassWithNoArgBeforeValidate,
+                                 ClassWithOverloadedBeforeValidate])
     }
 
     void "Test that validate works without a bound Session"() {

grails-data-hibernate5/core/src/test/groovy/grails/gorm/tests/LastUpdateWithDynamicUpdateSpec.groovy

@@ -27,7 +27,7 @@ import org.apache.grails.data.testing.tck.base.GrailsDataTckSpec
  */
 class LastUpdateWithDynamicUpdateSpec extends GrailsDataTckSpec<GrailsDataHibernate5TckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([LastUpdateTestA, LastUpdateTestB, LastUpdateTestC])
+        manager.addAllDomainClasses([LastUpdateTestA, LastUpdateTestB, LastUpdateTestC])
     }
 
     void "lastUpdated should work for dynamic update and no versioning on TestA"() {

grails-data-hibernate5/core/src/test/groovy/grails/gorm/tests/ManyToOneSpec.groovy

@@ -27,7 +27,7 @@ import org.apache.grails.data.testing.tck.base.GrailsDataTckSpec
  */
 class ManyToOneSpec extends GrailsDataTckSpec<GrailsDataHibernate5TckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([Foo, Bar])
+        manager.addAllDomainClasses([Foo, Bar])
     }
 
     static {

grails-data-hibernate5/core/src/test/groovy/grails/gorm/tests/MultiColumnUniqueConstraintSpec.groovy

@@ -27,7 +27,7 @@ import spock.lang.Issue
 @Issue('https://github.com/apache/grails-data-mapping/issues/617')
 class MultiColumnUniqueConstraintSpec extends GrailsDataTckSpec<GrailsDataHibernate5TckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([DomainOne, Task1, TaskLink])
+        manager.addAllDomainClasses([DomainOne, Task1, TaskLink])
     }
 
     void "test generated unique constraints"() {

grails-data-hibernate5/core/src/test/groovy/grails/gorm/tests/RLikeSpec.groovy

@@ -24,7 +24,7 @@ import org.apache.grails.data.testing.tck.base.GrailsDataTckSpec
 
 class RLikeSpec extends GrailsDataTckSpec<GrailsDataHibernate5TckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([RlikeFoo])
+        manager.addAllDomainClasses([RlikeFoo])
     }
 
     void "test rlike works with H2"() {

grails-data-hibernate5/core/src/test/groovy/grails/gorm/tests/ReadOperationSpec.groovy

@@ -24,7 +24,7 @@ import org.apache.grails.data.testing.tck.base.GrailsDataTckSpec
 
 class ReadOperationSpec extends GrailsDataTckSpec<GrailsDataHibernate5TckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([TestEntity])
+        manager.addAllDomainClasses([TestEntity])
     }
 
     void "test read operation for non existent"() {

grails-data-hibernate5/core/src/test/groovy/grails/gorm/tests/SizeConstraintSpec.groovy

@@ -30,7 +30,7 @@ import spock.lang.Issue
  */
 class SizeConstraintSpec extends GrailsDataTckSpec<GrailsDataHibernate5TckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([SizeConstrainedUser])
+        manager.addAllDomainClasses([SizeConstrainedUser])
     }
 
     @Issue('https://github.com/apache/grails-data-mapping/issues/846')

grails-data-hibernate5/core/src/test/groovy/grails/gorm/tests/ToOneProxySpec.groovy

@@ -27,7 +27,7 @@ import org.grails.orm.hibernate.proxy.HibernateProxyHandler
  */
 class ToOneProxySpec extends GrailsDataTckSpec<GrailsDataHibernate5TckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([Team, Club])
+        manager.addAllDomainClasses([Team, Club])
     }
 
     void "test that a proxy is not initialized on get"() {

grails-data-hibernate5/core/src/test/groovy/grails/gorm/tests/WhereQueryWithAssociationSortSpec.groovy

@@ -28,7 +28,7 @@ import spock.lang.Issue
  */
 class WhereQueryWithAssociationSortSpec extends GrailsDataTckSpec<GrailsDataHibernate5TckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([Club, Team])
+        manager.addAllDomainClasses([Club, Team])
     }
 
     @Issue('https://github.com/apache/grails-core/issues/9860')

grails-data-hibernate5/core/src/test/groovy/grails/gorm/tests/WithNewSessionAndExistingTransactionSpec.groovy

@@ -35,7 +35,7 @@ import javax.sql.DataSource
  */
 class WithNewSessionAndExistingTransactionSpec extends GrailsDataTckSpec<GrailsDataHibernate5TckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([Book])
+        manager.addAllDomainClasses([Book])
     }
 
     void "Test withNewSession when an existing transaction is present"() {

grails-data-hibernate5/core/src/test/groovy/grails/gorm/tests/autoimport/AutoImportSpec.groovy

@@ -25,7 +25,7 @@ import org.apache.grails.data.testing.tck.base.GrailsDataTckSpec
 
 class AutoImportSpec extends GrailsDataTckSpec<GrailsDataHibernate5TckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([A, grails.gorm.tests.autoimport.other.A])
+        manager.addAllDomainClasses([A, grails.gorm.tests.autoimport.other.A])
     }
 
     void "test a domain with a getter"() {

grails-data-hibernate5/core/src/test/groovy/grails/gorm/tests/belongsto/BidirectionalOneToOneWithUniqueSpec.groovy

@@ -27,7 +27,7 @@ import org.apache.grails.data.testing.tck.base.GrailsDataTckSpec
  */
 class BidirectionalOneToOneWithUniqueSpec extends GrailsDataTckSpec<GrailsDataHibernate5TckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([HibernateFace, HibernateNose])
+        manager.addAllDomainClasses([HibernateFace, HibernateNose])
     }
 
     void "test bidirectional one-to-one with unique"() {

grails-data-hibernate5/core/src/test/groovy/grails/gorm/tests/inheritance/SubclassToOneProxySpec.groovy

@@ -24,7 +24,7 @@ import org.apache.grails.data.testing.tck.base.GrailsDataTckSpec
 
 class SubclassToOneProxySpec extends GrailsDataTckSpec<GrailsDataHibernate5TckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([SuperclassProxy, SubclassProxy, HasOneProxy])
+        manager.addAllDomainClasses([SuperclassProxy, SubclassProxy, HasOneProxy])
     }
 
     void "the hasOne is a proxy and unwraps"() {

grails-data-hibernate5/core/src/test/groovy/grails/gorm/tests/inheritance/TablePerConcreteClassAndDateCreatedSpec.groovy

@@ -29,7 +29,7 @@ import spock.lang.Issue
 @Issue('https://github.com/apache/grails-data-mapping/issues/937')
 class TablePerConcreteClassAndDateCreatedSpec extends GrailsDataTckSpec<GrailsDataHibernate5TckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([Vehicle, Spaceship])
+        manager.addAllDomainClasses([Vehicle, Spaceship])
     }
 
     void "should set the dateCreated automatically"() {

grails-data-hibernate5/core/src/test/groovy/grails/gorm/tests/inheritance/TablePerConcreteClassImportedSpec.groovy

@@ -25,7 +25,7 @@ import spock.lang.Issue
 @Issue('https://github.com/grails/grails-data-hibernate5/issues/151')
 class TablePerConcreteClassImportedSpec extends GrailsDataTckSpec<GrailsDataHibernate5TckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([Vehicle, Spaceship])
+        manager.addAllDomainClasses([Vehicle, Spaceship])
     }
 
     void "test that subclasses are added to the imports on the metamodel"() {

grails-data-hibernate5/core/src/test/groovy/grails/gorm/tests/mappedby/MultipleOneToOneSpec.groovy

@@ -28,7 +28,7 @@ import spock.lang.Issue
  */
 class MultipleOneToOneSpec extends GrailsDataTckSpec<GrailsDataHibernate5TckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([Org, OrgMember])
+        manager.addAllDomainClasses([Org, OrgMember])
     }
 
     @Issue('https://github.com/apache/grails-data-mapping/issues/950')

grails-data-hibernate5/core/src/test/groovy/grails/gorm/tests/proxy/ByteBuddyProxySpec.groovy

@@ -33,7 +33,7 @@ import spock.lang.Shared
  */
 class ByteBuddyProxySpec extends GrailsDataTckSpec<GrailsDataHibernate5TckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([Team, Club])
+        manager.addAllDomainClasses([Team, Club])
     }
 
     @Shared

grails-data-hibernate5/core/src/test/groovy/grails/gorm/tests/traits/InterfacePropertySpec.groovy

@@ -28,7 +28,7 @@ import spock.lang.Issue
  */
 class InterfacePropertySpec extends GrailsDataTckSpec<GrailsDataHibernate5TckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([TestDomain])
+        manager.addAllDomainClasses([TestDomain])
     }
 
     @Issue('https://github.com/grails/grails-data-hibernate5/issues/38')

grails-data-hibernate5/core/src/test/groovy/grails/gorm/tests/validation/DeepValidationSpec.groovy

@@ -30,7 +30,7 @@ import spock.lang.Issue
  */
 class DeepValidationSpec extends GrailsDataTckSpec<GrailsDataHibernate5TckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([AnotherCity, Market, Address])
+        manager.addAllDomainClasses([AnotherCity, Market, Address])
     }
 
     @Rollback

grails-data-mongodb/core/src/test/groovy/grails/gorm/tests/DirtyCheckEmbeddedCollectionSpec.groovy

@@ -26,7 +26,7 @@ import org.bson.types.ObjectId
 class DirtyCheckEmbeddedCollectionSpec extends GrailsDataTckSpec<GrailsDataMongoTckManager> {
 
     void setupSpec() {
-        manager.domainClasses.addAll([Foo, Bar])
+        manager.addAllDomainClasses([Foo, Bar])
     }
 
     def "Test that changes to basic collections are detected"() {

grails-data-mongodb/core/src/test/groovy/grails/gorm/tests/FindNativeSpec.groovy

@@ -34,7 +34,7 @@ import static com.mongodb.client.model.Filters.eq
 class FindNativeSpec extends GrailsDataTckSpec<GrailsDataMongoTckManager> {
 
     void setupSpec() {
-        manager.domainClasses.addAll([Product])
+        manager.addAllDomainClasses([Product])
     }
 
     void "test native find method"() {

grails-data-mongodb/core/src/test/groovy/grails/gorm/tests/listener/PersistenceEventListenerSpec.groovy

@@ -38,7 +38,7 @@ class PersistenceEventListenerSpec extends GrailsDataTckSpec<GrailsDataMongoTckM
     SpecPersistenceListener listener
 
     void setupSpec() {
-        manager.domainClasses.addAll([Simples])
+        manager.addAllDomainClasses([Simples])
     }
 
     def setup() {

grails-data-mongodb/core/src/test/groovy/grails/mongodb/cascade/MongoCascadeSpec.groovy

@@ -23,7 +23,7 @@ import org.apache.grails.data.testing.tck.base.GrailsDataTckSpec
 
 class MongoCascadeSpec extends GrailsDataTckSpec<GrailsDataMongoTckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([Product, ProductLine])
+        manager.addAllDomainClasses([Product, ProductLine])
     }
 
     void "test association is not cascaded on update or insert"() {

grails-data-mongodb/core/src/test/groovy/org/grails/datastore/gorm/mongo/AggregateMethodSpec.groovy

@@ -29,7 +29,7 @@ import org.bson.types.ObjectId
  */
 class AggregateMethodSpec extends GrailsDataTckSpec<GrailsDataMongoTckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([City])
+        manager.addAllDomainClasses([City])
     }
 
     void "Test aggregate method"() {

grails-data-mongodb/core/src/test/groovy/org/grails/datastore/gorm/mongo/AssignedIdentifierSpec.groovy

@@ -29,7 +29,7 @@ import spock.lang.Issue
  */
 class AssignedIdentifierSpec extends GrailsDataTckSpec<GrailsDataMongoTckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([River, Lake, Volcano])
+        manager.addAllDomainClasses([River, Lake, Volcano])
     }
 
     void "Test that entities can be saved, retrieved and updated with assigned ids"() {

grails-data-mongodb/core/src/test/groovy/org/grails/datastore/gorm/mongo/AutowireServicesSpec.groovy

@@ -25,7 +25,7 @@ import org.springframework.context.support.GenericApplicationContext
 
 class AutowireServicesSpec extends GrailsDataTckSpec<GrailsDataMongoTckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([Pizza])
+        manager.addAllDomainClasses([Pizza])
     }
 
     void "Test that services can be autowired"() {

grails-data-mongodb/core/src/test/groovy/org/grails/datastore/gorm/mongo/BasicArraySpec.groovy

@@ -30,7 +30,7 @@ import org.bson.types.ObjectId
  */
 class BasicArraySpec extends GrailsDataTckSpec<GrailsDataMongoTckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([Data])
+        manager.addAllDomainClasses([Data])
     }
 
     void "Test that arrays are saved correctly"() {

grails-data-mongodb/core/src/test/groovy/org/grails/datastore/gorm/mongo/BasicCollectionTypeSpec.groovy

@@ -25,7 +25,7 @@ import org.apache.grails.data.testing.tck.base.GrailsDataTckSpec
 class BasicCollectionTypeSpec extends GrailsDataTckSpec<GrailsDataMongoTckManager> {
 
     void setupSpec() {
-        manager.domainClasses.addAll([MyCollections])
+        manager.addAllDomainClasses([MyCollections])
     }
 
     def "Test persist basic collection types"() {

grails-data-mongodb/core/src/test/groovy/org/grails/datastore/gorm/mongo/BasicCollectionsSpec.groovy

@@ -24,7 +24,7 @@ import org.apache.grails.data.testing.tck.base.GrailsDataTckSpec
 
 class BasicCollectionsSpec extends GrailsDataTckSpec<GrailsDataMongoTckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([Linguist, Increment])
+        manager.addAllDomainClasses([Linguist, Increment])
     }
 
     void "Test that a Locale can be used inside a collection"() {

grails-data-mongodb/core/src/test/groovy/org/grails/datastore/gorm/mongo/BatchUpdateDeleteSpec.groovy

@@ -33,7 +33,7 @@ import org.grails.datastore.gorm.query.transform.ApplyDetachedCriteriaTransform
 class BatchUpdateDeleteSpec extends GrailsDataTckSpec<GrailsDataMongoTckManager> {
 
     void setupSpec() {
-        manager.domainClasses.addAll([BatchUser, BatchAddress, Plant])
+        manager.addAllDomainClasses([BatchUser, BatchAddress, Plant])
     }
 
     void "Test that batch delete works"() {

grails-data-mongodb/core/src/test/groovy/org/grails/datastore/gorm/mongo/BeforeInsertUpdateSpec.groovy

@@ -29,7 +29,7 @@ import spock.lang.Issue
  */
 class BeforeInsertUpdateSpec extends GrailsDataTckSpec<GrailsDataMongoTckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([BeforeInsertUser])
+        manager.addAllDomainClasses([BeforeInsertUser])
     }
 
     @Issue('GPMONGODB-251')

grails-data-mongodb/core/src/test/groovy/org/grails/datastore/gorm/mongo/BeforeUpdatePropertyPersistenceSpec.groovy

@@ -33,7 +33,7 @@ import spock.lang.Issue
 class BeforeUpdatePropertyPersistenceSpec extends GrailsDataTckSpec<GrailsDataMongoTckManager> {
 
     void setupSpec() {
-        manager.domainClasses.addAll([UserWithBeforeUpdate, UserWithBeforeUpdateAndAutoTimestamp])
+        manager.addAllDomainClasses([UserWithBeforeUpdate, UserWithBeforeUpdateAndAutoTimestamp])
     }
 
     @Issue('GRAILS-15139')

grails-data-mongodb/core/src/test/groovy/org/grails/datastore/gorm/mongo/BigDecimalSpec.groovy

@@ -30,7 +30,7 @@ import org.bson.types.Decimal128
 class BigDecimalSpec extends GrailsDataTckSpec<GrailsDataMongoTckManager> {
 
     void setupSpec() {
-        manager.domainClasses.addAll([BossMan])
+        manager.addAllDomainClasses([BossMan])
     }
 
     void "test save and retrieve big decimal value"() {

grails-data-mongodb/core/src/test/groovy/org/grails/datastore/gorm/mongo/BrokenManyToManyAssociationSpec.groovy

@@ -30,7 +30,7 @@ import org.bson.Document
 class BrokenManyToManyAssociationSpec extends GrailsDataTckSpec<GrailsDataMongoTckManager> {
 
     void setupSpec() {
-        manager.domainClasses.addAll([ReferencingEntity, ReferencedEntity])
+        manager.addAllDomainClasses([ReferencingEntity, ReferencedEntity])
     }
 
     def 'Perform a cascading delete on a broken many-to-many relationship'() {

grails-data-mongodb/core/src/test/groovy/org/grails/datastore/gorm/mongo/CascadeDeleteOneToOneSpec.groovy

@@ -28,7 +28,7 @@ import org.bson.types.ObjectId
  */
 class CascadeDeleteOneToOneSpec extends GrailsDataTckSpec<GrailsDataMongoTckManager> {
     void setupSpec() {
-        manager.domainClasses.addAll([SystemUser, UserSettings, Company, Executive, Employee])
+        manager.addAllDomainClasses([SystemUser, UserSettings, Company, Executive, Employee])
     }
 
     void "Test owner deletes child in one-to-one cascade"() {