borinquenkid opened a new pull request, #16188: URL: https://github.com/apache/grails-core/pull/16188
## Summary - Add `ReflectionUtilsSpec` covering `isMethodOverriddenFromParent`: a genuinely overridden public method, a method unique to the subclass, a method whose declaring class has no superclass (e.g. `Object`), and a package-private method that only shadows a same-named parent method. - Expand `ClasspathEntityScannerSpec` from a single happy-path test to cover `jakarta.persistence.Entity` detection, non-entity classes being excluded, empty package input, de-duplication across a repeated package, combining results across multiple packages, and the previously-untested "ignored package" branch (`com`, `org`, etc. are skipped rather than scanned). - Minor cleanup in `ReflectionUtils.isMethodOverriddenFromParent`: drop a dead null check — `Class#getMethod` never returns null, it throws `NoSuchMethodException` instead. ## Test plan - [x] `./gradlew :grails-datamapping-core:test --tests "org.grails.datastore.gorm.utils.ClasspathEntityScannerSpec" --tests "org.grails.datastore.gorm.utils.ReflectionUtilsSpec"` - [x] `./gradlew :grails-datamapping-core:test :grails-datamapping-core:codenarcMain :grails-datamapping-core:codenarcTest` -- 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]
