Use PathStore.setup/teardown for testing

Author: mjackson

modules/mixins/LocationContext.js

@@ -62,7 +62,7 @@ var LocationContext = {
     );
 
     if (location)
-      PathStore.useLocation(location);
+      PathStore.setup(location);
   },
 
   /**

modules/stores/PathStore.js

@@ -9,7 +9,8 @@ function notifyChange() {
   _events.emit(CHANGE_EVENT);
 }
 
-var _currentLocation, _currentPath, _currentActionType;
+var _currentLocation, _currentActionType;
+var _currentPath = '/';
 
 function handleLocationChangeAction(action) {
   if (_currentPath !== action.path) {
@@ -39,7 +40,7 @@ var PathStore = {
   /**
    * Setup the PathStore to use the given location.
    */
-  useLocation: function (location) {
+  setup: function (location) {
     invariant(
       _currentLocation == null || _currentLocation === location,
       'You cannot use %s and %s on the same page',
@@ -56,6 +57,19 @@ var PathStore = {
     _currentLocation = location;
   },
 
+  /**
+   * Tear down the PathStore. Really only used for testing.
+   */
+  teardown: function () {
+    if (_currentLocation && _currentLocation.teardown)
+      _currentLocation.teardown();
+
+    _currentLocation = _currentActionType = null;
+    _currentPath = '/';
+
+    PathStore.removeAllChangeListeners();
+  },
+
   /**
    * Returns the current URL path.
    */

modules/stores/__tests__/PathStore-test.js

@@ -1,18 +1,29 @@
 var assert = require('assert');
 var expect = require('expect');
 var LocationActions = require('../../actions/LocationActions');
-var LocationDispatcher = require('../../dispatchers/LocationDispatcher');
 var PathStore = require('../PathStore');
 
 describe('PathStore', function () {
 
+  var _onChange;
+  var MockLocation = {
+    setup: function (onChange) {
+      _onChange = onChange;
+    },
+    getCurrentPath: function () {
+      return '/';
+    },
+    toString: function () {
+      return '<MockLocation>';
+    }
+  };
+
   beforeEach(function () {
-    LocationDispatcher.handleViewAction({
-      type: LocationActions.PUSH,
-      path: '/'
-    });
+    PathStore.setup(MockLocation);
   });
 
+  afterEach(PathStore.teardown);
+
   var changeWasFired;
   function changeListener() {
     changeWasFired = true;
@@ -30,7 +41,7 @@ describe('PathStore', function () {
   describe('when a new URL path is pushed', function () {
     beforeEach(setupChangeListener);
     beforeEach(function () {
-      LocationDispatcher.handleViewAction({
+      _onChange({
         type: LocationActions.PUSH,
         path: '/push'
       });
@@ -54,7 +65,7 @@ describe('PathStore', function () {
   describe('when a URL path is replaced', function () {
     beforeEach(setupChangeListener);
     beforeEach(function () {
-      LocationDispatcher.handleViewAction({
+      _onChange({
         type: LocationActions.REPLACE,
         path: '/replace'
       });
@@ -78,7 +89,7 @@ describe('PathStore', function () {
   describe('when a URL path is popped', function () {
     beforeEach(setupChangeListener);
     beforeEach(function () {
-      LocationDispatcher.handleViewAction({
+      _onChange({
         type: LocationActions.POP,
         path: '/pop'
       });