Review Feedback Vol. 1

Author: Vampire

gradle/libs.versions.toml

@@ -3,7 +3,7 @@ asciidoctorj = '3.0.0'
 groovy2 = '2.5.23'
 groovy3 = '3.0.25'
 groovy4 = '4.0.29'
-groovy5 = '5.0.1'
+groovy5 = '5.0.2'
 jacoco = '0.8.14'
 junit = '5.14.0'
 # The VersionRange used by OSGi to check compatibility with JUnit Platform.

spock-specs/src/test/groovy/org/spockframework/smoke/mock/GroovyMockAbstractGlobalClass.groovy

@@ -138,24 +138,15 @@ class GroovyMockAbstractGlobalClass extends Specification {
   }
 
   static abstract class AbstractClassA {
-    //TODO: verify whether it is necessity to qualify the field access here due to a change in Groovy 5
-    //      or whether it is a Spock bug that needs to be fixed
-    //      https://issues.apache.org/jira/browse/GROOVY-11781
-    static String getStaticName() { GroovyMockAbstractGlobalClass.NAME }
-
-    //TODO: verify whether it is necessity to qualify the field access here due to a change in Groovy 5
-    //      or whether it is a Spock bug that needs to be fixed
-    //      https://issues.apache.org/jira/browse/GROOVY-11781
+    static String getStaticName() { NAME }
+
     @SuppressWarnings('GrMethodMayBeStatic')
-    String getName() { GroovyMockAbstractGlobalClass.NAME }
+    String getName() { NAME }
   }
 
   static class SubClassA extends AbstractClassA {}
 
   static abstract class ClassB {
-    //TODO: verify whether it is necessity to qualify the field access here due to a change in Groovy 5
-    //      or whether it is a Spock bug that needs to be fixed
-    //      https://issues.apache.org/jira/browse/GROOVY-11781
-    static String getStaticName() { GroovyMockAbstractGlobalClass.NAME }
+    static String getStaticName() { NAME }
   }
 }