Skip to content

add ad-hoc mock por usePArams in GetCheck test#3

Open
merunga wants to merge 1 commit into
shari12:masterfrom
merunga:master
Open

add ad-hoc mock por usePArams in GetCheck test#3
merunga wants to merge 1 commit into
shari12:masterfrom
merunga:master

Conversation

@merunga

@merunga merunga commented Jul 27, 2020

Copy link
Copy Markdown

@shari12 @danleonca no funcionaba porque estaba mockeando mal useParams (retorna un objecto, con la propieda id, yo estaba retornando un string)

Ahora hice lo mismo q intente en la maniana, pero con el mock localmente en el test, para q aplique solamente para este componente.

Fijense q cuando corren el test, ya no falla, pero pasan 2 cosas:

  • hay un promise q falla no lo estan controlando (creo q es en showInfoTables -hay un await sin try/catch y creo q es porque estan intentando de usar firebase sin iniciarlo). Deberian o bien mockear firebase o mockear showInfoTables

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