Merge pull request #4083 from shakdoesgithub/mssql-trigger-support

adds support for for table triggers in mssql dialect for Model.update

lib/model.js

@@ -2306,6 +2306,7 @@ Model.prototype.update = function(values, options) {
 
     valuesUse = Utils.mapValueFieldNames(valuesUse, options.fields, self);
     options = Utils.mapOptionFieldNames(options, self);
+    options.hasTrigger =  self.options ? self.options.hasTrigger : false;
 
     // Run query to update all rows
     return self.QueryInterface.bulkUpdate(self.getTableName(options), valuesUse, options.where, options, self.tableAttributes).then(function(affectedRows) {

test/integration/trigger.test.js

@@ -45,7 +45,7 @@ if (current.dialect.supports.tmpTableTrigger) {
         });
       });
 
-      it('should return output rows after update', function() {
+      it('should return output rows after instance update', function() {
         return User.create({
           username: 'triggertest'
         }).then(function(user){
@@ -57,6 +57,23 @@ if (current.dialect.supports.tmpTableTrigger) {
         });
       });
 
+      it('should return output rows after Model update', function() {
+        return User.create({
+          username: 'triggertest'
+        }).then(function(user){
+          return User.update({
+            username: 'usernamechanged'
+          }, {
+            where: {
+              id: user.get('id')
+            }
+          });
+        })
+        .then(function (user) {
+          return expect(User.find({username: 'usernamechanged'})).to.eventually.have.property('username').which.equals('usernamechanged');
+        });
+      });
+
       it('should successfully delete with a trigger on the table', function() {
         return User.create({
           username: 'triggertest'