Prueba lo siguiente:
function updateNota(req, res){
let notaId = req.params.notaId
let update = req.body
console.log(update)
Nota.findByIdAndUpdate(notaId, update, (err, notaUpdated) => {
if(err) return res.status(500).send({message: `Error al actualizar la nota: ${err}`})
if(!notaUpdated) return res.status(500).send({message: 'No retornó objeto actualizado'})
res.status(200).send({ nota: notaUpdated })
})
}
Nota que agregué return en los if's, y que el if que agregué valida que te retorne el objeto actualizado. Si no te retorna el objeto, entonces no se está actualizando. Por cierto, que según tengas definido tu modelo, al hacer update a un documento, te retornará el documento antes de actualizar o el documento ya actualizado. La idea es que te retorne un objeto, si no te lo retorna es que no se hizo la operación.
Pero si sí te lo retorna, significa que sí se actualiza. Entra a la consola de mongo y haz:
use 'mi-db'
db.micoleccion.find().pretty()
para ver si te está modificando. Sólo reemplazas los valores de acuerdo a tu DB y tu colección. Esto es para no tener que instalar un programa tipo robomongo o mongobooster para consultar tu DB.
Como te decía, si no te marca ningún error, es que actualiza el doc con los mismos valores, puedes colocar esto al inicio de tu función:
if (!req.body.campo1) return res.status(500).send({ message: 'No se recibió valor del campo1' })
Así validas si estás recibiendo los valores a actualizar.
Nota1: colocar return en los if's de errores es para terminar de ejecutar la función, sino te continuará ejecutando el resto del código y dará error de "intentar enviar las cabeceras de retorno cuando ya han sido enviadas" (no puedes hacer más de un res.status... )
Nota2: los códigos http de retorno, por buenas prácticas, deben ser acorde a la causa de los mismos. El código 500 es "Error desconocido" y eso no da información suficiente. Consulta aquí para más.