Así es como se ve mi código:
userUpdate(req: Request, res: Response) { this.userTaskObj.userUpdate(req.params.id, req.body).then(() => { res.status(200).json({ status: 'OK', message: 'User updated', }); }) .catch((err) => ErrorHandler.getErrorMessage('User', res, err)); } (below function is inside UserTask class) userUpdate(id: string, data: any) { let query: Promise<any>; if (data.provides) { query = User.findByIdAndUpdate(id, {$addToSet: {provides: data.provides}}).exec(); } else { query = User.findByIdAndUpdate(id, data).exec(); } return new Promise((resolve, reject) => { resolve(query); // query.then(res => resolve(res)) // .catch(err => reject(err)); }); }Pensé que no sería capaz de detectar errores de la consulta usando resolver solo. Pero detecta errores de mangosta. Cuando lo reemplacé con la parte comentada, eso también funciona, pero los errores quedan atrapados dentro del catch() que se rechaza. ¿Cómo detecta errores el código sin catch() ?
.catch() se ejecuta cuando se rechaza la promesa. Un error da como resultado el rechazo de la promesa. Como consecuencia:
(new Promise(()=>{throw Error()}).then(()=>{ console.log("this will not display because the promise was rejected beforehand") })) .catch(()=>{ console.log("this will run"); }) En su caso, cuando se produce el error de mangoose, this.userTaskObj.userUpdate(req.params.id, req.body) ya se ha devuelto y el objeto devuelto es una instancia de promesa.
Esta instancia de promesa no se resuelve de acuerdo con el error de la consulta, por lo que finalmente se rechaza. De hecho, Promise.resolve(Promise.reject()) da como resultado una promesa rechazada [Nota: como comentaron otros, esto es antipatrón].
Por lo tanto, .catch() se hace cargo.