F!- Erstatter PostConstruct med init

navikt · Nov 6, 2023 · ef4bf12 · ef4bf12
1 parent b3501f5
commit ef4bf12
Show file tree

Hide file tree

Showing 18 changed files with 23 additions and 104 deletions.
diff --git a/build.gradle b/build.gradle
@@ -81,12 +81,12 @@ dependencies {
         exclude module: 'ep-metrics'
         exclude module: 'ep-logging'
     }
-    implementation("no.nav.eessi.pensjon:ep-personoppslag:13.1.34"){
+    implementation("no.nav.eessi.pensjon:ep-personoppslag:13.1.35"){
         exclude module: 'ep-eux'
         exclude module: 'ep-metrics'
         exclude module: 'ep-logging'
     }
-    implementation("no.nav.eessi.pensjon:ep-pensjonsinformasjon:3.0.68")
+    implementation("no.nav.eessi.pensjon:ep-pensjonsinformasjon:3.0.69")
 
     implementation("no.nav.tjenestespesifikasjoner:pensjonsinformasjon-v1-tjenestespesifikasjon:2618.0448179")
 

diff --git a/src/main/kotlin/no/nav/eessi/pensjon/prefill/InnhentingService.kt b/src/main/kotlin/no/nav/eessi/pensjon/prefill/InnhentingService.kt
@@ -1,6 +1,5 @@
 package no.nav.eessi.pensjon.prefill
 
-import jakarta.annotation.PostConstruct
 import no.nav.eessi.pensjon.eux.model.BucType.*
 import no.nav.eessi.pensjon.eux.model.SedType.*
 import no.nav.eessi.pensjon.metrics.MetricsHelper
@@ -34,8 +33,7 @@ class InnhentingService(
     private val logger = LoggerFactory.getLogger(InnhentingService::class.java)
 
 
-    @PostConstruct
-    fun initMetrics() {
+    init {
         HentPerson = metricsHelper.init("HentPerson", ignoreHttpCodes = listOf(HttpStatus.BAD_REQUEST))
         addInstutionAndDocumentBucUtils = metricsHelper.init(
             "AddInstutionAndDocumentBucUtils",

diff --git a/src/main/kotlin/no/nav/eessi/pensjon/prefill/PersonDataService.kt b/src/main/kotlin/no/nav/eessi/pensjon/prefill/PersonDataService.kt
@@ -1,6 +1,5 @@
 package no.nav.eessi.pensjon.prefill
 
-import jakarta.annotation.PostConstruct
 import no.nav.eessi.pensjon.eux.model.SedType.*
 import no.nav.eessi.pensjon.metrics.MetricsHelper
 import no.nav.eessi.pensjon.personoppslag.pdl.PersonService
@@ -26,8 +25,7 @@ class PersonDataService(private val personService: PersonService,
 
     private lateinit var HentPerson: MetricsHelper.Metric
 
-    @PostConstruct
-    fun initMetrics() {
+    init {
         HentPerson = metricsHelper.init("HentPerson", ignoreHttpCodes = listOf(HttpStatus.BAD_REQUEST))
     }
 

diff --git a/src/main/kotlin/no/nav/eessi/pensjon/prefill/PrefillService.kt b/src/main/kotlin/no/nav/eessi/pensjon/prefill/PrefillService.kt
@@ -1,7 +1,6 @@
 package no.nav.eessi.pensjon.prefill
 
 import io.micrometer.core.instrument.Metrics
-import jakarta.annotation.PostConstruct
 import no.nav.eessi.pensjon.eux.model.sed.SED
 import no.nav.eessi.pensjon.metrics.MetricsHelper
 import no.nav.eessi.pensjon.prefill.sed.PrefillSEDService
@@ -29,8 +28,7 @@ class PrefillService(
 
     private val LATEST_RINA_SED_VERSION = "v4.2"
 
-    @PostConstruct
-    fun initMetrics() {
+    init {
         PrefillSed = metricsHelper.init("PrefillSed",
             ignoreHttpCodes = listOf(HttpStatus.BAD_REQUEST, HttpStatus.FORBIDDEN))
     }

diff --git a/src/main/kotlin/no/nav/eessi/pensjon/prefill/person/PrefillPDLAdresse.kt b/src/main/kotlin/no/nav/eessi/pensjon/prefill/person/PrefillPDLAdresse.kt
@@ -1,6 +1,5 @@
 package no.nav.eessi.pensjon.prefill.person
 
-import jakarta.annotation.PostConstruct
 import no.nav.eessi.pensjon.eux.model.sed.Adresse
 import no.nav.eessi.pensjon.kodeverk.KodeverkClient
 import no.nav.eessi.pensjon.kodeverk.PostnummerService
@@ -24,8 +23,7 @@ class PrefillPDLAdresse (private val postnummerService: PostnummerService,
     private lateinit var hentLandkodeMetric: MetricsHelper.Metric
     private val logger: Logger by lazy { LoggerFactory.getLogger(PrefillPDLAdresse::class.java) }
 
-    @PostConstruct
-    fun initMetrics() {
+    init {
         hentLandkodeMetric = metricsHelper.init("hentLandkodeMetric")
     }
 

diff --git a/src/test/kotlin/no/nav/eessi/pensjon/config/PrefillAdresseKodeverkTest.kt b/src/test/kotlin/no/nav/eessi/pensjon/config/PrefillAdresseKodeverkTest.kt
@@ -79,7 +79,7 @@ class PrefillAdresseKodeverkTest {
     class Config {
         @Bean
         fun kodeVerkHentLandkoder(): KodeVerkHentLandkoder {
-            return KodeVerkHentLandkoder("testApp", restTemplate, MetricsHelper.ForTest()).apply { initMetrics() }
+            return KodeVerkHentLandkoder("testApp", restTemplate, MetricsHelper.ForTest())
         }
         @Bean
         fun kodeverkClient(): KodeverkClient {
@@ -88,7 +88,7 @@ class PrefillAdresseKodeverkTest {
 
         @Bean
         fun prefillPDLAdresse(): PrefillPDLAdresse {
-            return PrefillPDLAdresse(PostnummerService(), kodeverkClient(), mockk()).apply { initMetrics() }
+            return PrefillPDLAdresse(PostnummerService(), kodeverkClient(), mockk())
         }
     }
 

diff --git a/src/test/kotlin/no/nav/eessi/pensjon/prefill/InnhentingServiceTest.kt b/src/test/kotlin/no/nav/eessi/pensjon/prefill/InnhentingServiceTest.kt
@@ -3,25 +3,13 @@ package no.nav.eessi.pensjon.prefill
 import io.mockk.every
 import io.mockk.mockk
 import no.nav.eessi.pensjon.eux.model.BucType
-import no.nav.eessi.pensjon.eux.model.BucType.P_BUC_01
-import no.nav.eessi.pensjon.eux.model.BucType.P_BUC_02
-import no.nav.eessi.pensjon.eux.model.BucType.P_BUC_05
+import no.nav.eessi.pensjon.eux.model.BucType.*
 import no.nav.eessi.pensjon.eux.model.SedType
-import no.nav.eessi.pensjon.eux.model.SedType.P15000
-import no.nav.eessi.pensjon.eux.model.SedType.P2000
-import no.nav.eessi.pensjon.eux.model.SedType.P2100
-import no.nav.eessi.pensjon.eux.model.SedType.P5000
-import no.nav.eessi.pensjon.eux.model.SedType.P8000
+import no.nav.eessi.pensjon.eux.model.SedType.*
 import no.nav.eessi.pensjon.pensjonsinformasjon.clients.PensjonsinformasjonClient
 import no.nav.eessi.pensjon.personoppslag.pdl.model.AktoerId
-import no.nav.eessi.pensjon.personoppslag.pdl.model.Ident
-import no.nav.eessi.pensjon.personoppslag.pdl.model.IdentGruppe
-import no.nav.eessi.pensjon.personoppslag.pdl.model.IdentGruppe.*
-import no.nav.eessi.pensjon.shared.api.ApiRequest
-import no.nav.eessi.pensjon.shared.api.ApiSubject
-import no.nav.eessi.pensjon.shared.api.PersonId
-import no.nav.eessi.pensjon.shared.api.PrefillDataModel
-import no.nav.eessi.pensjon.shared.api.SubjectFnr
+import no.nav.eessi.pensjon.personoppslag.pdl.model.IdentGruppe.AKTORID
+import no.nav.eessi.pensjon.shared.api.*
 import no.nav.pensjon.v1.brukerssakerliste.V1BrukersSakerListe
 import no.nav.pensjon.v1.pensjonsinformasjon.Pensjonsinformasjon
 import no.nav.pensjon.v1.sak.V1Sak
@@ -48,9 +36,7 @@ class InnhentingServiceTest {
 
     @BeforeEach
     fun before() {
-        innhentingService =
-            InnhentingService(personDataService, pensjonsinformasjonService = pensjonsinformasjonService)
-        innhentingService.initMetrics()
+        innhentingService = InnhentingService(personDataService, pensjonsinformasjonService = pensjonsinformasjonService)
     }
 
     @Test
@@ -139,11 +125,6 @@ class InnhentingServiceTest {
         val pensjonsinformasjonService = PensjonsinformasjonService(pensjonsinformasjonClient)
         val innhentingsservice = InnhentingService(personDataService, pensjonsinformasjonService = pensjonsinformasjonService)
 
-        @BeforeEach
-        fun setUp() {
-            innhentingsservice.initMetrics()
-        }
-
         @Test
         fun `Gitt en P2100 med saktype ALDER saa skal hentPensjoninformasjonCollection sitt resultat paa saktype gi ut ALDER`() {
             val prefillData = prefillDataModel(sedType = P2100)

diff --git a/src/test/kotlin/no/nav/eessi/pensjon/prefill/PersonDataServiceTest.kt b/src/test/kotlin/no/nav/eessi/pensjon/prefill/PersonDataServiceTest.kt
@@ -18,7 +18,6 @@ import no.nav.eessi.pensjon.shared.person.FodselsnummerGenerator
 import org.junit.jupiter.api.AfterEach
 import org.junit.jupiter.api.Assertions.assertEquals
 import org.junit.jupiter.api.Assertions.assertNull
-import org.junit.jupiter.api.BeforeEach
 import org.junit.jupiter.api.Test
 import org.junit.jupiter.api.assertThrows
 import org.springframework.web.server.ResponseStatusException
@@ -44,11 +43,6 @@ internal class PersonDataServiceTest {
         personService = personService
     )
 
-    @BeforeEach
-    fun setup() {
-         persondataService.initMetrics()
-    }
-
     @AfterEach
     fun after() {
         clearAllMocks()

diff --git a/src/test/kotlin/no/nav/eessi/pensjon/prefill/PrefillControllerTest.kt b/src/test/kotlin/no/nav/eessi/pensjon/prefill/PrefillControllerTest.kt
@@ -6,11 +6,7 @@ import io.mockk.impl.annotations.SpyK
 import io.mockk.mockk
 import no.nav.eessi.pensjon.eux.model.BucType.P_BUC_06
 import no.nav.eessi.pensjon.eux.model.SedType.P6000
-import no.nav.eessi.pensjon.eux.model.sed.Bruker
-import no.nav.eessi.pensjon.eux.model.sed.Krav
-import no.nav.eessi.pensjon.eux.model.sed.Nav
-import no.nav.eessi.pensjon.eux.model.sed.Person
-import no.nav.eessi.pensjon.eux.model.sed.SED
+import no.nav.eessi.pensjon.eux.model.sed.*
 import no.nav.eessi.pensjon.logging.AuditLogger
 import no.nav.eessi.pensjon.personoppslag.pdl.model.NorskIdent
 import no.nav.eessi.pensjon.prefill.models.PersonDataCollection
@@ -40,10 +36,7 @@ class PrefillControllerTest {
     fun before() {
 
         val innhentingService = InnhentingService(personDataService, pensjonsinformasjonService = pensjonsinformasjonService)
-        innhentingService.initMetrics()
-
         val prefillService = PrefillService(mockPrefillSEDService, innhentingService, automatiseringStatistikkService)
-        prefillService.initMetrics()
 
         prefillController = PrefillController(
             prefillService, auditLogger

diff --git a/src/test/kotlin/no/nav/eessi/pensjon/prefill/person/PrefillPDLAdresseTest.kt b/src/test/kotlin/no/nav/eessi/pensjon/prefill/person/PrefillPDLAdresseTest.kt
@@ -8,18 +8,7 @@ import io.mockk.mockk
 import no.nav.eessi.pensjon.kodeverk.KodeverkClient
 import no.nav.eessi.pensjon.kodeverk.PostnummerService
 import no.nav.eessi.pensjon.personoppslag.pdl.PersonService
-import no.nav.eessi.pensjon.personoppslag.pdl.model.AdressebeskyttelseGradering
-import no.nav.eessi.pensjon.personoppslag.pdl.model.Bostedsadresse
-import no.nav.eessi.pensjon.personoppslag.pdl.model.Kontaktadresse
-import no.nav.eessi.pensjon.personoppslag.pdl.model.KontaktadresseType
-import no.nav.eessi.pensjon.personoppslag.pdl.model.Metadata
-import no.nav.eessi.pensjon.personoppslag.pdl.model.Navn
-import no.nav.eessi.pensjon.personoppslag.pdl.model.NorskIdent
-import no.nav.eessi.pensjon.personoppslag.pdl.model.Person
-import no.nav.eessi.pensjon.personoppslag.pdl.model.PostadresseIFrittFormat
-import no.nav.eessi.pensjon.personoppslag.pdl.model.UtenlandskAdresse
-import no.nav.eessi.pensjon.personoppslag.pdl.model.UtenlandskAdresseIFrittFormat
-import no.nav.eessi.pensjon.personoppslag.pdl.model.Vegadresse
+import no.nav.eessi.pensjon.personoppslag.pdl.model.*
 import no.nav.eessi.pensjon.prefill.PersonPDLMock
 import no.nav.eessi.pensjon.prefill.PersonPDLMock.medBeskyttelse
 import no.nav.eessi.pensjon.utils.mapJsonToAny
@@ -48,7 +37,6 @@ class PrefillPDLAdresseTest{
         debugLogger.addAppender(listAppender)
         listAppender.start()
         prefillAdresse = PrefillPDLAdresse(PostnummerService(), kodeverkClient, personService)
-        prefillAdresse.initMetrics()
     }
 
     @AfterEach

diff --git a/src/test/kotlin/no/nav/eessi/pensjon/prefill/person/PrefillPDLNavTest.kt b/src/test/kotlin/no/nav/eessi/pensjon/prefill/person/PrefillPDLNavTest.kt
@@ -3,22 +3,8 @@ package no.nav.eessi.pensjon.prefill.person
 import io.mockk.every
 import io.mockk.mockk
 import no.nav.eessi.pensjon.eux.model.SedType
-import no.nav.eessi.pensjon.eux.model.sed.Adresse
-import no.nav.eessi.pensjon.eux.model.sed.ArbeidsforholdItem
-import no.nav.eessi.pensjon.eux.model.sed.Bank
-import no.nav.eessi.pensjon.eux.model.sed.BarnItem
-import no.nav.eessi.pensjon.eux.model.sed.Bruker
-import no.nav.eessi.pensjon.eux.model.sed.EessisakItem
-import no.nav.eessi.pensjon.eux.model.sed.Ektefelle
-import no.nav.eessi.pensjon.eux.model.sed.Foedested
-import no.nav.eessi.pensjon.eux.model.sed.Foreldre
-import no.nav.eessi.pensjon.eux.model.sed.Innehaver
-import no.nav.eessi.pensjon.eux.model.sed.Konto
-import no.nav.eessi.pensjon.eux.model.sed.Nav
+import no.nav.eessi.pensjon.eux.model.sed.*
 import no.nav.eessi.pensjon.eux.model.sed.Person
-import no.nav.eessi.pensjon.eux.model.sed.PinItem
-import no.nav.eessi.pensjon.eux.model.sed.Sepa
-import no.nav.eessi.pensjon.eux.model.sed.StatsborgerskapItem
 import no.nav.eessi.pensjon.kodeverk.KodeverkClient
 import no.nav.eessi.pensjon.kodeverk.PostnummerService
 import no.nav.eessi.pensjon.personoppslag.pdl.PersonService
@@ -38,9 +24,7 @@ import no.nav.eessi.pensjon.shared.person.Fodselsnummer
 import no.nav.eessi.pensjon.shared.person.FodselsnummerGenerator
 import no.nav.eessi.pensjon.utils.mapAnyToJson
 import no.nav.eessi.pensjon.utils.toJsonSkipEmpty
-import org.junit.jupiter.api.Assertions.assertEquals
-import org.junit.jupiter.api.Assertions.assertNotNull
-import org.junit.jupiter.api.Assertions.assertNull
+import org.junit.jupiter.api.Assertions.*
 import org.junit.jupiter.api.BeforeEach
 import org.junit.jupiter.api.Test
 import org.skyscreamer.jsonassert.JSONAssert
@@ -64,7 +48,7 @@ class PrefillPDLNavTest {
         every { kodeverkClient.finnLandkode(eq("SWE")) } returns "SE"
 
         prefillPDLNav = PrefillPDLNav(
-            PrefillPDLAdresse(PostnummerService(), kodeverkClient, personService).apply { initMetrics() },
+            PrefillPDLAdresse(PostnummerService(), kodeverkClient, personService),
             someInstitutionId,
             someIntitutionNavn)
     }

diff --git a/src/test/kotlin/no/nav/eessi/pensjon/prefill/sed/PrefillP8000APUtlandInnvTest.kt b/src/test/kotlin/no/nav/eessi/pensjon/prefill/sed/PrefillP8000APUtlandInnvTest.kt
@@ -18,9 +18,7 @@ import no.nav.eessi.pensjon.shared.person.Fodselsnummer
 import no.nav.eessi.pensjon.shared.person.FodselsnummerGenerator
 import no.nav.pensjon.v1.kravhistorikkliste.V1KravHistorikkListe
 import no.nav.pensjon.v1.sak.V1Sak
-import org.junit.jupiter.api.Assertions.assertEquals
-import org.junit.jupiter.api.Assertions.assertNotNull
-import org.junit.jupiter.api.Assertions.assertNull
+import org.junit.jupiter.api.Assertions.*
 import org.junit.jupiter.api.BeforeEach
 import org.junit.jupiter.api.Test
 
@@ -42,7 +40,7 @@ class PrefillP8000APUtlandInnvTest {
 
         every { kodeverkClient.finnLandkode(any()) } returns "NO"
 
-        val prefillAdresse = PrefillPDLAdresse(PostnummerService(),kodeverkClient, personService).apply { initMetrics() }
+        val prefillAdresse = PrefillPDLAdresse(PostnummerService(),kodeverkClient, personService)
 
         prefillNav = PrefillPDLNav(
                 prefillAdresse = prefillAdresse,

diff --git a/src/test/kotlin/no/nav/eessi/pensjon/prefill/sed/PrefillP8000P_BUC_05Test.kt b/src/test/kotlin/no/nav/eessi/pensjon/prefill/sed/PrefillP8000P_BUC_05Test.kt
@@ -55,7 +55,7 @@ class PrefillP8000P_BUC_05Test {
         every { kodeverkClient.finnLandkode("NOR") } returns "NO"
         every { kodeverkClient.finnLandkode("SWE") } returns "SE"
 
-        prefillAdresse = PrefillPDLAdresse(PostnummerService(), kodeverkClient, personService).apply { initMetrics() }
+        prefillAdresse = PrefillPDLAdresse(PostnummerService(), kodeverkClient, personService)
         prefillNav = PrefillPDLNav( prefillAdresse,
                 institutionid = "NO:noinst002",
                 institutionnavn = "NOINST002, NO INST002, NO")

diff --git a/src/test/kotlin/no/nav/eessi/pensjon/prefill/sed/PrefillTestHelper.kt b/src/test/kotlin/no/nav/eessi/pensjon/prefill/sed/PrefillTestHelper.kt
@@ -26,7 +26,6 @@ object PrefillTestHelper {
         } returns readXMLVedtakresponse(responseXMLfilename)
 
         val pensjonsinformasjonClient = PensjonsinformasjonClient(pensjonsinformasjonRestTemplate, PensjonRequestBuilder())
-        pensjonsinformasjonClient.initMetrics()
         return PensjonsinformasjonService(pensjonsinformasjonClient)
     }
 
@@ -39,7 +38,6 @@ object PrefillTestHelper {
         } returns readXMLresponse(responseXMLfilename)
 
         val pensjonsinformasjonClient = PensjonsinformasjonClient(pensjonsinformasjonRestTemplate, PensjonRequestBuilder())
-        pensjonsinformasjonClient.initMetrics()
         return PensjonsinformasjonService(pensjonsinformasjonClient)
     }
 

diff --git a/src/test/kotlin/no/nav/eessi/pensjon/prefill/sed/krav/PrefillP2100AdresseTest.kt b/src/test/kotlin/no/nav/eessi/pensjon/prefill/sed/krav/PrefillP2100AdresseTest.kt
@@ -42,7 +42,6 @@ class PrefillP2100AdresseTest {
         prefillPDLAdresse = PrefillPDLAdresse(mockk(relaxed = true), mockk(relaxed = true){
             every { finnLandkode(eq("NOR")) } returns "NO"
         }, personService)
-        prefillPDLAdresse.initMetrics()
 
         val prefillNav = PrefillPDLNav(prefillPDLAdresse,
                 institutionid = "NO:noinst002",

diff --git a/src/test/kotlin/no/nav/eessi/pensjon/prefill/sed/krav/PrefillP2200_AP_21975717Test.kt b/src/test/kotlin/no/nav/eessi/pensjon/prefill/sed/krav/PrefillP2200_AP_21975717Test.kt
@@ -54,7 +54,7 @@ class PrefillP2200_AP_21975717Test {
         personDataCollection = PersonPDLMock.createEnkelFamilie(personFnr, ekteFnr)
 
         prefillNav = PrefillPDLNav(
-            prefillAdresse = PrefillPDLAdresse(PostnummerService(), kodeverkClient, personService).apply { initMetrics() },
+            prefillAdresse = PrefillPDLAdresse(PostnummerService(), kodeverkClient, personService),
             institutionid = "NO:noinst002",
             institutionnavn = "NOINST002, NO INST002, NO"
         )

diff --git a/...tlin/no/nav/eessi/pensjon/services/pensjonsinformasjon/PensjonsinformasjonClientMother.kt b/...tlin/no/nav/eessi/pensjon/services/pensjonsinformasjon/PensjonsinformasjonClientMother.kt
@@ -17,14 +17,7 @@ object PensjonsinformasjonClientMother {
         val readXMLresponse = ResponseEntity(resource, HttpStatus.OK)
 
         val mockRestTemplate: RestTemplate = mockk()
-/*
-        whenever(mockRestTemplate.exchange(any<String>(), any(), any<HttpEntity<Unit>>(), ArgumentMatchers.eq(String::class.java)))
-                .thenReturn(readXMLresponse)
-*/
         every { mockRestTemplate.exchange(any<String>(), any(), any<HttpEntity<Unit>>(), eq(String::class.java)) } returns readXMLresponse
-        val pensjonsinformasjonClient = PensjonsinformasjonClient(mockRestTemplate, PensjonRequestBuilder())
-        pensjonsinformasjonClient.initMetrics()
-        return pensjonsinformasjonClient
+        return PensjonsinformasjonClient(mockRestTemplate, PensjonRequestBuilder())
     }
-
 }
diff --git a/...kotlin/no/nav/eessi/pensjon/services/pensjonsinformasjon/PensjonsinformasjonClientTest.kt b/...kotlin/no/nav/eessi/pensjon/services/pensjonsinformasjon/PensjonsinformasjonClientTest.kt
@@ -32,7 +32,6 @@ class PensjonsinformasjonClientTest {
     @BeforeEach
     fun setup() {
         pensjonsinformasjonClient = PensjonsinformasjonClient(mockrestTemplate, PensjonRequestBuilder())
-        pensjonsinformasjonClient.initMetrics()
     }
 
     @Test