Fix order of Hamcrest migration recipes

Make sure to run `HamcrestOfMatchersToAssertJ` before `HamcrestIsMatcherToAssertJ`, so that `assertThat(x, allOf(..))` is first converted to `assertThat(x).satisfies(..)`, where any previous `is(..)` assertion will now be expanded to a `arg -> assertThat(arg, is(..))` lambda, which can then be handled by `HamcrestIsMatcherToAssertJ`.

See:
 - #858

Author: knutwannheden

src/main/resources/META-INF/rewrite/hamcrest.yml

@@ -93,16 +93,16 @@ recipeList:
       onlyIfUsing: org.hamcrest.*
       acceptTransitive: true
 
-  # First change `is(..)` to `Matchers.is(..)` for consistent matching
+  # Normalize Hamcrest imports to `org.hamcrest.Matchers` and remove wrapping `is(Matcher)` calls
   - org.openrewrite.java.testing.hamcrest.ConsistentHamcrestMatcherImports
 
-  # Then remove calls to `MatcherAssert.assertThat(String, is(Matcher))`
-  - org.openrewrite.java.testing.hamcrest.HamcrestIsMatcherToAssertJ
-
-  # Then remove calls to `MatcherAssert.assertThat(String, anyOf(..))` and `allOf(..)`
+  # Convert `assertThat(x, allOf(..))` to `assertThat(x).satisfies(..)` and `anyOf` to `satisfiesAnyOf`
   - org.openrewrite.java.testing.hamcrest.HamcrestOfMatchersToAssertJ
 
-  # Then remove calls to `MatcherAssert.assertThat(String, boolean)`
+  # Convert `assertThat(x, is(value))` to `assertThat(x).isEqualTo(value)`
+  - org.openrewrite.java.testing.hamcrest.HamcrestIsMatcherToAssertJ
+
+  # Convert `assertThat(reason, boolean)` to AssertJ
   - org.openrewrite.java.testing.hamcrest.AssertThatBooleanToAssertJ
 
   - org.openrewrite.java.testing.hamcrest.HamcrestMatcherToAssertJ:

src/test/java/org/openrewrite/java/testing/hamcrest/MigrateHamcrestToAssertJTest.java

@@ -802,6 +802,49 @@ void bar(List<String> list) {
             );
         }
 
+        @Issue("https://github.com/openrewrite/rewrite-testing-frameworks/issues/858#issuecomment-3600925842")
+        @Test
+        void allOfWithIsAndNotNullValue() {
+            rewriteRun(
+              //language=java
+              java(
+                """
+                  import org.junit.jupiter.api.Test;
+
+                  import static org.hamcrest.MatcherAssert.assertThat;
+                  import static org.hamcrest.Matchers.allOf;
+                  import static org.hamcrest.Matchers.is;
+                  import static org.hamcrest.Matchers.notNullValue;
+
+                  class ATest {
+                      @Test
+                      void test() {
+                          String str = "str";
+                          assertThat(str, allOf(notNullValue(), is("SQL")));
+                      }
+                  }
+                  """,
+                """
+                  import org.junit.jupiter.api.Test;
+
+                  import static org.assertj.core.api.Assertions.assertThat;
+
+                  class ATest {
+                      @Test
+                      void test() {
+                          String str = "str";
+                          assertThat(str)
+                                  .satisfies(
+                                          arg -> assertThat(arg).isNotNull(),
+                                          arg -> assertThat(arg).isEqualTo("SQL")
+                                  );
+                      }
+                  }
+                  """
+              )
+            );
+        }
+
         @Issue("https://github.com/openrewrite/rewrite-testing-frameworks/issues/526")
         @ParameterizedTest
         @ValueSource(