fix(ngResource): do not eat exceptions in success callback

src/ngResource/resource.js

@@ -784,18 +784,18 @@ angular.module('ngResource', ['ng']).
                 return value;
               },
               (hasError || hasResponseErrorInterceptor) ?
-              function(response) {
-                if (hasError) error(response);
-                return hasResponseErrorInterceptor ?
+                function(response) {
+                  if (hasError && !hasResponseErrorInterceptor) {
+                    // Avoid `Possibly Unhandled Rejection` error,
+                    // but still fulfill the returned promise with a rejection
+                    promise.catch(noop);
+                  }
+                  if (hasError) error(response);
+                  return hasResponseErrorInterceptor ?
                     responseErrorInterceptor(response) :
                     $q.reject(response);
-              } :
-              undefined);
-            if (hasError && !hasResponseErrorInterceptor) {
-              // Avoid `Possibly Unhandled Rejection` error,
-              // but still fulfill the returned promise with a rejection
-              promise.catch(noop);
-            }
+                } :
+                undefined);
 
             if (!isInstanceCall) {
               // we are creating instance / collection

test/ngResource/resourceSpec.js

@@ -1978,4 +1978,39 @@ describe('configuring `cancellable` on the provider', function() {
     expect(creditCard3.$cancelRequest).toBeDefined();
   });
 });
+
+  describe('exception handling in callbacks', function() {
+    var $resource;
+    var $httpBackend;
+    var CreditCard;
+    beforeEach(module('ngResource'));
+
+    beforeEach(inject(function($injector) {
+      $httpBackend = $injector.get('$httpBackend');
+      $resource = $injector.get('$resource');
+      CreditCard = $resource('/CreditCard/:id:verb', { id: '@id.key' });
+    }));
+
+    it('should throw exception in success callback when error callback provided', function() {
+      $httpBackend.expect('GET', '/CreditCard/123').respond({ id: 123, number: '9876' });
+
+      var cc = CreditCard.get({ id: 123 },
+        function(res) {
+          throw new Error('should be caught');
+        }, function() {});
+
+      expect($httpBackend.flush).toThrow(new Error('should be caught'));
+    });
+
+    it('should throw exception in success callback when error callback not provided', function() {
+      $httpBackend.expect('GET', '/CreditCard/123').respond({ id: 123, number: '9876' });
+
+      var cc = CreditCard.get({ id: 123 },
+        function(res) {
+          throw new Error('should be caught');
+        });
+
+      expect($httpBackend.flush).toThrow(new Error('should be caught'));
+    });
+  });
 });