Merge branch 'master' into dependabot/maven/nav-deps-c73ae7febc

Author: tendestad

domenetjenester/perioder/src/main/java/no/nav/ung/sak/perioder/ProsessTriggerPeriodeUtleder.java

@@ -1,8 +1,5 @@
 package no.nav.ung.sak.perioder;
 
-import java.util.Collection;
-import java.util.Set;
-
 import jakarta.enterprise.context.Dependent;
 import jakarta.inject.Inject;
 import no.nav.fpsak.tidsserie.LocalDateInterval;
@@ -13,18 +10,12 @@
 import no.nav.ung.sak.trigger.ProsessTriggereRepository;
 import no.nav.ung.sak.trigger.Trigger;
 
+import java.util.Collection;
+import java.util.Set;
+
 @Dependent
 public class ProsessTriggerPeriodeUtleder {
 
-    // Prosesstriggere som er relevante. RE_HENDELSE_OPPHØR_UNGDOMSPROGRAM håndteres i UtledPeriodeTilVurderingFraUngdomsprogram
-    public static final Set<BehandlingÅrsakType> RELEVANTE_ÅRSAKER = Set.of(
-        BehandlingÅrsakType.RE_HENDELSE_DØD_FORELDER,
-        BehandlingÅrsakType.RE_HENDELSE_DØD_BARN,
-        BehandlingÅrsakType.RE_HENDELSE_FØDSEL,
-        BehandlingÅrsakType.RE_TRIGGER_BEREGNING_HØY_SATS,
-        BehandlingÅrsakType.RE_RAPPORTERING_INNTEKT,
-        BehandlingÅrsakType.NY_SØKT_PROGRAM_PERIODE
-    );
     private final ProsessTriggereRepository prosessTriggereRepository;
     private final UngdomsytelseSøknadsperiodeTjeneste ungdomsytelseSøknadsperiodeTjeneste;
 
@@ -41,11 +32,13 @@ public ProsessTriggerPeriodeUtleder(ProsessTriggereRepository prosessTriggereRep
      * @return Tidslinje for perioder til vurdering
      */
     public LocalDateTimeline<Set<BehandlingÅrsakType>> utledTidslinje(Long behandligId) {
-        return prosessTriggereRepository.hentGrunnlag(behandligId)
+        final var triggere = prosessTriggereRepository.hentGrunnlag(behandligId)
             .stream()
             .map(ProsessTriggere::getTriggere)
             .flatMap(Collection::stream)
-            .filter(it -> RELEVANTE_ÅRSAKER.contains(it.getÅrsak()))
+            .toList();
+        return triggere
+            .stream()
             .map(p -> new LocalDateTimeline<>(finnPeriodeForBehandlingsårsak(behandligId, p, p.getÅrsak()), Set.of(p.getÅrsak())))
             .reduce((t1, t2) -> t1.crossJoin(t2, StandardCombinators::union))
             .orElse(LocalDateTimeline.empty());

domenetjenester/perioder/src/test/java/no/nav/ung/sak/perioder/ProsessTriggerPeriodeUtlederTest.java

@@ -48,17 +48,7 @@ void skal_ikke_finne_perioder_for_ingen_triggere() {
     }
 
     @Test
-    void skal_ikke_finne_perioder_for_ikke_relevant_trigger() {
-        // Arrange
-        prosessTriggereRepository.leggTil(behandlingId, Set.of(new Trigger(BehandlingÅrsakType.RE_ENDRET_FORDELING, DatoIntervallEntitet.fraOgMedTilOgMed(LocalDate.now(), LocalDate.now()))));
-        // Act
-        final var resultat = prosessTriggerPeriodeUtleder.utledTidslinje(behandlingId);
-        // Assert
-        assertThat(resultat.isEmpty()).isTrue();
-    }
-
-    @Test
-    void skal_finne_en_periode_for_relevant_trigger() {
+    void skal_finne_en_periode_trigger() {
         // Arrange
         final var fom = LocalDate.now();
         final var tom = LocalDate.now().plusDays(10);
@@ -70,7 +60,7 @@ void skal_finne_en_periode_for_relevant_trigger() {
     }
 
     @Test
-    void skal_finne_en_periode_for_to_overlappende_relevante_triggere() {
+    void skal_finne_en_periode_for_to_overlappende_triggere() {
         // Arrange
         final var fom = LocalDate.now();
         final var tom = LocalDate.now().plusDays(10);
@@ -85,7 +75,7 @@ void skal_finne_en_periode_for_to_overlappende_relevante_triggere() {
     }
 
     @Test
-    void skal_finne_tidslinje_for_to_ikke_overlappende_relevante_triggere() {
+    void skal_finne_tidslinje_for_to_ikke_overlappende_triggere() {
         // Arrange
         final var fom = LocalDate.now();
         final var tom = LocalDate.now().plusDays(10);

pom.xml

@@ -36,8 +36,7 @@
         <project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>
         <maven.deploy.skip>true</maven.deploy.skip>
         <felles.version>4.4.16</felles.version>
-        <swagger.version>2.2.28</swagger.version>
-        <junit.version>5.12.0</junit.version>
+        <swagger.version>2.2.29</swagger.version>
         <k9format.version>11.4.2</k9format.version>
         <jackson.version>2.18.3</jackson.version>
         <asm.version>9.7.1</asm.version>
@@ -318,7 +317,7 @@
             <dependency>
                 <groupId>no.nav.sif.abac</groupId>
                 <artifactId>kontrakt</artifactId>
-                <version>1.0.6</version>
+                <version>1.1.0</version>
             </dependency>
 
             <dependency>

web/src/main/java/no/nav/ung/sak/web/server/abac/AppPdpKlient.java

@@ -8,7 +8,7 @@
 import no.nav.k9.felles.sikkerhet.abac.PdpKlient;
 import no.nav.k9.felles.sikkerhet.abac.PdpRequest;
 import no.nav.k9.felles.sikkerhet.abac.Tilgangsbeslutning;
-import no.nav.sif.abac.kontrakt.abac.dto.SaksinformasjonTilgangskontrollInputDto;
+import no.nav.sif.abac.kontrakt.abac.dto.SaksinformasjonOgPersonerTilgangskontrollInputDto;
 
 import java.util.Set;
 
@@ -26,7 +26,7 @@ public AppPdpKlient(SifAbacPdpRestKlient sifAbacPdpRestKlient) {
 
     @Override
     public Tilgangsbeslutning forespørTilgang(PdpRequest pdpRequest) {
-        SaksinformasjonTilgangskontrollInputDto tilgangskontrollInput = PdpRequestMapper.map(pdpRequest);
+        SaksinformasjonOgPersonerTilgangskontrollInputDto tilgangskontrollInput = PdpRequestMapper.map(pdpRequest);
         Decision decision = sifAbacPdpRestKlient.sjekkTilgangForInnloggetBruker(tilgangskontrollInput);
         return new Tilgangsbeslutning(decision == Decision.Permit, Set.of(), pdpRequest);
     }

web/src/main/java/no/nav/ung/sak/web/server/abac/PdpRequestMapper.java

@@ -1,10 +1,14 @@
 package no.nav.ung.sak.web.server.abac;
 
 import no.nav.k9.felles.sikkerhet.abac.PdpRequest;
-import no.nav.sif.abac.kontrakt.abac.*;
+import no.nav.sif.abac.kontrakt.abac.AbacBehandlingStatus;
+import no.nav.sif.abac.kontrakt.abac.AbacFagsakStatus;
+import no.nav.sif.abac.kontrakt.abac.AksjonspunktType;
+import no.nav.sif.abac.kontrakt.abac.BeskyttetRessursActionAttributt;
+import no.nav.sif.abac.kontrakt.abac.ResourceType;
 import no.nav.sif.abac.kontrakt.abac.dto.OperasjonDto;
 import no.nav.sif.abac.kontrakt.abac.dto.SaksinformasjonDto;
-import no.nav.sif.abac.kontrakt.abac.dto.SaksinformasjonTilgangskontrollInputDto;
+import no.nav.sif.abac.kontrakt.abac.dto.SaksinformasjonOgPersonerTilgangskontrollInputDto;
 import no.nav.sif.abac.kontrakt.person.AktørId;
 import no.nav.sif.abac.kontrakt.person.PersonIdent;
 
@@ -15,8 +19,8 @@
 
 public class PdpRequestMapper {
 
-    public static SaksinformasjonTilgangskontrollInputDto map(PdpRequest pdpRequest){
-        return new SaksinformasjonTilgangskontrollInputDto(
+    public static SaksinformasjonOgPersonerTilgangskontrollInputDto map(PdpRequest pdpRequest){
+        return new SaksinformasjonOgPersonerTilgangskontrollInputDto(
             pdpRequest.getListOfString(AbacAttributter.RESOURCE_FELLES_PERSON_AKTOERID_RESOURCE).stream().map(AktørId::new).toList(),
             pdpRequest.getListOfString(AbacAttributter.RESOURCE_FELLES_PERSON_FNR).stream().map(PersonIdent::new).toList(),
             operasjon(pdpRequest),
@@ -53,10 +57,10 @@ private static Set<AksjonspunktType> aksjonspunktTyperFraKoder(List<String> kode
     private static AksjonspunktType aksjonspunktTypeFraKode(String kode) {
         return switch (kode) {
             case null -> null;
-            case "AUTO" -> AksjonspunktType.AUTOPUNKT;
-            case "MANU" -> AksjonspunktType.MANUELL;
-            case "OVST" -> AksjonspunktType.OVERSTYRING;
-            case "SAOV" -> AksjonspunktType.SAKSBEHANDLEROVERSTYRING;
+            case "Autopunkt" -> AksjonspunktType.AUTOPUNKT;
+            case "Manuell" -> AksjonspunktType.MANUELL;
+            case "Overstyring" -> AksjonspunktType.OVERSTYRING;
+            case "Saksbehandleroverstyring" -> AksjonspunktType.SAKSBEHANDLEROVERSTYRING;
             default -> throw new IllegalStateException("Unexpected value: " + kode);
         };
     }

web/src/main/java/no/nav/ung/sak/web/server/abac/SifAbacPdpRestKlient.java

@@ -6,8 +6,7 @@
 import no.nav.k9.felles.integrasjon.rest.ScopedRestIntegration;
 import no.nav.k9.felles.konfigurasjon.konfig.KonfigVerdi;
 import no.nav.k9.felles.sikkerhet.abac.Decision;
-import no.nav.sif.abac.kontrakt.abac.InnloggetAnsattDto;
-import no.nav.sif.abac.kontrakt.abac.dto.SaksinformasjonTilgangskontrollInputDto;
+import no.nav.sif.abac.kontrakt.abac.dto.SaksinformasjonOgPersonerTilgangskontrollInputDto;
 
 import java.net.URI;
 import java.net.URISyntaxException;
@@ -30,7 +29,7 @@ public SifAbacPdpRestKlient(OidcRestClient restClient,
         this.uriTilgangskontrollMedSaksinformasjon = tilUri(urlSifAbacPdp, "saksinformasjon");
     }
 
-    public Decision sjekkTilgangForInnloggetBruker(SaksinformasjonTilgangskontrollInputDto input) {
+    public Decision sjekkTilgangForInnloggetBruker(SaksinformasjonOgPersonerTilgangskontrollInputDto input) {
         return restClient.post(uriTilgangskontrollMedSaksinformasjon, input, Decision.class);
     }