Plataforma de desarrolladores/programadores junior

#1505 empiezo yo, ojo con la mezcla de inglés y español, preferiblemente todo en inglés.

Y no veo los tests por ningún lado, se te ha olvidado pushearlos no? No?

#1506 preferiblemente todo en inglés, fixed

#1506 Tests? eso se come?

Fuera coñas, todo el backend estaba con tests unitarios y de integracion. El problema vino cuando pase de gestionar yo la creacion de usuario y gestion de tokens a usar auth0, ahi no conseguia recuperar el token ya que se generan desde el front, por lo que tenia que hacer e2e (o eso creo)

En la parte del front hice algunos pero no le he dado suficiente caña a cypress por lo que al meter auth0 volvemos al mismo problema, asi que los deje de lado.

Como ya puse por arriba, se que tengo que refactorizar los nombres y dejarlos todos en ingles con esa mezcla de espanglish que hay.

En tema organizacion de codigo, que tal lo veis? Algun fallo gordo a arreglar en lo que es el codigo o esta medio decente?

#1513 A que te refieres con lo de codigo "mio"?

Lo de validar la request, en su momento lo tenia hecho pero al meter el validateToken para que haga los checks de que el token llega desde auth (lo comprueba con su propia libreria), se me jodia todo. Por lo que la unica opcion era o seguir igual y confiar en que el token llegaba directamente desde auth0 sin usar la libreria, o usarlo pero ya me perdia con los tests =/

Pero si, ahora que lo pones, es mas facil desacoplarlo, y testear simplemente el metodo que trata con la bbdd y pista. Aun asi, no podria hacer realmente un e2e sin saltarme validateToken no?

Edit, no tenia puesto el link en el repo para probarla, si alguien quiere echarle un ojo aqui os lo dejo

#1515 Thx, me lo apunto como tarea, que como digo, en tema picar codigo ando oxidado, pero en tema testing ando muy muy perdido

A que te refieres con lo de codigo "mio"?

Tampoco miré mucho pero me dio la sensación de que casi todo era pillar la request http y enchufársela a mongo; eso se suele llamar "glue code" porque básicamente tiras cuatro líneas de código par pegar componentes (lo que te da el framework http con lo que te da la lib de persistencia de mongodb).

Esto que te digo para desacoplar sería para los unitarios más que e2e. E2E y unitarios ambos testean "componentes", pero en el caso de los unitarios tu "componente" es una función o un método de una clase, en el caso de E2E tu componente es una ruta http.

El token cuando llega al controler debería estar bien formado siempre. En el middleware, en vez de

  if (!authorization.toLowerCase().startsWith('bearer ')) {
    console.log('nope')
  }

Tendrías que devolver un 401. Y después de eso comprobar también que el token es correcto y está bien firmado, y si no, otro 401 de vuelta.

Luego en el controller puedes llamar a otra función que sea, por ejemplo buildUser(token) y el resultado de eso sería lo que le pasas a tu "servcio de domino". Así también te quitarías de inyectar datos en la request dentro del middleware, que es algo bastante feo con bastantes peligros.

#1518 si, tampoco hago demasiado en cada endpoint más allá de llamadas a la BBDD y manejar/transformar lo que me devuelve. No deja de ser una lista de tareas un poco glorificada.

El comprobar que esté bien firmado lo hace la librería de auth0 en el primer middleware, en el segundo es verdad que me queda mucha mierda por quitar, que es donde hacia las comprobaciones cuando era yo el que firmaba y creaba el token.

Tendré en cuenta lo de modificar la request y tengo que repasar bien el back, que al final sí que es verdad que lo metí todo a mogollón y me despreocupe una vez que vi que funcionaba.

#1519 por algo en concreto o por ser solo llamadas a mongo?

#1523 Tambien lo pense y asi usar docker que nunca lo he mirado, pero realmente vale la pena ternerlo en contenedores? La app es simplemente la build de react, y la api, supongo que seria crear el contenedor y listo, ya que ahora mismo lo gestiono con pm2