Skip to content

GROOVY-11858: defer non-closure method for in-closure implicit-this call#2379

Open
eric-milles wants to merge 2 commits intomasterfrom
GROOVY-11858
Open

GROOVY-11858: defer non-closure method for in-closure implicit-this call#2379
eric-milles wants to merge 2 commits intomasterfrom
GROOVY-11858

Conversation

@eric-milles
Copy link
Member

Wait to execute an Object extension method until no method has been found on delegate or owner.

@codecov-commenter
Copy link

codecov-commenter commented Feb 15, 2026

Codecov Report

❌ Patch coverage is 63.63636% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 66.7223%. Comparing base (9b4ec31) to head (a688de6).
⚠️ Report is 11 commits behind head on master.

Files with missing lines Patch % Lines
...aus/groovy/runtime/metaclass/ClosureMetaClass.java 63.6364% 0 Missing and 4 partials ⚠️
Additional details and impacted files

Impacted file tree graph

@@                Coverage Diff                 @@
##               master      #2379        +/-   ##
==================================================
+ Coverage     66.7154%   66.7223%   +0.0069%     
- Complexity      29846      29854         +8     
==================================================
  Files            1382       1382                
  Lines          116180     116189         +9     
  Branches        20491      20497         +6     
==================================================
+ Hits            77510      77524        +14     
+ Misses          32323      32318         -5     
  Partials         6347       6347                
Files with missing lines Coverage Δ
src/main/java/groovy/lang/MetaClassImpl.java 77.7716% <ø> (ø)
...aus/groovy/runtime/metaclass/ClosureMetaClass.java 72.6027% <63.6364%> (-0.1888%) ⬇️

... and 3 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments