Fix sonar

Author: ComputerDaddyGuy

src/main/java/io/github/computerdaddyguy/jfiletreeprettyprinter/renderer/RenderingOptions.java

@@ -4,6 +4,7 @@
 import java.nio.file.Path;
 import java.util.function.Function;
 import org.jspecify.annotations.NullMarked;
+import org.jspecify.annotations.Nullable;
 
 @NullMarked
 public interface RenderingOptions {
@@ -30,6 +31,7 @@ public interface RenderingOptions {
 	 * The line extension function.
 	 * @return
 	 */
+	@Nullable
 	Function<Path, String> getLineExtension();
 
 }

src/test/java/io/github/computerdaddyguy/jfiletreeprettyprinter/LineExtensionTest.java

@@ -69,33 +69,13 @@ void example_dir_match() {
 	@Test
 	void compact_dir_first_dir() {
 
-		// @formatter:off
-		var path = FileStructureCreator
-			.forTargetPath(root)
-			.createAndEnterDirectory("dirA")
-				.createAndEnterDirectory("dirB")
-					.createAndEnterDirectory("dirC")
-						.createFiles("file", 3)
-					.up()
-				.up()
-			.up()
-			.createDirectory("dirX")
-			.getPath();
-		// @formatter:on
-
 		Function<Path, String> lineExtension = p -> {
 			if (PathUtils.hasName(p, "dirA")) {
 				return " // 1";
 			}
 			return null;
 		};
 
-		var printer = FileTreePrettyPrinter.builder()
-			.customizeOptions(options -> options.withLineExtension(lineExtension))
-			.customizeOptions(options -> options.withCompactDirectories(true))
-			.build();
-
-		var result = printer.prettyPrint(path);
 		var expected = """
 			targetPath/
 			├─ dirA/ // 1
@@ -104,39 +84,20 @@ void compact_dir_first_dir() {
 			│     ├─ file2
 			│     └─ file3
 			└─ dirX/""";
-		assertThat(result).isEqualTo(expected);
+
+		compact_dir(lineExtension, expected);
 	}
 
 	@Test
 	void compact_dir_middle_dir() {
 
-		// @formatter:off
-		var path = FileStructureCreator
-			.forTargetPath(root)
-			.createAndEnterDirectory("dirA")
-			.createAndEnterDirectory("dirB")
-			.createAndEnterDirectory("dirC")
-			.createFiles("file", 3)
-			.up()
-			.up()
-			.up()
-			.createDirectory("dirX")
-			.getPath();
-		// @formatter:on
-
 		Function<Path, String> lineExtension = p -> {
 			if (PathUtils.hasName(p, "dirB")) {
 				return " // 2";
 			}
 			return null;
 		};
 
-		var printer = FileTreePrettyPrinter.builder()
-			.customizeOptions(options -> options.withLineExtension(lineExtension))
-			.customizeOptions(options -> options.withCompactDirectories(true))
-			.build();
-
-		var result = printer.prettyPrint(path);
 		var expected = """
 			targetPath/
 			├─ dirA/dirB/ // 2
@@ -145,46 +106,53 @@ void compact_dir_middle_dir() {
 			│     ├─ file2
 			│     └─ file3
 			└─ dirX/""";
-		assertThat(result).isEqualTo(expected);
+
+		compact_dir(lineExtension, expected);
 	}
 
 	@Test
 	void compact_dir_last_dir() {
 
+		Function<Path, String> lineExtension = p -> {
+			if (PathUtils.hasName(p, "dirC")) {
+				return " // 3";
+			}
+			return null;
+		};
+
+		var expected = """
+			targetPath/
+			├─ dirA/dirB/dirC/ // 3
+			│  ├─ file1
+			│  ├─ file2
+			│  └─ file3
+			└─ dirX/""";
+
+		compact_dir(lineExtension, expected);
+	}
+
+	private void compact_dir(Function<Path, String> lineExtension, String expected) {
+
 		// @formatter:off
 		var path = FileStructureCreator
 			.forTargetPath(root)
 			.createAndEnterDirectory("dirA")
-			.createAndEnterDirectory("dirB")
-			.createAndEnterDirectory("dirC")
-			.createFiles("file", 3)
-			.up()
-			.up()
+				.createAndEnterDirectory("dirB")
+					.createAndEnterDirectory("dirC")
+						.createFiles("file", 3)
+					.up()
+				.up()
 			.up()
 			.createDirectory("dirX")
 			.getPath();
 		// @formatter:on
 
-		Function<Path, String> lineExtension = p -> {
-			if (PathUtils.hasName(p, "dirC")) {
-				return " // 3";
-			}
-			return null;
-		};
-
 		var printer = FileTreePrettyPrinter.builder()
 			.customizeOptions(options -> options.withLineExtension(lineExtension))
 			.customizeOptions(options -> options.withCompactDirectories(true))
 			.build();
 
 		var result = printer.prettyPrint(path);
-		var expected = """
-			targetPath/
-			├─ dirA/dirB/dirC/ // 3
-			│  ├─ file1
-			│  ├─ file2
-			│  └─ file3
-			└─ dirX/""";
 		assertThat(result).isEqualTo(expected);
 	}