Rename getLastDate -> getLastInclusiveDate

navikt · Feb 26, 2025 · 84e008e · 84e008e
1 parent e9ab578
commit 84e008e
Show file tree

Hide file tree

Showing 12 changed files with 18 additions and 19 deletions.
diff --git a/common/domain/src/main/kotlin/no/nav/mulighetsrommet/model/Periode.kt b/common/domain/src/main/kotlin/no/nav/mulighetsrommet/model/Periode.kt
@@ -42,14 +42,14 @@ data class Periode(
     }
 
     operator fun contains(periode: Periode): Boolean {
-        return periode.start in this && periode.getLastDate() in this
+        return periode.start in this && periode.getLastInclusiveDate() in this
     }
 
     fun getDurationInDays(): Long {
         return ChronoUnit.DAYS.between(start, slutt)
     }
 
-    fun getLastDate(): LocalDate {
+    fun getLastInclusiveDate(): LocalDate {
         return slutt.minusDays(1)
     }
 

diff --git a/common/domain/src/test/kotlin/no/nav/mulighetsrommet/model/PeriodeTest.kt b/common/domain/src/test/kotlin/no/nav/mulighetsrommet/model/PeriodeTest.kt
@@ -26,7 +26,7 @@ class PeriodeTest : FunSpec({
     test("should get last date") {
         val period = Periode(LocalDate.of(2021, 1, 1), LocalDate.of(2021, 1, 31))
 
-        period.getLastDate() shouldBe LocalDate.of(2021, 1, 30)
+        period.getLastInclusiveDate() shouldBe LocalDate.of(2021, 1, 30)
     }
 
     test("should check if date is in period") {

diff --git a/...ommet-api/src/main/kotlin/no/nav/mulighetsrommet/api/arrangorflate/ArrangorflateRoutes.kt b/...ommet-api/src/main/kotlin/no/nav/mulighetsrommet/api/arrangorflate/ArrangorflateRoutes.kt
@@ -300,7 +300,7 @@ suspend fun toArrFlateUtbetaling(
             gjennomforing = utbetaling.gjennomforing,
             arrangor = utbetaling.arrangor,
             periodeStart = utbetaling.periode.start,
-            periodeSlutt = utbetaling.periode.getLastDate(),
+            periodeSlutt = utbetaling.periode.getLastInclusiveDate(),
             beregning = Beregning.Forhandsgodkjent(
                 antallManedsverk = antallManedsverk,
                 belop = beregning.output.belop,
@@ -319,7 +319,7 @@ suspend fun toArrFlateUtbetaling(
         gjennomforing = utbetaling.gjennomforing,
         arrangor = utbetaling.arrangor,
         periodeStart = utbetaling.periode.start,
-        periodeSlutt = utbetaling.periode.getLastDate(),
+        periodeSlutt = utbetaling.periode.getLastInclusiveDate(),
         beregning = Beregning.Fri(
             belop = beregning.output.belop,
             digest = beregning.getDigest(),

diff --git a/...c/main/kotlin/no/nav/mulighetsrommet/api/arrangorflate/model/ArrFlateUtbetalingKompakt.kt b/...c/main/kotlin/no/nav/mulighetsrommet/api/arrangorflate/model/ArrFlateUtbetalingKompakt.kt
@@ -34,7 +34,7 @@ data class ArrFlateUtbetalingKompakt(
             gjennomforing = utbetaling.gjennomforing,
             arrangor = utbetaling.arrangor,
             periodeStart = utbetaling.periode.start,
-            periodeSlutt = utbetaling.periode.getLastDate(),
+            periodeSlutt = utbetaling.periode.getLastInclusiveDate(),
             belop = utbetaling.beregning.output.belop,
         )
     }

diff --git a/...mmet-api/src/main/kotlin/no/nav/mulighetsrommet/api/gjennomforing/GjennomforingService.kt b/...mmet-api/src/main/kotlin/no/nav/mulighetsrommet/api/gjennomforing/GjennomforingService.kt
@@ -224,7 +224,7 @@ class GjennomforingService(
                 "Registrerte stengt hos arrangør i perioden",
                 periode.start.formaterDatoTilEuropeiskDatoformat(),
                 "-",
-                periode.getLastDate().formaterDatoTilEuropeiskDatoformat(),
+                periode.getLastInclusiveDate().formaterDatoTilEuropeiskDatoformat(),
             ).joinToString(separator = " ")
             logEndring(operation, dto, bruker)
             dto

diff --git a/mulighetsrommet-api/src/main/kotlin/no/nav/mulighetsrommet/api/tilsagn/TilsagnRoutes.kt b/mulighetsrommet-api/src/main/kotlin/no/nav/mulighetsrommet/api/tilsagn/TilsagnRoutes.kt
@@ -148,7 +148,7 @@ fun Route.tilsagnRoutes() {
         val satser = ForhandsgodkjenteSatser.satser(tiltakstype).map {
             AvtaltSats(
                 periodeStart = it.periode.start,
-                periodeSlutt = it.periode.getLastDate(),
+                periodeSlutt = it.periode.getLastInclusiveDate(),
                 pris = it.belop,
                 valuta = "NOK",
             )

diff --git a/mulighetsrommet-api/src/main/kotlin/no/nav/mulighetsrommet/api/tilsagn/TilsagnValidator.kt b/mulighetsrommet-api/src/main/kotlin/no/nav/mulighetsrommet/api/tilsagn/TilsagnValidator.kt
@@ -23,7 +23,7 @@ object TilsagnValidator {
         }
 
         val errors = buildList {
-            if (next.periode.start.year != next.periode.getLastDate().year) {
+            if (next.periode.start.year != next.periode.getLastInclusiveDate().year) {
                 add(
                     FieldError.of(
                         TilsagnRequest::periodeSlutt,

diff --git a/mulighetsrommet-api/src/main/kotlin/no/nav/mulighetsrommet/api/tilsagn/db/TilsagnQueries.kt b/mulighetsrommet-api/src/main/kotlin/no/nav/mulighetsrommet/api/tilsagn/db/TilsagnQueries.kt
@@ -157,7 +157,7 @@ class TilsagnQueries(private val session: Session) {
         val params = mapOf(
             "gjennomforing_id" to gjennomforingId,
             "periode_start" to periode.start,
-            "periode_slutt" to periode.getLastDate(),
+            "periode_slutt" to periode.getLastInclusiveDate(),
         )
 
         return session.list(queryOf(query, params)) { it.toTilsagnDto() }
@@ -179,7 +179,7 @@ class TilsagnQueries(private val session: Session) {
         val params = mapOf(
             "gjennomforing_id" to gjennomforingId,
             "periode_start" to periode.start,
-            "periode_slutt" to periode.getLastDate(),
+            "periode_slutt" to periode.getLastInclusiveDate(),
         )
 
         return session.list(queryOf(query, params)) { it.toArrangorflateTilsagn() }

diff --git a/...ghetsrommet-api/src/main/kotlin/no/nav/mulighetsrommet/api/utbetaling/UtbetalingRoutes.kt b/...ghetsrommet-api/src/main/kotlin/no/nav/mulighetsrommet/api/utbetaling/UtbetalingRoutes.kt
@@ -21,7 +21,6 @@ import no.nav.mulighetsrommet.api.tilsagn.model.Besluttelse
 import no.nav.mulighetsrommet.api.utbetaling.model.*
 import no.nav.mulighetsrommet.model.Kid
 import no.nav.mulighetsrommet.model.Kontonummer
-import no.nav.mulighetsrommet.model.Periode
 import no.nav.mulighetsrommet.serializers.LocalDateSerializer
 import no.nav.mulighetsrommet.serializers.LocalDateTimeSerializer
 import no.nav.mulighetsrommet.serializers.UUIDSerializer
@@ -210,7 +209,7 @@ data class UtbetalingKompakt(
             status = utbetaling.status,
             beregning = Beregning(
                 periodeStart = utbetaling.periode.start,
-                periodeSlutt = utbetaling.periode.getLastDate(),
+                periodeSlutt = utbetaling.periode.getLastInclusiveDate(),
                 belop = utbetaling.beregning.output.belop,
             ),
             godkjentAvArrangorTidspunkt = utbetaling.godkjentAvArrangorTidspunkt,

diff --git a/mulighetsrommet-api/src/test/kotlin/no/nav/mulighetsrommet/api/tilsagn/TilsagnQueriesTest.kt b/mulighetsrommet-api/src/test/kotlin/no/nav/mulighetsrommet/api/tilsagn/TilsagnQueriesTest.kt
@@ -208,7 +208,7 @@ class TilsagnQueriesTest : FunSpec({
                         navn = TiltakstypeFixtures.AFT.navn,
                     ),
                     periodeStart = periodeMedTilsagn.start,
-                    periodeSlutt = periodeMedTilsagn.getLastDate(),
+                    periodeSlutt = periodeMedTilsagn.getLastInclusiveDate(),
                     arrangor = ArrangorflateTilsagn.Arrangor(
                         navn = ArrangorFixtures.underenhet1.navn,
                         id = ArrangorFixtures.underenhet1.id,

diff --git a/mulighetsrommet-api/src/test/kotlin/no/nav/mulighetsrommet/api/tilsagn/TilsagnServiceTest.kt b/mulighetsrommet-api/src/test/kotlin/no/nav/mulighetsrommet/api/tilsagn/TilsagnServiceTest.kt
@@ -348,7 +348,7 @@ class TilsagnServiceTest : FunSpec({
                     gjennomforingId = Tilsagn1.gjennomforingId,
                     type = Tilsagn1.type,
                     periodeStart = Tilsagn1.periode.start,
-                    periodeSlutt = Tilsagn1.periode.getLastDate(),
+                    periodeSlutt = Tilsagn1.periode.getLastInclusiveDate(),
                     kostnadssted = Tilsagn1.kostnadssted,
                     beregning = Tilsagn1.beregning.input,
                 ),
@@ -384,7 +384,7 @@ class TilsagnServiceTest : FunSpec({
                     gjennomforingId = Tilsagn1.gjennomforingId,
                     type = Tilsagn1.type,
                     periodeStart = Tilsagn1.periode.start,
-                    periodeSlutt = Tilsagn1.periode.getLastDate(),
+                    periodeSlutt = Tilsagn1.periode.getLastInclusiveDate(),
                     kostnadssted = Tilsagn1.kostnadssted,
                     beregning = Tilsagn1.beregning.input,
                 ),
@@ -407,7 +407,7 @@ class TilsagnServiceTest : FunSpec({
                     gjennomforingId = Tilsagn1.gjennomforingId,
                     type = Tilsagn1.type,
                     periodeStart = Tilsagn1.periode.start,
-                    periodeSlutt = Tilsagn1.periode.getLastDate(),
+                    periodeSlutt = Tilsagn1.periode.getLastInclusiveDate(),
                     kostnadssted = Tilsagn1.kostnadssted,
                     beregning = TilsagnBeregningFri.Input(belop = 7),
                 ),

diff --git a/mulighetsrommet-tiltaksokonomi/src/main/kotlin/no/nav/tiltak/okonomi/oebs/OebsService.kt b/mulighetsrommet-tiltaksokonomi/src/main/kotlin/no/nav/tiltak/okonomi/oebs/OebsService.kt
@@ -149,7 +149,7 @@ private fun toOebsBestillingMelding(
             antall = linje.belop,
             periode = linje.periode.start.monthValue.toString().padStart(2, '0'),
             startDato = linje.periode.start,
-            sluttDato = linje.periode.getLastDate(),
+            sluttDato = linje.periode.getLastInclusiveDate(),
         )
     }
 
@@ -165,7 +165,7 @@ private fun toOebsBestillingMelding(
         saksbehandler = bestilling.behandletAv.part,
         bdmGodkjenner = bestilling.behandletAv.part,
         startDato = bestilling.periode.start,
-        sluttDato = bestilling.periode.getLastDate(),
+        sluttDato = bestilling.periode.getLastInclusiveDate(),
         bestillingsLinjer = linjer,
         statsregnskapsKonto = kontering.statligRegnskapskonto,
         artsKonto = kontering.statligArtskonto,