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.

Sign up for GitHub

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

Jump to bottom

Implementar pruebas de sistema #196

Closed
aliciafp15 opened this issue Apr 14, 2023 · 0 comments
Closed

Implementar pruebas de sistema #196

aliciafp15 opened this issue Apr 14, 2023 · 0 comments
Assignees
@aliciafp15 @uo283840 @manlopdev
Labels
Despliegue 🚀 Integración continua ♾️ Pruebas 🧪

Comments

@aliciafp15
Copy link
Contributor

No description provided.

aliciafp15 added a commit that referenced this issue Apr 24, 2023
@aliciafp15
Primeros pasos hacia delante con #196
0a909b0 
* Hace una semana no conseguiamos ejecutar ningún test porque teniamos dos
fallos al renderizar y la estructura de los test mal montada.
En este commit se ha conseguido que pase como válido el test de prueba
AddPlaceSidebar.test.js, muy simple pero funciona.

* OJO: En la clase AddPlaceSidebar salta un error "There are multiple `InputBase` components inside a FormControl"
* Ahora se ejecutan los test, pero no se renderiza el mapa al ejecutar la aplicación.
El resto de la página va, pero el mapa está en blanco. Estamos buscando el porqué.
aliciafp15 added a commit that referenced this issue Apr 25, 2023
@aliciafp15
Solucionado bug #196
f83e300 
En el commit anterior expliqué que se había logrado arrancar el primer test
válido pero el mapa se habia estropeado por el camino.

SOLUCIÓN: En la clase Map.jsx se ha eliminado un contenedor <div> que rodeaba
al MapContainer.

De cara a redactar las pruebas se seguirá el proceso igual que en AddPlaceSidebarTest.
Se arrancan con "npm run test" y después dar a la opción 'a' para que muestre todos.
@aliciafp15 aliciafp15 mentioned this issue Apr 25, 2023
Merged
aliciafp15 added a commit that referenced this issue Apr 27, 2023
@aliciafp15
avance con #196 y cambio en yml
60ce74f
@manlopdev manlopdev mentioned this issue May 5, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@aliciafp15 aliciafp15

@uo283840 uo283840

@manlopdev manlopdev

Labels
Despliegue 🚀 Integración continua ♾️ Pruebas 🧪
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@aliciafp15 @uo283840 @manlopdev