Håndter manglende felter i Azure nav ansatt

Finnes brukere i dev som mangler epost f. eks. Bare filtrerer de vekk i søk
navikt · Oct 3, 2024 · e5bf598 · e5bf598
1 parent 05da039
commit e5bf598
Show file tree

Hide file tree

Showing 5 changed files with 26 additions and 19 deletions.
diff --git a/...et-api/src/main/kotlin/no/nav/mulighetsrommet/api/clients/msgraph/MicrosoftGraphClient.kt b/...et-api/src/main/kotlin/no/nav/mulighetsrommet/api/clients/msgraph/MicrosoftGraphClient.kt
@@ -60,13 +60,13 @@ class MicrosoftGraphClient(
 
         val user = response.body<MsGraphUserDto>()
 
-        return toNavAnsatt(user)
+        return toNavAnsatt(user) ?: throw Exception("Ansatt med azureId ${user.id} manglet required felter")
     }
 
     suspend fun getNavAnsattByNavIdent(navIdent: NavIdent, accessType: AccessType): AzureAdNavAnsatt? {
         val response = client.get("$baseUrl/v1.0/users") {
             bearerAuth(tokenProvider.exchange(accessType))
-            parameter("\$filter", "onPremisesSamAccountName eq '$navIdent'")
+            parameter("\$search", "\"onPremisesSamAccountName:$navIdent\"")
             parameter("\$select", azureAdNavAnsattFields)
         }
 
@@ -102,10 +102,7 @@ class MicrosoftGraphClient(
 
         return response.body<GetUserSearchResponse>()
             .value
-            .filter { isNavAnsatt(it) }
-            .map {
-                toNavAnsatt(it)
-            }
+            .mapNotNull { toNavAnsatt(it) }
     }
 
     suspend fun getMemberGroups(navAnsattAzureId: UUID, accessType: AccessType): List<AdGruppe> {
@@ -142,7 +139,7 @@ class MicrosoftGraphClient(
 
         return result.value
             .filter { isNavAnsatt(it) }
-            .map { toNavAnsatt(it) }
+            .map { toNavAnsatt(it) ?: throw Exception("Ansatt med azureId ${it.id} manglet required felter") }
     }
 
     suspend fun addToGroup(objectId: UUID, groupId: UUID) {
@@ -164,23 +161,28 @@ class MicrosoftGraphClient(
 
     private fun toNavAnsatt(user: MsGraphUserDto) = when {
         user.onPremisesSamAccountName == null -> {
-            throw RuntimeException("NAVident mangler for bruker med id=${user.id}")
+            log.warn("NAVident mangler for bruker med id=${user.id}")
+            null
         }
-
         user.streetAddress == null -> {
-            throw RuntimeException("NAV Enhetskode mangler for bruker med id=${user.id}")
+            log.warn("NAV Enhetskode mangler for bruker med id=${user.id}")
+            null
         }
-
         user.city == null -> {
-            throw RuntimeException("NAV Enhetsnavn mangler for bruker med id=${user.id}")
+            log.warn("NAV Enhetsnavn mangler for bruker med id=${user.id}")
+            null
         }
-
         user.givenName == null -> {
-            throw RuntimeException("Fornavn på ansatt mangler for bruker med id=${user.id}")
+            log.warn("Fornavn på ansatt mangler for bruker med id=${user.id}")
+            null
         }
-
         user.surname == null -> {
-            throw RuntimeException("Etternavn på ansatt mangler for bruker med id=${user.id}")
+            log.warn("Etternavn på ansatt mangler for bruker med id=${user.id}")
+            null
+        }
+        user.mail == null -> {
+            log.warn("Epost på ansatt mangler for bruker med id=${user.id}")
+            null
         }
 
         else -> AzureAdNavAnsatt(

diff --git a/...api/src/main/kotlin/no/nav/mulighetsrommet/api/clients/msgraph/MsGraphDirectoryObjects.kt b/...api/src/main/kotlin/no/nav/mulighetsrommet/api/clients/msgraph/MsGraphDirectoryObjects.kt
@@ -29,7 +29,7 @@ internal data class MsGraphUserDto(
     /**
      * E-postadresse
      */
-    val mail: String,
+    val mail: String?,
     /**
      * NAV Enhetskode
      *

diff --git a/mulighetsrommet-api/src/main/kotlin/no/nav/mulighetsrommet/api/domain/dbo/NavAnsattDbo.kt b/mulighetsrommet-api/src/main/kotlin/no/nav/mulighetsrommet/api/domain/dbo/NavAnsattDbo.kt
@@ -14,10 +14,10 @@ data class NavAnsattDbo(
      */
     val hovedenhet: String,
     val azureId: UUID,
-    val mobilnummer: String? = null,
+    val mobilnummer: String?,
     val epost: String,
     val roller: Set<NavAnsattRolle>,
-    val skalSlettesDato: LocalDate? = null,
+    val skalSlettesDato: LocalDate?,
 ) {
     companion object {
         fun fromNavAnsattDto(dto: NavAnsattDto): NavAnsattDbo = NavAnsattDbo(

diff --git a/mulighetsrommet-api/src/test/kotlin/no/nav/mulighetsrommet/api/fixtures/NavAnsattFixture.kt b/mulighetsrommet-api/src/test/kotlin/no/nav/mulighetsrommet/api/fixtures/NavAnsattFixture.kt
@@ -19,6 +19,7 @@ object NavAnsattFixture {
             NavAnsattRolle.TILTAKSGJENNOMFORINGER_SKRIV,
             NavAnsattRolle.AVTALER_SKRIV,
         ),
+        skalSlettesDato = null,
     )
     val ansatt2: NavAnsattDbo = NavAnsattDbo(
         navIdent = NavIdent("DD2"),
@@ -33,5 +34,6 @@ object NavAnsattFixture {
             NavAnsattRolle.TILTAKSGJENNOMFORINGER_SKRIV,
             NavAnsattRolle.AVTALER_SKRIV,
         ),
+        skalSlettesDato = null,
     )
 }
diff --git a/...et-api/src/test/kotlin/no/nav/mulighetsrommet/api/repositories/NavAnsattRepositoryTest.kt b/...et-api/src/test/kotlin/no/nav/mulighetsrommet/api/repositories/NavAnsattRepositoryTest.kt
@@ -69,6 +69,7 @@ class NavAnsattRepositoryTest : FunSpec({
             mobilnummer = "12345678",
             epost = "[email protected]",
             roller = setOf(NavAnsattRolle.TILTAKADMINISTRASJON_GENERELL),
+            skalSlettesDato = null,
         )
 
         val ansatt2 = NavAnsattDbo(
@@ -80,6 +81,7 @@ class NavAnsattRepositoryTest : FunSpec({
             mobilnummer = "12345678",
             epost = "[email protected]",
             roller = setOf(NavAnsattRolle.KONTAKTPERSON),
+            skalSlettesDato = null,
         )
 
         val ansatt3 = NavAnsattDbo(
@@ -91,6 +93,7 @@ class NavAnsattRepositoryTest : FunSpec({
             mobilnummer = "12345678",
             epost = "[email protected]",
             roller = setOf(NavAnsattRolle.KONTAKTPERSON, NavAnsattRolle.TILTAKADMINISTRASJON_GENERELL),
+            skalSlettesDato = null,
         )
 
         test("CRUD") {