Major state refactor

.size-snapshot.json

@@ -1,21 +1,21 @@
 {
   "dist/react-beautiful-dnd.js": {
-    "bundled": 400269,
-    "minified": 151652,
-    "gzipped": 43139
+    "bundled": 391074,
+    "minified": 148259,
+    "gzipped": 42162
   },
   "dist/react-beautiful-dnd.min.js": {
-    "bundled": 358556,
-    "minified": 134751,
-    "gzipped": 38201
+    "bundled": 353559,
+    "minified": 135149,
+    "gzipped": 37985
   },
   "dist/react-beautiful-dnd.esm.js": {
-    "bundled": 177759,
-    "minified": 89807,
-    "gzipped": 22649,
+    "bundled": 174113,
+    "minified": 88585,
+    "gzipped": 22372,
     "treeshaked": {
-      "rollup": 78288,
-      "webpack": 80060
+      "rollup": 76698,
+      "webpack": 78485
     }
   }
 }

jest.config.js

@@ -2,4 +2,11 @@
 
 module.exports = {
   setupFiles: ['./test/setup.js'],
+  // node_modules is default.
+  // We want to ignore any tests in the website(sometimes they are added to / website /.cache)
+  testPathIgnorePatterns: ['/node_modules/', '/website/.cache/'],
+  watchPlugins: [
+    'jest-watch-typeahead/filename',
+    'jest-watch-typeahead/testname',
+  ],
 };

package.json

@@ -47,58 +47,57 @@
     "babel-runtime": "^6.26.0",
     "css-box-model": "^0.0.14",
     "memoize-one": "^3.1.1",
-    "prop-types": "^15.6.0",
+    "prop-types": "^15.6.1",
     "raf-schd": "^2.1.2",
     "react-motion": "^0.5.2",
     "react-redux": "^5.0.7",
     "redux": "^4.0.0",
-    "redux-thunk": "^2.2.0",
-    "reselect": "^3.0.1",
     "tiny-invariant": "^0.0.3"
   },
   "devDependencies": {
-    "@atlaskit/css-reset": "^2.0.0",
-    "@storybook/react": "^3.4.3",
+    "@atlaskit/css-reset": "^2.0.2",
+    "@storybook/react": "^3.4.6",
     "babel-cli": "^6.26.0",
     "babel-core": "^6.26.3",
     "babel-eslint": "^8.2.3",
-    "babel-jest": "^22.4.3",
+    "babel-jest": "^23.0.1",
     "babel-plugin-styled-components": "^1.5.1",
     "babel-plugin-transform-class-properties": "^6.24.1",
     "babel-plugin-transform-es2015-modules-commonjs": "^6.26.2",
     "babel-plugin-transform-export-extensions": "^6.22.0",
     "babel-plugin-transform-object-rest-spread": "^6.26.0",
     "babel-plugin-transform-runtime": "^6.23.0",
-    "babel-preset-env": "^1.6.1",
+    "babel-preset-env": "^1.7.0",
     "babel-preset-flow": "^6.23.0",
     "babel-preset-react": "^6.24.1",
-    "cross-env": "^5.1.4",
+    "cross-env": "^5.1.6",
     "enzyme": "^3.3.0",
     "enzyme-adapter-react-16": "^1.1.1",
     "eslint": "^4.19.1",
     "eslint-config-airbnb": "^16.1.0",
-    "eslint-plugin-flowtype": "^2.46.3",
-    "eslint-plugin-import": "^2.11.0",
-    "eslint-plugin-jest": "^21.15.1",
+    "eslint-plugin-flowtype": "^2.49.3",
+    "eslint-plugin-import": "^2.12.0",
+    "eslint-plugin-jest": "^21.17.0",
     "eslint-plugin-jsx-a11y": "^6.0.3",
-    "eslint-plugin-react": "^7.7.0",
-    "flow-bin": "0.71.0",
-    "jest": "^22.4.3",
-    "puppeteer": "^1.3.0",
+    "eslint-plugin-react": "^7.9.1",
+    "flow-bin": "0.73.0",
+    "jest": "^23.1.0",
+    "jest-watch-typeahead": "^0.1.0",
+    "puppeteer": "^1.4.0",
     "raf-stub": "^2.0.2",
     "react": "^16.3.1",
     "react-dom": "^16.3.1",
     "react-test-renderer": "^16.3.1",
     "rimraf": "^2.6.2",
-    "rollup": "^0.58.2",
-    "rollup-plugin-babel": "^3.0.3",
-    "rollup-plugin-commonjs": "^9.1.0",
+    "rollup": "^0.59.4",
+    "rollup-plugin-babel": "^3.0.4",
+    "rollup-plugin-commonjs": "^9.1.3",
     "rollup-plugin-node-resolve": "^3.3.0",
     "rollup-plugin-replace": "^2.0.0",
     "rollup-plugin-size-snapshot": "^0.4.0",
     "rollup-plugin-strip": "^1.1.1",
     "rollup-plugin-uglify": "^3.0.0",
-    "styled-components": "^3.2.6",
+    "styled-components": "^3.3.2",
     "stylelint": "9.2.1",
     "stylelint-config-standard": "^18.2.0",
     "stylelint-config-styled-components": "^0.1.1",

src/debug/timings.js

@@ -1,4 +1,5 @@
 // @flow
+import invariant from 'tiny-invariant';
 
 type Records = {
   [key: string]: number,
@@ -10,7 +11,7 @@ const flag: string = '__react-beautiful-dnd-debug-timings-hook__';
 
 const isTimingsEnabled = (): boolean => Boolean(window[flag]);
 
-// TEMP
+// Debug: uncomment to enable
 // window[flag] = true;
 
 export const start = (key: string) => {
@@ -35,10 +36,7 @@ export const finish = (key: string) => {
 
   const previous: ?number = records[key];
 
-  if (previous == null) {
-    console.error('cannot finish timing as no previous time found');
-    return;
-  }
+  invariant(previous, 'cannot finish timing as no previous time found');
 
   const result: number = now - previous;
   const rounded: string = result.toFixed(2);