Merge pull request #689 from navikt/periodeRegel17

Lagt inn periodesjekk i regel 17. Lagt til cucumber
navikt · Oct 31, 2024 · af3add1 · af3add1
2 parents 7276096 + 41b4476
commit af3add1
Show file tree

Hide file tree

Showing 3 changed files with 42 additions and 2 deletions.
diff --git a/src/main/kotlin/no/nav/medlemskap/domene/arbeidsforhold/Arbeidsforhold.kt b/src/main/kotlin/no/nav/medlemskap/domene/arbeidsforhold/Arbeidsforhold.kt
@@ -548,7 +548,7 @@ data class Arbeidsforhold(
         private fun List<Utenlandsopphold>.hentLandkoder(): List<String> =
             this.map { it.landkode }
 
-        private fun List<Arbeidsforhold>.arbeidsforholdForKontrollPeriode(kontrollPeriode: Kontrollperiode) =
+        fun List<Arbeidsforhold>.arbeidsforholdForKontrollPeriode(kontrollPeriode: Kontrollperiode) =
             this.filter {
                 it.periode.overlapper(kontrollPeriode.periode)
             }

diff --git a/src/main/kotlin/no/nav/medlemskap/regler/v1/arbeidsforhold/HarBrukerArbeidsforholdRegel.kt b/src/main/kotlin/no/nav/medlemskap/regler/v1/arbeidsforhold/HarBrukerArbeidsforholdRegel.kt
@@ -3,6 +3,7 @@ package no.nav.medlemskap.regler.v1.arbeidsforhold
 import no.nav.medlemskap.domene.Datagrunnlag
 import no.nav.medlemskap.domene.Ytelse
 import no.nav.medlemskap.domene.arbeidsforhold.Arbeidsforhold
+import no.nav.medlemskap.domene.arbeidsforhold.Arbeidsforhold.Companion.arbeidsforholdForKontrollPeriode
 import no.nav.medlemskap.regler.common.RegelId
 import no.nav.medlemskap.regler.common.Resultat
 import no.nav.medlemskap.regler.common.Resultat.Companion.ja
@@ -18,11 +19,13 @@ class HarBrukerArbeidsforholdRegel(
 
     override fun operasjon(): Resultat {
         return when {
-            arbeidsforhold.isEmpty() -> nei(regelId)
+            arbeidsforhold.arbeidsforholdForKontrollPeriode(kontrollPeriodeForArbeidsforhold).isEmpty() -> nei(regelId)
             else -> ja(regelId)
         }
     }
 
+
+
     companion object {
 
         fun fraDatagrunnlag(datagrunnlag: Datagrunnlag): HarBrukerArbeidsforholdRegel {

diff --git a/...umentasjon/features/hovedregler/arbeidsforhold/regel_17_har_bruker_arbeidsforhold.feature b/...umentasjon/features/hovedregler/arbeidsforhold/regel_17_har_bruker_arbeidsforhold.feature
@@ -21,4 +21,41 @@ Egenskap: Regel 17: Har bruker arbeidsforhold?
       | Fra og med dato | Til og med dato | Har hatt arbeid utenfor Norge |
       | 30.01.2020      | 30.01.2021      | Nei                           |
 
+    Så skal svaret være "Ja"
+
+  Scenario: Bruker med arbeidsforhold som starter i kontrollperiode får "Ja"
+
+    Gitt følgende arbeidsforhold fra AAReg
+      | Fra og med dato | Til og med dato | Arbeidsgivertype | Arbeidsforholdstype |
+      | 30.03.2019      |                 | Organisasjon     | NORMALT             |
+
+    Når regel "17" kjøres med følgende parametre
+      | Fra og med dato | Til og med dato | Har hatt arbeid utenfor Norge |
+      | 30.01.2020      | 30.01.2021      | Nei                           |
+
+    Så skal svaret være "Ja"
+
+  Scenario: Bruker med arbeidsforhold som starter før og slutter i kontrollperiode får "Ja"
+
+    Gitt følgende arbeidsforhold fra AAReg
+      | Fra og med dato | Til og med dato | Arbeidsgivertype | Arbeidsforholdstype |
+      | 01.01.2018      | 30.10.2019      | Organisasjon     | NORMALT             |
+
+    Når regel "17" kjøres med følgende parametre
+      | Fra og med dato | Til og med dato | Har hatt arbeid utenfor Norge |
+      | 30.01.2020      | 30.01.2021      | Nei                           |
+
+    Så skal svaret være "Ja"
+
+
+  Scenario: Bruker med arbeidsforhold som starter og slutter i kontrollperiode får "Ja"
+
+    Gitt følgende arbeidsforhold fra AAReg
+      | Fra og med dato | Til og med dato | Arbeidsgivertype | Arbeidsforholdstype |
+      | 01.02.2019      | 30.10.2019      | Organisasjon     | NORMALT             |
+
+    Når regel "17" kjøres med følgende parametre
+      | Fra og med dato | Til og med dato | Har hatt arbeid utenfor Norge |
+      | 30.01.2020      | 30.01.2021      | Nei                           |
+
     Så skal svaret være "Ja"