The GitHub Actions job "CI" on 
grails-core.git/test/document-datamapping-core-finders has succeeded.
Run started by GitHub user borinquenkid (triggered by borinquenkid).

Head commit for run:
63ba0e911bf82a39d56f2f5761c52b31814a8adf / Walter Duque de Estrada 
<[email protected]>
Close remaining PR #16154 patch-coverage gaps and fix a silent-hang bug in 
RxSingleResultFinder

Adds tests for setPattern delegation, the null-detachedCriteria branch of the
reflective 4-arg invoke() overload, the 
ConversionException->MissingMethodException
mapping, DynamicFinder's buildMatchSpec/configureQueryWithArguments/
populateArgumentsForCriteria branches, and ListOrderByFinder's Map-argument
handling, across the six composed finder classes (sync + rx).

Also fixes a genuine latent bug surfaced while closing these gaps:
RxSingleResultFinder's findOrCreateBy*/findOrSaveBy* empty-result path ran
inside a spawned Thread with no exception handling, so a thrown
MissingMethodException (e.g. from a non-Equal expression) killed the thread
silently instead of reaching the Observable's error channel - any blocking
caller would hang forever. Wrapped the thread body so failures now propagate
via Subscriber#onError, and added a test proving the fix.

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

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

With regards,
GitHub Actions via GitBox

Reply via email to