Skip to content

Commit

Permalink
F!- Erstatter PostConstruct med init
Browse files Browse the repository at this point in the history
  • Loading branch information
dskarpas committed Nov 3, 2023
1 parent 7e6703d commit 8c93ca4
Show file tree
Hide file tree
Showing 5 changed files with 4 additions and 15 deletions.
2 changes: 1 addition & 1 deletion build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ dependencies {
implementation("no.nav.eessi.pensjon:ep-eux:2.1.69")
implementation("no.nav.eessi.pensjon:ep-metrics:2.0.40")
implementation("no.nav.eessi.pensjon:ep-logging:2.1.34")
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-logging'
exclude module: 'ep-metrics'
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
package no.nav.eessi.pensjon.begrens.innsyn

import jakarta.annotation.PostConstruct
import no.nav.eessi.pensjon.eux.model.SedHendelse
import no.nav.eessi.pensjon.metrics.MetricsHelper
import no.nav.eessi.pensjon.utils.mapJsonToAny
Expand Down Expand Up @@ -37,8 +36,7 @@ class SedListener(private val begrensInnsynService: BegrensInnsynService,
return latchMottatt
}

@PostConstruct
fun initMetrics() {
init {
consumeOutgoingSed = metricsHelper.init("consumeOutgoingSed")
consumeIncomingSed = metricsHelper.init("consumeIncomingSed")
}
Expand Down
4 changes: 1 addition & 3 deletions src/main/kotlin/no/nav/eessi/pensjon/eux/EuxService.kt
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
package no.nav.eessi.pensjon.eux

import jakarta.annotation.PostConstruct
import no.nav.eessi.pensjon.eux.klient.EuxKlientLib
import no.nav.eessi.pensjon.eux.model.buc.Buc
import no.nav.eessi.pensjon.eux.model.document.ForenkletSED
Expand All @@ -27,8 +26,7 @@ class EuxService(
private lateinit var hentBuc: MetricsHelper.Metric
private lateinit var settSensitiv: MetricsHelper.Metric

@PostConstruct
fun initMetrics() {
init {
hentSed = metricsHelper.init("hentSed", alert = MetricsHelper.Toggle.OFF)
hentBuc = metricsHelper.init("hentBuc", alert = MetricsHelper.Toggle.OFF)
settSensitiv = metricsHelper.init("settSensitiv", alert = MetricsHelper.Toggle.OFF)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -176,9 +176,7 @@ class BegrensInnsynIntegrationTest {
@TestConfiguration
class TestConfig{
@Bean
fun personService(): PersonService = mockk {
every { initMetrics() } just Runs
}
fun personService(): PersonService = mockk()

@Bean
fun downstreamClientCredentialsResourceRestTemplate(restTemplateBuilder: RestTemplateBuilder): RestTemplate? {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -21,11 +21,6 @@ internal class SedListenerTest {

private val sedHendelse = javaClass.getResource("/sed/P_BUC_01.json").readText()

@BeforeEach
fun setup() {
sedListener.initMetrics()
}

@Test
fun `gitt en gyldig sedHendelse når sedSendt hendelse konsumeres så ack melding`() {
sedListener.consumeSedSendt(sedHendelse, cr, acknowledgment)
Expand Down

0 comments on commit 8c93ca4

Please sign in to comment.