Endret utility class #deploy-test-dolly-backend

navikt · Aug 3, 2023 · ef5838d · ef5838d
1 parent 6dfcace
commit ef5838d
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 16 deletions.
diff --git a/...ly-backend/src/main/java/no/nav/dolly/bestilling/arenaforvalter/ArenaForvalterClient.java b/...ly-backend/src/main/java/no/nav/dolly/bestilling/arenaforvalter/ArenaForvalterClient.java
@@ -8,7 +8,7 @@
 import no.nav.dolly.bestilling.arenaforvalter.service.ArenaAapService;
 import no.nav.dolly.bestilling.arenaforvalter.service.ArenaBrukerService;
 import no.nav.dolly.bestilling.arenaforvalter.service.ArenaDagpengerService;
-import no.nav.dolly.bestilling.arenaforvalter.utils.ArenaEksisterendeVedtakUtil;
+import no.nav.dolly.bestilling.arenaforvalter.utils.ArenaEksisterendeVedtakService;
 import no.nav.dolly.domain.jpa.BestillingProgress;
 import no.nav.dolly.domain.resultset.RsDollyUtvidetBestilling;
 import no.nav.dolly.domain.resultset.arenaforvalter.Arenadata;
@@ -45,6 +45,7 @@ public class ArenaForvalterClient implements ClientRegister {
  private final ArenaAap115Service arenaAap115Service;
  private final ArenaAapService arenaAapService;
  private final ArenaDagpengerService arenaDagpengerService;
+ private final ArenaEksisterendeVedtakService arenaEksisterendeVedtakService;
 
  @Override
  public Flux<ClientFuture> gjenopprett(RsDollyUtvidetBestilling bestilling, DollyPerson dollyPerson, BestillingProgress progress, boolean isOpprettEndre) {
@@ -69,7 +70,7 @@ private Mono<String> doArenaOpprett(Arenadata arenadata, String ident, List<Stri
 
  return Flux.fromIterable(miljoer)
  .flatMap(miljoe -> arenaForvalterConsumer.getArenaBruker(ident, miljoe)
- .map(arenaArbeidsokerStatus -> ArenaEksisterendeVedtakUtil.getArenaOperasjoner(arenadata, arenaArbeidsokerStatus))
+ .map(arenaArbeidsokerStatus -> arenaEksisterendeVedtakService.getArenaOperasjoner(arenadata, arenaArbeidsokerStatus))
  .flatMapMany(arenaOperasjoner -> Flux.concat(
 
  arenaBrukerService.sendBruker(arenadata, arenaOperasjoner, ident, miljoe)

diff --git a/...er/utils/ArenaEksisterendeVedtakUtil.java → ...utils/ArenaEksisterendeVedtakService.java b/...er/utils/ArenaEksisterendeVedtakUtil.java → ...utils/ArenaEksisterendeVedtakService.java
@@ -1,13 +1,14 @@
 package no.nav.dolly.bestilling.arenaforvalter.utils;
 
-import lombok.experimental.UtilityClass;
+import lombok.RequiredArgsConstructor;
 import no.nav.dolly.bestilling.arenaforvalter.dto.ArenaStatusResponse;
 import no.nav.dolly.bestilling.arenaforvalter.dto.ArenaVedtakOperasjoner;
 import no.nav.dolly.domain.resultset.arenaforvalter.ArenaPeriode;
 import no.nav.dolly.domain.resultset.arenaforvalter.Arenadata;
 import no.nav.dolly.domain.resultset.arenaforvalter.RsArenaAap;
 import no.nav.dolly.domain.resultset.arenaforvalter.RsArenaDagpenger;
 import no.nav.dolly.util.NullcheckUtil;
+import org.springframework.stereotype.Service;
 
 import java.time.LocalDate;
 import java.time.LocalDateTime;
@@ -20,12 +21,13 @@
 import static java.util.Objects.isNull;
 import static java.util.Objects.nonNull;
 
-@UtilityClass
-public class ArenaEksisterendeVedtakUtil {
+@Service
+@RequiredArgsConstructor
+public class ArenaEksisterendeVedtakService {
 
- enum Ytelse {AAP, DAGO}
+ private enum Ytelse {AAP, DAGO}
 
- public static ArenaVedtakOperasjoner getArenaOperasjoner(Arenadata arenadata, ArenaStatusResponse response) {
+ public ArenaVedtakOperasjoner getArenaOperasjoner(Arenadata arenadata, ArenaStatusResponse response) {
 
  return ArenaVedtakOperasjoner.builder()
  .registrertDato(response.getRegistrertDato())
@@ -147,7 +149,7 @@ private static List<ArenaStatusResponse.Vedtak> getVedtaker(List<ArenaStatusResp
  .toList();
  }
 
- private LocalDate toLocalDate(LocalDateTime tid) {
+ private static LocalDate toLocalDate(LocalDateTime tid) {
 
  return nonNull(tid) ? tid.toLocalDate() : null;
  }

diff --git a/...tils/ArenaEksisterendeVedtakUtilTest.java → ...s/ArenaEksisterendeVedtakServiceTest.java b/...tils/ArenaEksisterendeVedtakUtilTest.java → ...s/ArenaEksisterendeVedtakServiceTest.java
@@ -6,7 +6,8 @@
 import no.nav.dolly.domain.resultset.arenaforvalter.RsArenaDagpenger;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.ExtendWith;
-import org.springframework.test.context.junit.jupiter.SpringExtension;
+import org.mockito.InjectMocks;
+import org.mockito.junit.jupiter.MockitoExtension;
 
 import java.time.LocalDate;
 import java.time.LocalDateTime;
@@ -17,13 +18,16 @@
 import static org.hamcrest.Matchers.is;
 import static org.hamcrest.Matchers.nullValue;
 
-@ExtendWith(SpringExtension.class)
-class ArenaEksisterendeVedtakUtilTest {
+@ExtendWith(MockitoExtension.class)
+class ArenaEksisterendeVedtakServiceTest {
+
+ @InjectMocks
+ private ArenaEksisterendeVedtakService arenaEksisterendeVedtakService;
 
  @Test
  void vedtakAapEksistererIkke() {
 
- var util = ArenaEksisterendeVedtakUtil.getArenaOperasjoner(Arenadata.builder()
+ var util = arenaEksisterendeVedtakService.getArenaOperasjoner(Arenadata.builder()
  .aap(List.of(RsArenaAap.builder()
  .fraDato(LocalDateTime.of(2023, 3, 1, 0, 0))
  .build()))
@@ -42,7 +46,7 @@ void vedtakAapEksistererIkke() {
  @Test
  void vedtakAapEksistereAllerede() {
 
- var util = ArenaEksisterendeVedtakUtil.getArenaOperasjoner(Arenadata.builder()
+ var util = arenaEksisterendeVedtakService.getArenaOperasjoner(Arenadata.builder()
  .aap(List.of(RsArenaAap.builder()
  .fraDato(LocalDateTime.of(2023, 1, 1, 0, 0))
  .build()))
@@ -60,7 +64,7 @@ void vedtakAapEksistereAllerede() {
  @Test
  void vedtakAapOverlapperEksisterende() {
 
- var util = ArenaEksisterendeVedtakUtil.getArenaOperasjoner(Arenadata.builder()
+ var util = arenaEksisterendeVedtakService.getArenaOperasjoner(Arenadata.builder()
  .aap(List.of(RsArenaAap.builder()
  .fraDato(LocalDateTime.of(2023, 3, 1, 0, 0))
  .build()))
@@ -80,7 +84,7 @@ void vedtakAapOverlapperEksisterende() {
  @Test
  void vedtakAapForanledningerEksisterende() {
 
- var util = ArenaEksisterendeVedtakUtil.getArenaOperasjoner(Arenadata.builder()
+ var util = arenaEksisterendeVedtakService.getArenaOperasjoner(Arenadata.builder()
  .aap(List.of(RsArenaAap.builder()
  .fraDato(LocalDateTime.of(2022, 10, 1, 0, 0))
  .build()))
@@ -99,7 +103,7 @@ void vedtakAapForanledningerEksisterende() {
  @Test
  void vedtakDagpengerEksisterAllleredeIngenEndring() {
 
- var util = ArenaEksisterendeVedtakUtil.getArenaOperasjoner(Arenadata.builder()
+ var util = arenaEksisterendeVedtakService.getArenaOperasjoner(Arenadata.builder()
  .dagpenger(List.of(RsArenaDagpenger.builder()
  .fraDato(LocalDateTime.of(2023, 1, 1, 0, 0))
  .tilDato(LocalDateTime.of(2023, 4, 30, 0, 0))