fluxible-router: remove currentNavigate.route from dehydrated state (#…

…670)

Besides avoiding unnecessary data to be sent to the client (given that
RouteStore currently doesn't need currentNavigate.route to
re-hydrate), we prevent browsers to break given that
currentNavigate.route.action and currentNavigate.route.handler may
contain server transpiled code that is not compatible with them.

packages/fluxible-router/lib/RouteStore.js

@@ -151,8 +151,9 @@ var RouteStore = createStore({
     dehydrate: function () {
         // no need to dehydrate this._prevNavigate, because it will always
         // be null on server request
+        var currentNavigate = Object.assign({}, this._currentNavigate, {route: null});
         return {
-            currentNavigate: this._currentNavigate,
+            currentNavigate: currentNavigate,
             routes: this._routes
         };
     },

packages/fluxible-router/tests/unit/lib/RouteStore-test.js

@@ -35,6 +35,7 @@ describe('RouteStore', function () {
                 expect(state.currentNavigate.transactionId).to.equal('first');
                 expect(state.currentNavigate.url).to.equal('/foo');
                 expect(state.currentNavigate.method).to.equal('get');
+                expect(state.currentNavigate.route).to.equal(null);
                 expect(state.routes).to.equal(null);
             });
         });