Altinn / altinn-access-management-frontend

Frontend for access management
MIT License
3 stars 1 forks source link

Single rights | Add data-testid #846

Open Albertlarsen opened 5 months ago

Albertlarsen commented 5 months ago

Description

For å gjøre det lettere å targete klikkbare komponenter i ende til ende testene våre så legger jeg til data-testid på alle komponenter som vi mener trenger det, for å gjøre det lettere å targete komponentene våre i ende-til-ende tester.

Under kan dere se komponentene jeg har kommer over som trenger det og foreslått data-testid til dem. Jeg tenker å gjøre det i 2 omganger hvor jeg prioriterer komponentene i Enkeltrettigheter først. Jeg tenker det er lurt å skille våre lokale komponenter i data-testid ved å legge til prefiks am (står for access management) slik at dersom vi tar i bruk en komponent fra et eksternt design system så går ikke data-testids i beina på hverandre.

De komponentene jeg først foreslår at jeg legger til data-testid på er i single rights. Under kan foreslått data-testid ses.

Legger til mer senere når jeg har tid men kom gjerne med flere forslag :blush:

image

Additional Information

No response