fix(MovePreview): add connections only once

related to bpmn-io/bpmn-js#2179
bpmn-io · May 29, 2024 · 7e2bfe3 · 7e2bfe3
1 parent 442438d
commit 7e2bfe3
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 8 deletions.
diff --git a/lib/features/move/MovePreview.js b/lib/features/move/MovePreview.js
@@ -68,7 +68,10 @@ export default function MovePreview(
       return (shape.incoming || []).concat(shape.outgoing || []);
     });
 
-    return flatten(allShapes.concat(allConnections));
+    var allElements = flatten(allShapes.concat(allConnections));
+    var uniqueElements = [ ...new Set(allElements) ];
+
+    return uniqueElements;
   }
 
   /**

diff --git a/test/spec/features/move/MovePreviewSpec.js b/test/spec/features/move/MovePreviewSpec.js
@@ -587,16 +587,16 @@ describe('features/move - MovePreview', function() {
       // then
       var container = canvas.getContainer();
 
-      // var clonedMarkers = domQueryAll('marker.djs-dragger', container);
-      // expect(clonedMarkers).to.have.length(6);
+      var clonedMarkers = domQueryAll('marker.djs-dragger', container);
+      expect(clonedMarkers).to.have.length(6);
 
       var markerStartClones = [ ...domQueryAll('marker[id^="marker-start-djs-dragger-clone"]', container) ],
           markerMidClones = [ ...domQueryAll('marker[id^="marker-mid-djs-dragger-clone"]', container) ],
           markerEndClones = [ ...domQueryAll('marker[id^="marker-end-djs-dragger-clone"]', container) ];
 
-      expect(markerStartClones).to.have.length(4);
-      expect(markerMidClones).to.exist.length(4);
-      expect(markerEndClones).to.exist.length(4);
+      expect(markerStartClones).to.have.length(2);
+      expect(markerMidClones).to.exist.length(2);
+      expect(markerEndClones).to.exist.length(2);
 
       var markerStartCloneIds = markerStartClones.map(marker => marker.id),
           markerMidCloneIds = markerMidClones.map(marker => marker.id),
@@ -743,8 +743,8 @@ describe('features/move - MovePreview', function() {
       // then
       var container = canvas.getContainer();
 
-      // var clonedMarkers = domQueryAll('marker.djs-dragger', container);
-      // expect(clonedMarkers).to.have.length(7);
+      var clonedMarkers = domQueryAll('marker.djs-dragger', container);
+      expect(clonedMarkers).to.have.length(7);
 
       var markerStartClone = domQuery('marker[id^="marker-start-connection3-djs-dragger-clone"]', container),
           markerMidClone = domQuery('marker[id^="marker-mid-connection3-djs-dragger-clone"]', container),
@@ -801,6 +801,7 @@ describe('features/move - MovePreview', function() {
     }));
 
   });
+
 });
 
 // helpers //////////