Flytte DTO-mapping av behandling fra interntjeneste til kontroller.

navikt · Nov 28, 2024 · ccfcc1c · ccfcc1c
1 parent d5f5984
commit ccfcc1c
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 8 deletions.
diff --git a/src/main/kotlin/no/nav/bidrag/behandling/controller/v2/BehandlingControllerV2.kt b/src/main/kotlin/no/nav/bidrag/behandling/controller/v2/BehandlingControllerV2.kt
@@ -272,7 +272,10 @@ class BehandlingControllerV2(
     fun henteBehandlingV2(
         @PathVariable behandlingsid: Long,
         @RequestParam("inkluderHistoriskeInntekter") inkluderHistoriskeInntekter: Boolean = false,
-    ): BehandlingDtoV2 = behandlingService.henteBehandling(behandlingsid, inkluderHistoriskeInntekter)
+    ): BehandlingDtoV2 {
+        val behandling = behandlingService.henteBehandling(behandlingsid, inkluderHistoriskeInntekter)
+        return dtomapper.tilDto(behandling, true)
+    }
 
     @Suppress("unused")
     @DeleteMapping("/behandling/{behandlingsid}")

diff --git a/src/main/kotlin/no/nav/bidrag/behandling/service/BehandlingService.kt b/src/main/kotlin/no/nav/bidrag/behandling/service/BehandlingService.kt
@@ -19,7 +19,6 @@ import no.nav.bidrag.behandling.dto.v1.forsendelse.BehandlingInfoDto
 import no.nav.bidrag.behandling.dto.v2.behandling.AktivereGrunnlagRequestV2
 import no.nav.bidrag.behandling.dto.v2.behandling.AktivereGrunnlagResponseV2
 import no.nav.bidrag.behandling.dto.v2.behandling.BehandlingDetaljerDtoV2
-import no.nav.bidrag.behandling.dto.v2.behandling.BehandlingDtoV2
 import no.nav.bidrag.behandling.dto.v2.underhold.BarnDto
 import no.nav.bidrag.behandling.transformers.Dtomapper
 import no.nav.bidrag.behandling.transformers.behandling.tilBehandlingDetaljerDtoV2
@@ -376,11 +375,11 @@ class BehandlingService(
     fun henteBehandling(
         behandlingsid: Long,
         inkluderHistoriskeInntekter: Boolean = false,
-    ): BehandlingDtoV2 {
+    ): Behandling {
         val behandling = hentBehandlingById(behandlingsid)
         grunnlagService.oppdatereGrunnlagForBehandling(behandling)
         behandling.oppdatereVirkningstidspunktSærbidrag()
-        return mapper.tilDto(behandling, true)
+        return behandling
     }
 
     @Transactional

diff --git a/src/test/kotlin/no/nav/bidrag/behandling/service/BehandlingServiceTest.kt b/src/test/kotlin/no/nav/bidrag/behandling/service/BehandlingServiceTest.kt
@@ -30,6 +30,7 @@ import no.nav.bidrag.behandling.dto.v1.behandling.OpprettKategoriRequestDto
 import no.nav.bidrag.behandling.dto.v1.behandling.OpprettRolleDto
 import no.nav.bidrag.behandling.dto.v2.behandling.AktivereGrunnlagRequestV2
 import no.nav.bidrag.behandling.dto.v2.behandling.Grunnlagsdatatype
+import no.nav.bidrag.behandling.transformers.Dtomapper
 import no.nav.bidrag.behandling.transformers.Jsonoperasjoner.Companion.jsonListeTilObjekt
 import no.nav.bidrag.behandling.transformers.Jsonoperasjoner.Companion.tilJson
 import no.nav.bidrag.behandling.transformers.behandling.hentEndringerSivilstand
@@ -107,6 +108,9 @@ class BehandlingServiceTest : TestContainerRunner() {
     @Autowired
     lateinit var testdataManager: TestdataManager
 
+    @Autowired
+    lateinit var dtomapper: Dtomapper
+
     @PersistenceContext
     lateinit var entityManager: EntityManager
 
@@ -213,9 +217,12 @@ class BehandlingServiceTest : TestContainerRunner() {
             kjøreStubber(behandling, true)
 
             // hvis
-            val behandlingDto = behandlingService.henteBehandling(behandling.id!!)
+            val hentetBehandling = behandlingService.henteBehandling(behandling.id!!)
 
             // så
+            // TODO: flytte til kontroller test - lagt inn her for enkelhetsskyld ifbm refaktorisering
+            val behandlingDto = dtomapper.tilDto(hentetBehandling, true)
+
             val ytelser =
                 setOf(
                     Inntektsrapportering.BARNETILLEGG,
@@ -407,7 +414,10 @@ class BehandlingServiceTest : TestContainerRunner() {
             behandlingRepository.save(behandling)
             kjøreStubber(behandling)
 
-            val behandlingDto = behandlingService.henteBehandling(behandling.id!!)
+            val hentetBehandling = behandlingService.henteBehandling(behandling.id!!)
+
+            // TODO: flytte til kontroller test - lagt inn her for enkelhetsskyld ifbm refaktorisering
+            val behandlingDto = dtomapper.tilDto(hentetBehandling, true)
 
             assertSoftly(behandlingDto) {
                 it.inntekter.beregnetInntekter shouldHaveSize 3
@@ -525,7 +535,10 @@ class BehandlingServiceTest : TestContainerRunner() {
             kjøreStubber(behandling)
 
             // hvis
-            val behandlingDto = behandlingService.henteBehandling(behandling.id!!)
+            val hentetBehandling = behandlingService.henteBehandling(behandling.id!!)
+
+            // TODO: flytte til kontroller test - lagt inn her for enkelhetsskyld ifbm refaktorisering
+            val behandlingDto = dtomapper.tilDto(hentetBehandling, true)
 
             // så
             val ytelser =
@@ -605,9 +618,12 @@ class BehandlingServiceTest : TestContainerRunner() {
             )
 
             // hvis
-            val behandlingDto = behandlingService.henteBehandling(behandling.id!!)
+            val hentetBehandling = behandlingService.henteBehandling(behandling.id!!)
 
             // så
+            // TODO: flytte til kontroller test - lagt inn her for enkelhetsskyld ifbm refaktorisering
+            val behandlingDto = dtomapper.tilDto(hentetBehandling, true)
+
             assertSoftly {
                 behandlingDto.ikkeAktiverteEndringerIGrunnlagsdata shouldNotBe null
                 behandlingDto.ikkeAktiverteEndringerIGrunnlagsdata.inntekter.årsinntekter shouldHaveSize 0