EY-3355 Forutsette hendelser #3692
Conversation
| * Det tar veldig mye tid å kjøre opp stadig nye Postgres-containere og kjøre Flyway migreringer. | ||
| * Denne extensionen kjører opp èn instans, som så gjenbrukes av de som måtte ønske det. | ||
| */ | ||
| object DatabaseExtension : BeforeAllCallback, AfterAllCallback, ExtensionContext.Store.CloseableResource { |
There was a problem hiding this comment.
Samme som i behandling, bare med andre tabeller i resetDb. Burde nok se på å gjøres mer gjenbrukbar, er object pga enkel tilgang til datasource fra testene men kanskje extensionen også kan injisere dette inn i testene.
There was a problem hiding this comment.
Har lyst til at vi får flytta denne over i libs/etterlatte-database/textfixtures, men det med tabellnamn i resetDb er jo da ei utfordring.
| @@ -6,6 +6,7 @@ enum class EventNames(val eventname: String) : EventnameHendelseType { | |||
| GRUNNLAGSVERSJONERING_EVENT_NAME("GRUNNLAGSVERSJONERING_EVENT_NAME"), | |||
| FEILA("FEILA"), | |||
| FORDELER_STATISTIKK("FORDELER:STATISTIKK"), | |||
| ALDERSOVERGANG("ALDERSOVERGANG"), | |||
There was a problem hiding this comment.
Bør nok være generisk (a la appnavnet) slik at man ikke trenger flere lyttere f.eks. for aldersovergang, og noe annet for revurdering oms pga x måneder siden dødsfall.
| enum class JobbType(val beskrivelse: String) { | ||
| AO_BP18("Aldersovergang barnepensjon ved 18 år"), | ||
| AO_BP20("Aldersovergang barnepensjon ved 20 år"), | ||
| } |
There was a problem hiding this comment.
tenker denne utvides med kategori (aldersovergang, annet), slik at komponenten som sender ut på kafka har lett tilgang til å sette ymse props
apps/etterlatte-tidshendelser/src/main/resources/logback-secure.xml
Outdated
Show resolved
Hide resolved
apps/etterlatte-tidshendelser/src/main/kotlin/no/nav/etterlatte/tidshendelser/HendelseDao.kt
Outdated
Show resolved
Hide resolved
apps/etterlatte-tidshendelser/src/main/kotlin/no/nav/etterlatte/tidshendelser/HendelsePoller.kt
Outdated
Show resolved
Hide resolved
...tterlatte-tidshendelser/src/main/kotlin/no/nav/etterlatte/tidshendelser/HendelsePublisher.kt
Outdated
Show resolved
Hide resolved
apps/etterlatte-tidshendelser/src/main/kotlin/no/nav/etterlatte/tidshendelser/HendelseDao.kt
Outdated
Show resolved
Hide resolved
| logger.info("Hentet ${saker.size} saker for brukere født i $foedselsmaaned") | ||
|
|
||
| if (saker.isNotEmpty()) { | ||
| hendelseDao.opprettHendelserForSaker(jobb.id, saker, Steg.IDENTIFISERT_SAK) |
There was a problem hiding this comment.
Skal det ikke kjøres oppdaterJobbstatusFerdig når alle hendelser er ferdig?
There was a problem hiding this comment.
Lytting på hendelser er ikke påbegynt ennå, dvs jobben skal ikke ha status ferdig før alle hendelser som er knyttet til den er ferdig (på et eller annet vis - oppgave er ferdigstilt)
There was a problem hiding this comment.
Ah var det du nevnte på standup. Jeg trodde det var for hver enkelte hendelse.
Når en jobb da inneholder mange hendelser så kan det jo gå lang tid før samtlige hendelser er plukket opp eller håndtert?
Hva er egentlig hensikten/gevinsten med å holde en overordna jobb for alle hendelsene åpen?
There was a problem hiding this comment.
Tenker vi kan fortsette diskusjon på zoom etter hvert, er Helge sin idé :p
apps/etterlatte-tidshendelser/src/main/kotlin/no/nav/etterlatte/tidshendelser/HendelsePoller.kt
Outdated
Show resolved
Hide resolved
…en ha en property som angir hendelsen
…se, annet enn status
ingleivj
force-pushed
the
feature/EY-3355_ao20
branch
from
2978a42 to
e28861e
Compare
|
stor force-push for å korrigere ymse commits med feil i importsene som følge av #3680 |
Første case: aldersovergang barnepensjon ved 20 år. Se gjerne mural lenket i jira.
Konsept:
Noe utestående, da PRen er svær nok allerede:
NB! ref navikt/pensjon-etterlatte-felles#200 så er det helt greit med diskusjon rundt hva barnet skal hete :)