refactor: Add a null method argument ChangeType constructor

Use this link to re-run the recipe: https://app.moderne.io/recipes/org.openrewrite.java.AddNullMethodArgument?organizationId=T3BlblJld3JpdGU%3D#defaults%3DW3sibmFtZSI6Im1ldGhvZFBhdHRlcm4iLCJ2YWx1ZSI6Im9yZy5vcGVucmV3cml0ZS5qYXZhLkNoYW5nZVR5cGUgPGNvbnN0cnVjdG9yPiguLikifSx7Im5hbWUiOiJhcmd1bWVudEluZGV4IiwidmFsdWUiOiIzIn0seyJuYW1lIjoicGFyYW1ldGVyVHlwZSIsInZhbHVlIjoiamF2YS5sYW5nLkJvb2xlYW4ifSx7Im5hbWUiOiJwYXJhbWV0ZXJOYW1lIiwidmFsdWUiOiJ2aXNpdFN0cmluZ0xpdGVyYWxzIn0seyJuYW1lIjoiZXhwbGljaXRDYXN0IiwidmFsdWUiOiJGYWxzZSJ9XQ%3D%3D

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

Author: Laurens-W

src/main/java/org/openrewrite/java/testing/arquillian/ReplaceArquillianInSequenceAnnotation.java

@@ -51,7 +51,7 @@ public TreeVisitor<?, ExecutionContext> getVisitor() {
 
                     @Override
                     public J.ClassDeclaration visitClassDeclaration(J.ClassDeclaration classDecl, ExecutionContext ctx) {
-                        doAfterVisit(new ChangeType(IN_SEQUENCE, "org.junit.jupiter.api.Order", true).getVisitor());
+                        doAfterVisit(new ChangeType(IN_SEQUENCE, "org.junit.jupiter.api.Order", true, null).getVisitor());
                         return super.visitClassDeclaration(classDecl, ctx);
                     }
 

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

@@ -73,8 +73,8 @@ public J.CompilationUnit visitCompilationUnit(J.CompilationUnit cu, ExecutionCon
                         J.CompilationUnit c = super.visitCompilationUnit(cu, ctx);
                         doAfterVisit(new TestCaseVisitor());
                         // ChangeType for org.junit.Assert method invocations because TestCase extends org.junit.Assert
-                        doAfterVisit(new ChangeType("junit.framework.TestCase", "org.junit.Assert", true).getVisitor());
-                        doAfterVisit(new ChangeType("junit.framework.Assert", "org.junit.Assert", true).getVisitor());
+                        doAfterVisit(new ChangeType("junit.framework.TestCase", "org.junit.Assert", true, null).getVisitor());
+                        doAfterVisit(new ChangeType("junit.framework.Assert", "org.junit.Assert", true, null).getVisitor());
                         doAfterVisit(new AssertToAssertions.AssertToAssertionsVisitor());
                         doAfterVisit(new UseStaticImport("org.junit.jupiter.api.Assertions assert*(..)").getVisitor());
                         doAfterVisit(new UseStaticImport("org.junit.jupiter.api.Assertions fail*(..)").getVisitor());

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

@@ -66,7 +66,7 @@ public J visitCompilationUnit(J.CompilationUnit cu, ExecutionContext ctx) {
                 J.CompilationUnit c = (J.CompilationUnit) super.visitCompilationUnit(cu, ctx);
                 if (c != cu) {
                     c = (J.CompilationUnit) new ChangeType(
-                            "org.junit.rules.TemporaryFolder", "java.io.File", true).getVisitor()
+                            "org.junit.rules.TemporaryFolder", "java.io.File", true, null).getVisitor()
                             .visit(c, ctx);
                     maybeAddImport("java.io.File");
                     maybeAddImport("org.junit.jupiter.api.io.TempDir");

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

@@ -80,8 +80,8 @@ public J visitMethodInvocation(J.MethodInvocation method, ExecutionContext ctx)
                     return mi;
                 }
             });
-            doAfterVisit(new ChangeType("org.junit.rules.TestName", "java.lang.String", true).getVisitor());
-            doAfterVisit(new ChangeType("org.junit.Before", "org.junit.jupiter.api.BeforeEach", true).getVisitor());
+            doAfterVisit(new ChangeType("org.junit.rules.TestName", "java.lang.String", true, null).getVisitor());
+            doAfterVisit(new ChangeType("org.junit.Before", "org.junit.jupiter.api.BeforeEach", true, null).getVisitor());
             return compilationUnit;
         }
 

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

@@ -51,10 +51,10 @@ public static class UpdateBeforeAfterAnnotationsVisitor extends JavaIsoVisitor<E
         @Override
         public J preVisit(J tree, ExecutionContext ctx) {
             stopAfterPreVisit();
-            doAfterVisit(new ChangeType("org.junit.Before", "org.junit.jupiter.api.BeforeEach", true).getVisitor());
-            doAfterVisit(new ChangeType("org.junit.After", "org.junit.jupiter.api.AfterEach", true).getVisitor());
-            doAfterVisit(new ChangeType("org.junit.BeforeClass", "org.junit.jupiter.api.BeforeAll", true).getVisitor());
-            doAfterVisit(new ChangeType("org.junit.AfterClass", "org.junit.jupiter.api.AfterAll", true).getVisitor());
+            doAfterVisit(new ChangeType("org.junit.Before", "org.junit.jupiter.api.BeforeEach", true, null).getVisitor());
+            doAfterVisit(new ChangeType("org.junit.After", "org.junit.jupiter.api.AfterEach", true, null).getVisitor());
+            doAfterVisit(new ChangeType("org.junit.BeforeClass", "org.junit.jupiter.api.BeforeAll", true, null).getVisitor());
+            doAfterVisit(new ChangeType("org.junit.AfterClass", "org.junit.jupiter.api.AfterAll", true, null).getVisitor());
             return tree;
         }
     }

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

@@ -71,7 +71,7 @@ public J.CompilationUnit visitCompilationUnit(J.CompilationUnit cu, ExecutionCon
             Set<NameTree> nameTreeSet = c.findType("org.junit.Test");
             if (!nameTreeSet.isEmpty()) {
                 // Update other references like `Test.class`.
-                c = (J.CompilationUnit) new ChangeType("org.junit.Test", "org.junit.jupiter.api.Test", true)
+                c = (J.CompilationUnit) new ChangeType("org.junit.Test", "org.junit.jupiter.api.Test", true, null)
                         .getVisitor().visitNonNull(c, ctx);
             }
 

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

@@ -51,7 +51,7 @@ public TreeVisitor<?, ExecutionContext> getVisitor() {
             public J preVisit(J tree, ExecutionContext ctx) {
                 if (tree instanceof JavaSourceFile) {
                     doAfterVisit(new ChangeType("com.github.tomakehurst.wiremock.junit.WireMockRule",
-                            "com.github.tomakehurst.wiremock.junit5.WireMockExtension", true).getVisitor());
+                            "com.github.tomakehurst.wiremock.junit5.WireMockExtension", true, null).getVisitor());
                 }
                 return tree;
             }
@@ -62,7 +62,7 @@ public J visitNewClass(J.NewClass newClass, ExecutionContext ctx) {
                 J.NewClass n = (J.NewClass) super.visitNewClass(newClass, ctx);
                 if (newWiremockRule.matches(n)) {
                     maybeAddImport("com.github.tomakehurst.wiremock.junit5.WireMockExtension");
-                    doAfterVisit(new ChangeType("org.junit.Rule", "org.junit.jupiter.api.extension.RegisterExtension", true)
+                    doAfterVisit(new ChangeType("org.junit.Rule", "org.junit.jupiter.api.extension.RegisterExtension", true, null)
                             .getVisitor());
 
                     Expression arg = n.getArguments().get(0);