Skip to content

Commit 3c16fbc

Browse files
timtebeekTeamModerne
authored andcommitted
Replace getDisplayName() and getDescription() methods with fields
Use this link to re-run the recipe: https://app.moderne.io/recipes/org.openrewrite.java.recipes.UseDisplayNameAndDescriptionFields?organizationId=QUxML01vZGVybmUvTW9kZXJuZSArIE9wZW5SZXdyaXRl Co-authored-by: Moderne <team@moderne.io>
1 parent 7150900 commit 3c16fbc

106 files changed

Lines changed: 557 additions & 875 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

src/main/java/org/openrewrite/java/testing/arquillian/ReplaceArquillianInSequenceAnnotation.java

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
*/
1616
package org.openrewrite.java.testing.arquillian;
1717

18+
import lombok.Getter;
1819
import org.openrewrite.ExecutionContext;
1920
import org.openrewrite.Preconditions;
2021
import org.openrewrite.Recipe;
@@ -28,15 +29,11 @@
2829

2930
public class ReplaceArquillianInSequenceAnnotation extends Recipe {
3031

31-
@Override
32-
public String getDisplayName() {
33-
return "Arquillian JUnit 4 `@InSequence` to JUnit Jupiter `@Order`";
34-
}
32+
@Getter
33+
final String displayName = "Arquillian JUnit 4 `@InSequence` to JUnit Jupiter `@Order`";
3534

36-
@Override
37-
public String getDescription() {
38-
return "Transforms the Arquillian JUnit 4 `@InSequence` to the JUnit Jupiter `@Order`.";
39-
}
35+
@Getter
36+
final String description = "Transforms the Arquillian JUnit 4 `@InSequence` to the JUnit Jupiter `@Order`.";
4037

4138
@Override
4239
public TreeVisitor<?, ExecutionContext> getVisitor() {

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

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
*/
1616
package org.openrewrite.java.testing.assertj;
1717

18+
import lombok.Getter;
1819
import org.openrewrite.ExecutionContext;
1920
import org.openrewrite.Preconditions;
2021
import org.openrewrite.Recipe;
@@ -75,15 +76,11 @@ public class AdoptAssertJDurationAssertions extends Recipe {
7576
put("isEqualTo", "isZero");
7677
}};
7778

78-
@Override
79-
public String getDisplayName() {
80-
return "Adopt AssertJ Duration assertions";
81-
}
79+
@Getter
80+
final String displayName = "Adopt AssertJ Duration assertions";
8281

83-
@Override
84-
public String getDescription() {
85-
return "Adopt AssertJ `DurationAssert` assertions for more expressive messages.";
86-
}
82+
@Getter
83+
final String description = "Adopt AssertJ `DurationAssert` assertions for more expressive messages.";
8784

8885
@Override
8986
public TreeVisitor<?, ExecutionContext> getVisitor() {

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

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
*/
1616
package org.openrewrite.java.testing.assertj;
1717

18+
import lombok.Getter;
1819
import org.openrewrite.ExecutionContext;
1920
import org.openrewrite.Preconditions;
2021
import org.openrewrite.Recipe;
@@ -36,15 +37,11 @@
3637
public class CollapseConsecutiveAssertThatStatements extends Recipe {
3738
private static final MethodMatcher ASSERT_THAT = new MethodMatcher("org.assertj.core.api.Assertions assertThat(..)");
3839

39-
@Override
40-
public String getDisplayName() {
41-
return "Collapse consecutive `assertThat` statements";
42-
}
40+
@Getter
41+
final String displayName = "Collapse consecutive `assertThat` statements";
4342

44-
@Override
45-
public String getDescription() {
46-
return "Collapse consecutive `assertThat` statements into single `assertThat` chained statement. This recipe ignores `assertThat` statements that have method invocation as parameter.";
47-
}
43+
@Getter
44+
final String description = "Collapse consecutive `assertThat` statements into single `assertThat` chained statement. This recipe ignores `assertThat` statements that have method invocation as parameter.";
4845

4946
@Override
5047
public TreeVisitor<?, ExecutionContext> getVisitor() {

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

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
*/
1616
package org.openrewrite.java.testing.assertj;
1717

18+
import lombok.Getter;
1819
import org.openrewrite.ExecutionContext;
1920
import org.openrewrite.Preconditions;
2021
import org.openrewrite.Recipe;
@@ -37,15 +38,11 @@ public class JUnitAssertArrayEqualsToAssertThat extends Recipe {
3738
private static final String ASSERTJ = "org.assertj.core.api.Assertions";
3839
private static final MethodMatcher ASSERT_ARRAY_EQUALS_MATCHER = new MethodMatcher(JUNIT + " assertArrayEquals(..)", true);
3940

40-
@Override
41-
public String getDisplayName() {
42-
return "JUnit `assertArrayEquals` to assertJ";
43-
}
41+
@Getter
42+
final String displayName = "JUnit `assertArrayEquals` to assertJ";
4443

45-
@Override
46-
public String getDescription() {
47-
return "Convert JUnit-style `assertArrayEquals()` to AssertJ's `assertThat().contains()` equivalents.";
48-
}
44+
@Getter
45+
final String description = "Convert JUnit-style `assertArrayEquals()` to AssertJ's `assertThat().contains()` equivalents.";
4946

5047
@Override
5148
public TreeVisitor<?, ExecutionContext> getVisitor() {

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

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
*/
1616
package org.openrewrite.java.testing.assertj;
1717

18+
import lombok.Getter;
1819
import org.openrewrite.ExecutionContext;
1920
import org.openrewrite.Preconditions;
2021
import org.openrewrite.Recipe;
@@ -37,15 +38,11 @@ public class JUnitAssertEqualsToAssertThat extends Recipe {
3738
private static final String ASSERTJ = "org.assertj.core.api.Assertions";
3839
private static final MethodMatcher ASSERT_EQUALS_MATCHER = new MethodMatcher(JUNIT + " assertEquals(..)", true);
3940

40-
@Override
41-
public String getDisplayName() {
42-
return "JUnit `assertEquals` to AssertJ";
43-
}
41+
@Getter
42+
final String displayName = "JUnit `assertEquals` to AssertJ";
4443

45-
@Override
46-
public String getDescription() {
47-
return "Convert JUnit-style `assertEquals()` to AssertJ's `assertThat().isEqualTo()`.";
48-
}
44+
@Getter
45+
final String description = "Convert JUnit-style `assertEquals()` to AssertJ's `assertThat().isEqualTo()`.";
4946

5047
@Override
5148
public TreeVisitor<?, ExecutionContext> getVisitor() {

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

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
*/
1616
package org.openrewrite.java.testing.assertj;
1717

18+
import lombok.Getter;
1819
import org.openrewrite.ExecutionContext;
1920
import org.openrewrite.Preconditions;
2021
import org.openrewrite.Recipe;
@@ -33,15 +34,11 @@ public class JUnitAssertFalseToAssertThat extends Recipe {
3334

3435
private static final MethodMatcher ASSERT_FALSE_MATCHER = new MethodMatcher("org.junit.jupiter.api.Assertions assertFalse(boolean, ..)", true);
3536

36-
@Override
37-
public String getDisplayName() {
38-
return "JUnit `assertFalse` to AssertJ";
39-
}
37+
@Getter
38+
final String displayName = "JUnit `assertFalse` to AssertJ";
4039

41-
@Override
42-
public String getDescription() {
43-
return "Convert JUnit-style `assertFalse()` to AssertJ's `assertThat().isFalse()`.";
44-
}
40+
@Getter
41+
final String description = "Convert JUnit-style `assertFalse()` to AssertJ's `assertThat().isFalse()`.";
4542

4643
@Override
4744
public TreeVisitor<?, ExecutionContext> getVisitor() {

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

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
*/
1616
package org.openrewrite.java.testing.assertj;
1717

18+
import lombok.Getter;
1819
import org.openrewrite.ExecutionContext;
1920
import org.openrewrite.Preconditions;
2021
import org.openrewrite.Recipe;
@@ -31,15 +32,11 @@ public class JUnitAssertInstanceOfToAssertThat extends Recipe {
3132

3233
private static final MethodMatcher ASSERT_INSTANCE_OF_MATCHER = new MethodMatcher("org.junit.jupiter.api.Assertions assertInstanceOf(..)", true);
3334

34-
@Override
35-
public String getDisplayName() {
36-
return "JUnit `assertInstanceOf` to AssertJ";
37-
}
35+
@Getter
36+
final String displayName = "JUnit `assertInstanceOf` to AssertJ";
3837

39-
@Override
40-
public String getDescription() {
41-
return "Convert JUnit-style `assertInstanceOf()` to AssertJ's `assertThat().isInstanceOf()`.";
42-
}
38+
@Getter
39+
final String description = "Convert JUnit-style `assertInstanceOf()` to AssertJ's `assertThat().isInstanceOf()`.";
4340

4441
@Override
4542
public TreeVisitor<?, ExecutionContext> getVisitor() {

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

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
*/
1616
package org.openrewrite.java.testing.assertj;
1717

18+
import lombok.Getter;
1819
import org.openrewrite.ExecutionContext;
1920
import org.openrewrite.Preconditions;
2021
import org.openrewrite.Recipe;
@@ -37,15 +38,11 @@ public class JUnitAssertNotEqualsToAssertThat extends Recipe {
3738
private static final String ASSERTJ = "org.assertj.core.api.Assertions";
3839
private static final MethodMatcher ASSERT_NOT_EQUALS_MATCHER = new MethodMatcher(JUNIT + " assertNotEquals(..)", true);
3940

40-
@Override
41-
public String getDisplayName() {
42-
return "JUnit `assertNotEquals` to AssertJ";
43-
}
41+
@Getter
42+
final String displayName = "JUnit `assertNotEquals` to AssertJ";
4443

45-
@Override
46-
public String getDescription() {
47-
return "Convert JUnit-style `assertNotEquals()` to AssertJ's `assertThat().isNotEqualTo()`.";
48-
}
44+
@Getter
45+
final String description = "Convert JUnit-style `assertNotEquals()` to AssertJ's `assertThat().isNotEqualTo()`.";
4946

5047
@Override
5148
public TreeVisitor<?, ExecutionContext> getVisitor() {

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

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
*/
1616
package org.openrewrite.java.testing.assertj;
1717

18+
import lombok.Getter;
1819
import org.openrewrite.ExecutionContext;
1920
import org.openrewrite.Preconditions;
2021
import org.openrewrite.Recipe;
@@ -33,15 +34,11 @@ public class JUnitAssertNotNullToAssertThat extends Recipe {
3334

3435
private static final MethodMatcher ASSERT_NOT_NULL_MATCHER = new MethodMatcher("org.junit.jupiter.api.Assertions assertNotNull(..)", true);
3536

36-
@Override
37-
public String getDisplayName() {
38-
return "JUnit `assertNotNull` to AssertJ";
39-
}
37+
@Getter
38+
final String displayName = "JUnit `assertNotNull` to AssertJ";
4039

41-
@Override
42-
public String getDescription() {
43-
return "Convert JUnit-style `assertNotNull()` to AssertJ's `assertThat().isNotNull()`.";
44-
}
40+
@Getter
41+
final String description = "Convert JUnit-style `assertNotNull()` to AssertJ's `assertThat().isNotNull()`.";
4542

4643
@Override
4744
public TreeVisitor<?, ExecutionContext> getVisitor() {

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

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
*/
1616
package org.openrewrite.java.testing.assertj;
1717

18+
import lombok.Getter;
1819
import org.openrewrite.ExecutionContext;
1920
import org.openrewrite.Preconditions;
2021
import org.openrewrite.Recipe;
@@ -33,15 +34,11 @@ public class JUnitAssertNullToAssertThat extends Recipe {
3334

3435
private static final MethodMatcher ASSERT_NULL_MATCHER = new MethodMatcher("org.junit.jupiter.api.Assertions assertNull(..)", true);
3536

36-
@Override
37-
public String getDisplayName() {
38-
return "JUnit `assertNull` to AssertJ";
39-
}
37+
@Getter
38+
final String displayName = "JUnit `assertNull` to AssertJ";
4039

41-
@Override
42-
public String getDescription() {
43-
return "Convert JUnit-style `assertNull()` to AssertJ's `assertThat().isNull()`.";
44-
}
40+
@Getter
41+
final String description = "Convert JUnit-style `assertNull()` to AssertJ's `assertThat().isNull()`.";
4542

4643
@Override
4744
public TreeVisitor<?, ExecutionContext> getVisitor() {

0 commit comments

Comments
 (0)