Skip to content
This repository was archived by the owner on Jul 17, 2023. It is now read-only.

M3 refactor good practices #24

Merged
merged 16 commits into from
Oct 22, 2021
Merged

Conversation

AgustinMDominguez
Copy link
Collaborator

👁️

Copy link
Owner

@shirosweets shirosweets left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@AgustinMDominguez

  • LogOut no cierra sesión
  • Home del Shop no anda
  • Update de la ubicación actual crashea la app

@RicardoLopez9908
Copy link
Collaborator

Los test unitarios disminuyeron y no se realizan comprobaciones de contraseña con mínimo 8 caracteres.

@shirosweets
Copy link
Owner

shirosweets commented Oct 22, 2021

@AgustinMDominguez

  • Actualizar la branch con main,
  • crear los issues correspondientes a lo implementado, linkearlos a la PR y
  • mergear a main

Mencionarme cuando estos pasos estén hechos

@AgustinMDominguez
Copy link
Collaborator Author

Lineamientos

  • Deben existir al menos 5 test unitarios que corran en la JVM, sin dependencia del framework de Android. Si no existe una clase que pueda ser testeada, crearla.
  • Deben existir al menos 2 tests que involucren el uso del framework de android en un test unitario local (no instrumentado).
  • Debe de existir al menos 4 tests de integración instrumentados (funciones simples como interactuar con la UI y verificar que una acción esté funcionando correctamente).

Tips:

  • Escribe primero los tests antes que el código, esto asegurará que la calidad de la funcionalidad desarrollada sea alta y sabremos que el código está completo cuando pase el test.
  • Desarrolla primero los tests unitarios, después los de integración y finalmente los de end-to-end.

@shirosweets shirosweets mentioned this pull request Oct 22, 2021
@AgustinMDominguez
Copy link
Collaborator Author

@RicardoLopez9908 ahi agregué 2 test más. Podemos agregar más después pero con esto se cumple el postwork

@AgustinMDominguez
Copy link
Collaborator Author

@shirosweets @RicardoLopez9908

@RicardoLopez9908
Copy link
Collaborator

Genial! Están muy buenos los tests de consumo de Api

@AgustinMDominguez AgustinMDominguez merged commit 10d4eea into main Oct 22, 2021
@AgustinMDominguez AgustinMDominguez deleted the m3_refactor_good_practices branch October 22, 2021 16:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement improvement, refactor or request
Projects
None yet
3 participants