MarinaVVsoftware / Novonautica-old

Back Office Web de Marina V&V
1 stars 0 forks source link

Combustible/Nuevo Related to M-002 #568

Closed EduardoHidalgo closed 5 years ago

EduardoHidalgo commented 5 years ago

Se cumple la generación de cotizaciones de combustible sin importar la cantidad de litros en inventario. Sin embargo, permite realizar la primera validación (hecha por un empleado de novonáutica). image

Al intentar validar como cliente (Segunda validación), falla la validación, pero no muestra ninguna alerta relacionada.

El candado se debe encontrar desde la primera validación, no la segunda, y debe mostrar una alerta para avisar al usuario que existe un candado contra existencias en inventario.

inrumi commented 5 years ago

image

@EduardoHidalgo Esto se reparo en el PR #564, la imagen es de hoy en beta y se realizo la validacion correctamente, puedes testearlo de nuevo?

EduardoHidalgo commented 5 years ago

1) eliminé las existencias negativas y añadí 20k litros. 2) luego se creó una cotización de combustible por 30k. 3) Luego se intentó validar, y se validó, generando en inventario -10k de existencias.

1 image 2 image 3 image

Nuevamente, pese a este error, la cotización se validó. Si intento seguir el flujo no se puede aceptar como cliente.

inrumi commented 5 years ago

@EduardoHidalgo Tienes razon, no estoy seguro que paso ahi, pero empezare a repararlo.

EduardoHidalgo commented 5 years ago

@EduardoHidalgo Tienes razon, no estoy seguro que paso ahi, pero empezare a repararlo.

@inrumi considera los siguientes flujos;

  1. inventarios en números negativos, crea la cotización, e intenta validar.
  2. inventarios en números negativos, crea la cotización, e intenta validar, luego modifica el inventario y vuelve a intentar validar (así es como me salió el error).
  3. inventarios en número positivos, crea la cotización con una cantidad mayor, e intenta validar.
inrumi commented 5 years ago

Okas, seguire esos flujos, gracias!

inrumi commented 5 years ago

@EduardoHidalgo ya estan los cambios en pruebas, tambien segui los flujos sugeridos.

EduardoHidalgo commented 5 years ago

@EduardoHidalgo ya estan los cambios en pruebas, tambien segui los flujos sugeridos.

ya lo testeé (: todo en orden. puedes cerrar el issue.