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

Oppdater sanityflagg for å spisse test #4660

Closed

Conversation

kristeine
Copy link
Contributor

💰 Hva skal gjøres, og hvorfor?

Favro: https://favro.com/organization/98c34fb974ce445eac854de0/1844bbac3b6605eacc8f5543?card=NAV-21583
Oppfølging av #4651

Oppdaterer sanityflagg for endret utb-begrunnelse. Tidligere har flagget vært satt til UTBETALING_IKKE_RELEVANT, og da passerer også testene for begrunnelsen. Men ved endring til SKAL_UTBETALES vil testene feile. Endrer flagget i sanity + i ressursfilen til SKAL_IKKE_UTBETALES, som er mest korrekt og spisst og forhåpentligvis forebygger at noen endrer flagget til feil verdi. Dersom det skulle skje igjen vil vi ha git-historikk på hva korrekt flagg er for å få funksjonaliteten vi forventer

🔎️ Er det noe spesielt du ønsker tilbakemelding om?

Er det noe du er usikker på eller ønsker å diskutere? Beskriv det gjerne her eller kommenter koden det gjelder.

✅ Checklist

Har du husket alle punktene i listen?

  • Jeg har testet mine endringer i henhold til akseptansekriteriene 🕵️
  • Jeg har config- eller sql-endringer. I så fall, husk manuell deploy til miljø for å verifisere endringene.
  • Jeg har skrevet tester. Hvis du ikke har skrevet tester, beskriv hvorfor under 👇

Jeg har ikke skrevet tester fordi:
Dekket av eksisterende begrunnelsestester

💬 Ønsker du en muntlig gjennomgang?

  • Ja
  • Nei

Oppdater sanityflagg for endret utb-begrunnelse. Tidligere har flagget vært satt til `UTBETALING_IKKE_RELEVANT`, og da passerer også testene for begrunnelsen. Men ved endring til `SKAL_UTBETALES` vil testene feile. Endrer flagget i sanity + i ressursfilen til `SKAL_IKKE_UTBETALES`, som er mest korrekt og spisst og forhåpentligvis forebygger at noen endrer flagget til feil verdi. Dersom det skulle skje igjen vil vi ha git-historikk på hva korrekt flagg er for å få funksjonaliteten vi forventer
@kristeine kristeine requested a review from a team as a code owner June 24, 2024 07:55
@fredrikpf fredrikpf closed this Oct 10, 2024
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