From 28a94490604d7b3759e240ebb6767f7fdf2739c1 Mon Sep 17 00:00:00 2001
From: Ugur Alpay Cenar <cenar93@gmail.com>
Date: Fri, 20 Oct 2023 16:26:37 +0200
Subject: [PATCH] =?UTF-8?q?Mulighet=20til=20=C3=A5=20sette=20ingen=20distr?=
 =?UTF-8?q?ibusjon=20ved=20retur=20(#426)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

* 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
---
 .../bidrag/dokument/arkiv/dto/Journalpost.kt  | 21 +++++++++++--------
 .../dokument/arkiv/dto/JournalpostTest.kt     |  6 +++---
 2 files changed, 15 insertions(+), 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
index dea48d5b..99cc967b 100644
--- 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
index 1a5e57f2..eee3e352 100644
--- 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,8 +233,7 @@ 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
@@ -242,9 +241,10 @@ internal class JournalpostTest {
         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