Disable patching - but keep code for now (#720)

navikt · Nov 10, 2023 · 3608720 · 3608720
1 parent 1493065
commit 3608720
Show file tree

Hide file tree

Showing 3 changed files with 27 additions and 33 deletions.
diff --git a/src/main/kotlin/no/nav/veilarbaktivitet/aktivitetskort/service/AktivitetskortService.kt b/src/main/kotlin/no/nav/veilarbaktivitet/aktivitetskort/service/AktivitetskortService.kt
@@ -52,9 +52,7 @@ class AktivitetskortService(
                             }
                             return UpsertActionResult.IGNORER
                         }
-                        arenaAktivitetskortService.oppdaterAktivitet(
-                            bestilling,
-                            gammelAktivitet)
+                        oppdaterAktivitet(gammelAktivitet, bestilling.toAktivitetsDataUpdate())
                     }
                     else -> oppdaterAktivitet(gammelAktivitet, bestilling.toAktivitetsDataUpdate())
                 }

diff --git a/src/main/kotlin/no/nav/veilarbaktivitet/aktivitetskort/service/ArenaAktivitetskortService.kt b/src/main/kotlin/no/nav/veilarbaktivitet/aktivitetskort/service/ArenaAktivitetskortService.kt
@@ -4,10 +4,7 @@ import lombok.extern.slf4j.Slf4j
 import no.nav.veilarbaktivitet.aktivitet.AktivitetDAO
 import no.nav.veilarbaktivitet.aktivitet.AktivitetService
 import no.nav.veilarbaktivitet.aktivitet.domain.AktivitetData
-import no.nav.veilarbaktivitet.aktivitet.domain.AktivitetStatus
-import no.nav.veilarbaktivitet.aktivitet.domain.AktivitetTransaksjonsType
 import no.nav.veilarbaktivitet.aktivitetskort.AktivitetIdMappingProducer
-import no.nav.veilarbaktivitet.aktivitetskort.AktivitetskortMapper.toAktivitet
 import no.nav.veilarbaktivitet.aktivitetskort.AktivitetskortMapper.toAktivitetsDataInsert
 import no.nav.veilarbaktivitet.aktivitetskort.bestilling.ArenaAktivitetskortBestilling
 import no.nav.veilarbaktivitet.aktivitetskort.dto.Aktivitetskort
@@ -17,11 +14,8 @@ import no.nav.veilarbaktivitet.arena.model.ArenaId
 import no.nav.veilarbaktivitet.avtalt_med_nav.AvtaltMedNavService
 import no.nav.veilarbaktivitet.avtalt_med_nav.ForhaandsorienteringDAO
 import no.nav.veilarbaktivitet.brukernotifikasjon.BrukerNotifikasjonDAO
-import no.nav.veilarbaktivitet.util.DateUtils
 import org.slf4j.LoggerFactory
 import org.springframework.stereotype.Service
-import java.time.ZoneOffset
-import java.time.ZonedDateTime
 
 @Slf4j
 @Service
@@ -97,28 +91,28 @@ class ArenaAktivitetskortService (
         return true
     }
 
-    fun oppdaterAktivitet(
-        bestilling: ArenaAktivitetskortBestilling,
-        gammelAktivitet: AktivitetData
-    ): AktivitetData {
-        val historiskTidspunkt = gammelAktivitet.eksternAktivitetData.oppfolgingsperiodeSlutt
-        val opprettetDato = DateUtils.dateToZonedDateTime(gammelAktivitet.opprettetDato)
-        val oppfolgingsperiode = bestilling.oppfolgingsperiode
-        val aktivitetsData = bestilling.toAktivitet(opprettetDato, historiskTidspunkt
-            ?.let { ZonedDateTime.of(it, ZoneOffset.UTC) } )
-            .withId(gammelAktivitet.id)
-            .withTransaksjonsType(AktivitetTransaksjonsType.OPPRETTET)
-            .withVersjon(gammelAktivitet.versjon)
-            .withOppfolgingsperiodeId(oppfolgingsperiode)
-            .withOpprettetDato(gammelAktivitet.opprettetDato)
-            .withFhoId(gammelAktivitet.fhoId)
-
-        val ferdigstatus = listOf(AktivitetStatus.AVBRUTT, AktivitetStatus.FULLFORT)
-        if (!ferdigstatus.contains(gammelAktivitet.status) && ferdigstatus.contains(aktivitetsData.status)) {
-            gammelAktivitet.fhoId?.let { avtaltMedNavService.settVarselFerdig(it) }
-        }
-
-        val opprettetAktivitetsData = aktivitetDAO.overskrivMenMedNyVersjon(aktivitetsData)
-        return opprettetAktivitetsData
-    }
+//    fun oppdaterAktivitet(
+//        bestilling: ArenaAktivitetskortBestilling,
+//        gammelAktivitet: AktivitetData
+//    ): AktivitetData {
+//        val historiskTidspunkt = gammelAktivitet.eksternAktivitetData.oppfolgingsperiodeSlutt
+//        val opprettetDato = DateUtils.dateToZonedDateTime(gammelAktivitet.opprettetDato)
+//        val oppfolgingsperiode = bestilling.oppfolgingsperiode
+//        val aktivitetsData = bestilling.toAktivitet(opprettetDato, historiskTidspunkt
+//            ?.let { ZonedDateTime.of(it, ZoneOffset.UTC) } )
+//            .withId(gammelAktivitet.id)
+//            .withTransaksjonsType(AktivitetTransaksjonsType.OPPRETTET)
+//            .withVersjon(gammelAktivitet.versjon)
+//            .withOppfolgingsperiodeId(oppfolgingsperiode)
+//            .withOpprettetDato(gammelAktivitet.opprettetDato)
+//            .withFhoId(gammelAktivitet.fhoId)
+//
+//        val ferdigstatus = listOf(AktivitetStatus.AVBRUTT, AktivitetStatus.FULLFORT)
+//        if (!ferdigstatus.contains(gammelAktivitet.status) && ferdigstatus.contains(aktivitetsData.status)) {
+//            gammelAktivitet.fhoId?.let { avtaltMedNavService.settVarselFerdig(it) }
+//        }
+//
+//        val opprettetAktivitetsData = aktivitetDAO.overskrivMenMedNyVersjon(aktivitetsData)
+//        return opprettetAktivitetsData
+//    }
 }
diff --git a/...st/kotlin/no/nav/veilarbaktivitet/aktivitetskort/AktivitetskortConsumerIntegrationTest.kt b/...st/kotlin/no/nav/veilarbaktivitet/aktivitetskort/AktivitetskortConsumerIntegrationTest.kt
@@ -46,6 +46,7 @@ import org.assertj.core.api.Assertions.assertThat
 import org.assertj.core.api.Assertions.within
 import org.junit.jupiter.api.Assertions.*
 import org.junit.jupiter.api.BeforeEach
+import org.junit.jupiter.api.Disabled
 import org.junit.jupiter.api.Test
 import org.mockito.Mockito
 import org.mockito.kotlin.any
@@ -906,6 +907,7 @@ internal class AktivitetskortConsumerIntegrationTest : SpringBootTestBase() {
     }
 
     @Test
+    @Disabled
     fun `relast skal overskrive første feilede migrering`() {
         val arenaaktivitetId = ArenaId("TA31212")
         val arenaAktivitetDTO =