util fn

Author: janolaveide

src/main/kotlin/no/nav/tilgangsmaskin/populasjonstilgangskontroll/integrasjoner/nom/NomJPAAdapter.kt

@@ -18,5 +18,5 @@ class NomJPAAdapter(private val repo: NomRepository) {
              startdato = start?.toInstant()
              gyldigtil = slutt?.toInstant()
         } ?: NomEntity(ansattId, ansattFnr, start?.toInstant(),slutt?.toInstant())).id!!
-    fun fnrForAnsatt(ansattId: String) = repo.finnGyldigAnsattFnr(ansattId)?.let { AnsattFnr(it) }
+    fun fnrForAnsatt(ansattId: String) = repo.ansattFødselsnummer(ansattId)?.let { AnsattFnr(it) }
     private fun LocalDate.toInstant(): Instant = atStartOfDay(systemDefault()).toInstant()}

src/main/kotlin/no/nav/tilgangsmaskin/populasjonstilgangskontroll/integrasjoner/nom/NomRepository.kt

@@ -6,8 +6,7 @@ import java.time.Instant
 
 interface NomRepository : JpaRepository<NomEntity, Long> {
   @Query("SELECT n.fnr FROM NomEntity n WHERE n.navid = :navId AND (n.gyldigtil IS NULL OR n.gyldigtil >= CURRENT_DATE)")
-  fun finnGyldigAnsattFnr(navId: String): String?
-
-  fun  deleteByGyldigtilBefore(before: Instant) : Int
+  fun ansattFødselsnummer(navId: String): String?
+  fun deleteByGyldigtilBefore(before: Instant) : Int
   fun findByNavid(navId: String): NomEntity?
 }

src/main/kotlin/no/nav/tilgangsmaskin/populasjonstilgangskontroll/integrasjoner/nom/NomTjeneste.kt

@@ -22,6 +22,9 @@ class NomTjeneste(private val adapter: NomJPAAdapter) : NomOperasjoner {
 
     override fun ryddOpp() = adapter.ryddOpp()
 }
+/**
+ * NOM har en rekke fødsesnummer i dev som ikke finnes i PDL
+ */
 @ConditionalOnDev
 class NomDevTjeneste(adapter: NomJPAAdapter): NomTjeneste(adapter) {
     override fun fnrForAnsatt(ansattId: AnsattId) = null

src/main/kotlin/no/nav/tilgangsmaskin/populasjonstilgangskontroll/regelmotor/regler/OverstyrbareRegler.kt

@@ -17,7 +17,6 @@ import org.springframework.core.Ordered.LOWEST_PRECEDENCE
 import org.springframework.core.annotation.Order
 import org.springframework.stereotype.Component
 import java.time.LocalDate
-import java.time.Period
 import java.util.*
 
 @Component
@@ -61,19 +60,19 @@ class UtlandUdefinertGeoRegel(@Value("\${gruppe.utland}") private val id: UUID)
 
 @Component
 @Order(LOWEST_PRECEDENCE - 3)
-class AvdødBrukerRegel(private val handler: AvdødHandler) : Regel {
+class AvdødBrukerRegel(private val teller: AvdødAksessTeller) : Regel {
     override fun test(ansatt: Ansatt,bruker: Bruker) =
         bruker.dødsdato?.let {
-            handler.håndterAvdødBruker(ansatt.ansattId, bruker.brukerId, it)
+            teller.tellAvdødBruker(ansatt.ansattId, bruker.brukerId, it)
         } ?: true
     override val metadata = RegelBeskrivelse("Avdød bruker", AVVIST_AVDØD)
 }
 
 @Component
-class AvdødHandler(private val meterRegistry: MeterRegistry) {
+class AvdødAksessTeller(private val meterRegistry: MeterRegistry) {
 
     private val log = LoggerFactory.getLogger(javaClass)
-    fun håndterAvdødBruker(ansattId: AnsattId, brukerId: BrukerId, dødsdato: LocalDate) =
+    fun tellAvdødBruker(ansattId: AnsattId, brukerId: BrukerId, dødsdato: LocalDate) =
         true.also {  // TODO Endre til false når vi faktisk skal håndtere døde
             Counter.builder("dead.attempted.total")
                 .description("Number of deceased users attempted to be accessed")