fix(delete): cascade getter does not necessarily return array

lib/query-interface.js

@@ -631,6 +631,8 @@ module.exports = (function() {
 
 
     return Promise.reduce(cascades, function (memo, cascade) {
     return Promise.reduce(cascades, function (memo, cascade) {
       return dao[cascade]().then(function (instances) {
       return dao[cascade]().then(function (instances) {
+        if (!Array.isArray(instances)) instances = [instances];
+        
         return Promise.reduce(instances, function (memo, instance) {
         return Promise.reduce(instances, function (memo, instance) {
           return instance.destroy();
           return instance.destroy();
         }, []);
         }, []);

test/integration/hooks.test.js

@@ -4676,12 +4676,13 @@ describe(Support.getTestDialectTeaser('Hooks'), function() {
             });
             });
           });
           });
 
 
-          it('with errors', function(done) {
+          it('with errors', function() {
             var self = this
             var self = this
               , beforeProject = false
               , beforeProject = false
               , afterProject = false
               , afterProject = false
               , beforeTask = false
               , beforeTask = false
-              , afterTask = false;
+              , afterTask = false
+              , VeryCustomError = function() {};
 
 
             this.Projects.beforeCreate(function(project, options, fn) {
             this.Projects.beforeCreate(function(project, options, fn) {
               beforeProject = true;
               beforeProject = true;
@@ -4695,7 +4696,7 @@ describe(Support.getTestDialectTeaser('Hooks'), function() {
 
 
             this.Tasks.beforeDestroy(function(task, options, fn) {
             this.Tasks.beforeDestroy(function(task, options, fn) {
               beforeTask = true;
               beforeTask = true;
-              fn(new Error('Whoops!'));
+              fn(new VeryCustomError('Whoops!'));
             });
             });
 
 
             this.Tasks.afterDestroy(function(task, options, fn) {
             this.Tasks.afterDestroy(function(task, options, fn) {
@@ -4703,16 +4704,14 @@ describe(Support.getTestDialectTeaser('Hooks'), function() {
               fn();
               fn();
             });
             });
 
 
-            this.Projects.create({title: 'New Project'}).success(function(project) {
-              self.Tasks.create({title: 'New Task'}).success(function(task) {
-                project.setTask(task).success(function() {
-                  project.destroy().error(function(err) {
-                    expect(err).to.be.instanceOf(Error);
+            return this.Projects.create({title: 'New Project'}).then(function(project) {
+              return self.Tasks.create({title: 'New Task'}).then(function(task) {
+                return project.setTask(task).then(function() {
+                  return expect(project.destroy()).to.eventually.be.rejectedWith(VeryCustomError).then(function () {
                     expect(beforeProject).to.be.true;
                     expect(beforeProject).to.be.true;
                     expect(afterProject).to.be.true;
                     expect(afterProject).to.be.true;
                     expect(beforeTask).to.be.true;
                     expect(beforeTask).to.be.true;
                     expect(afterTask).to.be.false;
                     expect(afterTask).to.be.false;
-                    done();
                   });
                   });
                 });
                 });
               });
               });