Rename and add Lombok tag

Author: timtebeek

…e/java/migrate/lombok/ConvertGetter.java …java/migrate/lombok/UseLombokGetter.javasrc/main/java/org/openrewrite/java/migrate/lombok/ConvertGetter.java renamed to src/main/java/org/openrewrite/java/migrate/lombok/UseLombokGetter.java src/main/java/org/openrewrite/java/migrate/lombok/ConvertGetter.java renamed to src/main/java/org/openrewrite/java/migrate/lombok/UseLombokGetter.java

@@ -27,6 +27,7 @@
 import org.openrewrite.java.JavaTemplate;
 import org.openrewrite.java.tree.J;
 
+import java.util.Collections;
 import java.util.HashSet;
 import java.util.Optional;
 import java.util.Set;
@@ -36,7 +37,7 @@
 
 @Value
 @EqualsAndHashCode(callSuper = false)
-public class ConvertGetter extends Recipe {
+public class UseLombokGetter extends Recipe {
 
     @Override
     public String getDisplayName() {
@@ -54,6 +55,11 @@ public String getDescription() {
                 " - Does not offer any of the configuration keys listed in https://projectlombok.org/features/GetterSetter.";
     }
 
+    @Override
+    public Set<String> getTags() {
+        return Collections.singleton("lombok");
+    }
+
     @Override
     public TreeVisitor<?, ExecutionContext> getVisitor() {
         return new MethodRemover();
@@ -84,9 +90,7 @@ public J.ClassDeclaration visitClassDeclaration(J.ClassDeclaration classDecl, Ex
 
         @Override
         public J.@Nullable MethodDeclaration visitMethodDeclaration(J.MethodDeclaration method, ExecutionContext ctx) {
-            assert method.getMethodType() != null;
-
-            if (LombokUtils.isEffectivelyGetter(method)) {
+            if (method.getMethodType() != null && LombokUtils.isEffectivelyGetter(method)) {
                 J.Return return_ = (J.Return) method.getBody().getStatements().get(0);
                 Variable fieldType = ((J.Identifier) return_.getExpression()).getFieldType();
                 boolean nameMatch = method.getSimpleName().equals(LombokUtils.deriveGetterMethodName(fieldType));
@@ -106,7 +110,6 @@ private static class Finding {
         AccessLevel accessLevel;
     }
 
-
     @Value
     @EqualsAndHashCode(callSuper = false)
     static class FieldAnnotator extends JavaIsoVisitor<ExecutionContext> {

…va/migrate/lombok/ConvertGetterTest.java …/migrate/lombok/UseLombokGetterTest.javasrc/test/java/org/openrewrite/java/migrate/lombok/ConvertGetterTest.java renamed to src/test/java/org/openrewrite/java/migrate/lombok/UseLombokGetterTest.java src/test/java/org/openrewrite/java/migrate/lombok/ConvertGetterTest.java renamed to src/test/java/org/openrewrite/java/migrate/lombok/UseLombokGetterTest.java

@@ -22,11 +22,11 @@
 
 import static org.openrewrite.java.Assertions.java;
 
-class ConvertGetterTest implements RewriteTest {
+class UseLombokGetterTest implements RewriteTest {
 
     @Override
     public void defaults(RecipeSpec spec) {
-        spec.recipe(new ConvertGetter());
+        spec.recipe(new UseLombokGetter());
     }
 
     @DocumentExample