Es medio confuso que digan "transaccion2EnPepe", no se entiende qué pasa sino hasta ver más abajo en el archivo.
Es más, en el enunciado decíamos cosas como "transacción1" para no repetir tanto, pero sería mejor darle un nombre de qué pasa, como "transacción1LuchoCierraLaCuenta".
Si quieren separar los tests, ok, pero denle mejores nombres.
Tienen separadas cosas como "ejecutarTestNuevosEventos". Está perfecto si quieren hacer describes separados, que solo tengan los tests de esa sección. Pero abajo de todo hagan una única función que agrupe todos los describes con un solo hspec. ¿Me explico? Así con una función ejecutan todos los tests siempre.
También, mejoren los nombres. Por ejemplo, acá dicen 2 veces que la billetera inicial es de 10:
Depositar 10 monedas en una billetera de 10 monedas. Billetera inicial: 10, Billetera final: 20
Sería mejor "Al depositar 10 monedas en una billetera de 10, produce una billetera de 20" o algo similar. Revisen en todos los casos.
Es medio confuso que digan "transaccion2EnPepe", no se entiende qué pasa sino hasta ver más abajo en el archivo.
Es más, en el enunciado decíamos cosas como "transacción1" para no repetir tanto, pero sería mejor darle un nombre de qué pasa, como "transacción1LuchoCierraLaCuenta".
Si quieren separar los tests, ok, pero denle mejores nombres.
Tienen separadas cosas como "ejecutarTestNuevosEventos". Está perfecto si quieren hacer describes separados, que solo tengan los tests de esa sección. Pero abajo de todo hagan una única función que agrupe todos los describes con un solo hspec. ¿Me explico? Así con una función ejecutan todos los tests siempre.
También, mejoren los nombres. Por ejemplo, acá dicen 2 veces que la billetera inicial es de 10:
Sería mejor "Al depositar 10 monedas en una billetera de 10, produce una billetera de 20" o algo similar. Revisen en todos los casos.