fix: remove @notempty on List (unsupported by HV) and correct test expectations #35203

Author: hassandotcms

dotCMS/src/main/java/com/dotcms/rest/api/v1/maintenance/DeleteContentletsForm.java

@@ -6,7 +6,6 @@
 import com.fasterxml.jackson.annotation.JsonProperty;
 import io.swagger.v3.oas.annotations.media.Schema;
 
-import javax.validation.constraints.NotEmpty;
 import java.util.List;
 
 /**
@@ -19,7 +18,6 @@
 public class DeleteContentletsForm extends Validated {
 
     @JsonProperty("identifiers")
-    @NotEmpty(message = "identifiers is required and must not be empty")
     @Schema(
             description = "List of contentlet identifiers. All language siblings of each "
                     + "identifier will be permanently destroyed.",

dotcms-integration/src/test/java/com/dotcms/rest/api/v1/maintenance/MaintenanceResourceIntegrationTest.java

@@ -329,13 +329,13 @@ public void test_deleteContentlets_nullForm_throwsBadRequest() {
         resource.deleteContentlets(request, mockResponse, null);
     }
 
-    @Test(expected = ValidationException.class)
-    public void test_deleteContentlets_emptyIdentifiers_throwsValidation() {
+    @Test(expected = BadRequestException.class)
+    public void test_deleteContentlets_emptyIdentifiers_throwsBadRequest() {
         new DeleteContentletsForm(Collections.emptyList());
     }
 
-    @Test(expected = ValidationException.class)
-    public void test_deleteContentlets_nullIdentifiers_throwsValidation() {
+    @Test(expected = BadRequestException.class)
+    public void test_deleteContentlets_nullIdentifiers_throwsBadRequest() {
         new DeleteContentletsForm(null);
     }