Skip to content

Commit be9cadd

Browse files
committed
Expect to add import as needed
1 parent 6935d02 commit be9cadd

File tree

2 files changed

+3
-9
lines changed

2 files changed

+3
-9
lines changed

src/main/java/org/openrewrite/java/recipes/UseRewriteTestDefaults.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -69,6 +69,7 @@ public J.ClassDeclaration visitClassDeclaration(J.ClassDeclaration classDecl, Ex
6969
if (!allSpecsAreIdentical(specInfos)) {
7070
return cd;
7171
}
72+
maybeAddImport("org.openrewrite.test.RecipeSpec");
7273
cd = newlineBeforeFirstStatement(cd);
7374
cd = addDefaultsMethod(cd, specInfos.get(0));
7475
return removeSpecsFromRewriteRuns(cd, ctx);

src/test/java/org/openrewrite/java/recipes/UseRewriteTestDefaultsTest.java

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,6 @@ void shouldRefactorWhenAllTestsUseSameRecipe() {
3838
java(
3939
"""
4040
import org.junit.jupiter.api.Test;
41-
import org.openrewrite.test.RecipeSpec;
4241
import org.openrewrite.test.RewriteTest;
4342
4443
class MyTest implements RewriteTest {
@@ -96,7 +95,6 @@ void shouldNotRefactorWhenOnlyOneTest() {
9695
java(
9796
"""
9897
import org.junit.jupiter.api.Test;
99-
import org.openrewrite.test.RecipeSpec;
10098
import org.openrewrite.test.RewriteTest;
10199
102100
class MyTest implements RewriteTest {
@@ -119,7 +117,6 @@ void shouldNotRefactorWhenTestsUseDifferentRecipes() {
119117
java(
120118
"""
121119
import org.junit.jupiter.api.Test;
122-
import org.openrewrite.test.RecipeSpec;
123120
import org.openrewrite.test.RewriteTest;
124121
125122
class MyTest implements RewriteTest {
@@ -150,7 +147,6 @@ void shouldHandleComplexRecipeSpec() {
150147
java(
151148
"""
152149
import org.junit.jupiter.api.Test;
153-
import org.openrewrite.test.RecipeSpec;
154150
import org.openrewrite.test.RewriteTest;
155151
import org.openrewrite.java.JavaParser;
156152
@@ -216,7 +212,6 @@ void shouldNotRefactorWhenSomeTestsHaveDifferentSpecs() {
216212
java(
217213
"""
218214
import org.junit.jupiter.api.Test;
219-
import org.openrewrite.test.RecipeSpec;
220215
import org.openrewrite.test.RewriteTest;
221216
222217
class MyTest implements RewriteTest {
@@ -311,9 +306,8 @@ void shouldHandleNestedClasses() {
311306
rewriteRun(
312307
java(
313308
"""
314-
import org.junit.jupiter.api.Test;
315309
import org.junit.jupiter.api.Nested;
316-
import org.openrewrite.test.RecipeSpec;
310+
import org.junit.jupiter.api.Test;
317311
import org.openrewrite.test.RewriteTest;
318312
319313
class MyTest implements RewriteTest {
@@ -354,8 +348,8 @@ void test4() {
354348
}
355349
""",
356350
"""
357-
import org.junit.jupiter.api.Test;
358351
import org.junit.jupiter.api.Nested;
352+
import org.junit.jupiter.api.Test;
359353
import org.openrewrite.test.RecipeSpec;
360354
import org.openrewrite.test.RewriteTest;
361355
@@ -410,7 +404,6 @@ void shouldPreserveOtherRewriteRunOverloads() {
410404
java(
411405
"""
412406
import org.junit.jupiter.api.Test;
413-
import org.openrewrite.test.RecipeSpec;
414407
import org.openrewrite.test.RewriteTest;
415408
416409
class MyTest implements RewriteTest {

0 commit comments

Comments
 (0)