From f5615a63c4d8e7f295e83f413c13e2e61696522b Mon Sep 17 00:00:00 2001 From: Ragnhild Emblem Holte <193506135+remhol@users.noreply.github.com> Date: Tue, 4 Feb 2025 13:49:40 +0100 Subject: [PATCH] Sonar: Fikser feil fra sonar (#7044) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * Lagt inn kommentarer i tomme metoder * Tar i bruk etablerte konstanter * Oppretter konstanter for tekster som bruker minst 3 steder * Trekker ut i egne metoder der det er hensiktsmessig * Renaming av konstanter for å matche definert regex * Splittet finnPapirSøknad() i to metoder for å unngå å returnere null for Boolean * Disablet test hvor asserts er kommentert ut midlertidig * Slettet testfil som ikke hadde noen tester * Endret til static klasser for LagOppdragTjeneste og KontoerGrunnlagBygger * get(0)/iterator().next() -> getFirst() i berørte filer --- .../FamilieHendelseRepository.java | 2 +- .../fagsak/FagsakRelasjonRepository.java | 7 +- .../fagsak/FagsakRepository.java | 38 ++-- .../Oppdrag110EntityTest.java" | 4 +- .../FpinntektsmeldingKlient.java | 64 +++--- .../behandling/kabal/KabalKlient.java | 6 +- .../behandling/kabal/MottaFraKabalTask.java | 2 +- ...omatiskEtterkontrollBatchTjenesteTest.java | 32 +-- ...resultatVedAvslagP\303\245AvslagTest.java" | 25 --- .../BehandlingskontrollTjenesteImplTest.java | 1 + ...beidsforholdHistorikkKalkulusTjeneste.java | 10 +- .../tjeneste/BehandlingDvhMapper.java | 11 +- .../FeilPraksisUtsettelseRepository.java | 8 +- .../MedlemskapVurderingPeriodeTjeneste.java | 8 +- .../dokumentmottak/impl/FileToStringUtil.java | 3 - .../hendelser/DummyDatavarehustjeneste.java | 8 +- .../oppdrag/mapper/LagOppdragTjeneste.java" | 9 +- .../OppdragskontrollTjenesteImpl.java" | 7 +- .../queue/config/FellesJmsKonfig.java" | 5 +- .../SimulerOppdragTjenesteESTest.java" | 3 +- .../NyOppdragskontrollTjenesteENDRTest.java" | 1 + ...ontrollTjenesteImplKontantytelseTest.java" | 25 ++- .../NyOppdragskontrollTjenesteTestBase.java" | 8 +- .../StartpunktUtlederFamilieHendelse.java | 8 +- .../StartpunktUtlederPersonopplysning.java | 8 +- .../tjeneste/FpriskTjeneste.java | 32 +-- .../FastsettePerioderRegelGrunnlagBygger.java | 5 +- .../KontoerGrunnlagBygger.java | 12 +- .../St\303\270nadskontoSaldoTjeneste.java" | 6 +- .../FastsettePerioderRegelAdapterTest.java | 184 +++++++++--------- ...tsettePerioderRegelGrunnlagByggerTest.java | 24 ++- .../FastsettePerioderTjenesteTest.java | 36 ++-- .../saldo/MaksDatoUttakTjenesteImplTest.java | 4 +- ...St\303\270nadskontoSaldoTjenesteTest.java" | 3 +- .../dto/behandling/BehandlingDtoTjeneste.java | 24 +-- .../St\303\270nadskontoMigreringTask.java" | 2 +- ...t\303\270nadsstatistikkMigreringTask.java" | 2 +- ...apporterUnmappedKolonnerIDatabaseTest.java | 8 +- .../AksjonspunktRestTjenesteTest.java | 32 +-- .../uttak/app/SaldoerDtoTjenesteTest.java | 10 +- 40 files changed, 318 insertions(+), 369 deletions(-) delete mode 100644 "domenetjenester/behandling-revurdering/src/test/java/no/nav/foreldrepenger/behandling/revurdering/felles/FastsettBehandlingsresultatVedAvslagP\303\245AvslagTest.java" diff --git a/behandlingslager/domene/src/main/java/no/nav/foreldrepenger/behandlingslager/behandling/familiehendelse/FamilieHendelseRepository.java b/behandlingslager/domene/src/main/java/no/nav/foreldrepenger/behandlingslager/behandling/familiehendelse/FamilieHendelseRepository.java index 5fe1a56b154..3c1d46f0217 100644 --- a/behandlingslager/domene/src/main/java/no/nav/foreldrepenger/behandlingslager/behandling/familiehendelse/FamilieHendelseRepository.java +++ b/behandlingslager/domene/src/main/java/no/nav/foreldrepenger/behandlingslager/behandling/familiehendelse/FamilieHendelseRepository.java @@ -122,7 +122,7 @@ public void lagre(Long behandlingId, FamilieHendelseBuilder hendelseBuilder) { } public void lagreRegisterHendelse(Long behandlingId, FamilieHendelseBuilder hendelse) { - Objects.requireNonNull(behandlingId, "behandlingId"); + Objects.requireNonNull(behandlingId, BEHANDLING_ID); Objects.requireNonNull(hendelse, "hendelse"); var aggregatBuilder = opprettAggregatBuilderFor(behandlingId); diff --git a/behandlingslager/domene/src/main/java/no/nav/foreldrepenger/behandlingslager/fagsak/FagsakRelasjonRepository.java b/behandlingslager/domene/src/main/java/no/nav/foreldrepenger/behandlingslager/fagsak/FagsakRelasjonRepository.java index 0fd12546671..68b61e5e17b 100644 --- a/behandlingslager/domene/src/main/java/no/nav/foreldrepenger/behandlingslager/fagsak/FagsakRelasjonRepository.java +++ b/behandlingslager/domene/src/main/java/no/nav/foreldrepenger/behandlingslager/fagsak/FagsakRelasjonRepository.java @@ -26,6 +26,7 @@ public class FagsakRelasjonRepository { private static final String FAGSAK_QP = "fagsak"; + private static final String STØNADSKONTOBEREGNING = "stønadskontoberegning"; private EntityManager entityManager; private FagsakLåsRepository fagsakLåsRepository; @@ -98,7 +99,7 @@ public Optional finnRelasjonForHvisEksisterer(long fagsakId, Loc } public void lagre(Fagsak fagsak, Stønadskontoberegning stønadskontoberegning) { - Objects.requireNonNull(stønadskontoberegning, "stønadskontoberegning"); + Objects.requireNonNull(stønadskontoberegning, STØNADSKONTOBEREGNING); var fagsak1Lås = fagsakLåsRepository.taLås(fagsak.getId()); FagsakLås fagsak2Lås = null; @@ -300,7 +301,7 @@ public List finnFagsakerForAvsluttning(LocalDate dato) { } private void persisterStønadskontoberegning(Stønadskontoberegning stønadskontoberegning) { - Objects.requireNonNull(stønadskontoberegning, "stønadskontoberegning"); + Objects.requireNonNull(stønadskontoberegning, STØNADSKONTOBEREGNING); entityManager.persist(stønadskontoberegning); for (var stønadskonto : stønadskontoberegning.getStønadskontoer()) { @@ -309,7 +310,7 @@ public List finnFagsakerForAvsluttning(LocalDate dato) { } public void persisterFlushStønadskontoberegning(Stønadskontoberegning stønadskontoberegning) { - Objects.requireNonNull(stønadskontoberegning, "stønadskontoberegning"); + Objects.requireNonNull(stønadskontoberegning, STØNADSKONTOBEREGNING); persisterStønadskontoberegning(stønadskontoberegning); entityManager.flush(); } diff --git a/behandlingslager/domene/src/main/java/no/nav/foreldrepenger/behandlingslager/fagsak/FagsakRepository.java b/behandlingslager/domene/src/main/java/no/nav/foreldrepenger/behandlingslager/fagsak/FagsakRepository.java index a33e36ca9c4..f4e070bc9ce 100644 --- a/behandlingslager/domene/src/main/java/no/nav/foreldrepenger/behandlingslager/fagsak/FagsakRepository.java +++ b/behandlingslager/domene/src/main/java/no/nav/foreldrepenger/behandlingslager/fagsak/FagsakRepository.java @@ -12,6 +12,8 @@ import jakarta.persistence.EntityManager; import jakarta.persistence.LockModeType; +import jakarta.persistence.TypedQuery; + import org.hibernate.jpa.HibernateHints; import org.hibernate.query.NativeQuery; @@ -49,24 +51,21 @@ public FagsakRepository(EntityManager entityManager) { } public Fagsak finnEksaktFagsak(long fagsakId) { - var query = entityManager.createQuery("from Fagsak where id=:fagsakId", Fagsak.class); - query.setParameter("fagsakId", fagsakId); + var query = getFagsakQuery(fagsakId); var fagsak = HibernateVerktøy.hentEksaktResultat(query); entityManager.refresh(fagsak); // hent alltid på nytt return fagsak; } public Fagsak finnEksaktFagsak(Saksnummer saksnummer) { - var query = entityManager.createQuery("from Fagsak where saksnummer=:saksnummer", Fagsak.class) - .setParameter("saksnummer", saksnummer); + var query = getFagsakWhereSaksnummerQuery(saksnummer); var fagsak = HibernateVerktøy.hentEksaktResultat(query); entityManager.refresh(fagsak); // hent alltid på nytt return fagsak; } public Fagsak finnEksaktFagsakReadOnly(long fagsakId) { - var query = entityManager.createQuery("from Fagsak where id=:fagsakId", Fagsak.class) - .setParameter("fagsakId", fagsakId) + var query = getFagsakQuery(fagsakId) .setHint(HibernateHints.HINT_READ_ONLY, "true"); var fagsak = HibernateVerktøy.hentEksaktResultat(query); entityManager.refresh(fagsak); // hent alltid på nytt @@ -74,8 +73,7 @@ public Fagsak finnEksaktFagsakReadOnly(long fagsakId) { } public Fagsak finnEksaktFagsakReadOnly(Saksnummer saksnummer) { - var query = entityManager.createQuery("from Fagsak where saksnummer=:saksnummer", Fagsak.class) - .setParameter("saksnummer", saksnummer) + var query = getFagsakWhereSaksnummerQuery(saksnummer) .setHint(HibernateHints.HINT_READ_ONLY, "true"); var fagsak = HibernateVerktøy.hentEksaktResultat(query); entityManager.refresh(fagsak); // hent alltid på nytt @@ -83,8 +81,7 @@ public Fagsak finnEksaktFagsakReadOnly(Saksnummer saksnummer) { } public Optional finnUnikFagsak(long fagsakId) { - var query = entityManager.createQuery("from Fagsak where id=:fagsakId", Fagsak.class); - query.setParameter("fagsakId", fagsakId); + var query = getFagsakQuery(fagsakId); var opt = HibernateVerktøy.hentUniktResultat(query); if (opt.isPresent()) { entityManager.refresh(opt.get()); @@ -113,19 +110,18 @@ public Optional hentJournalpost(JournalpostId journalpostId) { Journalpost.class); query.setParameter("journalpost", journalpostId); var journalposter = query.getResultList(); - return journalposter.isEmpty() ? Optional.empty() : Optional.ofNullable(journalposter.get(0)); + return journalposter.isEmpty() ? Optional.empty() : Optional.ofNullable(journalposter.getFirst()); } public Optional hentSakGittSaksnummer(Saksnummer saksnummer) { - var query = entityManager.createQuery("from Fagsak where saksnummer=:saksnummer", Fagsak.class); - query.setParameter("saksnummer", saksnummer); + var query = getFagsakWhereSaksnummerQuery(saksnummer); var fagsaker = query.getResultList(); if (fagsaker.size() > 1) { throw flereEnnEnFagsakFeil(saksnummer); } - return fagsaker.isEmpty() ? Optional.empty() : Optional.of(fagsaker.get(0)); + return fagsaker.isEmpty() ? Optional.empty() : Optional.of(fagsaker.getFirst()); } private TekniskException flereEnnEnFagsakFeil(Saksnummer saksnummer) { @@ -174,8 +170,7 @@ public void oppdaterBruker(Long fagsakId, NavBruker bruker) { } public Optional hentSakGittSaksnummer(Saksnummer saksnummer, boolean taSkriveLås) { - var query = entityManager.createQuery("from Fagsak where saksnummer=:saksnummer", Fagsak.class); - query.setParameter("saksnummer", saksnummer); + var query = getFagsakWhereSaksnummerQuery(saksnummer); if (taSkriveLås) { query.setLockMode(LockModeType.PESSIMISTIC_WRITE); } @@ -185,7 +180,7 @@ public Optional hentSakGittSaksnummer(Saksnummer saksnummer, boolean taS throw flereEnnEnFagsakFeil(saksnummer); } - return fagsaker.isEmpty() ? Optional.empty() : Optional.of(fagsaker.get(0)); + return fagsaker.isEmpty() ? Optional.empty() : Optional.of(fagsaker.getFirst()); } public Long lagre(Journalpost journalpost) { @@ -292,4 +287,13 @@ public List hentFagsakNotater(Long fagsakId) { return query.getResultList(); } + private TypedQuery getFagsakQuery(long fagsakId) { + var query = entityManager.createQuery("from Fagsak where id=:fagsakId", Fagsak.class); + query.setParameter("fagsakId", fagsakId); + return query; + } + + private TypedQuery getFagsakWhereSaksnummerQuery(Saksnummer saksnummer) { + return entityManager.createQuery("from Fagsak where saksnummer=:saksnummer", Fagsak.class).setParameter( "saksnummer", saksnummer); + } } diff --git "a/behandlingslager/domene/src/test/java/no/nav/foreldrepenger/behandlingslager/\303\270konomioppdrag/Oppdrag110EntityTest.java" "b/behandlingslager/domene/src/test/java/no/nav/foreldrepenger/behandlingslager/\303\270konomioppdrag/Oppdrag110EntityTest.java" index 8fdf34d8c8e..907cdc38053 100644 --- "a/behandlingslager/domene/src/test/java/no/nav/foreldrepenger/behandlingslager/\303\270konomioppdrag/Oppdrag110EntityTest.java" +++ "b/behandlingslager/domene/src/test/java/no/nav/foreldrepenger/behandlingslager/\303\270konomioppdrag/Oppdrag110EntityTest.java" @@ -110,7 +110,7 @@ void skal_ha_refleksiv_equalsOgHashCode() { assertThat(oppdrag110).isEqualTo(oppdrag110_2); assertThat(oppdrag110_2).isEqualTo(oppdrag110); - oppdrag110_2 = oppdrag110Builder.medKodeEndring(KodeEndring.UEND).build(); + oppdrag110_2 = oppdrag110Builder.medKodeEndring(KodeEndring.NY).build(); assertThat(oppdrag110).isNotEqualTo(oppdrag110_2); assertThat(oppdrag110_2).isNotEqualTo(oppdrag110); } @@ -119,7 +119,7 @@ void skal_ha_refleksiv_equalsOgHashCode() { void skal_bruke_KodeEndring_i_equalsOgHashCode() { oppdrag110Builder = lagBuilderMedPaakrevdeFelter(); oppdrag110 = oppdrag110Builder.build(); - oppdrag110Builder.medKodeEndring(KodeEndring.UEND); + oppdrag110Builder.medKodeEndring(KodeEndring.NY); oppdrag110_2 = oppdrag110Builder.build(); assertThat(oppdrag110).isNotEqualTo(oppdrag110_2); diff --git a/domenetjenester/arbeidsforhold/src/main/java/no/nav/foreldrepenger/domene/fpinntektsmelding/FpinntektsmeldingKlient.java b/domenetjenester/arbeidsforhold/src/main/java/no/nav/foreldrepenger/domene/fpinntektsmelding/FpinntektsmeldingKlient.java index 09a85fd6abf..00ab0d27076 100644 --- a/domenetjenester/arbeidsforhold/src/main/java/no/nav/foreldrepenger/domene/fpinntektsmelding/FpinntektsmeldingKlient.java +++ b/domenetjenester/arbeidsforhold/src/main/java/no/nav/foreldrepenger/domene/fpinntektsmelding/FpinntektsmeldingKlient.java @@ -22,6 +22,8 @@ public class FpinntektsmeldingKlient { private static final Logger LOG = LoggerFactory.getLogger(FpinntektsmeldingKlient.class); + private static final String REQUEST = "request"; + private final RestClient restClient; private final RestConfig restConfig; private final URI uriOpprettForesporsel; @@ -41,60 +43,60 @@ public FpinntektsmeldingKlient() { this.uriSendNyBeskjedPåForespørsel = toUri(restConfig.fpContextPath(), "/api/foresporsel/ny-beskjed"); } - public OpprettForespørselResponsNy opprettForespørsel(OpprettForespørselRequest request) { - Objects.requireNonNull(request, "request"); + public OpprettForespørselResponsNy opprettForespørsel(OpprettForespørselRequest opprettForespørselRequest) { + Objects.requireNonNull(opprettForespørselRequest, REQUEST); try { - LOG.info("Sender request til fpinntektsmelding for saksnummer {} ", request.fagsakSaksnummer().saksnr()); - var rrequest = RestRequest.newPOSTJson(request, uriOpprettForesporsel, restConfig); - return restClient.send(rrequest, OpprettForespørselResponsNy.class); + LOG.info("Sender request til fpinntektsmelding for saksnummer {} ", opprettForespørselRequest.fagsakSaksnummer().saksnr()); + var request = RestRequest.newPOSTJson(opprettForespørselRequest, uriOpprettForesporsel, restConfig); + return restClient.send(request, OpprettForespørselResponsNy.class); } catch (Exception e) { - LOG.warn("Feil ved opprettelse av inntektsmelding med request: {}", request); + LOG.warn("Feil ved opprettelse av inntektsmelding med request: {}", opprettForespørselRequest); throw feilVedKallTilFpinntektsmelding(e.getMessage()); } } - public void overstyrInntektsmelding(OverstyrInntektsmeldingRequest request) { - Objects.requireNonNull(request, "request"); + public void overstyrInntektsmelding(OverstyrInntektsmeldingRequest overstyrInntektsmeldingRequest) { + Objects.requireNonNull(overstyrInntektsmeldingRequest, REQUEST); try { - LOG.info("Overstyrer inntektsmelding for arbeidsgiver {}", request.arbeidsgiverIdent().ident()); - var rrequest = RestRequest.newPOSTJson(request, uriOverstyrInntektsmelding, restConfig); - restClient.send(rrequest, String.class); + LOG.info("Overstyrer inntektsmelding for arbeidsgiver {}", overstyrInntektsmeldingRequest.arbeidsgiverIdent().ident()); + var request = RestRequest.newPOSTJson(overstyrInntektsmeldingRequest, uriOverstyrInntektsmelding, restConfig); + restClient.send(request, String.class); } catch (Exception e) { - LOG.warn("Feil ved overstyring av inntektsmelding med request: {}", request); + LOG.warn("Feil ved overstyring av inntektsmelding med request: {}", overstyrInntektsmeldingRequest); throw feilVedKallTilFpinntektsmelding(e.getMessage()); } } - public void lukkForespørsel(LukkForespørselRequest request) { - Objects.requireNonNull(request, "request"); + public void lukkForespørsel(LukkForespørselRequest lukkForespørselRequest) { + Objects.requireNonNull(lukkForespørselRequest, REQUEST); try { - LOG.info("Sender lukk forespørsel request til fpinntektsmelding for saksnummer {} med organisasjonsnummer {}", request.fagsakSaksnummer().saksnr(), request.orgnummer()); - var restRequest = RestRequest.newPOSTJson(request, uriLukkForesporsel, restConfig); - restClient.send(restRequest, String.class); + LOG.info("Sender lukk forespørsel request til fpinntektsmelding for saksnummer {} med organisasjonsnummer {}", lukkForespørselRequest.fagsakSaksnummer().saksnr(), lukkForespørselRequest.orgnummer()); + var request = RestRequest.newPOSTJson(lukkForespørselRequest, uriLukkForesporsel, restConfig); + restClient.send(request, String.class); } catch (Exception e) { - skrivTilLogg(request); + skrivTilLogg(lukkForespørselRequest); throw feilVedKallTilFpinntektsmelding(e.getMessage()); } } - public void settForespørselTilUtgått(LukkForespørselRequest request) { - Objects.requireNonNull(request, "request"); + public void settForespørselTilUtgått(LukkForespørselRequest lukkForespørselRequest) { + Objects.requireNonNull(lukkForespørselRequest, REQUEST); try { - LOG.info("Sender Lukk forespørsel request til fpinntektsmelding for å sette forespørsel til utgått for saksnummer {} ", request.fagsakSaksnummer().saksnr()); - var restRequest = RestRequest.newPOSTJson(request, uriSettForesporselTilUtgaatt, restConfig); - restClient.send(restRequest, String.class); + LOG.info("Sender Lukk forespørsel request til fpinntektsmelding for å sette forespørsel til utgått for saksnummer {} ", lukkForespørselRequest.fagsakSaksnummer().saksnr()); + var request = RestRequest.newPOSTJson(lukkForespørselRequest, uriSettForesporselTilUtgaatt, restConfig); + restClient.send(request, String.class); } catch (Exception e) { - skrivTilLogg(request); + skrivTilLogg(lukkForespørselRequest); throw feilVedKallTilFpinntektsmelding(e.getMessage()); } } - public SendNyBeskjedResponse sendNyBeskjedPåForespørsel(NyBeskjedRequest request) { - Objects.requireNonNull(request, "request"); + public SendNyBeskjedResponse sendNyBeskjedPåForespørsel(NyBeskjedRequest nyBeskjedRequest) { + Objects.requireNonNull(nyBeskjedRequest, REQUEST); try { - LOG.info("Sender ny beskjed request til fpinntektsmelding for å legge til ny beskjed på eksisterende forespørsel for saksnummer {} ", request.fagsakSaksnummer().saksnr()); - var restRequest = RestRequest.newPOSTJson(request, uriSendNyBeskjedPåForespørsel, restConfig); - return restClient.send(restRequest, SendNyBeskjedResponse.class); + LOG.info("Sender ny beskjed request til fpinntektsmelding for å legge til ny beskjed på eksisterende forespørsel for saksnummer {} ", nyBeskjedRequest.fagsakSaksnummer().saksnr()); + var request = RestRequest.newPOSTJson(nyBeskjedRequest, uriSendNyBeskjedPåForespørsel, restConfig); + return restClient.send(request, SendNyBeskjedResponse.class); } catch (Exception e) { throw feilVedKallTilFpinntektsmelding(e.getMessage()); } @@ -112,8 +114,8 @@ private URI toUri(URI endpointURI, String path) { } } - private static void skrivTilLogg(LukkForespørselRequest request) { - LOG.warn("Feil ved oversending til fpinntektsmelding med lukk forespørsel request: {}", request); + private static void skrivTilLogg(LukkForespørselRequest lukkForespørselRequest) { + LOG.warn("Feil ved oversending til fpinntektsmelding med lukk forespørsel request: {}", lukkForespørselRequest); } } diff --git a/domenetjenester/behandling-klage/src/main/java/no/nav/foreldrepenger/behandling/kabal/KabalKlient.java b/domenetjenester/behandling-klage/src/main/java/no/nav/foreldrepenger/behandling/kabal/KabalKlient.java index 36cbef83924..287c41e5492 100644 --- a/domenetjenester/behandling-klage/src/main/java/no/nav/foreldrepenger/behandling/kabal/KabalKlient.java +++ b/domenetjenester/behandling-klage/src/main/java/no/nav/foreldrepenger/behandling/kabal/KabalKlient.java @@ -29,11 +29,11 @@ public KabalKlient() { this.restConfig = RestConfig.forClient(KabalKlient.class); } - public void sendTilKabal(TilKabalDto request) { + public void sendTilKabal(TilKabalDto tilKabalDto) { if (IS_LOCAL) return; // Har ikke laget kabal-mock i VTP try { - var rrequest = RestRequest.newPOSTJson(request, restConfig.endpoint(), restConfig); - restClient.sendExpectConflict(rrequest, String.class); + var request = RestRequest.newPOSTJson(tilKabalDto, restConfig.endpoint(), restConfig); + restClient.sendExpectConflict(request, String.class); } catch (Exception e) { LOG.warn("KABAL oversend: feil ved sending til KABAL {}", restConfig.endpoint(), e); throw new TekniskException( "FP-180127", String.format("KABAL %s gir feil, ta opp med team klage.", restConfig.endpoint()), e); diff --git a/domenetjenester/behandling-klage/src/main/java/no/nav/foreldrepenger/behandling/kabal/MottaFraKabalTask.java b/domenetjenester/behandling-klage/src/main/java/no/nav/foreldrepenger/behandling/kabal/MottaFraKabalTask.java index f273c27dcee..dfd10541570 100644 --- a/domenetjenester/behandling-klage/src/main/java/no/nav/foreldrepenger/behandling/kabal/MottaFraKabalTask.java +++ b/domenetjenester/behandling-klage/src/main/java/no/nav/foreldrepenger/behandling/kabal/MottaFraKabalTask.java @@ -74,7 +74,6 @@ public MottaFraKabalTask(BehandlingRepositoryProvider repositoryProvider, @Override protected void prosesser(ProsessTaskData prosessTaskData, Long behandlingId) { - var hendelsetype = Optional.ofNullable(prosessTaskData.getPropertyValue(HENDELSETYPE_KEY)) .map(KabalHendelse.BehandlingEventType::valueOf) .orElseThrow(() -> new IllegalStateException("Utviklerfeil: Mottatt ikke-støtte kabalisme")); @@ -86,6 +85,7 @@ protected void prosesser(ProsessTaskData prosessTaskData, Long behandlingId) { case ANKE_I_TRYGDERETTENBEHANDLING_OPPRETTET -> ankeTrygdrett(prosessTaskData, behandlingId, ref); case ANKEBEHANDLING_AVSLUTTET, BEHANDLING_ETTER_TRYGDERETTEN_OPPHEVET_AVSLUTTET -> ankeAvsluttet(prosessTaskData, behandlingId, ref); case BEHANDLING_FEILREGISTRERT -> henleggFeilopprettet(prosessTaskData, behandlingId, ref); + case OMGJOERINGSKRAVBEHANDLING_AVSLUTTET -> throw new IllegalArgumentException("Utviklerfeil: hvorfor kom vi hit"); } } diff --git a/domenetjenester/behandling-revurdering/src/test/java/no/nav/foreldrepenger/behandling/revurdering/etterkontroll/batch/AutomatiskEtterkontrollBatchTjenesteTest.java b/domenetjenester/behandling-revurdering/src/test/java/no/nav/foreldrepenger/behandling/revurdering/etterkontroll/batch/AutomatiskEtterkontrollBatchTjenesteTest.java index 941ad27aeb8..f907cc965ea 100644 --- a/domenetjenester/behandling-revurdering/src/test/java/no/nav/foreldrepenger/behandling/revurdering/etterkontroll/batch/AutomatiskEtterkontrollBatchTjenesteTest.java +++ b/domenetjenester/behandling-revurdering/src/test/java/no/nav/foreldrepenger/behandling/revurdering/etterkontroll/batch/AutomatiskEtterkontrollBatchTjenesteTest.java @@ -45,7 +45,7 @@ @ExtendWith(JpaExtension.class) class AutomatiskEtterkontrollBatchTjenesteTest { - private final static int revurderingDagerTilbake = 60; + private static final int REVURDERING_DAGER_TILBAKE = 60; @Mock private ProsessTaskTjeneste taskTjeneste; @@ -66,10 +66,10 @@ void setUp(EntityManager entityManager) { @Test void skal_finne_kandidat_til_revurdering() { - var behandling = opprettRevurderingsKandidat(revurderingDagerTilbake + 2); + var behandling = opprettRevurderingsKandidat(REVURDERING_DAGER_TILBAKE + 2); var etterkontroll = new Etterkontroll.Builder(behandling.getFagsakId()).medErBehandlet(false) - .medKontrollTidspunkt(LocalDate.now().atStartOfDay().minusDays(revurderingDagerTilbake)) + .medKontrollTidspunkt(LocalDate.now().atStartOfDay().minusDays(REVURDERING_DAGER_TILBAKE)) .medKontrollType(KontrollType.MANGLENDE_FØDSEL) .build(); etterkontrollRepository.lagre(etterkontroll); @@ -81,12 +81,12 @@ void skal_finne_kandidat_til_revurdering() { @Test void sjekk_at_behandlingen_blir_etterkontrollert_og_ikke_klagen() { - var behandling = opprettRevurderingsKandidat(revurderingDagerTilbake + 2); + var behandling = opprettRevurderingsKandidat(REVURDERING_DAGER_TILBAKE + 2); var klage = Behandling.forKlage(behandling.getFagsak()).build(); behandlingRepository.lagre(klage, behandlingRepository.taSkriveLås(klage)); var etterkontroll = new Etterkontroll.Builder(klage.getFagsakId()).medErBehandlet(false) - .medKontrollTidspunkt(LocalDate.now().atStartOfDay().minusDays(revurderingDagerTilbake)) + .medKontrollTidspunkt(LocalDate.now().atStartOfDay().minusDays(REVURDERING_DAGER_TILBAKE)) .medKontrollType(KontrollType.MANGLENDE_FØDSEL) .build(); etterkontrollRepository.lagre(etterkontroll); @@ -99,10 +99,10 @@ void sjekk_at_behandlingen_blir_etterkontrollert_og_ikke_klagen() { @Test void behandling_som_har_vært_etterkontrollert_skal_ikke_være_kandidat_til_revurdering() { - var behandling = opprettRevurderingsKandidat(revurderingDagerTilbake + 2); + var behandling = opprettRevurderingsKandidat(REVURDERING_DAGER_TILBAKE + 2); var etterkontroll = new Etterkontroll.Builder(behandling.getFagsakId()).medErBehandlet(false) - .medKontrollTidspunkt(LocalDate.now().atStartOfDay().minusDays(revurderingDagerTilbake)) + .medKontrollTidspunkt(LocalDate.now().atStartOfDay().minusDays(REVURDERING_DAGER_TILBAKE)) .medKontrollType(KontrollType.MANGLENDE_FØDSEL) .build(); etterkontrollRepository.lagre(etterkontroll); @@ -115,7 +115,7 @@ void sjekk_at_behandlingen_blir_etterkontrollert_og_ikke_klagen() { @Test void skal_ikke_velge_henlagt_behandling() { - var behandling = opprettRevurderingsKandidat(revurderingDagerTilbake); + var behandling = opprettRevurderingsKandidat(REVURDERING_DAGER_TILBAKE); var innvilget = new Behandlingsresultat.Builder().medBehandlingResultatType( BehandlingResultatType.INNVILGET).buildFor(behandling); @@ -124,7 +124,7 @@ void skal_ikke_velge_henlagt_behandling() { behandlingRepository.lagre(behandling, behandlingRepository.taSkriveLås(behandling)); var etterkontroll = new Etterkontroll.Builder(behandling.getFagsakId()).medErBehandlet(false) - .medKontrollTidspunkt(LocalDate.now().atStartOfDay().minusDays(revurderingDagerTilbake)) + .medKontrollTidspunkt(LocalDate.now().atStartOfDay().minusDays(REVURDERING_DAGER_TILBAKE)) .medKontrollType(KontrollType.MANGLENDE_FØDSEL) .build(); etterkontrollRepository.lagre(etterkontroll); @@ -144,10 +144,10 @@ void skal_ikke_velge_henlagt_behandling() { @Test void fagsak_som_har_eksisterende_etterkontrollsbehandling_skal_ikke_være_kandidat_til_revurdering() { - var behandling = opprettRevurderingsKandidat(revurderingDagerTilbake + 2); + var behandling = opprettRevurderingsKandidat(REVURDERING_DAGER_TILBAKE + 2); var etterkontroll = new Etterkontroll.Builder(behandling.getFagsakId()).medErBehandlet(false) - .medKontrollTidspunkt(LocalDate.now().atStartOfDay().minusDays(revurderingDagerTilbake)) + .medKontrollTidspunkt(LocalDate.now().atStartOfDay().minusDays(REVURDERING_DAGER_TILBAKE)) .medKontrollType(KontrollType.MANGLENDE_FØDSEL) .build(); etterkontrollRepository.lagre(etterkontroll); @@ -190,8 +190,8 @@ static Optional finnTaskFor(ArgumentCaptor cap @Test void skal_hente_ut_siste_vedtak_til_revurdering(EntityManager entityManager) { var grunnlagRepository = new FamilieHendelseRepository(entityManager); - var behandling = opprettRevurderingsKandidat(revurderingDagerTilbake + 2); - var terminDato = LocalDate.now().minusDays(revurderingDagerTilbake + 2); + var behandling = opprettRevurderingsKandidat(REVURDERING_DAGER_TILBAKE + 2); + var terminDato = LocalDate.now().minusDays(REVURDERING_DAGER_TILBAKE + 2); var revurderingBuilder = Behandling.fraTidligereBehandling(behandling, BehandlingType.REVURDERING).medBehandlingÅrsak(BehandlingÅrsak.builder(BehandlingÅrsakType.RE_ANNET)); @@ -219,7 +219,7 @@ void skal_hente_ut_siste_vedtak_til_revurdering(EntityManager entityManager) { var etterkontroll = new Etterkontroll.Builder(revurderingsBehandling.getFagsakId()).medErBehandlet( false) - .medKontrollTidspunkt(LocalDate.now().atStartOfDay().minusDays(revurderingDagerTilbake)) + .medKontrollTidspunkt(LocalDate.now().atStartOfDay().minusDays(REVURDERING_DAGER_TILBAKE)) .medKontrollType(KontrollType.MANGLENDE_FØDSEL) .build(); etterkontrollRepository.lagre(etterkontroll); @@ -235,7 +235,7 @@ void skal_hente_ut_siste_vedtak_til_revurdering(EntityManager entityManager) { var behandling = opprettRevurderingsKandidat(0); var etterkontroll = new Etterkontroll.Builder(behandling.getFagsakId()).medErBehandlet(false) - .medKontrollTidspunkt(LocalDate.now().plusDays(revurderingDagerTilbake).atStartOfDay()) + .medKontrollTidspunkt(LocalDate.now().plusDays(REVURDERING_DAGER_TILBAKE).atStartOfDay()) .medKontrollType(KontrollType.MANGLENDE_FØDSEL) .build(); etterkontrollRepository.lagre(etterkontroll); @@ -279,7 +279,7 @@ private Behandling opprettRevurderingsKandidat(int dagerTilbake) { @Test void skal_finne_nyeste_innvilgete_avsluttede_behandling_som_ikke_er_henlagt() { - var behandling = opprettRevurderingsKandidat(revurderingDagerTilbake + 2); + var behandling = opprettRevurderingsKandidat(REVURDERING_DAGER_TILBAKE + 2); var innvilget = new Behandlingsresultat.Builder().medBehandlingResultatType( BehandlingResultatType.INNVILGET).buildFor(behandling); diff --git "a/domenetjenester/behandling-revurdering/src/test/java/no/nav/foreldrepenger/behandling/revurdering/felles/FastsettBehandlingsresultatVedAvslagP\303\245AvslagTest.java" "b/domenetjenester/behandling-revurdering/src/test/java/no/nav/foreldrepenger/behandling/revurdering/felles/FastsettBehandlingsresultatVedAvslagP\303\245AvslagTest.java" deleted file mode 100644 index 3da9e3fd03a..00000000000 --- "a/domenetjenester/behandling-revurdering/src/test/java/no/nav/foreldrepenger/behandling/revurdering/felles/FastsettBehandlingsresultatVedAvslagP\303\245AvslagTest.java" +++ /dev/null @@ -1,25 +0,0 @@ -package no.nav.foreldrepenger.behandling.revurdering.felles; - -import jakarta.persistence.EntityManager; - -import org.junit.jupiter.api.BeforeEach; -import org.junit.jupiter.api.extension.ExtendWith; - -import no.nav.foreldrepenger.behandlingslager.behandling.repository.BehandlingRepository; -import no.nav.foreldrepenger.behandlingslager.behandling.repository.BehandlingRepositoryProvider; -import no.nav.foreldrepenger.dbstoette.JpaExtension; - -@ExtendWith(JpaExtension.class) -class FastsettBehandlingsresultatVedAvslagPåAvslagTest { - - private BehandlingRepository behandlingRepository; - private BehandlingRepositoryProvider repositoryProvider; - - @BeforeEach - public void setUp(EntityManager entityManager) { - behandlingRepository = new BehandlingRepository(entityManager); - repositoryProvider = new BehandlingRepositoryProvider(entityManager); - } - - -} diff --git a/domenetjenester/behandlingskontroll/src/test/java/no/nav/foreldrepenger/behandlingskontroll/impl/BehandlingskontrollTjenesteImplTest.java b/domenetjenester/behandlingskontroll/src/test/java/no/nav/foreldrepenger/behandlingskontroll/impl/BehandlingskontrollTjenesteImplTest.java index db5e0eadd96..fbaabc9e6ce 100644 --- a/domenetjenester/behandlingskontroll/src/test/java/no/nav/foreldrepenger/behandlingskontroll/impl/BehandlingskontrollTjenesteImplTest.java +++ b/domenetjenester/behandlingskontroll/src/test/java/no/nav/foreldrepenger/behandlingskontroll/impl/BehandlingskontrollTjenesteImplTest.java @@ -340,6 +340,7 @@ private static final class BehandlingskontrollEventPublisererForTest extends Beh @Override protected void doFireEvent(BehandlingEvent event) { + // NOSONAR } } diff --git a/domenetjenester/beregningsgrunnlag/src/main/java/no/nav/foreldrepenger/domene/rest/historikk/kalkulus/FastsettBGTidsbegrensetArbeidsforholdHistorikkKalkulusTjeneste.java b/domenetjenester/beregningsgrunnlag/src/main/java/no/nav/foreldrepenger/domene/rest/historikk/kalkulus/FastsettBGTidsbegrensetArbeidsforholdHistorikkKalkulusTjeneste.java index d3511a5cb45..eb05b649c31 100644 --- a/domenetjenester/beregningsgrunnlag/src/main/java/no/nav/foreldrepenger/domene/rest/historikk/kalkulus/FastsettBGTidsbegrensetArbeidsforholdHistorikkKalkulusTjeneste.java +++ b/domenetjenester/beregningsgrunnlag/src/main/java/no/nav/foreldrepenger/domene/rest/historikk/kalkulus/FastsettBGTidsbegrensetArbeidsforholdHistorikkKalkulusTjeneste.java @@ -17,8 +17,8 @@ import no.nav.foreldrepenger.behandling.aksjonspunkt.AksjonspunktOppdaterParameter; import no.nav.foreldrepenger.behandlingslager.behandling.historikk.HistorikkAktør; import no.nav.foreldrepenger.behandlingslager.behandling.historikk.Historikkinnslag; -import no.nav.foreldrepenger.behandlingslager.behandling.historikk.HistorikkinnslagRepository; import no.nav.foreldrepenger.behandlingslager.behandling.historikk.HistorikkinnslagLinjeBuilder; +import no.nav.foreldrepenger.behandlingslager.behandling.historikk.HistorikkinnslagRepository; import no.nav.foreldrepenger.behandlingslager.behandling.skjermlenke.SkjermlenkeType; import no.nav.foreldrepenger.domene.aksjonspunkt.BeløpEndring; import no.nav.foreldrepenger.domene.aksjonspunkt.BeregningsgrunnlagEndring; @@ -35,6 +35,8 @@ @ApplicationScoped public class FastsettBGTidsbegrensetArbeidsforholdHistorikkKalkulusTjeneste { + private static final String FRILANSINNTEKT = "Frilansinntekt"; + private ArbeidsgiverHistorikkinnslag arbeidsgiverHistorikkinnslagTjeneste; private InntektArbeidYtelseTjeneste inntektArbeidYtelseTjeneste; private HistorikkinnslagRepository historikkinnslagRepository; @@ -152,13 +154,13 @@ public static List oppdaterFrilansInntektVedEndret if (forrigeFrilansInntekt != null && nyFrilansInntekt != null) { var fraInntekt = (int) Math.round(forrigeFrilansInntekt.doubleValue()); if (Objects.equals(fraInntekt, nyFrilansInntekt)) { - linjeBuilderList.add(linjeBuilder.til("Frilansinntekt", nyFrilansInntekt)); + linjeBuilderList.add(linjeBuilder.til(FRILANSINNTEKT, nyFrilansInntekt)); } else { - linjeBuilderList.add(linjeBuilder.fraTil("Frilansinntekt", fraInntekt, nyFrilansInntekt)); + linjeBuilderList.add(linjeBuilder.fraTil(FRILANSINNTEKT, fraInntekt, nyFrilansInntekt)); } linjeBuilderList.add(HistorikkinnslagLinjeBuilder.LINJESKIFT); } else if (nyFrilansInntekt != null) { - linjeBuilderList.add(linjeBuilder.til("Frilansinntekt", nyFrilansInntekt)); + linjeBuilderList.add(linjeBuilder.til(FRILANSINNTEKT, nyFrilansInntekt)); linjeBuilderList.add(HistorikkinnslagLinjeBuilder.LINJESKIFT); } return linjeBuilderList; diff --git a/domenetjenester/datavarehus/src/main/java/no/nav/foreldrepenger/datavarehus/tjeneste/BehandlingDvhMapper.java b/domenetjenester/datavarehus/src/main/java/no/nav/foreldrepenger/datavarehus/tjeneste/BehandlingDvhMapper.java index ab3dd2b5041..b67315ed4fd 100644 --- a/domenetjenester/datavarehus/src/main/java/no/nav/foreldrepenger/datavarehus/tjeneste/BehandlingDvhMapper.java +++ b/domenetjenester/datavarehus/src/main/java/no/nav/foreldrepenger/datavarehus/tjeneste/BehandlingDvhMapper.java @@ -72,7 +72,7 @@ public static BehandlingDvh map(Behandling behandling, // NOSONAR .relatertBehandlingFagsystem(getRelatertBehandlingFagsystem(behandling, klageResultat)) .familieHendelseType(mapFamilieHendelse(fh)) .medFoersteStoenadsdag(skjæringstidspunkt.orElse(null)) - .medPapirSøknad(finnPapirSøknad(behandling, mottatteDokument)) + .medPapirSøknad(erYtelsesbehandlingMedDokumenter(behandling, mottatteDokument) ? harPapirsøknad(mottatteDokument) : null) .medBehandlingMetode(utledBehandlingMetode(behandling, behandlingsresultat)) .medRevurderingÅrsak(utledRevurderingÅrsak(behandling, fagsakMarkering)) .medOmgjøringÅrsak(omgjøringÅrsak) @@ -156,13 +156,14 @@ private static String mapFamilieHendelse(Optional mottatteDokumenter) { - if (!behandling.erYtelseBehandling() || mottatteDokumenter.isEmpty()) { - return null; - } + private static boolean harPapirsøknad(List mottatteDokumenter) { return mottatteDokumenter.stream().anyMatch(md -> !md.getElektroniskRegistrert()); } + private static boolean erYtelsesbehandlingMedDokumenter(Behandling behandling, List mottatteDokumenter) { + return !behandling.erYtelseBehandling() || mottatteDokumenter.isEmpty(); + } + private static LocalDateTime finnMottattTidspunkt(List mottatteDokumenter) { return mottatteDokumenter.stream() .map(d -> d.getMottattTidspunkt().isBefore(d.getOpprettetTidspunkt()) ? d.getMottattTidspunkt() : d.getOpprettetTidspunkt()) diff --git a/domenetjenester/dokumentbestiller/src/main/java/no/nav/foreldrepenger/dokumentbestiller/infobrev/FeilPraksisUtsettelseRepository.java b/domenetjenester/dokumentbestiller/src/main/java/no/nav/foreldrepenger/dokumentbestiller/infobrev/FeilPraksisUtsettelseRepository.java index 6e9a826e1ad..b2213bb0aa7 100644 --- a/domenetjenester/dokumentbestiller/src/main/java/no/nav/foreldrepenger/dokumentbestiller/infobrev/FeilPraksisUtsettelseRepository.java +++ b/domenetjenester/dokumentbestiller/src/main/java/no/nav/foreldrepenger/dokumentbestiller/infobrev/FeilPraksisUtsettelseRepository.java @@ -18,6 +18,8 @@ @ApplicationScoped public class FeilPraksisUtsettelseRepository { + private static final String FRA_FAGSAK_ID = "fraFagsakId"; + private EntityManager entityManager; FeilPraksisUtsettelseRepository() { @@ -76,7 +78,7 @@ public List finnNesteHundreSakerForMerkingMor(Long fraFagsakId) { * levende barn - Begrenset til at det er oppgitt annen part */ var query = entityManager.createNativeQuery(QUERY_MERKING_MOR) - .setParameter("fraFagsakId", fraFagsakId == null ? 0 : fraFagsakId) + .setParameter(FRA_FAGSAK_ID, fraFagsakId == null ? 0 : fraFagsakId) .setHint(HibernateHints.HINT_READ_ONLY, "true"); @SuppressWarnings("unchecked") var resultat = query.getResultList(); @@ -136,7 +138,7 @@ and f.id not in (select fagsak_id from fagsak_egenskap where egenskap_value = 'P public List finnNesteHundreSakerForMerkingFarBeggeEllerAlene(Long fraFagsakId) { var query = entityManager.createNativeQuery(QUERY_MERKING_FAR_BEGGE_ALENE) - .setParameter("fraFagsakId", fraFagsakId == null ? 0 : fraFagsakId) + .setParameter(FRA_FAGSAK_ID, fraFagsakId == null ? 0 : fraFagsakId) .setHint(HibernateHints.HINT_READ_ONLY, "true"); @SuppressWarnings("unchecked") var resultat = query.getResultList(); @@ -157,7 +159,7 @@ and fagsak_id not in (select fagsak_id from behandling bi2 join behandling_arsak public List finnNesteHundreSakerSomErMerketPraksisUtsettelse(Long fraFagsakId) { var query = entityManager.createNativeQuery(QUERY_MERKET) - .setParameter("fraFagsakId", fraFagsakId == null ? 0 : fraFagsakId) + .setParameter(FRA_FAGSAK_ID, fraFagsakId == null ? 0 : fraFagsakId) .setParameter("feilp", BehandlingÅrsakType.FEIL_PRAKSIS_UTSETTELSE.getKode()) .setHint(HibernateHints.HINT_READ_ONLY, "true"); @SuppressWarnings("unchecked") diff --git a/domenetjenester/medlem/src/main/java/no/nav/foreldrepenger/domene/medlem/MedlemskapVurderingPeriodeTjeneste.java b/domenetjenester/medlem/src/main/java/no/nav/foreldrepenger/domene/medlem/MedlemskapVurderingPeriodeTjeneste.java index 86d7ad58af3..88bb567791e 100644 --- a/domenetjenester/medlem/src/main/java/no/nav/foreldrepenger/domene/medlem/MedlemskapVurderingPeriodeTjeneste.java +++ b/domenetjenester/medlem/src/main/java/no/nav/foreldrepenger/domene/medlem/MedlemskapVurderingPeriodeTjeneste.java @@ -18,8 +18,10 @@ public class MedlemskapVurderingPeriodeTjeneste { private static final Period BOSATT_TILBAKE_TID = Period.ofMonths(12); private static final Period MEDLEMSKAP_ES = BotidCore2024.FORUTGÅENDE_MEDLEMSKAP_TIDSPERIODE; + private static final String MANGLER_YTELSE = "Mangler ytelse"; private BotidCore2024 botidCore2024; + MedlemskapVurderingPeriodeTjeneste() { // CDI } @@ -35,7 +37,7 @@ public LocalDateInterval bosattVurderingsintervall(BehandlingReferanse ref, Skj var intervallSluttdato = switch (ref.fagsakYtelseType()) { case ENGANGSTØNAD -> referansedato; case FORELDREPENGER, SVANGERSKAPSPENGER -> stp.getUttaksintervall().map(LocalDateInterval::getTomDato).orElse(referansedato); - case null, default -> throw new IllegalArgumentException("Mangler ytelse"); + case null, default -> throw new IllegalArgumentException(MANGLER_YTELSE); }; var baseForStartdato = minDato(referansedato, LocalDate.now()); var intervallStartdato = startBosatt(ref.fagsakYtelseType(), stp, baseForStartdato); @@ -47,7 +49,7 @@ public LocalDateInterval lovligOppholdVurderingsintervall(BehandlingReferanse re var intervallSluttdato = switch (ref.fagsakYtelseType()) { case ENGANGSTØNAD -> referansedato; case FORELDREPENGER, SVANGERSKAPSPENGER -> stp.getUttaksintervall().map(LocalDateInterval::getTomDato).orElse(referansedato); - case null, default -> throw new IllegalArgumentException("Mangler ytelse"); + case null, default -> throw new IllegalArgumentException(MANGLER_YTELSE); }; var intervallStartdato = startLovligOpphold(ref.fagsakYtelseType(), stp, referansedato); return new LocalDateInterval(intervallStartdato, intervallSluttdato); @@ -57,7 +59,7 @@ private LocalDate getReferansedato(BehandlingReferanse ref, Skjæringstidspunkt return switch (ref.fagsakYtelseType()) { case ENGANGSTØNAD -> referansedatoES(stp); case FORELDREPENGER, SVANGERSKAPSPENGER -> stp.getUtledetSkjæringstidspunkt(); - case null, default -> throw new IllegalArgumentException("Mangler ytelse"); + case null, default -> throw new IllegalArgumentException(MANGLER_YTELSE); }; } diff --git a/domenetjenester/mottak/src/test/java/no/nav/foreldrepenger/mottak/dokumentmottak/impl/FileToStringUtil.java b/domenetjenester/mottak/src/test/java/no/nav/foreldrepenger/mottak/dokumentmottak/impl/FileToStringUtil.java index 36b6aa8b015..072d06eb682 100644 --- a/domenetjenester/mottak/src/test/java/no/nav/foreldrepenger/mottak/dokumentmottak/impl/FileToStringUtil.java +++ b/domenetjenester/mottak/src/test/java/no/nav/foreldrepenger/mottak/dokumentmottak/impl/FileToStringUtil.java @@ -7,9 +7,6 @@ import java.nio.file.Paths; public class FileToStringUtil { - public FileToStringUtil() { - } - public String readFile(String filename) throws URISyntaxException, IOException { var path = Paths.get(getClass().getClassLoader().getResource(filename).toURI()); return new String(Files.readAllBytes(path), StandardCharsets.UTF_8); diff --git a/domenetjenester/mottak/src/test/java/no/nav/foreldrepenger/mottak/hendelser/DummyDatavarehustjeneste.java b/domenetjenester/mottak/src/test/java/no/nav/foreldrepenger/mottak/hendelser/DummyDatavarehustjeneste.java index 3f9ced109a0..97742968508 100644 --- a/domenetjenester/mottak/src/test/java/no/nav/foreldrepenger/mottak/hendelser/DummyDatavarehustjeneste.java +++ b/domenetjenester/mottak/src/test/java/no/nav/foreldrepenger/mottak/hendelser/DummyDatavarehustjeneste.java @@ -1,12 +1,9 @@ package no.nav.foreldrepenger.mottak.hendelser; -import java.util.Collection; - import jakarta.enterprise.context.ApplicationScoped; import jakarta.enterprise.inject.Alternative; import no.nav.foreldrepenger.behandlingslager.behandling.Behandling; -import no.nav.foreldrepenger.behandlingslager.behandling.aksjonspunkt.Aksjonspunkt; import no.nav.foreldrepenger.behandlingslager.behandling.vedtak.BehandlingVedtak; import no.nav.foreldrepenger.datavarehus.tjeneste.DatavarehusTjeneste; @@ -21,12 +18,11 @@ public class DummyDatavarehustjeneste implements DatavarehusTjeneste { @Override public void lagreNedBehandling(Long behandlingId) { - + // NOSONAR } @Override public void lagreNedBehandling(Behandling behandling, BehandlingVedtak vedtak) { - + // NOSONAR } - } diff --git "a/domenetjenester/okonomistotte/src/main/java/no/nav/foreldrepenger/\303\270konomist\303\270tte/oppdrag/mapper/LagOppdragTjeneste.java" "b/domenetjenester/okonomistotte/src/main/java/no/nav/foreldrepenger/\303\270konomist\303\270tte/oppdrag/mapper/LagOppdragTjeneste.java" index 8605a17fd65..283e2aa79b0 100644 --- "a/domenetjenester/okonomistotte/src/main/java/no/nav/foreldrepenger/\303\270konomist\303\270tte/oppdrag/mapper/LagOppdragTjeneste.java" +++ "b/domenetjenester/okonomistotte/src/main/java/no/nav/foreldrepenger/\303\270konomist\303\270tte/oppdrag/mapper/LagOppdragTjeneste.java" @@ -3,19 +3,18 @@ import java.util.List; import java.util.Optional; -import jakarta.enterprise.context.Dependent; - import no.nav.foreldrepenger.behandlingslager.økonomioppdrag.Oppdragskontroll; import no.nav.foreldrepenger.økonomistøtte.oppdrag.domene.Oppdrag; import no.nav.foreldrepenger.økonomistøtte.oppdrag.tjeneste.EndringsdatoTjeneste; import no.nav.foreldrepenger.økonomistøtte.oppdrag.tjeneste.OppdragFactory; -@Dependent public class LagOppdragTjeneste { - public LagOppdragTjeneste() { } + private LagOppdragTjeneste() { + // Skjuler default konstruktør + } - public Optional lagOppdrag(OppdragInput input, boolean brukFellesEndringstidspunkt, final Oppdragskontroll eksisterendeOppdragskontroll) { + public static Optional lagOppdrag(OppdragInput input, boolean brukFellesEndringstidspunkt, final Oppdragskontroll eksisterendeOppdragskontroll) { var målbilde = input.getTilkjentYtelse(); var tidligereOppdrag = input.getTidligereOppdrag(); diff --git "a/domenetjenester/okonomistotte/src/main/java/no/nav/foreldrepenger/\303\270konomist\303\270tte/oppdrag/tjeneste/OppdragskontrollTjenesteImpl.java" "b/domenetjenester/okonomistotte/src/main/java/no/nav/foreldrepenger/\303\270konomist\303\270tte/oppdrag/tjeneste/OppdragskontrollTjenesteImpl.java" index e58016de294..e5becf2ac1e 100644 --- "a/domenetjenester/okonomistotte/src/main/java/no/nav/foreldrepenger/\303\270konomist\303\270tte/oppdrag/tjeneste/OppdragskontrollTjenesteImpl.java" +++ "b/domenetjenester/okonomistotte/src/main/java/no/nav/foreldrepenger/\303\270konomist\303\270tte/oppdrag/tjeneste/OppdragskontrollTjenesteImpl.java" @@ -16,17 +16,14 @@ public class OppdragskontrollTjenesteImpl implements OppdragskontrollTjeneste { private ØkonomioppdragRepository økonomioppdragRepository; - private LagOppdragTjeneste lagOppdragTjeneste; OppdragskontrollTjenesteImpl() { //for cdi proxy } @Inject - public OppdragskontrollTjenesteImpl(LagOppdragTjeneste lagOppdragTjeneste, - ØkonomioppdragRepository økonomioppdragRepository) { + public OppdragskontrollTjenesteImpl(ØkonomioppdragRepository økonomioppdragRepository) { this.økonomioppdragRepository = økonomioppdragRepository; - this.lagOppdragTjeneste = lagOppdragTjeneste; } /** @@ -53,7 +50,7 @@ public void lagre(Oppdragskontroll oppdragskontroll) { private Optional opprettOppdrag(OppdragInput input, boolean brukFellesEndringstidspunkt) { var oppdragFraFør = økonomioppdragRepository.finnOppdragForBehandling(input.getBehandlingId()); - var oppdragskontroll = lagOppdragTjeneste.lagOppdrag(input, brukFellesEndringstidspunkt, oppdragFraFør.orElse(null)); + var oppdragskontroll = LagOppdragTjeneste.lagOppdrag(input, brukFellesEndringstidspunkt, oppdragFraFør.orElse(null)); oppdragskontroll.ifPresent(OppdragskontrollPostConditionCheck::valider); return oppdragskontroll; } diff --git "a/domenetjenester/okonomistotte/src/main/java/no/nav/foreldrepenger/\303\270konomist\303\270tte/queue/config/FellesJmsKonfig.java" "b/domenetjenester/okonomistotte/src/main/java/no/nav/foreldrepenger/\303\270konomist\303\270tte/queue/config/FellesJmsKonfig.java" index 6eac4484c48..4995c5b08ad 100644 --- "a/domenetjenester/okonomistotte/src/main/java/no/nav/foreldrepenger/\303\270konomist\303\270tte/queue/config/FellesJmsKonfig.java" +++ "b/domenetjenester/okonomistotte/src/main/java/no/nav/foreldrepenger/\303\270konomist\303\270tte/queue/config/FellesJmsKonfig.java" @@ -1,11 +1,12 @@ package no.nav.foreldrepenger.økonomistøtte.queue.config; +import static com.ibm.msg.client.jakarta.wmq.common.CommonConstants.WMQ_MESSAGE_BODY_MQ; + import jakarta.jms.JMSException; import com.ibm.mq.jakarta.jms.MQConnectionFactory; import com.ibm.mq.jakarta.jms.MQQueue; import com.ibm.msg.client.jakarta.jms.JmsConstants; -import com.ibm.msg.client.jakarta.wmq.WMQConstants; import com.ibm.msg.client.jakarta.wmq.compat.jms.internal.JMSC; import no.nav.foreldrepenger.felles.jms.JmsKonfig; @@ -16,7 +17,7 @@ public class FellesJmsKonfig { protected MQConnectionFactory mqConnectionFactory; protected MQQueue mqQueue; - private static final int MQ_TARGET_CLIENT = WMQConstants.WMQ_MESSAGE_BODY_MQ; + private static final int MQ_TARGET_CLIENT = WMQ_MESSAGE_BODY_MQ; protected static MQQueue settOppMessageQueue(String queueName) throws JMSException { return settOppMessageQueue(queueName, false); diff --git "a/domenetjenester/okonomistotte/src/test/java/no/nav/foreldrepenger/\303\270konomist\303\270tte/SimulerOppdragTjenesteESTest.java" "b/domenetjenester/okonomistotte/src/test/java/no/nav/foreldrepenger/\303\270konomist\303\270tte/SimulerOppdragTjenesteESTest.java" index 0de0f181d23..4bc86b1645c 100644 --- "a/domenetjenester/okonomistotte/src/test/java/no/nav/foreldrepenger/\303\270konomist\303\270tte/SimulerOppdragTjenesteESTest.java" +++ "b/domenetjenester/okonomistotte/src/test/java/no/nav/foreldrepenger/\303\270konomist\303\270tte/SimulerOppdragTjenesteESTest.java" @@ -34,7 +34,6 @@ import no.nav.foreldrepenger.domene.typer.AktørId; import no.nav.foreldrepenger.domene.typer.PersonIdent; import no.nav.foreldrepenger.domene.typer.Saksnummer; -import no.nav.foreldrepenger.økonomistøtte.oppdrag.mapper.LagOppdragTjeneste; import no.nav.foreldrepenger.økonomistøtte.oppdrag.tjeneste.OppdragskontrollTjenesteImpl; import no.nav.foreldrepenger.økonomistøtte.simulering.klient.OppdragsKontrollDtoMapper; @@ -85,7 +84,7 @@ public void setup() { var oppdragInputTjeneste = new OppdragInputTjeneste(behandlingRepository, null, behandlingVedtakRepository, familieHendelseRepository, tilbakekrevingRepository, personinfoAdapter, økonomioppdragRepository, beregningRepository); - simulerOppdragTjeneste = new SimulerOppdragTjeneste(new OppdragskontrollTjenesteImpl(new LagOppdragTjeneste(), økonomioppdragRepository), + simulerOppdragTjeneste = new SimulerOppdragTjeneste(new OppdragskontrollTjenesteImpl(økonomioppdragRepository), oppdragInputTjeneste); } diff --git "a/domenetjenester/okonomistotte/src/test/java/no/nav/foreldrepenger/\303\270konomist\303\270tte/oppdrag/NyOppdragskontrollTjenesteENDRTest.java" "b/domenetjenester/okonomistotte/src/test/java/no/nav/foreldrepenger/\303\270konomist\303\270tte/oppdrag/NyOppdragskontrollTjenesteENDRTest.java" index e951afa1417..5213b3a3a60 100644 --- "a/domenetjenester/okonomistotte/src/test/java/no/nav/foreldrepenger/\303\270konomist\303\270tte/oppdrag/NyOppdragskontrollTjenesteENDRTest.java" +++ "b/domenetjenester/okonomistotte/src/test/java/no/nav/foreldrepenger/\303\270konomist\303\270tte/oppdrag/NyOppdragskontrollTjenesteENDRTest.java" @@ -334,6 +334,7 @@ void test_manglende_inntrekk_tfp_5130() { } @Test + @Disabled(value = "Midlertidig slått av Ompostering siden økonomi klarer å utlede selv den beste datoen for ompostering.") void skalSendeOppdragMedOmpostering116OgSetteDatoOmposterFomTilFørsteUttaksdatoFraForrigeBehandlingForBrukerNårEndringsdatoErTidligere() { // Arrange var beregningsresultat = buildEmptyBeregningsresultatFP(); diff --git "a/domenetjenester/okonomistotte/src/test/java/no/nav/foreldrepenger/\303\270konomist\303\270tte/oppdrag/NyOppdragskontrollTjenesteImplKontantytelseTest.java" "b/domenetjenester/okonomistotte/src/test/java/no/nav/foreldrepenger/\303\270konomist\303\270tte/oppdrag/NyOppdragskontrollTjenesteImplKontantytelseTest.java" index aa241c8d45b..e9db12af242 100644 --- "a/domenetjenester/okonomistotte/src/test/java/no/nav/foreldrepenger/\303\270konomist\303\270tte/oppdrag/NyOppdragskontrollTjenesteImplKontantytelseTest.java" +++ "b/domenetjenester/okonomistotte/src/test/java/no/nav/foreldrepenger/\303\270konomist\303\270tte/oppdrag/NyOppdragskontrollTjenesteImplKontantytelseTest.java" @@ -31,7 +31,6 @@ import no.nav.foreldrepenger.økonomistøtte.oppdrag.domene.samlinger.GruppertYtelse; import no.nav.foreldrepenger.økonomistøtte.oppdrag.domene.samlinger.OverordnetOppdragKjedeOversikt; import no.nav.foreldrepenger.økonomistøtte.oppdrag.mapper.EksisterendeOppdragMapper; -import no.nav.foreldrepenger.økonomistøtte.oppdrag.mapper.LagOppdragTjeneste; import no.nav.foreldrepenger.økonomistøtte.oppdrag.mapper.OppdragInput; import no.nav.foreldrepenger.økonomistøtte.oppdrag.tjeneste.OppdragskontrollTjenesteImpl; @@ -49,7 +48,7 @@ class NyOppdragskontrollTjenesteImplKontantytelseTest { @BeforeEach public void setUp() { - oppdragskontrollTjeneste = new OppdragskontrollTjenesteImpl(new LagOppdragTjeneste(), mock(ØkonomioppdragRepository.class)); + oppdragskontrollTjeneste = new OppdragskontrollTjenesteImpl(mock(ØkonomioppdragRepository.class)); } @Test @@ -83,12 +82,12 @@ void hentOppdragskontrollTestES() { assertThat(oppdragskontroll).isPresent(); assertThat(oppdragskontroll.get().getOppdrag110Liste()).hasSize(1); - var oppdrag110 = oppdragskontroll.get().getOppdrag110Liste().get(0); + var oppdrag110 = oppdragskontroll.get().getOppdrag110Liste().getFirst(); assertThat(oppdrag110).isNotNull(); assertThat(oppdrag110.getOppdragslinje150Liste()).hasSize(1); assertThat(oppdrag110.getAvstemming()).isNotNull(); - var oppdrlinje150 = oppdrag110.getOppdragslinje150Liste().get(0); + var oppdrlinje150 = oppdrag110.getOppdragslinje150Liste().getFirst(); assertThat(oppdrlinje150).isNotNull(); assertThat(oppdrlinje150.getOppdrag110()).isNotNull(); } @@ -104,7 +103,7 @@ void hentOppdragskontrollTestES() { var originaltOppdrag = oppdragskontrollTjeneste.opprettOppdrag(inputBuilder.build()); assertThat(originaltOppdrag).isPresent(); - var originaltOppdrag110 = originaltOppdrag.get().getOppdrag110Liste().get(0); + var originaltOppdrag110 = originaltOppdrag.get().getOppdrag110Liste().getFirst(); // Arrange 2: Revurdering var gruppertYtelseBuilder2 = getGruppertYtelseBuilder(KodeKlassifik.ES_FØDSEL, VEDTAKSDATO, SATS_ES * 2); @@ -114,7 +113,7 @@ void hentOppdragskontrollTestES() { // Assert 2: Revurdering assertThat(oppdragRevurdering).isPresent(); - var originalOppdragslinje150 = originaltOppdrag110.getOppdragslinje150Liste().get(0); + var originalOppdragslinje150 = originaltOppdrag110.getOppdragslinje150Liste().getFirst(); var oppdragslinje150 = verifiserOppdrag110(oppdragRevurdering.get(), KodeEndring.ENDR, originaltOppdrag110.getFagsystemId()); verifiserOppdragslinje150(oppdragslinje150, KodeEndringLinje.NY, null, @@ -133,7 +132,7 @@ void hentOppdragskontrollTestES() { var originaltOppdrag = oppdragskontrollTjeneste.opprettOppdrag(inputBuilder.build()); assertThat(originaltOppdrag).isPresent(); - var originaltOppdrag110 = originaltOppdrag.get().getOppdrag110Liste().get(0); + var originaltOppdrag110 = originaltOppdrag.get().getOppdrag110Liste().getFirst(); // Arrange 2: Revurdering var inputBuilder2 = getInputStandardBuilder(GruppertYtelse.TOM).medTidligereOppdrag(mapTidligereOppdrag(List.of(originaltOppdrag.get()))); @@ -142,7 +141,7 @@ void hentOppdragskontrollTestES() { // Assert 2: Revurdering assertThat(oppdragRevurdering).isPresent(); - var originalOppdragslinje150 = originaltOppdrag110.getOppdragslinje150Liste().get(0); + var originalOppdragslinje150 = originaltOppdrag110.getOppdragslinje150Liste().getFirst(); var oppdragslinje150 = verifiserOppdrag110(oppdragRevurdering.get(), KodeEndring.ENDR, originaltOppdrag110.getFagsystemId()); verifiserOppdragslinje150(oppdragslinje150, KodeEndringLinje.ENDR, KodeStatusLinje.OPPH, @@ -175,9 +174,9 @@ void hentOppdragskontrollTestES() { // Assert 3: Revurdering assertThat(oppdragAndreRevurdering).isPresent(); - var oppdrag110 = førsteOppdrag.getOppdrag110Liste().get(0); + var oppdrag110 = førsteOppdrag.getOppdrag110Liste().getFirst(); assertThat(oppdrag110.getOppdragslinje150Liste()).hasSize(1); - var førstegangsOpp150 = oppdrag110.getOppdragslinje150Liste().get(0); + var førstegangsOpp150 = oppdrag110.getOppdragslinje150Liste().getFirst(); var andreRevurderingopp150 = verifiserOppdrag110(oppdragAndreRevurdering.get(), KodeEndring.ENDR, oppdrag110.getFagsystemId()); assertThat(andreRevurderingopp150.getDatoVedtakFom()).isEqualTo(førstegangsOpp150.getDatoVedtakFom()); @@ -192,11 +191,11 @@ private Oppdragslinje150 verifiserOppdrag110(Oppdragskontroll oppdragskontroll, KodeEndring kodeEndring, Long fagsystemId) { assertThat(oppdragskontroll.getOppdrag110Liste()).hasSize(1); - var oppdrag110 = oppdragskontroll.getOppdrag110Liste().get(0); + var oppdrag110 = oppdragskontroll.getOppdrag110Liste().getFirst(); assertThat(oppdrag110.getKodeEndring()).isEqualTo(kodeEndring); assertThat(oppdrag110.getFagsystemId()).isEqualTo(fagsystemId); assertThat(oppdrag110.getOppdragslinje150Liste()).hasSize(1); - return oppdrag110.getOppdragslinje150Liste().get(0); + return oppdrag110.getOppdragslinje150Liste().getFirst(); } private void verifiserOppdragslinje150(Oppdragslinje150 oppdragslinje150, @@ -226,7 +225,7 @@ private List verifiserOppdragslinje150(List oppdra var løpenummer = 100L; for (var oppdrag110 : oppdrag110Liste) { assertThat(oppdrag110.getOppdragslinje150Liste()).hasSize(1); - var oppdragslinje150 = oppdragslinje150List.get(0); + var oppdragslinje150 = oppdragslinje150List.getFirst(); assertThat(oppdragslinje150.getKodeEndringLinje()).isEqualTo(KodeEndringLinje.NY); assertThat(oppdragslinje150.getVedtakId()).isEqualTo(VEDTAKSDATO.toString()); assertThat(oppdragslinje150.getDelytelseId()).isEqualTo( diff --git "a/domenetjenester/okonomistotte/src/test/java/no/nav/foreldrepenger/\303\270konomist\303\270tte/oppdrag/NyOppdragskontrollTjenesteTestBase.java" "b/domenetjenester/okonomistotte/src/test/java/no/nav/foreldrepenger/\303\270konomist\303\270tte/oppdrag/NyOppdragskontrollTjenesteTestBase.java" index 0bf2b868090..81ade1a8431 100644 --- "a/domenetjenester/okonomistotte/src/test/java/no/nav/foreldrepenger/\303\270konomist\303\270tte/oppdrag/NyOppdragskontrollTjenesteTestBase.java" +++ "b/domenetjenester/okonomistotte/src/test/java/no/nav/foreldrepenger/\303\270konomist\303\270tte/oppdrag/NyOppdragskontrollTjenesteTestBase.java" @@ -25,7 +25,6 @@ import no.nav.foreldrepenger.behandlingslager.økonomioppdrag.Oppdragskontroll; import no.nav.foreldrepenger.behandlingslager.økonomioppdrag.Oppdragslinje150; import no.nav.foreldrepenger.behandlingslager.økonomioppdrag.koder.KodeEndring; -import no.nav.foreldrepenger.behandlingslager.økonomioppdrag.koder.KodeFagområde; import no.nav.foreldrepenger.behandlingslager.økonomioppdrag.koder.KodeKlassifik; import no.nav.foreldrepenger.behandlingslager.økonomioppdrag.ØkonomioppdragRepository; import no.nav.foreldrepenger.domene.typer.AktørId; @@ -41,7 +40,6 @@ import no.nav.foreldrepenger.økonomistøtte.oppdrag.domene.samlinger.GruppertYtelse; import no.nav.foreldrepenger.økonomistøtte.oppdrag.domene.samlinger.OverordnetOppdragKjedeOversikt; import no.nav.foreldrepenger.økonomistøtte.oppdrag.mapper.EksisterendeOppdragMapper; -import no.nav.foreldrepenger.økonomistøtte.oppdrag.mapper.LagOppdragTjeneste; import no.nav.foreldrepenger.økonomistøtte.oppdrag.mapper.OppdragInput; import no.nav.foreldrepenger.økonomistøtte.oppdrag.mapper.TilkjentYtelseMapper; import no.nav.foreldrepenger.økonomistøtte.oppdrag.tjeneste.OppdragskontrollTjenesteImpl; @@ -94,7 +92,7 @@ protected OverordnetOppdragKjedeOversikt mapTidligereOppdrag(List nyOpp110Liste, var opp110 = refusjonsinfo156.get().getOppdragslinje150().getOppdrag110(); assertThat(opp110.getKodeEndring()).isEqualTo(KodeEndring.NY); } else { - assertThat(oppdr110Revurd.getKodeEndring()).isEqualTo(KodeFagområde.FP - .equals( - oppdr110Revurd.getKodeFagomrade()) ? KodeEndring.ENDR : KodeEndring.ENDR); + assertThat(oppdr110Revurd.getKodeEndring()).isEqualTo(KodeEndring.ENDR); } assertThat(oppdr110Revurd.getOppdragslinje150Liste()).isNotEmpty(); var nyMottaker = erMottakerNy(oppdr110Revurd); diff --git a/domenetjenester/registerinnhenting/src/main/java/no/nav/foreldrepenger/domene/registerinnhenting/impl/startpunkt/StartpunktUtlederFamilieHendelse.java b/domenetjenester/registerinnhenting/src/main/java/no/nav/foreldrepenger/domene/registerinnhenting/impl/startpunkt/StartpunktUtlederFamilieHendelse.java index 005e3809449..a942e395f51 100644 --- a/domenetjenester/registerinnhenting/src/main/java/no/nav/foreldrepenger/domene/registerinnhenting/impl/startpunkt/StartpunktUtlederFamilieHendelse.java +++ b/domenetjenester/registerinnhenting/src/main/java/no/nav/foreldrepenger/domene/registerinnhenting/impl/startpunkt/StartpunktUtlederFamilieHendelse.java @@ -23,6 +23,8 @@ @GrunnlagRef(FamilieHendelseGrunnlagEntitet.ENTITY_NAME) class StartpunktUtlederFamilieHendelse implements StartpunktUtleder { + private static final String ANTALL_BARN = "antall barn"; + private SkjæringstidspunktTjeneste skjæringstidspunktTjeneste; private FamilieHendelseTjeneste familieHendelseTjeneste; private DekningsgradTjeneste dekningsgradTjeneste; @@ -50,15 +52,15 @@ public StartpunktType utledStartpunkt(BehandlingReferanse ref, Skjæringstidspun return StartpunktType.INNGANGSVILKÅR_OPPLYSNINGSPLIKT; } if (erAntallBekreftedeBarnEndret(grunnlag1, grunnlag2)) { - FellesStartpunktUtlederLogger.loggEndringSomFørteTilStartpunkt(this.getClass().getSimpleName(), StartpunktType.SØKERS_RELASJON_TIL_BARNET, "antall barn", grunnlag1, grunnlag2); + FellesStartpunktUtlederLogger.loggEndringSomFørteTilStartpunkt(this.getClass().getSimpleName(), StartpunktType.SØKERS_RELASJON_TIL_BARNET, ANTALL_BARN, grunnlag1, grunnlag2); return StartpunktType.SØKERS_RELASJON_TIL_BARNET; } if (erTilkommetRegisterDødfødselMedDekningsgrad80(ref, grunnlag1, grunnlag2)) { if (ref.fagsakYtelseType() == FagsakYtelseType.FORELDREPENGER) { - FellesStartpunktUtlederLogger.loggEndringSomFørteTilStartpunkt(this.getClass().getSimpleName(), StartpunktType.DEKNINGSGRAD, "antall barn", grunnlag1, grunnlag2); + FellesStartpunktUtlederLogger.loggEndringSomFørteTilStartpunkt(this.getClass().getSimpleName(), StartpunktType.DEKNINGSGRAD, ANTALL_BARN, grunnlag1, grunnlag2); return StartpunktType.DEKNINGSGRAD; } else { - FellesStartpunktUtlederLogger.loggEndringSomFørteTilStartpunkt(this.getClass().getSimpleName(), StartpunktType.BEREGNING, "antall barn", grunnlag1, grunnlag2); + FellesStartpunktUtlederLogger.loggEndringSomFørteTilStartpunkt(this.getClass().getSimpleName(), StartpunktType.BEREGNING, ANTALL_BARN, grunnlag1, grunnlag2); return StartpunktType.BEREGNING; } } diff --git a/domenetjenester/registerinnhenting/src/main/java/no/nav/foreldrepenger/domene/registerinnhenting/impl/startpunkt/StartpunktUtlederPersonopplysning.java b/domenetjenester/registerinnhenting/src/main/java/no/nav/foreldrepenger/domene/registerinnhenting/impl/startpunkt/StartpunktUtlederPersonopplysning.java index 6657a581c44..b73b54cfaab 100644 --- a/domenetjenester/registerinnhenting/src/main/java/no/nav/foreldrepenger/domene/registerinnhenting/impl/startpunkt/StartpunktUtlederPersonopplysning.java +++ b/domenetjenester/registerinnhenting/src/main/java/no/nav/foreldrepenger/domene/registerinnhenting/impl/startpunkt/StartpunktUtlederPersonopplysning.java @@ -41,6 +41,8 @@ @GrunnlagRef(PersonInformasjonEntitet.ENTITY_NAME) class StartpunktUtlederPersonopplysning implements StartpunktUtleder { + private static final String BARNETS_DØDSDATO = "barnets dødsdato"; + private PersonopplysningRepository personopplysningRepository; private BehandlingRepository behandlingRepository; @@ -108,14 +110,14 @@ private List hentAlleStartpunktForPersonopplysninger(BehandlingR if (poDiff.erBarnDødsdatoEndret()) { if (ref.fagsakYtelseType() == FagsakYtelseType.FORELDREPENGER) { if (har80Dekningsgrad(ref)) { - FellesStartpunktUtlederLogger.loggEndringSomFørteTilStartpunkt(this.getClass().getSimpleName(), StartpunktType.DEKNINGSGRAD, "barnets dødsdato", grunnlag1.getId(), grunnlag2.getId()); + FellesStartpunktUtlederLogger.loggEndringSomFørteTilStartpunkt(this.getClass().getSimpleName(), StartpunktType.DEKNINGSGRAD, BARNETS_DØDSDATO, grunnlag1.getId(), grunnlag2.getId()); startpunkter.add(StartpunktType.DEKNINGSGRAD); } else { - FellesStartpunktUtlederLogger.loggEndringSomFørteTilStartpunkt(this.getClass().getSimpleName(), StartpunktType.UTTAKSVILKÅR, "barnets dødsdato", grunnlag1.getId(), grunnlag2.getId()); + FellesStartpunktUtlederLogger.loggEndringSomFørteTilStartpunkt(this.getClass().getSimpleName(), StartpunktType.UTTAKSVILKÅR, BARNETS_DØDSDATO, grunnlag1.getId(), grunnlag2.getId()); startpunkter.add(StartpunktType.UTTAKSVILKÅR); } } else { - FellesStartpunktUtlederLogger.loggEndringSomFørteTilStartpunkt(this.getClass().getSimpleName(), StartpunktType.BEREGNING, "barnets dødsdato", grunnlag1.getId(), grunnlag2.getId()); + FellesStartpunktUtlederLogger.loggEndringSomFørteTilStartpunkt(this.getClass().getSimpleName(), StartpunktType.BEREGNING, BARNETS_DØDSDATO, grunnlag1.getId(), grunnlag2.getId()); startpunkter.add(StartpunktType.BEREGNING); } } diff --git a/domenetjenester/risikoklassifisering/src/main/java/no/nav/foreldrepenger/domene/risikoklassifisering/tjeneste/FpriskTjeneste.java b/domenetjenester/risikoklassifisering/src/main/java/no/nav/foreldrepenger/domene/risikoklassifisering/tjeneste/FpriskTjeneste.java index 767a951babb..e0e8b65ccf7 100644 --- a/domenetjenester/risikoklassifisering/src/main/java/no/nav/foreldrepenger/domene/risikoklassifisering/tjeneste/FpriskTjeneste.java +++ b/domenetjenester/risikoklassifisering/src/main/java/no/nav/foreldrepenger/domene/risikoklassifisering/tjeneste/FpriskTjeneste.java @@ -26,6 +26,8 @@ public class FpriskTjeneste { private static final Logger LOG = LoggerFactory.getLogger(FpriskTjeneste.class); + private static final String REQUEST = "request"; + private final URI sendOppdragEndpoint; private final URI lagreVurderingEndpoint; private final URI hentRisikoklassifiseringEndpoint; @@ -42,14 +44,14 @@ public FpriskTjeneste() { /** * Henter resultat av risikoklassifisering og evt vurdering og faresignaler fra fprisk - * @param request + * @param hentRisikovurderingDto * @return RisikovurderingResultatDto som inneholder både risikoklasse, vurdering og faresignaler */ - public Optional hentFaresignalerForBehandling(HentRisikovurderingDto request) { - Objects.requireNonNull(request, "request"); + public Optional hentFaresignalerForBehandling(HentRisikovurderingDto hentRisikovurderingDto) { + Objects.requireNonNull(hentRisikovurderingDto, REQUEST); try { - var rrequest = RestRequest.newPOSTJson(request, hentRisikoklassifiseringEndpoint, restConfig); - return restClient.sendReturnOptional(rrequest, RisikovurderingResultatDto.class); + var request = RestRequest.newPOSTJson(hentRisikovurderingDto, hentRisikoklassifiseringEndpoint, restConfig); + return restClient.sendReturnOptional(request, RisikovurderingResultatDto.class); } catch (Exception e) { LOG.warn("Klarte ikke hente faresignaler fra fprisk", e); return Optional.empty(); @@ -58,13 +60,13 @@ public Optional hentFaresignalerForBehandling(HentRi /** * Sender risikovurdering til fprisk - * @param request + * @param lagreFaresignalVurderingDto */ - protected void sendRisikovurderingTilFprisk(LagreFaresignalVurderingDto request) { - Objects.requireNonNull(request, "request"); + protected void sendRisikovurderingTilFprisk(LagreFaresignalVurderingDto lagreFaresignalVurderingDto) { + Objects.requireNonNull(lagreFaresignalVurderingDto, REQUEST); try { - var rrequest = RestRequest.newPOSTJson(request, lagreVurderingEndpoint, restConfig); - restClient.sendReturnOptional(rrequest, String.class); + var request = RestRequest.newPOSTJson(lagreFaresignalVurderingDto, lagreVurderingEndpoint, restConfig); + restClient.sendReturnOptional(request, String.class); } catch (Exception e) { LOG.warn("Klarte ikke lagre risikovurdering i fprisk", e); throw e; @@ -73,13 +75,13 @@ protected void sendRisikovurderingTilFprisk(LagreFaresignalVurderingDto request) /** * Sender oppdrag om risikoklassifisering til fprisk - * @param request + * @param risikovurderingRequestDto */ - protected void sendRisikoklassifiseringsoppdrag(RisikovurderingRequestDto request) { - Objects.requireNonNull(request, "request"); + protected void sendRisikoklassifiseringsoppdrag(RisikovurderingRequestDto risikovurderingRequestDto) { + Objects.requireNonNull(risikovurderingRequestDto, REQUEST); try { - var rrequest = RestRequest.newPOSTJson(request, sendOppdragEndpoint, restConfig); - restClient.sendReturnOptional(rrequest, String.class); + var request = RestRequest.newPOSTJson(risikovurderingRequestDto, sendOppdragEndpoint, restConfig); + restClient.sendReturnOptional(request, String.class); } catch (Exception e) { LOG.warn("Klarte ikke sende risikovurdering til fprisk", e); throw e; diff --git a/domenetjenester/uttak/src/main/java/no/nav/foreldrepenger/domene/uttak/fastsetteperioder/FastsettePerioderRegelGrunnlagBygger.java b/domenetjenester/uttak/src/main/java/no/nav/foreldrepenger/domene/uttak/fastsetteperioder/FastsettePerioderRegelGrunnlagBygger.java index 2525912ff0c..0130ceb0672 100644 --- a/domenetjenester/uttak/src/main/java/no/nav/foreldrepenger/domene/uttak/fastsetteperioder/FastsettePerioderRegelGrunnlagBygger.java +++ b/domenetjenester/uttak/src/main/java/no/nav/foreldrepenger/domene/uttak/fastsetteperioder/FastsettePerioderRegelGrunnlagBygger.java @@ -35,7 +35,6 @@ public class FastsettePerioderRegelGrunnlagBygger { private InngangsvilkårGrunnlagBygger inngangsvilkårGrunnlagBygger; private OpptjeningGrunnlagBygger opptjeningGrunnlagBygger; private AdopsjonGrunnlagBygger adopsjonGrunnlagBygger; - private KontoerGrunnlagBygger kontoerGrunnlagBygger; private YtelserGrunnlagBygger ytelserGrunnlagBygger; public FastsettePerioderRegelGrunnlagBygger() { @@ -54,7 +53,6 @@ public FastsettePerioderRegelGrunnlagBygger(AnnenPartGrunnlagBygger annenPartGru InngangsvilkårGrunnlagBygger inngangsvilkårGrunnlagBygger, OpptjeningGrunnlagBygger opptjeningGrunnlagBygger, AdopsjonGrunnlagBygger adopsjonGrunnlagBygger, - KontoerGrunnlagBygger kontoerGrunnlagBygger, YtelserGrunnlagBygger ytelserGrunnlagBygger) { this.annenPartGrunnlagBygger = annenPartGrunnlagBygger; this.arbeidGrunnlagBygger = arbeidGrunnlagBygger; @@ -67,7 +65,6 @@ public FastsettePerioderRegelGrunnlagBygger(AnnenPartGrunnlagBygger annenPartGru this.inngangsvilkårGrunnlagBygger = inngangsvilkårGrunnlagBygger; this.opptjeningGrunnlagBygger = opptjeningGrunnlagBygger; this.adopsjonGrunnlagBygger = adopsjonGrunnlagBygger; - this.kontoerGrunnlagBygger = kontoerGrunnlagBygger; this.ytelserGrunnlagBygger = ytelserGrunnlagBygger; } @@ -85,7 +82,7 @@ public RegelGrunnlag byggGrunnlag(UttakInput input, Stønadskontoberegning støn .inngangsvilkår(inngangsvilkårGrunnlagBygger.byggGrunnlag(input)) .opptjening(opptjeningGrunnlagBygger.byggGrunnlag(input)) .adopsjon(adopsjonGrunnlagBygger.byggGrunnlag(foreldrepengerGrunnlag).orElse(null)) - .kontoer(kontoerGrunnlagBygger.byggGrunnlag(input, stønadskontoberegning.getStønadskontoutregning())) + .kontoer(KontoerGrunnlagBygger.byggGrunnlag(input, stønadskontoberegning.getStønadskontoutregning())) .ytelser(ytelserGrunnlagBygger.byggGrunnlag(input)) .build(); } diff --git a/domenetjenester/uttak/src/main/java/no/nav/foreldrepenger/domene/uttak/fastsetteperioder/grunnlagbyggere/KontoerGrunnlagBygger.java b/domenetjenester/uttak/src/main/java/no/nav/foreldrepenger/domene/uttak/fastsetteperioder/grunnlagbyggere/KontoerGrunnlagBygger.java index 256c11ab3a9..8e71b3a2e98 100644 --- a/domenetjenester/uttak/src/main/java/no/nav/foreldrepenger/domene/uttak/fastsetteperioder/grunnlagbyggere/KontoerGrunnlagBygger.java +++ b/domenetjenester/uttak/src/main/java/no/nav/foreldrepenger/domene/uttak/fastsetteperioder/grunnlagbyggere/KontoerGrunnlagBygger.java @@ -2,8 +2,6 @@ import java.util.Map; -import jakarta.enterprise.context.ApplicationScoped; - import no.nav.foreldrepenger.behandlingslager.behandling.personopplysning.RelasjonsRolleType; import no.nav.foreldrepenger.behandlingslager.uttak.fp.StønadskontoType; import no.nav.foreldrepenger.domene.uttak.UttakEnumMapper; @@ -11,10 +9,10 @@ import no.nav.foreldrepenger.regler.uttak.fastsetteperiode.grunnlag.Konto; import no.nav.foreldrepenger.regler.uttak.fastsetteperiode.grunnlag.Kontoer; -@ApplicationScoped public class KontoerGrunnlagBygger { - public KontoerGrunnlagBygger() { + private KontoerGrunnlagBygger() { + // Skjuler default konstruktør } /* @@ -31,7 +29,7 @@ public KontoerGrunnlagBygger() { * - kan utsettes og utvide stønadsperioden * - Brukes framfor utenAktivitetskravDager fom FAB */ - public Kontoer.Builder byggGrunnlag(UttakInput uttakInput, Map stønadskontoutregning) { + public static Kontoer.Builder byggGrunnlag(UttakInput uttakInput, Map stønadskontoutregning) { var kontoer = stønadskontoutregning.entrySet().stream() .filter(k -> k.getKey().erStønadsdager()) .map(k -> new Konto.Builder().trekkdager(k.getValue()).type(UttakEnumMapper.map(k.getKey()))) @@ -39,7 +37,7 @@ public Kontoer.Builder byggGrunnlag(UttakInput uttakInput, Map stønadskontoer) { + private static Kontoer.Builder getBuilder(UttakInput uttakInput, Map stønadskontoer) { var erMor = RelasjonsRolleType.MORA.equals(uttakInput.getBehandlingReferanse().relasjonRolle()); int toTette = erMor ? finnKontoVerdi(stønadskontoer, StønadskontoType.TETTE_SAKER_MOR) : finnKontoVerdi(stønadskontoer, StønadskontoType.TETTE_SAKER_FAR); return new Kontoer.Builder() @@ -50,7 +48,7 @@ private Kontoer.Builder getBuilder(UttakInput uttakInput, Map konti, StønadskontoType stønadskontoType) { + private static Integer finnKontoVerdi(Map konti, StønadskontoType stønadskontoType) { return konti.getOrDefault(stønadskontoType, 0); } } diff --git "a/domenetjenester/uttak/src/main/java/no/nav/foreldrepenger/domene/uttak/saldo/St\303\270nadskontoSaldoTjeneste.java" "b/domenetjenester/uttak/src/main/java/no/nav/foreldrepenger/domene/uttak/saldo/St\303\270nadskontoSaldoTjeneste.java" index b7e019fc5cb..d7cdcc2bf02 100644 --- "a/domenetjenester/uttak/src/main/java/no/nav/foreldrepenger/domene/uttak/saldo/St\303\270nadskontoSaldoTjeneste.java" +++ "b/domenetjenester/uttak/src/main/java/no/nav/foreldrepenger/domene/uttak/saldo/St\303\270nadskontoSaldoTjeneste.java" @@ -40,17 +40,15 @@ public class StønadskontoSaldoTjeneste { private UtregnetStønadskontoTjeneste utregnetStønadskontoTjeneste; private FpUttakRepository fpUttakRepository; - private KontoerGrunnlagBygger kontoerGrunnlagBygger; StønadskontoSaldoTjeneste() { //For CDI } @Inject - public StønadskontoSaldoTjeneste(UttakRepositoryProvider repositoryProvider, KontoerGrunnlagBygger kontoerGrunnlagBygger, UtregnetStønadskontoTjeneste utregnetStønadskontoTjeneste) { + public StønadskontoSaldoTjeneste(UttakRepositoryProvider repositoryProvider, UtregnetStønadskontoTjeneste utregnetStønadskontoTjeneste) { this.utregnetStønadskontoTjeneste = utregnetStønadskontoTjeneste; this.fpUttakRepository = repositoryProvider.getFpUttakRepository(); - this.kontoerGrunnlagBygger = kontoerGrunnlagBygger; } public SaldoUtregning finnSaldoUtregning(UttakInput uttakInput) { @@ -77,7 +75,7 @@ private SaldoUtregningGrunnlag saldoUtregningGrunnlag(List .orElse(null); if (!stønadskontoer.isEmpty() && !perioderSøker.isEmpty()) { var perioderAnnenpart = perioderAnnenpart(fpGrunnlag); - var kontoer = kontoerGrunnlagBygger.byggGrunnlag(uttakInput, stønadskontoer).build(); + var kontoer = KontoerGrunnlagBygger.byggGrunnlag(uttakInput, stønadskontoer).build(); return SaldoUtregningGrunnlag.forUtregningAvHeleUttaket(perioderSøker, berørtBehandling, perioderAnnenpart, kontoer, søknadOpprettetTidspunkt, sisteSøknadOpprettetTidspunktAnnenpart, UtsettelseCore2021.kreverSammenhengendeUttakTilOgMed()); diff --git a/domenetjenester/uttak/src/test/java/no/nav/foreldrepenger/domene/uttak/fastsetteperioder/FastsettePerioderRegelAdapterTest.java b/domenetjenester/uttak/src/test/java/no/nav/foreldrepenger/domene/uttak/fastsetteperioder/FastsettePerioderRegelAdapterTest.java index d9426443324..759999e0117 100644 --- a/domenetjenester/uttak/src/test/java/no/nav/foreldrepenger/domene/uttak/fastsetteperioder/FastsettePerioderRegelAdapterTest.java +++ b/domenetjenester/uttak/src/test/java/no/nav/foreldrepenger/domene/uttak/fastsetteperioder/FastsettePerioderRegelAdapterTest.java @@ -12,7 +12,6 @@ import java.util.List; import java.util.Objects; import java.util.Optional; -import java.util.stream.Collectors; import org.junit.jupiter.api.BeforeEach; import org.junit.jupiter.api.Test; @@ -71,7 +70,6 @@ import no.nav.foreldrepenger.domene.uttak.fastsetteperioder.grunnlagbyggere.BehandlingGrunnlagBygger; import no.nav.foreldrepenger.domene.uttak.fastsetteperioder.grunnlagbyggere.DatoerGrunnlagBygger; import no.nav.foreldrepenger.domene.uttak.fastsetteperioder.grunnlagbyggere.InngangsvilkårGrunnlagBygger; -import no.nav.foreldrepenger.domene.uttak.fastsetteperioder.grunnlagbyggere.KontoerGrunnlagBygger; import no.nav.foreldrepenger.domene.uttak.fastsetteperioder.grunnlagbyggere.MedlemskapGrunnlagBygger; import no.nav.foreldrepenger.domene.uttak.fastsetteperioder.grunnlagbyggere.OpptjeningGrunnlagBygger; import no.nav.foreldrepenger.domene.uttak.fastsetteperioder.grunnlagbyggere.RettOgOmsorgGrunnlagBygger; @@ -129,7 +127,6 @@ void setUp() { new SøknadGrunnlagBygger(repositoryProvider.getYtelsesFordelingRepository()), new InngangsvilkårGrunnlagBygger(repositoryProvider), new OpptjeningGrunnlagBygger(), new AdopsjonGrunnlagBygger(), - new KontoerGrunnlagBygger(), new YtelserGrunnlagBygger()); iayTjeneste = new AbakusInMemoryInntektArbeidYtelseTjeneste(); fastsettePerioderRegelAdapter = new FastsettePerioderRegelAdapter(grunnlagBygger, @@ -174,20 +171,20 @@ void setUp() { assertThat(fpffPeriode.getResultatType()).isEqualTo(PeriodeResultatType.INNVILGET); assertThat(fpffPeriode.getAktiviteter()).hasSize(1); - assertThat(fpffPeriode.getAktiviteter().get(0).getTrekkonto()).isEqualTo(UttakPeriodeType.FORELDREPENGER_FØR_FØDSEL); - assertThat(fpffPeriode.getAktiviteter().get(0).getTrekkdager()).isEqualTo(new Trekkdager(15)); + assertThat(fpffPeriode.getAktiviteter().getFirst().getTrekkonto()).isEqualTo(UttakPeriodeType.FORELDREPENGER_FØR_FØDSEL); + assertThat(fpffPeriode.getAktiviteter().getFirst().getTrekkdager()).isEqualTo(new Trekkdager(15)); // mødrekvote innvilget assertThat(mkPeriode1.getResultatType()).isEqualTo(PeriodeResultatType.INNVILGET); assertThat(mkPeriode1.isGraderingInnvilget()).isFalse(); assertThat(mkPeriode1.getAktiviteter()).hasSize(1); - assertThat(mkPeriode1.getAktiviteter().get(0).getTrekkonto()).isEqualTo(UttakPeriodeType.MØDREKVOTE); - assertThat(mkPeriode1.getAktiviteter().get(0).getTrekkdager()).isEqualTo( + assertThat(mkPeriode1.getAktiviteter().getFirst().getTrekkonto()).isEqualTo(UttakPeriodeType.MØDREKVOTE); + assertThat(mkPeriode1.getAktiviteter().getFirst().getTrekkdager()).isEqualTo( new Trekkdager(new SimpleLocalDateInterval(mkPeriode1.getFom(), mkPeriode1.getTom()).antallArbeidsdager())); assertThat(mkPeriode2.getResultatType()).isEqualTo(PeriodeResultatType.INNVILGET); assertThat(mkPeriode2.getAktiviteter()).hasSize(1); assertThat(mkPeriode2.isGraderingInnvilget()).isTrue(); - assertThat(mkPeriode2.getAktiviteter().get(0).getTrekkonto()).isEqualTo(UttakPeriodeType.MØDREKVOTE); - assertThat(mkPeriode2.getAktiviteter().get(0).getTrekkdager().merEnn0()).isTrue(); + assertThat(mkPeriode2.getAktiviteter().getFirst().getTrekkonto()).isEqualTo(UttakPeriodeType.MØDREKVOTE); + assertThat(mkPeriode2.getAktiviteter().getFirst().getTrekkdager().merEnn0()).isTrue(); } @Test @@ -217,21 +214,21 @@ void setUp() { var foreldrepengerUttakPerioder = uttakResultatPerioder.stream() .filter(uttakResultatPeriode -> uttakResultatPeriode.getAktiviteter() - .get(0) + .getFirst() .getTrekkonto() .getKode() .equals(StønadskontoType.FORELDREPENGER_FØR_FØDSEL.getKode())) .toList(); assertThat(foreldrepengerUttakPerioder).hasSize(1); - var foreldrePengerUttakPeriode = foreldrepengerUttakPerioder.get(0); + var foreldrePengerUttakPeriode = foreldrepengerUttakPerioder.getFirst(); assertThat(foreldrePengerUttakPeriode.getFom()).isEqualTo(fødselsdato.minusWeeks(3)); assertThat(foreldrePengerUttakPeriode.getTom()).isEqualTo(fødselsdato.minusDays(1)); var gjenståendeDager_foreldrepenger = uker_før_fødsel_foreldrepenger_grense * 5 - maxDagerForeldrepengerFørFødsel; assertThat(maxDagerForeldrepengerFørFødsel - foreldrePengerUttakPeriode.getAktiviteter() - .get(0) + .getFirst() .getTrekkdager() .decimalValue() .intValue()).isEqualTo(gjenståendeDager_foreldrepenger); @@ -241,9 +238,9 @@ void setUp() { var mangledeSøktmkPeriode = finnPeriode(uttakResultatPerioder, fødselsdato, fødselsdato.plusWeeks(veker_etter_fødsel_mødrekvote_grense).minusDays(1)); - assertThat(mangledeSøktmkPeriode.getAktiviteter().get(0).getTrekkdager()).isEqualTo( + assertThat(mangledeSøktmkPeriode.getAktiviteter().getFirst().getTrekkdager()).isEqualTo( new Trekkdager(veker_etter_fødsel_mødrekvote_grense * 5)); - assertThat(fpffPeriode.getAktiviteter().get(0).getTrekkdager()).isEqualTo(new Trekkdager(15)); + assertThat(fpffPeriode.getAktiviteter().getFirst().getTrekkdager()).isEqualTo(new Trekkdager(15)); } private UttakInput lagInput(Behandling behandling, @@ -311,7 +308,7 @@ void skalReturnereManuellBehandlingForPlanMedForTidligOppstartAvFedrekvote() { var uttakResultatPerioder = resultat.getPerioder(); assertThat(uttakResultatPerioder).hasSize(2); - var periode1 = uttakResultatPerioder.get(0); + var periode1 = uttakResultatPerioder.getFirst(); assertThat(periode1.getFom()).isEqualTo(startDato); assertThat(periode1.getTom()).isEqualTo(fødselsdato.plusWeeks(veker_etter_fødsel_mødrekvote_grense).minusDays(1)); assertThat(periode1.getResultatType()).isEqualTo(PeriodeResultatType.MANUELL_BEHANDLING); @@ -357,7 +354,7 @@ void skalReturnereManuellBehandlingForPlanMedForTidligOppstartAvFedrekvote() { var uttakResultatPerioder = resultat.getPerioder(); assertThat(uttakResultatPerioder).hasSize(3); - var periode1 = uttakResultatPerioder.get(0); + var periode1 = uttakResultatPerioder.getFirst(); assertThat(periode1.getFom()).isEqualTo(startDato); assertThat(periode1.getTom()).isEqualTo(fødselsdato.minusWeeks(2).minusDays(1)); assertThat(periode1.getResultatType()).isEqualTo(PeriodeResultatType.MANUELL_BEHANDLING); @@ -401,18 +398,17 @@ void skalReturnereManuellBehandlingForPlanMedForTidligOppstartAvFedrekvote() { // Assert var uttakResultatPerioder = resultat.getPerioder(); - var manuelleFellesPerioderStream = uttakResultatPerioder.stream() + var manuelleFellesPerioder = uttakResultatPerioder.stream() .filter(uttakResultatPeriode -> uttakResultatPeriode.getAktiviteter() - .get(0) + .getFirst() .getTrekkonto() .getKode() .equals(UttakPeriodeType.FELLESPERIODE.getKode()) && uttakResultatPeriode.getResultatType() - .equals(PeriodeResultatType.MANUELL_BEHANDLING)); - var manuelleFellesPerioder = manuelleFellesPerioderStream.collect( - Collectors.toList()); + .equals(PeriodeResultatType.MANUELL_BEHANDLING)) + .toList(); assertThat(manuelleFellesPerioder).hasSize(1); - var manuellFellesPeriode = manuelleFellesPerioder.get(0); + var manuellFellesPeriode = manuelleFellesPerioder.getFirst(); assertThat(manuellFellesPeriode.getFom()).isEqualTo(startDatoFellesperiode); assertThat(manuellFellesPeriode.getTom()).isEqualTo(sluttDatoFellesperiode); } @@ -452,11 +448,11 @@ void skalReturnereManuellBehandlingForPlanMedForTidligOppstartAvFedrekvote() { var manglendeSøktPeriodeMK = finnPeriode(uttakResultatPerioder, fødselsdato, fødselsdato.plusWeeks(6).minusDays(1)); - assertThat(tidligUttakFP.getAktiviteter().get(0).getTrekkonto()).isEqualTo(UttakPeriodeType.FELLESPERIODE); + assertThat(tidligUttakFP.getAktiviteter().getFirst().getTrekkonto()).isEqualTo(UttakPeriodeType.FELLESPERIODE); assertThat(tidligUttakFP.getResultatType()).isEqualTo(PeriodeResultatType.INNVILGET); - assertThat(manglendeSøktPeriodeFP.getAktiviteter().get(0).getTrekkonto()).isEqualTo(UttakPeriodeType.FELLESPERIODE); - assertThat(manglendeSøktPeriodeFPFF.getAktiviteter().get(0).getTrekkonto()).isEqualTo(UttakPeriodeType.FORELDREPENGER_FØR_FØDSEL); - assertThat(manglendeSøktPeriodeMK.getAktiviteter().get(0).getTrekkonto()).isEqualTo(UttakPeriodeType.MØDREKVOTE); + assertThat(manglendeSøktPeriodeFP.getAktiviteter().getFirst().getTrekkonto()).isEqualTo(UttakPeriodeType.FELLESPERIODE); + assertThat(manglendeSøktPeriodeFPFF.getAktiviteter().getFirst().getTrekkonto()).isEqualTo(UttakPeriodeType.FORELDREPENGER_FØR_FØDSEL); + assertThat(manglendeSøktPeriodeMK.getAktiviteter().getFirst().getTrekkonto()).isEqualTo(UttakPeriodeType.MØDREKVOTE); assertThat(manglendeSøktPeriodeMK.getResultatType()).isEqualTo(PeriodeResultatType.MANUELL_BEHANDLING); } @@ -509,17 +505,17 @@ void skalReturnereManuellBehandlingForPlanMedForTidligOppstartAvFedrekvote() { var manglendeSøktPeriodeMK = finnPeriode(uttakResultatPerioder, fødselsdato, fødselsdato.plusWeeks(6).minusDays(1)); - assertThat(tidligUttakFP1.getAktiviteter().get(0).getTrekkonto()).isEqualTo(UttakPeriodeType.FELLESPERIODE); + assertThat(tidligUttakFP1.getAktiviteter().getFirst().getTrekkonto()).isEqualTo(UttakPeriodeType.FELLESPERIODE); assertThat(tidligUttakFP1.getResultatType()).isEqualTo(PeriodeResultatType.INNVILGET); - assertThat(manglendeSøktPeriodeFP.getAktiviteter().get(0).getTrekkonto()).isEqualTo(UttakPeriodeType.FELLESPERIODE); + assertThat(manglendeSøktPeriodeFP.getAktiviteter().getFirst().getTrekkonto()).isEqualTo(UttakPeriodeType.FELLESPERIODE); assertThat(manglendeSøktPeriodeFP.getResultatType()).isEqualTo(PeriodeResultatType.MANUELL_BEHANDLING); - assertThat(tidligUttakFP2.getAktiviteter().get(0).getTrekkonto()).isEqualTo(UttakPeriodeType.FELLESPERIODE); + assertThat(tidligUttakFP2.getAktiviteter().getFirst().getTrekkonto()).isEqualTo(UttakPeriodeType.FELLESPERIODE); assertThat(tidligUttakFP2.getResultatType()).isEqualTo(PeriodeResultatType.INNVILGET); - assertThat(manglendeSøktPeriodeFPFF.getAktiviteter().get(0).getTrekkonto()).isEqualTo(UttakPeriodeType.FORELDREPENGER_FØR_FØDSEL); + assertThat(manglendeSøktPeriodeFPFF.getAktiviteter().getFirst().getTrekkonto()).isEqualTo(UttakPeriodeType.FORELDREPENGER_FØR_FØDSEL); assertThat(manglendeSøktPeriodeFPFF.getResultatType()).isEqualTo(PeriodeResultatType.AVSLÅTT); - assertThat(manglendeSøktPeriodeMK.getAktiviteter().get(0).getTrekkonto()).isEqualTo(UttakPeriodeType.MØDREKVOTE); + assertThat(manglendeSøktPeriodeMK.getAktiviteter().getFirst().getTrekkonto()).isEqualTo(UttakPeriodeType.MØDREKVOTE); assertThat(manglendeSøktPeriodeMK.getResultatType()).isEqualTo(PeriodeResultatType.MANUELL_BEHANDLING); } @@ -575,12 +571,12 @@ void skalReturnereManuellBehandlingForPlanMedForTidligOppstartAvFedrekvote() { var uttakResultatPerioder = resultat.getPerioder(); assertThat(uttakResultatPerioder).hasSize(5); - var periode = uttakResultatPerioder.get(0); - assertThat(periode.getAktiviteter().get(0).getTrekkonto()).isEqualTo(UttakPeriodeType.FORELDREPENGER_FØR_FØDSEL); + var periode = uttakResultatPerioder.getFirst(); + assertThat(periode.getAktiviteter().getFirst().getTrekkonto()).isEqualTo(UttakPeriodeType.FORELDREPENGER_FØR_FØDSEL); assertThat(periode.getResultatType()).isEqualTo(PeriodeResultatType.INNVILGET); periode = uttakResultatPerioder.get(1); - assertThat(periode.getAktiviteter().get(0).getTrekkonto()).isEqualTo(UttakPeriodeType.MØDREKVOTE); + assertThat(periode.getAktiviteter().getFirst().getTrekkonto()).isEqualTo(UttakPeriodeType.MØDREKVOTE); assertThat(periode.getResultatType()).isEqualTo(PeriodeResultatType.INNVILGET); // Oppholdsperioder for annen forelder skal returneres som en uttakResultatPeriode uten aktivitet @@ -592,13 +588,13 @@ void skalReturnereManuellBehandlingForPlanMedForTidligOppstartAvFedrekvote() { // Periode knukket pga ikke nok dager igjen på konto. periode = uttakResultatPerioder.get(3); - assertThat(periode.getAktiviteter().get(0).getTrekkonto()).isEqualTo(UttakPeriodeType.FELLESPERIODE); + assertThat(periode.getAktiviteter().getFirst().getTrekkonto()).isEqualTo(UttakPeriodeType.FELLESPERIODE); assertThat(periode.getResultatType()).isEqualTo(PeriodeResultatType.INNVILGET); assertThat(periode.getFom()).isEqualTo(fellesperiode.getFom()); assertThat(periode.getTom()).isEqualTo(sluttDatoMødrekvote.plusWeeks(16)); periode = uttakResultatPerioder.get(4); - assertThat(periode.getAktiviteter().get(0).getTrekkonto()).isEqualTo(UttakPeriodeType.FELLESPERIODE); + assertThat(periode.getAktiviteter().getFirst().getTrekkonto()).isEqualTo(UttakPeriodeType.FELLESPERIODE); assertThat(periode.getResultatType()).isEqualTo(PeriodeResultatType.MANUELL_BEHANDLING); assertThat(periode.getFom()).isEqualTo(sluttDatoMødrekvote.plusWeeks(16).plusDays(1)); assertThat(periode.getTom()).isEqualTo(fellesperiode.getTom()); @@ -650,12 +646,12 @@ void skalReturnereManuellBehandlingForPlanMedForTidligOppstartAvFedrekvote() { var uttakResultatPerioder = resultat.getPerioder(); assertThat(uttakResultatPerioder).hasSize(4); - var periode = uttakResultatPerioder.get(0); - assertThat(periode.getAktiviteter().get(0).getTrekkonto()).isEqualTo(UttakPeriodeType.FORELDREPENGER_FØR_FØDSEL); + var periode = uttakResultatPerioder.getFirst(); + assertThat(periode.getAktiviteter().getFirst().getTrekkonto()).isEqualTo(UttakPeriodeType.FORELDREPENGER_FØR_FØDSEL); assertThat(periode.getResultatType()).isEqualTo(PeriodeResultatType.INNVILGET); periode = uttakResultatPerioder.get(1); - assertThat(periode.getAktiviteter().get(0).getTrekkonto()).isEqualTo(UttakPeriodeType.MØDREKVOTE); + assertThat(periode.getAktiviteter().getFirst().getTrekkonto()).isEqualTo(UttakPeriodeType.MØDREKVOTE); assertThat(periode.getResultatType()).isEqualTo(PeriodeResultatType.INNVILGET); // Oppholdsperioder for annen forelder skal returneres som en uttakResultatPeriode uten aktivitet @@ -711,32 +707,32 @@ void skalReturnereManuellBehandlingForPlanMedForTidligOppstartAvFedrekvote() { var fedrekvotePerioder = uttakResultatPerioder.stream() .filter(uttakResultatPeriode -> uttakResultatPeriode.getAktiviteter() - .get(0) + .getFirst() .getTrekkonto() .getKode() .equals(StønadskontoType.FEDREKVOTE.getKode())) .toList(); assertThat(fedrekvotePerioder).hasSize(1); - assertThat(fedrekvotePerioder.get(0).getFom()).isEqualTo(startDatoFedrekvote); - assertThat(fedrekvotePerioder.get(0).getTom()).isEqualTo(sluttDatoFedrekvote); + assertThat(fedrekvotePerioder.getFirst().getFom()).isEqualTo(startDatoFedrekvote); + assertThat(fedrekvotePerioder.getFirst().getTom()).isEqualTo(sluttDatoFedrekvote); - assertThat(fedrekvotePerioder.get(0).getResultatType()).isEqualTo(PeriodeResultatType.MANUELL_BEHANDLING); - assertThat(fedrekvotePerioder.get(0).getResultatÅrsak().getKode()).isEqualTo("4007"); - assertThat(fedrekvotePerioder.get(0).getAktiviteter().get(0).getTrekkonto()).isEqualTo(UttakPeriodeType.FEDREKVOTE); + assertThat(fedrekvotePerioder.getFirst().getResultatType()).isEqualTo(PeriodeResultatType.MANUELL_BEHANDLING); + assertThat(fedrekvotePerioder.getFirst().getResultatÅrsak().getKode()).isEqualTo("4007"); + assertThat(fedrekvotePerioder.getFirst().getAktiviteter().getFirst().getTrekkonto()).isEqualTo(UttakPeriodeType.FEDREKVOTE); var mødrekvotePerioder = uttakResultatPerioder.stream() .filter(uttakResultatPeriode -> uttakResultatPeriode.getAktiviteter() - .get(0) + .getFirst() .getTrekkonto() .getKode() .equals(StønadskontoType.MØDREKVOTE.getKode())) .toList(); assertThat(mødrekvotePerioder).hasSize(1); - assertThat(mødrekvotePerioder.get(0).getFom()).isEqualTo(fødselsdato); - assertThat(mødrekvotePerioder.get(0).getTom()).isEqualTo(sluttDatoMødrekvote); - assertThat(mødrekvotePerioder.get(0).getResultatType()).isEqualTo(PeriodeResultatType.INNVILGET); - assertThat(mødrekvotePerioder.get(0).getResultatÅrsak().getKode()).isEqualTo("2003"); - assertThat(mødrekvotePerioder.get(0).getAktiviteter().get(0).getTrekkonto()).isEqualTo(UttakPeriodeType.MØDREKVOTE); + assertThat(mødrekvotePerioder.getFirst().getFom()).isEqualTo(fødselsdato); + assertThat(mødrekvotePerioder.getFirst().getTom()).isEqualTo(sluttDatoMødrekvote); + assertThat(mødrekvotePerioder.getFirst().getResultatType()).isEqualTo(PeriodeResultatType.INNVILGET); + assertThat(mødrekvotePerioder.getFirst().getResultatÅrsak().getKode()).isEqualTo("2003"); + assertThat(mødrekvotePerioder.getFirst().getAktiviteter().getFirst().getTrekkonto()).isEqualTo(UttakPeriodeType.MØDREKVOTE); } @Test @@ -831,9 +827,9 @@ void skalReturnereManuellBehandlingForPlanMedForTidligOppstartAvFedrekvote() { var resultat = fastsettePerioderRegelAdapter.fastsettePerioder(input, lagStønadskontoer()); assertThat(resultat.getPerioder()).hasSize(1); - assertThat(resultat.getPerioder().get(0).getUtsettelseType()).isEqualTo(UttakUtsettelseType.SYKDOM_SKADE); - assertThat(resultat.getPerioder().get(0).getResultatType()).isEqualTo(PeriodeResultatType.AVSLÅTT); - assertThat(resultat.getPerioder().get(0).getAktiviteter().get(0).getTrekkdager().decimalValue()).isZero(); + assertThat(resultat.getPerioder().getFirst().getUtsettelseType()).isEqualTo(UttakUtsettelseType.SYKDOM_SKADE); + assertThat(resultat.getPerioder().getFirst().getResultatType()).isEqualTo(PeriodeResultatType.AVSLÅTT); + assertThat(resultat.getPerioder().getFirst().getAktiviteter().getFirst().getTrekkdager().decimalValue()).isZero(); } private InntektArbeidYtelseGrunnlag tomIay() { @@ -864,10 +860,10 @@ private InntektArbeidYtelseGrunnlag tomIay() { var uttakResultatPerioder = resultat.getPerioder(); assertThat(uttakResultatPerioder).hasSize(1); - var periode = uttakResultatPerioder.iterator().next(); + var periode = uttakResultatPerioder.getFirst(); assertThat(periode.getResultatType()).isEqualTo(PeriodeResultatType.AVSLÅTT); assertThat(periode.getAktiviteter()).hasSize(1); - var aktivitet = periode.getAktiviteter().get(0); + var aktivitet = periode.getAktiviteter().getFirst(); assertThat(aktivitet.getTrekkdager()).isEqualTo(new Trekkdager(5 * 5)); assertThat(aktivitet.getTrekkonto()).isEqualTo(UttakPeriodeType.FEDREKVOTE); assertThat(aktivitet.getUtbetalingsgrad()).isEqualTo(Utbetalingsgrad.ZERO); @@ -901,9 +897,9 @@ private InntektArbeidYtelseGrunnlag tomIay() { assertThat(uttakResultatPerioder).hasSize(1); var fkPeriode = finnPeriode(uttakResultatPerioder, start, slutt); - assertThat(fkPeriode.getAktiviteter().get(0).getTrekkonto()).isEqualTo(UttakPeriodeType.FEDREKVOTE); - assertThat(fkPeriode.getAktiviteter().get(0).getTrekkdager()).isEqualTo(new Trekkdager((int) (5 * 5 * 0.4))); - assertThat(fkPeriode.getAktiviteter().get(0).getArbeidsprosent()).isEqualTo(arbeidsprosent); + assertThat(fkPeriode.getAktiviteter().getFirst().getTrekkonto()).isEqualTo(UttakPeriodeType.FEDREKVOTE); + assertThat(fkPeriode.getAktiviteter().getFirst().getTrekkdager()).isEqualTo(new Trekkdager((int) (5 * 5 * 0.4))); + assertThat(fkPeriode.getAktiviteter().getFirst().getArbeidsprosent()).isEqualTo(arbeidsprosent); } @Test @@ -1126,10 +1122,10 @@ private UttakResultatPeriodeAktivitetEntitet aktivitetForArbeidsgiver(List p.getAktiviteter().get(0).getTrekkonto().equals(UttakPeriodeType.FORELDREPENGER_FØR_FØDSEL)) + .filter(p -> p.getAktiviteter().getFirst().getTrekkonto().equals(UttakPeriodeType.FORELDREPENGER_FØR_FØDSEL)) .findFirst() .get(); var mkPeriode = uttakResultatPerioder.stream() - .filter(p -> p.getAktiviteter().get(0).getTrekkonto().equals(UttakPeriodeType.MØDREKVOTE)) + .filter(p -> p.getAktiviteter().getFirst().getTrekkonto().equals(UttakPeriodeType.MØDREKVOTE)) .findFirst() .get(); // Innvilget FPFF assertThat(fpffPeriode.getResultatType()).isEqualTo(PeriodeResultatType.INNVILGET); - assertThat(fpffPeriode.getAktiviteter().get(0).getTrekkdager()).isEqualTo(new Trekkdager(15)); + assertThat(fpffPeriode.getAktiviteter().getFirst().getTrekkdager()).isEqualTo(new Trekkdager(15)); // Innvilget mødrekvote, men avslag på gradering. assertThat(mkPeriode.getResultatType()).isEqualTo(PeriodeResultatType.INNVILGET); assertThat(mkPeriode.getGraderingAvslagÅrsak()).isEqualTo(GraderingAvslagÅrsak.GRADERING_FØR_UKE_7); - assertThat(mkPeriode.getAktiviteter().get(0).getTrekkdager()).isEqualTo( + assertThat(mkPeriode.getAktiviteter().getFirst().getTrekkdager()).isEqualTo( new Trekkdager(new SimpleLocalDateInterval(fødselsdato, fødselsdato.plusWeeks(6).minusDays(1)).antallArbeidsdager())); } @@ -1361,11 +1357,11 @@ void samtidigUttaksprosentSkalHa2Desimaler() { var resultatRevurdering = fastsettePerioderRegelAdapter.fastsettePerioder(input, lagStønadskontoer()); assertThat(resultatRevurdering.getPerioder()).hasSize(3); - assertThat(resultatRevurdering.getPerioder().get(0).getAktiviteter()).hasSize(1); + assertThat(resultatRevurdering.getPerioder().getFirst().getAktiviteter()).hasSize(1); assertThat(resultatRevurdering.getPerioder().get(1).getAktiviteter()).hasSize(1); assertThat(resultatRevurdering.getPerioder().get(2).getAktiviteter()).hasSize(1); - assertThat(resultatRevurdering.getPerioder().get(0).getFom()).isEqualTo(opprinneligFpff.getFom()); - assertThat(resultatRevurdering.getPerioder().get(0).getTom()).isEqualTo(opprinneligFpff.getTom()); + assertThat(resultatRevurdering.getPerioder().getFirst().getFom()).isEqualTo(opprinneligFpff.getFom()); + assertThat(resultatRevurdering.getPerioder().getFirst().getTom()).isEqualTo(opprinneligFpff.getTom()); assertThat(resultatRevurdering.getPerioder().get(1).getFom()).isEqualTo(opprinneligMødrekvote.getFom()); assertThat(resultatRevurdering.getPerioder().get(1).getTom()).isEqualTo( revurderingSøknadsperiodeFellesperiode.getFom().minusDays(1)); @@ -1373,23 +1369,23 @@ void samtidigUttaksprosentSkalHa2Desimaler() { revurderingSøknadsperiodeFellesperiode.getFom()); assertThat(resultatRevurdering.getPerioder().get(2).getTom()).isEqualTo( revurderingSøknadsperiodeFellesperiode.getTom()); - assertThat(resultatRevurdering.getPerioder().get(0).getAktiviteter().get(0).getTrekkdager()).isEqualTo( - opprinneligFpff.getAktiviteter().get(0).getTrekkdager()); - assertThat(resultatRevurdering.getPerioder().get(0).getAktiviteter().get(0).getUttakAktivitet()).isEqualTo( - opprinneligFpff.getAktiviteter().get(0).getUttakAktivitet()); - assertThat(resultatRevurdering.getPerioder().get(0).getAktiviteter().get(0).getArbeidsprosent()).isEqualTo( - opprinneligFpff.getAktiviteter().get(0).getArbeidsprosent()); - assertThat(resultatRevurdering.getPerioder().get(0).getAktiviteter().get(0).getTrekkonto()).isEqualTo( - opprinneligFpff.getAktiviteter().get(0).getTrekkonto()); - assertThat(resultatRevurdering.getPerioder().get(1).getAktiviteter().get(0).getTrekkdager()).isEqualTo( + assertThat(resultatRevurdering.getPerioder().getFirst().getAktiviteter().getFirst().getTrekkdager()).isEqualTo( + opprinneligFpff.getAktiviteter().getFirst().getTrekkdager()); + assertThat(resultatRevurdering.getPerioder().getFirst().getAktiviteter().getFirst().getUttakAktivitet()).isEqualTo( + opprinneligFpff.getAktiviteter().getFirst().getUttakAktivitet()); + assertThat(resultatRevurdering.getPerioder().getFirst().getAktiviteter().getFirst().getArbeidsprosent()).isEqualTo( + opprinneligFpff.getAktiviteter().getFirst().getArbeidsprosent()); + assertThat(resultatRevurdering.getPerioder().getFirst().getAktiviteter().getFirst().getTrekkonto()).isEqualTo( + opprinneligFpff.getAktiviteter().getFirst().getTrekkonto()); + assertThat(resultatRevurdering.getPerioder().get(1).getAktiviteter().getFirst().getTrekkdager()).isEqualTo( new Trekkdager(new SimpleLocalDateInterval(opprinneligMødrekvote.getFom(), revurderingSøknadsperiodeFellesperiode.getFom().minusDays(1)).antallArbeidsdager())); - assertThat(resultatRevurdering.getPerioder().get(1).getAktiviteter().get(0).getTrekkonto()).isEqualTo( - opprinneligMødrekvote.getAktiviteter().get(0).getTrekkonto()); - assertThat(resultatRevurdering.getPerioder().get(2).getAktiviteter().get(0).getTrekkdager()).isEqualTo( + assertThat(resultatRevurdering.getPerioder().get(1).getAktiviteter().getFirst().getTrekkonto()).isEqualTo( + opprinneligMødrekvote.getAktiviteter().getFirst().getTrekkonto()); + assertThat(resultatRevurdering.getPerioder().get(2).getAktiviteter().getFirst().getTrekkdager()).isEqualTo( new Trekkdager(new SimpleLocalDateInterval(revurderingSøknadsperiodeFellesperiode.getFom(), revurderingSøknadsperiodeFellesperiode.getTom()).antallArbeidsdager())); - assertThat(resultatRevurdering.getPerioder().get(2).getAktiviteter().get(0).getTrekkonto()).isEqualTo(UttakPeriodeType.FELLESPERIODE); + assertThat(resultatRevurdering.getPerioder().get(2).getAktiviteter().getFirst().getTrekkonto()).isEqualTo(UttakPeriodeType.FELLESPERIODE); } @@ -1469,8 +1465,8 @@ void arbeidstaker_uten_arbeidsgiver_skal_fastsettes() { periodeMedSamtidigUttak.getTom()); assertThat(fkPeriode.getSamtidigUttaksprosent()).isEqualTo(new SamtidigUttaksprosent(100)); - assertThat(fkPeriode.getAktiviteter().get(0).getUtbetalingsgrad()).isEqualTo(new Utbetalingsgrad(100)); - assertThat(fkPeriode.getAktiviteter().get(0).getTrekkdager()).isEqualTo(new Trekkdager(5)); + assertThat(fkPeriode.getAktiviteter().getFirst().getUtbetalingsgrad()).isEqualTo(new Utbetalingsgrad(100)); + assertThat(fkPeriode.getAktiviteter().getFirst().getTrekkdager()).isEqualTo(new Trekkdager(5)); } @Test @@ -1525,7 +1521,7 @@ void arbeidstaker_uten_arbeidsgiver_skal_fastsettes() { assertThat(utsettelseResultat.getUtsettelseType()).isEqualTo(UttakUtsettelseType.ARBEID); assertThat(utsettelseResultat.getManuellBehandlingÅrsak()).isEqualTo(ManuellBehandlingÅrsak.IKKE_HELTIDSARBEID); - assertThat(utsettelseResultat.getAktiviteter().get(0).getArbeidsprosent()).isEqualTo(BigDecimal.valueOf(50)); + assertThat(utsettelseResultat.getAktiviteter().getFirst().getArbeidsprosent()).isEqualTo(BigDecimal.valueOf(50)); } @Test @@ -1682,8 +1678,8 @@ private YrkesaktivitetBuilder lagYrkesaktivitet(Arbeidsgiver arbeidsgiver, Local var mødrekvote = finnPeriode(uttakResultatPerioder, oppgittPeriode2.getFom(), oppgittPeriode2.getTom()); - assertThat(mødrekvote.getAktiviteter().get(0).getArbeidsprosent()).isEqualTo(BigDecimal.ZERO); - assertThat(mødrekvote.getAktiviteter().get(0).getUtbetalingsgrad()).isEqualTo(new Utbetalingsgrad(100.00)); + assertThat(mødrekvote.getAktiviteter().getFirst().getArbeidsprosent()).isEqualTo(BigDecimal.ZERO); + assertThat(mødrekvote.getAktiviteter().getFirst().getUtbetalingsgrad()).isEqualTo(new Utbetalingsgrad(100.00)); } @Test @@ -1745,8 +1741,8 @@ private YrkesaktivitetBuilder lagYrkesaktivitet(Arbeidsgiver arbeidsgiver, Local var mødrekvote = finnPeriode(uttakResultatPerioder, oppgittPeriode2.getFom(), oppgittPeriode2.getTom()); - assertThat(mødrekvote.getAktiviteter().get(0).getArbeidsprosent()).isEqualTo(BigDecimal.ZERO); - assertThat(mødrekvote.getAktiviteter().get(0).getUtbetalingsgrad()).isEqualTo(new Utbetalingsgrad(100.00)); + assertThat(mødrekvote.getAktiviteter().getFirst().getArbeidsprosent()).isEqualTo(BigDecimal.ZERO); + assertThat(mødrekvote.getAktiviteter().getFirst().getUtbetalingsgrad()).isEqualTo(new Utbetalingsgrad(100.00)); } private static UttakResultatPeriodeEntitet finnPeriode(List perioder, diff --git a/domenetjenester/uttak/src/test/java/no/nav/foreldrepenger/domene/uttak/fastsetteperioder/FastsettePerioderRegelGrunnlagByggerTest.java b/domenetjenester/uttak/src/test/java/no/nav/foreldrepenger/domene/uttak/fastsetteperioder/FastsettePerioderRegelGrunnlagByggerTest.java index 9a8fc0dde87..b55d00545ec 100644 --- a/domenetjenester/uttak/src/test/java/no/nav/foreldrepenger/domene/uttak/fastsetteperioder/FastsettePerioderRegelGrunnlagByggerTest.java +++ b/domenetjenester/uttak/src/test/java/no/nav/foreldrepenger/domene/uttak/fastsetteperioder/FastsettePerioderRegelGrunnlagByggerTest.java @@ -56,7 +56,6 @@ import no.nav.foreldrepenger.domene.uttak.fastsetteperioder.grunnlagbyggere.BehandlingGrunnlagBygger; import no.nav.foreldrepenger.domene.uttak.fastsetteperioder.grunnlagbyggere.DatoerGrunnlagBygger; import no.nav.foreldrepenger.domene.uttak.fastsetteperioder.grunnlagbyggere.InngangsvilkårGrunnlagBygger; -import no.nav.foreldrepenger.domene.uttak.fastsetteperioder.grunnlagbyggere.KontoerGrunnlagBygger; import no.nav.foreldrepenger.domene.uttak.fastsetteperioder.grunnlagbyggere.MedlemskapGrunnlagBygger; import no.nav.foreldrepenger.domene.uttak.fastsetteperioder.grunnlagbyggere.OpptjeningGrunnlagBygger; import no.nav.foreldrepenger.domene.uttak.fastsetteperioder.grunnlagbyggere.RettOgOmsorgGrunnlagBygger; @@ -103,8 +102,7 @@ void setUp() { repositoryProvider.getFpUttakRepository()), new SøknadGrunnlagBygger(repositoryProvider.getYtelsesFordelingRepository()), new InngangsvilkårGrunnlagBygger(repositoryProvider), new OpptjeningGrunnlagBygger(), - new AdopsjonGrunnlagBygger(), new KontoerGrunnlagBygger(), - new YtelserGrunnlagBygger()); + new AdopsjonGrunnlagBygger(), new YtelserGrunnlagBygger()); } private Behandling lagre(AbstractTestScenario scenario) { @@ -139,7 +137,7 @@ private Behandling lagre(AbstractTestScenario scenario) { var grunnlag = grunnlagBygger.byggGrunnlag(input, lagStønadskontoer()); assertThat(grunnlag.getSøknad().getOppgittePerioder()).hasSize(1); - var oppgittPeriodeIGrunnlag = grunnlag.getSøknad().getOppgittePerioder().get(0); + var oppgittPeriodeIGrunnlag = grunnlag.getSøknad().getOppgittePerioder().getFirst(); assertThat(oppgittPeriodeIGrunnlag.getArbeidsprosent()).isEqualTo(arbeidsprosentFraSøknad); } @@ -239,10 +237,10 @@ private Optional finnGradertUttakPeriode(BigDecimal arbeidsprose var grunnlag = grunnlagBygger.byggGrunnlag(input, lagStønadskontoer()); assertThat(grunnlag.getSøknad().getOppgittePerioder()).hasSize(1); - assertThat(grunnlag.getSøknad().getOppgittePerioder().get(0).getArbeidsprosent()).isEqualTo(arbeidsprosent); + assertThat(grunnlag.getSøknad().getOppgittePerioder().getFirst().getArbeidsprosent()).isEqualTo(arbeidsprosent); assertThat(grunnlag.getSøknad() .getOppgittePerioder() - .get(0) + .getFirst() .getGradertAktiviteter() .stream() .findFirst() @@ -289,11 +287,11 @@ void gradertArbeidsforholdArbeidstakerMedSamtUgradertFrilansSkalBareGiEnGradertA var grunnlag = grunnlagBygger.byggGrunnlag(input, lagStønadskontoer()); assertThat(grunnlag.getSøknad().getOppgittePerioder()).hasSize(1); - assertThat(grunnlag.getSøknad().getOppgittePerioder().get(0).getArbeidsprosent()).isEqualTo(arbeidsprosent); - assertThat(grunnlag.getSøknad().getOppgittePerioder().get(0).getGradertAktiviteter()).hasSize(1); + assertThat(grunnlag.getSøknad().getOppgittePerioder().getFirst().getArbeidsprosent()).isEqualTo(arbeidsprosent); + assertThat(grunnlag.getSøknad().getOppgittePerioder().getFirst().getGradertAktiviteter()).hasSize(1); var aktivitetIdentifikator1 = grunnlag.getSøknad() .getOppgittePerioder() - .get(0) + .getFirst() .getGradertAktiviteter() .stream() .findFirst() @@ -414,7 +412,7 @@ void mapperAnnenPartsUttaksperioder() { var uttakPerioderAnnenPart = grunnlag.getAnnenPart().getUttaksperioder(); assertThat(uttakPerioderAnnenPart).hasSize(3); - var annenPartGrunnlag = uttakPerioderAnnenPart.get(0); + var annenPartGrunnlag = uttakPerioderAnnenPart.getFirst(); assertThat(annenPartGrunnlag.getFom()).isEqualTo(uttakMødrekvote.getFom()); assertThat(annenPartGrunnlag.getTom()).isEqualTo(uttakMødrekvote.getTom()); assertThat(annenPartGrunnlag.getAktiviteter()).hasSize(1); @@ -423,7 +421,7 @@ void mapperAnnenPartsUttaksperioder() { assertThat(aktivitetMødrekvote.getAktivitetIdentifikator()).isEqualTo(forventetAktivitetIdentifikator); assertThat(aktivitetMødrekvote.getStønadskontotype()).isEqualTo(Stønadskontotype.MØDREKVOTE); assertThat(new Trekkdager(aktivitetMødrekvote.getTrekkdager().decimalValue())).isEqualTo( - uttakMødrekvote.getAktiviteter().get(0).getTrekkdager()); + uttakMødrekvote.getAktiviteter().getFirst().getTrekkdager()); var mappedFellesperiode = uttakPerioderAnnenPart.get(1); assertThat(mappedFellesperiode.getFom()).isEqualTo(uttakFellesperiode.getFom()); @@ -434,7 +432,7 @@ void mapperAnnenPartsUttaksperioder() { assertThat(aktivitetFellesperiode.getAktivitetIdentifikator()).isEqualTo(forventetAktivitetIdentifikator); assertThat(aktivitetFellesperiode.getStønadskontotype()).isEqualTo(Stønadskontotype.FELLESPERIODE); assertThat(new Trekkdager(aktivitetFellesperiode.getTrekkdager().decimalValue())).isEqualTo( - uttakFellesperiode.getAktiviteter().get(0).getTrekkdager()); + uttakFellesperiode.getAktiviteter().getFirst().getTrekkdager()); } @Test @@ -471,7 +469,7 @@ void testeSamtidigUttak() { var oppittePerioder = grunnlag.getSøknad().getOppgittePerioder(); assertThat(oppittePerioder).isNotEmpty(); - assertThat(oppittePerioder.get(0).getSamtidigUttaksprosent().decimalValue()).isEqualTo( + assertThat(oppittePerioder.getFirst().getSamtidigUttaksprosent().decimalValue()).isEqualTo( SamtidigUttaksprosent.TEN.decimalValue()); } diff --git a/domenetjenester/uttak/src/test/java/no/nav/foreldrepenger/domene/uttak/fastsetteperioder/FastsettePerioderTjenesteTest.java b/domenetjenester/uttak/src/test/java/no/nav/foreldrepenger/domene/uttak/fastsetteperioder/FastsettePerioderTjenesteTest.java index 11f67d8fae0..ef13f844333 100644 --- a/domenetjenester/uttak/src/test/java/no/nav/foreldrepenger/domene/uttak/fastsetteperioder/FastsettePerioderTjenesteTest.java +++ b/domenetjenester/uttak/src/test/java/no/nav/foreldrepenger/domene/uttak/fastsetteperioder/FastsettePerioderTjenesteTest.java @@ -20,7 +20,6 @@ import org.junit.jupiter.api.Test; import no.nav.foreldrepenger.behandling.BehandlingReferanse; -import no.nav.foreldrepenger.behandling.FagsakRelasjonTjeneste; import no.nav.foreldrepenger.behandling.Skjæringstidspunkt; import no.nav.foreldrepenger.behandlingslager.behandling.Behandling; import no.nav.foreldrepenger.behandlingslager.behandling.ytelsefordeling.AvklarteUttakDatoerEntitet; @@ -73,7 +72,6 @@ import no.nav.foreldrepenger.domene.uttak.fastsetteperioder.grunnlagbyggere.BehandlingGrunnlagBygger; import no.nav.foreldrepenger.domene.uttak.fastsetteperioder.grunnlagbyggere.DatoerGrunnlagBygger; import no.nav.foreldrepenger.domene.uttak.fastsetteperioder.grunnlagbyggere.InngangsvilkårGrunnlagBygger; -import no.nav.foreldrepenger.domene.uttak.fastsetteperioder.grunnlagbyggere.KontoerGrunnlagBygger; import no.nav.foreldrepenger.domene.uttak.fastsetteperioder.grunnlagbyggere.MedlemskapGrunnlagBygger; import no.nav.foreldrepenger.domene.uttak.fastsetteperioder.grunnlagbyggere.OpptjeningGrunnlagBygger; import no.nav.foreldrepenger.domene.uttak.fastsetteperioder.grunnlagbyggere.RettOgOmsorgGrunnlagBygger; @@ -99,8 +97,6 @@ class FastsettePerioderTjenesteTest { private final YtelsesFordelingRepository ytelsesFordelingRepository = repositoryProvider.getYtelsesFordelingRepository(); - private final FagsakRelasjonTjeneste fagsakRelasjonTjeneste; - private final FpUttakRepository fpUttakRepository = repositoryProvider.getFpUttakRepository(); private final UttaksperiodegrenseRepository uttaksperiodegrenseRepository = repositoryProvider.getUttaksperiodegrenseRepository(); @@ -110,7 +106,6 @@ class FastsettePerioderTjenesteTest { { var rettOgOmsorgGrunnlagBygger = new RettOgOmsorgGrunnlagBygger(repositoryProvider, new ForeldrepengerUttakTjeneste(repositoryProvider.getFpUttakRepository())); - fagsakRelasjonTjeneste = new FagsakRelasjonTjeneste(repositoryProvider.getFagsakRepository(), repositoryProvider.getFagsakRelasjonRepository()); regelAdapter = new FastsettePerioderRegelAdapter( new FastsettePerioderRegelGrunnlagBygger(new AnnenPartGrunnlagBygger(repositoryProvider.getFpUttakRepository()), new ArbeidGrunnlagBygger(repositoryProvider), new BehandlingGrunnlagBygger(), @@ -119,8 +114,7 @@ class FastsettePerioderTjenesteTest { repositoryProvider.getFpUttakRepository()), new SøknadGrunnlagBygger(repositoryProvider.getYtelsesFordelingRepository()), new InngangsvilkårGrunnlagBygger(repositoryProvider), new OpptjeningGrunnlagBygger(), - new AdopsjonGrunnlagBygger(), new KontoerGrunnlagBygger(), - new YtelserGrunnlagBygger()), + new AdopsjonGrunnlagBygger(), new YtelserGrunnlagBygger()), new FastsettePerioderRegelResultatKonverterer(fpUttakRepository, ytelsesFordelingRepository)); } @@ -173,8 +167,8 @@ void skalInnvilgeFedrekvoteForMedmor() { .getPerioder(); assertThat(uttakResultatPerioder).hasSize(1); - var resultatPeriode = uttakResultatPerioder.iterator().next(); - assertThat(resultatPeriode.getAktiviteter().get(0).getTrekkonto()).isEqualTo(UttakPeriodeType.FEDREKVOTE); + var resultatPeriode = uttakResultatPerioder.getFirst(); + assertThat(resultatPeriode.getAktiviteter().getFirst().getTrekkonto()).isEqualTo(UttakPeriodeType.FEDREKVOTE); assertThat(resultatPeriode.getResultatType()).isEqualTo(PeriodeResultatType.INNVILGET); } @@ -292,8 +286,8 @@ void arbeidstidsprosentOgUtbetalingsgradSkalHa2Desimaler() { var uttakResultatPerioder = uttakResultat.get() .getOpprinneligPerioder() .getPerioder(); - assertThat(uttakResultatPerioder.get(3).getAktiviteter().get(0).getArbeidsprosent()).isEqualTo(arbeidsprosent); - assertThat(uttakResultatPerioder.get(3).getAktiviteter().get(0).getUtbetalingsgrad()).isEqualTo( + assertThat(uttakResultatPerioder.get(3).getAktiviteter().getFirst().getArbeidsprosent()).isEqualTo(arbeidsprosent); + assertThat(uttakResultatPerioder.get(3).getAktiviteter().getFirst().getUtbetalingsgrad()).isEqualTo( new Utbetalingsgrad(49.45)); } @@ -328,7 +322,7 @@ void skal_filtrere_ut_fri_utsettelse() { var resultat = fpUttakRepository.hentUttakResultat(input.getBehandlingReferanse().behandlingId()); assertThat(resultat.getGjeldendePerioder().getPerioder()).hasSize(2); - assertThat(resultat.getGjeldendePerioder().getPerioder().get(0).isUtsettelse()).isFalse(); + assertThat(resultat.getGjeldendePerioder().getPerioder().getFirst().isUtsettelse()).isFalse(); assertThat(resultat.getGjeldendePerioder().getPerioder().get(1).isUtsettelse()).isFalse(); } @@ -371,9 +365,9 @@ void skal_filtrere_ut_fri_utsettelse() { var resultat = fpUttakRepository.hentUttakResultat(input.getBehandlingReferanse().behandlingId()); assertThat(resultat.getGjeldendePerioder().getPerioder()).hasSize(4); - assertThat(resultat.getGjeldendePerioder().getPerioder().get(0).isUtsettelse()).isTrue(); + assertThat(resultat.getGjeldendePerioder().getPerioder().getFirst().isUtsettelse()).isTrue(); //Innvilget pga oppfylt aktkrav - assertThat(resultat.getGjeldendePerioder().getPerioder().get(0).isInnvilget()).isTrue(); + assertThat(resultat.getGjeldendePerioder().getPerioder().getFirst().isInnvilget()).isTrue(); assertThat(resultat.getGjeldendePerioder().getPerioder().get(2).isUtsettelse()).isTrue(); //Avslag pga ikke oppfylt aktkrav assertThat(resultat.getGjeldendePerioder().getPerioder().get(2).isInnvilget()).isFalse(); @@ -419,7 +413,7 @@ private OppgittRettighetEntitet bareFarHarRett() { var mødrekvote = uttakResultatPerioder.stream() .filter( - p -> StønadskontoType.MØDREKVOTE.getKode().equals(p.getAktiviteter().get(0).getTrekkonto().getKode())) + p -> StønadskontoType.MØDREKVOTE.getKode().equals(p.getAktiviteter().getFirst().getTrekkonto().getKode())) .findFirst(); assertThat(mødrekvote).isPresent(); @@ -441,7 +435,7 @@ private OppgittRettighetEntitet bareFarHarRett() { var foreldrepengerPeriode = resultat.stream() .filter(p -> StønadskontoType.FORELDREPENGER.getKode() - .equals(p.getAktiviteter().get(0).getTrekkonto().getKode())) + .equals(p.getAktiviteter().getFirst().getTrekkonto().getKode())) .findFirst(); assertThat(foreldrepengerPeriode).isPresent(); } @@ -558,9 +552,9 @@ private OppgittRettighetEntitet bareFarHarRett() { var uttakResultat = fpUttakRepository.hentUttakResultatHvisEksisterer( behandling.getId()); var uttakAktivitetVirksomhet = aktivitetMedArbeidsgiverIPeriode(virksomhet, - uttakResultat.get().getGjeldendePerioder().getPerioder().get(0)); + uttakResultat.get().getGjeldendePerioder().getPerioder().getFirst()); var uttakAktivitetPerson = aktivitetMedArbeidsgiverIPeriode(person, - uttakResultat.get().getGjeldendePerioder().getPerioder().get(0)); + uttakResultat.get().getGjeldendePerioder().getPerioder().getFirst()); assertThat(uttakAktivitetVirksomhet).isPresent(); assertThat(uttakAktivitetPerson).isPresent(); assertThat(uttakAktivitetVirksomhet.get()).isNotEqualTo(uttakAktivitetPerson.get()); @@ -612,7 +606,7 @@ private OppgittRettighetEntitet bareFarHarRett() { var resultat = fpUttakRepository.hentUttakResultat(input.getBehandlingReferanse().behandlingId()); assertThat(resultat.getGjeldendePerioder().getPerioder()).hasSize(3); - var aktiviteterPeriode1 = resultat.getGjeldendePerioder().getPerioder().get(0).getAktiviteter(); + var aktiviteterPeriode1 = resultat.getGjeldendePerioder().getPerioder().getFirst().getAktiviteter(); var aktiviteterPeriode2 = resultat.getGjeldendePerioder().getPerioder().get(1).getAktiviteter(); //Knekker mødrekvote på startdato på arbeidsforholdet var aktiviteterPeriode3 = resultat.getGjeldendePerioder().getPerioder().get(2).getAktiviteter(); @@ -730,8 +724,8 @@ void overstyrtSkalLeggesTilOpprinnelig() { .sorted(Comparator.comparing(UttakResultatPeriodeEntitet::getTom)) .toList(); assertThat(overstyrtePerioder).hasSize(3); - assertThat(overstyrtePerioder.get(0).getFom()).isEqualTo(fødselsdato); - assertThat(overstyrtePerioder.get(0).getTom()).isEqualTo(opprinneligMødreKvoteSlutt); + assertThat(overstyrtePerioder.getFirst().getFom()).isEqualTo(fødselsdato); + assertThat(overstyrtePerioder.getFirst().getTom()).isEqualTo(opprinneligMødreKvoteSlutt); assertThat(overstyrtePerioder.get(1).getFom()).isEqualTo(opprinneligMødreKvoteSlutt.plusDays(1)); assertThat(overstyrtePerioder.get(1).getTom()).isEqualTo(opprinneligFellesPeriodeSlutt.minusWeeks(2)); assertThat(overstyrtePerioder.get(2).getFom()).isEqualTo( diff --git a/domenetjenester/uttak/src/test/java/no/nav/foreldrepenger/domene/uttak/saldo/MaksDatoUttakTjenesteImplTest.java b/domenetjenester/uttak/src/test/java/no/nav/foreldrepenger/domene/uttak/saldo/MaksDatoUttakTjenesteImplTest.java index d8d7fa8ef41..d10b1ec050b 100644 --- a/domenetjenester/uttak/src/test/java/no/nav/foreldrepenger/domene/uttak/saldo/MaksDatoUttakTjenesteImplTest.java +++ b/domenetjenester/uttak/src/test/java/no/nav/foreldrepenger/domene/uttak/saldo/MaksDatoUttakTjenesteImplTest.java @@ -28,7 +28,6 @@ import no.nav.foreldrepenger.domene.uttak.ForeldrepengerUttakTjeneste; import no.nav.foreldrepenger.domene.uttak.UttakRepositoryProvider; import no.nav.foreldrepenger.domene.uttak.beregnkontoer.UtregnetStønadskontoTjeneste; -import no.nav.foreldrepenger.domene.uttak.fastsetteperioder.grunnlagbyggere.KontoerGrunnlagBygger; import no.nav.foreldrepenger.domene.uttak.input.FamilieHendelse; import no.nav.foreldrepenger.domene.uttak.input.FamilieHendelser; import no.nav.foreldrepenger.domene.uttak.input.ForeldrepengerGrunnlag; @@ -45,10 +44,9 @@ class MaksDatoUttakTjenesteImplTest { { var fagsakRelasjonTjeneste = new FagsakRelasjonTjeneste(repositoryProvider.getFagsakRepository(), repositoryProvider.getFagsakRelasjonRepository()); - var kontoerGrunnlagBygger = new KontoerGrunnlagBygger(); var uttakTjeneste = new ForeldrepengerUttakTjeneste(repositoryProvider.getFpUttakRepository()); var utregnetTjeneste = new UtregnetStønadskontoTjeneste(fagsakRelasjonTjeneste, uttakTjeneste); - var stønadskontoSaldoTjeneste = new StønadskontoSaldoTjeneste(repositoryProvider, kontoerGrunnlagBygger, utregnetTjeneste); + var stønadskontoSaldoTjeneste = new StønadskontoSaldoTjeneste(repositoryProvider, utregnetTjeneste); maksDatoUttakTjeneste = new MaksDatoUttakTjenesteImpl( repositoryProvider.getFpUttakRepository(), stønadskontoSaldoTjeneste); } diff --git "a/domenetjenester/uttak/src/test/java/no/nav/foreldrepenger/domene/uttak/saldo/St\303\270nadskontoSaldoTjenesteTest.java" "b/domenetjenester/uttak/src/test/java/no/nav/foreldrepenger/domene/uttak/saldo/St\303\270nadskontoSaldoTjenesteTest.java" index b7a27e0d284..91118ac1892 100644 --- "a/domenetjenester/uttak/src/test/java/no/nav/foreldrepenger/domene/uttak/saldo/St\303\270nadskontoSaldoTjenesteTest.java" +++ "b/domenetjenester/uttak/src/test/java/no/nav/foreldrepenger/domene/uttak/saldo/St\303\270nadskontoSaldoTjenesteTest.java" @@ -34,7 +34,6 @@ import no.nav.foreldrepenger.domene.uttak.ForeldrepengerUttakTjeneste; import no.nav.foreldrepenger.domene.uttak.UttakRepositoryProvider; import no.nav.foreldrepenger.domene.uttak.beregnkontoer.UtregnetStønadskontoTjeneste; -import no.nav.foreldrepenger.domene.uttak.fastsetteperioder.grunnlagbyggere.KontoerGrunnlagBygger; import no.nav.foreldrepenger.domene.uttak.input.FamilieHendelse; import no.nav.foreldrepenger.domene.uttak.input.FamilieHendelser; import no.nav.foreldrepenger.domene.uttak.input.ForeldrepengerGrunnlag; @@ -281,7 +280,7 @@ private Behandling behandlingMedKonto(AktørId aktørId) { var fagsakRelasjonTjeneste = new FagsakRelasjonTjeneste(repositoryProvider.getFagsakRepository(), repositoryProvider.getFagsakRelasjonRepository()); var uttakTjeneste = new ForeldrepengerUttakTjeneste(repositoryProvider.getFpUttakRepository()); var utregnetTjeneste = new UtregnetStønadskontoTjeneste(fagsakRelasjonTjeneste, uttakTjeneste); - return new StønadskontoSaldoTjeneste(repositoryProvider, new KontoerGrunnlagBygger(), utregnetTjeneste); + return new StønadskontoSaldoTjeneste(repositoryProvider, utregnetTjeneste); } } diff --git a/web/src/main/java/no/nav/foreldrepenger/web/app/tjenester/behandling/dto/behandling/BehandlingDtoTjeneste.java b/web/src/main/java/no/nav/foreldrepenger/web/app/tjenester/behandling/dto/behandling/BehandlingDtoTjeneste.java index a0e29f48e8c..52070e8508d 100644 --- a/web/src/main/java/no/nav/foreldrepenger/web/app/tjenester/behandling/dto/behandling/BehandlingDtoTjeneste.java +++ b/web/src/main/java/no/nav/foreldrepenger/web/app/tjenester/behandling/dto/behandling/BehandlingDtoTjeneste.java @@ -305,20 +305,14 @@ private UtvidetBehandlingDto utvideBehandlingDtoKlage(Behandling behandling, Utv var uuidDto = new UuidDto(behandling.getUuid()); dto.leggTil(get(KlageRestTjeneste.KLAGE_V2_PATH, "klage-vurdering", uuidDto)); dto.leggTil(get(KlageRestTjeneste.MOTTATT_KLAGEDOKUMENT_V2_PATH, "mottatt-klagedokument", uuidDto)); - if (behandling.harAksjonspunktMedType(AksjonspunktDefinisjon.AVKLAR_VERGE) || vergeRepository.hentAggregat(behandling.getId()).isPresent()) { - dto.leggTil(get(PersonRestTjeneste.VERGE_PATH, "soeker-verge", uuidDto)); - dto.leggTil(get(PersonRestTjeneste.VERGE_BACKEND_PATH, "verge-backend", uuidDto)); - } + leggTilVergeHvisAksjonspunkt(behandling, dto, uuidDto); return dto; } private UtvidetBehandlingDto utvideBehandlingDtoAnke(Behandling behandling, UtvidetBehandlingDto dto) { var uuidDto = new UuidDto(behandling.getUuid()); dto.leggTil(get(AnkeRestTjeneste.ANKEVURDERING_V2_PATH, "anke-vurdering", uuidDto)); - if (behandling.harAksjonspunktMedType(AksjonspunktDefinisjon.AVKLAR_VERGE) || vergeRepository.hentAggregat(behandling.getId()).isPresent()) { - dto.leggTil(get(PersonRestTjeneste.VERGE_PATH, "soeker-verge", uuidDto)); - dto.leggTil(get(PersonRestTjeneste.VERGE_BACKEND_PATH, "verge-backend", uuidDto)); - } + leggTilVergeHvisAksjonspunkt(behandling, dto, uuidDto); return dto; } @@ -347,12 +341,7 @@ private UtvidetBehandlingDto utvideBehandlingDto(Behandling behandling, UtvidetB dto.leggTil(get(FamiliehendelseRestTjeneste.FAMILIEHENDELSE_V2_PATH, "familiehendelse-v2", uuidDto)); dto.leggTil(get(PersonRestTjeneste.PERSONOVERSIKT_PATH, "behandling-personoversikt", uuidDto)); dto.leggTil(get(PersonRestTjeneste.MEDLEMSKAP_V3_PATH, "soeker-medlemskap-v3", uuidDto)); - - if (behandling.harAksjonspunktMedType(AksjonspunktDefinisjon.AVKLAR_VERGE) || vergeRepository.hentAggregat(behandling.getId()).isPresent()) { - dto.leggTil(get(PersonRestTjeneste.VERGE_PATH, "soeker-verge", uuidDto)); - dto.leggTil(get(PersonRestTjeneste.VERGE_BACKEND_PATH, "verge-backend", uuidDto)); - } - + leggTilVergeHvisAksjonspunkt(behandling, dto, uuidDto); dto.leggTil(get(InntektArbeidYtelseRestTjeneste.INNTEKT_ARBEID_YTELSE_PATH, "inntekt-arbeid-ytelse", uuidDto)); dto.leggTil(get(InntektArbeidYtelseRestTjeneste.ARBEIDSGIVERE_OPPLYSNINGER_PATH, "arbeidsgivere-oversikt", uuidDto)); dto.leggTil(get(ArbeidOgInntektsmeldingRestTjeneste.HENT_ALLE_INNTEKTSMELDINGER_PATH, "inntektsmeldinger", uuidDto)); @@ -520,4 +509,11 @@ private Optional lagTilbakekrevingValgLink(Behandling behandling) private Behandlingsresultat getBehandlingsresultat(Long behandlingId) { return behandlingsresultatRepository.hentHvisEksisterer(behandlingId).orElse(null); } + + private void leggTilVergeHvisAksjonspunkt(Behandling behandling, UtvidetBehandlingDto dto, UuidDto uuidDto) { + if (behandling.harAksjonspunktMedType(AksjonspunktDefinisjon.AVKLAR_VERGE) || vergeRepository.hentAggregat(behandling.getId()).isPresent()) { + dto.leggTil(get(PersonRestTjeneste.VERGE_PATH, "soeker-verge", uuidDto)); + dto.leggTil(get(PersonRestTjeneste.VERGE_BACKEND_PATH, "verge-backend", uuidDto)); + } + } } diff --git "a/web/src/main/java/no/nav/foreldrepenger/web/app/tjenester/forvaltning/St\303\270nadskontoMigreringTask.java" "b/web/src/main/java/no/nav/foreldrepenger/web/app/tjenester/forvaltning/St\303\270nadskontoMigreringTask.java" index 743345f3e99..e14b5c0b216 100644 --- "a/web/src/main/java/no/nav/foreldrepenger/web/app/tjenester/forvaltning/St\303\270nadskontoMigreringTask.java" +++ "b/web/src/main/java/no/nav/foreldrepenger/web/app/tjenester/forvaltning/St\303\270nadskontoMigreringTask.java" @@ -78,7 +78,7 @@ public void doTask(ProsessTaskData prosessTaskData) { """; var query = entityManager.createNativeQuery(sql, Stønadskontoberegning.class) - .setParameter("fraId", fraId == null ? 0 : fraId); + .setParameter(FRA_ID, fraId == null ? 0 : fraId); return query.getResultStream(); } diff --git "a/web/src/main/java/no/nav/foreldrepenger/web/app/tjenester/forvaltning/stonadsstatistikk/St\303\270nadsstatistikkMigreringTask.java" "b/web/src/main/java/no/nav/foreldrepenger/web/app/tjenester/forvaltning/stonadsstatistikk/St\303\270nadsstatistikkMigreringTask.java" index c236eb92ca6..850e7f90142 100644 --- "a/web/src/main/java/no/nav/foreldrepenger/web/app/tjenester/forvaltning/stonadsstatistikk/St\303\270nadsstatistikkMigreringTask.java" +++ "b/web/src/main/java/no/nav/foreldrepenger/web/app/tjenester/forvaltning/stonadsstatistikk/St\303\270nadsstatistikkMigreringTask.java" @@ -108,7 +108,7 @@ and trunc(bv.OPPRETTET_TID) >=:fomDato var query = entityManager.createNativeQuery(sql, BehandlingVedtak.class) .setParameter("fomDato", fomDato) - .setParameter("fraId", fraId == null ? 0 : fraId) + .setParameter(FRA_ID, fraId == null ? 0 : fraId) .setHint(HibernateHints.HINT_READ_ONLY, "true"); return query.getResultStream(); } diff --git a/web/src/test/java/no/nav/foreldrepenger/RapporterUnmappedKolonnerIDatabaseTest.java b/web/src/test/java/no/nav/foreldrepenger/RapporterUnmappedKolonnerIDatabaseTest.java index e69afc60081..989b4e9be92 100644 --- a/web/src/test/java/no/nav/foreldrepenger/RapporterUnmappedKolonnerIDatabaseTest.java +++ b/web/src/test/java/no/nav/foreldrepenger/RapporterUnmappedKolonnerIDatabaseTest.java @@ -48,9 +48,6 @@ class RapporterUnmappedKolonnerIDatabaseTest { private static EntityManagerFactory entityManagerFactory; - public RapporterUnmappedKolonnerIDatabaseTest() { - } - @BeforeAll public static void setup() { @@ -172,9 +169,8 @@ public void integrate( } @Override - public void disintegrate( - SessionFactoryImplementor sessionFactory, - SessionFactoryServiceRegistry serviceRegistry) { + public void disintegrate(SessionFactoryImplementor sessionFactory, SessionFactoryServiceRegistry serviceRegistry) { + // NOSONAR } } diff --git a/web/src/test/java/no/nav/foreldrepenger/web/app/tjenester/behandling/aksjonspunkt/AksjonspunktRestTjenesteTest.java b/web/src/test/java/no/nav/foreldrepenger/web/app/tjenester/behandling/aksjonspunkt/AksjonspunktRestTjenesteTest.java index 5da39c602db..a2d3ed9773d 100644 --- a/web/src/test/java/no/nav/foreldrepenger/web/app/tjenester/behandling/aksjonspunkt/AksjonspunktRestTjenesteTest.java +++ b/web/src/test/java/no/nav/foreldrepenger/web/app/tjenester/behandling/aksjonspunkt/AksjonspunktRestTjenesteTest.java @@ -41,8 +41,8 @@ class AksjonspunktRestTjenesteTest { // skal_håndtere_overlappende_perioder data private static final LocalDate now = LocalDate.now(); private static final UUID behandlingUuid = UUID.randomUUID(); - private static final Long behandlingVersjon = 2L; - private static final String begrunnelse = "skal_håndtere_overlappende_perioder"; + private static final Long BEHANDLING_VERSJON = 2L; + private static final String BEGRUNNELSE = "skal_håndtere_overlappende_perioder"; private static final LocalDate fødselsdato = now.plusDays(40); private static final LocalDate termindato = now.plusDays(30); private static final LocalDate utstedtdato = now.minusDays(10); @@ -69,10 +69,10 @@ public void setUp() { @Test void skal_bekrefte_terminbekreftelse() throws Exception { Collection aksjonspunkt = new ArrayList<>(); - aksjonspunkt.add(new BekreftTerminbekreftelseAksjonspunktDto(begrunnelse, termindato, utstedtdato, 1)); + aksjonspunkt.add(new BekreftTerminbekreftelseAksjonspunktDto(BEGRUNNELSE, termindato, utstedtdato, 1)); aksjonspunktRestTjeneste.bekreft(mock(HttpServletRequest.class), - BekreftedeAksjonspunkterDto.lagDto(behandlingUuid, behandlingVersjon, aksjonspunkt)); + BekreftedeAksjonspunkterDto.lagDto(behandlingUuid, BEHANDLING_VERSJON, aksjonspunkt)); verify(aksjonspunktTjenesteMock).bekreftAksjonspunkter(ArgumentMatchers.anyCollection(), anyLong()); } @@ -81,10 +81,10 @@ void skal_bekrefte_terminbekreftelse() throws Exception { void skal_bekrefte_fødsel() throws Exception { Collection aksjonspunkt = new ArrayList<>(); var uidentifiserteBarn = new UidentifisertBarnDto[]{new UidentifisertBarnDto(fødselsdato, null)}; - aksjonspunkt.add(new SjekkManglendeFodselDto(begrunnelse, true, false, List.of(uidentifiserteBarn))); + aksjonspunkt.add(new SjekkManglendeFodselDto(BEGRUNNELSE, true, false, List.of(uidentifiserteBarn))); aksjonspunktRestTjeneste.bekreft(mock(HttpServletRequest.class), - BekreftedeAksjonspunkterDto.lagDto(behandlingUuid, behandlingVersjon, aksjonspunkt)); + BekreftedeAksjonspunkterDto.lagDto(behandlingUuid, BEHANDLING_VERSJON, aksjonspunkt)); verify(aksjonspunktTjenesteMock).bekreftAksjonspunkter(ArgumentMatchers.anyCollection(), anyLong()); } @@ -92,10 +92,10 @@ void skal_bekrefte_terminbekreftelse() throws Exception { @Test void skal_bekrefte_antall_barn() throws Exception { Collection aksjonspunkt = new ArrayList<>(); - aksjonspunkt.add(new SjekkManglendeFodselDto(begrunnelse, false, false, new ArrayList<>())); + aksjonspunkt.add(new SjekkManglendeFodselDto(BEGRUNNELSE, false, false, new ArrayList<>())); aksjonspunktRestTjeneste.bekreft(mock(HttpServletRequest.class), - BekreftedeAksjonspunkterDto.lagDto(behandlingUuid, behandlingVersjon, aksjonspunkt)); + BekreftedeAksjonspunkterDto.lagDto(behandlingUuid, BEHANDLING_VERSJON, aksjonspunkt)); verify(aksjonspunktTjenesteMock).bekreftAksjonspunkter(ArgumentMatchers.anyCollection(), anyLong()); @@ -108,10 +108,10 @@ void skal_bekrefte_fatte_vedtak_med_aksjonspunkt_godkjent() throws Exception { Collection aksjonspunktGodkjenningDtos = new ArrayList<>(); var godkjentAksjonspunkt = opprettetGodkjentAksjonspunkt(); aksjonspunktGodkjenningDtos.add(godkjentAksjonspunkt); - aksjonspunkt.add(new FatterVedtakAksjonspunktDto(begrunnelse, aksjonspunktGodkjenningDtos)); + aksjonspunkt.add(new FatterVedtakAksjonspunktDto(BEGRUNNELSE, aksjonspunktGodkjenningDtos)); aksjonspunktRestTjeneste.beslutt(mock(HttpServletRequest.class), - BekreftedeAksjonspunkterDto.lagDto(behandlingUuid, behandlingVersjon, aksjonspunkt)); + BekreftedeAksjonspunkterDto.lagDto(behandlingUuid, BEHANDLING_VERSJON, aksjonspunkt)); verify(aksjonspunktTjenesteMock).bekreftAksjonspunkter(ArgumentMatchers.anyCollection(), anyLong()); } @@ -120,23 +120,23 @@ void skal_bekrefte_fatte_vedtak_med_aksjonspunkt_godkjent() throws Exception { void skal_ikke_kunne_bekrefte_andre_aksjonspunkt_ved_status_fatter_vedtak() { when(behandling.getStatus()).thenReturn(BehandlingStatus.FATTER_VEDTAK); Collection aksjonspunkt = new ArrayList<>(); - aksjonspunkt.add(new SjekkManglendeFodselDto(begrunnelse, false, false, new ArrayList<>())); - var dto = BekreftedeAksjonspunkterDto.lagDto(behandlingUuid, behandlingVersjon, aksjonspunkt); + aksjonspunkt.add(new SjekkManglendeFodselDto(BEGRUNNELSE, false, false, new ArrayList<>())); + var dto = BekreftedeAksjonspunkterDto.lagDto(behandlingUuid, BEHANDLING_VERSJON, aksjonspunkt); var request = mock(HttpServletRequest.class); assertThrows(FunksjonellException.class, () -> aksjonspunktRestTjeneste.bekreft(request, dto)); } @Test void skal_kunne_sende_fatte_vedtak_til_beslutter_endepunkt() throws URISyntaxException { - aksjonspunktRestTjeneste.beslutt(mock(HttpServletRequest.class), BekreftedeAksjonspunkterDto.lagDto(behandlingUuid, behandlingVersjon, - List.of(new FatterVedtakAksjonspunktDto(begrunnelse, List.of(new AksjonspunktGodkjenningDto()))))); + aksjonspunktRestTjeneste.beslutt(mock(HttpServletRequest.class), BekreftedeAksjonspunkterDto.lagDto(behandlingUuid, BEHANDLING_VERSJON, + List.of(new FatterVedtakAksjonspunktDto(BEGRUNNELSE, List.of(new AksjonspunktGodkjenningDto()))))); verify(aksjonspunktTjenesteMock).bekreftAksjonspunkter(ArgumentMatchers.anyCollection(), anyLong()); } @Test void skal_ikke_kunne_sende_andre_ap_til_beslutter_endepunkt() { - var dto = BekreftedeAksjonspunkterDto.lagDto(behandlingUuid, behandlingVersjon, + var dto = BekreftedeAksjonspunkterDto.lagDto(behandlingUuid, BEHANDLING_VERSJON, List.of(new FastsetteUttakDto.FastsetteUttakPerioderDto(List.of()))); assertThatThrownBy(() -> aksjonspunktRestTjeneste.beslutt(mock(HttpServletRequest.class), dto)).isExactlyInstanceOf( IllegalArgumentException.class); @@ -144,7 +144,7 @@ void skal_ikke_kunne_sende_andre_ap_til_beslutter_endepunkt() { @Test void skal_ikke_kunne_sende_fatter_vedtak_ap_til_aksjonspunkt_endepunkt() { - var dto = BekreftedeAksjonspunkterDto.lagDto(behandlingUuid, behandlingVersjon, List.of(new FatterVedtakAksjonspunktDto())); + var dto = BekreftedeAksjonspunkterDto.lagDto(behandlingUuid, BEHANDLING_VERSJON, List.of(new FatterVedtakAksjonspunktDto())); assertThatThrownBy(() -> aksjonspunktRestTjeneste.bekreft(mock(HttpServletRequest.class), dto)).isExactlyInstanceOf( IllegalArgumentException.class); } diff --git a/web/src/test/java/no/nav/foreldrepenger/web/app/tjenester/behandling/uttak/app/SaldoerDtoTjenesteTest.java b/web/src/test/java/no/nav/foreldrepenger/web/app/tjenester/behandling/uttak/app/SaldoerDtoTjenesteTest.java index 6a95d557183..75e6b3b5fba 100644 --- a/web/src/test/java/no/nav/foreldrepenger/web/app/tjenester/behandling/uttak/app/SaldoerDtoTjenesteTest.java +++ b/web/src/test/java/no/nav/foreldrepenger/web/app/tjenester/behandling/uttak/app/SaldoerDtoTjenesteTest.java @@ -66,7 +66,6 @@ import no.nav.foreldrepenger.domene.uttak.TapteDagerFpffTjeneste; import no.nav.foreldrepenger.domene.uttak.UttakRepositoryProvider; import no.nav.foreldrepenger.domene.uttak.beregnkontoer.UtregnetStønadskontoTjeneste; -import no.nav.foreldrepenger.domene.uttak.fastsetteperioder.grunnlagbyggere.KontoerGrunnlagBygger; import no.nav.foreldrepenger.domene.uttak.input.Annenpart; import no.nav.foreldrepenger.domene.uttak.input.Barn; import no.nav.foreldrepenger.domene.uttak.input.FamilieHendelse; @@ -102,8 +101,7 @@ public void setUp() { var fagsakRelasjonTjeneste = new FagsakRelasjonTjeneste(repositoryProvider); var uttakTjeneste = new ForeldrepengerUttakTjeneste(fpUttakRepository); var utregnetKontoTjeneste = new UtregnetStønadskontoTjeneste(fagsakRelasjonTjeneste, uttakTjeneste); - var stønadskontoSaldoTjeneste = new StønadskontoSaldoTjeneste(uttakRepositoryProvider, new KontoerGrunnlagBygger(), - utregnetKontoTjeneste); + var stønadskontoSaldoTjeneste = new StønadskontoSaldoTjeneste(uttakRepositoryProvider, utregnetKontoTjeneste); var tapteDagerFpffTjeneste = new TapteDagerFpffTjeneste(uttakRepositoryProvider, new YtelseFordelingTjeneste(new YtelsesFordelingRepository(entityManager))); tjeneste = new SaldoerDtoTjeneste(stønadskontoSaldoTjeneste, uttakTjeneste, tapteDagerFpffTjeneste, utregnetKontoTjeneste); @@ -543,13 +541,13 @@ void riktig_saldo_for_mors_uttaksplan_med_flere_arbeidsforhold() { var fpffDto = saldoer.stonadskontoer().get(SaldoerDto.SaldoVisningStønadskontoType.FORELDREPENGER_FØR_FØDSEL); assertThat(fpffDto.maxDager()).isEqualTo(maxDagerFPFF); assertThat(fpffDto.aktivitetSaldoDtoList()).hasSize(2); - assertThat(fpffDto.aktivitetSaldoDtoList().get(0).saldo()).isZero(); + assertThat(fpffDto.aktivitetSaldoDtoList().getFirst().saldo()).isZero(); assertThat(fpffDto.aktivitetSaldoDtoList().get(1).saldo()).isZero(); var mkDto = saldoer.stonadskontoer().get(SaldoerDto.SaldoVisningStønadskontoType.MØDREKVOTE); assertThat(mkDto.maxDager()).isEqualTo(maxDagerMK); assertThat(mkDto.aktivitetSaldoDtoList()).hasSize(2); - assertThat(mkDto.aktivitetSaldoDtoList().get(0).saldo()).isEqualTo(maxDagerMK - 6 * 5); + assertThat(mkDto.aktivitetSaldoDtoList().getFirst().saldo()).isEqualTo(maxDagerMK - 6 * 5); assertThat(mkDto.aktivitetSaldoDtoList().get(1).saldo()).isEqualTo(maxDagerMK - 6 * 5); var fpDto = saldoer.stonadskontoer().get(SaldoerDto.SaldoVisningStønadskontoType.FELLESPERIODE); @@ -1040,7 +1038,7 @@ private void assertKonto(StønadskontoDto stønadskontoDto, int maxDager, int sa assertThat(stønadskontoDto.maxDager()).isEqualTo(maxDager); assertThat(stønadskontoDto.aktivitetSaldoDtoList()).hasSize(1); assertThat(stønadskontoDto.saldo()).isEqualTo(saldo); - assertThat(stønadskontoDto.aktivitetSaldoDtoList().get(0).saldo()).isEqualTo(saldo); + assertThat(stønadskontoDto.aktivitetSaldoDtoList().getFirst().saldo()).isEqualTo(saldo); }