Fix failing tests

heydemo · Oct 10, 2014 · bd23312 · bd23312
1 parent 6b9982c
commit bd23312
Show file tree

Hide file tree

Showing 2 changed files with 31 additions and 7 deletions.
diff --git a/modules/components/__tests__/Link-test.js b/modules/components/__tests__/Link-test.js
@@ -24,7 +24,11 @@ describe('A Link', function () {
         )
       );
 
-      component.dispatch('/mjackson/home', done);
+      component.dispatch('/mjackson/home', function (error, abortReason, nextState) {
+        expect(error).toBe(null);
+        expect(abortReason).toBe(null);
+        component.setState(nextState, done);
+      });
     });
 
     afterEach(function () {
@@ -52,7 +56,11 @@ describe('A Link', function () {
         )
       );
 
-      component.dispatch('/', done);
+      component.dispatch('/', function (error, abortReason, nextState) {
+        expect(error).toBe(null);
+        expect(abortReason).toBe(null);
+        component.setState(nextState, done);
+      });
     });
 
     afterEach(function () {

diff --git a/modules/mixins/__tests__/Navigation-test.js b/modules/mixins/__tests__/Navigation-test.js
@@ -22,12 +22,16 @@ describe('Navigation', function () {
       var component;
       beforeEach(function (done) {
         component = ReactTestUtils.renderIntoDocument(
-          Routes({ location: 'none' }, 
+          Routes({ location: 'none', onChange: done }, 
             Route({ name: 'home', path: '/:username/home', handler: NavigationHandler })
           )
         );
 
-        component.dispatch('/anybody/home', done);
+        component.dispatch('/anybody/home', function (error, abortReason, nextState) {
+          expect(error).toBe(null);
+          expect(abortReason).toBe(null);
+          component.setState(nextState, done);
+        });
       });
 
       afterEach(function () {
@@ -50,7 +54,11 @@ describe('Navigation', function () {
           )
         );
 
-        component.dispatch('/home', done);
+        component.dispatch('/home', function (error, abortReason, nextState) {
+          expect(error).toBe(null);
+          expect(abortReason).toBe(null);
+          component.setState(nextState, done);
+        });
       });
 
       afterEach(function () {
@@ -78,7 +86,11 @@ describe('Navigation', function () {
           )
         );
 
-        component.dispatch('/home', done);
+        component.dispatch('/home', function (error, abortReason, nextState) {
+          expect(error).toBe(null);
+          expect(abortReason).toBe(null);
+          component.setState(nextState, done);
+        });
       });
 
       afterEach(function () {
@@ -101,7 +113,11 @@ describe('Navigation', function () {
           )
         );
 
-        component.dispatch('/home', done);
+        component.dispatch('/home', function (error, abortReason, nextState) {
+          expect(error).toBe(null);
+          expect(abortReason).toBe(null);
+          component.setState(nextState, done);
+        });
       });
 
       afterEach(function () {