Merge pull request cerebral#92 from etienne-dldc/next

fix(overmind): use operators in compose

Author: christianalfoni

packages/node_modules/overmind/src/Action.ts

@@ -274,9 +274,7 @@ export default class ActionClass<
   ) => [InitialValue] extends [void]
     ? INoValueAction<State, Effects, InitialValue, T>
     : IValueAction<State, Effects, InitialValue, T> = (action) => {
-    const composedAction = (action as any)(
-      () => new ActionClass(this.proxyStateTree, this.getActionChain())
-    )
+    const composedAction = (action as any)(this.operators)
     const operator = (effects, value) => {
       return composedAction(value, effects)
     }

packages/node_modules/overmind/src/index.test.ts

@@ -336,4 +336,24 @@ describe('OPERATORS', () => {
       expect(end - start).toBeGreaterThanOrEqual(100)
     })
   })
+  test('compose', () => {
+    const mockCallback = jest.fn()
+    const runMock: Action = ({ run }) =>
+      run(() => {
+        mockCallback()
+      })
+    const doThis: Action = ({ compose }) => compose(runMock)
+    const config = {
+      actions: {
+        doThis,
+      },
+    }
+    type Config = TConfig<{
+      actions: typeof config.actions
+    }>
+    type Action<Input = void, Output = any> = TAction<Input, Output, Config>
+    const app = new App(config)
+    app.actions.doThis()
+    expect(mockCallback.mock.calls.length).toBe(1)
+  })
 })