Merge pull request remix-run#163 from bripkens/failing-navigation

[fixed] Navigation to root URL can fail

modules/stores/URLStore.js

@@ -58,18 +58,18 @@ var URLStore = {
   getCurrentPath: function () {
     if (_location === 'history')
       return getWindowPath();
-    
+
     if (_location === 'hash')
       return window.location.hash.substr(1);
-    
+
     return _currentPath;
   },
 
   /**
    * Pushes the given path onto the browser navigation stack.
    */
   push: function (path) {
-    if (path === _currentPath)
+    if (path === this.getCurrentPath())
       return;
 
     if (_location === 'disabledHistory')
@@ -193,6 +193,7 @@ var URLStore = {
     }
 
     _location = null;
+    _currentPath = '/';
   }
 
 };

specs/URLStore.spec.js

@@ -56,3 +56,26 @@ describe('when going back in history', function () {
     expect(error).toEqual(true);
   });
 });
+
+describe('when navigating back to the root', function() {
+  beforeEach(function () {
+    // not all tests are constructing and tearing down the URLStore.
+    // Let's set it up correctly once and then tear it down to ensure that all
+    // variables in the URLStore module are reset.
+    URLStore.setup('hash');
+    URLStore.teardown();
+
+    // simulating that the browser opens a page with #/dashboard
+    window.location.hash = '/dashboard';
+    URLStore.setup('hash');
+  });
+
+  afterEach(function () {
+    URLStore.teardown();
+  });
+
+  it('should have the correct path', function () {
+    URLStore.push('/');
+    expect(window.location.hash).toEqual('#/');
+  });
+});