Fjerner endepunkt for etterbetalingsbelop som ikke er i bruk

navikt · Sep 16, 2023 · 4b780d5 · 4b780d5
1 parent acb55de
commit 4b780d5
Show file tree

Hide file tree

Showing 7 changed files with 8 additions and 224 deletions.
diff --git a/src/main/kotlin/no/nav/familie/oppdrag/rest/SimuleringController.kt b/src/main/kotlin/no/nav/familie/oppdrag/rest/SimuleringController.kt
@@ -2,7 +2,6 @@ package no.nav.familie.oppdrag.rest
 
 import jakarta.validation.Valid
 import no.nav.familie.kontrakter.felles.Ressurs
-import no.nav.familie.kontrakter.felles.oppdrag.RestSimulerResultat
 import no.nav.familie.kontrakter.felles.oppdrag.Utbetalingsoppdrag
 import no.nav.familie.kontrakter.felles.simulering.DetaljertSimuleringResultat
 import no.nav.familie.kontrakter.felles.simulering.FeilutbetalingerFraSimulering
@@ -22,21 +21,16 @@ import org.springframework.web.bind.annotation.RequestMapping
 import org.springframework.web.bind.annotation.RestController
 
 @RestController
-@RequestMapping("/api/simulering", consumes = [MediaType.APPLICATION_JSON_VALUE], produces = [MediaType.APPLICATION_JSON_VALUE])
+@RequestMapping(
+    "/api/simulering",
+    consumes = [MediaType.APPLICATION_JSON_VALUE],
+    produces = [MediaType.APPLICATION_JSON_VALUE],
+)
 @ProtectedWithClaims(issuer = "azuread")
 class SimuleringController(@Autowired val simuleringTjeneste: SimuleringTjeneste) {
 
     val logger: Logger = LoggerFactory.getLogger(SimuleringController::class.java)
 
-    @PostMapping(path = ["/etterbetalingsbelop"])
-    fun hentEtterbetalingsbeløp(
-        @Valid @RequestBody
-        utbetalingsoppdrag: Utbetalingsoppdrag,
-    ): ResponseEntity<Ressurs<RestSimulerResultat>> {
-        logger.info("Hente simulert etterbetaling for saksnr ${utbetalingsoppdrag.saksnummer}")
-        return ok(simuleringTjeneste.utførSimulering(utbetalingsoppdrag))
-    }
-
     @PostMapping(path = ["/v1"])
     fun utførSimuleringOgHentResultat(
         @Valid @RequestBody

diff --git a/src/main/kotlin/no/nav/familie/oppdrag/simulering/SimulerBeregningResponseMapper.kt b/src/main/kotlin/no/nav/familie/oppdrag/simulering/SimulerBeregningResponseMapper.kt
diff --git a/src/main/kotlin/no/nav/familie/oppdrag/simulering/SimuleringTjeneste.kt b/src/main/kotlin/no/nav/familie/oppdrag/simulering/SimuleringTjeneste.kt
@@ -1,6 +1,5 @@
 package no.nav.familie.oppdrag.simulering
 
-import no.nav.familie.kontrakter.felles.oppdrag.RestSimulerResultat
 import no.nav.familie.kontrakter.felles.oppdrag.Utbetalingsoppdrag
 import no.nav.familie.kontrakter.felles.simulering.DetaljertSimuleringResultat
 import no.nav.familie.kontrakter.felles.simulering.FeilutbetalingerFraSimulering
@@ -9,7 +8,6 @@ import no.nav.system.os.tjenester.simulerfpservice.simulerfpservicegrensesnitt.S
 
 interface SimuleringTjeneste {
 
-    fun utførSimulering(utbetalingsoppdrag: Utbetalingsoppdrag): RestSimulerResultat
     fun utførSimuleringOghentDetaljertSimuleringResultat(utbetalingsoppdrag: Utbetalingsoppdrag): DetaljertSimuleringResultat
     fun hentSimulerBeregningResponse(utbetalingsoppdrag: Utbetalingsoppdrag): SimulerBeregningResponse
     fun hentFeilutbetalinger(request: HentFeilutbetalingerFraSimuleringRequest): FeilutbetalingerFraSimulering

diff --git a/src/main/kotlin/no/nav/familie/oppdrag/simulering/SimuleringTjenesteE2E.kt b/src/main/kotlin/no/nav/familie/oppdrag/simulering/SimuleringTjenesteE2E.kt
@@ -1,6 +1,5 @@
 package no.nav.familie.oppdrag.simulering
 
-import no.nav.familie.kontrakter.felles.oppdrag.RestSimulerResultat
 import no.nav.familie.kontrakter.felles.oppdrag.Utbetalingsoppdrag
 import no.nav.familie.kontrakter.felles.simulering.DetaljertSimuleringResultat
 import no.nav.familie.kontrakter.felles.simulering.FeilutbetalingerFraSimulering
@@ -14,8 +13,6 @@ import org.springframework.web.context.annotation.ApplicationScope
 @ApplicationScope
 @Profile("e2e")
 class SimuleringTjenesteE2E : SimuleringTjeneste {
-
-    override fun utførSimulering(utbetalingsoppdrag: Utbetalingsoppdrag): RestSimulerResultat = RestSimulerResultat(0)
     override fun utførSimuleringOghentDetaljertSimuleringResultat(utbetalingsoppdrag: Utbetalingsoppdrag): DetaljertSimuleringResultat =
         DetaljertSimuleringResultat(simuleringMottaker = emptyList())
 

diff --git a/src/main/kotlin/no/nav/familie/oppdrag/simulering/SimuleringTjenesteImpl.kt b/src/main/kotlin/no/nav/familie/oppdrag/simulering/SimuleringTjenesteImpl.kt
@@ -1,12 +1,12 @@
 package no.nav.familie.oppdrag.simulering
 
 import com.fasterxml.jackson.module.kotlin.jacksonObjectMapper
-import no.nav.familie.kontrakter.felles.oppdrag.RestSimulerResultat
 import no.nav.familie.kontrakter.felles.oppdrag.Utbetalingsoppdrag
 import no.nav.familie.kontrakter.felles.simulering.DetaljertSimuleringResultat
 import no.nav.familie.kontrakter.felles.simulering.FeilutbetalingerFraSimulering
 import no.nav.familie.kontrakter.felles.simulering.FeilutbetaltPeriode
 import no.nav.familie.kontrakter.felles.simulering.HentFeilutbetalingerFraSimuleringRequest
+import no.nav.familie.kontrakter.felles.simulering.PosteringType
 import no.nav.familie.oppdrag.common.logSoapFaultException
 import no.nav.familie.oppdrag.config.FinnesIkkeITps
 import no.nav.familie.oppdrag.config.IntegrasjonException
@@ -41,10 +41,6 @@ class SimuleringTjenesteImpl(
     val mapper = jacksonObjectMapper()
     val simuleringResultatTransformer = SimuleringResultatTransformer()
 
-    override fun utførSimulering(utbetalingsoppdrag: Utbetalingsoppdrag): RestSimulerResultat {
-        return hentSimulerBeregningResponse(utbetalingsoppdrag).toRestSimulerResult()
-    }
-
     override fun hentSimulerBeregningResponse(utbetalingsoppdrag: Utbetalingsoppdrag): SimulerBeregningResponse {
         val simulerBeregningRequest = simulerBeregningRequestMapper.tilSimulerBeregningRequest(utbetalingsoppdrag)
 
@@ -130,7 +126,7 @@ class SimuleringTjenesteImpl(
         return simulering.beregningsPeriode.map { beregningsperiode ->
             beregningsperiode to beregningsperiode.beregningStoppnivaa.map { stoppNivå ->
                 stoppNivå.beregningStoppnivaaDetaljer.filter { detalj ->
-                    detalj.typeKlasse == TypeKlasse.FEIL.name &&
+                    detalj.typeKlasse == PosteringType.FEILUTBETALING.kode &&
                         detalj.belop > BigDecimal.ZERO
                 }
             }.flatten()
@@ -141,7 +137,7 @@ class SimuleringTjenesteImpl(
         return simulering.beregningsPeriode.associateWith { beregningsperiode ->
             beregningsperiode.beregningStoppnivaa.map { stoppNivå ->
                 stoppNivå.beregningStoppnivaaDetaljer.filter { detalj ->
-                    detalj.typeKlasse == TypeKlasse.YTEL.name
+                    detalj.typeKlasse == PosteringType.YTELSE.kode
                 }
             }.flatten()
         }

diff --git a/src/test/kotlin/no/nav/familie/oppdrag/rest/SimuleringControllerIntegrationTest.kt b/src/test/kotlin/no/nav/familie/oppdrag/rest/SimuleringControllerIntegrationTest.kt
diff --git a/src/test/kotlin/no/nav/familie/oppdrag/simulering/SimulerBeregningResponseMapperTest.kt b/src/test/kotlin/no/nav/familie/oppdrag/simulering/SimulerBeregningResponseMapperTest.kt