Fix TemporaryFolderToTempDir for chained newFolder() calls

The recipe was not properly traversing the containment hierarchy formed by chained method invocations and thus didn't transform `newFolder()` invocations when they were chained.

Fixes: #311

Author: knutwannheden

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

@@ -238,10 +238,10 @@ public TranslateNewFolderMethodInvocation(J.MethodInvocation method, JavaType.Me
 
             @Override
             public J.MethodInvocation visitMethodInvocation(J.MethodInvocation method, ExecutionContext ctx) {
-                if (!method.isScope(methodScope)) {
-                    return method;
-                }
                 J.MethodInvocation mi = (J.MethodInvocation) super.visitMethodInvocation(method, ctx);
+                if (!mi.isScope(methodScope)) {
+                    return mi;
+                }
                 if (mi.getSelect() != null) {
                     J tempDir = mi.getSelect().withType(JavaType.ShallowClass.build("java.io.File"));
                     List<Expression> args = mi.getArguments().stream().filter(arg -> !(arg instanceof J.Empty)).collect(Collectors.toList());

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

@@ -613,4 +613,60 @@ void doSomething(TemporaryFolder tempFolder) {
           )
         );
     }
+
+    @Test
+    @Issue("https://github.com/openrewrite/rewrite-testing-frameworks/issues/311")
+    void newFolderChainedCall() {
+        //language=java
+        rewriteRun(
+          java(
+            """
+              import java.io.File;
+              import java.io.IOException;
+              import java.nio.file.Path;
+              import org.junit.Rule;
+              import org.junit.Test;
+              import org.junit.rules.TemporaryFolder;
+
+              public class TempDirTest
+              {
+                  @Rule
+                  public TemporaryFolder folder = new TemporaryFolder();
+              
+                  @Test
+                  public void testPath() throws IOException {
+                      Path newFolder = folder.newFolder().toPath();
+                  }
+              }
+              """,
+            """
+              import java.io.File;
+              import java.io.IOException;
+              import java.nio.file.Path;
+              import org.junit.Test;
+              import org.junit.jupiter.api.io.TempDir;
+
+              public class TempDirTest
+              {
+                  @TempDir
+                  public File folder;
+              
+                  @Test
+                  public void testPath() throws IOException {
+                      Path newFolder = newFolder(folder, "junit").toPath();
+                  }
+
+                  private static File newFolder(File root, String... subDirs) throws IOException {
+                      String subFolder = String.join("/", subDirs);
+                      File result = new File(root, subFolder);
+                      if (!result.mkdirs()) {
+                          throw new IOException("Couldn't create folders " + root);
+                      }
+                      return result;
+                  }
+              }
+              """
+          )
+        );
+    }
 }