fix(overmind): fix not having state on root when merging config

Author: christianalfoni

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

@@ -95,6 +95,18 @@ describe('Config', () => {
     expect(merged.state.configB.bar).toEqual('baz')
   })
 
+  test('should merge without root state', () => {
+    const configA = {}
+    const configB = {
+      state: {
+        bar: 'baz',
+      },
+    }
+    const merged = merge(configA, namespaced({ configB }))
+
+    expect(merged.state.configB.bar).toEqual('baz')
+  })
+
   test('should keep context when lazy loading namespaces', async () => {
     const config = {
       state: {
@@ -103,20 +115,21 @@ describe('Config', () => {
       actions: {
         changeFoo(context) {
           context.state.config.foo = 'bar2'
-        }
-      }
+        },
+      },
     }
-    
-    const overmind = createOvermindMock(lazy({
-      config: () => Promise.resolve(config)
-    }))
+
+    const overmind = createOvermindMock(
+      lazy({
+        config: () => Promise.resolve(config),
+      })
+    )
 
     await overmind.onInitialize()
 
     await overmind.actions.lazy.loadConfig('config')
     // @ts-ignore
     overmind.actions.config.changeFoo()
     expect(overmind.state.config!.foo).toEqual('bar2')
-    
   })
 })

packages/node_modules/overmind/src/config/merge.ts

@@ -123,8 +123,6 @@ export function merge<
   configI: I
 ): A & B & C & D & E & F & G & H & I
 
-
-
 export function merge(...configurations: IConfiguration[]): IConfiguration {
   const initializers = configurations.reduce(
     (aggr, config) =>
@@ -178,6 +176,9 @@ export function merge(...configurations: IConfiguration[]): IConfiguration {
       }
     },
     {
+      state: {},
+      actions: {},
+      effects: {},
       ...rootConfiguration,
       onInitialize: initializers.length
         ? (context, value) =>