From b1f7b895921912e80d24e8d95d479bc29d2d5eb9 Mon Sep 17 00:00:00 2001 From: Hans Petter Simonsen Date: Mon, 21 Oct 2024 13:33:19 +0200 Subject: [PATCH] =?UTF-8?q?Slutte=20=C3=A5=20bruke=20deprekert=20MockNavSe?= =?UTF-8?q?rvice=20(#833)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * Slutte å bruke deprekert MockNavService * Use with instead of builder in kotlin classes * Fiks tester --------- Co-authored-by: sigurdgroneng Co-authored-by: johannetronstad --- build.gradle.kts | 4 +- .../avtalt_med_nav/AvtaltMedNavTest.java | 7 ++- .../BrukernotifikasjonAsserts.java | 7 --- .../BrukernotifikasjonTest.java | 35 +++++++------ .../controller/AktivitetsplanRSTest.java | 15 +++--- .../internapi/InternApiControllerTest.java | 4 +- .../mock_nav_modell/BrukerOptions.java | 2 + .../mock_nav_modell/MockNavService.java | 5 -- .../veilarbaktivitet/motesms/MoteSmsTest.java | 14 +++--- .../DelingAvCvManueltAvbruttServiceTest.java | 2 +- .../StillingFraNavControllerITest.java | 4 +- .../AktiviteterTilKafkaAivenServiceTest.java | 4 +- .../admin/KasserControllerTest.kt | 4 +- .../aktivitet/AktivitetsplanControllerTest.kt | 47 +++++++++--------- .../AktivitetskortConsumerIntegrationTest.kt | 2 +- .../graphql/AktivitetskortControllerTest.kt | 11 ++--- .../arkivering/ArkiveringsControllerTest.kt | 7 ++- .../InnsynrettControllerTest.kt | 14 +++--- .../mock_nav_modell/NavMockService.kt | 49 ++++++++++++++----- 19 files changed, 125 insertions(+), 112 deletions(-) diff --git a/build.gradle.kts b/build.gradle.kts index b6608cb1b..25ee30d66 100644 --- a/build.gradle.kts +++ b/build.gradle.kts @@ -17,8 +17,8 @@ plugins { id("jacoco") id("org.sonarqube") version "4.4.1.3373" id("org.springframework.boot") version "3.0.2" - kotlin("plugin.lombok") version "2.0.0" - id("io.freefair.lombok") version "8.1.0" + kotlin("plugin.lombok") version "2.0.21" + id("io.freefair.lombok") version "8.10" } java { diff --git a/src/test/java/no/nav/veilarbaktivitet/avtalt_med_nav/AvtaltMedNavTest.java b/src/test/java/no/nav/veilarbaktivitet/avtalt_med_nav/AvtaltMedNavTest.java index caf1ab20c..7371d4a51 100644 --- a/src/test/java/no/nav/veilarbaktivitet/avtalt_med_nav/AvtaltMedNavTest.java +++ b/src/test/java/no/nav/veilarbaktivitet/avtalt_med_nav/AvtaltMedNavTest.java @@ -8,6 +8,7 @@ import no.nav.veilarbaktivitet.mock_nav_modell.MockBruker; import no.nav.veilarbaktivitet.mock_nav_modell.MockNavService; import no.nav.veilarbaktivitet.mock_nav_modell.MockVeileder; +import no.nav.veilarbaktivitet.mock_nav_modell.NavMockService; import no.nav.veilarbaktivitet.testutils.AktivitetAssertUtils; import no.nav.veilarbaktivitet.testutils.AktivitetDtoTestBuilder; import org.junit.jupiter.api.BeforeEach; @@ -19,6 +20,8 @@ class AvtaltMedNavTest extends SpringBootTestBase { @Autowired BrukernotifikasjonAssertsConfig config; BrukernotifikasjonAsserts brukernotifikasjonAsserts; + @Autowired + NavMockService navMockService; @BeforeEach void before() { @@ -27,7 +30,7 @@ void before() { @Test void IkkeSendeFhoForBrukerSomIkkeKanVarsles() { - MockBruker brukerSomIkkeKanVarsles = BrukernotifikasjonAsserts.getBrukerSomIkkeKanVarsles(); + MockBruker brukerSomIkkeKanVarsles = navMockService.getBrukerSomIkkeKanVarsles(); MockVeileder veileder = MockNavService.createVeileder(brukerSomIkkeKanVarsles); ForhaandsorienteringDTO fho = ForhaandsorienteringDTO @@ -111,7 +114,7 @@ void IkkeOppretteFHOFEilAktivitetVersion() { @Test void setteAvtaltUtenFHOForBrukerSomIkkeKanVarsles() { - MockBruker brukerSomIkkeKanVarsles = BrukernotifikasjonAsserts.getBrukerSomIkkeKanVarsles(); + MockBruker brukerSomIkkeKanVarsles = navMockService.getBrukerSomIkkeKanVarsles(); MockVeileder veileder = MockNavService.createVeileder(brukerSomIkkeKanVarsles); ForhaandsorienteringDTO fho = ForhaandsorienteringDTO diff --git a/src/test/java/no/nav/veilarbaktivitet/brukernotifikasjon/BrukernotifikasjonAsserts.java b/src/test/java/no/nav/veilarbaktivitet/brukernotifikasjon/BrukernotifikasjonAsserts.java index edc4fa34c..9fcd5ec07 100644 --- a/src/test/java/no/nav/veilarbaktivitet/brukernotifikasjon/BrukernotifikasjonAsserts.java +++ b/src/test/java/no/nav/veilarbaktivitet/brukernotifikasjon/BrukernotifikasjonAsserts.java @@ -7,9 +7,6 @@ import no.nav.brukernotifikasjon.schemas.input.OppgaveInput; import no.nav.doknotifikasjon.schemas.DoknotifikasjonStatus; import no.nav.veilarbaktivitet.config.kafka.kafkatemplates.KafkaStringAvroTemplate; -import no.nav.veilarbaktivitet.mock_nav_modell.BrukerOptions; -import no.nav.veilarbaktivitet.mock_nav_modell.MockBruker; -import no.nav.veilarbaktivitet.mock_nav_modell.MockNavService; import no.nav.veilarbaktivitet.person.Person; import no.nav.veilarbaktivitet.util.KafkaTestService; import org.apache.avro.specific.SpecificRecord; @@ -34,10 +31,6 @@ public class BrukernotifikasjonAsserts { BrukernotifikasjonAssertsConfig config; KafkaTestService kafkaTestService; - public static MockBruker getBrukerSomIkkeKanVarsles() { - return MockNavService.createBruker(BrukerOptions.happyBrukerBuilder().erManuell(true).build()); - } - public BrukernotifikasjonAsserts(BrukernotifikasjonAssertsConfig config) { oppgaveConsumer = config.createOppgaveConsumer(); beskjedConsumer = config.createBeskjedConsumer(); diff --git a/src/test/java/no/nav/veilarbaktivitet/brukernotifikasjon/BrukernotifikasjonTest.java b/src/test/java/no/nav/veilarbaktivitet/brukernotifikasjon/BrukernotifikasjonTest.java index e209e3344..356aa6836 100644 --- a/src/test/java/no/nav/veilarbaktivitet/brukernotifikasjon/BrukernotifikasjonTest.java +++ b/src/test/java/no/nav/veilarbaktivitet/brukernotifikasjon/BrukernotifikasjonTest.java @@ -27,7 +27,6 @@ import no.nav.veilarbaktivitet.db.DbTestUtils; import no.nav.veilarbaktivitet.mock_nav_modell.BrukerOptions; import no.nav.veilarbaktivitet.mock_nav_modell.MockBruker; -import no.nav.veilarbaktivitet.mock_nav_modell.MockNavService; import no.nav.veilarbaktivitet.testutils.AktivitetDataTestBuilder; import no.nav.veilarbaktivitet.util.AktivitetTestService; import no.nav.veilarbaktivitet.util.KafkaTestService; @@ -122,7 +121,7 @@ void assertNoUnkowns() { @SneakyThrows @Test void happy_case_oppgave() { - MockBruker mockBruker = MockNavService.createHappyBruker(); + MockBruker mockBruker = navMockService.createBruker(BrukerOptions.happyBruker()); AktivitetData aktivitetData = AktivitetDataTestBuilder.nyEgenaktivitet(); AktivitetDTO skalOpprettes = AktivitetDTOMapper.mapTilAktivitetDTO(aktivitetData, false); AktivitetDTO aktivitetDTO = aktivitetTestService.opprettAktivitet(mockBruker, skalOpprettes); @@ -135,7 +134,7 @@ void happy_case_oppgave() { @SneakyThrows @Test void skalSendeOppgaveMedEgentTekst() { - MockBruker mockBruker = MockNavService.createHappyBruker(); + MockBruker mockBruker = navMockService.createBruker(BrukerOptions.happyBruker()); AktivitetData aktivitetData = AktivitetDataTestBuilder.nyEgenaktivitet(); AktivitetDTO skalOpprettes = AktivitetDTOMapper.mapTilAktivitetDTO(aktivitetData, false); AktivitetDTO aktivitetDTO = aktivitetTestService.opprettAktivitet(mockBruker, skalOpprettes); @@ -176,7 +175,7 @@ void skalSendeOppgaveMedEgentTekst() { @SneakyThrows @Test void skalSendeBeskjedMedEgentTekst() { - MockBruker mockBruker = MockNavService.createHappyBruker(); + MockBruker mockBruker = navMockService.createHappyBruker(); AktivitetData aktivitetData = AktivitetDataTestBuilder.nyEgenaktivitet(); AktivitetDTO skalOpprettes = AktivitetDTOMapper.mapTilAktivitetDTO(aktivitetData, false); AktivitetDTO aktivitetDTO = aktivitetTestService.opprettAktivitet(mockBruker, skalOpprettes); @@ -211,7 +210,7 @@ void skalSendeBeskjedMedEgentTekst() { @SneakyThrows @Test void skal_sendeBeskjed() { - MockBruker mockBruker = MockNavService.createHappyBruker(); + MockBruker mockBruker = navMockService.createHappyBruker(); AktivitetData aktivitetData = AktivitetDataTestBuilder.nyEgenaktivitet(); AktivitetDTO skalOpprettes = AktivitetDTOMapper.mapTilAktivitetDTO(aktivitetData, false); AktivitetDTO aktivitetDTO = aktivitetTestService.opprettAktivitet(mockBruker, skalOpprettes); @@ -237,7 +236,7 @@ void skal_sendeBeskjed() { @Test void skal_ikke_produsere_meldinger_for_avsluttet_oppgave() { - MockBruker mockBruker = MockNavService.createHappyBruker(); + MockBruker mockBruker = navMockService.createHappyBruker(); AktivitetData aktivitetData = AktivitetDataTestBuilder.nyEgenaktivitet(); AktivitetDTO skalOpprettes = AktivitetDTOMapper.mapTilAktivitetDTO(aktivitetData, false); AktivitetDTO aktivitetDTO = aktivitetTestService.opprettAktivitet(mockBruker, skalOpprettes); @@ -254,7 +253,7 @@ void skal_ikke_produsere_meldinger_for_avsluttet_oppgave() { @Test void skal_ikke_sende_meldinger_for_avbrutte_aktiviteter() { - MockBruker mockBruker = MockNavService.createHappyBruker(); + MockBruker mockBruker = navMockService.createHappyBruker(); AktivitetData aktivitetData = AktivitetDataTestBuilder.nyEgenaktivitet(); AktivitetDTO skalOpprettes = AktivitetDTOMapper.mapTilAktivitetDTO(aktivitetData, false); @@ -293,7 +292,7 @@ void skal_ikke_sende_meldinger_for_avbrutte_aktiviteter() { @Test void skal_lukke_brukernotifikasjonsOppgave_nar_aktivitet_blir_avbrutt() { - MockBruker mockBruker = MockNavService.createHappyBruker(); + MockBruker mockBruker = navMockService.createHappyBruker(); AktivitetData aktivitetData = AktivitetDataTestBuilder.nyEgenaktivitet(); AktivitetDTO skalOpprettes = AktivitetDTOMapper.mapTilAktivitetDTO(aktivitetData, false); AktivitetDTO aktivitetDTO = aktivitetTestService.opprettAktivitet(mockBruker, skalOpprettes); @@ -343,8 +342,8 @@ private void oppgaveSendtOk(ConsumerRecord oppgaveRec @Test void skal_kunne_opprette_brukernotifikasjon_pa_fho_pa_arena_aktiviteter_som_ikke_er_migrert_og_ha_lenke_med_riktig_id() { - var mockBruker = navMockService.createHappyBruker(BrukerOptions.happyBruker()); - var mockVeileder = MockNavService.createVeileder(mockBruker); + var mockBruker = navMockService.createBruker(BrukerOptions.happyBruker()); + var mockVeileder = navMockService.createVeileder(mockBruker); var arenaId = new ArenaId("ARENATA123"); aktivitetTestService.opprettFHOForArenaAktivitet(mockBruker, arenaId, mockVeileder); @@ -357,8 +356,8 @@ void skal_kunne_opprette_brukernotifikasjon_pa_fho_pa_arena_aktiviteter_som_ikke @Test void skal_kunne_opprette_brukernotifications_pa_fho_pa_arena_aktiviteter_som_ER_migrert_og_ha_lenke_med_riktig_id() { when(unleash.isEnabled(MigreringService.VIS_MIGRERTE_ARENA_AKTIVITETER_TOGGLE)).thenReturn(true); - var mockBruker = navMockService.createHappyBruker(BrukerOptions.happyBruker()); - var mockVeileder = MockNavService.createVeileder(mockBruker); + var mockBruker = navMockService.createBruker(BrukerOptions.happyBruker()); + var mockVeileder = navMockService.createVeileder(mockBruker); var arenaId = new ArenaId("ARENATA123"); // Opprett ekstern aktivitet var aktivitetskortMelding = new KafkaAktivitetskortWrapperDTO( @@ -387,8 +386,8 @@ void skal_kunne_opprette_brukernotifications_pa_fho_pa_arena_aktiviteter_som_ER_ @Test void skal_lukke_brukernotifikasjonsOppgave_nar_eksterne_lonnstilskudd_blir_avbrutt() { - var mockBruker = navMockService.createHappyBruker(BrukerOptions.happyBruker()); - var mockVeileder = MockNavService.createVeileder(mockBruker); + var mockBruker = navMockService.createBruker(BrukerOptions.happyBruker()); + var mockVeileder = navMockService.createVeileder(mockBruker); // Opprett ekstern aktivitet var serie = new AktivitetskortSerie(mockBruker, AktivitetskortType.MIDLERTIDIG_LONNSTILSKUDD); var aktivitetskortMelding = serie.ny(AktivitetskortStatus.GJENNOMFORES, ZonedDateTime.now()); @@ -410,8 +409,8 @@ void skal_lukke_brukernotifikasjonsOppgave_nar_eksterne_lonnstilskudd_blir_avbru @Test void skal_lukke_brukernotifikasjonsOppgave_nar_eksterne_arena_tiltak_blir_avbrutt() { - var mockBruker = navMockService.createHappyBruker(BrukerOptions.happyBruker()); - var mockVeileder = MockNavService.createVeileder(mockBruker); + var mockBruker = navMockService.createBruker(BrukerOptions.happyBruker()); + var mockVeileder = navMockService.createVeileder(mockBruker); var serie = ArenaAktivitetskortSerie.of(mockBruker, "MIDL"); // Opprett FHO aktivitetTestService.opprettFHOForArenaAktivitet(mockBruker, serie.getArenaId(), mockVeileder); @@ -427,8 +426,8 @@ void skal_lukke_brukernotifikasjonsOppgave_nar_eksterne_arena_tiltak_blir_avbrut @Test void skal_lukke_brukernotifikasjonsOppgave_nar_eksterne_arena_tiltak_blir_avbrutt_men_fho_opprettet_etter_migrering() { - var mockBruker = navMockService.createHappyBruker(BrukerOptions.happyBruker()); - var mockVeileder = MockNavService.createVeileder(mockBruker); + var mockBruker = navMockService.createBruker(BrukerOptions.happyBruker()); + var mockVeileder = navMockService.createVeileder(mockBruker); // Opprett ekstern aktivitet og avbryter den var serie = ArenaAktivitetskortSerie.of(mockBruker, "MIDL"); var aktivitetskortMelding = serie.ny(AktivitetskortStatus.GJENNOMFORES, ZonedDateTime.now()); diff --git a/src/test/java/no/nav/veilarbaktivitet/controller/AktivitetsplanRSTest.java b/src/test/java/no/nav/veilarbaktivitet/controller/AktivitetsplanRSTest.java index e052d08fe..1b7b45960 100644 --- a/src/test/java/no/nav/veilarbaktivitet/controller/AktivitetsplanRSTest.java +++ b/src/test/java/no/nav/veilarbaktivitet/controller/AktivitetsplanRSTest.java @@ -21,7 +21,6 @@ import no.nav.veilarbaktivitet.db.DbTestUtils; import no.nav.veilarbaktivitet.mock_nav_modell.BrukerOptions; import no.nav.veilarbaktivitet.mock_nav_modell.MockBruker; -import no.nav.veilarbaktivitet.mock_nav_modell.MockNavService; import no.nav.veilarbaktivitet.mock_nav_modell.MockVeileder; import org.junit.jupiter.api.AfterEach; import org.junit.jupiter.api.BeforeEach; @@ -74,9 +73,9 @@ class AktivitetsplanRSTest extends SpringBootTestBase { @BeforeEach void moreSettup() { - mockBruker = navMockService.createHappyBruker(BrukerOptions.happyBruker()); - mockBrukersVeileder = MockNavService.createVeileder(mockBruker); - annenMockVeilederMedNasjonalTilgang = MockNavService.createVeilederMedNasjonalTilgang(); + mockBruker = navMockService.createBruker(BrukerOptions.happyBruker()); + mockBrukersVeileder = navMockService.createVeileder(mockBruker); + annenMockVeilederMedNasjonalTilgang = navMockService.createVeilederMedNasjonalTilgang(); aktivVeileder = mockBrukersVeileder; } @@ -235,13 +234,13 @@ private void gitt_at_jeg_har_aktiviter() { private void gitt_at_jeg_har_aktiviteter_med_kontorsperre() { var enableKvp = mockBruker.getBrukerOptions().toBuilder().erUnderKvp(true).build(); - MockNavService.updateBruker(mockBruker, enableKvp); + navMockService.updateBruker(mockBruker, enableKvp); gitt_at_jeg_har_folgende_aktiviteter(List.of( nyttStillingssok(), nyttStillingssok() )); var removeKvp = mockBruker.getBrukerOptions().toBuilder().erUnderKvp(false).build(); - MockNavService.updateBruker(mockBruker, removeKvp); + navMockService.updateBruker(mockBruker, removeKvp); gitt_at_jeg_har_folgende_aktiviteter(List.of( nyttStillingssok(), nyttStillingssok() @@ -250,10 +249,10 @@ private void gitt_at_jeg_har_aktiviteter_med_kontorsperre() { private void gitt_at_jeg_har_en_aktivitet_med_kontorsperre() { var enableKvp = mockBruker.getBrukerOptions().toBuilder().erUnderKvp(true).build(); - MockNavService.updateBruker(mockBruker, enableKvp); + navMockService.updateBruker(mockBruker, enableKvp); gitt_at_jeg_har_folgende_aktiviteter(List.of(nyttStillingssok())); var removeKvp = mockBruker.getBrukerOptions().toBuilder().erUnderKvp(false).build(); - MockNavService.updateBruker(mockBruker, removeKvp); + navMockService.updateBruker(mockBruker, removeKvp); } private void gitt_at_jeg_har_folgende_aktiviteter(List aktiviteter) { diff --git a/src/test/java/no/nav/veilarbaktivitet/internapi/InternApiControllerTest.java b/src/test/java/no/nav/veilarbaktivitet/internapi/InternApiControllerTest.java index 71d073308..a72c4ba5a 100644 --- a/src/test/java/no/nav/veilarbaktivitet/internapi/InternApiControllerTest.java +++ b/src/test/java/no/nav/veilarbaktivitet/internapi/InternApiControllerTest.java @@ -23,7 +23,7 @@ class InternApiControllerTest extends SpringBootTestBase { @Test void hentAktiviteterTest() { - MockBruker mockBruker = navMockService.createHappyBruker(BrukerOptions.happyBruker()); + MockBruker mockBruker = navMockService.createBruker(BrukerOptions.happyBruker()); MockVeileder mockVeileder = MockNavService.createVeileder(mockBruker); AktivitetData aktivitetData = AktivitetDataTestBuilder.nyMoteAktivitet(); @@ -71,7 +71,7 @@ void hentAktiviteterTest() { @Test void skalFunkeForAlleAktivitettyper() { - MockBruker mockBruker = navMockService.createHappyBruker(BrukerOptions.happyBruker()); + MockBruker mockBruker = navMockService.createBruker(BrukerOptions.happyBruker()); MockVeileder mockVeileder = MockNavService.createVeileder(mockBruker); for (AktivitetTypeDTO type : AktivitetTypeDTO.values()) { diff --git a/src/test/java/no/nav/veilarbaktivitet/mock_nav_modell/BrukerOptions.java b/src/test/java/no/nav/veilarbaktivitet/mock_nav_modell/BrukerOptions.java index 730029bc0..177e6d507 100644 --- a/src/test/java/no/nav/veilarbaktivitet/mock_nav_modell/BrukerOptions.java +++ b/src/test/java/no/nav/veilarbaktivitet/mock_nav_modell/BrukerOptions.java @@ -2,10 +2,12 @@ import lombok.Builder; import lombok.Getter; +import lombok.With; import no.nav.veilarbaktivitet.person.Navn; @Builder(toBuilder = true) @Getter +@With public class BrukerOptions { private boolean underOppfolging; private boolean erManuell; diff --git a/src/test/java/no/nav/veilarbaktivitet/mock_nav_modell/MockNavService.java b/src/test/java/no/nav/veilarbaktivitet/mock_nav_modell/MockNavService.java index 5625b89bb..64fc9ca7e 100644 --- a/src/test/java/no/nav/veilarbaktivitet/mock_nav_modell/MockNavService.java +++ b/src/test/java/no/nav/veilarbaktivitet/mock_nav_modell/MockNavService.java @@ -54,10 +54,5 @@ public static MockVeileder createVeileder() { return new MockVeileder(navAnsatt); } - public static MockVeileder createVeilederMedNasjonalTilgang() { - NavAnsatt navAnsatt = NAV_CONTEXT.getNavAnsatt().nyNksAnsatt(); - - return new MockVeileder(navAnsatt); - } } diff --git a/src/test/java/no/nav/veilarbaktivitet/motesms/MoteSmsTest.java b/src/test/java/no/nav/veilarbaktivitet/motesms/MoteSmsTest.java index f06bd3f9f..508281af7 100644 --- a/src/test/java/no/nav/veilarbaktivitet/motesms/MoteSmsTest.java +++ b/src/test/java/no/nav/veilarbaktivitet/motesms/MoteSmsTest.java @@ -57,7 +57,7 @@ void setUp() { @Test void skalSendeServiceMelding() { - MockBruker happyBruker = navMockService.createHappyBruker(BrukerOptions.happyBruker()); + MockBruker happyBruker = navMockService.createBruker(BrukerOptions.happyBruker()); MockVeileder veileder = MockNavService.createVeileder(happyBruker); AktivitetDTO aktivitetDTO = AktivitetDtoTestBuilder.nyAktivitet(AktivitetTypeDTO.MOTE); ZonedDateTime startTid = ZonedDateTime.now().plusHours(2); @@ -93,7 +93,7 @@ void skalSendeServiceMelding() { @Test void skalSendeForAlleMoteTyper() { - MockBruker happyBruker = navMockService.createHappyBruker(BrukerOptions.happyBruker()); + MockBruker happyBruker = navMockService.createBruker(BrukerOptions.happyBruker()); MockVeileder veileder = MockNavService.createVeileder(happyBruker); AktivitetDTO aktivitetDTO = AktivitetDtoTestBuilder.nyAktivitet(AktivitetTypeDTO.MOTE); ZonedDateTime fraDato = ZonedDateTime.now().plusHours(4); @@ -111,7 +111,7 @@ void skalSendeForAlleMoteTyper() { @Test void bareSendeForMote() { - MockBruker happyBruker = navMockService.createHappyBruker(BrukerOptions.happyBruker()); + MockBruker happyBruker = navMockService.createBruker(BrukerOptions.happyBruker()); MockVeileder veileder = MockNavService.createVeileder(happyBruker); for (AktivitetTypeDTO type : AktivitetTypeDTO.values()) { @@ -133,7 +133,7 @@ void bareSendeForMote() { @Test void skalFjereneGamleMoter() { - MockBruker happyBruker = navMockService.createHappyBruker(BrukerOptions.happyBruker()); + MockBruker happyBruker = navMockService.createBruker(BrukerOptions.happyBruker()); MockVeileder veileder = MockNavService.createVeileder(happyBruker); AktivitetDTO aktivitet = AktivitetDtoTestBuilder.nyAktivitet(AktivitetTypeDTO.MOTE); ZonedDateTime startTid = ZonedDateTime.now().minusDays(10); @@ -152,7 +152,7 @@ void skalFjereneGamleMoter() { @Test void skalIkkeOppreteVarsleHistorisk() { - MockBruker happyBruker = navMockService.createHappyBruker(BrukerOptions.happyBruker()); + MockBruker happyBruker = navMockService.createBruker(BrukerOptions.happyBruker()); MockVeileder veileder = MockNavService.createVeileder(happyBruker); AktivitetDTO aktivitetDTO = AktivitetDtoTestBuilder.nyAktivitet(AktivitetTypeDTO.MOTE); ZonedDateTime startTid = ZonedDateTime.now().plusHours(2); @@ -171,7 +171,7 @@ void skalIkkeOppreteVarsleHistorisk() { @Test void skalIkkeOppreteVarsleFulfort() { - MockBruker happyBruker = navMockService.createHappyBruker(BrukerOptions.happyBruker()); + MockBruker happyBruker = navMockService.createBruker(BrukerOptions.happyBruker()); MockVeileder veileder = MockNavService.createVeileder(happyBruker); AktivitetDTO aktivitetDTO = AktivitetDtoTestBuilder.nyAktivitet(AktivitetTypeDTO.MOTE); ZonedDateTime startTid = ZonedDateTime.now().plusHours(2); @@ -188,7 +188,7 @@ void skalIkkeOppreteVarsleFulfort() { @Test void skalIkkeOppreteVarsleAvbrutt() { - MockBruker happyBruker = navMockService.createHappyBruker(BrukerOptions.happyBruker()); + MockBruker happyBruker = navMockService.createBruker(BrukerOptions.happyBruker()); MockVeileder veileder = MockNavService.createVeileder(happyBruker); AktivitetDTO aktivitetDTO = AktivitetDtoTestBuilder.nyAktivitet(AktivitetTypeDTO.MOTE); ZonedDateTime startTid = ZonedDateTime.now().plusHours(2); diff --git a/src/test/java/no/nav/veilarbaktivitet/stilling_fra_nav/DelingAvCvManueltAvbruttServiceTest.java b/src/test/java/no/nav/veilarbaktivitet/stilling_fra_nav/DelingAvCvManueltAvbruttServiceTest.java index c349860b0..ec870b0ad 100644 --- a/src/test/java/no/nav/veilarbaktivitet/stilling_fra_nav/DelingAvCvManueltAvbruttServiceTest.java +++ b/src/test/java/no/nav/veilarbaktivitet/stilling_fra_nav/DelingAvCvManueltAvbruttServiceTest.java @@ -57,7 +57,7 @@ void cleanupBetweenTests() { @Test void happy_case() { - MockBruker mockBruker = navMockService.createHappyBruker(BrukerOptions.happyBruker()); + MockBruker mockBruker = navMockService.createBruker(BrukerOptions.happyBruker()); AktivitetDTO skalBehandles = aktivitetTestService.opprettStillingFraNav(mockBruker); AktivitetDTO skalIkkeBehandles = aktivitetTestService.opprettStillingFraNav(mockBruker); diff --git a/src/test/java/no/nav/veilarbaktivitet/stilling_fra_nav/StillingFraNavControllerITest.java b/src/test/java/no/nav/veilarbaktivitet/stilling_fra_nav/StillingFraNavControllerITest.java index 74a2374c2..4dde0578f 100644 --- a/src/test/java/no/nav/veilarbaktivitet/stilling_fra_nav/StillingFraNavControllerITest.java +++ b/src/test/java/no/nav/veilarbaktivitet/stilling_fra_nav/StillingFraNavControllerITest.java @@ -160,7 +160,7 @@ void svar_naar_frist_utlopt_feiler() { @Test void historikk_del_cv_transaksjoner() { - MockBruker mockBruker = navMockService.createHappyBruker(BrukerOptions.happyBruker()); + MockBruker mockBruker = navMockService.createBruker(BrukerOptions.happyBruker()); MockVeileder veileder = MockNavService.createVeileder(mockBruker); AktivitetDTO aktivitetDTO = aktivitetTestService.opprettStillingFraNav(mockBruker); @@ -184,7 +184,7 @@ void oppdaterKanCvDeles_feilAktivitetstype_feiler() { private void oppdaterKanCvDeles_feilAktivitetstype_feiler(AktivitetTypeDTO typeDTO) { if (typeDTO.equals(AktivitetTypeDTO.STILLING_FRA_NAV) || typeDTO.equals(AktivitetTypeDTO.EKSTERNAKTIVITET)) return; - MockBruker mockBruker = navMockService.createHappyBruker(BrukerOptions.happyBruker()); + MockBruker mockBruker = navMockService.createBruker(BrukerOptions.happyBruker()); MockVeileder veileder = MockNavService.createVeileder(mockBruker); AktivitetDTO aktivitetDTO = AktivitetDtoTestBuilder.nyAktivitet(typeDTO); diff --git a/src/test/java/no/nav/veilarbaktivitet/veilarbportefolje/AktiviteterTilKafkaAivenServiceTest.java b/src/test/java/no/nav/veilarbaktivitet/veilarbportefolje/AktiviteterTilKafkaAivenServiceTest.java index a3222da1c..f662f16a1 100644 --- a/src/test/java/no/nav/veilarbaktivitet/veilarbportefolje/AktiviteterTilKafkaAivenServiceTest.java +++ b/src/test/java/no/nav/veilarbaktivitet/veilarbportefolje/AktiviteterTilKafkaAivenServiceTest.java @@ -84,7 +84,7 @@ public void cleanupBetweenTests() { @Test void skal_sende_meldinger_til_portefolje() { - MockBruker mockBruker = navMockService.createHappyBruker(BrukerOptions.happyBruker()); + MockBruker mockBruker = navMockService.createBruker(BrukerOptions.happyBruker()); AktivitetData aktivitetData = AktivitetDataTestBuilder.nyEgenaktivitet(); AktivitetDTO skalSendes = AktivitetDTOMapper.mapTilAktivitetDTO(aktivitetData, false); @@ -153,7 +153,7 @@ void skal_ikke_sende_tiltak_opprettet_som_historisk() { @Test void skal_committe_hver_melding() { - MockBruker mockBruker = navMockService.createHappyBruker(BrukerOptions.happyBruker()); + MockBruker mockBruker = navMockService.createBruker(BrukerOptions.happyBruker()); AktivitetData aktivitetData1 = AktivitetDataTestBuilder.nyEgenaktivitet(); AktivitetData aktivitetData2 = AktivitetDataTestBuilder.nyEgenaktivitet(); AktivitetDTO skalSendes1 = AktivitetDTOMapper.mapTilAktivitetDTO(aktivitetData1, false); diff --git a/src/test/kotlin/no/nav/veilarbaktivitet/admin/KasserControllerTest.kt b/src/test/kotlin/no/nav/veilarbaktivitet/admin/KasserControllerTest.kt index 623565ad5..1d255bdb3 100644 --- a/src/test/kotlin/no/nav/veilarbaktivitet/admin/KasserControllerTest.kt +++ b/src/test/kotlin/no/nav/veilarbaktivitet/admin/KasserControllerTest.kt @@ -2,7 +2,6 @@ package no.nav.veilarbaktivitet.admin import no.nav.veilarbaktivitet.SpringBootTestBase import no.nav.veilarbaktivitet.aktivitet.domain.AktivitetStatus -import no.nav.veilarbaktivitet.aktivitet.domain.AktivitetTransaksjonsType import no.nav.veilarbaktivitet.aktivitet.domain.AktivitetTransaksjonsType.KASSERT import no.nav.veilarbaktivitet.aktivitet.dto.AktivitetTypeDTO import no.nav.veilarbaktivitet.mock_nav_modell.BrukerOptions @@ -11,7 +10,6 @@ import no.nav.veilarbaktivitet.mock_nav_modell.MockVeileder import no.nav.veilarbaktivitet.person.Innsender import no.nav.veilarbaktivitet.testutils.AktivitetDtoTestBuilder import org.assertj.core.api.Assertions.assertThat -import org.assertj.core.api.Assertions.within import org.junit.jupiter.api.BeforeAll import org.junit.jupiter.api.Test import org.junit.jupiter.api.TestInstance @@ -26,7 +24,7 @@ internal class KasserControllerTest : SpringBootTestBase() { @BeforeAll fun beforeAll() { - mockBruker = navMockService.createHappyBruker(BrukerOptions.happyBruker()) + mockBruker = navMockService.createBruker(BrukerOptions.happyBruker()) veilederSomKanKassere = navMockService.createVeileder(ident = "Z999999", mockBruker = mockBruker) } diff --git a/src/test/kotlin/no/nav/veilarbaktivitet/aktivitet/AktivitetsplanControllerTest.kt b/src/test/kotlin/no/nav/veilarbaktivitet/aktivitet/AktivitetsplanControllerTest.kt index 4eb9628d8..7e988b7e6 100644 --- a/src/test/kotlin/no/nav/veilarbaktivitet/aktivitet/AktivitetsplanControllerTest.kt +++ b/src/test/kotlin/no/nav/veilarbaktivitet/aktivitet/AktivitetsplanControllerTest.kt @@ -3,7 +3,6 @@ package no.nav.veilarbaktivitet.aktivitet import no.nav.common.json.JsonUtils import no.nav.veilarbaktivitet.SpringBootTestBase import no.nav.veilarbaktivitet.aktivitet.dto.AktivitetTypeDTO -import no.nav.veilarbaktivitet.mock_nav_modell.MockNavService import no.nav.veilarbaktivitet.testutils.AktivitetDtoTestBuilder import org.junit.jupiter.api.Assertions import org.junit.jupiter.api.Test @@ -12,8 +11,8 @@ import org.springframework.http.HttpStatus internal class AktivitetsplanControllerTest : SpringBootTestBase() { @Test fun veileder_skal_kunne_oprette_aktivitet() { - val happyBruker = navMockService.createHappyBruker() - val veileder = MockNavService.createVeileder(happyBruker) + val happyBruker = navMockService.createBruker() + val veileder = navMockService.createVeileder(happyBruker) aktivitetTestService.opprettAktivitet( happyBruker, veileder, AktivitetDtoTestBuilder.nyAktivitet( AktivitetTypeDTO.EGEN @@ -23,14 +22,14 @@ internal class AktivitetsplanControllerTest : SpringBootTestBase() { @Test fun bruker_skal_kunne_oprette_aktivitet() { - val happyBruker = navMockService.createHappyBruker() + val happyBruker = navMockService.createBruker() aktivitetTestService.opprettAktivitet(happyBruker, AktivitetDtoTestBuilder.nyAktivitet(AktivitetTypeDTO.EGEN)) } @Test fun bruker_skal_ikke_kunne_oprette_aktivitet_på_annen_bruker() { - val happyBruker = navMockService.createHappyBruker() - val evilUser = MockNavService.createHappyBruker() + val happyBruker = navMockService.createBruker() + val evilUser = navMockService.createBruker() val aktivitetPayloadJson = JsonUtils.toJson(AktivitetDtoTestBuilder.nyAktivitet(AktivitetTypeDTO.EGEN)) val response = evilUser .createRequest() @@ -51,8 +50,8 @@ internal class AktivitetsplanControllerTest : SpringBootTestBase() { @Test fun veileder_uten_tilgang_skal_ikke_kunne_opprette_aktiviteter_på_bruker() { - val happyBruker = navMockService.createHappyBruker() - val veileder = MockNavService.createVeileder() + val happyBruker = navMockService.createBruker() + val veileder = navMockService.createVeileder() val aktivitetPayloadJson = JsonUtils.toJson(AktivitetDtoTestBuilder.nyAktivitet(AktivitetTypeDTO.EGEN)) veileder .createRequest(happyBruker) @@ -67,8 +66,8 @@ internal class AktivitetsplanControllerTest : SpringBootTestBase() { @Test fun veileder_uten_tilgang_skal_ikke_kunne_hente_aktiviteter_på_bruker() { - val happyBruker = navMockService.createHappyBruker() - val veileder = MockNavService.createVeileder() + val happyBruker = navMockService.createBruker() + val veileder = navMockService.createVeileder() veileder .createRequest(happyBruker) .get("http://localhost:$port/veilarbaktivitet/api/aktivitet") @@ -79,8 +78,8 @@ internal class AktivitetsplanControllerTest : SpringBootTestBase() { @Test fun veileder_uten_tilgang_skal_ikke_kunne_hente_en_aktivitet() { - val happyBruker = navMockService.createHappyBruker() - val veileder = MockNavService.createVeileder() + val happyBruker = navMockService.createBruker() + val veileder = navMockService.createVeileder() val aktivitet = aktivitetTestService.opprettAktivitet(happyBruker, AktivitetDtoTestBuilder.nyAktivitet(AktivitetTypeDTO.EGEN)) veileder .createRequest(happyBruker) @@ -92,8 +91,8 @@ internal class AktivitetsplanControllerTest : SpringBootTestBase() { @Test fun veileder_uten_tilgang_skal_ikke_kunne_hente_aktivitetsversjoner() { - val happyBruker = navMockService.createHappyBruker() - val veileder = MockNavService.createVeileder() + val happyBruker = navMockService.createBruker() + val veileder = navMockService.createVeileder() val aktivitet = aktivitetTestService.opprettAktivitet(happyBruker, AktivitetDtoTestBuilder.nyAktivitet(AktivitetTypeDTO.EGEN)) veileder .createRequest(happyBruker) @@ -105,8 +104,8 @@ internal class AktivitetsplanControllerTest : SpringBootTestBase() { @Test fun veileder_uten_tilgang_skal_ikke_kunne_oppdatere_aktiviteter() { - val happyBruker = navMockService.createHappyBruker() - val veileder = MockNavService.createVeileder() + val happyBruker = navMockService.createBruker() + val veileder = navMockService.createVeileder() val aktivitet = aktivitetTestService.opprettAktivitet(happyBruker, AktivitetDtoTestBuilder.nyAktivitet(AktivitetTypeDTO.EGEN)) val aktivitetPayloadJson = JsonUtils.toJson(aktivitet) veileder @@ -120,8 +119,8 @@ internal class AktivitetsplanControllerTest : SpringBootTestBase() { @Test fun veileder_uten_tilgang_skal_ikke_kunne_oppdatere_etiketter() { - val happyBruker = navMockService.createHappyBruker() - val veileder = MockNavService.createVeileder() + val happyBruker = navMockService.createBruker() + val veileder = navMockService.createVeileder() val aktivitet = aktivitetTestService.opprettAktivitet(happyBruker, AktivitetDtoTestBuilder.nyAktivitet(AktivitetTypeDTO.EGEN)) val aktivitetPayloadJson = JsonUtils.toJson(aktivitet) veileder @@ -135,8 +134,8 @@ internal class AktivitetsplanControllerTest : SpringBootTestBase() { @Test fun veileder_uten_tilgang_skal_ikke_kunne_oppdatere_status() { - val happyBruker = navMockService.createHappyBruker() - val veileder = MockNavService.createVeileder() + val happyBruker = navMockService.createBruker() + val veileder = navMockService.createVeileder() val aktivitet = aktivitetTestService.opprettAktivitet(happyBruker, AktivitetDtoTestBuilder.nyAktivitet(AktivitetTypeDTO.EGEN)) val aktivitetPayloadJson = JsonUtils.toJson(aktivitet) veileder @@ -150,8 +149,8 @@ internal class AktivitetsplanControllerTest : SpringBootTestBase() { @Test fun veileder_uten_tilgang_skal_ikke_kunne_oppdatere_referat() { - val happyBruker = navMockService.createHappyBruker() - val veileder = MockNavService.createVeileder() + val happyBruker = navMockService.createBruker() + val veileder = navMockService.createVeileder() val aktivitet = aktivitetTestService.opprettAktivitet(happyBruker, AktivitetDtoTestBuilder.nyAktivitet(AktivitetTypeDTO.EGEN)) val aktivitetPayloadJson = JsonUtils.toJson(aktivitet) veileder @@ -165,8 +164,8 @@ internal class AktivitetsplanControllerTest : SpringBootTestBase() { @Test fun bruker_skal_ikke_kunne_publisere_referat() { - val happyBruker = navMockService.createHappyBruker() - val veileder = MockNavService.createVeileder(happyBruker) + val happyBruker = navMockService.createBruker() + val veileder = navMockService.createVeileder(happyBruker) val aktivitet = aktivitetTestService.opprettAktivitet(happyBruker, veileder, AktivitetDtoTestBuilder.nyAktivitet(AktivitetTypeDTO.MOTE)) val aktivitetPayloadJson = JsonUtils.toJson(aktivitet) happyBruker diff --git a/src/test/kotlin/no/nav/veilarbaktivitet/aktivitetskort/AktivitetskortConsumerIntegrationTest.kt b/src/test/kotlin/no/nav/veilarbaktivitet/aktivitetskort/AktivitetskortConsumerIntegrationTest.kt index 68701bcd2..b39292210 100644 --- a/src/test/kotlin/no/nav/veilarbaktivitet/aktivitetskort/AktivitetskortConsumerIntegrationTest.kt +++ b/src/test/kotlin/no/nav/veilarbaktivitet/aktivitetskort/AktivitetskortConsumerIntegrationTest.kt @@ -1055,7 +1055,7 @@ open class AktivitetskortConsumerIntegrationTest( } private val brukerUtenOppfolging = MockNavService.createBruker(BrukerOptions.happyBruker().toBuilder().underOppfolging(false).build()) - private val mockBruker by lazy { navMockService.createHappyBruker() } + private val mockBruker by lazy { navMockService.createBruker() } private val veileder by lazy { MockNavService.createVeileder(mockBruker) } private val endretDato = ZonedDateTime.now() } diff --git a/src/test/kotlin/no/nav/veilarbaktivitet/aktivitetskort/graphql/AktivitetskortControllerTest.kt b/src/test/kotlin/no/nav/veilarbaktivitet/aktivitetskort/graphql/AktivitetskortControllerTest.kt index 1ba20c32c..bba920c83 100644 --- a/src/test/kotlin/no/nav/veilarbaktivitet/aktivitetskort/graphql/AktivitetskortControllerTest.kt +++ b/src/test/kotlin/no/nav/veilarbaktivitet/aktivitetskort/graphql/AktivitetskortControllerTest.kt @@ -5,7 +5,6 @@ import no.nav.veilarbaktivitet.SpringBootTestBase import no.nav.veilarbaktivitet.aktivitet.domain.AktivitetStatus import no.nav.veilarbaktivitet.aktivitet.dto.AktivitetTypeDTO import no.nav.veilarbaktivitet.mock_nav_modell.BrukerOptions -import no.nav.veilarbaktivitet.mock_nav_modell.MockNavService import no.nav.veilarbaktivitet.mock_nav_modell.MockVeileder import no.nav.veilarbaktivitet.testutils.AktivitetDtoTestBuilder import no.nav.veilarbaktivitet.util.DateUtils @@ -15,8 +14,8 @@ import java.util.* class AktivitetskortControllerTest: SpringBootTestBase() { - private val mockBruker by lazy { navMockService.createHappyBruker() } - private val mockVeileder: MockVeileder by lazy { MockNavService.createVeileder(mockBruker) } + private val mockBruker by lazy { navMockService.createBruker() } + private val mockVeileder: MockVeileder by lazy { navMockService.createVeileder(mockBruker) } @Test fun `skal gruppere pa oppfolgingsperiode (bruker)`() { @@ -70,7 +69,7 @@ class AktivitetskortControllerTest: SpringBootTestBase() { @Test fun `veileder skal ikke ha tilgang til aktiviteter hvis ikke tilgang på bruker`() { - val bruker = MockNavService.createHappyBruker() + val bruker = navMockService.createHappyBruker() // Escaping $ does not work in multiline strings so use variable instead val fnrParam = "\$fnr" val query = """ @@ -90,12 +89,12 @@ class AktivitetskortControllerTest: SpringBootTestBase() { @Test fun `veileder skal ikke ha tilgang på kvp aktiviteter hvis ikke tilgang på enhet`() { - val kvpBruker = MockNavService.createBruker( + val kvpBruker = navMockService.createBruker( BrukerOptions.happyBruker().toBuilder() .erUnderKvp(true) .build() ) - val veileder = MockNavService.createVeilederMedNasjonalTilgang() + val veileder = navMockService.createVeilederMedNasjonalTilgang() // Escaping $ does not work in multiline strings so use variable instead val fnrParam = "\$fnr" val query = """ diff --git a/src/test/kotlin/no/nav/veilarbaktivitet/arkivering/ArkiveringsControllerTest.kt b/src/test/kotlin/no/nav/veilarbaktivitet/arkivering/ArkiveringsControllerTest.kt index 9b3c70ba8..c2a630e4b 100644 --- a/src/test/kotlin/no/nav/veilarbaktivitet/arkivering/ArkiveringsControllerTest.kt +++ b/src/test/kotlin/no/nav/veilarbaktivitet/arkivering/ArkiveringsControllerTest.kt @@ -29,7 +29,6 @@ import org.assertj.core.api.Assertions.within import org.junit.jupiter.api.Test import org.springframework.http.HttpStatus import java.net.URI -import java.net.URL import java.time.Instant import java.time.ZoneId import java.time.ZonedDateTime @@ -800,7 +799,7 @@ internal class ArkiveringsControllerTest : SpringBootTestBase() { @Test fun `Kall mot arkiveringsendepunkt kaster 403 når oppfølgingsperiodeId mangler`() { - val bruker = navMockService.createHappyBruker() + val bruker = navMockService.createBruker() val veileder = navMockService.createVeileder(mockBruker = bruker) val arkiveringsUrl = "http://localhost:$port/veilarbaktivitet/api/arkivering/forhaandsvisning" @@ -975,7 +974,7 @@ internal class ArkiveringsControllerTest : SpringBootTestBase() { private fun hentBrukerOgVeileder(brukerFornavn: String, brukerEtternavn: String): Pair { val navn = Navn(brukerFornavn, null, brukerEtternavn) val brukerOptions = BrukerOptions.happyBruker().toBuilder().navn(navn).build() - val bruker = navMockService.createHappyBruker(brukerOptions) + val bruker = navMockService.createBruker(brukerOptions) val veileder = navMockService.createVeileder(mockBruker = bruker) return Pair(bruker, veileder) } @@ -986,7 +985,7 @@ internal class ArkiveringsControllerTest : SpringBootTestBase() { ): Pair { val navn = Navn(brukerFornavn, null, brukerEtternavn) val brukerOptions = BrukerOptions.happyBruker().toBuilder().erUnderKvp(true).navn(navn).build() - val bruker = navMockService.createHappyBruker(brukerOptions) + val bruker = navMockService.createBruker(brukerOptions) val veileder = navMockService.createVeileder(mockBruker = bruker) return Pair(bruker, veileder) } diff --git a/src/test/kotlin/no/nav/veilarbaktivitet/foresatteinnsynsrett/InnsynrettControllerTest.kt b/src/test/kotlin/no/nav/veilarbaktivitet/foresatteinnsynsrett/InnsynrettControllerTest.kt index 48f09fab8..050f92da0 100644 --- a/src/test/kotlin/no/nav/veilarbaktivitet/foresatteinnsynsrett/InnsynrettControllerTest.kt +++ b/src/test/kotlin/no/nav/veilarbaktivitet/foresatteinnsynsrett/InnsynrettControllerTest.kt @@ -15,7 +15,7 @@ internal class InnsynrettControllerTest : SpringBootTestBase() { @Test fun `for bruker som er over 18 har ikke foresatte innsynsrett`() { val fødselsdato = LocalDate.now().minusYears(18) - val bruker = navMockService.createHappyBruker(BrukerOptions.happyBruker(), Fnr.of("${fødselsdato.tilFødselsDato()}60000")) + val bruker = navMockService.createBruker(BrukerOptions.happyBruker(), Fnr.of("${fødselsdato.tilFødselsDato()}60000")) val response = bruker .createRequest() @@ -34,7 +34,7 @@ internal class InnsynrettControllerTest : SpringBootTestBase() { @Test fun `for bruker som er under 18 har foresatte innsynsrett`() { val fødselsdato = LocalDate.now().minusYears(18).plusDays(1) - val bruker = navMockService.createHappyBruker(BrukerOptions.happyBruker(), Fnr.of("${fødselsdato.tilFødselsDato()}60000")) + val bruker = navMockService.createBruker(BrukerOptions.happyBruker(), Fnr.of("${fødselsdato.tilFødselsDato()}60000")) val response = bruker .createRequest() @@ -52,7 +52,7 @@ internal class InnsynrettControllerTest : SpringBootTestBase() { @Test fun `for bruker som er født på 1900 tallet har foresatte aldri innsynsrett`() { - val bruker = navMockService.createHappyBruker(BrukerOptions.happyBruker(), Fnr.of("01017120000")) + val bruker = navMockService.createBruker(BrukerOptions.happyBruker(), Fnr.of("01017120000")) val response = bruker @@ -72,7 +72,7 @@ internal class InnsynrettControllerTest : SpringBootTestBase() { @Test fun `skal støtte syntetiske føldselsnumre fra TestNorge`() { val fødselsdato = LocalDate.now().minusYears(18).plusDays(1) - val bruker = navMockService.createHappyBruker(BrukerOptions.happyBruker(), Fnr.of("${fødselsdato.tilSyntetiskFødselsdato(FødselsnummerType.TEST_NORGE)}50000")) + val bruker = navMockService.createBruker(BrukerOptions.happyBruker(), Fnr.of("${fødselsdato.tilSyntetiskFødselsdato(FødselsnummerType.TEST_NORGE)}50000")) val response = bruker @@ -92,7 +92,7 @@ internal class InnsynrettControllerTest : SpringBootTestBase() { @Test fun `skal støtte syntetiske føldselsnumre fra Dolly`() { val fødselsdato = LocalDate.now().minusYears(18).plusDays(1) - val bruker = navMockService.createHappyBruker(BrukerOptions.happyBruker(), Fnr.of("${fødselsdato.tilSyntetiskFødselsdato(FødselsnummerType.DOLLY)}40000")) + val bruker = navMockService.createBruker(BrukerOptions.happyBruker(), Fnr.of("${fødselsdato.tilSyntetiskFødselsdato(FødselsnummerType.DOLLY)}40000")) val response = bruker @@ -111,7 +111,7 @@ internal class InnsynrettControllerTest : SpringBootTestBase() { @Test fun `skal kunne se at en person er født på 1900-tallet selv om personnummeret starter på 9`() { - val bruker = navMockService.createHappyBruker(BrukerOptions.happyBruker(), Fnr.of("16917197656")) + val bruker = navMockService.createBruker(BrukerOptions.happyBruker(), Fnr.of("16917197656")) val response = bruker @@ -130,7 +130,7 @@ internal class InnsynrettControllerTest : SpringBootTestBase() { @Test fun `Veileder skal også kunne sjekke om foresatte har innsynsrett`() { - val bruker = navMockService.createHappyBruker() + val bruker = navMockService.createBruker() val veileder = navMockService.createVeileder(mockBruker = bruker) veileder diff --git a/src/test/kotlin/no/nav/veilarbaktivitet/mock_nav_modell/NavMockService.kt b/src/test/kotlin/no/nav/veilarbaktivitet/mock_nav_modell/NavMockService.kt index 3b941b25b..407ad7902 100644 --- a/src/test/kotlin/no/nav/veilarbaktivitet/mock_nav_modell/NavMockService.kt +++ b/src/test/kotlin/no/nav/veilarbaktivitet/mock_nav_modell/NavMockService.kt @@ -13,7 +13,7 @@ class NavMockService( val oppfolgingsperiodeService: OppfolgingsperiodeService, ) { - fun createHappyBruker(brukerOptions: BrukerOptions = BrukerOptions.happyBruker(), fnr: Fnr? = null): MockBruker { + fun createBruker(brukerOptions: BrukerOptions = BrukerOptions.happyBruker(), fnr: Fnr? = null): MockBruker { val bruker = MockNavService.createBruker(brukerOptions, fnr) val oppfolgingsperiode = bruker.oppfolgingsperioder.first() oppfolgingsperiodeService.upsertOppfolgingsperiode( @@ -26,8 +26,12 @@ class NavMockService( return bruker } - fun createHappyBruker(brukerOptions: BrukerOptions = BrukerOptions.happyBruker()): MockBruker { - return createHappyBruker(brukerOptions, null) + fun createBruker(brukerOptions: BrukerOptions = BrukerOptions.happyBruker()): MockBruker { + return createBruker(brukerOptions, null) + } + + fun createHappyBruker(): MockBruker { + return createBruker(BrukerOptions.happyBruker()) } fun updateBruker(mockBruker: MockBruker, brukerOptions: BrukerOptions) { @@ -35,19 +39,42 @@ class NavMockService( WireMockUtil.stubBruker(mockBruker) } - fun createVeileder(ident: String? = null, mockBruker: MockBruker): MockVeileder { - val navAnsatt = if(ident != null) { - MockNavService.NAV_CONTEXT.navAnsatt.get(ident)?.let { MockVeileder(it) } - NavAnsatt(ident) - } else { - NavAnsatt() - } + fun createVeileder(ident: String, mockBruker: MockBruker): MockVeileder { + val navAnsatt = MockNavService.NAV_CONTEXT.navAnsatt.get(ident)?: NavAnsatt(ident) MockNavService.NAV_CONTEXT.navAnsatt.add(navAnsatt) navAnsatt.adGrupper.add(tilgjengligeAdGrupper.modiaOppfolging) - val veileder = MockVeileder(navAnsatt) veileder.addBruker(mockBruker) + return veileder + } + fun createVeileder(mockBruker: MockBruker): MockVeileder { + val navAnsatt = NavAnsatt() + MockNavService.NAV_CONTEXT.navAnsatt.add(navAnsatt) + navAnsatt.adGrupper.add(tilgjengligeAdGrupper.modiaOppfolging) + + val veileder = MockVeileder(navAnsatt) + veileder.addBruker(mockBruker) return veileder } + + fun createVeileder() :MockVeileder { + val navAnsatt = NavAnsatt() + MockNavService.NAV_CONTEXT.navAnsatt.add(navAnsatt) + navAnsatt.adGrupper.add(tilgjengligeAdGrupper.modiaOppfolging) + return MockVeileder(navAnsatt) + } + + fun createVeilederMedNasjonalTilgang(): MockVeileder { + val navAnsatt = MockNavService.NAV_CONTEXT.navAnsatt.nyNksAnsatt() + + return MockVeileder(navAnsatt) + } + + fun getBrukerSomIkkeKanVarsles(): MockBruker { + val brukerOptions = BrukerOptions.happyBruker() + .withErManuell(true) + + return this.createBruker(brukerOptions) + } }