fix(modeling): do not remove default sequence flow on type change

* do not remove default sequence flow when changing type to intermediate catch

Closes #1197

Author: ssoorriiin

lib/features/modeling/BpmnUpdater.js

@@ -234,10 +234,13 @@ export default function BpmnUpdater(
 
     // on reconnectEnd -> default flow
     if ((businessObject.sourceRef && businessObject.sourceRef.default) &&
-        !(is(newTarget, 'bpmn:Activity') ||
-          is(newTarget, 'bpmn:EndEvent') ||
-          is(newTarget, 'bpmn:Gateway') ||
-          is(newTarget, 'bpmn:IntermediateThrowEvent'))) {
+        !isAny(newTarget, [
+          'bpmn:Activity',
+          'bpmn:EndEvent',
+          'bpmn:Gateway',
+          'bpmn:IntermediateThrowEvent',
+          'bpmn:IntermediateCatchEvent'
+        ])) {
       context.default = businessObject.sourceRef.default;
       businessObject.sourceRef.default = undefined;
     }

test/spec/features/popup-menu/ReplaceMenuProviderSpec.js

@@ -1459,7 +1459,8 @@ describe('features/popup-menu - replace menu provider', function() {
       [
         'bpmn:Activity',
         'bpmn:EndEvent',
-        'bpmn:IntermediateThrowEvent'
+        'bpmn:IntermediateThrowEvent',
+        'bpmn:IntermediateCatchEvent'
       ].forEach(function(type) {
 
         it('should keep DefaultFlow when changing target to ' + type,