Revert "refactor: Replace Stream.collect(Collectors.toUnmodifiableList()) with Stream.toList()"

This reverts commit 2fe03e5.

Author: timtebeek

rewrite-core/src/test/java/org/openrewrite/RecipeEstimatedEffortTest.java

@@ -37,6 +37,7 @@
 
 import static java.util.Collections.emptyList;
 import static java.util.Objects.requireNonNull;
+import static java.util.stream.Collectors.toList;
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.openrewrite.Tree.randomId;
 import static org.openrewrite.marker.GitProvenance.AutoCRLF.False;
@@ -231,7 +232,7 @@ public Collection<SourceFile> generate(AtomicBoolean shouldCreate, ExecutionCont
             if (shouldCreate.get()) {
                 return PlainTextParser.builder().build().parse(fileContents)
                   .map(brandNewFile -> (SourceFile) brandNewFile.withSourcePath(Paths.get(relativeFileName)))
-                  .toList();
+                  .collect(toList());
             }
             return emptyList();
         }

rewrite-java-test/src/test/java/org/openrewrite/java/RenameVariableTest.java

@@ -27,6 +27,7 @@
 import java.util.ArrayList;
 import java.util.List;
 
+import static java.util.stream.Collectors.toList;
 import static org.openrewrite.java.Assertions.java;
 import static org.openrewrite.test.RewriteTest.toRecipe;
 
@@ -82,17 +83,17 @@ public J visitClassDeclaration(J.ClassDeclaration classDecl, ExecutionContext ct
                     List<J.VariableDeclarations> variableDecls = classDecl.getBody().getStatements().stream()
                       .filter(J.VariableDeclarations.class::isInstance)
                       .map(J.VariableDeclarations.class::cast)
-                      .toList();
+                      .collect(toList());
 
                     if (includeMethodParameters) {
                         variableDecls.addAll(
-                                classDecl.getBody().getStatements().stream()
+                          classDecl.getBody().getStatements().stream()
                             .filter(J.MethodDeclaration.class::isInstance)
                             .map(J.MethodDeclaration.class::cast)
                             .flatMap(it -> it.getParameters().stream())
                             .filter(J.VariableDeclarations.class::isInstance)
                             .map(J.VariableDeclarations.class::cast)
-                            .toList()
+                            .collect(toList())
                         );
                     }
 

rewrite-java-test/src/test/java/org/openrewrite/java/UpdateSourcePositionsTest.java

@@ -25,6 +25,7 @@
 
 import java.util.List;
 import java.util.function.UnaryOperator;
+import java.util.stream.Collectors;
 
 import static org.assertj.core.api.Assertions.assertThat;
 
@@ -66,7 +67,7 @@ public Consumer<String> demo() {
                 }
             }
             """
-        ).toList();
+        ).collect(Collectors.toList());
         Result result = new UpdateSourcePositions().run(new InMemoryLargeSourceSet(cus), new InMemoryExecutionContext(), 1).getChangeset().getAllResults().getFirst();
         assertThat(printWithLines(result.getAfter())).isEqualTo(
           """
@@ -97,7 +98,7 @@ void test() {
                 }
             }
             """
-        ).toList();
+        ).collect(Collectors.toList());
         Result result = new UpdateSourcePositions().run(new InMemoryLargeSourceSet(cus), new InMemoryExecutionContext(), 1).getChangeset().getAllResults().getFirst();
         assertThat(printWithLines(result.getAfter())).isEqualTo(
           """

rewrite-java-test/src/test/java/org/openrewrite/java/internal/TypesInUseTest.java

@@ -20,6 +20,8 @@
 import org.openrewrite.java.tree.TypeUtils;
 import org.openrewrite.test.RewriteTest;
 
+import java.util.stream.Collectors;
+
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.openrewrite.java.Assertions.java;
 
@@ -54,7 +56,7 @@ public class Foo {}
             spec -> spec.afterRecipe(cu -> {
                 var foundTypes = cu.getTypesInUse().getVariables().stream()
                   .map(v -> TypeUtils.asFullyQualified(v.getType()).getFullyQualifiedName())
-                  .toList();
+                  .collect(Collectors.toList());
                 assertThat(foundTypes).containsExactlyInAnyOrder("org.openrewrite.test.YesOrNo$Status");
             })
           )