Working

Signed-off-by: Arthur DANJOU <arthurdanjou@outlook.fr>

app/Controllers/Http/UsersController.ts

@@ -5,34 +5,33 @@ import UserUpdateValidator from "App/Validators/users/UserUpdateValidator";
 
 export default class UsersController {
 
-  public async index () {
-    return User.query()
+  public async index({response}: HttpContextContract) {
+    return response.status(200).send({
+      users: User.query()
+    })
   }
 
-  public async store ({request}: HttpContextContract) {
+  public async store({request, response}: HttpContextContract) {
     const data = await request.validate(UserStoreValidator)
-    return await User.create(data)
+    return response.status(200).send({
+      user: await User.create(data)
+    })
   }
 
-  public async show ({params}: HttpContextContract) {
-    return await User.findOrFail(params.id)
+  public async show({params, response}: HttpContextContract) {
+    return response.status(200).send({
+      user: await User.findOrFail(params.id)
+    })
   }
 
-  public async update({ request, params, response }: HttpContextContract) {
+  public async update({request, params, response}: HttpContextContract) {
     const user = await User.findOrFail(params.id)
     const data = await request.validate(UserUpdateValidator)
-    const { email } = data
-    const user2 = await User.findBy('email', email)
-
-    if (user2 !== null && user.email !== email) {
-      return response.abort({
-        message: 'L\' adresse mail n\'est pas unique !'
-      })
-    }
-
     await user.merge(data).save()
 
-    return { message: 'Le compte a été mis à jour' }
+    return response.status(200).send({
+      message: 'User successfully updated!'
+    })
   }
 
   public async destroy({ response, params, auth }: HttpContextContract) {
@@ -44,7 +43,9 @@ export default class UsersController {
     }
 
     await user.delete()
-    return { message: "L'utilisateur a bien été supprimé" }
+    return response.status(200).send({
+      message: 'User successfully deleted!'
+    })
   }
 
 }