Use this link to re-run the recipe: https://app.moderne.io/recipes/org.openrewrite.recipes.rewrite.OpenRewriteRecipeBestPractices?organizationId=QUxML01vZGVybmUvTW9kZXJuZSArIE9wZW5SZXdyaXRl

Co-authored-by: Moderne <team@moderne.io>

Use this link to re-run the recipe: https://app.moderne.io/recipes/org.openrewrite.recipes.rewrite.OpenRewriteRecipeBestPractices?organizationId=QUxML01vZGVybmUvTW9kZXJuZSArIE9wZW5SZXdyaXRl

Co-authored-by: Moderne <team@moderne.io>

The "remove bare mockStatic" guard didn't recognize J.Return as a valid
ancestor, so mockStatic(X.class) returned from a helper was nullified —
collapsing the return statement to `return;` and breaking compilation
against the helper's MockedStatic<X> return type.

* Upgrade junit-bom to 6.x in JUnit 5 to 6 migration

The JUnit5to6Migration recipe upgraded individual junit-jupiter
artifacts but left the org.junit:junit-bom import managed in
dependencyManagement on 5.x, including when its version was
referenced through a property.

* Use recipeFromResources in JUnit5to6MigrationTest

* Return assertion actual() instead of explicit return

Restrict to files that actually import PowerMockito, so plain
Mockito-only tests no longer run through PowerMockito conversion logic.

Fixes #980

Switch the JavaTemplate to take `Class<?>` directly and construct a
proper class literal so the resulting `assertInstanceOf` invocation
carries type info, allowing `maybeAddImport` to add the missing static
import.

…...")` (#983)

* Guard PrepareForTest fullyQualifiedNames cast against ClassCastException

Add an `instanceof J.NewArray` check before casting the assignment value
in `extractPrepareForTestClasses`, so `@PrepareForTest(fullyQualifiedNames = "...")`
with a string literal no longer crashes the recipe.

* Leave @PrepareForTest unchanged when no class references can be extracted

Filter the `J.NewArray` initializers to class references only and skip the
annotation-removal scheduling when nothing was extracted, so usages such as
`@PrepareForTest(fullyQualifiedNames = "...")` or
`@PrepareForTest(fullyQualifiedNames = {"a","b"})` leave the source intact
instead of stripping an annotation the recipe cannot migrate.

The captor registers a matcher used to capture values during later
real invocations; removing the stubbing breaks the capture.

#990)

Removing an explicit `openMocks(this)` call shifts `@Mock` fields onto
the extension/runner's strict mocking session, which may surface
`UnnecessaryStubbingException` for tests that previously benefited from
the lenient mocks created by `openMocks`. Mention the opt-out via
`@MockitoSettings(strictness = Strictness.LENIENT)`.

Refs #987

* Support `textBlock` attribute in `CsvSourceToValueSource`

Resolves #985

* Skip textBlock conversion when values contain single quotes

Single quote is CsvSource's default quote character, so converting
'apple' to a ValueSource string literal would change the parsed value.

* Use `StringUtils.containsWhitespace` and inline trivial checks

Replaces three local helpers with `StringUtils.containsWhitespace` for
the whitespace check and inline `String#indexOf` calls for the single-
character checks.

Converting `assertTrue(a == b)` to `assertSame(a, b)` changes value
equality into reference equality on the boxed types, which fails for
mixed primitive types like `123L == 123`.