fix(resource): check whether response matches action.isArray

When using $resource you must setup your actions carefully based on what the server returns. If the server responds to a request with an array then you must configure the action with `isArray:true` and vice versa.  The built-in `get` action defaults to `isArray:false` and the `query` action defaults to `isArray:true`, which is must be changed if the server does not do this.
Before the error message was an exception inside angular.copy, which didn't explain what the real problem was.
Rather than changing the way that angular.copy works, this change ensures that a better error message is provided to the programmer if they do not set up their resource actions correctly.

Closes angular#2255, angular#1044

Author: petebacondarwin

src/ngResource/resource.js

@@ -473,6 +473,10 @@ angular.module('ngResource', ['ng']).
                 promise = value.$promise;
 
             if (data) {
+              if ( angular.isArray(data) != !!action.isArray ) {
+                return $q.reject("Error in resource configuration. Expected response to contain an " +
+                  (action.isArray?'array':'object') + " but got an " + (angular.isArray(data)?'array':'object'));
+              }
               if (action.isArray) {
                 value.length = 0;
                 forEach(data, function(item) {

test/ngResource/resourceSpec.js

@@ -807,6 +807,27 @@ describe("resource", function() {
     });
   });
 
+  it('should fail if action expects an object but response is an array', function() {
+    var successHandler = jasmine.createSpy('successHandler');
+    var failureHandler = jasmine.createSpy('failureHandler');
+    $httpBackend.expect('GET', '/Customer/123').respond({id: 'abc'});
+    $resource('/Customer/123').query().$promise.then(successHandler, failureHandler);
+    $httpBackend.flush();
+    expect(successHandler).not.toHaveBeenCalled();
+    expect(failureHandler).toHaveBeenCalledWith(
+      'Error in resource configuration. Expected response to contain an array but got an object');
+  });
+
+  it('should fail if action expects an array but response is an object', function() {
+    var successHandler = jasmine.createSpy('successHandler');
+    var failureHandler = jasmine.createSpy('failureHandler');
+    $httpBackend.expect('GET', '/Customer/123').respond([1,2,3]);
+    $resource('/Customer/123').get().$promise.then(successHandler, failureHandler);
+    $httpBackend.flush();
+    expect(successHandler).not.toHaveBeenCalled();
+    expect(failureHandler).toHaveBeenCalledWith(
+      'Error in resource configuration. Expected response to contain an object but got an array');
+  });
 
   it('should transform request/response', function() {
     var Person = $resource('/Person/:id', {}, {