Optimizations (planttheidea#9)

* Updates...
* Move all constants to constants (no duplicate declarations of default property names)
* Refactor a few things to be more functional
* Return dev sourcemap to cheap-module-eval-source-map

* Updates...
* Correct alphabetization

DEV_ONLY/App.js

@@ -38,10 +38,10 @@ class Div extends Component {
       children
     } = this.props;
 
-    // console.group('Div');
-    // console.log('div position', this.props.position);
-    // console.log('div size', this.props.size);
-    // console.groupEnd();
+    console.group('Div');
+    console.log('div position', this.props.position);
+    console.log('div size', this.props.size);
+    console.groupEnd();
 
     return (
       <div style={DIV_STYLES}>
@@ -100,10 +100,10 @@ class Main extends Component {
       children
     } = this.props;
 
-    console.group('Main');
-    console.log('main position', this.props.position);
-    console.log('main size', this.props.size);
-    console.groupEnd();
+    // console.group('Main');
+    // console.log('main position', this.props.position);
+    // console.log('main size', this.props.size);
+    // console.groupEnd();
 
     return (
       <main>

package.json

@@ -95,5 +95,5 @@
     "test:watch": "NODE_ENV=test ava --no-cache --watch",
     "transpile": "./node_modules/babel-cli/lib/babel/index.js -d lib/ src/"
   },
-  "version": "1.4.0"
+  "version": "1.4.1"
 }

src/constants.js

@@ -1,3 +1,7 @@
+const POSITION_PROP_DEFAULT = 'position';
+const RENDER_ON_RESIZE_DEFAULT = true;
+const SIZE_PROP_DEFAULT = 'size';
+
 const BOUNDING_CLIENT_RECT_SIZE_KEYS = [
   'height',
   'width'
@@ -75,3 +79,6 @@ export {BOUNDING_CLIENT_RECT_SIZE_KEYS};
 export {DOM_ELEMENT_POSITION_KEYS};
 export {DOM_ELEMENT_SIZE_KEYS};
 export {initialState};
+export {POSITION_PROP_DEFAULT};
+export {RENDER_ON_RESIZE_DEFAULT};
+export {SIZE_PROP_DEFAULT};

src/getHigherOrderComponent.js

@@ -19,13 +19,12 @@ import {
 import {
   ALL_BOUNDING_CLIENT_RECT_KEYS,
   ALL_DOM_ELEMENT_KEYS,
-  ALL_KEYS
+  ALL_KEYS,
+  POSITION_PROP_DEFAULT,
+  RENDER_ON_RESIZE_DEFAULT,
+  SIZE_PROP_DEFAULT
 } from './constants';
 
-const POSITION_PROP_DEFAULT = 'position';
-const RENDER_ON_RESIZE_DEFAULT = true;
-const SIZE_PROP_DEFAULT = 'size';
-
 let raf;
 
 /**
@@ -43,6 +42,26 @@ const setRaf = () => {
   );
 };
 
+/**
+ * based on desiredKeys, build the initialState object
+ *
+ * @param {array<string>} allKeys
+ * @param {array<string>} desiredKeys
+ * @returns {array<T>}
+ */
+const reduceStateToMatchingKeys = (allKeys, desiredKeys) => {
+  return allKeys.reduce((accumulatedInitialState, key) => {
+    if (desiredKeys.includes(key)) {
+      return {
+        ...accumulatedInitialState,
+        [key]: 0
+      };
+    }
+
+    return accumulatedInitialState;
+  }, {});
+};
+
 /**
  * create the HOC that injects the position and size props
  * into the child (assuming they have keys that are valid
@@ -67,17 +86,7 @@ const getHigherOrderComponent = (OriginalComponent, keys, options = {}) => {
 
   const boundingClientRectKeys = arraySubset(ALL_BOUNDING_CLIENT_RECT_KEYS, keys);
   const domElementKeys = arraySubset(ALL_DOM_ELEMENT_KEYS, keys);
-
-  const initialState = ALL_KEYS.reduce((accumulatedInitialState, key) => {
-    if (keys.includes(key)) {
-      return {
-        ...accumulatedInitialState,
-        [key]: 0
-      };
-    }
-
-    return accumulatedInitialState;
-  }, {});
+  const initialState = reduceStateToMatchingKeys(ALL_KEYS, keys);
 
   if (!raf) {
     setRaf();