Add error checks

Signed-off-by: Scott Marlow <[email protected]>
scottmarlow · Feb 11, 2025 · fb2a854 · fb2a854
1 parent 5df0184
commit fb2a854
Showing 1 changed file with 10 additions and 0 deletions.
diff --git a/...revalidation/src/main/java/com/sun/ts/tests/signaturetest/ClientSignatureServletTest.java b/...revalidation/src/main/java/com/sun/ts/tests/signaturetest/ClientSignatureServletTest.java
@@ -111,6 +111,8 @@ public static WebArchive createDeploymentVehicle(@ArquillianResource TestArchive
         URL warResURL = ClientSignatureServletTest.class.getResource("/com/sun/ts/tests/common/vehicle/servlet/servlet_vehicle_web.xml");
         if (warResURL != null) {
             signaturetest_servlet_vehicle_web.addAsWebInfResource(warResURL, "web.xml");
+        } else {
+            throw new IllegalStateException("missing web.xml");
         }
 
         // add jakarta.tck:sigtest-maven-plugin jar to the war
@@ -122,10 +124,18 @@ public static WebArchive createDeploymentVehicle(@ArquillianResource TestArchive
                 .withTransitivity()
                 .asFile();
         // add signature test artifact
+        final int[] addedSigtestCount = {0};
         Arrays.stream(files).filter(file -> file.getName().contains("sigtest-maven-plugin"))
                 .forEach(file -> {
                     signaturetest_servlet_vehicle_web.addAsLibrary(file);
+                    if (addedSigtestCount[0] > 0) {
+                        throw new IllegalStateException("multiple sigtest-maven-plugin libraries found: " + file.getName());
+                    }
+                    addedSigtestCount[0]++;
                 });
+        if (addedSigtestCount[0] == 0) {
+            throw new IllegalStateException("Missing sigtest-maven-plugin library");
+        }
         // Call the archive processor
         archiveProcessor.processWebArchive(signaturetest_servlet_vehicle_web, ClientSignatureServletTest.class, warResURL);
         return signaturetest_servlet_vehicle_web;