More tests, finished porting

Author: mjackson

modules/components/__tests__/DefaultRoute-test.js

@@ -8,6 +8,11 @@ var PathStore = require('../../stores/PathStore');
 var Route = require('../Route');
 var DefaultRoute = require('../DefaultRoute');
 
+afterEach(function () {
+  // For some reason unmountComponentAtNode doesn't call componentWillUnmount :/
+  PathStore.removeAllChangeListeners();
+});
+
 describe('A DefaultRoute', function () {
   it('has a null path', function () {
     expect(DefaultRoute({ path: '/' }).props.path).toBe(null);
@@ -85,8 +90,6 @@ describe('when no child routes match a URL, but the parent\'s path matches', fun
 
   afterEach(function () {
     React.unmountComponentAtNode(component.getDOMNode());
-    // For some reason unmountComponentAtNode doesn't call componentWillUnmount :/
-    PathStore.removeAllChangeListeners();
   });
 
   it('matches the default route', function () {

modules/components/__tests__/NotFoundRoute-test.js

@@ -8,6 +8,11 @@ var PathStore = require('../../stores/PathStore');
 var Route = require('../Route');
 var NotFoundRoute = require('../NotFoundRoute');
 
+afterEach(function () {
+  // For some reason unmountComponentAtNode doesn't call componentWillUnmount :/
+  PathStore.removeAllChangeListeners();
+});
+
 describe('A NotFoundRoute', function () {
   it('has a null path', function () {
     expect(NotFoundRoute({ path: '/' }).props.path).toBe(null);
@@ -85,8 +90,6 @@ describe('when no child routes match a URL, but the beginning of the parent\'s p
 
   afterEach(function () {
     React.unmountComponentAtNode(component.getDOMNode());
-    // For some reason unmountComponentAtNode doesn't call componentWillUnmount :/
-    PathStore.removeAllChangeListeners();
   });
 
   it('matches the NotFoundRoute', function () {

modules/components/__tests__/Routes-test.js

@@ -10,6 +10,11 @@ function getRootMatch(matches) {
   return matches[matches.length - 1];
 }
 
+afterEach(function () {
+  // For some reason unmountComponentAtNode doesn't call componentWillUnmount :/
+  PathStore.removeAllChangeListeners();
+});
+
 describe('A Routes', function () {
 
   var App = React.createClass({
@@ -32,8 +37,6 @@ describe('A Routes', function () {
 
     afterEach(function () {
       React.unmountComponentAtNode(component.getDOMNode());
-      // For some reason unmountComponentAtNode doesn't call componentWillUnmount :/
-      PathStore.removeAllChangeListeners();
     });
 
     it('returns an array', function () {
@@ -60,8 +63,6 @@ describe('A Routes', function () {
 
     afterEach(function () {
       React.unmountComponentAtNode(component.getDOMNode());
-      // For some reason unmountComponentAtNode doesn't call componentWillUnmount :/
-      PathStore.removeAllChangeListeners();
     });
 
     it('returns an array with the correct params', function () {
@@ -74,4 +75,58 @@ describe('A Routes', function () {
     });
   });
 
+
+  describe('when a transition is aborted', function () {
+    it('triggers onAbortedTransition', function (done) {
+      var App = React.createClass({
+        statics: {
+          willTransitionTo: function (transition) {
+            transition.abort();
+          }
+        },
+        render: function () {
+          return React.DOM.div();
+        }
+      });
+
+      function handleAbortedTransition(transition) {
+        assert(transition);
+        done();
+      }
+
+      ReactTestUtils.renderIntoDocument(
+        Routes({ onAbortedTransition: handleAbortedTransition },
+          Route({ handler: App })
+        )
+      );
+    });
+  });
+
+  describe('when there is an error in a transition hook', function () {
+    it('triggers onTransitionError', function (done) {
+      var App = React.createClass({
+        statics: {
+          willTransitionTo: function (transition) {
+            throw new Error('boom!');
+          }
+        },
+        render: function () {
+          return React.DOM.div();
+        }
+      });
+
+      function handleTransitionError(error) {
+        assert(error);
+        expect(error.message).toEqual('boom!');
+        done();
+      }
+
+      ReactTestUtils.renderIntoDocument(
+        Routes({ onTransitionError: handleTransitionError },
+          Route({ handler: App })
+        )
+      );
+    });
+  });
+
 });