Skip to content

Commit 8ba56a1

Browse files
committed
Only change from apply to test when not using any filter methods
1 parent 295edd0 commit 8ba56a1

File tree

2 files changed

+19
-13
lines changed

2 files changed

+19
-13
lines changed

src/main/java/org/openrewrite/java/migrate/guava/NoGuavaPredicate.java

Lines changed: 19 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -15,10 +15,8 @@
1515
*/
1616
package org.openrewrite.java.migrate.guava;
1717

18-
import org.openrewrite.ExecutionContext;
19-
import org.openrewrite.Preconditions;
20-
import org.openrewrite.Recipe;
21-
import org.openrewrite.TreeVisitor;
18+
import org.openrewrite.*;
19+
import org.openrewrite.java.ChangeMethodName;
2220
import org.openrewrite.java.ChangeType;
2321
import org.openrewrite.java.JavaIsoVisitor;
2422
import org.openrewrite.java.MethodMatcher;
@@ -42,11 +40,23 @@ public String getDescription() {
4240
public TreeVisitor<?, ExecutionContext> getVisitor() {
4341
return Preconditions.check(
4442
Preconditions.not(new UsesPredicateMethod<>()),
45-
new ChangeType(
46-
"com.google.common.base.Predicate",
47-
"java.util.function.Predicate",
48-
false)
49-
.getVisitor()
43+
new TreeVisitor<Tree, ExecutionContext>() {
44+
@Override
45+
public Tree preVisit(Tree tree, ExecutionContext ctx) {
46+
Tree t = tree;
47+
t = new ChangeMethodName(
48+
"com.google.common.base.Predicate apply(..)",
49+
"test",
50+
true,
51+
false
52+
).getVisitor().visitNonNull(t, ctx, getCursor().getParentOrThrow());
53+
return new ChangeType(
54+
"com.google.common.base.Predicate",
55+
"java.util.function.Predicate",
56+
false
57+
).getVisitor().visitNonNull(t, ctx, getCursor().getParentOrThrow());
58+
}
59+
}
5060
);
5161
}
5262

src/main/resources/META-INF/rewrite/no-guava.yml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -191,10 +191,6 @@ recipeList:
191191
methodPattern: com.google.common.base.Predicates isEqual(..)
192192
fullyQualifiedTargetTypeName: java.util.function.Predicate
193193

194-
- org.openrewrite.java.ChangeMethodName:
195-
methodPattern: com.google.common.base.Predicate apply(..)
196-
newMethodName: test
197-
matchOverrides: true
198194
- org.openrewrite.java.migrate.guava.NoGuavaPredicate
199195

200196
---

0 commit comments

Comments
 (0)