#3156 = do not transform interfaces

coiouhkc · Jun 17, 2023 · 722f905 · 722f905
1 parent acb5cf5
commit 722f905
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 1 deletion.
diff --git a/rewrite-java/src/main/java/org/openrewrite/java/LombokUtilityClass.java b/rewrite-java/src/main/java/org/openrewrite/java/LombokUtilityClass.java
@@ -138,6 +138,9 @@ public J.ClassDeclaration visitClassDeclaration(
                 final J.ClassDeclaration classDecl,
                 final AtomicBoolean shouldPerformChanges
         ) {
+            if (classDecl.getType().hasFlags(Flag.Interface)) {
+                shouldPerformChanges.set(false);
+            }
             if (classDecl.hasModifier(J.Modifier.Type.Abstract)) {
                 shouldPerformChanges.set(false);
             }

diff --git a/rewrite-java/src/test/java/org/openrewrite/java/LombokUtilityClassTest.java b/rewrite-java/src/test/java/org/openrewrite/java/LombokUtilityClassTest.java
@@ -23,7 +23,6 @@
 
 /**
  * Cases to test:
- * - Interfaces
  * - Empty classes & interfaces
  * - inheritance
  * - instantiations of changed classes
@@ -280,6 +279,22 @@ public static void doSmth() {
             );
         }
 
+        @Test
+        void givenInteface() {
+            rewriteRun(
+                    recipeSpec -> recipeSpec.recipe(new LombokUtilityClass()),
+                    java(
+                            """
+                                    public interface A {                                    
+                                       int CONST = 1;
+                                       static void doSmth() {
+                                       }
+                                    }
+                                    """
+                    )
+            );
+        }
+
     }
 
     @Test