The GitHub Actions job "Coverage" on 
grails-core.git/feat/gorm-registry-core-impl has failed.
Run started by GitHub user borinquenkid (triggered by borinquenkid).

Head commit for run:
6ea005194da81d50dadc9d49b62d13f9b239f909 / Walter Duque de Estrada 
<[email protected]>
fix: resolve null datastore in H5/H7 validation and dynamic finder dispatch

Three targeted fixes that eliminate H5/H7 runtime NPEs introduced by the
GormRegistry refactor:

1. Remove setDatastore(Datastore) from AbstractDatastoreApi — adding a public
   setter for 'datastore' caused Groovy @CompileStatic to route constructor
   field assignments (e.g. this.datastore = hds in AbstractHibernateGorm-
   ValidationApi) through the setter instead of the declared local field,
   leaving that field null and causing ValidationEvent.<init> to throw
   IllegalArgumentException: null source.

2. Fix deprecated GormStaticApi(Class, Datastore, List[, 
PlatformTransactionManager])
   constructors to wire a real DatastoreResolver closure instead of null,
   so getDatastore() returns the correct Datastore at runtime for H5/H7
   adapters that still call these constructors.

3. Fix GormEnhancer.addStaticMethods mc.static.propertyMissing to convert
   any non-MissingPropertyException (e.g. ConfigurationException from H5's
   HibernateGormStaticApi.propertyMissing treating the name as a datasource
   qualifier) into MissingPropertyException so Groovy can fall through to
   methodMissing for dynamic finder dispatch (e.g. Person.countByTitle).

Co-Authored-By: Claude Sonnet 4.6 <[email protected]>

Report URL: https://github.com/apache/grails-core/actions/runs/28306160661

With regards,
GitHub Actions via GitBox

Reply via email to