Endret regel eval. til å ikludere en default regel som brukes når ing…

…en andre regler slår inn
navikt · Aug 21, 2024 · 86ba9e6 · 86ba9e6
1 parent df361bd
commit 86ba9e6
Show file tree

Hide file tree

Showing 6 changed files with 35 additions and 18 deletions.
diff --git a/...erioder/src/main/kotlin/no/nav/paw/arbeidssokerregisteret/application/RequestValidator.kt b/...erioder/src/main/kotlin/no/nav/paw/arbeidssokerregisteret/application/RequestValidator.kt
@@ -7,6 +7,7 @@ import no.nav.paw.arbeidssokerregisteret.RequestScope
 import no.nav.paw.arbeidssokerregisteret.application.authfaktka.navAnsattTilgangFakta
 import no.nav.paw.arbeidssokerregisteret.application.authfaktka.tokenXPidFakta
 import no.nav.paw.arbeidssokerregisteret.application.opplysninger.*
+import no.nav.paw.arbeidssokerregisteret.application.regler.standardTilgangsregel
 import no.nav.paw.arbeidssokerregisteret.application.regler.tilgangsReglerIPrioritertRekkefolge
 import no.nav.paw.arbeidssokerregisteret.domain.Identitetsnummer
 import no.nav.paw.arbeidssokerregisteret.services.AutorisasjonService
@@ -31,7 +32,7 @@ class RequestValidator(
                 } else {
                     emptySet()
                 }
-        return tilgangsReglerIPrioritertRekkefolge.evaluer(autentiseringsFakta)
+        return tilgangsReglerIPrioritertRekkefolge.evaluer(defaultRegel = standardTilgangsregel, autentiseringsFakta)
     }
 
     context(RequestScope)
@@ -44,10 +45,12 @@ class RequestValidator(
             .flatMap { tilgangsResultat ->
                 val person = personInfoService.hentPersonInfo(identitetsnummer.verdi)
                 val opplysning = person?.let { genererPersonFakta(it) } ?: setOf(DomeneOpplysning.PersonIkkeFunnet)
-                reglerForInngangIPrioritertRekkefolge.evaluer(opplysning + tilgangsResultat.opplysning)
+                reglerForInngangIPrioritertRekkefolge.evaluer(
+                    defaultRegel = standardInngangsregel,
+                    opplysning + tilgangsResultat.opplysning
+                )
             }
 
-
     fun genererPersonFakta(person: Person): Set<Opplysning> {
         require(person.foedsel.size <= 1) { "Personen har flere fødselsdatoer enn forventet" }
         require(person.bostedsadresse.size <= 1) { "Personen har flere bostedsadresser enn forventet" }

diff --git a/...er/src/main/kotlin/no/nav/paw/arbeidssokerregisteret/application/regler/TilgangsRegler.kt b/...er/src/main/kotlin/no/nav/paw/arbeidssokerregisteret/application/regler/TilgangsRegler.kt
@@ -26,12 +26,13 @@ val tilgangsReglerIPrioritertRekkefolge: List<Regel> = listOf(
     AnsattIkkeTilgangTilBruker(
         AuthOpplysning.AnsattIkkeTilgang,
         vedTreff = ::skalAvises
-    ),
-    IkkeTilgang(
-        vedTreff = ::muligGrunnlagForAvvisning
     )
 )
 
+val standardTilgangsregel = IkkeTilgang(
+    vedTreff = ::skalAvises
+)
+
 sealed interface AuthRegelId: RegelId
 
 data object IkkeTilgang : AuthRegelId {

diff --git a/...art-stopp-perioder/src/test/kotlin/no/nav/paw/arbeidssokerregisteret/TilgansReglerTest.kt b/...art-stopp-perioder/src/test/kotlin/no/nav/paw/arbeidssokerregisteret/TilgansReglerTest.kt
@@ -11,12 +11,14 @@ import no.nav.paw.arbeidssokerregisteret.application.authfaktka.AuthOpplysning.I
 import no.nav.paw.arbeidssokerregisteret.application.opplysninger.DomeneOpplysning
 import no.nav.paw.arbeidssokerregisteret.application.regler.AnsattHarTilgangTilBruker
 import no.nav.paw.arbeidssokerregisteret.application.regler.IkkeAnsattOgForhaandsgodkjentAvAnsatt
+import no.nav.paw.arbeidssokerregisteret.application.regler.standardTilgangsregel
 import no.nav.paw.arbeidssokerregisteret.application.regler.tilgangsReglerIPrioritertRekkefolge
 
 class TilgansReglerTest : FreeSpec({
     "eval av tilgang skal gi" - {
         "ugydlig request ved IKKE_ANSATT kombinert med GODKJENT_AV_ANSATT" {
             val resultat = tilgangsReglerIPrioritertRekkefolge.evaluer(
+                standardTilgangsregel,
                 setOf(
                     IkkeAnsatt,
                     DomeneOpplysning.ErForhaandsgodkjent
@@ -27,6 +29,7 @@ class TilgansReglerTest : FreeSpec({
         }
         "lovlig kombinasjon av ANSATT og FORHANDSGODKJENT_AV_ANSATT" {
             val resultat = tilgangsReglerIPrioritertRekkefolge.evaluer(
+                standardTilgangsregel,
                 setOf(
                     AnsattTilgang,
                     DomeneOpplysning.ErForhaandsgodkjent

diff --git a/...ain/kotlin/no/nav/paw/arbeidssoekerregisteret/utgang/pdl/kafka/ScheduleAvsluttPerioder.kt b/...ain/kotlin/no/nav/paw/arbeidssoekerregisteret/utgang/pdl/kafka/ScheduleAvsluttPerioder.kt
@@ -14,11 +14,7 @@ import no.nav.paw.arbeidssoekerregisteret.utgang.pdl.utils.negativeOpplysninger
 import no.nav.paw.arbeidssoekerregisteret.utgang.pdl.utils.statusToOpplysningMap
 import no.nav.paw.arbeidssoekerregisteret.utgang.pdl.utils.toAarsak
 import no.nav.paw.arbeidssoekerregisteret.utgang.pdl.utils.toPerson
-import no.nav.paw.arbeidssokerregisteret.application.GrunnlagForGodkjenning
-import no.nav.paw.arbeidssokerregisteret.application.Problem
-import no.nav.paw.arbeidssokerregisteret.application.evaluer
-import no.nav.paw.arbeidssokerregisteret.application.hendelseOpplysningTilDomeneOpplysninger
-import no.nav.paw.arbeidssokerregisteret.application.reglerForInngangIPrioritertRekkefolge
+import no.nav.paw.arbeidssokerregisteret.application.*
 import no.nav.paw.arbeidssokerregisteret.intern.v1.Avsluttet
 import no.nav.paw.arbeidssokerregisteret.intern.v1.Hendelse
 import no.nav.paw.arbeidssokerregisteret.intern.v1.vo.Bruker
@@ -152,8 +148,14 @@ private fun List<HentPersonBolkResult>.processResultsV2(
         .map { hendelseOpplysningTilDomeneOpplysninger(it) as no.nav.paw.arbeidssokerregisteret.application.opplysninger.Opplysning }
         .toSet()
 
-    val opplysningerEvaluering = reglerForInngangIPrioritertRekkefolge.evaluer(domeneOpplysninger)
-    val pdlEvaluering = reglerForInngangIPrioritertRekkefolge.evaluer(genererPersonFakta(person.toPerson()))
+    val opplysningerEvaluering = reglerForInngangIPrioritertRekkefolge.evaluer(
+        defaultRegel = standardInngangsregel,
+        domeneOpplysninger
+    )
+    val pdlEvaluering = reglerForInngangIPrioritertRekkefolge.evaluer(
+        defaultRegel = standardInngangsregel,
+        genererPersonFakta(person.toPerson())
+    )
 
     val erForhaandsgodkjent = hendelseOpplysninger.contains(Opplysning.FORHAANDSGODKJENT_AV_ANSATT)
 

diff --git a/...er-regler/src/main/kotlin/no/nav/paw/arbeidssokerregisteret/application/InngangsRegler.kt b/...er-regler/src/main/kotlin/no/nav/paw/arbeidssokerregisteret/application/InngangsRegler.kt
@@ -33,8 +33,9 @@ val reglerForInngangIPrioritertRekkefolge: List<Regel> = listOf(
         ErOver18Aar,
         BosattEtterFregLoven,
         vedTreff = ::grunnlagForGodkjenning
-    ),
-    IkkeBosattINorgeIHenholdTilFolkeregisterloven(
-        vedTreff = ::muligGrunnlagForAvvisning
     )
 )
+
+val standardInngangsregel = IkkeBosattINorgeIHenholdTilFolkeregisterloven(
+    vedTreff = ::muligGrunnlagForAvvisning
+)
diff --git a/...r-regler/src/main/kotlin/no/nav/paw/arbeidssokerregisteret/application/RegelExtensions.kt b/...r-regler/src/main/kotlin/no/nav/paw/arbeidssokerregisteret/application/RegelExtensions.kt
@@ -17,10 +17,17 @@ operator fun RegelId.invoke(
 
 fun Regel.evaluer(samletOpplysning: Iterable<Opplysning>): Boolean = opplysninger.all { samletOpplysning.contains(it) }
 
-fun List<Regel>.evaluer(opplysninger: Iterable<Opplysning>): Either<Problem, GrunnlagForGodkjenning> =
+/**
+ * Evaluerer en liste med regler mot en liste med opplysninger. Returnerer første regel som evalueres til sann,
+ * eller defaultRegel om ingen regler evalueres til sann.
+ */
+fun List<Regel>.evaluer(
+    defaultRegel: Regel,
+    opplysninger: Iterable<Opplysning>
+): Either<Problem, GrunnlagForGodkjenning> =
     filter { regel -> regel.evaluer(opplysninger) }
         .map { regel -> regel.vedTreff(opplysninger) }
-        .first()
+        .firstOrNull() ?: defaultRegel.vedTreff(opplysninger)
 
 fun domeneOpplysningTilHendelseOpplysning(opplysning: DomeneOpplysning): HendelseOpplysning =
     when (opplysning) {