fix: exclude interfaces from transformation

Signed-off-by: Joke de Buhr <[email protected]>
joke · Nov 27, 2022 · b48f9dc · b48f9dc
1 parent 1420bd9
commit b48f9dc
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 2 deletions.
diff --git a/spock-mockable/build.gradle b/spock-mockable/build.gradle
@@ -81,7 +81,8 @@ test {
     useJUnitPlatform()
     jvmArgs += [
             '-Dorg.slf4j.simpleLogger.defaultLogLevel=debug',
-            '-Dspock-mockable.disabled=true'
+            '-Dspock-mockable.disabled=true',
+//            '-agentlib:jdwp=transport=dt_socket,server=n,address=localhost:5005,suspend=n'
     ]
 }
 
@@ -94,3 +95,10 @@ publishing {
         }
     }
 }
+
+tasks.withType(GroovyCompile) {
+//    groovyOptions.fork = true
+//    groovyOptions.forkOptions.jvmArgs = [
+//            '-agentlib:jdwp=transport=dt_socket,server=n,address=localhost:5006,suspend=n'
+//    ]
+}
diff --git a/spock-mockable/src/main/java/io/github/joke/spockmockable/ast/visitors/MockVisitor.java b/spock-mockable/src/main/java/io/github/joke/spockmockable/ast/visitors/MockVisitor.java
@@ -93,10 +93,16 @@ protected Optional<Class<?>> extractClass(final ClassNode classNode) {
         protected void checkAndLogError(@Nullable final Class<?> clazz) {
             if (clazz != null) {
                 classDetected = true;
-                classCollector.addClass(clazz);
+                if (isTransformable(clazz)) {
+                    classCollector.addClass(clazz);
+                }
             } else if (!classDetected) {
                 logError();
             }
         }
+
+        protected boolean isTransformable(final Class<?> clazz) {
+            return clazz != Object.class && !clazz.isInterface();
+        }
     }
 }