feat(modeling): don't select start-event inside newly created sub-pro…

…cess The canonical modeling operation shall not be drill down, but continue to model.
bpmn-io · Dec 8, 2021 · 6214772 · 6214772
1 parent 88a484e
commit 6214772
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/lib/features/palette/PaletteProvider.js b/lib/features/palette/PaletteProvider.js
@@ -90,7 +90,7 @@ PaletteProvider.prototype.getPaletteEntries = function(element) {
 
     create.start(event, [ subProcess, startEvent ], {
       hints: {
-        autoSelect: [ startEvent ]
+        autoSelect: [ subProcess ]
       }
     });
   }

diff --git a/test/spec/features/palette/PaletteProviderSpec.js b/test/spec/features/palette/PaletteProviderSpec.js
@@ -65,7 +65,7 @@ describe('features/palette', function() {
     }));
 
 
-    it('should select start event', inject(function(canvas, dragging, selection) {
+    it('should select sub-process', inject(function(canvas, dragging, selection) {
 
       // given
       var rootElement = canvas.getRootElement(),
@@ -82,8 +82,10 @@ describe('features/palette', function() {
       dragging.end();
 
       // then
-      expect(selection.get()).to.have.length(1);
-      expect(is(selection.get()[0], 'bpmn:StartEvent')).to.be.true;
+      var selected = selection.get();
+
+      expect(selected).to.have.length(1);
+      expect(is(selected[0], 'bpmn:SubProcess')).to.be.true;
     }));
 
   });