refactor: Set default estimated effort

Co-authored-by: Moderne <[email protected]>
openrewrite · Mar 11, 2022 · a305a16 · a305a16
1 parent 13be4ea
commit a305a16
Show file tree

Hide file tree

Showing 10 changed files with 65 additions and 0 deletions.
diff --git a/src/main/java/org/openrewrite/java/quarkus/ConfigPropertiesToConfigMapping.java b/src/main/java/org/openrewrite/java/quarkus/ConfigPropertiesToConfigMapping.java
@@ -25,6 +25,8 @@
 import org.openrewrite.java.search.UsesType;
 import org.openrewrite.java.tree.J;
 
+import java.time.Duration;
+
 public class ConfigPropertiesToConfigMapping extends Recipe {
     @Override
     public String getDisplayName() {
@@ -59,4 +61,9 @@ public J.ClassDeclaration visitClassDeclaration(J.ClassDeclaration classDecl, Ex
             return cd;
         }
     }
+
+    @Override
+    public Duration getEstimatedEffortPerOccurrence() {
+        return Duration.ofMinutes(5);
+    }
 }
diff --git a/.../java/org/openrewrite/java/quarkus/ConfigureQuarkusMavenPluginWithReasonableDefaults.java b/.../java/org/openrewrite/java/quarkus/ConfigureQuarkusMavenPluginWithReasonableDefaults.java
@@ -27,6 +27,7 @@
 import org.openrewrite.xml.ChangeTagValueVisitor;
 import org.openrewrite.xml.tree.Xml;
 
+import java.time.Duration;
 import java.util.Optional;
 
 @Value
@@ -42,6 +43,11 @@ public String getDescription() {
         return "Configures the `quarkus-maven-plugin` with reasonable defaults, such as default activated `goals` and `<extensions>` configuration.";
     }
 
+    @Override
+    public Duration getEstimatedEffortPerOccurrence() {
+        return Duration.ofMinutes(5);
+    }
+
     @Override
     protected TreeVisitor<?, ExecutionContext> getVisitor() {
         return new ConfigureQuarkusMavenPluginWithReasonableDefaultsVisitor();

diff --git a/src/main/java/org/openrewrite/java/quarkus/MigrateQuarkusMavenPluginNativeImageGoal.java b/src/main/java/org/openrewrite/java/quarkus/MigrateQuarkusMavenPluginNativeImageGoal.java
@@ -26,6 +26,7 @@
 import org.openrewrite.xml.search.FindTags;
 import org.openrewrite.xml.tree.Xml;
 
+import java.time.Duration;
 import java.util.Optional;
 
 public class MigrateQuarkusMavenPluginNativeImageGoal extends Recipe {
@@ -95,4 +96,9 @@ public Xml.Document visitDocument(Xml.Document document, ExecutionContext ctx) {
         }
     }
 
+    @Override
+    public Duration getEstimatedEffortPerOccurrence() {
+        return Duration.ofMinutes(5);
+    }
+
 }
diff --git a/src/main/java/org/openrewrite/java/quarkus/MultiTransformHotStreamToMultiHotStream.java b/src/main/java/org/openrewrite/java/quarkus/MultiTransformHotStreamToMultiHotStream.java
@@ -24,6 +24,8 @@
 import org.openrewrite.java.search.UsesMethod;
 import org.openrewrite.java.tree.J;
 
+import java.time.Duration;
+
 public class MultiTransformHotStreamToMultiHotStream extends Recipe {
     private static final MethodMatcher HOT_STREAM_METHOD_MATCHER = new MethodMatcher("io.smallrye.mutiny.groups.MultiTransform toHotStream()");
 
@@ -37,6 +39,11 @@ public String getDescription() {
         return "Replace Mutiny API usages of `multi.transform().toHotStream()` with `multi.toHotStream()`.";
     }
 
+    @Override
+    public Duration getEstimatedEffortPerOccurrence() {
+        return Duration.ofMinutes(5);
+    }
+
     @Override
     protected @Nullable TreeVisitor<?, ExecutionContext> getSingleSourceApplicableTest() {
         return new UsesMethod<>(HOT_STREAM_METHOD_MATCHER);

diff --git a/src/main/java/org/openrewrite/java/quarkus/quarkus2/GrpcServiceAnnotationToGrpcClient.java b/src/main/java/org/openrewrite/java/quarkus/quarkus2/GrpcServiceAnnotationToGrpcClient.java
@@ -26,6 +26,8 @@
 import org.openrewrite.java.tree.J;
 import org.openrewrite.java.tree.TypeUtils;
 
+import java.time.Duration;
+
 public class GrpcServiceAnnotationToGrpcClient extends Recipe {
     private static final String GRPC_SERVICE_ANNOTATION_FQN = "io.quarkus.grpc.runtime.annotations.GrpcService";
     private static final String GRPC_CLIENT_ANNOTATION_FQN = "io.quarkus.grpc.GrpcClient";
@@ -40,6 +42,11 @@ public String getDescription() {
         return "The `@GrpcService` annotation is replaced with `@GrpcClient` in Quarkus 2.x. Removes the optional `@GrpcClient.value()` unless the service name is different from the name of annotated element.";
     }
 
+    @Override
+    public Duration getEstimatedEffortPerOccurrence() {
+        return Duration.ofMinutes(5);
+    }
+
     @Override
     protected TreeVisitor<?, ExecutionContext> getSingleSourceApplicableTest() {
         return new UsesType<>(GRPC_SERVICE_ANNOTATION_FQN);

diff --git a/src/main/java/org/openrewrite/java/quarkus/quarkus2/RemoveAvroMavenPlugin.java b/src/main/java/org/openrewrite/java/quarkus/quarkus2/RemoveAvroMavenPlugin.java
@@ -24,6 +24,8 @@
 import org.openrewrite.maven.search.FindPlugin;
 import org.openrewrite.xml.tree.Xml;
 
+import java.time.Duration;
+
 public class RemoveAvroMavenPlugin extends Recipe {
     @Override
     public String getDisplayName() {
@@ -35,6 +37,11 @@ public String getDescription() {
         return "Removes the `avro-maven-plugin` if the `quarkus-maven-plugin` is found in the project's `pom.xml`. Avro has been integrated with the Quarkus code generation mechanism. This replaces the need to use the Avro plugin.";
     }
 
+    @Override
+    public Duration getEstimatedEffortPerOccurrence() {
+        return Duration.ofMinutes(5);
+    }
+
     @Override
     protected @Nullable TreeVisitor<?, ExecutionContext> getApplicableTest() {
         return new MavenIsoVisitor<ExecutionContext>() {

diff --git a/src/main/java/org/openrewrite/java/quarkus/quarkus2/UseIdentifierOnDefaultKafkaBroker.java b/src/main/java/org/openrewrite/java/quarkus/quarkus2/UseIdentifierOnDefaultKafkaBroker.java
@@ -26,6 +26,7 @@
 import org.openrewrite.java.search.UsesType;
 import org.openrewrite.java.tree.J;
 
+import java.time.Duration;
 import java.util.Collections;
 
 public class UseIdentifierOnDefaultKafkaBroker extends Recipe {
@@ -51,6 +52,11 @@ public String getDescription() {
         return "Use `@io.smallrye.common.annotation.Identifier` on default kafka broker configuration.";
     }
 
+    @Override
+    public Duration getEstimatedEffortPerOccurrence() {
+        return Duration.ofMinutes(5);
+    }
+
     @Override
     protected TreeVisitor<?, ExecutionContext> getSingleSourceApplicableTest() {
         return new UsesType<>("javax.inject.Named");

diff --git a/src/main/java/org/openrewrite/java/quarkus/quarkus2/UsePanacheEntityBaseStaticMethods.java b/src/main/java/org/openrewrite/java/quarkus/quarkus2/UsePanacheEntityBaseStaticMethods.java
@@ -27,6 +27,8 @@
 import org.openrewrite.java.tree.JavaType;
 import org.openrewrite.java.tree.TypeUtils;
 
+import java.time.Duration;
+
 public class UsePanacheEntityBaseStaticMethods extends Recipe {
     private static final MethodMatcher GET_ENTITY_MANAGER = new MethodMatcher("io.quarkus.hibernate.orm.panache.PanacheEntityBase getEntityManager()");
     private static final MethodMatcher FLUSH = new MethodMatcher("io.quarkus.hibernate.orm.panache.PanacheEntityBase flush()");
@@ -41,6 +43,11 @@ public String getDescription() {
         return "The `getEntityManager()` and the `flush()` methods of `PanacheEntityBase` are now static methods.";
     }
 
+    @Override
+    public Duration getEstimatedEffortPerOccurrence() {
+        return Duration.ofMinutes(5);
+    }
+
     @Override
     protected TreeVisitor<?, ExecutionContext> getSingleSourceApplicableTest() {
         return new JavaIsoVisitor<ExecutionContext>() {

diff --git a/src/main/java/org/openrewrite/java/quarkus/quarkus2/UsePanacheEntityBaseUniT.java b/src/main/java/org/openrewrite/java/quarkus/quarkus2/UsePanacheEntityBaseUniT.java
@@ -29,6 +29,7 @@
 import org.openrewrite.java.tree.JavaType;
 import org.openrewrite.java.tree.TypeUtils;
 
+import java.time.Duration;
 import java.util.List;
 import java.util.stream.Collectors;
 import java.util.stream.Stream;
@@ -66,6 +67,11 @@ public String getDescription() {
         return "The `persist()` and `persistAndFlush()` methods now return an `Uni<T extends PanacheEntityBase>` instead of an `Uni<Void>` to allow chaining the methods.";
     }
 
+    @Override
+    public Duration getEstimatedEffortPerOccurrence() {
+        return Duration.ofMinutes(5);
+    }
+
     @Override
     protected @Nullable TreeVisitor<?, ExecutionContext> getSingleSourceApplicableTest() {
         return new JavaIsoVisitor<ExecutionContext>() {

diff --git a/...ain/java/org/openrewrite/java/quarkus/quarkus2/UseReactivePanacheMongoEntityBaseUniT.java b/...ain/java/org/openrewrite/java/quarkus/quarkus2/UseReactivePanacheMongoEntityBaseUniT.java
@@ -29,6 +29,7 @@
 import org.openrewrite.java.tree.JavaType;
 import org.openrewrite.java.tree.TypeUtils;
 
+import java.time.Duration;
 import java.util.List;
 import java.util.stream.Collectors;
 import java.util.stream.Stream;
@@ -81,6 +82,11 @@ public J.CompilationUnit visitCompilationUnit(J.CompilationUnit cu, ExecutionCon
         };
     }
 
+    @Override
+    public Duration getEstimatedEffortPerOccurrence() {
+        return Duration.ofMinutes(5);
+    }
+
     @Override
     protected TreeVisitor<?, ExecutionContext> getVisitor() {
         return new UseReactivePanacheMongoEntityBaseUniTVisitor();