add updated_by to user and metadata table

Author: sshugsc

app/middleware/acl.js

@@ -117,13 +117,16 @@ module.exports = async (req, res, next) => {
   try {
     // Update last time the user logged in, limit to once a day
     const currentDate = new Date().toDateString();
-    let userMetadata = await db.models.userMetadata.findOrCreate({where: {userId: req.user.id}});
+    let userMetadata = await db.models.userMetadata.findOrCreate({
+      where: {userId: req.user.id},
+      defaults: {updatedBy: req.user.id},
+    });
     userMetadata = userMetadata[0];
     const userLastLogin = userMetadata.lastLoginAt
       ? new Date(userMetadata.lastLoginAt).toDateString()
       : '';
     if (userLastLogin !== currentDate) {
-      await userMetadata.update({lastLoginAt: new Date()});
+      await userMetadata.update({lastLoginAt: new Date(), updatedBy: req.user.id});
     }
 
     try {

app/routes/user/settings.js

@@ -21,11 +21,14 @@ router.route('/')
   })
   .put(async (req, res) => {
     try {
-      await db.models.userMetadata.update({settings: req.body}, {
+      await db.models.userMetadata.update({
+        settings: req.body,
+        updatedBy: req.user.id,
+      }, {
         where: {
           user_id: req.user.id,
         },
-        fields: ['settings'],
+        fields: ['settings', 'updatedBy'],
         hooks: false,
         limit: 1,
       });

app/routes/user/user.js

@@ -260,9 +260,15 @@ router.route('/')
       // Create transaction
       transaction = await db.transaction();
       // Create user
-      const createdUser = await db.models.user.create(req.body, {transaction});
+      const createdUser = await db.models.user.create({
+        ...req.body,
+        updatedBy: req.user.id,
+      }, {transaction});
       // Create user metadata
-      await db.models.userMetadata.create({userId: createdUser.id}, {transaction});
+      await db.models.userMetadata.create({
+        userId: createdUser.id,
+        updatedBy: req.user.id,
+      }, {transaction});
       // Commit changes
       await transaction.commit();
       // Return new user

test/routes/user/settings.test.js

@@ -68,6 +68,7 @@ describe('/user/settings', () => {
         where: {userId: testUser.id},
       });
       expect(results.length).toBe(1);
+      expect(results[0].updatedBy).toBe(testUser.id);
     });
   });
 });