From 246b64fd456d7ec116b4d99c6a9f9aca8925ec48 Mon Sep 17 00:00:00 2001 From: Michael Gundersen <21148282+theagilepadawan@users.noreply.github.com> Date: Tue, 20 Jul 2021 13:03:48 +0200 Subject: [PATCH] R!! Bruker PdlOppslagGateway direkte i stedet for PersonGateway --- .../bruker/PersonGateway.java | 8 ---- .../bruker/adapter/PersonGatewayConfig.java | 16 ------- .../bruker/adapter/PersonGatewayImpl.java | 24 ----------- .../config/ApplicationConfig.java | 4 +- .../config/ServiceBeansConfig.java | 24 ++++++----- .../oppgave/OppgaveRouter.java | 5 +-- .../StartRegistreringStatusService.java | 10 ++--- .../bruker/adapter/PersonGatewayConfig.kt | 14 ------ .../bruker/adapter/PersonGatewayTest.kt | 43 ------------------- .../oppgave/OppgaveRouterTest.kt | 34 ++++++--------- .../StartRegistreringStatusServiceTest.kt | 10 ++--- 11 files changed, 40 insertions(+), 152 deletions(-) delete mode 100644 src/main/java/no/nav/fo/veilarbregistrering/bruker/PersonGateway.java delete mode 100644 src/main/java/no/nav/fo/veilarbregistrering/bruker/adapter/PersonGatewayConfig.java delete mode 100644 src/main/java/no/nav/fo/veilarbregistrering/bruker/adapter/PersonGatewayImpl.java delete mode 100644 src/test/java/no/nav/fo/veilarbregistrering/bruker/adapter/PersonGatewayConfig.kt delete mode 100644 src/test/java/no/nav/fo/veilarbregistrering/bruker/adapter/PersonGatewayTest.kt diff --git a/src/main/java/no/nav/fo/veilarbregistrering/bruker/PersonGateway.java b/src/main/java/no/nav/fo/veilarbregistrering/bruker/PersonGateway.java deleted file mode 100644 index 2b39f1ddf..000000000 --- a/src/main/java/no/nav/fo/veilarbregistrering/bruker/PersonGateway.java +++ /dev/null @@ -1,8 +0,0 @@ -package no.nav.fo.veilarbregistrering.bruker; - -import java.util.Optional; - -public interface PersonGateway { - - Optional hentGeografiskTilknytning(Bruker bruker); -} diff --git a/src/main/java/no/nav/fo/veilarbregistrering/bruker/adapter/PersonGatewayConfig.java b/src/main/java/no/nav/fo/veilarbregistrering/bruker/adapter/PersonGatewayConfig.java deleted file mode 100644 index 0ea6b5567..000000000 --- a/src/main/java/no/nav/fo/veilarbregistrering/bruker/adapter/PersonGatewayConfig.java +++ /dev/null @@ -1,16 +0,0 @@ -package no.nav.fo.veilarbregistrering.bruker.adapter; - - -import no.nav.fo.veilarbregistrering.bruker.PdlOppslagGateway; -import org.springframework.context.annotation.Bean; -import org.springframework.context.annotation.Configuration; - - -@Configuration -public class PersonGatewayConfig { - - @Bean - PersonGatewayImpl personGateway(PdlOppslagGateway pdlOppslagGateway) { - return new PersonGatewayImpl(pdlOppslagGateway); - } -} diff --git a/src/main/java/no/nav/fo/veilarbregistrering/bruker/adapter/PersonGatewayImpl.java b/src/main/java/no/nav/fo/veilarbregistrering/bruker/adapter/PersonGatewayImpl.java deleted file mode 100644 index 009371ec6..000000000 --- a/src/main/java/no/nav/fo/veilarbregistrering/bruker/adapter/PersonGatewayImpl.java +++ /dev/null @@ -1,24 +0,0 @@ -package no.nav.fo.veilarbregistrering.bruker.adapter; - -import no.nav.fo.veilarbregistrering.bruker.Bruker; -import no.nav.fo.veilarbregistrering.bruker.GeografiskTilknytning; -import no.nav.fo.veilarbregistrering.bruker.PdlOppslagGateway; -import no.nav.fo.veilarbregistrering.bruker.PersonGateway; - -import java.util.Optional; - - -class PersonGatewayImpl implements PersonGateway { - - private final PdlOppslagGateway pdlOppslagGateway; - - PersonGatewayImpl(PdlOppslagGateway pdlOppslagGateway) { - this.pdlOppslagGateway = pdlOppslagGateway; - } - - @Override - public Optional hentGeografiskTilknytning(Bruker bruker) { - Optional geografiskTilknytningPDL = pdlOppslagGateway.hentGeografiskTilknytning(bruker.getAktorId()); - return geografiskTilknytningPDL; - } -} diff --git a/src/main/java/no/nav/fo/veilarbregistrering/config/ApplicationConfig.java b/src/main/java/no/nav/fo/veilarbregistrering/config/ApplicationConfig.java index f1035d299..335712ea9 100644 --- a/src/main/java/no/nav/fo/veilarbregistrering/config/ApplicationConfig.java +++ b/src/main/java/no/nav/fo/veilarbregistrering/config/ApplicationConfig.java @@ -5,13 +5,12 @@ import no.nav.fo.veilarbregistrering.arbeidsforhold.adapter.ArbeidsforholdGatewayConfig; import no.nav.fo.veilarbregistrering.arbeidssoker.adapter.FormidlingsgruppeGatewayConfig; import no.nav.fo.veilarbregistrering.autorisasjon.PepConfig; -import no.nav.fo.veilarbregistrering.bruker.adapter.PersonGatewayConfig; import no.nav.fo.veilarbregistrering.bruker.krr.KrrConfig; import no.nav.fo.veilarbregistrering.bruker.pdl.PdlOppslagConfig; import no.nav.fo.veilarbregistrering.db.DatabaseConfig; -import no.nav.fo.veilarbregistrering.helsesjekk.HelsesjekkConfig; import no.nav.fo.veilarbregistrering.db.RepositoryConfig; import no.nav.fo.veilarbregistrering.enhet.adapter.EnhetGatewayConfig; +import no.nav.fo.veilarbregistrering.helsesjekk.HelsesjekkConfig; import no.nav.fo.veilarbregistrering.kafka.KafkaConfig; import no.nav.fo.veilarbregistrering.metrics.MetricsConfig; import no.nav.fo.veilarbregistrering.oppfolging.adapter.OppfolgingGatewayConfig; @@ -38,7 +37,6 @@ CacheConfig.class, UnleashConfig.class, MetricsConfig.class, - PersonGatewayConfig.class, ArbeidsforholdGatewayConfig.class, OppfolgingGatewayConfig.class, OppgaveGatewayConfig.class, diff --git a/src/main/java/no/nav/fo/veilarbregistrering/config/ServiceBeansConfig.java b/src/main/java/no/nav/fo/veilarbregistrering/config/ServiceBeansConfig.java index 635fe1992..e9794b6f3 100644 --- a/src/main/java/no/nav/fo/veilarbregistrering/config/ServiceBeansConfig.java +++ b/src/main/java/no/nav/fo/veilarbregistrering/config/ServiceBeansConfig.java @@ -13,7 +13,10 @@ import no.nav.fo.veilarbregistrering.arbeidssoker.resources.ArbeidssokerResource; import no.nav.fo.veilarbregistrering.arbeidssoker.resources.InternalArbeidssokerServlet; import no.nav.fo.veilarbregistrering.autorisasjon.AutorisasjonService; -import no.nav.fo.veilarbregistrering.bruker.*; +import no.nav.fo.veilarbregistrering.bruker.KontaktinfoService; +import no.nav.fo.veilarbregistrering.bruker.KrrGateway; +import no.nav.fo.veilarbregistrering.bruker.PdlOppslagGateway; +import no.nav.fo.veilarbregistrering.bruker.UserService; import no.nav.fo.veilarbregistrering.bruker.resources.InternalIdentServlet; import no.nav.fo.veilarbregistrering.bruker.resources.KontaktinfoResource; import no.nav.fo.veilarbregistrering.enhet.EnhetGateway; @@ -22,21 +25,24 @@ import no.nav.fo.veilarbregistrering.metrics.InfluxMetricsService; import no.nav.fo.veilarbregistrering.metrics.PrometheusMetricsService; import no.nav.fo.veilarbregistrering.oppfolging.OppfolgingGateway; -import no.nav.fo.veilarbregistrering.oppgave.*; +import no.nav.fo.veilarbregistrering.oppgave.OppgaveGateway; +import no.nav.fo.veilarbregistrering.oppgave.OppgaveRepository; +import no.nav.fo.veilarbregistrering.oppgave.OppgaveRouter; +import no.nav.fo.veilarbregistrering.oppgave.OppgaveService; import no.nav.fo.veilarbregistrering.oppgave.resources.OppgaveResource; import no.nav.fo.veilarbregistrering.orgenhet.Norg2Gateway; import no.nav.fo.veilarbregistrering.profilering.ProfileringRepository; import no.nav.fo.veilarbregistrering.profilering.ProfileringService; import no.nav.fo.veilarbregistrering.registrering.bruker.*; import no.nav.fo.veilarbregistrering.registrering.bruker.resources.RegistreringResource; -import no.nav.fo.veilarbregistrering.registrering.manuell.ManuellRegistreringRepository; -import no.nav.fo.veilarbregistrering.registrering.publisering.ArbeidssokerProfilertProducer; -import no.nav.fo.veilarbregistrering.registrering.publisering.ArbeidssokerRegistrertProducer; -import no.nav.fo.veilarbregistrering.registrering.publisering.PubliseringAvEventsService; import no.nav.fo.veilarbregistrering.registrering.formidling.RegistreringTilstandRepository; import no.nav.fo.veilarbregistrering.registrering.formidling.RegistreringTilstandService; import no.nav.fo.veilarbregistrering.registrering.formidling.resources.InternalRegistreringStatusServlet; import no.nav.fo.veilarbregistrering.registrering.formidling.resources.InternalRegistreringStatusoversiktServlet; +import no.nav.fo.veilarbregistrering.registrering.manuell.ManuellRegistreringRepository; +import no.nav.fo.veilarbregistrering.registrering.publisering.ArbeidssokerProfilertProducer; +import no.nav.fo.veilarbregistrering.registrering.publisering.ArbeidssokerRegistrertProducer; +import no.nav.fo.veilarbregistrering.registrering.publisering.PubliseringAvEventsService; import no.nav.fo.veilarbregistrering.sykemelding.SykemeldingGateway; import no.nav.fo.veilarbregistrering.sykemelding.SykemeldingService; import no.nav.fo.veilarbregistrering.sykemelding.resources.SykemeldingResource; @@ -92,12 +98,12 @@ BrukerTilstandService brukerTilstandService( StartRegistreringStatusService startRegistreringStatusService( ArbeidsforholdGateway arbeidsforholdGateway, BrukerTilstandService brukerTilstandService, - PersonGateway personGateway, + PdlOppslagGateway pdlOppslagGateway, InfluxMetricsService influxMetricsService) { return new StartRegistreringStatusService( arbeidsforholdGateway, brukerTilstandService, - personGateway, + pdlOppslagGateway, influxMetricsService); } @@ -217,14 +223,12 @@ OppgaveRouter oppgaveRouter( ArbeidsforholdGateway arbeidsforholdGateway, EnhetGateway enhetGateway, Norg2Gateway norg2Gateway, - PersonGateway personGateway, PdlOppslagGateway pdlOppslagGateway, InfluxMetricsService influxMetricsService) { return new OppgaveRouter( arbeidsforholdGateway, enhetGateway, norg2Gateway, - personGateway, pdlOppslagGateway, influxMetricsService); } diff --git a/src/main/java/no/nav/fo/veilarbregistrering/oppgave/OppgaveRouter.java b/src/main/java/no/nav/fo/veilarbregistrering/oppgave/OppgaveRouter.java index 7b8c6afde..8f9e0c571 100644 --- a/src/main/java/no/nav/fo/veilarbregistrering/oppgave/OppgaveRouter.java +++ b/src/main/java/no/nav/fo/veilarbregistrering/oppgave/OppgaveRouter.java @@ -37,7 +37,6 @@ public class OppgaveRouter { private final ArbeidsforholdGateway arbeidsforholdGateway; private final EnhetGateway enhetGateway; private final Norg2Gateway norg2Gateway; - private final PersonGateway personGateway; private final PdlOppslagGateway pdlOppslagGateway; private final InfluxMetricsService influxMetricsService; @@ -45,13 +44,11 @@ public OppgaveRouter( ArbeidsforholdGateway arbeidsforholdGateway, EnhetGateway enhetGateway, Norg2Gateway norg2Gateway, - PersonGateway personGateway, PdlOppslagGateway pdlOppslagGateway, InfluxMetricsService influxMetricsService) { this.arbeidsforholdGateway = arbeidsforholdGateway; this.enhetGateway = enhetGateway; this.norg2Gateway = norg2Gateway; - this.personGateway = personGateway; this.pdlOppslagGateway = pdlOppslagGateway; this.influxMetricsService = influxMetricsService; } @@ -64,7 +61,7 @@ public Optional hentEnhetsnummerFor(Bruker bruker) { Optional geografiskTilknytning; try { - geografiskTilknytning = personGateway.hentGeografiskTilknytning(bruker); + geografiskTilknytning = pdlOppslagGateway.hentGeografiskTilknytning(bruker.getAktorId()); } catch (RuntimeException e) { LOG.warn("Henting av geografisk tilknytning feilet", e); influxMetricsService.reportTags(OPPGAVE_ROUTING_EVENT, GeografiskTilknytning_Feilet); diff --git a/src/main/java/no/nav/fo/veilarbregistrering/registrering/bruker/StartRegistreringStatusService.java b/src/main/java/no/nav/fo/veilarbregistrering/registrering/bruker/StartRegistreringStatusService.java index 84a6de9a3..9d06998e1 100644 --- a/src/main/java/no/nav/fo/veilarbregistrering/registrering/bruker/StartRegistreringStatusService.java +++ b/src/main/java/no/nav/fo/veilarbregistrering/registrering/bruker/StartRegistreringStatusService.java @@ -3,7 +3,7 @@ import no.nav.fo.veilarbregistrering.arbeidsforhold.ArbeidsforholdGateway; import no.nav.fo.veilarbregistrering.bruker.Bruker; import no.nav.fo.veilarbregistrering.bruker.GeografiskTilknytning; -import no.nav.fo.veilarbregistrering.bruker.PersonGateway; +import no.nav.fo.veilarbregistrering.bruker.PdlOppslagGateway; import no.nav.fo.veilarbregistrering.metrics.Events; import no.nav.fo.veilarbregistrering.metrics.InfluxMetricsService; import no.nav.fo.veilarbregistrering.registrering.bruker.resources.StartRegistreringStatusDto; @@ -23,17 +23,17 @@ public class StartRegistreringStatusService { private final ArbeidsforholdGateway arbeidsforholdGateway; private final BrukerTilstandService brukerTilstandService; - private final PersonGateway personGateway; + private final PdlOppslagGateway pdlOppslagGateway; private final InfluxMetricsService influxMetricsService; public StartRegistreringStatusService( ArbeidsforholdGateway arbeidsforholdGateway, BrukerTilstandService brukerTilstandService, - PersonGateway personGateway, + PdlOppslagGateway pdlOppslagGateway, InfluxMetricsService influxMetricsService) { this.arbeidsforholdGateway = arbeidsforholdGateway; this.brukerTilstandService = brukerTilstandService; - this.personGateway = personGateway; + this.pdlOppslagGateway = pdlOppslagGateway; this.influxMetricsService = influxMetricsService; } @@ -70,7 +70,7 @@ private Optional hentGeografiskTilknytning(Bruker bruker) Optional geografiskTilknytning = Optional.empty(); try { long t1 = System.currentTimeMillis(); - geografiskTilknytning = personGateway.hentGeografiskTilknytning(bruker); + geografiskTilknytning = pdlOppslagGateway.hentGeografiskTilknytning(bruker.getAktorId()); LOG.info("Henting av geografisk tilknytning tok {} ms.", System.currentTimeMillis() - t1); } catch (RuntimeException e) { diff --git a/src/test/java/no/nav/fo/veilarbregistrering/bruker/adapter/PersonGatewayConfig.kt b/src/test/java/no/nav/fo/veilarbregistrering/bruker/adapter/PersonGatewayConfig.kt deleted file mode 100644 index 8f6fd7b74..000000000 --- a/src/test/java/no/nav/fo/veilarbregistrering/bruker/adapter/PersonGatewayConfig.kt +++ /dev/null @@ -1,14 +0,0 @@ -package no.nav.fo.veilarbregistrering.bruker.adapter - -import no.nav.fo.veilarbregistrering.bruker.PersonGateway -import org.springframework.context.annotation.Bean -import org.springframework.context.annotation.Configuration -import java.util.* - -@Configuration -class PersonGatewayConfig { - @Bean - fun personGateway(): PersonGateway { - return PersonGateway { Optional.empty() } - } -} diff --git a/src/test/java/no/nav/fo/veilarbregistrering/bruker/adapter/PersonGatewayTest.kt b/src/test/java/no/nav/fo/veilarbregistrering/bruker/adapter/PersonGatewayTest.kt deleted file mode 100644 index c08800790..000000000 --- a/src/test/java/no/nav/fo/veilarbregistrering/bruker/adapter/PersonGatewayTest.kt +++ /dev/null @@ -1,43 +0,0 @@ -package no.nav.fo.veilarbregistrering.bruker.adapter - -import io.mockk.every -import io.mockk.mockk -import io.mockk.mockkStatic -import no.finn.unleash.UnleashContext -import no.nav.common.featuretoggle.UnleashClient -import no.nav.common.health.HealthCheckResult -import no.nav.fo.veilarbregistrering.bruker.* -import no.nav.fo.veilarbregistrering.config.RequestContext -import no.nav.fo.veilarbregistrering.config.RequestContext.servletRequest -import org.assertj.core.api.Assertions.assertThat -import org.junit.jupiter.api.BeforeEach -import org.junit.jupiter.api.Test -import org.junit.jupiter.api.extension.ExtendWith -import org.mockserver.integration.ClientAndServer -import org.mockserver.junit.jupiter.MockServerExtension -import org.mockserver.model.HttpRequest -import org.mockserver.model.HttpResponse -import org.mockserver.model.MediaType -import java.util.* -import javax.servlet.http.HttpServletRequest - -class PersonGatewayTest { - - private fun lagPdlOppslagGateway(gt: GeografiskTilknytning? = null): PdlOppslagGateway { - val pdlOppslagGatewayMock = mockk() - every { pdlOppslagGatewayMock.hentGeografiskTilknytning(any())} returns Optional.ofNullable(gt) - return pdlOppslagGatewayMock - } - - @Test - fun `hentGeografiskTilknytning skal returnere geografisk tilknytning fra PDL`() { - val foedselsnummer = Foedselsnummer.of("12345678910") - val bruker = Bruker.of(foedselsnummer, null) - val forventetGeografiskTilknytning = "1234" - - val personGateway = PersonGatewayImpl(lagPdlOppslagGateway(GeografiskTilknytning.of(forventetGeografiskTilknytning))) - - val geografiskTilknytning = personGateway.hentGeografiskTilknytning(bruker) - assertThat(geografiskTilknytning).hasValue(GeografiskTilknytning.of(forventetGeografiskTilknytning)) - } -} diff --git a/src/test/java/no/nav/fo/veilarbregistrering/oppgave/OppgaveRouterTest.kt b/src/test/java/no/nav/fo/veilarbregistrering/oppgave/OppgaveRouterTest.kt index 3814d0103..c1027380f 100644 --- a/src/test/java/no/nav/fo/veilarbregistrering/oppgave/OppgaveRouterTest.kt +++ b/src/test/java/no/nav/fo/veilarbregistrering/oppgave/OppgaveRouterTest.kt @@ -61,18 +61,17 @@ class OppgaveRouterTest { @Test fun `geografisk tilknytning med by med bydel skal gi intern brukerstotte`() { - val personGateway = PersonGateway { Optional.of(GeografiskTilknytning.of("0301")) } - - val oppgaveRouter = oppgaveRouter(personGateway = personGateway) + val pdlOppslagGateway = StubPdlOppslagGateway(geografiskTilknytning = GeografiskTilknytning.of("0301")) + val oppgaveRouter = oppgaveRouter(pdlOppslagGateway = pdlOppslagGateway) val enhetsnr = oppgaveRouter.hentEnhetsnummerFor(BRUKER) assertThat(enhetsnr).hasValue(Enhetnr.internBrukerstotte()) } @Test fun `geografisk tilknytning med unntak av landkode skal gi empty enhetsnummer`() { - val personGateway = PersonGateway { Optional.of(GeografiskTilknytning.of("030106")) } + val pdlOppslagGateway = StubPdlOppslagGateway(geografiskTilknytning = GeografiskTilknytning.of("030106")) - val oppgaveRouter = oppgaveRouter(personGateway = personGateway) + val oppgaveRouter = oppgaveRouter(pdlOppslagGateway = pdlOppslagGateway) val enhetsnr = oppgaveRouter.hentEnhetsnummerFor(BRUKER) assertThat(enhetsnr).isEmpty } @@ -83,9 +82,9 @@ class OppgaveRouterTest { Kommunenummer.of("1241"), Periode.of(LocalDate.of(2020, 1, 1), null)) val enhetGateway = EnhetGateway { Organisasjonsdetaljer.of(listOf(forretningsadresse), emptyList()) } - val personGateway = PersonGateway { Optional.of(GeografiskTilknytning.of("DNK")) } + val pdlOppslagGateway = StubPdlOppslagGateway(geografiskTilknytning = GeografiskTilknytning.of("DNK")) - val oppgaveRouter = oppgaveRouter(enhetGateway = enhetGateway, personGateway = personGateway) + val oppgaveRouter = oppgaveRouter(enhetGateway = enhetGateway, pdlOppslagGateway = pdlOppslagGateway) val enhetsnr = oppgaveRouter.hentEnhetsnummerFor(BRUKER) assertThat(enhetsnr).hasValue(Enhetnr.of("232")) } @@ -96,9 +95,9 @@ class OppgaveRouterTest { Kommunenummer.of(KommuneMedBydel.STAVANGER), Periode.of(LocalDate.of(2020, 1, 1), null)) val enhetGateway = EnhetGateway { Organisasjonsdetaljer.of(listOf(forretningsadresse), emptyList()) } - val personGateway = PersonGateway { Optional.of(GeografiskTilknytning.of("DNK")) } + val pdlOppslagGateway = StubPdlOppslagGateway(geografiskTilknytning = GeografiskTilknytning.of("DNK")) - val oppgaveRouter = oppgaveRouter(enhetGateway = enhetGateway, personGateway = personGateway) + val oppgaveRouter = oppgaveRouter(enhetGateway = enhetGateway, pdlOppslagGateway = pdlOppslagGateway) val enhetsnr = oppgaveRouter.hentEnhetsnummerFor(BRUKER) assertThat(enhetsnr).hasValue(Enhetnr.internBrukerstotte()) } @@ -122,10 +121,10 @@ class OppgaveRouterTest { } private fun hentEnhetsnummerForBrukerMedAdressebeskyttelse(adressebeskyttelseGradering: AdressebeskyttelseGradering): Optional { - val personGateway = PersonGateway { Optional.of(GeografiskTilknytning.of("0301")) } val person = Person.of(null, null, adressebeskyttelseGradering) - val pdlOppslagGateway = StubPdlOppslagGateway(users = mapOf(BRUKER.aktorId to person)) - val oppgaveRouter = oppgaveRouter(personGateway = personGateway, pdlOppslagGateway = pdlOppslagGateway) + val pdlOppslagGateway = StubPdlOppslagGateway(geografiskTilknytning = GeografiskTilknytning.of("0301"), + users = mapOf(BRUKER.aktorId to person)) + val oppgaveRouter = oppgaveRouter(pdlOppslagGateway = pdlOppslagGateway) return oppgaveRouter.hentEnhetsnummerFor(BRUKER) } @@ -134,11 +133,10 @@ class OppgaveRouterTest { arbeidsforholdGateway: ArbeidsforholdGateway = StubArbeidsforholdGateway(), enhetGateway: EnhetGateway = EnhetGateway { null }, norg2Gateway: Norg2Gateway = StubNorg2Gateway(), - personGateway: PersonGateway = StubPersonGateway(), pdlOppslagGateway: PdlOppslagGateway = StubPdlOppslagGateway(), influxMetricsService: InfluxMetricsService = mockk(relaxed = true) ) = - OppgaveRouter(arbeidsforholdGateway, enhetGateway, norg2Gateway, personGateway, pdlOppslagGateway, influxMetricsService) + OppgaveRouter(arbeidsforholdGateway, enhetGateway, norg2Gateway, pdlOppslagGateway, influxMetricsService) private class StubNorg2Gateway : Norg2Gateway { override fun hentEnhetFor(kommunenummer: Kommunenummer): Optional { @@ -158,11 +156,7 @@ class OppgaveRouterTest { FlereArbeidsforholdTestdataBuilder.flereArbeidsforholdTilfeldigSortert()!! } - private class StubPersonGateway : PersonGateway { - override fun hentGeografiskTilknytning(bruker: Bruker): Optional? = Optional.empty() - } - - private class StubPdlOppslagGateway(private val users: Map = emptyMap()) : PdlOppslagGateway { + private class StubPdlOppslagGateway(private val geografiskTilknytning: GeografiskTilknytning? = null ,private val users: Map = emptyMap()) : PdlOppslagGateway { override fun hentPerson(aktorId: AktorId?) = Optional.ofNullable(users[aktorId]) override fun hentIdenter(fnr: Foedselsnummer?): Identer { @@ -173,7 +167,7 @@ class OppgaveRouterTest { TODO("Not yet implemented") } - override fun hentGeografiskTilknytning(aktorId: AktorId?) = TODO("Not yet implemented") + override fun hentGeografiskTilknytning(aktorId: AktorId?) = Optional.ofNullable(geografiskTilknytning) } diff --git a/src/test/java/no/nav/fo/veilarbregistrering/registrering/bruker/StartRegistreringStatusServiceTest.kt b/src/test/java/no/nav/fo/veilarbregistrering/registrering/bruker/StartRegistreringStatusServiceTest.kt index 84efd313f..ce1a77c31 100644 --- a/src/test/java/no/nav/fo/veilarbregistrering/registrering/bruker/StartRegistreringStatusServiceTest.kt +++ b/src/test/java/no/nav/fo/veilarbregistrering/registrering/bruker/StartRegistreringStatusServiceTest.kt @@ -21,18 +21,18 @@ class StartRegistreringStatusServiceTest { private lateinit var brukerRegistreringService: StartRegistreringStatusService private lateinit var arbeidsforholdGateway: ArbeidsforholdGateway private lateinit var oppfolgingClient: OppfolgingClient - private lateinit var personGateway: PersonGateway + private lateinit var pdlOppslagGateway: PdlOppslagGateway @BeforeEach fun setup() { arbeidsforholdGateway = mockk() oppfolgingClient = mockk() - personGateway = mockk() + pdlOppslagGateway = mockk() val influxMetricsService: InfluxMetricsService = mockk(relaxed = true) val oppfolgingGateway = OppfolgingGatewayImpl(oppfolgingClient) brukerRegistreringService = StartRegistreringStatusService( arbeidsforholdGateway, BrukerTilstandService(oppfolgingGateway, mockk(relaxed = true)), - personGateway, + pdlOppslagGateway, influxMetricsService ) } @@ -96,7 +96,7 @@ class StartRegistreringStatusServiceTest { fun gitt_at_geografiskTilknytning_er_1234_skal_1234_returneres() { mockInaktivBrukerUtenReaktivering() mockArbeidssforholdSomOppfyllerBetingelseOmArbeidserfaring() - every { personGateway.hentGeografiskTilknytning(any()) } returns + every { pdlOppslagGateway.hentGeografiskTilknytning(any()) } returns Optional.of(GeografiskTilknytning.of("1234")) val startRegistreringStatus = getStartRegistreringStatus(BRUKER_INTERN) Assertions.assertThat(startRegistreringStatus).isNotNull @@ -107,7 +107,7 @@ class StartRegistreringStatusServiceTest { fun gitt_at_geografiskTilknytning_kaster_exception_skal_null_returneres() { mockInaktivBrukerUtenReaktivering() mockArbeidssforholdSomOppfyllerBetingelseOmArbeidserfaring() - every { personGateway.hentGeografiskTilknytning(any()) } throws RuntimeException("Ikke tilgang") + every { pdlOppslagGateway.hentGeografiskTilknytning(any()) } throws RuntimeException("Ikke tilgang") val startRegistreringStatus = getStartRegistreringStatus(BRUKER_INTERN) Assertions.assertThat(startRegistreringStatus).isNotNull Assertions.assertThat(startRegistreringStatus.geografiskTilknytning).isNull()