fix(state): fail transition on exceptions in transition handler

closes #2772

Author: metamatt

src/state.js

@@ -1170,7 +1170,7 @@ function $StateProvider(   $urlRouterProvider,   $urlMatcherFactory) {
         $urlRouter.update(true);
 
         return $state.current;
-      }, function (error) {
+      }).then(null, function (error) {
         if ($state.transition !== transition) return TransitionSuperseded;
 
         $state.transition = null;

test/stateSpec.js

@@ -120,6 +120,11 @@ describe('state', function () {
         },
         controller: function() { log += "controller;"}
       })
+      .state('onEnterFail', {
+        onEnter: function() {
+          throw new Error('negative onEnter');
+        }
+      })
       .state('badParam', {
         url: "/bad/{param:int}"
       })
@@ -551,6 +556,20 @@ describe('state', function () {
         'A.onEnter;');
     }));
 
+    it('sends $stateChangeError for exceptions in onEnter', inject(function ($state, $q, $rootScope) {
+      var called;
+      $rootScope.$on('$stateChangeError', function (ev, to, toParams, from, fromParams, options) {
+        called = true;
+      });
+
+      initStateTo(A);
+      $state.transitionTo('onEnterFail');
+      $q.flush();
+
+      expect(called).toBeTruthy();
+      expect($state.current.name).toEqual(A.name);
+    }));
+
     it('doesn\'t transition to parent state when child has no URL', inject(function ($state, $q) {
       $state.transitionTo('about.sidebar'); $q.flush();
       expect($state.current.name).toEqual('about.sidebar');
@@ -1044,6 +1063,7 @@ describe('state', function () {
         'logA',
         'logA.logB',
         'logA.logB.logC',
+        'onEnterFail',
         'resolveFail',
         'resolveTimeout',
         'root',