Skip to content

Commit

Permalink
Kjør resend brev som systembruker, slik at det ikke står utviklers na…
Browse files Browse the repository at this point in the history
…vn i brevet som resendes (#5020)

### 💰 Hva skal gjøres, og hvorfor?
Kjør resend brev som systembruker, slik at det ikke står utviklers navn
i brevet som resendes

### 🔎️ 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:_

### 💬 Ønsker du en muntlig gjennomgang?
- [ ] Ja
- [ ] Nei
  • Loading branch information
stigebil authored Jan 20, 2025
1 parent 43ca518 commit de07eb3
Show file tree
Hide file tree
Showing 3 changed files with 46 additions and 9 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@ import no.nav.familie.ba.sak.common.UtbetalingsikkerhetFeil
import no.nav.familie.ba.sak.common.førsteDagIInneværendeMåned
import no.nav.familie.ba.sak.common.secureLogger
import no.nav.familie.ba.sak.common.toYearMonth
import no.nav.familie.ba.sak.config.BehandlerRolle
import no.nav.familie.ba.sak.config.TaskRepositoryWrapper
import no.nav.familie.ba.sak.integrasjoner.infotrygd.InfotrygdService
import no.nav.familie.ba.sak.integrasjoner.økonomi.ØkonomiService
Expand All @@ -29,7 +28,6 @@ import no.nav.familie.ba.sak.kjerne.behandling.domene.BehandlingÅrsak
import no.nav.familie.ba.sak.kjerne.beregning.BeregningService
import no.nav.familie.ba.sak.kjerne.beregning.TilkjentYtelseValideringService
import no.nav.familie.ba.sak.kjerne.beregning.domene.AndelTilkjentYtelseRepository
import no.nav.familie.ba.sak.kjerne.brev.DokumentDistribueringService
import no.nav.familie.ba.sak.kjerne.fagsak.FagsakRepository
import no.nav.familie.ba.sak.kjerne.fagsak.FagsakService
import no.nav.familie.ba.sak.kjerne.grunnlag.personopplysninger.PersongrunnlagService
Expand All @@ -42,6 +40,7 @@ import no.nav.familie.ba.sak.kjerne.vilkårsvurdering.domene.Vilkår.UNDER_18_Å
import no.nav.familie.ba.sak.kjerne.vilkårsvurdering.domene.Vilkårsvurdering
import no.nav.familie.ba.sak.sikkerhet.SikkerhetContext
import no.nav.familie.ba.sak.task.DistribuerDokumentDTO
import no.nav.familie.ba.sak.task.ForvaltningRedistribuerBrevTask
import no.nav.familie.ba.sak.task.IverksettMotOppdragTask
import no.nav.familie.kontrakter.felles.objectMapper
import no.nav.familie.log.mdc.MDCConstants
Expand Down Expand Up @@ -76,7 +75,6 @@ class ForvalterService(
private val vilkårsvurderingService: VilkårsvurderingService,
private val persongrunnlagService: PersongrunnlagService,
private val andelTilkjentYtelseRepository: AndelTilkjentYtelseRepository,
private val dokumentDistribueringService: DokumentDistribueringService,
private val taskService: TaskService,
) {
private val logger = LoggerFactory.getLogger(ForvalterService::class.java)
Expand Down Expand Up @@ -396,8 +394,9 @@ class ForvalterService(

if (!dryRun) {
kjørMedCallId(UUID.randomUUID().toString()) {
logger.info("distribuerDokumentFraTaskForFerdigstiltBehandling: task: $it, distribuerDokumentDTO=$distribuerDokumentDTO")
dokumentDistribueringService.prøvDistribuerBrevOgLoggHendelseFraBehandling(distribuerDokumentDTO = distribuerDokumentDTO, loggBehandlerRolle = BehandlerRolle.SYSTEM)
ForvaltningRedistribuerBrevTask.opprettTask(distribuerDokumentDTO).also {
taskService.save(it)
}
}
} else {
logger.info("dry run: distribuerDokumentFraTaskForFerdigstiltBehandling: task: $it, distribuerDokumentDTO=$distribuerDokumentDTO")
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,42 @@
package no.nav.familie.ba.sak.task

import no.nav.familie.ba.sak.config.BehandlerRolle
import no.nav.familie.ba.sak.kjerne.brev.DokumentDistribueringService
import no.nav.familie.kontrakter.felles.objectMapper
import no.nav.familie.prosessering.AsyncTaskStep
import no.nav.familie.prosessering.TaskStepBeskrivelse
import no.nav.familie.prosessering.domene.Task
import org.slf4j.LoggerFactory
import org.springframework.stereotype.Service

@Service
@TaskStepBeskrivelse(
taskStepType = ForvaltningRedistribuerBrevTask.TASK_STEP_TYPE,
beskrivelse = "Forvaltning: Redistribuer brev",
maxAntallFeil = 1,
settTilManuellOppfølgning = true,
)
@Deprecated("Skal kun brukes for å patche feil fra dok")
class ForvaltningRedistribuerBrevTask(
private val dokumentDistribueringService: DokumentDistribueringService,
) : AsyncTaskStep {
private val logger = LoggerFactory.getLogger(this::class.java)

override fun doTask(task: Task) {
val distribuerDokumentDTO = objectMapper.readValue(task.payload, DistribuerDokumentDTO::class.java)
logger.info("distribuerDokumentFraTaskForFerdigstiltBehandling: task: ${task.id}, distribuerDokumentDTO=$distribuerDokumentDTO")
dokumentDistribueringService.prøvDistribuerBrevOgLoggHendelseFraBehandling(distribuerDokumentDTO = distribuerDokumentDTO, loggBehandlerRolle = BehandlerRolle.SYSTEM)
}

companion object {
const val TASK_STEP_TYPE = "forvaltningRedistribuerBrevTask"

fun opprettTask(
distribuerDokumentDTO: DistribuerDokumentDTO,
): Task =
Task(
type = TASK_STEP_TYPE,
payload = objectMapper.writeValueAsString(distribuerDokumentDTO),
)
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,6 @@ import no.nav.familie.ba.sak.kjerne.beregning.BeregningService
import no.nav.familie.ba.sak.kjerne.beregning.TilkjentYtelseValideringService
import no.nav.familie.ba.sak.kjerne.beregning.domene.AndelTilkjentYtelseRepository
import no.nav.familie.ba.sak.kjerne.beregning.domene.YtelseType
import no.nav.familie.ba.sak.kjerne.brev.DokumentDistribueringService
import no.nav.familie.ba.sak.kjerne.fagsak.FagsakRepository
import no.nav.familie.ba.sak.kjerne.fagsak.FagsakService
import no.nav.familie.ba.sak.kjerne.grunnlag.personopplysninger.Målform
Expand Down Expand Up @@ -109,9 +108,6 @@ class ForvalterServiceTest {
@MockK
lateinit var andelTilkjentYtelseRepository: AndelTilkjentYtelseRepository

@MockK
lateinit var dokumentDistribueringService: DokumentDistribueringService

@MockK
lateinit var taskService: TaskService

Expand Down

0 comments on commit de07eb3

Please sign in to comment.