fix

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

@@ -912,6 +912,13 @@
                 modificationUuidsByNodes.computeIfAbsent(nodeId, k -> new ArrayList<>()).add(ref.getReferenceId());
             }
         }));
-        modificationUuidsByNodes.entrySet().forEach(entry -> studyService.sharedModificationsUpdatedNotification(entry.getKey(), entry.getValue(), onlyMetadata));
+        // a failure on one node must not prevent notifying the other nodes
+        modificationUuidsByNodes.forEach((nodeUuid, modificationUuids) -> {
+            try {
+                studyService.sharedModificationsUpdatedNotification(nodeUuid, modificationUuids, onlyMetadata);
+            } catch (Exception e) {
+                LOGGER.error(String.format("Could not handle shared modifications update for node %s", nodeUuid), e);
+            }
+        });
     }
 }

src/test/java/org/gridsuite/study/server/service/ConsumerServiceSharedElementUpdateTest.java

@@ -137,6 +137,25 @@ void testStudyNodeAndCompositeReferencesInDifferentNode() {
         verify(studyService, times(1)).sharedModificationsUpdatedNotification(node2Uuid, List.of(compositeModificationUuid), false);
     }
 
+    @Test
+    void testFailureOnOneNodeDoesNotPreventOtherNodesNotification() {
+        UUID failingNodeUuid = UUID.randomUUID();
+        UUID nodeUuid = UUID.randomUUID();
+        UUID failingModificationUuid = UUID.randomUUID();
+        UUID modificationUuid = UUID.randomUUID();
+
+        doThrow(new RuntimeException("node notification failure"))
+            .when(studyService).sharedModificationsUpdatedNotification(eq(failingNodeUuid), anyList(), anyBoolean());
+
+        consumeSharedElementUpdate.accept(sharedElementUpdateMessage(
+            List.of(createStudyNodeReference(failingModificationUuid, failingNodeUuid)),
+            List.of(createCompositeReference(modificationUuid, nodeUuid))));
+
+        verify(studyService, times(2)).sharedModificationsUpdatedNotification(any(), anyList(), anyBoolean());
+        verify(studyService, times(1)).sharedModificationsUpdatedNotification(failingNodeUuid, List.of(failingModificationUuid), false);
+        verify(studyService, times(1)).sharedModificationsUpdatedNotification(nodeUuid, List.of(modificationUuid), false);
+    }
+
     @Test
     void testNoSharedCompositeUpdated() {
         consumeSharedElementUpdate.accept(sharedElementUpdateMessage(List.of(), List.of()));