Skip to content

Commit 415f9d9

Browse files
committed
Remove static qualifier from parser suppliers
1 parent 09bd8dc commit 415f9d9

File tree

3 files changed

+5
-4
lines changed

3 files changed

+5
-4
lines changed

src/main/java/org/openrewrite/java/testing/assertj/JUnitAssertSameToAssertThat.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ protected TreeVisitor<?, ExecutionContext> getVisitor() {
6060
}
6161

6262
public static class AssertSameToAssertThatVisitor extends JavaIsoVisitor<ExecutionContext> {
63-
private static final Supplier<JavaParser> ASSERTJ_JAVA_PARSER = () -> JavaParser.fromJavaVersion()
63+
private final Supplier<JavaParser> ASSERTJ_JAVA_PARSER = () -> JavaParser.fromJavaVersion()
6464
.dependsOn(Parser.Input.fromResource("/META-INF/rewrite/AssertJAssertions.java", "---")).build();
6565
private static final MethodMatcher JUNIT_ASSERT_SAME_MATCHER = new MethodMatcher("org.junit.jupiter.api.Assertions" + " assertSame(..)");
6666

src/main/java/org/openrewrite/java/testing/assertj/JUnitAssertTrueToAssertThat.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ protected TreeVisitor<?, ExecutionContext> getVisitor() {
5959
}
6060

6161
public static class AssertTrueToAssertThatVisitor extends JavaIsoVisitor<ExecutionContext> {
62-
private static final Supplier<JavaParser> ASSERTJ_JAVA_PARSER = () -> JavaParser.fromJavaVersion()
62+
private final Supplier<JavaParser> ASSERTJ_JAVA_PARSER = () -> JavaParser.fromJavaVersion()
6363
.dependsOn(Parser.Input.fromResource("/META-INF/rewrite/AssertJAssertions.java", "---")).build();
6464
private static final MethodMatcher JUNIT_ASSERT_TRUE = new MethodMatcher("org.junit.jupiter.api.Assertions" + " assertTrue(boolean, ..)");
6565

src/main/java/org/openrewrite/java/testing/assertj/JUnitFailToAssertJFail.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,6 @@
3131
public class JUnitFailToAssertJFail extends Recipe {
3232
@Override
3333
public String getDisplayName() {
34-
//noinspection GrazieInspection
3534
return "JUnit fail to AssertJ";
3635
}
3736

@@ -56,7 +55,7 @@ protected TreeVisitor<?, ExecutionContext> getVisitor() {
5655
}
5756

5857
public static class JUnitFailToAssertJFailVisitor extends JavaIsoVisitor<ExecutionContext> {
59-
private static final Supplier<JavaParser> ASSERTJ_JAVA_PARSER = () -> JavaParser.fromJavaVersion()
58+
private final Supplier<JavaParser> ASSERTJ_JAVA_PARSER = () -> JavaParser.fromJavaVersion()
6059
.dependsOn(Parser.Input.fromResource("/META-INF/rewrite/AssertJAssertions.java", "---")).build();
6160
private static final MethodMatcher JUNIT_FAIL_MATCHER = new MethodMatcher("org.junit.jupiter.api.Assertions" + " fail(..)");
6261

@@ -121,6 +120,8 @@ public J.MethodInvocation visitMethodInvocation(J.MethodInvocation method, Execu
121120
}
122121

123122
private static class UnqualifiedMethodInvocations extends JavaIsoVisitor<ExecutionContext> {
123+
private final Supplier<JavaParser> ASSERTJ_JAVA_PARSER = () -> JavaParser.fromJavaVersion()
124+
.dependsOn(Parser.Input.fromResource("/META-INF/rewrite/AssertJAssertions.java", "---")).build();
124125
private static final MethodMatcher ASSERTJ_FAIL_MATCHER = new MethodMatcher("org.assertj.core.api.Assertions" + " fail(..)");
125126

126127
@Override

0 commit comments

Comments
 (0)