Skip to content

Commit 1d2783b

Browse files
committed
Fix Groovy 4.0 compatibility: Use Closure.DELEGATE_FIRST instead of DELEGATE_FIRST
- In Groovy 4.0, DELEGATE_FIRST is no longer a top-level constant - Must be accessed via Closure.DELEGATE_FIRST - Fixes 13 out of 19 test failures (from 19 failures down to 6) - Remaining 6 failures appear to be test logic issues, not Groovy compatibility
1 parent 4b0ce34 commit 1d2783b

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/main/groovy/com/ullink/NuGetSpec.groovy

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -98,7 +98,7 @@ class NuGetSpec extends Exec {
9898
visitor = { entry ->
9999
switch (entry) {
100100
case Closure:
101-
entry.resolveStrategy = DELEGATE_FIRST
101+
entry.resolveStrategy = Closure.DELEGATE_FIRST
102102
entry.delegate = delegate
103103
entry.call()
104104
break

src/test/groovy/com/ullink/packageparser/PackageReferenceParserSpec.groovy

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ class PackageReferenceParserSpec extends Specification {
2828
def xml = new MarkupBuilder(writer)
2929
xml.dependencies() {
3030
result.each {
31-
it.resolveStrategy = DELEGATE_FIRST
31+
it.resolveStrategy = Closure.DELEGATE_FIRST
3232
it.delegate = delegate
3333
it.call()
3434
}

0 commit comments

Comments
 (0)