fix: increment/decrement dont map updatedAt field

changelog.md

 # Future
 # Future
-- [FIXED] `restore` now uses `field` from `deletedAt` 
+- [FIXED] `restore` now uses `field` from `deletedAt`
 - [FIXED] MSSQL bulkInsertQuery when options and attributes are not passed
 - [FIXED] MSSQL bulkInsertQuery when options and attributes are not passed
 - [FIXED] `DATEONLY` now returns `YYYY-MM-DD` date string [#4858] (https://github.com/sequelize/sequelize/issues/4858)
 - [FIXED] `DATEONLY` now returns `YYYY-MM-DD` date string [#4858] (https://github.com/sequelize/sequelize/issues/4858)
 - [FIXED] Issues with `createFunction` and `dropFunction` (PostgresSQL)
 - [FIXED] Issues with `createFunction` and `dropFunction` (PostgresSQL)
@@ -8,6 +8,7 @@
 - [FIXED] GroupedLimit when foreignKey has a field alias
 - [FIXED] GroupedLimit when foreignKey has a field alias
 - [FIXED] groupedLimit.through.where support
 - [FIXED] groupedLimit.through.where support
 - [ADDED] `option.silent` for increment and decrement [#6795](https://github.com/sequelize/sequelize/pull/6795)
 - [ADDED] `option.silent` for increment and decrement [#6795](https://github.com/sequelize/sequelize/pull/6795)
+- [FIXED] `Increment` / `Decrement` properly maps to timestamp fields [#6296](https://github.com/sequelize/sequelize/issues/6296)
 
 
 ## BC breaks:
 ## BC breaks:
 - `DATEONLY` now returns string in `YYYY-MM-DD` format rather than `Date` type
 - `DATEONLY` now returns string in `YYYY-MM-DD` format rather than `Date` type

lib/model.js

@@ -3660,6 +3660,7 @@ class Model {
   increment(fields, options) {
   increment(fields, options) {
     const identifier = this.where();
     const identifier = this.where();
     const updatedAtAttr = this.constructor._timestampAttributes.updatedAt;
     const updatedAtAttr = this.constructor._timestampAttributes.updatedAt;
+    const updatedAtAttribute = this.constructor.rawAttributes[updatedAtAttr];
 
 
     options = _.defaults({}, options, {
     options = _.defaults({}, options, {
       by: 1,
       by: 1,
@@ -3681,7 +3682,7 @@ class Model {
     }
     }
 
 
     if (!options.silent && updatedAtAttr && !values[updatedAtAttr]) {
     if (!options.silent && updatedAtAttr && !values[updatedAtAttr]) {
-      options.attributes[updatedAtAttr] = this.constructor._getDefaultTimestamp(updatedAtAttr) || Utils.now(this.sequelize.options.dialect);
+      options.attributes[updatedAtAttribute.field || updatedAtAttr] = this.constructor._getDefaultTimestamp(updatedAtAttr) || Utils.now(this.sequelize.options.dialect);
     }
     }
 
 
     for (const attr of Object.keys(values)) {
     for (const attr of Object.keys(values)) {

test/integration/model/attributes/field.test.js

@@ -73,17 +73,14 @@ describe(Support.getTestDialectTeaser('Model'), function() {
             autoIncrement: true,
             autoIncrement: true,
             field: 'commentId'
             field: 'commentId'
           },
           },
-          text: {
-            type: DataTypes.STRING,
-            field: 'comment_text'
-          },
-          notes: {
-            type: DataTypes.STRING,
-            field: 'notes'
-          }
+          text: { type: DataTypes.STRING, field: 'comment_text' },
+          notes: { type: DataTypes.STRING, field: 'notes' },
+          likes: { type: DataTypes.INTEGER, field: 'like_count' },
+          createdAt: { type: DataTypes.DATE, field: 'created_at', allowNull: false },
+          updatedAt: { type: DataTypes.DATE, field: 'updated_at', allowNull: false }
         }, {
         }, {
           tableName: 'comments',
           tableName: 'comments',
-          timestamps: false
+          timestamps: true
         });
         });
 
 
         this.User.hasMany(this.Task, {
         this.User.hasMany(this.Task, {
@@ -145,6 +142,16 @@ describe(Support.getTestDialectTeaser('Model'), function() {
             },
             },
             notes: {
             notes: {
               type: DataTypes.STRING
               type: DataTypes.STRING
+            },
+            like_count: {
+              type: DataTypes.INTEGER
+            },
+            created_at: {
+              type: DataTypes.DATE,
+              allowNull: false
+            },
+            updated_at: {
+              type: DataTypes.DATE
             }
             }
           })
           })
         ]);
         ]);
@@ -231,6 +238,35 @@ describe(Support.getTestDialectTeaser('Model'), function() {
         });
         });
       });
       });
 
 
+      it('increment should work', function() {
+        return this.Comment.destroy({ truncate: true })
+          .then(() => this.Comment.create({ note: 'oh boy, here I go again', likes: 23 }))
+          .then((comment) => comment.increment('likes'))
+          .then((comment) => comment.reload())
+          .then((comment) => {
+            expect(comment.likes).to.be.equal(24);
+          });
+      });
+
+      it('decrement should work', function() {
+        return this.Comment.destroy({ truncate: true })
+          .then(() => this.Comment.create({ note: 'oh boy, here I go again', likes: 23 }))
+          .then((comment) => comment.decrement('likes'))
+          .then((comment) => comment.reload())
+          .then((comment) => {
+            expect(comment.likes).to.be.equal(22);
+          });
+      });
+
+      it('sum should work', function() {
+        return this.Comment.destroy({ truncate: true })
+          .then(() => this.Comment.create({ note: 'oh boy, here I go again', likes: 23 }))
+          .then(() => this.Comment.sum('likes'))
+          .then((likes) => {
+            expect(likes).to.be.equal(23);
+          });
+      });
+
       it('should create, fetch and update with alternative field names from a simple model', function() {
       it('should create, fetch and update with alternative field names from a simple model', function() {
         var self = this;
         var self = this;