The GitHub Actions job "Code Analysis" on grails-core.git/fix/graphql-errors-response has succeeded. Run started by GitHub user jdaugherty (triggered by jdaugherty).
Head commit for run: 4632f8fcec3d770c467fcf56d604c31c3f9b2194 / Walter Duque de Estrada <[email protected]> Replace deprecated DataFetchingEnvironment#getContext() in getLocale() getContext() has been deprecated in graphql-java since 2021-07-05 in favor of getGraphQlContext(), but the latter only accepts Map entries and can't carry an arbitrary object such as a custom GraphQLContextBuilder result implementing LocaleAwareContext. GraphqlController already sets both .context(context) and .root(context) to the same value specifically so callers can move off the legacy context; switch getLocale() to read environment.root instead, which covers both the Map-of-locale and LocaleAwareContext cases without changing behavior. Updated the existing LocaleAwareContext spec to stub root instead of context, and added coverage for the root-Map-with-locale and no-locale-entry-falls-back-to-default branches, which weren't separately tested before. The handler spec resolves the message fetcher through the code registry rather than the protected field, GraphqlControllerSpec pins the root the handler now relies on, and the plugin guide documents that the context object is available as the execution root. Co-Authored-By: Claude Sonnet 5 <[email protected]> Report URL: https://github.com/apache/grails-core/actions/runs/36735160378 With regards, GitHub Actions via GitBox
