Prøv å håndtere utnulling av samtalereferat (#904)

* Prøv å håndtere utnulling av samtalereferat * fiks test for melding til oversikten --------- Co-authored-by: erikb <[email protected]>
navikt · Dec 23, 2024 · f6022e4 · f6022e4
1 parent 8f8c866
commit f6022e4
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 6 deletions.
diff --git a/src/main/java/no/nav/veilarbaktivitet/aktivitet/AktivitetAppService.java b/src/main/java/no/nav/veilarbaktivitet/aktivitet/AktivitetAppService.java
@@ -249,7 +249,7 @@ public AktivitetData oppdaterReferat(AktivitetData aktivitet) {
 
         var oppdatertAktivitet = aktivitetService.oppdaterReferat(originalAktivitet, aktivitet);
 
-        var maybeEventType = hentEventTypePåSamtalereferat(originalAktivitet, aktivitet);
+        var maybeEventType = hentEventType(originalAktivitet, aktivitet);
         maybeEventType.ifPresent(eventType -> {
             bigQueryClient.logEvent(oppdatertAktivitet, eventType);
             sendMeldingTilOversikten(oppdatertAktivitet, eventType);
@@ -265,11 +265,13 @@ private void sendMeldingTilOversikten(AktivitetData aktivitet, EventType eventTy
         }
     }
 
-    private Optional<EventType> hentEventTypePåSamtalereferat(AktivitetData originalAktivitet, AktivitetData oppdatertAktivitet) {
-        var forrigeReferat = Optional.ofNullable(originalAktivitet.getMoteData()).map(it -> it.getReferat()).orElse("");
+    private Optional<EventType> hentEventType(AktivitetData originalAktivitet, AktivitetData oppdatertAktivitet) {
+        var forrigeReferat = Optional.ofNullable(originalAktivitet.getMoteData()).map(it -> it.getReferat()).orElse(null);
         var nesteReferat = Optional.ofNullable(oppdatertAktivitet.getMoteData()).map(it -> it.getReferat()).orElse("");
 
-        var referatHarNåFåttInnhold = forrigeReferat.isEmpty() && !nesteReferat.isEmpty();
+        /* Hvis samtalereferatet har fått innhold men det hviskes bort så vil forreigeReferat være "",
+        * i samtalereferat er feltet referat obligatoprisk, dvs ikke sjekk mot null ikke isEmpty her */
+        var referatHarNåFåttInnhold = forrigeReferat == null && !nesteReferat.isEmpty();
         var referatHarNåBlittDeltMedBruker = !originalAktivitet.getMoteData().isReferatPublisert() && oppdatertAktivitet.getMoteData().isReferatPublisert();
 
         if (referatHarNåFåttInnhold && !referatHarNåBlittDeltMedBruker) {

diff --git a/src/main/kotlin/no/nav/veilarbaktivitet/eventsLogger/BigQueryClient.kt b/src/main/kotlin/no/nav/veilarbaktivitet/eventsLogger/BigQueryClient.kt
@@ -11,7 +11,7 @@ import java.time.ZoneOffset
 import java.time.ZonedDateTime
 
 enum class EventType {
-    SAMTALEREFERAT_OPPRETTET,
+    SAMTALEREFERAT_OPPRETTET, // TODO: Endre navn, dette er referat opprettet, ikke aktivitet med type samtalereferat
     SAMTALEREFERAT_FIKK_INNHOLD,
     SAMTALEREFERAT_DELT_MED_BRUKER,
     SAMTALEREFERAT_OPPRETTET_OG_DELT_MED_BRUKER,

diff --git a/src/test/kotlin/no/nav/veilarbaktivitet/aktivitet/AktivitetsplanControllerTest.kt b/src/test/kotlin/no/nav/veilarbaktivitet/aktivitet/AktivitetsplanControllerTest.kt
@@ -196,7 +196,7 @@ internal class AktivitetsplanControllerTest: SpringBootTestBase() {
         val aktivitet = aktivitetTestService.opprettAktivitet(
             happyBruker,
             veileder,
-            AktivitetDtoTestBuilder.nyAktivitet(AktivitetTypeDTO.MOTE).setErReferatPublisert(false).setReferat("")
+            AktivitetDtoTestBuilder.nyAktivitet(AktivitetTypeDTO.MOTE).setErReferatPublisert(false).setReferat(null)
         )
         val oppdatertAktivitet = aktivitet
         oppdatertAktivitet.setReferat("Et referat")