Make sure that before and after hooks are called

test/hooks.test.js

@@ -6995,20 +6995,24 @@ describe(Support.getTestDialectTeaser("Hooks"), function () {
     })
   })
 
-  describe('passing DAO instances', function() {
+  describe.only('passing DAO instances', function() {
 
     describe('beforeValidate / afterValidate', function() {
       it('should pass a DAO instance to the hook', function(done){
+        var beforeHooked = false
+        var afterHooked = false
         var User = this.sequelize.define('User', {
           username: DataTypes.STRING
         }, {
           hooks: {
             beforeValidate: function(user, fn) {
               expect(user).to.be.instanceof(User.DAO)
+              beforeHooked = true
               fn()
             },
             afterValidate: function(user, fn) {
               expect(user).to.be.instanceof(User.DAO)
+              afterHooked = true
               fn()
             }
           }
@@ -7016,6 +7020,8 @@ describe(Support.getTestDialectTeaser("Hooks"), function () {
 
         User.sync({ force: true }).success(function() {
           User.create({ username: 'bob' }).success(function(user) {
+            expect(beforeHooked).to.be.true
+            expect(afterHooked).to.be.true
             done()
           })
         })
@@ -7024,16 +7030,20 @@ describe(Support.getTestDialectTeaser("Hooks"), function () {
 
     describe('beforeCreate / afterCreate', function() {
       it('should pass a DAO instance to the hook', function(done){
+        var beforeHooked = false
+        var afterHooked = false
         var User = this.sequelize.define('User', {
           username: DataTypes.STRING
         }, {
           hooks: {
             beforeCreate: function(user, fn) {
               expect(user).to.be.instanceof(User.DAO)
+              beforeHooked = true
               fn()
             },
             afterCreate: function(user, fn) {
               expect(user).to.be.instanceof(User.DAO)
+              afterHooked = true
               fn()
             }
           }
@@ -7041,6 +7051,8 @@ describe(Support.getTestDialectTeaser("Hooks"), function () {
 
         User.sync({ force: true }).success(function() {
           User.create({ username: 'bob' }).success(function(user) {
+            expect(beforeHooked).to.be.true
+            expect(afterHooked).to.be.true
             done()
           })
         })
@@ -7049,16 +7061,20 @@ describe(Support.getTestDialectTeaser("Hooks"), function () {
 
     describe('beforeDestroy / afterDestroy', function() {
       it('should pass a DAO instance to the hook', function(done){
+        var beforeHooked = false
+        var afterHooked = false
         var User = this.sequelize.define('User', {
           username: DataTypes.STRING
         }, {
           hooks: {
             beforeDestroy: function(user, fn) {
               expect(user).to.be.instanceof(User.DAO)
+              beforeHooked = true
               fn()
             },
             afterDestroy: function(user, fn) {
               expect(user).to.be.instanceof(User.DAO)
+              afterHooked = true
               fn()
             }
           }
@@ -7067,6 +7083,8 @@ describe(Support.getTestDialectTeaser("Hooks"), function () {
         User.sync({ force: true }).success(function() {
           User.create({ username: 'bob' }).success(function(user) {
             user.destroy().success(function() {
+              expect(beforeHooked).to.be.true
+              expect(afterHooked).to.be.true
               done()
             })
           })
@@ -7076,16 +7094,20 @@ describe(Support.getTestDialectTeaser("Hooks"), function () {
 
     describe('beforeDelete / afterDelete', function() {
       it('should pass a DAO instance to the hook', function(done){
+        var beforeHooked = false
+        var afterHooked = false
         var User = this.sequelize.define('User', {
           username: DataTypes.STRING
         }, {
           hooks: {
             beforeDelete: function(user, fn) {
               expect(user).to.be.instanceof(User.DAO)
+              beforeHooked = true
               fn()
             },
             afterDelete: function(user, fn) {
               expect(user).to.be.instanceof(User.DAO)
+              afterHooked = true
               fn()
             }
           }
@@ -7094,6 +7116,8 @@ describe(Support.getTestDialectTeaser("Hooks"), function () {
         User.sync({ force: true }).success(function() {
           User.create({ username: 'bob' }).success(function(user) {
             user.destroy().success(function() {
+              expect(beforeHooked).to.be.true
+              expect(afterHooked).to.be.true
               done()
             })
           })
@@ -7103,16 +7127,20 @@ describe(Support.getTestDialectTeaser("Hooks"), function () {
 
     describe('beforeUpdate / afterUpdate', function() {
       it('should pass a DAO instance to the hook', function(done){
+        var beforeHooked = false
+        var afterHooked = false
         var User = this.sequelize.define('User', {
           username: DataTypes.STRING
         }, {
           hooks: {
             beforeUpdate: function(user, fn) {
               expect(user).to.be.instanceof(User.DAO)
+              beforeHooked = true
               fn()
             },
             afterUpdate: function(user, fn) {
               expect(user).to.be.instanceof(User.DAO)
+              afterHooked = true
               fn()
             }
           }
@@ -7121,6 +7149,8 @@ describe(Support.getTestDialectTeaser("Hooks"), function () {
         User.sync({ force: true }).success(function() {
           User.create({ username: 'bob' }).success(function(user) {
             user.save({ username: 'bawb' }).success(function() {
+              expect(beforeHooked).to.be.true
+              expect(afterHooked).to.be.true
               done()
             })
           })