Replace getEstimatedEffortPerOccurrence() method with field

Use this link to re-run the recipe: https://app.moderne.io/recipes/org.openrewrite.java.recipes.UseEstimatedEffortPerOccurrenceField?organizationId=QUxML01vZGVybmUvTW9kZXJuZSArIE9wZW5SZXdyaXRl

Co-authored-by: Moderne <[email protected]>

Author: 2 people

rewrite-core/src/main/java/org/openrewrite/search/RepositoryContainsFile.java

@@ -44,10 +44,7 @@ public class RepositoryContainsFile extends ScanningRecipe<AtomicBoolean> {
                "contains a specific file or files matching a pattern. If present all files in the repository are marked " +
                "with a `SearchResult` marker. If you want to get only the matching file as a search result, use `FindSourceFiles` instead.";
 
-    @Override
-    public Duration getEstimatedEffortPerOccurrence() {
-        return Duration.ZERO;
-    }
+    Duration estimatedEffortPerOccurrence = Duration.ZERO;
 
     @Override
     public AtomicBoolean getInitialValue(ExecutionContext ctx) {

rewrite-gradle/src/main/java/org/openrewrite/gradle/ChangeDependencyConfiguration.java

@@ -74,10 +74,7 @@ public Validated<Object> validate() {
         return super.validate().and(DependencyMatcher.build(groupId + ":" + artifactId));
     }
 
-    @Override
-    public Duration getEstimatedEffortPerOccurrence() {
-        return Duration.ofMinutes(5);
-    }
+    Duration estimatedEffortPerOccurrence = Duration.ofMinutes(5);
 
     @Override
     public TreeVisitor<?, ExecutionContext> getVisitor() {

rewrite-gradle/src/main/java/org/openrewrite/gradle/security/UseHttpsForRepositories.java

@@ -44,10 +44,8 @@ public class UseHttpsForRepositories extends Recipe {
     @Getter
     final Set<String> tags = singleton("security");
 
-    @Override
-    public Duration getEstimatedEffortPerOccurrence() {
-        return Duration.ofMinutes(5);
-    }
+    @Getter
+    final Duration estimatedEffortPerOccurrence = Duration.ofMinutes(5);
 
     @Override
     public TreeVisitor<?, ExecutionContext> getVisitor() {

rewrite-java/src/main/java/org/openrewrite/java/RemoveUnusedImports.java

@@ -52,10 +52,7 @@ public class RemoveUnusedImports extends Recipe {
 
     Set<String> tags = singleton("RSPEC-S1128");
 
-    @Override
-    public Duration getEstimatedEffortPerOccurrence() {
-        return Duration.ofMinutes(5);
-    }
+    Duration estimatedEffortPerOccurrence = Duration.ofMinutes(5);
 
     @Override
     public TreeVisitor<?, ExecutionContext> getVisitor() {

rewrite-java/src/main/java/org/openrewrite/java/ShortenFullyQualifiedTypeReferences.java

@@ -45,10 +45,8 @@ public class ShortenFullyQualifiedTypeReferences extends Recipe {
         "names and import statements, provided that it doesn't result in " +
         "any conflicts with other imports or types declared in the local compilation unit.";
 
-    @Override
-    public @Nullable Duration getEstimatedEffortPerOccurrence() {
-        return Duration.ofMinutes(2);
-    }
+    @Getter
+    final Duration estimatedEffortPerOccurrence = Duration.ofMinutes(2);
 
     @Override
     public JavaVisitor<ExecutionContext> getVisitor() {

rewrite-java/src/main/java/org/openrewrite/java/format/EmptyNewlineAtEndOfFile.java

@@ -48,10 +48,8 @@ public class EmptyNewlineAtEndOfFile extends Recipe {
     @Getter
     final Set<String> tags = singleton("RSPEC-S113");
 
-    @Override
-    public Duration getEstimatedEffortPerOccurrence() {
-        return Duration.ofMinutes(1);
-    }
+    @Getter
+    final Duration estimatedEffortPerOccurrence = Duration.ofMinutes(1);
 
     @Override
     public TreeVisitor<?, ExecutionContext> getVisitor() {

rewrite-java/src/main/java/org/openrewrite/java/format/WrappingAndBraces.java

@@ -39,10 +39,8 @@ public class WrappingAndBraces extends Recipe {
     @Getter
     final Set<String> tags = new LinkedHashSet<>(Arrays.asList("RSPEC-S121", "RSPEC-S2681", "RSPEC-S3972", "RSPEC-S3973"));
 
-    @Override
-    public Duration getEstimatedEffortPerOccurrence() {
-        return Duration.ofMinutes(10);
-    }
+    @Getter
+    final Duration estimatedEffortPerOccurrence = Duration.ofMinutes(10);
 
     @Override
     public TreeVisitor<?, ExecutionContext> getVisitor() {

rewrite-java/src/main/java/org/openrewrite/java/search/FindEmptyClasses.java

@@ -40,10 +40,8 @@ public class FindEmptyClasses extends Recipe {
     @Getter
     final Set<String> tags = singleton("RSPEC-S2094");
 
-    @Override
-    public Duration getEstimatedEffortPerOccurrence() {
-        return Duration.ofMinutes(5);
-    }
+    @Getter
+    final Duration estimatedEffortPerOccurrence = Duration.ofMinutes(5);
 
     @Override
     public TreeVisitor<?, ExecutionContext> getVisitor() {

rewrite-kotlin/src/main/java/org/openrewrite/kotlin/cleanup/EqualsMethodUsage.java

@@ -52,10 +52,7 @@ public class EqualsMethodUsage extends Recipe {
 
     Set<String> tags = singleton("RSPEC-S6519");
 
-    @Override
-    public Duration getEstimatedEffortPerOccurrence() {
-        return Duration.ofMinutes(3);
-    }
+    Duration estimatedEffortPerOccurrence = Duration.ofMinutes(3);
 
     @Override
     public TreeVisitor<?, ExecutionContext> getVisitor() {

rewrite-kotlin/src/main/java/org/openrewrite/kotlin/cleanup/ImplicitParameterInLambda.java

@@ -41,10 +41,7 @@ public class ImplicitParameterInLambda extends Recipe {
                "only one parameter, when this is true the compiler can determine the parameter type by context. Thus " +
                "when using it with single parameter lambda expressions, you do not need to declare the type.";
 
-    @Override
-    public Duration getEstimatedEffortPerOccurrence() {
-        return Duration.ofMinutes(2);
-    }
+    Duration estimatedEffortPerOccurrence = Duration.ofMinutes(2);
 
     Set<String> tags = singleton("RSPEC-S6558");