不要怂,就是干,撸起袖子干!

Commit 6091435e by shakib bawa

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

- adds hasTrigger option in model.js that is then passed to
querygenerator.update
- adds integration test for Model.update
1 parent 26582414
Showing with 19 additions and 1 deletions
...@@ -2252,6 +2252,7 @@ Model.prototype.update = function(values, options) { ...@@ -2252,6 +2252,7 @@ Model.prototype.update = function(values, options) {
valuesUse = Utils.mapValueFieldNames(valuesUse, options.fields, self); valuesUse = Utils.mapValueFieldNames(valuesUse, options.fields, self);
options = Utils.mapOptionFieldNames(options, self); options = Utils.mapOptionFieldNames(options, self);
options.hasTrigger = self.options ? self.options.hasTrigger : false;
// Run query to update all rows // Run query to update all rows
return self.QueryInterface.bulkUpdate(self.getTableName(options), valuesUse, options.where, options, self.tableAttributes).then(function(affectedRows) { return self.QueryInterface.bulkUpdate(self.getTableName(options), valuesUse, options.where, options, self.tableAttributes).then(function(affectedRows) {
......
...@@ -45,7 +45,7 @@ if (current.dialect.supports.tmpTableTrigger) { ...@@ -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({ return User.create({
username: 'triggertest' username: 'triggertest'
}).then(function(user){ }).then(function(user){
...@@ -57,6 +57,23 @@ if (current.dialect.supports.tmpTableTrigger) { ...@@ -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() { it('should successfully delete with a trigger on the table', function() {
return User.create({ return User.create({
username: 'triggertest' username: 'triggertest'
......
Markdown is supported
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!