The GitHub Actions job "Coverage" on grails-core.git/chore/lint-proxy-package 
has failed.
Run started by GitHub user jdaugherty (triggered by jdaugherty).

Head commit for run:
7b33002957bce1c674577316adaa3b9815e258a6 / James Daugherty 
<[email protected]>
Route every MetaClass dispatch variant through the proxy rules and extend proxy 
coverage

Groovy dispatches property assignment, field access and non-indy property
reads on GroovyObject instances through the sender-aware MetaClass variants
(getProperty/setProperty/getAttribute/setAttribute taking a sender Class).
DelegatingMetaClass forwards those straight to the delegate with the proxy
as the receiver, so the three-argument overrides in ProxyInstanceMetaClass
were skipped: `proxy.name = 'x'` wrote to the proxy shell instead of the
proxied instance and left the proxy uninitialized, and `proxy.@name` read
the shell's field. Share the proxy rules between both arities so every
dispatch path resolves the target consistently.

Extend the coverage added by this PR to exercise real Groovy dispatch
through a proxy (property and field reads and writes, method calls, proxy
state queries), the mapping-context and direct-id fallbacks in
GroovyProxyFactory, association initialization checks, null handling and
the sender-aware MetaClass variants. Add a TCK case that writes a property
through a proxy for both proxy factories. Rename the misleading
'initialized-looking' feature and drop the unused delegate stub from the
getTarget/initialize case, which returns the target without delegating.

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

With regards,
GitHub Actions via GitBox

Reply via email to