jamesfredley opened a new pull request, #15435: URL: https://github.com/apache/grails-core/pull/15435
## Summary - Add test coverage for GORM static method routing (`executeQuery`, `withCriteria`, `createCriteria`, `executeUpdate`, `withTransaction`) to verify they route to the correct datasource for entities mapped to non-default datasources - Covers the `allQualifiers()` fix from commit 4e04e96 ## Changes **Unit tests** (`GormEnhancerAllQualifiersSpec`): - Verify `registerEntity()` places static APIs under both DEFAULT and named qualifier for non-default datasource entities - Covers non-MultiTenant, MultiTenant, and default datasource scenarios **TCK integration tests** (`MultipleDataSourceConnectionsSpec`): - Verify `executeQuery`, `withCriteria`, `executeUpdate`, `withTransaction`, and `withNewSession` route to the correct H2 database for multi-datasource entities - Verify ALL-mapped entities still use the default datasource **Functional tests** (`DatasourceSwitchingSpec`): - Verify `executeQuery`, `withCriteria`, `createCriteria`, `executeUpdate` route to the secondary datasource in a full Grails application - Each test saves data to both datasources and verifies only the correct datasource is queried/updated All 32 tests pass (10 unit + 4 TCK + 18 functional). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
