Remove usages of Collectors.toList()

Author: ascopes

java-compiler-testing/src/main/java/io/github/ascopes/jct/compilers/impl/JctCompilationFactoryImpl.java

@@ -16,7 +16,6 @@
 package io.github.ascopes.jct.compilers.impl;
 
 import static java.util.Objects.requireNonNull;
-import static java.util.stream.Collectors.toList;
 
 import io.github.ascopes.jct.compilers.JctCompilation;
 import io.github.ascopes.jct.compilers.JctCompilationFactory;
@@ -139,7 +138,7 @@ public JctCompilation createCompilation(
           .arguments(flags)
           .compilationUnits(Set.copyOf(compilationUnits))
           .fileManager(fileManager)
-          .outputLines(writer.getContent().lines().collect(toList()))
+          .outputLines(writer.getContent().lines().toList())
           .diagnostics(diagnosticListener.getDiagnostics())
           .success(success)
           .failOnWarnings(compiler.isFailOnWarnings())

java-compiler-testing/src/main/java/io/github/ascopes/jct/ex/package-info.java

@@ -19,5 +19,3 @@
  */
 package io.github.ascopes.jct.ex;
 
-import org.apiguardian.api.API;
-import org.apiguardian.api.API.Status;

java-compiler-testing/src/main/java/io/github/ascopes/jct/filemanagers/config/package-info.java

@@ -19,5 +19,3 @@
  */
 package io.github.ascopes.jct.filemanagers.config;
 
-import org.apiguardian.api.API;
-import org.apiguardian.api.API.Status;

java-compiler-testing/src/main/java/io/github/ascopes/jct/junit/AbstractCompilersProvider.java

@@ -23,7 +23,6 @@
 import io.github.ascopes.jct.ex.JctIllegalInputException;
 import io.github.ascopes.jct.ex.JctJunitConfigurerException;
 import java.lang.reflect.Constructor;
-import java.lang.reflect.InaccessibleObjectException;
 import java.lang.reflect.InvocationTargetException;
 import java.util.stream.IntStream;
 import java.util.stream.Stream;

java-compiler-testing/src/main/java/io/github/ascopes/jct/utils/StringUtils.java

@@ -22,7 +22,6 @@
 import java.util.List;
 import java.util.Objects;
 import java.util.function.Function;
-import java.util.stream.Collectors;
 import me.xdrop.fuzzywuzzy.FuzzySearch;
 import me.xdrop.fuzzywuzzy.ToStringFunction;
 import me.xdrop.fuzzywuzzy.model.BoundExtractedResult;
@@ -282,7 +281,7 @@ public static <T> String resultNotFoundWithFuzzySuggestions(
         .map(BoundExtractedResult::getReferent)
         .map(userFriendlyStringifier)
         .sorted()
-        .collect(Collectors.toList());
+        .toList();
 
     if (!closestMatches.isEmpty()) {
       errorBuilder.append(" Maybe you meant:");

java-compiler-testing/src/main/java/io/github/ascopes/jct/utils/VisibleForTestingOnly.java

@@ -17,7 +17,6 @@
 
 import java.lang.annotation.Documented;
 import java.lang.annotation.ElementType;
-import java.lang.annotation.Inherited;
 import java.lang.annotation.Retention;
 import java.lang.annotation.RetentionPolicy;
 import java.lang.annotation.Target;

java-compiler-testing/src/test/java/io/github/ascopes/jct/compilers/impl/JavacJctFlagBuilderImplTest.java

@@ -25,7 +25,6 @@
 import io.github.ascopes.jct.compilers.DebuggingInfo;
 import io.github.ascopes.jct.fixtures.Fixtures;
 import java.util.List;
-import java.util.stream.Collectors;
 import java.util.stream.Stream;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.DisplayName;
@@ -467,7 +466,7 @@ void addsAnnotationProcessorOptions() {
       var options = Stream
           .generate(Fixtures::someText)
           .limit(5)
-          .collect(Collectors.toList());
+          .toList();
 
       // When
       flagBuilder.annotationProcessorOptions(options);
@@ -476,7 +475,7 @@ void addsAnnotationProcessorOptions() {
       assertThat(flagBuilder.build())
           .containsSequence(options.stream()
               .map("-A"::concat)
-              .collect(Collectors.toList()));
+              .toList());
     }
 
     @DisplayName(".annotationProcessorOptions(...) returns the flag builder")
@@ -486,7 +485,7 @@ void returnsFlagBuilder() {
       var options = Stream
           .generate(Fixtures::someText)
           .limit(5)
-          .collect(Collectors.toList());
+          .toList();
 
       // Then
       assertThat(flagBuilder.annotationProcessorOptions(options))
@@ -505,7 +504,7 @@ void addsCompilerOptions() {
       var options = Stream
           .generate(Fixtures::someText)
           .limit(5)
-          .collect(Collectors.toList());
+          .toList();
 
       // When
       flagBuilder.compilerOptions(options);
@@ -522,7 +521,7 @@ void returnsFlagBuilder() {
       var options = Stream
           .generate(Fixtures::someText)
           .limit(5)
-          .collect(Collectors.toList());
+          .toList();
 
       // Then
       assertThat(flagBuilder.compilerOptions(options))

java-compiler-testing/src/test/java/io/github/ascopes/jct/compilers/impl/JctCompilationFactoryImplTest.java

@@ -23,7 +23,6 @@
 import static io.github.ascopes.jct.fixtures.Fixtures.someText;
 import static io.github.ascopes.jct.fixtures.Fixtures.someTraceDiagnostic;
 import static io.github.ascopes.jct.utils.IterableUtils.flatten;
-import static java.util.stream.Collectors.toList;
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
 import static org.mockito.ArgumentMatchers.any;
@@ -388,7 +387,7 @@ void teeWriterLogsShouldBePlacedInTheCompilationResult() throws IOException {
 
       // Then
       assertThat(result.getOutputLines())
-          .containsExactlyElementsOf(lines.lines().collect(toList()));
+          .containsExactlyElementsOf(lines.lines().toList());
 
       // Ensure we flushed before we called toString, otherwise data might be missing.
       order.verify(teeWriter).flush();

java-compiler-testing/src/test/java/io/github/ascopes/jct/compilers/impl/JctCompilationImplTest.java

@@ -55,7 +55,7 @@ void getArgumentsReturnsExpectedValue(int argumentCount) {
     var arguments = Stream
         .generate(Fixtures::someFlag)
         .limit(argumentCount)
-        .collect(Collectors.toList());
+        .toList();
 
     var compilation = filledBuilder()
         .arguments(arguments)
@@ -102,7 +102,7 @@ void getOutputLinesReturnsExpectedValue(int lineCount) {
         .generate(UUID::randomUUID)
         .map(Objects::toString)
         .limit(lineCount)
-        .collect(Collectors.toList());
+        .toList();
 
     var compilation = filledBuilder()
         .outputLines(lines)
@@ -141,7 +141,7 @@ void getDiagnosticsReturnsExpectedValue(int diagnosticCount) {
     var diagnostics = Stream
         .generate(Fixtures::someTraceDiagnostic)
         .limit(diagnosticCount)
-        .collect(Collectors.toList());
+        .toList();
 
     var compilation = filledBuilder()
         .diagnostics(diagnostics)

java-compiler-testing/src/test/java/io/github/ascopes/jct/filemanagers/impl/JctFileManagerImplTest.java

@@ -135,7 +135,7 @@ void addPathsDelegatesMultipleCallsToTheRepository() {
     var pathRoots = Stream
         .generate(Fixtures::somePathRoot)
         .limit(10)
-        .collect(Collectors.toList());
+        .toList();
 
     // When
     fileManager.addPaths(location, pathRoots);