PROBLEMA
Tal como digo en mi comentario, el método (lo que llamas controlador) para crear un usario tiene un problema:
const createUser = async (data) => {
const newUser = await new User.insertOne({ // <- AQUÍ!
firstName: data.firstName,
lastName: data.lastName,
email: data.email,
password: data.password,
country: data.country,
});
return newUser;
};
El problema es que estás encadenando el método insertOne
al constructor del Modelo (new
). Lo cual es incorrecto.
SOLUCIÓN
Tal como se desprende de la documentación, para insertar un nuevo documento en tu colección tienes 3 formas de hacerlo a partir de un modelo de Mongoose.
Usando el constructor new
Si vas a usar el constructor new
, debe saber que el resultado obtenido no es almacenado directamentamente en la base de datos, simplemente estás creando una instancia del objeto Dcoument
de Mongoose.
Con esta instancia puedes modificar, validar y otras cosas antes de almacenar el documento en la base de datos. Sin embargo, puedes simplemente almacenarlo usando el método save()
que es parte de toda instancia de un Documento. Por ejemplo:
const createUser = async (data) => {
const newUser = await new User({
firstName: data.firstName,
lastName: data.lastName,
email: data.email,
password: data.password,
country: data.country,
});
await newUser.save();
return newUser;
};
Ahora bien, podría darse el caso de error al guardar un documento en la base de datos, es por ello que cada vez que uses async / await
, siempre es buena idea usar try ... catch
para capaturar cualquier excepción durante el proceso.
Por ejemplo:
const createUser = async (data) => {
try {
const newUser = await new User({
firstName: data.firstName,
lastName: data.lastName,
email: data.email,
password: data.password,
country: data.country,
});
await newUser.save();
return newUser;
}
catch(e) {
console.error(e.message);
return null;
}
};
Así, si ocurre un error podremos manejarlo, mostrar el mensaje y devolver null
que podría indicar que el usuario no fue creado.
Usando insertOne
Si deseas usar insertOne
, debes tomar en cuenta tres cosas:
- El método
insertOne
es un método del objeto Collection de MongoDB y no del Modelo, o del Documento de Mongoose (son cosas diferentes). Es por ello que new Model().insertOne()
no es correcto.
- El método
insertOne
no realiza las validaciones sobre el documento a insertar, ya que no forma parte del Modelo, sino de la colección, y como las colecciones en MongoDB son schemaless, no hay validaciones sobre las mismas, excepto las de tipo índice.
- El método que realmente debes usar es
insertMany
, que recibe una lista de documentos para insertar a la base de datos, validando cada uno de acuerdo a las validaciones del Modelo.
Por ejemplo:
const createUser = async (data) => {
try {
const newUser = await User.insertMany([
{
firstName: data.firstName,
lastName: data.lastName,
email: data.email,
password: data.password,
country: data.country,
}
]);
if(!newUser.length) {
throw new Error('Documento no pudo ser crewado);
}
return newUser[0]; // <- Como hemos creado 1 sólo documento, se usa [0]
}
catch(e) {
console.error(e.message);
return null;
}
};
Usando create
Otra forma de insertar un documento en la base de datos es usando el método create
del Modelo. Por ejemplo:
const createUser = async (data) => {
try {
const newUser = await User.create({
firstName: data.firstName,
lastName: data.lastName,
email: data.email,
password: data.password,
country: data.country,
});
return newUser;
}
catch(e) {
console.error(e.message);
return null;
}
};
Al igual que insertOne
, el método create
realiza la operación en la base de datos, y devuelve una Promesa.
Puedes leer la diferencia entre Modelo y Documento
Espero que esto te ayude a resolver tu problema.
await new User.insertOne({ ... });
. Estás mezclando el constructor del modelo (new
) con el métodoinsertOne
. En tu caso debería ser solo:await new User({ ... });
, luego debes usarawait newUser.save()
. Prueba eso y comenta el resultado. También puedes leer la documentación. Saludos