Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor ecs test #33

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Refactor ecs test #33

wants to merge 6 commits into from

Conversation

smmo1
Copy link

@smmo1 smmo1 commented Mar 31, 2019

Refactor de MTax, XTax + tests de Mtax/validateTax.

  • Se eliminaron los comentarios del codigo, para una lectura mas clara.
  • Se homologa el idioma de las propiedades de la clase XTax.
  • Se modifica el nombre del metodo validate de la clase MTax a validateTax, para acotarlo a su proposito.
  • Se crea el metodo setTaxsValidated, para ser consumido solo si se cumplen las condiciones apropiadas
  • Se cambia la variable del contador (cont) por una bandera de tipo boleano flagAllLocalTax, con la finalidad de solo ser consumida en caso de que se cumpla y simplificar la condicion.
  • Se agrega el mensaje de error en caso de que que no se cumpla el escenario si el id es nulo.
  • Se agrega la condición para acceder al metodo solo si cumple la condicion si cuenta con almenos una tasa local, (creo que tambien debo inlcuir, si todos tienen id, y si todos tienen valor tax).
  • Se modifica el metodo setTaxsValidated eliminando la validacion de (xt.size() != validIds.size(), ya que me parece redundante debido al resultado obtenido es apartir de la lista previamente validada y si los datos no han sido guardados previamente no es este el fragmento donde deberia de aplicar dicha validacion.
  • Se modifica el hashMap para trabajar mediante llave valor la fecha obtenida a partir de la consulta del metodo TaxsByListId
  • Se simplifica el ciclo for(int i = 0; i < xTaxList.size(); i++)
  • Se crea la clase para pruebas unitarias a partir de la libreria JUnit4, para probar que los escenarios contemplados esten efectuando la validacion esperada.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants