fjerner generasjonId fra dtoer

Co-authored-by: Erik Maximilian Forsman <[email protected]>
navikt · Mar 7, 2024 · 78550ee · 78550ee
1 parent 9cc09cc
commit 78550ee
Show file tree

Hide file tree

Showing 7 changed files with 9 additions and 37 deletions.
diff --git a/fabrikk/src/main/kotlin/no/nav/helse/spekemat/fabrikk/Pølse.kt b/fabrikk/src/main/kotlin/no/nav/helse/spekemat/fabrikk/Pølse.kt
@@ -4,8 +4,6 @@ import java.util.*
 
 data class Pølse(
     val vedtaksperiodeId: UUID,
-    @Deprecated("", ReplaceWith("behandlingId"))
-    val generasjonId: UUID,
     val behandlingId: UUID,
     // hvorvidt behandlingen er åpen for endringer (dvs. til behandling) eller ikke (vedtak fattet / behandling avsluttet)
     private val status: Pølsestatus,
@@ -28,16 +26,14 @@ data class Pølse(
 
     fun dto() = PølseDto(
         vedtaksperiodeId = vedtaksperiodeId,
-        generasjonId = generasjonId,
         behandlingId = behandlingId,
         status = status,
         kilde = kilde
     )
 
     fun oppdaterPølse(vedtaksperiodeId: UUID, behandlingId: UUID, status: Pølsestatus): Pølse {
         if (this.vedtaksperiodeId != vedtaksperiodeId) return this
-        if (this.generasjonId != behandlingId) throw OppdatererEldreBehandlingException("Det er gjort forsøk på å oppdatere en generasjon som ikke samsvarer med den som er registrert i nyeste rad")
-        if (this.behandlingId != this.behandlingId) throw OppdatererEldreBehandlingException("Det er gjort forsøk på å oppdatere en behandling som ikke samsvarer med den som er registrert i nyeste rad")
+        if (this.behandlingId != behandlingId) throw OppdatererEldreBehandlingException("Det er gjort forsøk på å oppdatere en behandling som ikke samsvarer med den som er registrert i nyeste rad")
         return this.copy(status = status)
     }
 
@@ -48,7 +44,6 @@ data class Pølse(
     companion object {
         fun fraDto(dto: PølseDto) = Pølse(
             vedtaksperiodeId = dto.vedtaksperiodeId,
-            generasjonId = dto.generasjonId,
             behandlingId = dto.behandlingId,
             status = dto.status,
             kilde = dto.kilde

diff --git a/fabrikk/src/main/kotlin/no/nav/helse/spekemat/fabrikk/PølseDto.kt b/fabrikk/src/main/kotlin/no/nav/helse/spekemat/fabrikk/PølseDto.kt
@@ -4,8 +4,6 @@ import java.util.*
 
 data class PølseDto(
     val vedtaksperiodeId: UUID,
-    @Deprecated("", ReplaceWith("behandlingId"))
-    val generasjonId: UUID,
     val behandlingId: UUID,
     val status: Pølsestatus,
     // tingen som gjorde at behandlingen ble opprettet

diff --git a/fabrikk/src/test/kotlin/no/nav/helse/spekemat/fabrikk/PølseTest.kt b/fabrikk/src/test/kotlin/no/nav/helse/spekemat/fabrikk/PølseTest.kt
@@ -17,11 +17,11 @@ abstract class PølseTest {
         vedtaksperiodeId: UUID,
         kilde: UUID = UUID.randomUUID(),
         status: Pølsestatus = Pølsestatus.ÅPEN
-    ) = UUID.randomUUID().let { behandlingId -> Pølse(vedtaksperiodeId, behandlingId, behandlingId, status, kilde) }
+    ) = UUID.randomUUID().let { behandlingId -> Pølse(vedtaksperiodeId, behandlingId, status, kilde) }
 
     protected infix fun LocalDate.til(tom: LocalDate) = pølse(UUID.randomUUID())
     protected infix fun Pølse.som(vedtaksperiodeId: UUID) = this.copy(vedtaksperiodeId = vedtaksperiodeId)
-    protected fun Pølse.nyBehandling(behandlingId: UUID = UUID.randomUUID(), kilde: UUID = UUID.randomUUID()) = fordi(kilde).copy(behandlingId = behandlingId, generasjonId = behandlingId)
+    protected fun Pølse.nyBehandling(behandlingId: UUID = UUID.randomUUID(), kilde: UUID = UUID.randomUUID()) = fordi(kilde).copy(behandlingId = behandlingId)
     protected infix fun Pølse.fordi(kilde: UUID) = this.copy(kilde = kilde)
 
     protected fun Pølse.lukket() = this.copy(status = Pølsestatus.LUKKET)

diff --git a/foredler/src/main/kotlin/no/nav/helse/spekemat/foredler/Api.kt b/foredler/src/main/kotlin/no/nav/helse/spekemat/foredler/Api.kt
@@ -39,8 +39,7 @@ fun Route.api(pølsetjeneste: Pølsetjeneste) {
             ))
             val pølse = PølseDto(
                 vedtaksperiodeId = request.pølse.vedtaksperiodeId,
-                generasjonId = checkNotNull(request.pølse.generasjonId ?: request.pølse.behandlingId),
-                behandlingId = checkNotNull(request.pølse.generasjonId ?: request.pølse.behandlingId),
+                behandlingId = request.pølse.behandlingId,
                 status = Pølsestatus.ÅPEN,
                 kilde = request.pølse.kilde
             )
@@ -72,7 +71,7 @@ fun Route.api(pølsetjeneste: Pølsetjeneste) {
                     request.fnr,
                     request.yrkesaktivitetidentifikator,
                     request.vedtaksperiodeId,
-                    checkNotNull(request.generasjonId ?: request.behandlingId),
+                    request.behandlingId,
                     status,
                     request.meldingsreferanseId,
                     request.hendelsedata,
@@ -111,36 +110,20 @@ data class NyPølseRequest(
 )
 data class NyPølseDto(
     val vedtaksperiodeId: UUID,
-    @Deprecated("", ReplaceWith("behandlingId"))
-    val generasjonId: UUID?,
-    val behandlingId: UUID?,
+    val behandlingId: UUID,
     // tingen som gjorde at behandlingen ble opprettet
     val kilde: UUID
-) {
-    init {
-        check(generasjonId != null || behandlingId != null) {
-            "Må ha enten generasjonId eller behandlingId"
-        }
-    }
-}
+)
 enum class PølsestatusDto { ÅPEN, LUKKET, FORKASTET }
 data class OppdaterPølseRequest(
     val fnr: String,
     val yrkesaktivitetidentifikator: String,
     val vedtaksperiodeId: UUID,
-    @Deprecated("", ReplaceWith("behandlingId"))
-    val generasjonId: UUID?,
-    val behandlingId: UUID?,
+    val behandlingId: UUID,
     val status: PølsestatusDto,
     val meldingsreferanseId: UUID,
     val hendelsedata: String
-) {
-    init {
-        check(generasjonId != null || behandlingId != null) {
-            "Må ha enten generasjonId eller behandlingId"
-        }
-    }
-}
+)
 
 data class YrkesaktivitetDto(
     val yrkesaktivitetidentifikator: String,

diff --git a/foredler/src/main/kotlin/no/nav/helse/spekemat/foredler/PølseDao.kt b/foredler/src/main/kotlin/no/nav/helse/spekemat/foredler/PølseDao.kt
@@ -160,7 +160,6 @@ class PølseDao(private val dataSource: DatasourceProvider) {
                 pølser = rad.pølser.map { pølse ->
                     PølseDto(
                         vedtaksperiodeId = pølse.vedtaksperiodeId,
-                        generasjonId = pølse.generasjonId,
                         behandlingId = pølse.generasjonId,
                         status = when (pølse.status) {
                             PølseradDbDto.PølseDbDto.PølseDbstatus.ÅPEN -> Pølsestatus.ÅPEN

diff --git a/foredler/src/main/kotlin/no/nav/helse/spekemat/foredler/SpleisClient.kt b/foredler/src/main/kotlin/no/nav/helse/spekemat/foredler/SpleisClient.kt
@@ -85,7 +85,6 @@ class SpleisClient(
     private fun mapPølse(pølse: SpleisPeriodeResponse): PølseDto {
         return PølseDto(
             vedtaksperiodeId = pølse.vedtaksperiodeId,
-            generasjonId = pølse.generasjonId,
             behandlingId = pølse.generasjonId,
             status = parseStatus(pølse.periodetilstand),
             kilde = pølse.kilde

diff --git a/foredler/src/test/kotlin/no/nav/helse/spekemat/foredler/E2ETest.kt b/foredler/src/test/kotlin/no/nav/helse/spekemat/foredler/E2ETest.kt
@@ -431,7 +431,6 @@ private class TestContext(
                 yrkesaktivitetidentifikator = yrkesaktivitetidentifikator,
                 pølse = NyPølseDto(
                     vedtaksperiodeId = vedtaksperiodeId,
-                    generasjonId = behandlingId,
                     behandlingId = behandlingId,
                     kilde = kildeId
                 ),
@@ -457,7 +456,6 @@ private class TestContext(
                 fnr = fnr,
                 yrkesaktivitetidentifikator = yrkesaktivitetidentifikator,
                 vedtaksperiodeId = vedtaksperiodeId,
-                generasjonId = behandlingId,
                 behandlingId = behandlingId,
                 status = status,
                 meldingsreferanseId = hendelseId,