feat: remove user from achievement

Author: nalves599

server/resources/achievement.js

@@ -15,6 +15,7 @@ server.method('achievement.getByUser', getByUser, {})
 server.method('achievement.removeAllFromUser', removeAllFromUser, {})
 server.method('achievement.list', list, {})
 server.method('achievement.remove', remove, {})
+server.method('achievement.removeUser', removeUser, {})
 server.method('achievement.addUser', addUser, {})
 server.method('achievement.addMultiUsers', addMultiUsers, {})
 server.method('achievement.addMultiUsersBySession', addMultiUsersBySession, {})
@@ -137,6 +138,10 @@ async function remove(id) {
   return Achievement.findOneAndRemove({ id: id })
 }
 
+async function removeUser(id, userId) {
+  return Achievement.updateOne({ id: id }, { $pull: { users: userId }})
+}
+
 // 500, 404
 async function addCV(userId) {
   const achievementKind = 'cv'

server/routes/achievement/handlers.js

@@ -184,6 +184,36 @@ exports.removeMe = {
   }
 }
 
+exports.removeUser = {
+  options: {
+    tags: ['api', 'achievement'],
+    auth: {
+      strategies: ['default'],
+      scope: ['team', 'admin']
+    },
+    description: 'Removes user from achievement',
+    validate: {
+      params: Joi.object({
+        id: Joi.string().required().description('Id of the achievement'),
+        userId: Joi.string().required().description('Id of the user')
+      })
+    },
+  },
+  handler: async (request, h) => {
+    try {
+      let ach = await request.server.methods.achievement.removeUser(request.params.id, request.params.userId)
+      if (!ach) {
+        log.error({ err: 'not found', id: request.params.id }, 'achievement not found')
+        throw Boom.notFound('achievement not found')
+      }
+      return h.response(render(ach))
+    } catch (err) {
+      log.error({ err: err, id: request }, 'error removing user from achievement')
+      throw Boom.internal('error getting achievement')
+    }
+  }
+}
+
 exports.getActive = {
   options: {
     tags: ['api', 'achievement'],

server/routes/achievement/index.js

@@ -79,6 +79,13 @@ server.route({
   handler: handlers.remove.handler
 })
 
+server.route({
+  method: 'DELETE',
+  path: '/achievements/{id}/users/{userId}',
+  options: handlers.removeUser.options,
+  handler: handlers.removeUser.handler
+})
+
 server.route({
   method: 'GET',
   path: '/achievements/code',