-
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
Historikkvask for punsj v3 #2293
Conversation
e24657f
to
facbc89
Compare
0ef61f9
to
7706e7f
Compare
|
||
// V3 | ||
val oppgaveV3 = hentV3Oppgave(event) | ||
// TODO: Skal status på oppgaven endres her, eller er testen ikke relevant? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Godt spørsmål. Finnes det eventer fra punsj uten aksjonspunkter? Skulle tro at sendtInn hadde vært en bedre ting å teste, men det kan skyldes midlertidig tilstand i punsj som vi ikke ønsker å vise oppgave for i los.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Det finnes eventer uten aksjonspunkter. Dette er reelle eksempler fra dev: https://github.com/navikt/k9-los-api/blob/master/src/test/resources/no/nav/k9/los/nyoppgavestyring/domeneadaptere/k9punsjtillos/uten_fnr_dnr.json og https://github.com/navikt/k9-los-api/blob/master/src/test/resources/no/nav/k9/los/nyoppgavestyring/domeneadaptere/k9punsjtillos/prosessEventEksempel2.json
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Den siste av dem er sendt inn, så da er det forståelig at det ikke er noen aksjonspunkter.
Vet du hva slags tilstand den første eksempelet (uten_fnr_dnr) tilsvarer i punsj? Er den satt på vent?
Lukker denne PR-en, mesteparten her er tatt inn allerede. |
No description provided.