Add AssertJ dependency if using Hamcrest (#648)

* Add AssertJ dependency if using Hamcrest * Restore `withToolingApi()`
openrewrite · Dec 4, 2024 · 433685e · 433685e
1 parent b0df4cc
commit 433685e
Show file tree

Hide file tree

Showing 2 changed files with 185 additions and 183 deletions.
diff --git a/src/main/resources/META-INF/rewrite/hamcrest.yml b/src/main/resources/META-INF/rewrite/hamcrest.yml
@@ -39,6 +39,14 @@ tags:
   - hamcrest
   - assertj
 recipeList:
+  # Add dependency if not already present
+  - org.openrewrite.java.dependencies.AddDependency:
+      groupId: org.assertj
+      artifactId: assertj-core
+      version: 3.x
+      onlyIfUsing: org.hamcrest.*
+      acceptTransitive: true
+
   # First change `is(..)` to `Matchers.is(..)` for consistent matching
   - org.openrewrite.java.ChangeMethodTargetToStatic:
       methodPattern: org.hamcrest.core.* *(..)
@@ -319,10 +327,3 @@ recipeList:
       notMatcher: empty
       assertion: isNotEmpty
 
-  # Add dependency if not already present
-  - org.openrewrite.java.dependencies.AddDependency:
-      groupId: org.assertj
-      artifactId: assertj-core
-      version: 3.x
-      onlyIfUsing: org.assertj.core.api.Assertions
-      acceptTransitive: true