feat(model): add support for pre-defining timestamp values

src/Lucid/Model/index.js

@@ -548,7 +548,7 @@ class Model extends BaseModel {
    */
   _setUpdatedAt (values, onInsert) {
     const updatedAtColumn = this.constructor.updatedAtColumn
-    if (updatedAtColumn && ((onInsert && !values[updatedAtColumn]) || !onInsert)) {
+    if (updatedAtColumn && !this.dirty[updatedAtColumn]) {
       values[updatedAtColumn] = this._getSetterValue(updatedAtColumn, new Date())
     }
   }

src/Lucid/QueryBuilder/index.js

@@ -518,6 +518,16 @@ class QueryBuilder {
 
     const valuesCopy = _.clone(valuesOrModelInstance)
     const fakeModel = new this.Model()
+
+    /**
+     * Here we fill attributes on the model, so that the logic to
+     * find if `updated_at` should be set or not can be
+     * evaluated by the model instance.
+     *
+     * For this model instance relies on `dirty` values.
+     */
+    fakeModel.fill(valuesCopy)
+
     fakeModel._setUpdatedAt(valuesCopy)
     fakeModel._formatDateFields(valuesCopy)
 

test/unit/lucid.spec.js

@@ -2028,4 +2028,44 @@ test.group('Model', (group) => {
 
     assert.equal(userQuery.sql, helpers.formatQuery('select * from "users" where (exists (select * from "profiles" where "users"."id" = "profiles"."user_id")) limit ?'))
   })
+
+  test('do not set created_at when explicitly set in values', async (assert) => {
+    class User extends Model {
+    }
+
+    User._bootIfNotBooted()
+    const createdAt = moment().subtract('1', 'day').format('YYYY-MM-DD HH:mm:ss')
+
+    const user = await User.create({ username: 'virk', created_at: createdAt })
+    await user.reload()
+
+    assert.equal(user.created_at, createdAt)
+  })
+
+  test('do not set updated_at when explicitly set in values', async (assert) => {
+    class User extends Model {
+    }
+
+    User._bootIfNotBooted()
+    const updatedAt = moment().subtract('1', 'day').format('YYYY-MM-DD HH:mm:ss')
+
+    const user = await User.create({ username: 'virk', updated_at: updatedAt })
+    await user.reload()
+
+    assert.equal(user.updated_at, updatedAt)
+  })
+
+  test('do not set updated_at when calling update on query builder', async (assert) => {
+    class User extends Model {
+    }
+
+    User._bootIfNotBooted()
+    const updatedAt = moment().subtract('1', 'day').format('YYYY-MM-DD HH:mm:ss')
+
+    const user = await User.create({ username: 'virk' })
+    await User.query().where('username', 'virk').update({ updated_at: updatedAt })
+    await user.reload()
+
+    assert.equal(user.updated_at, updatedAt)
+  })
 })