Polish

Author: snicoll

spring-core-test/src/main/java/org/springframework/aot/test/generate/compile/CompileWithTargetClassAccessClassLoader.java

@@ -21,6 +21,8 @@
 import java.net.URL;
 import java.util.Enumeration;
 
+import org.springframework.lang.Nullable;
+
 /**
  * {@link ClassLoader} implementation to support
  * {@link CompileWithTargetClassAccess @CompileWithTargetClassAccess}.
@@ -74,6 +76,7 @@ protected Enumeration<URL> findResources(String name) throws IOException {
 	}
 
 	@Override
+	@Nullable
 	protected URL findResource(String name) {
 		return this.testClassLoader.getResource(name);
 	}

spring-core-test/src/main/java/org/springframework/aot/test/generate/compile/DynamicClassLoader.java

@@ -158,7 +158,7 @@ private static class ResourceFileHandler extends URLStreamHandler {
 
 
 		@Override
-		protected URLConnection openConnection(URL url) throws IOException {
+		protected URLConnection openConnection(URL url) {
 			return new ResourceFileConnection(url, this.file);
 		}
 
@@ -177,11 +177,11 @@ protected ResourceFileConnection(URL url, ResourceFile file) {
 
 
 		@Override
-		public void connect() throws IOException {
+		public void connect() {
 		}
 
 		@Override
-		public InputStream getInputStream() throws IOException {
+		public InputStream getInputStream() {
 			return new ByteArrayInputStream(
 					this.file.getContent().getBytes(StandardCharsets.UTF_8));
 		}

spring-core-test/src/main/java/org/springframework/aot/test/generate/file/DynamicFile.java

@@ -38,8 +38,8 @@ public abstract sealed class DynamicFile permits SourceFile, ResourceFile {
 
 
 	protected DynamicFile(String path, String content) {
-		Assert.hasText(path, "Path must not be empty");
-		Assert.hasText(content, "Content must not be empty");
+		Assert.hasText(path, "'path' must not be empty");
+		Assert.hasText(content, "'content' must not be empty");
 		this.path = path;
 		this.content = content;
 	}

spring-core-test/src/main/java/org/springframework/aot/test/generate/file/DynamicFiles.java

@@ -38,8 +38,7 @@
 final class DynamicFiles<F extends DynamicFile> implements Iterable<F> {
 
 
-	private static final DynamicFiles<?> NONE = new DynamicFiles<>(
-			Collections.emptyMap());
+	private static final DynamicFiles<?> NONE = new DynamicFiles<>(Collections.emptyMap());
 
 
 	private final Map<String, F> files;

spring-core-test/src/main/java/org/springframework/aot/test/generate/file/ResourceFile.java

@@ -22,7 +22,6 @@
 import org.assertj.core.api.AssertProvider;
 
 import org.springframework.core.io.InputStreamSource;
-import org.springframework.lang.Nullable;
 import org.springframework.util.FileCopyUtils;
 
 /**
@@ -32,8 +31,7 @@
  * @author Phillip Webb
  * @since 6.0
  */
-public final class ResourceFile extends DynamicFile
-		implements AssertProvider<ResourceFileAssert> {
+public final class ResourceFile extends DynamicFile implements AssertProvider<ResourceFileAssert> {
 
 
 	private ResourceFile(String path, String content) {
@@ -46,7 +44,7 @@ private ResourceFile(String path, String content) {
 	 * {@link CharSequence}.
 	 * @param path the relative path of the file or {@code null} to have the
 	 * path deduced
-	 * @param charSequence a file containing the source contents
+	 * @param charSequence a file containing the file contents
 	 * @return a {@link ResourceFile} instance
 	 */
 	public static ResourceFile of(String path, CharSequence charSequence) {
@@ -56,24 +54,21 @@ public static ResourceFile of(String path, CharSequence charSequence) {
 	/**
 	 * Factory method to create a new {@link ResourceFile} from the given
 	 * {@link InputStreamSource}.
-	 * @param path the relative path of the file or {@code null} to have the
-	 * path deduced
+	 * @param path the relative path of the file
 	 * @param inputStreamSource the source for the file
-	 * @return a {@link SourceFile} instance
+	 * @return a {@link ResourceFile} instance
 	 */
-	public static ResourceFile of(@Nullable String path,
-			InputStreamSource inputStreamSource) {
+	public static ResourceFile of(String path, InputStreamSource inputStreamSource) {
 		return of(path, appendable -> appendable.append(FileCopyUtils.copyToString(
 				new InputStreamReader(inputStreamSource.getInputStream(), StandardCharsets.UTF_8))));
 	}
 
 	/**
 	 * Factory method to create a new {@link SourceFile} from the given
 	 * {@link WritableContent}.
-	 * @param path the relative path of the file or {@code null} to have the
-	 * path deduced
+	 * @param path the relative path of the file
 	 * @param writableContent the content to write to the file
-	 * @return a {@link SourceFile} instance
+	 * @return a {@link ResourceFile} instance
 	 */
 	public static ResourceFile of(String path, WritableContent writableContent) {
 		return new ResourceFile(path, toString(writableContent));

spring-core-test/src/main/java/org/springframework/aot/test/generate/file/ResourceFiles.java

@@ -111,7 +111,7 @@ public boolean isEmpty() {
 
 	/**
 	 * Get the {@link ResourceFile} with the given
-	 * {@code DynamicFile#getPath() path}.
+	 * {@linkplain  DynamicFile#getPath() path}.
 	 * @param path the path to find
 	 * @return a {@link ResourceFile} instance or {@code null}
 	 */
@@ -121,7 +121,7 @@ public ResourceFile get(String path) {
 	}
 
 	/**
-	 * Return the single source file contained in the collection.
+	 * Return the single {@link ResourceFile} contained in the collection.
 	 * @return the single file
 	 * @throws IllegalStateException if the collection doesn't contain exactly
 	 * one file

spring-core-test/src/main/java/org/springframework/aot/test/generate/file/SourceFile.java

@@ -35,18 +35,16 @@
 
 /**
  * {@link DynamicFile} that holds Java source code and provides
- * {@link SourceFileAssert} support. Usually created from an AOT generated type,
- * for example:
- *
+ * {@link SourceFileAssert} support. Usually created from an AOT generated
+ * type, for example:
  * <pre class="code">
  * SourceFile.of(generatedFile::writeTo)
  * </pre>
  *
  * @author Phillip Webb
  * @since 6.0
  */
-public final class SourceFile extends DynamicFile
-		implements AssertProvider<SourceFileAssert> {
+public final class SourceFile extends DynamicFile implements AssertProvider<SourceFileAssert> {
 
 
 	private final String className;

spring-core-test/src/main/java/org/springframework/aot/test/generate/file/SourceFiles.java

@@ -114,7 +114,7 @@ public boolean isEmpty() {
 
 	/**
 	 * Get the {@link SourceFile} with the given
-	 * {@code DynamicFile#getPath() path}.
+	 * {@linkplain  DynamicFile#getPath() path}.
 	 * @param path the path to find
 	 * @return a {@link SourceFile} instance or {@code null}
 	 */

spring-core-test/src/main/java/org/springframework/aot/test/generate/file/package-info.java

@@ -1,7 +1,5 @@
 /**
  * Support classes for running assertions on generated files.
- *
- * @author Stephane Nicoll
  */
 @NonNullApi
 @NonNullFields

spring-core-test/src/test/java/org/springframework/aot/test/generate/compile/CompiledTests.java

@@ -107,15 +107,13 @@ void getResourceFileWhenMultipleThrowsException() {
 				ResourceFile.of("META-INF/myfile1", "test1"),
 				ResourceFile.of("META-INF/myfile2", "test2"));
 		TestCompiler.forSystem().withResources(resourceFiles).compile(
-				compiled -> assertThatIllegalStateException().isThrownBy(
-						() -> compiled.getResourceFile()));
+				compiled -> assertThatIllegalStateException().isThrownBy(compiled::getResourceFile));
 	}
 
 	@Test
 	void getResourceFileWhenNoneThrowsException() {
 		TestCompiler.forSystem().compile(
-				compiled -> assertThatIllegalStateException().isThrownBy(
-						() -> compiled.getResourceFile()));
+				compiled -> assertThatIllegalStateException().isThrownBy(compiled::getResourceFile));
 	}
 
 	@Test