AssertToAssertions should not swap arguments for fail methods

Fixes #804

Author: timtebeek

src/main/java/org/openrewrite/java/testing/junit5/AssertToAssertions.java

@@ -96,6 +96,9 @@ public J.MethodInvocation visitMethodInvocation(J.MethodInvocation method, Execu
                     return m;
                 }
             }
+            if ("fail".equals(m.getSimpleName())) {
+                return m;
+            }
 
             if (TypeUtils.isString(firstArg.getType())) {
                 // Move the first arg to be the last argument

src/test/java/org/openrewrite/java/testing/junit5/JUnit5MigrationTest.java

@@ -26,6 +26,7 @@
 import org.openrewrite.marker.BuildTool;
 import org.openrewrite.test.RecipeSpec;
 import org.openrewrite.test.RewriteTest;
+import org.openrewrite.test.TypeValidation;
 
 import java.util.regex.Pattern;
 
@@ -755,4 +756,54 @@ public void testMethod() {}
             );
         }
     }
+
+    @Issue("https://github.com/openrewrite/rewrite-testing-frameworks/issues/804")
+    @Test
+    void dontSwapArgumentsForFailWithMissingTypeInformationInJavadoc() {
+        rewriteRun(
+          spec -> spec.typeValidationOptions(TypeValidation.none()),
+          java(
+            """
+              import org.junit.Assert;
+              import org.junit.Test;
+
+              public class ReproductionTest {
+
+                  public static void fail(String param1, String param2, String param3) {
+                      // No-op
+                  }
+
+                  /**
+                   * Method documentation with a broken link to {@link #fail(String, String,
+                   * Integer)} (the types in the parameter list don't match).
+                   */
+                  @Test
+                  public void test() {
+                      Assert.assertEquals(42, Integer.parseInt("2a", 16));
+                  }
+              }
+              """,
+            """
+              import org.junit.jupiter.api.Assertions;
+              import org.junit.jupiter.api.Test;
+
+              public class ReproductionTest {
+
+                  public static void fail(String param1, String param2, String param3) {
+                      // No-op
+                  }
+
+                  /**
+                   * Method documentation with a broken link to {@link #fail(String, String,
+                   * Integer)} (the types in the parameter list don't match).
+                   */
+                  @Test
+                  public void test() {
+                      Assertions.assertEquals(42, Integer.parseInt("2a", 16));
+                  }
+              }
+              """
+          )
+        );
+    }
 }