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

FizzBuzz Contribution #169

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

Conversation

LuceroLuciano
Copy link

Contribución Open Source

El siguiente código es un ejemplo de como se puede contribuir al código abierto.

  1. Como primer punto se realizo un fork al respositorio, esto crea una copia del código en nuestro repo.

  2. Abrir el repositorio, leer y entender las instrucciones. En este caso se requiere crear un nuevo endpoint que muestre la lista de explorers filtrados por stacks.

Solución

El proceso que se llevo a cabo para crear el nuevo endpoint es el siguiente:

  1. Crear un nuevo método static llamado getListExplorersByStack que recibirá dos parámetros y filtrara la lista de explores utilizando el metodo filter e include para que se muestre los valores del stack. Este método se creara en el archivo ExplorerService.js

Imagen1

  1. Crear la prueba de unidad en el archivo ExplorerService.test.js para verificar que funciona correctamente.
  • Para ejecutar la prueba corre el siguiente comando: npm test test/services/ExplorerService.test.js
    En este paso ya deberías tener instalado jest

Imagen2

  1. Crear un método static en el archivo ExplorerController.js que contiene el método que manda a llamar al método getListExplorersByStack de ExplorerService.js

Imagen3

  1. Finalmente se crea el nuevo endpoint en el archivo server.js
    Imagen4

  2. Ejecutar ESLInt como guía de diseño para corregir los errores de identación, comillas, punto y coma, entre otros.

  • npm run linter para ver los errores
  • npm linter --fix para corregir los errores
  1. Quitar del control de versiones el package-lock.json
  • Primero crear un archivo llamado .gitignore y agregar el **/pakcage-lock.json dentro del archivo .gitignore
  • Ejecutar el siguiente comando git rm --cache package-look.json
  • Aguarda los cambios y haz commit y push

Enpoint en funcionamiento

Lista de explorers filtrada por stacks

En el navegador:

imagen7

Con Postman:
Imagen8

Imagen9

@LuceroLuciano
Copy link
Author

Contribución Open Source

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