Merge branch 'main' into dependabot/maven/org.codehaus.mojo-build-hel…

…per-maven-plugin-3.6.0
navikt · Oct 21, 2024 · 5757309 · 5757309
2 parents a99b14c + 6eded8d
commit 5757309
Show file tree

Hide file tree

Showing 3 changed files with 67 additions and 32 deletions.
diff --git a/.nais/nais.yaml b/.nais/nais.yaml
@@ -47,10 +47,10 @@ spec:
   resources:
     requests:
       cpu: {{ cpu.requests }}
-      memory: 2500Mi
+      memory: 1Gi
     limits:
       cpu: {{ cpu.limits }}
-      memory: 4096Mi
+      memory: 2Gi
   ingresses:
   {{#each ingresses as |url|}}
       - {{url}}

diff --git a/src/main/kotlin/no/nav/bidrag/dokument/arkiv/dto/Avvik.kt b/src/main/kotlin/no/nav/bidrag/dokument/arkiv/dto/Avvik.kt
@@ -100,14 +100,10 @@ data class AvvikshendelseIntern(
     )
 }
 
-data class OverforEnhetRequest(
-    private var journalpostId: Long,
-    override var journalfoerendeEnhet: String?,
-) :
+data class OverforEnhetRequest(private var journalpostId: Long, override var journalfoerendeEnhet: String?) :
     OppdaterJournalpostRequest(journalpostId)
 
-data class EndreFagomradeRequest(private var journalpostId: Long, override var tema: String?) :
-    OppdaterJournalpostRequest(journalpostId)
+data class EndreFagomradeRequest(private var journalpostId: Long, override var tema: String?) : OppdaterJournalpostRequest(journalpostId)
 
 data class EndreFagomradeOgKnyttTilSakRequest(
     private var journalpostId: Long,
@@ -116,10 +112,7 @@ data class EndreFagomradeOgKnyttTilSakRequest(
     override var sak: Sak? = GenerellSak(),
 ) : OppdaterJournalpostRequest(journalpostId)
 
-data class EndreFagomradeJournalfortJournalpostRequest(
-    private var journalpostId: Long,
-    private var journalpost: Journalpost,
-) :
+data class EndreFagomradeJournalfortJournalpostRequest(private var journalpostId: Long, private var journalpost: Journalpost) :
     OppdaterJournalpostRequest(journalpostId) {
     init {
         journalpost.tilleggsopplysninger.setEndretTemaFlagg()
@@ -135,22 +128,15 @@ data class OppdaterOriginalBestiltFlagg(private var journalpost: Journalpost) :
     }
 }
 
-data class OpphevEndreFagomradeJournalfortJournalpostRequest(
-    private var journalpostId: Long,
-    private var journalpost: Journalpost,
-) :
+data class OpphevEndreFagomradeJournalfortJournalpostRequest(private var journalpostId: Long, private var journalpost: Journalpost) :
     OppdaterJournalpostRequest(journalpostId) {
     init {
         journalpost.tilleggsopplysninger.removeEndretTemaFlagg()
         tilleggsopplysninger = journalpost.tilleggsopplysninger
     }
 }
 
-data class EndreTittelRequest(
-    private var journalpostId: Long,
-    private var _tittel: String?,
-    private var journalpost: Journalpost,
-) :
+data class EndreTittelRequest(private var journalpostId: Long, private var _tittel: String?, private var journalpost: Journalpost) :
     OppdaterJournalpostRequest(journalpostId) {
 
     init {
@@ -159,7 +145,11 @@ data class EndreTittelRequest(
             dokumenter =
                 listOf(Dokument(hoveddokument.dokumentInfoId, _tittel, null))
         }
-        if (!journalpost.isUtgaaendeDokument()) tittel = _tittel
+        if (journalpost.isInngaaendeDokument() &&
+            journalpost.hentDatoRegistrert()?.isAfter(LocalDate.now().minusYears(1).minusDays(1)) == true
+        ) {
+            tittel = _tittel
+        }
     }
 }
 
@@ -170,10 +160,7 @@ data class EndreKnyttTilGenerellSakRequest(
     override var sak: Sak? = GenerellSak(),
 ) : OppdaterJournalpostRequest(journalpostId)
 
-data class InngaaendeTilUtgaaendeRequest(
-    private var journalpostId: Long,
-    override var tema: String?,
-) : OppdaterJournalpostRequest(journalpostId)
+data class InngaaendeTilUtgaaendeRequest(private var journalpostId: Long, override var tema: String?) : OppdaterJournalpostRequest(journalpostId)
 
 data class RegistrerReturRequest(
     private var journalpostId: Long,

diff --git a/src/test/java/no/nav/bidrag/dokument/arkiv/controller/AvvikControllerTest.kt b/src/test/java/no/nav/bidrag/dokument/arkiv/controller/AvvikControllerTest.kt
@@ -6,6 +6,7 @@ import no.nav.bidrag.commons.web.EnhetFilter
 import no.nav.bidrag.dokument.arkiv.consumer.BestemKanalResponse
 import no.nav.bidrag.dokument.arkiv.consumer.DistribusjonsKanal
 import no.nav.bidrag.dokument.arkiv.dto.AvsenderMottaker
+import no.nav.bidrag.dokument.arkiv.dto.DatoType
 import no.nav.bidrag.dokument.arkiv.dto.DokDistDistribuerJournalpostRequest
 import no.nav.bidrag.dokument.arkiv.dto.Dokument
 import no.nav.bidrag.dokument.arkiv.dto.JournalStatus
@@ -49,6 +50,7 @@ import org.springframework.http.HttpMethod
 import org.springframework.http.HttpStatus
 import org.springframework.http.ResponseEntity
 import java.io.IOException
+import java.time.LocalDateTime
 import java.util.Base64
 
 class AvvikControllerTest : AbstractControllerTest() {
@@ -119,6 +121,7 @@ class AvvikControllerTest : AbstractControllerTest() {
                         tittel = "tittel",
                     ),
                 ),
+                relevanteDatoer = listOf(DatoType(LocalDateTime.now().minusMonths(8).toString(), "DATO_REGISTRERT")),
                 tema = "FAR",
                 journalstatus = JournalStatus.JOURNALFOERT,
             ),
@@ -144,6 +147,53 @@ class AvvikControllerTest : AbstractControllerTest() {
         }
     }
 
+    @Test
+    fun `skal utfore avvik FARSKAP_UTELUKKET og ikke oppdatere tittel hvis eldre enn 1 år`() {
+        // given
+        val xEnhet = "1234"
+        val journalpostIdFraJson = 201028011L
+        val avvikHendelse = createAvvikHendelse(AvvikType.FARSKAP_UTELUKKET, emptyMap())
+        stubs.mockSafResponseTilknyttedeJournalposter(HttpStatus.OK)
+        stubs.mockSafResponseHentJournalpost(
+            opprettSafResponse(
+                journalpostIdFraJson.toString(),
+                avsenderMottaker = AvsenderMottaker(),
+                dokumenter = listOf(
+                    Dokument(
+                        dokumentInfoId = DOKUMENT_1_ID,
+                        tittel = DOKUMENT_1_TITTEL,
+                    ),
+                    Dokument(
+                        dokumentInfoId = "123213",
+                        tittel = "tittel",
+                    ),
+                ),
+                tema = "FAR",
+                relevanteDatoer = listOf(DatoType(LocalDateTime.now().minusYears(1).toString(), "DATO_REGISTRERT")),
+                journalstatus = JournalStatus.JOURNALFOERT,
+            ),
+        )
+        stubs.mockPersonResponse(PersonDto(PERSON_IDENT, aktørId = AKTOR_IDENT), HttpStatus.OK)
+        stubs.mockDokarkivOppdaterRequest(journalpostIdFraJson)
+
+        val response = sendAvvikRequest(xEnhet, journalpostIdFraJson, avvikHendelse)
+
+        assertSoftly {
+            response.statusCode shouldBe HttpStatus.OK
+            stubs.verifyStub.dokarkivOppdaterKalt(
+                journalpostIdFraJson,
+                "\"dokumenter\":[{\"dokumentInfoId\":\"123123\",\"tittel\":\"FARSKAP UTELUKKET: Tittel på dokument 1\"}]",
+            )
+            Mockito.verify(kafkaTemplateMock).send(
+                ArgumentMatchers.eq(topicJournalpost),
+                ArgumentMatchers.eq(
+                    "JOARK-$journalpostIdFraJson",
+                ),
+                ArgumentMatchers.any(),
+            )
+        }
+    }
+
     @Test
     fun `skal utfore avvik REGISTRER_RETUR`() {
         // given
@@ -1468,10 +1518,8 @@ class AvvikControllerTest : AbstractControllerTest() {
         )
     }
 
-    private fun createAvvikHendelse(avvikType: AvvikType, detaljer: Map<String, String>): Avvikshendelse {
-        return Avvikshendelse(
-            avvikType = avvikType,
-            detaljer = detaljer,
-        )
-    }
+    private fun createAvvikHendelse(avvikType: AvvikType, detaljer: Map<String, String>): Avvikshendelse = Avvikshendelse(
+        avvikType = avvikType,
+        detaljer = detaljer,
+    )
 }