fix: ensure logical order of task listener event types

Author: barmac

CHANGELOG.md

@@ -6,6 +6,8 @@ All notable changes to [bpmn-js-properties-panel](https://github.com/bpmn-io/bpm
 
 ___Note:__ Yet to be released changes appear here._
 
+* `FIX`: ensure logical order of task listener event types ([#1102](https://github.com/bpmn-io/bpmn-js-properties-panel/pull/1102))
+
 ## 5.30.0
 
 * `FEAT`: rename task listener event types ([#1098](https://github.com/bpmn-io/bpmn-js-properties-panel/pull/1098))

src/provider/zeebe/properties/TaskListener.js

@@ -6,11 +6,15 @@ import {
 
 import { ListenerType, Retries } from './shared/Listener';
 
-export const EVENT_TYPE = [ 'completing', 'assigning' ];
+// ensure types are in logical order
+export const EVENT_TYPE = [
+  'assigning',
+  'completing'
+];
 
 export const EVENT_TO_LABEL = {
-  completing: 'Completing',
-  assigning: 'Assigning'
+  assigning: 'Assigning',
+  completing: 'Completing'
 };
 
 export function TaskListenerEntries(props) {

test/spec/provider/zeebe/TaskListenerProps.spec.js

@@ -265,7 +265,7 @@ describe('provider/zeebe - TaskListenerProps', function() {
 
   describe('event type', function() {
 
-    it('should use a supported event type for a new listener', inject(
+    it('should use "assigning" as event type for a new listener', inject(
       async function(elementRegistry, selection) {
 
         // given
@@ -288,7 +288,7 @@ describe('provider/zeebe - TaskListenerProps', function() {
         const listeners = getListeners(element);
         const newListener = listeners[listeners.length - 1];
 
-        expect(newListener).to.have.property('eventType', 'completing');
+        expect(newListener).to.have.property('eventType', 'assigning');
       })
     );