Fixes problems with Spock and @Rollback - Fixes #9493

Author: graemerocher

grails-core/src/main/groovy/org/grails/transaction/transform/TransactionalTransform.groovy

@@ -117,7 +117,10 @@ class TransactionalTransform implements ASTTransformation{
             if (!md.isSynthetic() && Modifier.isPublic(modifiers) && !Modifier.isAbstract(modifiers) && !Modifier.isStatic(modifiers)) {
                 if(hasExcludedAnnotation(md)) continue
 
-                if( methodName.contains('$') && !methodName.startsWith('$spock') ) continue
+                def startsWithSpock = methodName.startsWith('$spock')
+                if( methodName.contains('$') && !startsWithSpock) continue
+
+                if(startsWithSpock && methodName.endsWith('proc') ) continue
 
                 if(md.getAnnotations().any { AnnotationNode an -> an.classNode.name == "org.spockframework.runtime.model.DataProviderMetadata"}) {
                     continue

grails-test-suite-uber/src/test/groovy/grails/transaction/TransactionalTransformSpec.groovy

@@ -21,6 +21,7 @@ import org.codehaus.groovy.control.MultipleCompilationErrorsException
 import org.springframework.beans.factory.annotation.Autowired
 import org.springframework.beans.factory.annotation.Qualifier
 import org.springframework.transaction.PlatformTransactionManager
+import org.springframework.util.ReflectionUtils
 
 import javax.annotation.PostConstruct
 import javax.sql.DataSource
@@ -112,12 +113,55 @@ class DemoSpec extends Specification {
 
         then:"It implements TransactionManagerAware"
         TransactionManagerAware.isAssignableFrom(mySpec)
-        mySpec.getDeclaredMethod('$spock_feature_0_0')
+        mySpec.getDeclaredMethod('$spock_feature_0_0', Object, Object)
+        mySpec.getDeclaredMethod('$spock_feature_0_0proc', Object, Object)
         mySpec.getDeclaredMethod('$spock_feature_0_0prov0')
-        !mySpec.getDeclaredMethod('$tt__$spock_feature_0_0prov0')
-        mySpec.getDeclaredMethod('$tt__$spock_feature_0_0', TransactionStatus)
+
+        !ReflectionUtils.findMethod(mySpec, '$tt__$spock_feature_0_0prov0', TransactionStatus)
+        ReflectionUtils.findMethod(mySpec, '$tt__$spock_feature_0_0', Object, Object, TransactionStatus)
+        !ReflectionUtils.findMethod(mySpec, '$tt__$spock_feature_0_0proc', Object, Object, TransactionStatus)
     }
 
+    void "Test @Rollback when applied to Spock specifications on a method and where blocks"() {
+        when:"A new instance of a class with a @Transactional method is created that subclasses another transactional class"
+        Class mySpec = new GroovyShell().evaluate('''
+import grails.test.mixin.integration.Integration
+import grails.transaction.Rollback
+import spock.lang.Specification
+
+@Integration
+
+class DemoSpec extends Specification {
+
+    @Rollback
+    def "test toUpperCase"() {
+        given:
+        def result = value.toUpperCase()
+
+        expect:
+        result == expectedResult
+
+        where:
+        value | expectedResult
+        'King Crimson' | 'KING CRIMSON\'
+        'Riverside'    | 'RIVERSIDE\'
+    }
+}
+    DemoSpec
+    ''')
+
+        then:"It implements TransactionManagerAware"
+        TransactionManagerAware.isAssignableFrom(mySpec)
+        mySpec.getDeclaredMethod('$spock_feature_0_0', Object, Object)
+        mySpec.getDeclaredMethod('$spock_feature_0_0proc', Object, Object)
+        mySpec.getDeclaredMethod('$spock_feature_0_0prov0')
+
+        !ReflectionUtils.findMethod(mySpec, '$tt__$spock_feature_0_0prov0', TransactionStatus)
+        ReflectionUtils.findMethod(mySpec, '$tt__$spock_feature_0_0', Object, Object, TransactionStatus)
+        !ReflectionUtils.findMethod(mySpec, '$tt__$spock_feature_0_0proc', Object, Object, TransactionStatus)
+    }
+
+
     @Issue('#701')
     void "Test @Transactional with a datasource specified isn't TransactionManager aware, but has appropriate autowired and qualifier"() {
         when:"A new instance of a class with a @Transactional method is created that subclasses another transactional class"