edit node description (#822)

* add/edit node description

 ---------
Signed-off-by: SOUISSI Maissa (Externe) <[email protected]>

Author: souissimai

src/main/java/org/gridsuite/study/server/networkmodificationtree/entities/AbstractNodeInfoEntity.java

@@ -42,7 +42,7 @@ public UUID getId() {
     @Column(columnDefinition = "CLOB")
     String name;
 
-    @Column
+    @Column(columnDefinition = "CLOB")
     String description;
 
     @Column

src/main/java/org/gridsuite/study/server/notification/NotificationService.java

@@ -119,7 +119,7 @@ public class NotificationService {
     public static final String NODE_DELETED = "nodeDeleted";
     public static final String NODE_CREATED = "nodeCreated";
     public static final String NODE_MOVED = "nodeMoved";
-    public static final String NODE_RENAMED = "nodeRenamed";
+    public static final String NODE_EDITED = "nodeEdited";
     public static final String NODE_BUILD_STATUS_UPDATED = "nodeBuildStatusUpdated";
     public static final String SUBTREE_MOVED = "subtreeMoved";
     public static final String NODES_COLUMN_POSITIONS_CHANGED = "nodesColumnPositionsChanged";
@@ -337,8 +337,8 @@ public void emitNodeBuildStatusUpdated(UUID studyUuid, List<UUID> nodes, UUID ro
     }
 
     @PostCompletion
-    public void emitNodeRenamed(UUID studyUuid, UUID nodeUuid) {
-        sendStudyUpdateMessage(studyUuid, NODE_RENAMED, MessageBuilder.withPayload("")
+    public void emitNodeEdited(UUID studyUuid, UUID nodeUuid) {
+        sendStudyUpdateMessage(studyUuid, NODE_EDITED, MessageBuilder.withPayload("")
                 .setHeader(HEADER_NODE, nodeUuid)
         );
     }

src/main/java/org/gridsuite/study/server/service/NetworkModificationTreeService.java

@@ -519,8 +519,8 @@ public void updateNode(UUID studyUuid, NetworkModificationNode node, String user
             networkModificationNodeEntity.setDescription(node.getDescription());
         }
 
-        if (isRenameNode(node)) {
-            notificationService.emitNodeRenamed(self.getStudyUuidForNodeId(node.getId()), node.getId());
+        if (isEditedNode(node)) {
+            notificationService.emitNodeEdited(self.getStudyUuidForNodeId(node.getId()), node.getId());
         } else {
             notificationService.emitNodesChanged(self.getStudyUuidForNodeId(node.getId()), Collections.singletonList(node.getId()));
         }
@@ -548,13 +548,14 @@ public void updateNodesColumnPositions(UUID studyUuid, UUID parentUuid, List<Net
         notificationService.emitElementUpdated(studyUuid, userId);
     }
 
-    private boolean isRenameNode(AbstractNode node) {
-        NetworkModificationNode renameNode = NetworkModificationNode.builder()
-            .id(node.getId())
-            .name(node.getName())
-            .type(node.getType())
-            .build();
-        return renameNode.equals(node);
+    private boolean isEditedNode(AbstractNode node) {
+        NetworkModificationNode editedNode = NetworkModificationNode.builder()
+                .id(node.getId())
+                .name(node.getName())
+                .type(node.getType())
+                .description(node.getDescription())
+                .build();
+        return editedNode.equals(node);
     }
 
     private NodeEntity getNodeEntity(UUID nodeId) {

src/main/resources/db/changelog/changesets/changelog_20250807T200045Z.xml

@@ -0,0 +1,6 @@
+<?xml version="1.1" encoding="UTF-8" standalone="no"?>
+<databaseChangeLog xmlns="http://www.liquibase.org/xml/ns/dbchangelog" xmlns:ext="http://www.liquibase.org/xml/ns/dbchangelog-ext" xmlns:pro="http://www.liquibase.org/xml/ns/pro" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://www.liquibase.org/xml/ns/dbchangelog-ext http://www.liquibase.org/xml/ns/dbchangelog/dbchangelog-ext.xsd http://www.liquibase.org/xml/ns/pro http://www.liquibase.org/xml/ns/pro/liquibase-pro-latest.xsd http://www.liquibase.org/xml/ns/dbchangelog http://www.liquibase.org/xml/ns/dbchangelog/dbchangelog-latest.xsd">
+    <changeSet author="souissimai (generated)" id="1754596868860">
+        <modifyDataType tableName="network_modification_node_info" columnName="description" newDataType="CLOB"/>
+    </changeSet>
+</databaseChangeLog>

src/main/resources/db/changelog/db.changelog-master.yaml

@@ -330,5 +330,8 @@ databaseChangeLog:
       file: changesets/changelog_20250719T002545Z.xml
       relativeToChangelogFile: true
   - include:
-      file: changesets/changelog_20250723T152653Z.xml
+      file: changesets/changelog_20250807T200045Z.xml
       relativeToChangelogFile: true
+  - include:
+      file: changesets/changelog_20250723T152653Z.xml
+      relativeToChangelogFile: true

src/test/java/org/gridsuite/study/server/NetworkModificationTreeTest.java

@@ -95,7 +95,7 @@
 @ContextConfigurationWithTestChannel
 class NetworkModificationTreeTest {
     private static final Logger LOGGER = LoggerFactory.getLogger(NetworkModificationTreeTest.class);
-    private static final String NODE_UPDATED = "nodeUpdated";
+    private static final String NODE_EDITED = "nodeEdited";
     private static final long TIMEOUT = 1000;
 
     @Autowired
@@ -1127,7 +1127,7 @@ void testNodeUpdate() throws Exception {
                 .content(objectWriter.writeValueAsString(justANameUpdate))
                 .header(USER_ID_HEADER, "userId"))
             .andExpect(status().isOk());
-        assertEquals(NODE_RENAMED, output.receive(TIMEOUT, STUDY_UPDATE_DESTINATION).getHeaders().get(HEADER_UPDATE_TYPE));
+        assertEquals(NODE_EDITED, output.receive(TIMEOUT, STUDY_UPDATE_DESTINATION).getHeaders().get(HEADER_UPDATE_TYPE));
         checkElementUpdatedMessageSent(root.getStudyId(), userId);
 
         var newNode = getNode(root.getStudyId(), node1.getId(), firstRootNetworkUuid);
@@ -1174,9 +1174,7 @@ void testUpdateNodesColumnPositions() throws Exception {
         checkElementUpdatedMessageSent(root.getStudyId(), userId);
     }
 
-    // This test is for a part of the code that is not used yet
-    // We update a node description (this is not used in the front) and we assume that it will emit a nodeUpdated notif
-    // If it's not the case or if this test causes problems feel free to update it / remove it as needed
+    // We edit a node description and assume that it will emit a nodeEdited notif
     @Test
     void testNodeDescriptionUpdate() throws Exception {
         String userId = "userId";
@@ -1194,7 +1192,7 @@ void testNodeDescriptionUpdate() throws Exception {
                 .content(objectWriter.writeValueAsString(nodeDescriptionUpdate))
                 .header(USER_ID_HEADER, "userId"))
                 .andExpect(status().isOk());
-        assertEquals(NODE_UPDATED, output.receive(TIMEOUT, STUDY_UPDATE_DESTINATION).getHeaders().get(HEADER_UPDATE_TYPE));
+        assertEquals(NODE_EDITED, output.receive(TIMEOUT, STUDY_UPDATE_DESTINATION).getHeaders().get(HEADER_UPDATE_TYPE));
         checkElementUpdatedMessageSent(root.getStudyId(), userId);
     }