Skip to content

Commit

Permalink
Slutte å bruke deprekert MockNavService (#833)
Browse files Browse the repository at this point in the history
* Slutte å bruke deprekert MockNavService

* Use with instead of builder in kotlin classes

* Fiks tester

---------

Co-authored-by: sigurdgroneng <[email protected]>
Co-authored-by: johannetronstad <[email protected]>
  • Loading branch information
3 people authored Oct 21, 2024
1 parent dd3f1a6 commit b1f7b89
Show file tree
Hide file tree
Showing 19 changed files with 125 additions and 112 deletions.
4 changes: 2 additions & 2 deletions build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -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 {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand All @@ -19,6 +20,8 @@ class AvtaltMedNavTest extends SpringBootTestBase {
@Autowired
BrukernotifikasjonAssertsConfig config;
BrukernotifikasjonAsserts brukernotifikasjonAsserts;
@Autowired
NavMockService navMockService;

@BeforeEach
void before() {
Expand All @@ -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
Expand Down Expand Up @@ -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
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand All @@ -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();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down Expand Up @@ -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);
Expand All @@ -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);
Expand Down Expand Up @@ -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);
Expand Down Expand Up @@ -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);
Expand All @@ -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);
Expand All @@ -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);

Expand Down Expand Up @@ -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);
Expand Down Expand Up @@ -343,8 +342,8 @@ private void oppgaveSendtOk(ConsumerRecord<NokkelInput, OppgaveInput> 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);

Expand All @@ -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(
Expand Down Expand Up @@ -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());
Expand All @@ -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);
Expand All @@ -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());
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down Expand Up @@ -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;
}

Expand Down Expand Up @@ -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()
Expand All @@ -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<AktivitetData> aktiviteter) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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();
Expand Down Expand Up @@ -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()) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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);
}

}
14 changes: 7 additions & 7 deletions src/test/java/no/nav/veilarbaktivitet/motesms/MoteSmsTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -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);
Expand Down Expand Up @@ -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);
Expand All @@ -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()) {
Expand All @@ -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);
Expand All @@ -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);
Expand All @@ -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);
Expand All @@ -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);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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);

Expand Down
Loading

0 comments on commit b1f7b89

Please sign in to comment.