Fjerner bruken av injectmockks (#5100)

Favrokort:
https://favro.com/organization/98c34fb974ce445eac854de0/1844bbac3b6605eacc8f5543?card=NAV-24408

Ref diskusjon i
[https://nav-it.slack.com/archives/C0205N5B9CK/p1739971349429849](https://github.com/navikt/familie-ba-sak/compare/Hadebra_injectmockks?expand=1)
så fjerner vi injectmockks.

src/test/enhetstester/kotlin/no/nav/familie/ba/sak/config/KafkaAivenErrorHandlerTest.kt

@@ -1,8 +1,7 @@
 package no.nav.familie.ba.sak.config
 
 import io.mockk.MockKAnnotations
-import io.mockk.impl.annotations.InjectMockKs
-import io.mockk.impl.annotations.MockK
+import io.mockk.mockk
 import org.apache.kafka.clients.consumer.Consumer
 import org.apache.kafka.clients.consumer.ConsumerRecord
 import org.assertj.core.api.Assertions.assertThatThrownBy
@@ -13,14 +12,10 @@ import org.springframework.kafka.listener.MessageListenerContainer
 
 @TestInstance(TestInstance.Lifecycle.PER_CLASS)
 class KafkaAivenErrorHandlerTest {
-    @MockK(relaxed = true)
-    lateinit var container: MessageListenerContainer
+    private val container = mockk<MessageListenerContainer>(relaxed = true)
+    private val consumer = mockk<Consumer<String, String>>(relaxed = true)
 
-    @MockK(relaxed = true)
-    lateinit var consumer: Consumer<*, *>
-
-    @InjectMockKs
-    lateinit var errorHandler: KafkaAivenErrorHandler
+    private val errorHandler = KafkaAivenErrorHandler()
 
     @BeforeEach
     internal fun setUp() {

src/test/enhetstester/kotlin/no/nav/familie/ba/sak/integrasjoner/ecb/ECBServiceTest.kt

@@ -1,9 +1,7 @@
 package no.nav.familie.ba.sak.integrasjoner.ecb
 
 import io.mockk.every
-import io.mockk.impl.annotations.InjectMockKs
-import io.mockk.impl.annotations.MockK
-import io.mockk.junit5.MockKExtension
+import io.mockk.mockk
 import io.mockk.unmockkAll
 import no.nav.familie.ba.sak.integrasjoner.ecb.domene.ECBValutakursCache
 import no.nav.familie.ba.sak.integrasjoner.ecb.domene.ECBValutakursCacheRepository
@@ -22,21 +20,15 @@ import org.junit.jupiter.api.Assertions.assertEquals
 import org.junit.jupiter.api.Test
 import org.junit.jupiter.api.TestInstance
 import org.junit.jupiter.api.assertThrows
-import org.junit.jupiter.api.extension.ExtendWith
 import java.math.BigDecimal
 import java.time.LocalDate
 
-@ExtendWith(MockKExtension::class)
 @TestInstance(TestInstance.Lifecycle.PER_CLASS)
 class ECBServiceTest {
-    @MockK
-    private lateinit var ecbClient: ValutakursRestClient
+    private val ecbClient = mockk<ValutakursRestClient>()
+    private val ecbValutakursCacheRepository = mockk<ECBValutakursCacheRepository>()
 
-    @MockK
-    private lateinit var evbValutakursCacheRepository: ECBValutakursCacheRepository
-
-    @InjectMockKs
-    private lateinit var ecbService: ECBService
+    private val ecbService = ECBService(ecbClient, ecbValutakursCacheRepository)
 
     @AfterAll
     fun tearDown() {
@@ -52,8 +44,8 @@ class ECBServiceTest {
                 listOf(Pair("NOK", BigDecimal.valueOf(10.337)), Pair("SEK", BigDecimal.valueOf(10.6543))),
                 valutakursDato.toString(),
             )
-        every { evbValutakursCacheRepository.findByValutakodeAndValutakursdato(any(), any()) } returns emptyList()
-        every { evbValutakursCacheRepository.save(any()) } returns ECBValutakursCache(kurs = BigDecimal.valueOf(10.6543), valutakode = "SEK", valutakursdato = valutakursDato)
+        every { ecbValutakursCacheRepository.findByValutakodeAndValutakursdato(any(), any()) } returns emptyList()
+        every { ecbValutakursCacheRepository.save(any()) } returns ECBValutakursCache(kurs = BigDecimal.valueOf(10.6543), valutakode = "SEK", valutakursdato = valutakursDato)
         every {
             ecbClient.hentValutakurs(
                 Frequency.Daily,
@@ -74,7 +66,7 @@ class ECBServiceTest {
                 listOf(Pair("NOK", BigDecimal.valueOf(10.337))),
                 valutakursDato.toString(),
             )
-        every { evbValutakursCacheRepository.findByValutakodeAndValutakursdato(any(), any()) } returns emptyList()
+        every { ecbValutakursCacheRepository.findByValutakodeAndValutakursdato(any(), any()) } returns emptyList()
         every {
             ecbClient.hentValutakurs(
                 Frequency.Daily,
@@ -94,7 +86,7 @@ class ECBServiceTest {
                 listOf(Pair("NOK", BigDecimal.valueOf(10.337)), Pair("SEK", BigDecimal.valueOf(10.6543))),
                 valutakursDato.minusDays(1).toString(),
             )
-        every { evbValutakursCacheRepository.findByValutakodeAndValutakursdato(any(), any()) } returns emptyList()
+        every { ecbValutakursCacheRepository.findByValutakodeAndValutakursdato(any(), any()) } returns emptyList()
         every {
             ecbClient.hentValutakurs(
                 Frequency.Daily,
@@ -115,8 +107,8 @@ class ECBServiceTest {
                 listOf(Pair("NOK", BigDecimal.valueOf(9.4567))),
                 valutakursDato.toString(),
             )
-        every { evbValutakursCacheRepository.findByValutakodeAndValutakursdato(any(), any()) } returns emptyList()
-        every { evbValutakursCacheRepository.save(any()) } returns ECBValutakursCache(kurs = BigDecimal.valueOf(9.4567), valutakode = "EUR", valutakursdato = valutakursDato)
+        every { ecbValutakursCacheRepository.findByValutakodeAndValutakursdato(any(), any()) } returns emptyList()
+        every { ecbValutakursCacheRepository.save(any()) } returns ECBValutakursCache(kurs = BigDecimal.valueOf(9.4567), valutakode = "EUR", valutakursdato = valutakursDato)
         every {
             ecbClient.hentValutakurs(
                 Frequency.Daily,

src/test/enhetstester/kotlin/no/nav/familie/ba/sak/integrasjoner/infotrygd/InfotrygdControllerTest.kt

@@ -1,64 +1,38 @@
 package no.nav.familie.ba.sak.integrasjoner.infotrygd
 
 import io.mockk.every
-import io.mockk.impl.annotations.InjectMockKs
-import io.mockk.impl.annotations.MockK
-import io.mockk.impl.annotations.SpyK
-import io.mockk.junit5.MockKExtension
+import io.mockk.mockk
+import io.mockk.spyk
 import no.nav.familie.ba.sak.common.clearAllCaches
 import no.nav.familie.ba.sak.config.IntegrasjonClientMock.Companion.mockSjekkTilgang
 import no.nav.familie.ba.sak.config.tilAktør
 import no.nav.familie.ba.sak.integrasjoner.familieintegrasjoner.FamilieIntegrasjonerTilgangskontrollClient
 import no.nav.familie.ba.sak.integrasjoner.familieintegrasjoner.FamilieIntegrasjonerTilgangskontrollService
-import no.nav.familie.ba.sak.integrasjoner.pdl.PersonopplysningerService
 import no.nav.familie.ba.sak.integrasjoner.pdl.SystemOnlyPdlRestClient
 import no.nav.familie.ba.sak.kjerne.personident.PersonidentService
 import no.nav.familie.kontrakter.ba.infotrygd.InfotrygdSøkResponse
 import no.nav.familie.kontrakter.ba.infotrygd.Sak
 import no.nav.familie.kontrakter.felles.personopplysning.ADRESSEBESKYTTELSEGRADERING
 import no.nav.familie.kontrakter.felles.personopplysning.Adressebeskyttelse
 import org.junit.jupiter.api.Assertions
-import org.junit.jupiter.api.BeforeAll
 import org.junit.jupiter.api.BeforeEach
 import org.junit.jupiter.api.Test
 import org.junit.jupiter.api.TestInstance
-import org.junit.jupiter.api.extension.ExtendWith
 import org.springframework.cache.concurrent.ConcurrentMapCacheManager
 import org.springframework.http.HttpStatus
 
-@ExtendWith(MockKExtension::class)
 @TestInstance(TestInstance.Lifecycle.PER_CLASS)
 class InfotrygdControllerTest {
-    @MockK
-    lateinit var personopplysningerService: PersonopplysningerService
+    private val systemOnlyPdlRestClient = mockk<SystemOnlyPdlRestClient>()
+    private val cacheManager = spyk(ConcurrentMapCacheManager())
+    private val familieIntegrasjonerTilgangskontrollClient = mockk<FamilieIntegrasjonerTilgangskontrollClient>()
 
-    @MockK
-    lateinit var systemOnlyPdlRestClient: SystemOnlyPdlRestClient
+    private val familieIntegrasjonerTilgangskontrollService = FamilieIntegrasjonerTilgangskontrollService(familieIntegrasjonerTilgangskontrollClient, cacheManager, systemOnlyPdlRestClient)
 
-    @SpyK
-    var cacheManager = ConcurrentMapCacheManager()
-
-    @MockK
-    lateinit var familieIntegrasjonerTilgangskontrollClient: FamilieIntegrasjonerTilgangskontrollClient
-
-    @InjectMockKs
-    lateinit var familieIntegrasjonerTilgangskontrollService: FamilieIntegrasjonerTilgangskontrollService
-
-    @MockK
-    lateinit var infotrygdBarnetrygdClient: InfotrygdBarnetrygdClient
-
-    @MockK
-    lateinit var personidentService: PersonidentService
-
-    @InjectMockKs
-    lateinit var infotrygdService: InfotrygdService
-
-    lateinit var infotrygdController: InfotrygdController
-
-    @BeforeAll
-    fun init() {
-        infotrygdController = InfotrygdController(infotrygdBarnetrygdClient, personidentService, infotrygdService)
-    }
+    private val infotrygdBarnetrygdClient = mockk<InfotrygdBarnetrygdClient>()
+    private val personidentService = mockk<PersonidentService>()
+    private val infotrygdService: InfotrygdService = InfotrygdService(infotrygdBarnetrygdClient, familieIntegrasjonerTilgangskontrollService, personidentService)
+    private val infotrygdController = InfotrygdController(infotrygdBarnetrygdClient, personidentService, infotrygdService)
 
     @BeforeEach
     fun setUp() {

src/test/enhetstester/kotlin/no/nav/familie/ba/sak/integrasjoner/oppgave/OppgaveControllerTest.kt

@@ -1,10 +1,8 @@
 package no.nav.familie.ba.sak.integrasjoner.oppgave
 
 import io.mockk.every
-import io.mockk.impl.annotations.InjectMockKs
-import io.mockk.impl.annotations.MockK
-import io.mockk.junit5.MockKExtension
 import io.mockk.just
+import io.mockk.mockk
 import io.mockk.runs
 import no.nav.familie.ba.sak.integrasjoner.familieintegrasjoner.IntegrasjonClient
 import no.nav.familie.ba.sak.integrasjoner.familieintegrasjoner.IntegrasjonException
@@ -22,35 +20,28 @@ import org.junit.jupiter.api.BeforeAll
 import org.junit.jupiter.api.Test
 import org.junit.jupiter.api.TestInstance
 import org.junit.jupiter.api.assertThrows
-import org.junit.jupiter.api.extension.ExtendWith
 import org.springframework.http.HttpStatus
 
-@ExtendWith(MockKExtension::class)
 @TestInstance(TestInstance.Lifecycle.PER_CLASS)
 class OppgaveControllerTest {
-    @MockK
-    lateinit var oppgaveService: OppgaveService
-
-    @MockK
-    lateinit var personopplysningerService: PersonopplysningerService
-
-    @MockK
-    lateinit var personidentService: PersonidentService
-
-    @MockK
-    lateinit var integrasjonClient: IntegrasjonClient
-
-    @MockK
-    lateinit var fagsakService: FagsakService
-
-    @MockK
-    lateinit var innkommendeJournalføringService: InnkommendeJournalføringService
-
-    @MockK
-    lateinit var tilgangService: TilgangService
-
-    @InjectMockKs
-    lateinit var oppgaveController: OppgaveController
+    private val oppgaveService = mockk<OppgaveService>()
+    private val personopplysningerService = mockk<PersonopplysningerService>()
+    private val personidentService = mockk<PersonidentService>()
+    private val integrasjonClient = mockk<IntegrasjonClient>()
+    private val fagsakService = mockk<FagsakService>()
+    private val innkommendeJournalføringService = mockk<InnkommendeJournalføringService>()
+    private val tilgangService = mockk<TilgangService>()
+
+    private val oppgaveController =
+        OppgaveController(
+            oppgaveService = oppgaveService,
+            fagsakService = fagsakService,
+            personidentService = personidentService,
+            integrasjonClient = integrasjonClient,
+            personopplysningerService = personopplysningerService,
+            tilgangService = tilgangService,
+            innkommendeJournalføringService = innkommendeJournalføringService,
+        )
 
     @BeforeAll
     fun init() {

src/test/enhetstester/kotlin/no/nav/familie/ba/sak/integrasjoner/sanity/SanityServiceTest.kt

@@ -1,25 +1,19 @@
 package no.nav.familie.ba.sak.integrasjoner.sanity
 
 import io.mockk.every
-import io.mockk.impl.annotations.InjectMockKs
-import io.mockk.impl.annotations.MockK
-import io.mockk.junit5.MockKExtension
+import io.mockk.mockk
 import no.nav.familie.ba.sak.kjerne.brev.domene.SanityBegrunnelse
 import no.nav.familie.ba.sak.kjerne.brev.domene.SanityEØSBegrunnelse
 import no.nav.familie.ba.sak.kjerne.vedtak.begrunnelser.EØSStandardbegrunnelse
 import no.nav.familie.ba.sak.kjerne.vedtak.begrunnelser.Standardbegrunnelse
 import org.assertj.core.api.Assertions.assertThat
 import org.junit.jupiter.api.Test
 import org.junit.jupiter.api.assertThrows
-import org.junit.jupiter.api.extension.ExtendWith
 
-@ExtendWith(MockKExtension::class)
 class SanityServiceTest {
-    @MockK
-    private lateinit var sanityKlient: SanityKlient
+    private val sanityKlient = mockk<SanityKlient>()
 
-    @InjectMockKs
-    private lateinit var sanityService: SanityService
+    private val sanityService = SanityService(sanityKlient)
 
     @Test
     fun `hentSanityBegrunnelser - skal hente standardbegrunnelser`() {

src/test/enhetstester/kotlin/no/nav/familie/ba/sak/internal/ForvalterServiceTest.kt

@@ -1,9 +1,7 @@
 package no.nav.familie.ba.sak.internal
 
 import io.mockk.every
-import io.mockk.impl.annotations.InjectMockKs
-import io.mockk.impl.annotations.MockK
-import io.mockk.junit5.MockKExtension
+import io.mockk.mockk
 import io.mockk.slot
 import no.nav.familie.ba.sak.common.Feil
 import no.nav.familie.ba.sak.common.førsteDagIInneværendeMåned
@@ -35,44 +33,34 @@ import org.assertj.core.api.Assertions.assertThat
 import org.junit.jupiter.api.Assertions.assertTrue
 import org.junit.jupiter.api.Test
 import org.junit.jupiter.api.assertThrows
-import org.junit.jupiter.api.extension.ExtendWith
 import java.time.LocalDate
 import kotlin.random.Random
 
-@ExtendWith(MockKExtension::class)
 class ForvalterServiceTest {
-    @MockK
-    lateinit var persongrunnlagService: PersongrunnlagService
-
-    @MockK
-    lateinit var økonomiService: ØkonomiService
-
-    @MockK
-    lateinit var vedtakService: VedtakService
-
-    @MockK
-    lateinit var beregningService: BeregningService
-
-    @MockK
-    lateinit var behandlingHentOgPersisterService: BehandlingHentOgPersisterService
-
-    @MockK
-    lateinit var vilkårsvurderingService: VilkårsvurderingService
-
-    @MockK
-    lateinit var fagsakRepository: FagsakRepository
-
-    @MockK
-    lateinit var behandlingRepository: BehandlingRepository
-
-    @MockK
-    lateinit var tilkjentYtelseValideringService: TilkjentYtelseValideringService
-
-    @MockK
-    lateinit var arbeidsfordelingService: ArbeidsfordelingService
-
-    @InjectMockKs
-    lateinit var forvalterService: ForvalterService
+    private val persongrunnlagService = mockk<PersongrunnlagService>()
+    private val økonomiService = mockk<ØkonomiService>()
+    private val vedtakService = mockk<VedtakService>()
+    private val beregningService = mockk<BeregningService>()
+    private val behandlingHentOgPersisterService = mockk<BehandlingHentOgPersisterService>()
+    private val vilkårsvurderingService = mockk<VilkårsvurderingService>()
+    private val fagsakRepository = mockk<FagsakRepository>()
+    private val behandlingRepository = mockk<BehandlingRepository>()
+    private val tilkjentYtelseValideringService = mockk<TilkjentYtelseValideringService>()
+    private val arbeidsfordelingService = mockk<ArbeidsfordelingService>()
+
+    private val forvalterService =
+        ForvalterService(
+            økonomiService = økonomiService,
+            vedtakService = vedtakService,
+            beregningService = beregningService,
+            behandlingHentOgPersisterService = behandlingHentOgPersisterService,
+            fagsakRepository = fagsakRepository,
+            behandlingRepository = behandlingRepository,
+            tilkjentYtelseValideringService = tilkjentYtelseValideringService,
+            arbeidsfordelingService = arbeidsfordelingService,
+            vilkårsvurderingService = vilkårsvurderingService,
+            persongrunnlagService = persongrunnlagService,
+        )
 
     @Test
     fun `Skal endre periodeFom på vilkårresultat når den er før fødselsdato på person`() {