refactor: OpenRewrite best practices (openrewrite#4329)

Use this link to re-run the recipe: https://app.moderne.io/recipes/org.openrewrite.recipes.OpenRewriteBestPractices?organizationId=T3BlblJld3JpdGU%3D Co-authored-by: Moderne <[email protected]>
dpozinen · Jul 15, 2024 · d9e0609 · d9e0609
1 parent 0fb7071
commit d9e0609
Show file tree

Hide file tree

Showing 5 changed files with 9 additions and 1 deletion.
diff --git a/rewrite-gradle/src/test/java/org/openrewrite/gradle/DependencyConstraintToRuleTest.java b/rewrite-gradle/src/test/java/org/openrewrite/gradle/DependencyConstraintToRuleTest.java
@@ -16,6 +16,7 @@
 package org.openrewrite.gradle;
 
 import org.junit.jupiter.api.Test;
+import org.openrewrite.DocumentExample;
 import org.openrewrite.test.RecipeSpec;
 import org.openrewrite.test.RewriteTest;
 
@@ -29,6 +30,7 @@ public void defaults(RecipeSpec spec) {
         spec.recipe(new DependencyConstraintToRule());
     }
 
+    @DocumentExample
     @Test
     void newResolutionStrategyBlock() {
         rewriteRun(

diff --git a/rewrite-java-test/src/test/java/org/openrewrite/java/search/SemanticallyEqualTest.java b/rewrite-java-test/src/test/java/org/openrewrite/java/search/SemanticallyEqualTest.java
@@ -16,8 +16,8 @@
 package org.openrewrite.java.search;
 
 import org.intellij.lang.annotations.Language;
-import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.Nested;
+import org.junit.jupiter.api.Test;
 import org.junitpioneer.jupiter.ExpectedToFail;
 import org.junitpioneer.jupiter.cartesian.CartesianTest;
 import org.openrewrite.java.JavaIsoVisitor;

diff --git a/rewrite-java/src/test/java/org/openrewrite/java/trait/LiteralTest.java b/rewrite-java/src/test/java/org/openrewrite/java/trait/LiteralTest.java
@@ -17,6 +17,7 @@
 
 import com.fasterxml.jackson.core.type.TypeReference;
 import org.junit.jupiter.api.Test;
+import org.openrewrite.DocumentExample;
 import org.openrewrite.marker.SearchResult;
 import org.openrewrite.test.RewriteTest;
 
@@ -28,6 +29,7 @@
 
 class LiteralTest implements RewriteTest {
 
+    @DocumentExample
     @Test
     void numericLiteral() {
         rewriteRun(

diff --git a/rewrite-java/src/test/java/org/openrewrite/java/trait/MethodAccessTest.java b/rewrite-java/src/test/java/org/openrewrite/java/trait/MethodAccessTest.java
@@ -16,6 +16,7 @@
 package org.openrewrite.java.trait;
 
 import org.junit.jupiter.api.Test;
+import org.openrewrite.DocumentExample;
 import org.openrewrite.Recipe;
 import org.openrewrite.java.MethodMatcher;
 import org.openrewrite.marker.SearchResult;
@@ -35,6 +36,7 @@ public void defaults(RecipeSpec spec) {
           new MethodMatcher("java.util.List add(..)", true))));
     }
 
+    @DocumentExample
     @Test
     void methodAccesses() {
         rewriteRun(

diff --git a/rewrite-java/src/test/java/org/openrewrite/java/trait/VariableAccessTest.java b/rewrite-java/src/test/java/org/openrewrite/java/trait/VariableAccessTest.java
@@ -16,6 +16,7 @@
 package org.openrewrite.java.trait;
 
 import org.junit.jupiter.api.Test;
+import org.openrewrite.DocumentExample;
 import org.openrewrite.Recipe;
 import org.openrewrite.marker.SearchResult;
 import org.openrewrite.test.RecipeSpec;
@@ -34,6 +35,7 @@ public void defaults(RecipeSpec spec) {
         spec.recipe(markVariableAccesses(variableAccess()));
     }
 
+    @DocumentExample
     @Test
     void variableAccesses() {
         rewriteRun(