borinquenkid opened a new pull request, #16155: URL: https://github.com/apache/grails-core/pull/16155
## Summary - Adds unit test coverage for `GroovyProxyFactory` and `ProxyInstanceMetaClass` in `grails-datamapping-core`, which previously had none in this module despite implementing the core Groovy-based proxy contract used outside Hibernate/Neo4j. - Fixes IntelliJ inspection warnings in both classes: methods with no instance-state dependency marked `static`, replaces deprecated `Class.newInstance()` with `getDeclaredConstructor().newInstance()`, makes never-reassigned fields `final`, suppresses the unchecked-generics warning inherent to `MetaClass.getTheClass()`'s raw return type, and fills in missing Javadoc tag descriptions. ## Test plan - [x] `./gradlew :grails-datamapping-core:test --tests "org.grails.datastore.gorm.proxy.*"` — 29 tests pass - [x] `./gradlew :grails-datamapping-core:test` — full module suite passes - [x] `./gradlew :grails-datamapping-core:codeStyle` — no Checkstyle/CodeNarc violations 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- 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]
