Fix Mockito1to3Migration not adding runner after removing PowerMockRunner (#942)

ReplacePowerMockito was ordered after AddMockitoExtensionIfAnnotationsUsed,
so the latter would see @RunWith(PowerMockRunner.class) and skip the class.
Then ReplacePowerMockito would remove it, leaving @mock fields uninitialized.

Move ReplacePowerMockito before AddMockitoExtensionIfAnnotationsUsed so the
runner gap is detected and filled with MockitoJUnitRunner or MockitoExtension.

Author: bmuschko

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

@@ -179,14 +179,14 @@ recipeList:
       newFullyQualifiedTypeName: org.mockito.junit.MockitoJUnitRunner
   - org.openrewrite.java.testing.mockito.CleanupMockitoImports
   - org.openrewrite.java.testing.mockito.MockUtilsToStatic
+  - org.openrewrite.java.testing.mockito.ReplacePowerMockito
   - org.openrewrite.java.testing.junit5.MockitoJUnitToMockitoExtension
   - org.openrewrite.java.testing.mockito.AddMockitoExtensionIfAnnotationsUsed
   - org.openrewrite.java.testing.mockito.AddMockitoJupiterDependency
   - org.openrewrite.java.testing.mockito.ReplaceMockitoTestExecutionListenerForJupiter
   - org.openrewrite.java.testing.mockito.ReplaceMockitoTestExecutionListenerForJUnit4
   - org.openrewrite.java.testing.mockito.ReplaceMockitoTestExecutionListenerForTestNG
   - org.openrewrite.java.testing.mockito.RemoveInitMocksIfRunnersSpecified
-  - org.openrewrite.java.testing.mockito.ReplacePowerMockito
   - org.openrewrite.java.dependencies.ChangeDependency:
       oldGroupId: org.mockito
       oldArtifactId: mockito-all

src/test/java/org/openrewrite/java/testing/mockito/Mockito1to3MigrationTest.java

@@ -406,6 +406,56 @@ void someTest() {
         );
     }
 
+    @Test
+    void powerMockRunnerReplacedWithMockitoJUnitRunnerWhenMockAnnotationsPresent() {
+        rewriteRun(
+          spec -> spec.parser(JavaParser.fromJavaVersion()
+            .logCompilationWarningsAndErrors(true)
+            .classpathFromResources(new InMemoryExecutionContext(),
+              "mockito-core-3.12",
+              "junit-4",
+              "powermock-core-1",
+              "powermock-api-mockito-1",
+              "powermock-api-support-1",
+              "powermock-module-junit4")),
+          //language=java
+          java(
+            """
+              import org.junit.Test;
+              import org.junit.runner.RunWith;
+              import org.mockito.Mock;
+              import org.powermock.modules.junit4.PowerMockRunner;
+
+              @RunWith(PowerMockRunner.class)
+              public class MyTest {
+                  @Mock
+                  Object myMock;
+
+                  @Test
+                  public void someTest() {
+                  }
+              }
+              """,
+            """
+              import org.junit.Test;
+              import org.junit.runner.RunWith;
+              import org.mockito.Mock;
+              import org.mockito.junit.MockitoJUnitRunner;
+
+              @RunWith(MockitoJUnitRunner.class)
+              public class MyTest {
+                  @Mock
+                  Object myMock;
+
+                  @Test
+                  public void someTest() {
+                  }
+              }
+              """
+          )
+        );
+    }
+
     @Test
     void handlesAnyObjectFromMockitoWildCardImport() {
         rewriteRun(