Merge pull request #27 from muke78/fixMessageRequestHttp

fix: Mejorar manejo de errores y mensajes en servicios de usuario

Author: muke78

src/controllers/users/functions/deleteUser.js

@@ -16,14 +16,8 @@ export const EliminarUsuario = async (req, res) => {
       message: `El usuario ${deletedUser.NameUser} fue eliminado correctamente`,
     });
   } catch (error) {
-    if (error.status === 400)
-      return methodIncorrect(req, res, "Faltan campos requeridos");
-    if (error.status === 404)
-      return methodNotFound(
-        req,
-        res,
-        "No se encontro el id del usuario que se quiere eliminar",
-      );
+    if (error.status === 400) return methodIncorrect(req, res, error.message);
+    if (error.status === 404) return methodNotFound(req, res, error.message);
     return methodError(req, res, { message: error });
   }
 };
@@ -36,8 +30,7 @@ export const DeleteUserBulk = async (req, res) => {
       message: `Se eliminaron ${req.body.ids.length} usuarios correctamente`,
     });
   } catch (error) {
-    if (error.status === 400)
-      return methodIncorrect(req, res, "Faltan campos requeridos");
+    if (error.status === 400) return methodIncorrect(req, res, error.message);
     return methodError(req, res, { message: error });
   }
 };

src/controllers/users/functions/editUser.js

@@ -21,15 +21,8 @@ export const EditarUsuario = async (req, res) => {
     }
   } catch (error) {
     if (error.status === 409)
-      return methodConflicts(req, res, {
-        message: "El correo ya existe y no se puede actualizar",
-      });
-    if (error.status === 404)
-      return methodNotFound(
-        req,
-        res,
-        "No se proporcionó un ID válido o el usuario no existe",
-      );
+      return methodConflicts(req, res, { message: error.message });
+    if (error.status === 404) return methodNotFound(req, res, error.message);
     return methodError(req, res, { message: error });
   }
 };

src/controllers/users/functions/insertUsers.js

@@ -14,12 +14,9 @@ export const InsertarUsario = async (req, res) => {
     const newuser = await insertUserService(req.body);
     return methodCreated(req, res, newuser);
   } catch (error) {
-    if (error.status === 400)
-      return methodIncorrect(req, res, "Faltan campos requeridos");
+    if (error.status === 400) return methodIncorrect(req, res, error.message);
     if (error.status === 409)
-      return methodConflicts(req, res, {
-        message: "El correo ya se encuentra registrado",
-      });
+      return methodConflicts(req, res, { message: error.message });
     return methodError(req, res, { message: error });
   }
 };
@@ -34,12 +31,9 @@ export const InsertarUsuariosRunnerMasive = async (req, res) => {
       `Se insertaron correctamente ${newUserMasive.length} usuarios como prueba`,
     );
   } catch (error) {
-    if (error.status === 400)
-      return methodIncorrect(req, res, "Faltan campos requeridos");
+    if (error.status === 400) return methodIncorrect(req, res, error.message);
     if (error.status === 409)
-      return methodConflicts(req, res, {
-        message: "El correo ya se encuentra registrado",
-      });
+      return methodConflicts(req, res, { message: error.message });
     return methodError(req, res, { message: error });
   }
 };

src/controllers/users/functions/login.js

@@ -13,20 +13,9 @@ export const Login = async (req, res) => {
 
     methodOK(req, res, token);
   } catch (error) {
-    if (error.status === 404)
-      return methodNotFound(req, res, "Usuario no encontrado");
-    if (error.status === 403)
-      return methodForbidden(
-        req,
-        res,
-        "El usuario está inactivo, pida la reactivación a un administrador",
-      );
-    if (error.status === 400)
-      return methodIncorrect(
-        req,
-        res,
-        "La contraseña es incorrecta o está mal escrita",
-      );
+    if (error.status === 404) return methodNotFound(req, res, error.message);
+    if (error.status === 403) return methodForbidden(req, res, error.message);
+    if (error.status === 400) return methodIncorrect(req, res, error.message);
 
     return methodError(req, res, { message: error });
   }

src/controllers/users/functions/registerUser.js

@@ -11,12 +11,9 @@ export const RegistrarUsuario = async (req, res) => {
     const newUser = await registerUserService(req.body);
     return methodCreated(req, res, newUser);
   } catch (error) {
-    if (error.status === 400)
-      return methodIncorrect(req, res, "Faltan campos requeridos");
+    if (error.status === 400) return methodIncorrect(req, res, error.message);
     if (error.status === 409)
-      return methodConflicts(req, res, {
-        message: "El correo ya se encuentra registrado",
-      });
+      return methodConflicts(req, res, { message: error.message });
     return methodError(req, res, { message: error });
   }
 };

src/controllers/users/functions/searchUser.js

@@ -11,12 +11,8 @@ export const BusquedaDeUsuarios = async (req, res) => {
 
     methodOK(req, res, resultSearch);
   } catch (error) {
-    if (error.status === 400)
-      return methodIncorrect(
-        req,
-        res,
-        `No se encontro el correo ${req.params.email}`,
-      );
+    console.log(error);
+    if (error.status === 400) return methodIncorrect(req, res, error.message);
     return methodError(req, res, { message: error });
   }
 };

src/services/users/functions/authService.js

@@ -8,20 +8,27 @@ export const loginService = async ({ email, password }) => {
   const user = await findUserByEmail(email);
 
   if (!user) {
-    throw { status: 404 };
+    throw { status: 404, message: "El usuario no ha podidio ser encontrado" };
+  }
+
+  if (user.AccountType === "google") {
+    throw { status: 400, message: "El correo ya esta registrado con google" };
   }
 
   const isPasswordValid = await hashedArg.verify(user.Password, password);
 
   if (!isPasswordValid) {
     throw {
       status: 400,
+      message: "La contraseña es incorrecta o está mal escrita",
     };
   }
 
   if (user.AccountStatus === "Inactivo") {
     throw {
       status: 403,
+      message:
+        "El usuario está inactivo, pida la reactivación a un administrador",
     };
   }
 

src/services/users/functions/deleteUserService.js

@@ -6,17 +6,20 @@ import {
 
 export const deleteUserService = async ({ id }) => {
   if (!id) {
-    throw { status: 400 };
+    throw { status: 400, message: "Faltan campos por completar" };
   }
 
   const foundUserToEliminated = await validateFoundUserToEliminated(id);
   if (foundUserToEliminated.length === 0) {
-    throw { status: 404 };
+    throw {
+      status: 404,
+      message: "No se encontro el id del usuario que se quiere eliminar",
+    };
   }
 
   const deleteUserFromID = await deleteUser(id);
   if (deleteUserFromID.affectedRows === 0) {
-    throw { status: 400 };
+    throw { status: 500 };
   }
 
   return foundUserToEliminated[0];
@@ -32,7 +35,7 @@ export const deleteUserBulkService = async ({ ids }) => {
 
   const MAX_IDS = 600;
   if (ids.length > MAX_IDS) {
-    throw { status: 400 };
+    throw { status: 400, message: "Faltan campos por completar" };
   }
 
   const batchSize = 100;

src/services/users/functions/insertUserService.js

@@ -13,12 +13,12 @@ export const insertUserService = async ({
   role,
 }) => {
   if (!nameUser || !email || !password || !accountStatus || !role) {
-    throw { status: 400 };
+    throw { status: 400, message: "Faltan campos por completar" };
   }
 
   const existingUser = await findUserByEmail(email);
   if (existingUser) {
-    throw { status: 409 };
+    throw { status: 409, message: "El correo ya se encuentra registrado" };
   }
 
   const hashedPassword = await hashedArg.hash(password);
@@ -40,7 +40,7 @@ export const insertUserService = async ({
 
 export const insertUserMasiveService = async ({ countInsert }) => {
   if (!countInsert || isNaN(countInsert)) {
-    throw { status: 400 };
+    throw { status: 400, message: "Faltan campos por completar" };
   }
 
   const insertados = [];
@@ -54,7 +54,7 @@ export const insertUserMasiveService = async ({ countInsert }) => {
 
     const existingUser = await findUserByEmail(email);
     if (existingUser) {
-      throw { status: 409 };
+      throw { status: 409, message: "El correo ya se encuentra registrado" };
     }
 
     const hashedPassword = await hashedArg.hash(password);

src/services/users/functions/registerUserService.js

@@ -6,13 +6,13 @@ import { registerUser } from "../../../models/users/functions/registerUserModel.
 
 export const registerUserService = async ({ nameUser, email, password }) => {
   if (!nameUser || !email || !password) {
-    throw { status: 400 };
+    throw { status: 400, message: "Faltan campos por completar" };
   }
 
   const existingUser = await findUserByEmail(email);
 
   if (existingUser) {
-    throw { status: 409 };
+    throw { status: 409, message: "El correo ya se encuentra registrado" };
   }
 
   const hashedPassword = await hashedArg.hash(password);