Merge pull request #348 from ASHWINIAJIT/user-status

Added status property in user object

Author: swarajpure

constants/users.js

@@ -0,0 +1,2 @@
+const userStatusEnum = ['ooo', 'idle', 'active']
+module.exports = userStatusEnum

docs/swaggerDefinition.js

@@ -445,6 +445,9 @@ const swaggerOptions = {
             },
             tokens: {
               type: 'object'
+            },
+            status: {
+              type: 'string'
             }
           }
         },

middlewares/validators/user.js

@@ -1,4 +1,5 @@
 const joi = require('joi')
+const userStatusEnum = require('../../constants/users')
 
 const updateUser = async (req, res, next) => {
   const schema = joi.object().keys({
@@ -14,7 +15,8 @@ const updateUser = async (req, res, next) => {
     linkedin_id: joi.string().optional(),
     twitter_id: joi.string().optional(),
     instagram_id: joi.string().optional(),
-    website: joi.string().optional()
+    website: joi.string().optional(),
+    status: joi.any().valid(...userStatusEnum).optional()
   })
 
   try {

test/fixtures/user/user.js

@@ -23,7 +23,8 @@ module.exports = () => {
       email: '[email protected]',
       tokens: {
         githubAccessToken: 'githubAccessToken'
-      }
+      },
+      status: 'active'
     },
     {
       username: 'nikhil',

test/integration/users.test.js

@@ -44,6 +44,46 @@ describe('Users', function () {
           return done()
         })
     })
+
+    it('Should update the user status', function (done) {
+      chai
+        .request(app)
+        .patch('/users/self')
+        .set('cookie', `${cookieName}=${jwt}`)
+        .send({
+          status: 'ooo'
+        })
+        .end((err, res) => {
+          if (err) { return done(err) }
+
+          expect(res).to.have.status(204)
+
+          return done()
+        })
+    })
+
+    it('Should return 400 for invalid status value', function (done) {
+      chai
+        .request(app)
+        .patch('/users/self')
+        .set('cookie', `${cookieName}=${jwt}`)
+        .send({
+          status: 'blah'
+        })
+        .end((err, res) => {
+          if (err) { return done(err) }
+
+          expect(res).to.have.status(400)
+          expect(res.body).to.be.an('object')
+          expect(res.body).to.eql({
+            statusCode: 400,
+            error: 'Bad Request',
+            message: '"status" must be one of [ooo, idle, active]'
+          })
+
+          return done()
+        })
+    })
   })
 
   describe('GET /users', function () {