fix(overmind): fix returning operator execution in tests

Author: christianalfoni

packages/node_modules/overmind/src/index.ts

@@ -293,7 +293,7 @@ export class Overmind<ThisConfig extends IConfiguration>
       const onInitialize = this.createAction(
         'onInitialize',
         configuration.onInitialize
-      )
+      ) as any
 
       this.initialized = Promise.resolve(onInitialize(this))
     } else {
@@ -371,7 +371,7 @@ export class Overmind<ThisConfig extends IConfiguration>
     this.actionReferences.push(action)
     const actionFunc = async (value?) => {
       if (IS_PRODUCTION || action[IS_OPERATOR]) {
-        await new Promise((resolve, reject) => {
+        return new Promise((resolve, reject) => {
           const execution = this.createExecution(name, action)
           this.eventHub.emit(EventType.ACTION_START, execution)
 
@@ -451,15 +451,23 @@ export class Overmind<ThisConfig extends IConfiguration>
           result: undefined,
         })
         this.eventHub.emit(EventType.ACTION_END, execution)
-        await result
+
+        if (result) {
+          await result
+        }
+
         return this.options.testMode && execution
       }
     }
 
     if (this.options.testMode) {
       const actionCallback = this.options.testMode.actionCallback
 
-      return (value?) => (actionFunc as any)(value).then(actionCallback)
+      return async (value?) => {
+        const result = await actionFunc(value)
+
+        return actionCallback(result)
+      }
     }
 
     return actionFunc

packages/node_modules/proxy-state-tree/src/index.ts

@@ -163,7 +163,10 @@ export class ProxyStateTree<T extends object> implements IProxyStateTree<T> {
     }
 
     if (!changes.mutations.length && !changes.objectChanges.size) {
-      return
+      return {
+        mutations: [],
+        flushId: null,
+      }
     }
 
     const paths = new Set()