You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Lage test i nais/api som fanger opp at nais-verification fungerer
Beskrivelse / Essensen
Featuren nais-verification bruker nais-api til å opprette nais-verification teamet.
I praksis gjøres dette kun ved opprettelse av en ny tenant, så det er fort gjort å glemme seg når man endrer API'et og ikke legge merke til at nais-verification har brukket før vi evt. oppretter en ny tenant.
Investeringsvilje
1 uke
Ikke-mål
Mulig løsning i grove trekk
Trekke ut de API-kallene nais-verification gjør, og lage tester i nais-api som kjører de kallene og sjekker forventet oppførsel.
Gjerne med litt dokumentasjon om hvorfor disse testene ligger der, slik at man vet hva man skal gjøre hvis de brekker.
For bonus-poeng kunne man kanskje gjort det slik at nais-verification dytter oppdaterte queries inn i nais-api repoet ved behov.
Eventuell annen relevant informasjon
The text was updated successfully, but these errors were encountered:
Lage test i nais/api som fanger opp at nais-verification fungerer
Beskrivelse / Essensen
Featuren nais-verification bruker nais-api til å opprette nais-verification teamet.
I praksis gjøres dette kun ved opprettelse av en ny tenant, så det er fort gjort å glemme seg når man endrer API'et og ikke legge merke til at nais-verification har brukket før vi evt. oppretter en ny tenant.
Investeringsvilje
1 uke
Ikke-mål
Mulig løsning i grove trekk
Trekke ut de API-kallene nais-verification gjør, og lage tester i nais-api som kjører de kallene og sjekker forventet oppførsel.
Gjerne med litt dokumentasjon om hvorfor disse testene ligger der, slik at man vet hva man skal gjøre hvis de brekker.
For bonus-poeng kunne man kanskje gjort det slik at nais-verification dytter oppdaterte queries inn i nais-api repoet ved behov.
Eventuell annen relevant informasjon
The text was updated successfully, but these errors were encountered: