search

jorgejcabrera / rent-app-service

0 stars 0 forks source link

si recibis por parametro el localdatetime, en vez de instanciarlo acá adentro, es mas facil para testear. #18

Open maurociancio opened 1 year ago

maurociancio commented 1 year ago

https://github.com/jorgejcabrera/rent-app-service/blob/08ed89e9d52c172bbe9f4138b2b4a669ffde29eb/src/main/groovy/com/fiuba/rent_app/domain/item/Item.groovy#L105 es un detalle

jorgejcabrera commented 1 year ago

El problema es que si lo recibo por parámetro es una posible entrada a inconsistencias ya que la fecha de chequeo podría ser una distinta a la actual, o la fecha en la que se está realziando el chequeo para saber si una orden expiró. De todas formas es fácil testear esto, ese test lo podés ver acá https://github.com/jorgejcabrera/rent-app-service/blob/master/src/test/groovy/com/fiuba/rent_app/domain/item/ItemTest.groovy#L16. Lo que hago es generar una orden vieja, siempre previa a la fecha actual.