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

Implementacion de endpoint que filtra explorers por stack #177

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

Conversation

mariel-rs
Copy link

Para la implementacion de esta nueva funcionalidad se hizo lo siguiente:

  1. Instalacion de dependencias,
    1.1 Ligero cambio en la ruta del script de test para Windows en package.json
    1.2 Instalacion de jest version 26 para que puedan ejecutarse las pruebas unitarias en github actions

  2. Se agrego un metodo estatico filterByStack en la class ExplorerService, que recibe dos parametros (explorers, stack). Este metodo busca que un stack determinado esta en la propiedad stacks del array explorers.

  3. Se agrego un metodo estatico getExplorerByStack en la clase ExplorerController, el cual recibe el parametro stack. Este metodo utiliza el metodo ReadJsonFile previamente implementado y el metodo filterByStack de ExplorerService.

  4. Se actualizo el server, con un nuevo endpoint con la url "/v1/explorers/stack/:stack", el cual toma el parametro dinamico stack via query params. Finalmente el endpoint despliega la lista de explorers que cumplan la condicion

  5. Se crearon pruebas unitarias para los dos metodos implementados en ExplorerController y ExplorerService.

  6. Se ejecuto linter para mantener estilo en codigo

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.

1 participant