add test assertion to resolve @SuppressWarnings("checkstyle:UnusedLocalVariable")

Author: Vincent Potucek

impl/maven-core/src/test/java/org/apache/maven/artifact/handler/ArtifactHandlerTest.java

@@ -20,7 +20,6 @@
 
 import javax.inject.Inject;
 
-import java.io.File;
 import java.nio.file.Files;
 import java.util.List;
 
@@ -29,39 +28,54 @@
 import org.codehaus.plexus.testing.PlexusTest;
 import org.junit.jupiter.api.Test;
 
+import static org.assertj.core.api.Assertions.assertThat;
 import static org.codehaus.plexus.testing.PlexusExtension.getTestFile;
 import static org.junit.jupiter.api.Assertions.assertEquals;
 
 @PlexusTest
 class ArtifactHandlerTest {
+    private static final String[] EXPECTED_COLUMN_HEADERS = new String[] {
+        "", "type", "classifier", "extension", "packaging", "language", "added to classpath", "includesDependencies"
+    };
+    private static final List<String> VALID_PACKAGING_TYPES = List.of(
+            "pom",
+            "jar",
+            "test-jar",
+            "maven-plugin",
+            "ejb",
+            "war",
+            "ear",
+            "rar",
+            "java-source",
+            "javadoc",
+            "aar",
+            "apk",
+            "bundle",
+            "eclipse-plugin",
+            "eclipse-test-plugin",
+            "hpi",
+            "jpi",
+            "kar",
+            "lpkg",
+            "maven-archetype",
+            "nar",
+            "par",
+            "sar",
+            "swc",
+            "swf",
+            "zip");
+
     @Inject
     PlexusContainer container;
 
     @Test
-    @SuppressWarnings("checkstyle:UnusedLocalVariable")
     void testAptConsistency() throws Exception {
-        File apt = getTestFile("src/site/apt/artifact-handlers.apt");
-
-        List<String> lines = Files.readAllLines(apt.toPath());
-
-        for (String line : lines) {
+        for (String line : Files.readAllLines(
+                getTestFile("src/site/apt/artifact-handlers.apt").toPath())) {
             if (line.startsWith("||")) {
-                String[] cols = line.split("\\|\\|");
-                String[] expected = new String[] {
-                    "",
-                    "type",
-                    "classifier",
-                    "extension",
-                    "packaging",
-                    "language",
-                    "added to classpath",
-                    "includesDependencies",
-                    ""
-                };
-
                 int i = 0;
-                for (String col : cols) {
-                    assertEquals(expected[i++], col.trim(), "Wrong column header");
+                for (String col : line.split("\\|\\|")) {
+                    assertEquals(EXPECTED_COLUMN_HEADERS[i++], col.trim(), "Wrong column header");
                 }
             } else if (line.startsWith("|")) {
                 String[] cols = line.split("\\|");
@@ -79,6 +93,8 @@ void testAptConsistency() throws Exception {
                 assertEquals(handler.getExtension(), extension, type + " extension");
                 // Packaging/Directory is Maven1 remnant!!!
                 // assertEquals(handler.getPackaging(), packaging, type + " packaging");
+                assertThat(handler.getPackaging()).isNotEmpty();
+                assertThat(VALID_PACKAGING_TYPES).contains(packaging);
                 assertEquals(handler.getClassifier(), classifier, type + " classifier");
                 assertEquals(handler.getLanguage(), language, type + " language");
                 assertEquals(
@@ -97,8 +113,7 @@ private String trimApt(String content, String type) {
     }
 
     private String trimApt(String content) {
-        content = content.replace('<', ' ').replace('>', ' ').trim();
-
-        return (content.length() == 0) ? null : content;
+        String value = content.replace('<', ' ').replace('>', ' ').trim();
+        return value.isEmpty() ? null : value;
     }
 }