fix(modeling/behavior): hook UpdateFlowNodeRefs behavior in on elemen…

…ts.create
bpmn-io · Dec 6, 2019 · 4455c3f · 4455c3f
1 parent d769e6d
commit 4455c3f
Show file tree

Hide file tree

Showing 2 changed files with 42 additions and 4 deletions.
diff --git a/lib/features/modeling/behavior/UpdateFlowNodeRefsBehavior.js b/lib/features/modeling/behavior/UpdateFlowNodeRefsBehavior.js
@@ -72,8 +72,9 @@ export default function UpdateFlowNodeRefsBehavior(eventBus, modeling, translate
     'lane.add',
     'lane.resize',
     'lane.split',
-    'elements.move',
+    'elements.create',
     'elements.delete',
+    'elements.move',
     'shape.create',
     'shape.delete',
     'shape.move',

diff --git a/test/spec/features/modeling/lanes/UpdateFlowNodeRefsSpec.js b/test/spec/features/modeling/lanes/UpdateFlowNodeRefsSpec.js
@@ -5,15 +5,22 @@ import {
 
 import modelingModule from 'lib/features/modeling';
 import coreModule from 'lib/core';
+import copyPasteModule from 'lib/features/copy-paste';
+
+/* global sinon */
 
 
 describe('features/modeling - lanes - flowNodeRefs', function() {
 
   var diagramXML = require('./flowNodeRefs.bpmn');
 
-  var testModules = [ coreModule, modelingModule ];
-
-  beforeEach(bootstrapModeler(diagramXML, { modules: testModules }));
+  beforeEach(bootstrapModeler(diagramXML, {
+    modules: [
+      coreModule,
+      modelingModule,
+      copyPasteModule
+    ]
+  }));
 
 
   describe('should unwire during move', function() {
@@ -290,4 +297,34 @@ describe('features/modeling - lanes - flowNodeRefs', function() {
     expect(lane1.flowNodeRef).to.have.length(1);
     expect(lane2.flowNodeRef).to.have.length(2);
   }));
+
+
+  describe('should wire once during paste', function() {
+
+    it('execute', inject(function(canvas, eventBus, elementRegistry, copyPaste) {
+
+      // given
+      var participant = elementRegistry.get('Participant_A');
+
+      var updateRefsSpy = sinon.spy();
+
+      eventBus.on('commandStack.lane.updateRefs.execute', updateRefsSpy);
+
+      // when
+      copyPaste.copy(participant);
+
+      copyPaste.paste({
+        element: canvas.getRootElement(),
+        point: {
+          x: 350,
+          y: 150
+        }
+      });
+
+      // then
+      expect(updateRefsSpy).to.have.been.calledOnce;
+    }));
+
+  });
+
 });