fix: allow to create connection + event-based gateway

Closes #1490

lib/features/modeling/behavior/EventBasedGatewayBehavior.js

@@ -43,12 +43,18 @@ export default function EventBasedGatewayBehavior(eventBus, modeling) {
     var sequenceFlows = [];
 
     if (is(source, 'bpmn:EventBasedGateway')) {
-      sequenceFlows = target.incoming.filter(isSequenceFlow);
+      sequenceFlows = target.incoming
+        .filter(flow =>
+          flow !== connection &&
+          isSequenceFlow(flow)
+        );
     } else {
-      sequenceFlows = target.incoming.filter(function(connection) {
-        return isSequenceFlow(connection)
-          && is(connection.source, 'bpmn:EventBasedGateway');
-      });
+      sequenceFlows = target.incoming
+        .filter(flow =>
+          flow !== connection &&
+          isSequenceFlow(flow) &&
+          is(flow.source, 'bpmn:EventBasedGateway')
+        );
     }
 
     sequenceFlows.forEach(function(sequenceFlow) {

test/spec/features/modeling/behavior/EventBasedGatewayBehaviorSpec.js

@@ -4,9 +4,13 @@ import {
 } from 'test/TestHelper';
 
 import coreModule from 'lib/core';
+import createModule from 'diagram-js/lib/features/create';
 import modelingModule from 'lib/features/modeling';
 import replaceModule from 'lib/features/replace';
 
+import { createCanvasEvent as canvasEvent } from '../../../../util/MockEvents';
+
+
 describe('features/modeling/behavior - event-based gateway', function() {
 
   var diagramXML = require('./EventBasedGatewayBehavior.bpmn');
@@ -145,3 +149,64 @@ describe('features/modeling/behavior - event-based gateway', function() {
   });
 
 });
+
+
+describe('features/modeling/behavior - event-based gateway - integration', function() {
+
+  var diagramXML = require('./EventBasedGatewayBehavior.bpmn');
+
+  var testModules = [
+    coreModule,
+    createModule,
+    modelingModule,
+    replaceModule
+  ];
+
+  beforeEach(bootstrapModeler(diagramXML, {
+    modules: testModules
+  }));
+
+
+  it('should create with target', inject(
+    function(create, elementRegistry, elementFactory, dragging) {
+      // given
+      const gateway = elementFactory.createShape({
+        type: "bpmn:EventBasedGateway",
+        x: 0,
+        y: 0
+      });
+
+      const event = elementFactory.createShape({
+        type: "bpmn:IntermediateCatchEvent",
+        eventDefinitionType: "bpmn:MessageEventDefinition",
+        x: 120,
+        y: 120
+      });
+
+      const waypoints = [
+        { x: gateway.x, y: gateway.y },
+        { x: event.x, y: event.y }
+      ];
+
+      const connection = elementFactory.createConnection({
+        type: "bpmn:SequenceFlow",
+        source: gateway,
+        target: event,
+        waypoints
+      });
+
+      const rootElement = elementRegistry.get('Process_1');
+      const rootGfx = elementRegistry.getGraphics('Process_1');
+
+      // when
+      create.start(canvasEvent({ x: 0, y: 0 }), [ gateway, event, connection ]);
+      dragging.hover({ element: rootElement, gfx: rootGfx });
+      dragging.move(canvasEvent({ x: 100, y: 200 }));
+      dragging.end();
+
+      // then
+      expect(elementRegistry.get(connection.id)).to.exist;
+    }
+  ));
+
+});