Legg til kommentar etter retur

navikt · Nov 13, 2023 · a9c2c7b · a9c2c7b
1 parent 6a28257
commit a9c2c7b
Show file tree

Hide file tree

Showing 6 changed files with 50 additions and 34 deletions.
diff --git a/src/main/java/no/nav/bidrag/dokument/arkiv/controller/AvvikController.kt b/src/main/java/no/nav/bidrag/dokument/arkiv/controller/AvvikController.kt
@@ -38,8 +38,8 @@ class AvvikController(private val avvikService: AvvikService) : BaseController()
     @Operation(
         security = [SecurityRequirement(name = "bearer-key")],
         summary = "Henter mulige avvik for en journalpost, id på formatet '" +
-                KildesystemIdenfikator.PREFIX_JOARK +
-                "<journalpostId>'"
+            KildesystemIdenfikator.PREFIX_JOARK +
+            "<journalpostId>'"
     )
     @ApiResponses(
         value = [
@@ -93,8 +93,8 @@ class AvvikController(private val avvikService: AvvikService) : BaseController()
     @Operation(
         security = [SecurityRequirement(name = "bearer-key")],
         summary = "Behandler et avvik for en journalpost, id på formatet '" +
-                KildesystemIdenfikator.PREFIX_JOARK_COMPLETE +
-                "<journalpostId>'"
+            KildesystemIdenfikator.PREFIX_JOARK_COMPLETE +
+            "<journalpostId>'"
     )
     @Transactional
     @ApiResponses(

diff --git a/src/main/java/no/nav/bidrag/dokument/arkiv/kafka/BehandleOppgaveHendelseService.kt b/src/main/java/no/nav/bidrag/dokument/arkiv/kafka/BehandleOppgaveHendelseService.kt
@@ -138,9 +138,11 @@ class BehandleOppgaveHendelseService(
 
     private fun Journalpost.harReturKommetFraNavNo() = distribuertTilAdresse() == null
     private fun opprettKommentarSomLeggesTilOppgave(journalpost: Journalpost): String? {
-        SECURE_LOGGER.info("Journalpost kommet retur med følgende detaljer ${journalpost.journalpostId} ${journalpost.journalstatus} ${journalpost.distribuertTilAdresse()} ${
+        SECURE_LOGGER.info(
+            "Journalpost kommet retur med følgende detaljer ${journalpost.journalpostId} ${journalpost.journalstatus} ${journalpost.distribuertTilAdresse()} ${
             journalpost.relevanteDatoer.joinToString(",") { "${it.datotype}:${it.dato}" }
-        }")
+            }"
+        )
         return if (journalpost.harReturKommetFraNavNo()) "Mottaker har ikke åpnet forsendelsen via www.nav.no innen 40 timer. Ingen postadresse er registrert. Vurder om mottaker har adresse forsendelsen kan sendes til." else null
     }
 

diff --git a/src/main/kotlin/no/nav/bidrag/dokument/arkiv/dto/Journalpost.kt b/src/main/kotlin/no/nav/bidrag/dokument/arkiv/dto/Journalpost.kt
@@ -227,9 +227,13 @@ data class Journalpost(
                 val postnummerPoststed = postadresseSplit.getOrNull(1)?.split(" ") ?: emptyList()
                 val postnummer = if (postnummerPoststed.size == 2) postnummerPoststed.getOrNull(0) else null
                 val poststed =
-                    if (postnummerPoststed.size == 1) postnummerPoststed.getOrNull(0) else postnummerPoststed.getOrNull(
-                        1
-                    )
+                    if (postnummerPoststed.size == 1) {
+                        postnummerPoststed.getOrNull(0)
+                    } else {
+                        postnummerPoststed.getOrNull(
+                            1
+                        )
+                    }
                 val adresselinje1 = when (postadresseSplit.size) {
                     3 -> postadresseSplit.getOrNull(2)
                     4 -> postadresseSplit.getOrNull(3)
@@ -893,9 +897,9 @@ data class ReturDetaljerLogDO(
     fun toMap(): List<Map<String, String>> = beskrivelse.chunked(100).mapIndexed { index, it ->
         mapOf(
             "nokkel" to "${if (locked == true) "L" else ""}$RETUR_DETALJER_KEY${index}_${
-                DateUtils.formatDate(
-                    dato
-                )
+            DateUtils.formatDate(
+                dato
+            )
             }",
             "verdi" to it
         )

diff --git a/src/main/kotlin/no/nav/bidrag/dokument/arkiv/dto/Oppgave.kt b/src/main/kotlin/no/nav/bidrag/dokument/arkiv/dto/Oppgave.kt
@@ -99,8 +99,8 @@ data class LeggTilKommentarPaaOppgave(
         versjon = oppgaveData.versjon,
         endretAvEnhetsnr = _endretAvEnhetsnr,
         beskrivelse = beskrivelseHeader(saksbehandlersInfo) +
-                "$kommentar\r\n\r\n" +
-                "${oppgaveData.beskrivelse}"
+            "$kommentar\r\n\r\n" +
+            "${oppgaveData.beskrivelse}"
     )
 
 data class FerdigstillOppgaveRequest(
@@ -155,7 +155,7 @@ data class BestillSplittingoppgaveRequest(
     ) {
     init {
         beskrivelse = "${beskrivelseHeader(saksbehandlerMedEnhet.hentSaksbehandlerInfo())}\n${
-            bestillSplittingKommentar(beskrivSplitting)
+        bestillSplittingKommentar(beskrivSplitting)
         }"
     }
 }
@@ -177,7 +177,7 @@ data class BestillReskanningOppgaveRequest(
     ) {
     init {
         beskrivelse = "${beskrivelseHeader(saksbehandlerMedEnhet.hentSaksbehandlerInfo())}\n${
-            bestillReskanningKommentar(kommentar)
+        bestillReskanningKommentar(kommentar)
         }"
     }
 }
@@ -299,7 +299,7 @@ internal fun lagVurderDokumentOppgaveBeskrivelse(
     regDato: LocalDate
 ): String {
     var description = "--- ${
-        LocalDate.now().format(NORSK_DATO_FORMAT)
+    LocalDate.now().format(NORSK_DATO_FORMAT)
     } ${saksbehandlerMedEnhet.hentSaksbehandlerInfo()} ---\n $brevKode $dokumentTittel"
     if (kommentar != null) {
         description += "\n\n $kommentar"

diff --git a/src/test/java/no/nav/bidrag/dokument/arkiv/controller/DistribuerControllerTest.kt b/src/test/java/no/nav/bidrag/dokument/arkiv/controller/DistribuerControllerTest.kt
@@ -162,11 +162,11 @@ internal class DistribuerControllerTest : AbstractControllerTest() {
             stubs.verifyStub.dokarkivOppdaterKalt(
                 JOURNALPOST_ID,
                 "{\"tilleggsopplysninger\":[" +
-                        "{\"nokkel\":\"dokdistBestillingsId\",\"verdi\":\"asdsadasdsadasdasd\"}," +
-                        "{\"nokkel\":\"journalfortAvIdent\",\"verdi\":\"Z99999\"}," +
-                        "{\"nokkel\":\"distAdresse0\",\"verdi\":\"{\\\"adresselinje1\\\":\\\"Adresselinje1\\\",\\\"adresselinje2\\\":\\\"Adresselinje2\\\",\\\"adresselinje3\\\":\\\"Adresselinje3\\\",\\\"la\"}," +
-                        "{\"nokkel\":\"distAdresse1\",\"verdi\":\"nd\\\":\\\"NO\\\",\\\"postnummer\\\":\\\"3000\\\",\\\"poststed\\\":\\\"Ingen\\\"}\"}," +
-                        "{\"nokkel\":\"distribusjonBestilt\",\"verdi\":\"true\"},{\"nokkel\":\"distribuertAvIdent\",\"verdi\":\"aud-localhost\"}],\"dokumenter\":[]}"
+                    "{\"nokkel\":\"dokdistBestillingsId\",\"verdi\":\"asdsadasdsadasdasd\"}," +
+                    "{\"nokkel\":\"journalfortAvIdent\",\"verdi\":\"Z99999\"}," +
+                    "{\"nokkel\":\"distAdresse0\",\"verdi\":\"{\\\"adresselinje1\\\":\\\"Adresselinje1\\\",\\\"adresselinje2\\\":\\\"Adresselinje2\\\",\\\"adresselinje3\\\":\\\"Adresselinje3\\\",\\\"la\"}," +
+                    "{\"nokkel\":\"distAdresse1\",\"verdi\":\"nd\\\":\\\"NO\\\",\\\"postnummer\\\":\\\"3000\\\",\\\"poststed\\\":\\\"Ingen\\\"}\"}," +
+                    "{\"nokkel\":\"distribusjonBestilt\",\"verdi\":\"true\"},{\"nokkel\":\"distribuertAvIdent\",\"verdi\":\"aud-localhost\"}],\"dokumenter\":[]}"
             )
         }
     }
@@ -258,10 +258,10 @@ internal class DistribuerControllerTest : AbstractControllerTest() {
             stubs.verifyStub.dokarkivOppdaterKalt(
                 JOURNALPOST_ID,
                 "{\"tilleggsopplysninger\":[" +
-                        "{\"nokkel\":\"dokdistBestillingsId\",\"verdi\":\"asdsadasdsadasdasd\"}," +
-                        "{\"nokkel\":\"journalfortAvIdent\",\"verdi\":\"Z99999\"},{\"nokkel\":\"distribusjonBestilt\",\"verdi\":\"true\"}," +
-                        "{\"nokkel\":\"origDistDigitalt\",\"verdi\":\"true\"}," +
-                        "{\"nokkel\":\"distribuertAvIdent\",\"verdi\":\"aud-localhost\"}],\"dokumenter\":[]}"
+                    "{\"nokkel\":\"dokdistBestillingsId\",\"verdi\":\"asdsadasdsadasdasd\"}," +
+                    "{\"nokkel\":\"journalfortAvIdent\",\"verdi\":\"Z99999\"},{\"nokkel\":\"distribusjonBestilt\",\"verdi\":\"true\"}," +
+                    "{\"nokkel\":\"origDistDigitalt\",\"verdi\":\"true\"}," +
+                    "{\"nokkel\":\"distribuertAvIdent\",\"verdi\":\"aud-localhost\"}],\"dokumenter\":[]}"
             )
         }
     }
@@ -351,11 +351,11 @@ internal class DistribuerControllerTest : AbstractControllerTest() {
             stubs.verifyStub.dokarkivOppdaterKalt(
                 JOURNALPOST_ID,
                 "{\"tilleggsopplysninger\":[" +
-                        "{\"nokkel\":\"dokdistBestillingsId\",\"verdi\":\"asdsadasdsadasdasd\"}," +
-                        "{\"nokkel\":\"journalfortAvIdent\",\"verdi\":\"Z99999\"}," +
-                        "{\"nokkel\":\"distAdresse0\",\"verdi\":\"{\\\"adresselinje1\\\":\\\"Adresselinje1\\\",\\\"adresselinje2\\\":\\\"Adresselinje2\\\",\\\"adresselinje3\\\":\\\"Adresselinje3\\\",\\\"la\"}," +
-                        "{\"nokkel\":\"distAdresse1\",\"verdi\":\"nd\\\":\\\"NO\\\",\\\"postnummer\\\":\\\"3000\\\",\\\"poststed\\\":\\\"Ingen\\\"}\"}," +
-                        "{\"nokkel\":\"distribusjonBestilt\",\"verdi\":\"true\"},{\"nokkel\":\"distribuertAvIdent\",\"verdi\":\"aud-localhost\"}],\"dokumenter\":[]}"
+                    "{\"nokkel\":\"dokdistBestillingsId\",\"verdi\":\"asdsadasdsadasdasd\"}," +
+                    "{\"nokkel\":\"journalfortAvIdent\",\"verdi\":\"Z99999\"}," +
+                    "{\"nokkel\":\"distAdresse0\",\"verdi\":\"{\\\"adresselinje1\\\":\\\"Adresselinje1\\\",\\\"adresselinje2\\\":\\\"Adresselinje2\\\",\\\"adresselinje3\\\":\\\"Adresselinje3\\\",\\\"la\"}," +
+                    "{\"nokkel\":\"distAdresse1\",\"verdi\":\"nd\\\":\\\"NO\\\",\\\"postnummer\\\":\\\"3000\\\",\\\"poststed\\\":\\\"Ingen\\\"}\"}," +
+                    "{\"nokkel\":\"distribusjonBestilt\",\"verdi\":\"true\"},{\"nokkel\":\"distribuertAvIdent\",\"verdi\":\"aud-localhost\"}],\"dokumenter\":[]}"
             )
         }
     }

diff --git a/src/test/java/no/nav/bidrag/dokument/arkiv/hendelser/OppgaveHendelseListenerTest.kt b/src/test/java/no/nav/bidrag/dokument/arkiv/hendelser/OppgaveHendelseListenerTest.kt
@@ -6,10 +6,12 @@ import io.kotest.assertions.assertSoftly
 import no.nav.bidrag.dokument.arkiv.BidragDokumentArkivConfig
 import no.nav.bidrag.dokument.arkiv.BidragDokumentArkivTest
 import no.nav.bidrag.dokument.arkiv.dto.DatoType
+import no.nav.bidrag.dokument.arkiv.dto.FysiskpostSendt
 import no.nav.bidrag.dokument.arkiv.dto.OppgaveData
 import no.nav.bidrag.dokument.arkiv.dto.ReturDetaljerLogDO
 import no.nav.bidrag.dokument.arkiv.dto.Sak
 import no.nav.bidrag.dokument.arkiv.dto.TilleggsOpplysninger
+import no.nav.bidrag.dokument.arkiv.dto.UtsendingsInfo
 import no.nav.bidrag.dokument.arkiv.kafka.HendelseListener
 import no.nav.bidrag.dokument.arkiv.kafka.dto.OppgaveKafkaHendelse
 import no.nav.bidrag.dokument.arkiv.model.OppgaveStatus
@@ -85,7 +87,7 @@ class OppgaveHendelseListenerTest {
             relevanteDatoer = listOf(
                 DatoType("2021-08-18T13:20:33", "DATO_DOKUMENT")
             )
-        )
+        ).copy(utsendingsinfo = UtsendingsInfo(fysiskpostSendt = FysiskpostSendt("Adresselinje1\n3033 Drammen\nNO")))
         safResponse.antallRetur = 1
 
         stubs.mockSafResponseHentJournalpost(safResponse)
@@ -188,7 +190,8 @@ class OppgaveHendelseListenerTest {
             relevanteDatoer = listOf(
                 DatoType("2021-08-18T13:20:33", "DATO_DOKUMENT")
             )
-        )
+        ).copy(utsendingsinfo = UtsendingsInfo(fysiskpostSendt = FysiskpostSendt("Adresselinje1\n3033 Drammen\nNO")))
+
         safResponse.antallRetur = 1
         safResponse.sak = Sak("5276661")
 
@@ -236,7 +239,8 @@ class OppgaveHendelseListenerTest {
             relevanteDatoer = listOf(
                 DatoType("2021-08-18T13:20:33", "DATO_DOKUMENT")
             )
-        )
+        ).copy(utsendingsinfo = UtsendingsInfo(fysiskpostSendt = FysiskpostSendt("Adresselinje1\n3033 Drammen\nNO")))
+
         safResponse.antallRetur = 1
 
         stubs.mockSafResponseHentJournalpost(opprettUtgaendeSafResponse(), null, "NO_RETUR")
@@ -365,6 +369,12 @@ class OppgaveHendelseListenerTest {
                 "\"versjon\":22",
                 "Mottaker har ikke åpnet forsendelsen via www.nav.no innen 40 timer. Ingen postadresse er registrert. Vurder om mottaker har adresse forsendelsen kan sendes til"
             )
+            stubs.verifyStub.dokarkivOppdaterKalt(
+                journalpostId,
+                "\"tilleggsopplysninger\":" +
+                        "[{\"nokkel\":\"distribusjonBestilt\",\"verdi\":\"true\"}," +
+                        "{\"nokkel\":\"retur0_${DateUtils.formatDate(LocalDate.now())}\",\"verdi\":\"Distribusjon feilet, mottaker mangler postadresse\"}]"
+            )
         }
     }