bugfiks: behandling har preferanse over å vise søknad. Sorterer på opprettet

Author: ramnav990

app/src/main/kotlin/no/nav/tiltakspenger/vedtak/repository/benk/BenkOversiktPostgresRepo.kt

@@ -78,6 +78,7 @@ class BenkOversiktPostgresRepo(
                             saksbehandler = saksbehandler,
                             beslutter = beslutter,
                             sakId = row.stringOrNull("sak_id")?.let { SakId.fromString(it) },
+                            opprettet = row.localDateTime("opprettet"),
                         )
                     }.asList,
                 )
@@ -116,6 +117,7 @@ class BenkOversiktPostgresRepo(
                             saksbehandler = null,
                             beslutter = null,
                             sakId = row.stringOrNull("sak_id")?.let { SakId.fromString(it) },
+                            opprettet = opprettet,
                         )
                     }.asList,
                 )

app/src/main/kotlin/no/nav/tiltakspenger/vedtak/routes/sak/SakDTO.kt

@@ -22,10 +22,9 @@ fun Sak.toDTO() = SakDTO(
     saksnummer = saksnummer.verdi,
     sakId = id.toString(),
     fnr = fnr.verdi,
-    // vi kan enten sende med søknadene til frontend, så kan den gjøre dette. Den vil ta også ha alle søknadene hvis den vil gjøre noe videre
-    // ellers bare gjør vi det her
-    behandlingsoversikt = behandlinger.filterNot { behandling -> this.soknader.any { it.id == behandling.søknad?.id } }
-        .toSaksoversiktDTO() + this.soknader.toSaksoversiktDTO(),
+    behandlingsoversikt = behandlinger.toSaksoversiktDTO() +
+        this.soknader.filter { soknad -> behandlinger.none { it.søknad?.id == soknad.id } }
+            .toSaksoversiktDTO().sortedBy { it.opprettet },
     meldeperiodeKjeder = toMeldeperiodeKjederDTO(),
     førsteLovligeStansdato = førsteLovligeStansdato(),
     sisteDagSomGirRett = sisteDagSomGirRett,

app/src/main/kotlin/no/nav/tiltakspenger/vedtak/routes/sak/SaksoversiktDTO.kt

@@ -28,6 +28,7 @@ data class SaksoversiktDTO(
     val sakId: String?,
     val saksbehandler: String?,
     val beslutter: String?,
+    val opprettet: String,
 )
 
 internal fun Saksoversikt.toDTO(): List<SaksoversiktDTO> = this.map { it.toSaksoversiktDTO() }
@@ -48,6 +49,7 @@ fun BehandlingEllerSøknadForSaksoversikt.toSaksoversiktDTO() = SaksoversiktDTO(
     saksbehandler = saksbehandler,
     beslutter = beslutter,
     sakId = sakId.toString(),
+    opprettet = this.opprettet.toString(),
 )
 
 fun List<Behandling>.toSaksoversiktDTO(): List<SaksoversiktDTO> =
@@ -69,6 +71,7 @@ fun Behandling.toSaksoversiktDTO() = SaksoversiktDTO(
     sakId = sakId.toString(),
     saksbehandler = saksbehandler,
     beslutter = beslutter,
+    opprettet = this.opprettet.toString(),
 )
 
 fun Søknad.toSaksoversiktDTO() = SaksoversiktDTO(
@@ -83,4 +86,5 @@ fun Søknad.toSaksoversiktDTO() = SaksoversiktDTO(
     sakId = this.sakId.toString(),
     saksbehandler = null,
     beslutter = null,
+    opprettet = this.opprettet.toString(),
 )

app/src/test/kotlin/no/nav/tiltakspenger/vedtak/repository/benk/BenkOversiktPostgresRepoTest.kt

@@ -45,6 +45,7 @@ class BenkOversiktPostgresRepoTest {
                                 underkjent = false,
                                 kravtidspunkt = LocalDateTime.from(1.januarDateTime(2022)),
                                 id = søknad1.id,
+                                opprettet = søknad1.opprettet,
                             ),
                             BehandlingEllerSøknadForSaksoversikt(
                                 periode = null,
@@ -58,6 +59,7 @@ class BenkOversiktPostgresRepoTest {
                                 underkjent = false,
                                 kravtidspunkt = LocalDateTime.from(1.januarDateTime(2022)),
                                 id = førstegangsBehandlingSak.førstegangsbehandling!!.id,
+                                opprettet = førstegangsBehandlingSak.førstegangsbehandling!!.opprettet,
                             ),
                             BehandlingEllerSøknadForSaksoversikt(
                                 periode = null,
@@ -71,6 +73,7 @@ class BenkOversiktPostgresRepoTest {
                                 underkjent = false,
                                 kravtidspunkt = null,
                                 id = revurderingSak.revurderinger.first().id,
+                                opprettet = revurderingSak.revurderinger.first().opprettet,
                             ),
                         ),
                     )
@@ -103,6 +106,7 @@ class BenkOversiktPostgresRepoTest {
                                 underkjent = false,
                                 kravtidspunkt = LocalDateTime.from(1.januarDateTime(2022)),
                                 id = sak.førstegangsbehandling!!.id,
+                                opprettet = sak.førstegangsbehandling!!.opprettet,
                             ),
                         ),
                     )

app/src/test/kotlin/no/nav/tiltakspenger/vedtak/routes/sak/SakDtoTest.kt

@@ -2,6 +2,7 @@ package no.nav.tiltakspenger.vedtak.routes.sak
 
 import io.kotest.matchers.shouldBe
 import no.nav.tiltakspenger.objectmothers.ObjectMother
+import no.nav.tiltakspenger.vedtak.routes.behandling.dto.BehandlingstypeDTO
 import org.junit.jupiter.api.Test
 
 class SakDtoTest {
@@ -13,6 +14,13 @@ class SakDtoTest {
         // TODO - burde muligens ha en sak.nySøknad()
         val sakMedSøknadOgBehandling = sak.copy(soknader = sak.soknader + nySøknad)
 
-        sakMedSøknadOgBehandling.toDTO().behandlingsoversikt.size shouldBe 2
+        val actual = sakMedSøknadOgBehandling.toDTO()
+        actual.behandlingsoversikt.size shouldBe 2
+        actual.behandlingsoversikt.first().let {
+            it.typeBehandling shouldBe BehandlingstypeDTO.FØRSTEGANGSBEHANDLING
+        }
+        actual.behandlingsoversikt.last().let {
+            it.typeBehandling shouldBe BehandlingstypeDTO.SØKNAD
+        }
     }
 }

domene/src/main/kotlin/no/nav/tiltakspenger/saksbehandling/domene/benk/Saksoversikt.kt

@@ -35,6 +35,7 @@ data class BehandlingEllerSøknadForSaksoversikt(
     val saksbehandler: String?,
     val beslutter: String?,
     val sakId: SakId?,
+    val opprettet: LocalDateTime,
 ) {
     sealed interface Status {
         data object Søknad : Status

test-common/src/main/kotlin/no/nav/tiltakspenger/fakes/repos/SaksoversiktFakeRepo.kt

@@ -28,6 +28,7 @@ class SaksoversiktFakeRepo(
                 saksbehandler = behandling.saksbehandler,
                 beslutter = behandling.beslutter,
                 sakId = behandling.sakId,
+                opprettet = behandling.opprettet,
             )
         }
     }
@@ -47,6 +48,7 @@ class SaksoversiktFakeRepo(
                 saksbehandler = null,
                 beslutter = null,
                 sakId = null,
+                opprettet = søknad.opprettet,
             )
         }
     }