-
Notifications
You must be signed in to change notification settings - Fork 0
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
Frontend testen afwerken #167
Comments
Hier is de checklist van de testen die nog moeten gebeuren:
Als je aan een van de puntjes gaat beginnen, zeg dat dan in deze issue op voorhand zodat niemand anders er al ondertussen mee begint. Als het af is, zet er dan een vinkje. |
Ik ga eens kijken om testen de schrijven voor de subjectsPage. |
Ik heb de component testen geupdatet: nieuwe aangemaakt, verouderde verwijderd en eentje verplaatst naar de scoresPage. |
Ik heb gezorgd dat alle component testen de UI elementen opzoeken via een id, in plaats van de naam van het element. Dit is handiger aangezien er soms meerdere van hetzelfde element zijn en het dan lastig kan zijn om diegene te selecteren die je eigenlijk wilt. Vooral voor de e2e testen zal dit veel gemakkelijker zijn. |
Ok, dat is goed, ik ben momenteel wat tutorials aan het kijken over e2e testen omdat ik e nog nooit mee gewerkt heb. Maar vanavond nog zal ik al proberen om wat e2e testen te schrijven. |
Het is eigenlijk heel gelijkaardig aan de component testen, maar in plaats van 1 component te mounten, ga je op de site gaan werken. |
Ik heb trouwens net iets interessants gezien dat wel handig kan zijn voor ons. Je kan aan de jsx elementen een data-cy tag toevoegen. Dan kan je zeer gemakkelijk daarop zoeken in de tests. |
Dat is gelijkaardig als met id's werken vind ik. Cypress zelf zegt wel dat het beter is om met cy-data te werken, maar ik zie niet echt het verschil tussen cy-data of id, het moet sowieso op een pagina uniek zijn, want anders krijgt je terug meerdere objecten met cy.get |
Deze page testen zijn al in orde werd er daarnet op de vergadering gezegd, dus mss aanvullen :) |
No description provided.
The text was updated successfully, but these errors were encountered: