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

Cypress #20

Open
wants to merge 79 commits into
base: main
Choose a base branch
from
Open

Cypress #20

wants to merge 79 commits into from

Conversation

vicweber
Copy link

No description provided.

@andersrebner
Copy link
Collaborator

@vicweber Hva er status på denne? Er det skrevet noe dokumentasjon? Vi ønsker å få det deployet.

@vicweber
Copy link
Author

@vicweber Hva er status på denne? Er det skrevet noe dokumentasjon? Vi ønsker å få det deployet.

Hei! Det må sjekkes og evt oppdateres. Har ikke sett på det siden september

@andersrebner
Copy link
Collaborator

andersrebner commented Feb 20, 2023

Vi ble enige om at dette kan videreutvikles av de som jobber på Kompetansekartlegging mellom prosjekt. Start gjerne fra scratch i ny branch.

Endringsforslag ref #57:

Gjøre om testene til bedre praksis. Her og nå er mange av testene bassert på feks at en knapp har tekst i seg med innhold slik som "Sign In". Dette er ikke robust, da testene feiler hvis man gjør endringer i frontenden slik som å forandre på teksten. I stedet burde testene være bassert på HTML-attributter som er lagt til spesifik for testing, slik som "data-cv=sign-in-button". (se https://docs.cypress.io/guides/references/best-practices)

@andersrebner
Copy link
Collaborator

andersrebner commented Apr 11, 2023

Etter migrering til Vite er det muligens problemer med Cypress, ref: #28 (comment)

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.

2 participants