Replace NONE event with INDEXING/DEINDEXING

Refactor DumAemEvent enums to replace the previous NONE/UNPUBLISHING mapping with explicit INDEXING and DEINDEXING values (keeping UNPUBLISHING and PUBLISHING), and update all callers and defaults to use INDEXING. Adjusted constructors, session builders, plugin/service calls, and event handlers to use the new default. Updated and reorganized many unit tests to reflect the enum changes (including DumAemAttrProcessTest restructuring and additional assertions) and fixed a mapper instantiation in DumAemSourceApiTest. No other functional behavior changes intended beyond the event enum rename/expansion and related test updates.

Author: alegauss

aem-commons/src/main/java/com/viglet/dumont/connector/aem/commons/DumAemObjectGeneric.java

@@ -79,7 +79,7 @@ public class DumAemObjectGeneric {
     public final SimpleDateFormat aemJsonDateFormat = new SimpleDateFormat(DATE_JSON_FORMAT, Locale.ENGLISH);
 
     public DumAemObjectGeneric(String nodePath, JSONObject jcrNode) {
-        this(nodePath, jcrNode, DumAemEvent.NONE);
+        this(nodePath, jcrNode, DumAemEvent.INDEXING);
     }
 
     public DumAemObjectGeneric(String nodePath, JSONObject jcrNode, DumAemEvent event) {

aem-commons/src/main/java/com/viglet/dumont/connector/aem/commons/bean/DumAemEvent.java

@@ -1,10 +1,16 @@
 package com.viglet.dumont.connector.aem.commons.bean;
 
 public enum DumAemEvent {
-    UNPUBLISHING {
+    INDEXING {
         @Override
         public String toString() {
-            return "UNPUBLISHING";
+            return "INDEXING";
+        }
+    },
+    DEINDEXING {
+        @Override
+        public String toString() {
+            return "DEINDEXING";
         }
     },
     PUBLISHING {
@@ -13,10 +19,10 @@ public String toString() {
             return "PUBLISHING";
         }
     },
-    NONE {
+    UNPUBLISHING {
         @Override
         public String toString() {
-            return "NONE";
+            return "UNPUBLISHING";
         }
     }
 }

aem-commons/src/test/java/com/viglet/dumont/connector/aem/commons/DumAemObjectGenericTest.java

@@ -17,7 +17,10 @@
 
 package com.viglet.dumont.connector.aem.commons;
 
-import static org.junit.jupiter.api.Assertions.*;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertTrue;
 
 import org.json.JSONObject;
 import org.junit.jupiter.api.DisplayName;
@@ -98,7 +101,7 @@ void shouldHandleEventUnpublishing() {
         void shouldHandleEventNone() {
             JSONObject jcrNode = createJcrNodeWithContent();
 
-            DumAemObjectGeneric object = new DumAemObjectGeneric(TEST_PATH, jcrNode, DumAemEvent.NONE);
+            DumAemObjectGeneric object = new DumAemObjectGeneric(TEST_PATH, jcrNode, DumAemEvent.INDEXING);
 
             assertNotNull(object);
         }

aem-commons/src/test/java/com/viglet/dumont/connector/aem/commons/bean/DumAemEnvTest.java

@@ -17,7 +17,7 @@
 
 package com.viglet.dumont.connector.aem.commons.bean;
 
-import static org.junit.jupiter.api.Assertions.*;
+import static org.junit.jupiter.api.Assertions.assertEquals;
 
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Test;

aem-commons/src/test/java/com/viglet/dumont/connector/aem/commons/bean/DumAemEventTest.java

@@ -17,7 +17,7 @@
 
 package com.viglet.dumont.connector.aem.commons.bean;
 
-import static org.junit.jupiter.api.Assertions.*;
+import static org.junit.jupiter.api.Assertions.assertEquals;
 
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Test;
@@ -40,20 +40,21 @@ void shouldHavePublishingEnumValue() {
     @Test
     @DisplayName("Should have NONE enum value")
     void shouldHaveNoneEnumValue() {
-        assertEquals("NONE", DumAemEvent.NONE.toString());
+        assertEquals("INDEXING", DumAemEvent.INDEXING.toString());
     }
 
     @Test
     @DisplayName("Should have correct number of enum values")
     void shouldHaveCorrectNumberOfEnumValues() {
-        assertEquals(3, DumAemEvent.values().length);
+        assertEquals(4, DumAemEvent.values().length);
     }
 
     @Test
     @DisplayName("Should get enum by name")
     void shouldGetEnumByName() {
         assertEquals(DumAemEvent.UNPUBLISHING, DumAemEvent.valueOf("UNPUBLISHING"));
         assertEquals(DumAemEvent.PUBLISHING, DumAemEvent.valueOf("PUBLISHING"));
-        assertEquals(DumAemEvent.NONE, DumAemEvent.valueOf("NONE"));
+        assertEquals(DumAemEvent.INDEXING, DumAemEvent.valueOf("INDEXING"));
+        assertEquals(DumAemEvent.DEINDEXING, DumAemEvent.valueOf("DEINDEXING"));
     }
 }

aem/aem-plugin/src/main/java/com/viglet/dumont/connector/plugin/aem/impl/DumAemPlugin.java

@@ -59,7 +59,7 @@ public void indexAll(String source) {
     public void indexById(String source, List<String> contentId) {
         DumAemPathList dumAemPathList = DumAemPathList.builder()
                 .paths(contentId)
-                .event(DumAemEvent.NONE)
+                .event(DumAemEvent.INDEXING)
                 .recursive(false)
                 .build();
         dumAemPluginProcess.sentToIndexStandalone(source, dumAemPathList);

aem/aem-plugin/src/main/java/com/viglet/dumont/connector/plugin/aem/service/DumAemSessionService.java

@@ -50,7 +50,7 @@ public DumAemSession getDumAemSession(DumAemSource dumAemSource,
                 // Extract event and recursion settings with null-safe operations
                 DumAemEvent event = Optional.ofNullable(dumAemPathList)
                                 .map(DumAemPathList::getEvent)
-                                .orElse(DumAemEvent.NONE);
+                                .orElse(DumAemEvent.INDEXING);
 
                 boolean recursive = Optional.ofNullable(dumAemPathList)
                                 .map(DumAemPathList::getRecursive)