codeconsole commented on PR #16208: URL: https://github.com/apache/grails-core/pull/16208#issuecomment-5687969048
The reported MongoDB matrix, coverage, and Groovy canary failures reduce to two test issues: - Nine logging assertions attach to `AbstractDatastore`, but CI's merge revision `f0ee12d16a1994016702dc431b302eb976fc1819` includes the newer `MongoDatastore` logger. The assertions now capture the shared `org.grails.datastore.mapping` namespace, covering both revisions while retaining their message assertions. - `LocalMongoUnitSpec` now correctly throws `ValidationException` because the supplied-client settings fix honors `grails.gorm.failOnError`. Removed its obsolete `@PendingFeature` marker so this becomes an active regression test. These changes are committed locally as `211ef8de17`; pushing is awaiting authorization. No Gradle checks were rerun, as requested. The diff was reviewed and `git diff --check` passed; CI has not yet validated this follow-up. -- 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]
