feat(build): lighter/faster build (#58)

* wip

* yarn lock

* upgrade deps

* clarify connect

* done

Author: adrienharnay

__tests__/internals/reducer/generateReducer/__snapshots__/getReducerCases.test.js.snap

@@ -61,9 +61,6 @@ Object {
     },
   },
   "resources": Object {
-    "animals": Object {
-      "1": "worm",
-    },
     "fruits": Object {
       "1": "banana",
       "2": "cherry",
@@ -132,9 +129,6 @@ Object {
     },
   },
   "resources": Object {
-    "animals": Object {
-      "1": "worm",
-    },
     "fruits": Object {
       "1": "banana",
       "2": "cherry",
@@ -207,9 +201,6 @@ Object {
     },
   },
   "resources": Object {
-    "animals": Object {
-      "1": "worm",
-    },
     "fruits": Object {
       "1": "banana",
       "2": "cherry",
@@ -278,9 +269,6 @@ Object {
     },
   },
   "resources": Object {
-    "animals": Object {
-      "1": "worm",
-    },
     "fruits": Object {
       "1": "banana",
       "2": "cherry",
@@ -636,9 +624,6 @@ Object {
     },
   },
   "resources": Object {
-    "animals": Object {
-      "1": "worm",
-    },
     "fruits": Object {
       "1": "banana",
       "2": "cherry",
@@ -725,9 +710,6 @@ Object {
     },
   },
   "resources": Object {
-    "animals": Object {
-      "1": "worm",
-    },
     "fruits": Object {
       "1": "banana",
       "2": "cherry",
@@ -872,9 +854,6 @@ Object {
     },
   },
   "resources": Object {
-    "animals": Object {
-      "1": "worm",
-    },
     "fruits": undefined,
     "seeds": Object {
       "2": "pumpkin",

__tests__/internals/utils/__snapshots__/mergeResources.test.js.snap

@@ -1,59 +1,38 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`mergeResources merge empty in filled 1`] = `
+exports[`mergeResources happy path 1`] = `
 Object {
-  "fruits": Object {
-    "apple": Object {
-      "juicy": true,
-      "red": true,
-    },
-    "cherry": Object {
-      "juicy": false,
-    },
-  },
-  "vegetables": Object {
-    "carrot": Object {
-      "orange": true,
+  "eggs": Object {
+    "1": Object {
+      "a": 2,
     },
   },
-}
-`;
-
-exports[`mergeResources merge filled in empty 1`] = `
-Object {
   "fruits": Object {
-    "apple": Object {
-      "juicy": true,
-      "red": true,
-    },
-    "cherry": Object {
-      "juicy": false,
-    },
-  },
-  "vegetables": Object {
-    "carrot": Object {
-      "orange": true,
-    },
-  },
-}
-`;
-
-exports[`mergeResources merge filled2 in filled 1`] = `
-Object {
-  "fruits": Object {
-    "apple": Object {
-      "juicy": false,
-    },
-    "banana": Object {
-      "yellow": true,
+    "1": Object {
+      "a": 1,
     },
-    "cherry": Object {
-      "juicy": false,
+    "2": Object {
+      "c": 3,
     },
   },
   "vegetables": Object {
-    "carrot": Object {
-      "orange": true,
+    "1": Object {
+      "a": 1,
+      "b": 2,
+      "c": 3,
+    },
+    "2": Object {
+      "a": 1,
+      "b": 2,
+      "c": 3,
+    },
+    "3": Object {
+      "a": 1,
+      "b": 2,
+      "c": 3,
+    },
+    "4": Object {
+      "a": 1,
     },
   },
 }

__tests__/internals/utils/__snapshots__/shallowMergeResources.test.js.snap

@@ -0,0 +1,23 @@
+import isPlainObject from '../../../src/internals/utils/isPlainObject';
+
+describe('isPlainObject', () => {
+  test('not plain objects', () => {
+    expect(isPlainObject()).toBe(false);
+    expect(isPlainObject(null)).toBe(false);
+    expect(isPlainObject(false)).toBe(false);
+    expect(isPlainObject(true)).toBe(false);
+    expect(isPlainObject(0)).toBe(false);
+    expect(isPlainObject(1)).toBe(false);
+    expect(isPlainObject(-1)).toBe(false);
+    expect(isPlainObject('')).toBe(false);
+    expect(isPlainObject('hello')).toBe(false);
+    expect(isPlainObject([])).toBe(false);
+    expect(isPlainObject([''])).toBe(false);
+    expect(isPlainObject([0])).toBe(false);
+  });
+
+  test('plain objects', () => {
+    expect(isPlainObject({})).toBe(true);
+    expect(isPlainObject({ a: 1 })).toBe(true);
+  });
+});

__tests__/internals/utils/isPlainObject.test.js

@@ -1,53 +1,75 @@
 import mergeResources from '../../../src/internals/utils/mergeResources';
 
-const EMPTY_STATE = {};
-const FILLED_STATE = {
+const PREVIOUS_STATE = {
   resources: {
+    eggs: {
+      1: {
+        a: 1,
+      },
+    },
     fruits: {
-      apple: {
-        juicy: true,
-        red: true,
+      1: {
+        a: 1,
+        b: 2,
       },
-      cherry: {
-        juicy: false,
+      2: {
+        a: 1,
+        b: 2,
       },
     },
     vegetables: {
-      carrot: {
-        orange: true,
+      1: {
+        a: 1,
+        b: 2,
+        c: 3,
+      },
+      2: {
+        a: 1,
+        b: 2,
+        c: 3,
+      },
+      3: {
+        a: 1,
+        b: 2,
+        c: 3,
       },
     },
   },
 };
-const FILLED_STATE2 = {
-  resources: {
-    fruits: {
-      apple: {
-        juicy: false,
-      },
-      banana: {
-        yellow: true,
-      },
+
+const NORMALIZED_PAYLOAD = {
+  eggs: {
+    1: {
+      a: 2,
+    },
+  },
+  fruits: {
+    1: {
+      a: 1,
+    },
+    2: {
+      c: 3,
+    },
+  },
+  vegetables: {
+    4: {
+      a: 1,
     },
   },
 };
 
 describe('mergeResources', () => {
-  test('merge filled in empty', () => {
-    expect(
-      mergeResources(EMPTY_STATE, FILLED_STATE.resources),
-    ).toMatchSnapshot();
-  });
-
-  test('merge empty in filled', () => {
-    expect(
-      mergeResources(FILLED_STATE, EMPTY_STATE.resources),
-    ).toMatchSnapshot();
+  test('bad arguments', () => {
+    expect(Object.keys(mergeResources()).length).toBe(0);
+    expect(mergeResources(PREVIOUS_STATE)).toBe(PREVIOUS_STATE.resources);
+    expect(mergeResources(PREVIOUS_STATE, null)).toBe(PREVIOUS_STATE.resources);
+    expect(mergeResources(PREVIOUS_STATE, 1)).toBe(PREVIOUS_STATE.resources);
+    expect(mergeResources(PREVIOUS_STATE, {})).toBe(PREVIOUS_STATE.resources);
   });
 
-  test('merge filled2 in filled', () => {
+  test('happy path', () => {
     expect(
-      mergeResources(FILLED_STATE, FILLED_STATE2.resources),
+      mergeResources(PREVIOUS_STATE, NORMALIZED_PAYLOAD),
     ).toMatchSnapshot();
   });
 });