feat(action-chain): add name of operators to events

Author: christianalfoni

packages/node_modules/action-chain/src/ActionBase.ts

@@ -35,7 +35,7 @@ export class ActionBase<Context, InitialValue, Value = InitialValue> {
     this.initialActionId = initialActionId
 
     let currentExecutionId = 0
-    return Object.assign(function(value) {
+    const func = Object.assign(function(value) {
       const initialOperator = typeof arguments[1] === 'undefined'
       const newPath = typeof arguments[2] === 'undefined' ? null : arguments[2]
       const executionContext: ExecutionContext = initialOperator
@@ -52,28 +52,34 @@ export class ActionBase<Context, InitialValue, Value = InitialValue> {
         actionChain.emit('action:start', {
           actionId: executionContext.__execution.actionId,
           executionId: executionContext.__execution.executionId,
+          actionName: func.displayName,
+          value,
         })
       }
       const returnValue = runOperators
         ? runOperators(value, executionContext, newPath)
         : value
 
       if (initialOperator && returnValue instanceof Promise) {
-        returnValue.then(() => {
+        returnValue.then((promiseValue) => {
           actionChain.emit('action:end', {
             actionId: executionContext.__execution.actionId,
             executionId: executionContext.__execution.executionId,
+            actionName: func.displayName,
           })
         })
       } else if (initialOperator) {
         actionChain.emit('action:end', {
           actionId: executionContext.__execution.actionId,
           executionId: executionContext.__execution.executionId,
+          actionName: func.displayName,
         })
       }
 
       return returnValue
     }, this) as any
+
+    return func
   }
   getActionChain = () => {
     return this.actionChain

packages/node_modules/action-chain/src/ActionChain.ts

@@ -22,9 +22,13 @@ export type Execution = {
 export type ActionExecution = {
   actionId: number
   executionId: number
+  actionName: string
+  value?: any
 }
 
-export type OperatorExecution = ActionExecution & {
+export type OperatorExecution = {
+  actionId: number
+  executionId: number
   operatorId: number
   type: string
   name: string

packages/node_modules/action-chain/src/index.test.ts

@@ -80,6 +80,18 @@ beforeEach(() => {
   ActionBase.nextActionId = 0
 })
 
+describe('ACTION', () => {
+  test('should be able to give the action a name', () => {
+    expect.assertions(1)
+    const test = action() as any
+    test.displayName = 'My Name'
+    actionChain.once('action:start', (execution) => {
+      expect(execution.actionName).toEqual('My Name')
+    })
+    test()
+  })
+})
+
 describe('VALUE', () => {
   test('should run and return result', () => {
     const test = action<string>()
@@ -91,8 +103,6 @@ describe('VALUE', () => {
 describe('CONTEXT', () => {
   test('should pass default context', () => {
     expect.assertions(2)
-    const wut = action()
-    console.log(wut.test)
     const test = action().test(({ __execution, __path }: any) => {
       expect(__execution).toBeTruthy()
       expect(__path).toBeTruthy()