Mulighet til å sette ingen distribusjon ved retur (#426)

* Mulighet til å sette ingen distribusjon ved retur * Mulighet til å sette ingen distribusjon ved retur * Mulighet til å sette ingen distribusjon ved retur * Mulighet til å sette ingen distribusjon ved retur
navikt · Oct 20, 2023 · 28a9449 · 28a9449
1 parent af5c087
commit 28a9449
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 12 deletions.
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
@@ -208,7 +208,7 @@ data class Journalpost(
 
     fun hentAvsenderMottakerId(): String? = avsenderMottaker?.id
     fun hentJournalStatus(): JournalpostStatus? {
-        return if (isDistribusjonKommetIRetur()) {
+        return if (isDistribusjonKommetIRetur() && kanal != JournalpostKanal.INGEN_DISTRIBUSJON) {
             JournalpostStatus.RETUR
         } else {
             when (journalstatus) {
@@ -460,11 +460,14 @@ data class Journalpost(
                 AvvikType.BESTILL_NY_DISTRIBUSJON
             )
         }
-        if (isUtgaaendeDokument() && isStatusFerdigsstilt() && !isDistribusjonBestilt() && kanal != JournalpostKanal.INGEN_DISTRIBUSJON) {
-            avvikTypeList.add(
-                AvvikType.MANGLER_ADRESSE
-            )
+        if (isUtgaaendeDokument() && kanal != JournalpostKanal.INGEN_DISTRIBUSJON) {
+            if (isStatusEkspedert() && isDistribusjonKommetIRetur() || isStatusFerdigsstilt() && !isDistribusjonBestilt()) {
+                avvikTypeList.add(
+                    AvvikType.MANGLER_ADRESSE
+                )
+            }
         }
+
         if (isFarskap() && !isFarskapUtelukket() && !isStatusMottatt()) avvikTypeList.add(AvvikType.FARSKAP_UTELUKKET)
         return avvikTypeList
     }
@@ -817,9 +820,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
         )
@@ -983,7 +986,7 @@ data class EndreJournalpostCommandIntern(
     fun sjekkGyldigEndringAvReturDato(journalpost: Journalpost, violations: MutableList<String>) {
         val endreReturDetaljer =
             endreJournalpostCommand.endreReturDetaljer?.filter { Strings.isNotEmpty(it.beskrivelse) }
-        if (endreReturDetaljer != null && endreReturDetaljer.isNotEmpty()) {
+        if (!endreReturDetaljer.isNullOrEmpty()) {
             val kanEndreReturDetaljer = journalpost.isDistribusjonKommetIRetur()
             if (!kanEndreReturDetaljer) {
                 violations.add("Kan ikke endre returdetaljer på journalpost som ikke har kommet i retur")

diff --git a/src/test/java/no/nav/bidrag/dokument/arkiv/dto/JournalpostTest.kt b/src/test/java/no/nav/bidrag/dokument/arkiv/dto/JournalpostTest.kt
@@ -233,18 +233,18 @@ internal class JournalpostTest {
     }
 
     @Test
-    @DisplayName("skal hente avvik hvis Journalpost er status er EKSPEDERT og utgående")
-    fun skalHenteAvvikForEKSPEDERT_Utgaaende() {
+    fun `skal hente avvik for eksepdert utgående som har kommet i retur`() {
         val journalpost = Journalpost()
         journalpost.tema = "BID"
         journalpost.journalstatus = JournalStatus.EKSPEDERT
         journalpost.journalposttype = JournalpostType.U
         journalpost.antallRetur = 1
         journalpost.sak = Sak("")
         val avvikListe = journalpost.tilAvvik()
-        assertThat(avvikListe).hasSize(2)
+        assertThat(avvikListe).hasSize(3)
         assertThat(avvikListe).contains(AvvikType.BESTILL_NY_DISTRIBUSJON)
         assertThat(avvikListe).contains(AvvikType.FEILFORE_SAK)
+        assertThat(avvikListe).contains(AvvikType.MANGLER_ADRESSE)
     }
 
     @Test