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

Co-authored-by: Moderne <[email protected]>

…eMockitoTestExecutionListener` (#925)

* Use FindImports for test framework detection in ReplaceMockitoTestExecutionListener

Replace manual import loop in detectFramework() with FindImports
search visitors for cleaner, more idiomatic framework detection.

* Check JUnit 4 imports before TestNG to match original priority order

* Add dependency-aware framework detection via targetFramework option

Add a targetFramework @option that YAML wrapper recipes set based on
ModuleHasDependency preconditions. Import-based detection still takes
priority; the option is a fallback for files with no framework imports.

Three YAML wrapper recipes gate on JUnit 5, JUnit 4, and TestNG
dependencies respectively, replacing the single bare reference in
Mockito1to3Migration.

* Simplify JUnit 4 import detection to single FindImports pattern

Since JUnit 5 (org.junit.jupiter..*) is matched first, any remaining
org.junit..* import is necessarily JUnit 4.

* Use `valid = {"jupiter", "junit4", "testng"}`

* Use same naming in precondition guarded recipes

---------

Co-authored-by: Tim te Beek <[email protected]>

The switch on `targetFramework` used `"junit5"` but the `@Option`
annotation declared valid values as `"jupiter"`, `"junit4"`, `"testng"`,
and the YAML wrapper recipes already passed `targetFramework: jupiter`.
This mismatch meant the Jupiter fallback path was unreachable.

Regenerated recipes.csv.

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

Co-authored-by: Moderne <[email protected]>

Fixes #930

Use this link to re-run the recipe: https://app.moderne.io/recipes/org.openrewrite.java.migrate.lombok.LombokBestPractices?organizationId=QUxML01vZGVybmUvTW9kZXJuZSArIE9wZW5SZXdyaXRl

Co-authored-by: Moderne <[email protected]>

#931)

The upstream AddDependency recipe now correctly scopes onlyIfUsing checks to match the dependency's scope. Test-scoped dependencies are only added when the usage is detected in test sources, so move the test class from main sources to test sources.

Co-authored-by: Tim te Beek <[email protected]>