Return all oppfolgingsperioder, also when no aktiviteter

navikt · Dec 19, 2024 · 853de96 · 853de96
1 parent fb119c3
commit 853de96
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 9 deletions.
diff --git a/src/main/kotlin/no/nav/veilarbaktivitet/aktivitetskort/graphql/AktivitetskortController.kt b/src/main/kotlin/no/nav/veilarbaktivitet/aktivitetskort/graphql/AktivitetskortController.kt
@@ -43,13 +43,12 @@ class AktivitetskortController(
         val aktorId = aktorOppslagClient.hentAktorId(eksternBrukerId.otherFnr())
         val oppfolgingsPerioder = oppfolgingsperiodeService.hentOppfolgingsPerioder(Person.AktorId(aktorId.get()))
         val aktiviteter = getAktiviteter(eksternBrukerId)
-            .groupBy { it.oppfolgingsperiodeId }
-            .toList()
-            .map { (oppfolgingPeriodeId, aktiviteter) ->
-                oppfolgingsPerioder.first { it.oppfolgingsperiodeId.equals(oppfolgingPeriodeId) }
-                    .let { OppfolgingsPeriode(oppfolgingPeriodeId, aktiviteter, it.startTid, it.sluttTid) }
+        return oppfolgingsPerioder
+            .map { periode ->
+                val periodeAktiviteter = aktiviteter.filter { it.oppfolgingsperiodeId == periode.oppfolgingsperiodeId }
+                OppfolgingsPeriode(periode.oppfolgingsperiodeId, periodeAktiviteter, periode.startTid, periode.sluttTid)
             }
-        return aktiviteter
+            .sortedByDescending { it.start }
     }
 
     private fun getAktiviteter(userFnr: Person): List<AktivitetDTO> {

diff --git a/...est/kotlin/no/nav/veilarbaktivitet/aktivitetskort/graphql/AktivitetskortControllerTest.kt b/...est/kotlin/no/nav/veilarbaktivitet/aktivitetskort/graphql/AktivitetskortControllerTest.kt
@@ -57,8 +57,8 @@ class AktivitetskortControllerTest: SpringBootTestBase() {
         )
         aktivitetTestService.opprettAktivitet(mockBruker, mockBruker, aktivitet)
         val result = aktivitetTestService.queryAktivitetskort(mockBruker, mockBruker, query)
-        val nyestePeriode = result.data?.perioder?.last()
-        val gammelPeriode = result.data?.perioder?.first()
+        val nyestePeriode = result.data?.perioder?.first()
+        val gammelPeriode = result.data?.perioder?.last()
         assertThat(result.errors).isNull()
         assertThat(result.data?.perioder).hasSize(2)
         assertThat(nyestePeriode?.start).isCloseTo(mockBruker.oppfolgingsperioder.first().startTid, within(1, ChronoUnit.MILLIS))
@@ -129,7 +129,7 @@ class AktivitetskortControllerTest: SpringBootTestBase() {
         val jobbAktivitet = AktivitetDtoTestBuilder.nyAktivitet(AktivitetTypeDTO.IJOBB)
         aktivitetTestService.opprettAktivitet(kvpBruker, kvpBruker, jobbAktivitet)
         val result = aktivitetTestService.queryAktivitetskort(kvpBruker, veileder, query)
-        assertThat(result.data?.perioder).isEmpty()
+        assertThat(result.data?.perioder?.firstOrNull()?.aktiviteter).isEmpty()
         assertThat(result.errors).isNull()
     }