b - hentIdenter returner null dersom ønsket ident ikke finnes

Co-authored-by: Daniel.skarpas@nav.no
navikt · Sep 6, 2023 · d344021 · d344021
1 parent f586a1a
commit d344021
Show file tree

Hide file tree

Showing 2 changed files with 56 additions and 35 deletions.
diff --git a/src/main/kotlin/no/nav/eessi/pensjon/personoppslag/pdl/PersonService.kt b/src/main/kotlin/no/nav/eessi/pensjon/personoppslag/pdl/PersonService.kt
@@ -244,13 +244,13 @@ class PersonService(
         return hentIdentMetric.measure {
             val result = hentIdenter(ident)
                 .firstOrNull { it.gruppe == identTypeWanted }
-                ?.ident
+                ?.ident ?: return@measure null
 
             @Suppress("USELESS_CAST", "UNCHECKED_CAST")
             return@measure when (identTypeWanted) {
-                IdentGruppe.FOLKEREGISTERIDENT-> result?.let { NorskIdent(it) } as Ident
-                IdentGruppe.AKTORID-> result?.let { AktoerId(it) } as Ident
-                IdentGruppe.NPID -> result?.let { Npid(it) } as Ident
+                IdentGruppe.FOLKEREGISTERIDENT-> NorskIdent(result) as Ident
+                IdentGruppe.AKTORID-> AktoerId(result) as Ident
+                IdentGruppe.NPID -> Npid(result) as Ident
                 else -> null
             }
         }

diff --git a/src/test/kotlin/no/nav/eessi/pensjon/personoppslag/pdl/PersonServiceTest.kt b/src/test/kotlin/no/nav/eessi/pensjon/personoppslag/pdl/PersonServiceTest.kt
@@ -288,37 +288,58 @@ internal class PersonServiceTest {
 
     @Test
     fun hentIdent() {
-//        val identer = listOf(
-//                IdentInformasjon("1", AKTORID),
-//                IdentInformasjon("2", FOLKEREGISTERIDENT),
-//                IdentInformasjon("3", NPID)
-//        )
-//
-//        every { client.hentIdenter(any()) } returns IdenterResponse(IdenterDataResponse(HentIdenter(identer)))
-//
-//        // Hente ut NorskIdent med AktørID
-//        val norskIdentFraAktorId = service.hentIdent(IdentType.NorskIdent, AktoerId("1"))
-//        assertEquals("2", norskIdentFraAktorId.id)
-//
-//        // Hente ut NPID med AktørID
-//        val npidFraAktorId = service.hentIdent(IdentType.Npid, AktoerId("1"))
-//        assertEquals("3", npidFraAktorId.id)
-//
-//        // Hente ut AktørID med NorskIdent
-//        val aktoeridFraNorskIdent = service.hentIdent(IdentType.AktoerId, NorskIdent("2"))
-//        assertEquals("1", aktoeridFraNorskIdent.id)
-//
-//        // Hente ut NPID med NorskIdent
-//        val npidFraNorskIdent = service.hentIdent(IdentType.Npid, NorskIdent("2"))
-//        assertEquals("3", npidFraNorskIdent.id)
-//
-//        // Hente ut AktørID med Npid
-//        val aktoeridFraNpid = service.hentIdent(IdentType.AktoerId, Npid("2"))
-//        assertEquals("1", aktoeridFraNpid.id)
-//
-//        // Hente ut NorskIdent med Npid
-//        val norskIdentFraNpid = service.hentIdent(IdentType.NorskIdent, Npid("2"))
-//        assertEquals("2", norskIdentFraNpid.id)
+        val identer = listOf(
+            IdentInformasjon("1", AKTORID),
+            IdentInformasjon("2", FOLKEREGISTERIDENT),
+            IdentInformasjon("3", NPID)
+        )
+
+        every { client.hentIdenter(any()) } returns IdenterResponse(IdenterDataResponse(HentIdenter(identer)))
+
+        // Hente ut NorskIdent med AktørID
+        val tomNorskIdentFraAktorId = service.hentIdent(FOLKEREGISTERIDENT, AktoerId(""))
+        assertEquals("2", tomNorskIdentFraAktorId?.id)
+
+        // Hente ut NorskIdent med AktørID
+        val norskIdentFraAktorId = service.hentIdent(FOLKEREGISTERIDENT, AktoerId("1"))
+        assertEquals("2", norskIdentFraAktorId?.id)
+
+        // Hente ut NPID med AktørID
+        val npidFraAktorId = service.hentIdent(NPID, AktoerId("1"))
+        assertEquals("3", npidFraAktorId?.id)
+
+        // Hente ut AktørID med NorskIdent
+        val aktoeridFraNorskIdent = service.hentIdent(AKTORID, NorskIdent("2"))
+        assertEquals("1", aktoeridFraNorskIdent?.id)
+
+        // Hente ut NPID med NorskIdent
+        val npidFraNorskIdent = service.hentIdent(NPID, NorskIdent("2"))
+        assertEquals("3", npidFraNorskIdent?.id)
+
+        // Hente ut AktørID med Npid
+        val aktoeridFraNpid = service.hentIdent(AKTORID, Npid("2"))
+        assertEquals("1", aktoeridFraNpid?.id)
+
+        // Hente ut NorskIdent med Npid
+        val norskIdentFraNpid = service.hentIdent(FOLKEREGISTERIDENT, Npid("2"))
+        assertEquals("2", norskIdentFraNpid?.id)
+    }
+
+    @Test
+    fun `Gitt at vi forsøker å hente fnr eller npid for en tom aktoerId så returneres null`() {
+        every { client.hentIdenter(any()) } returns IdenterResponse(IdenterDataResponse(HentIdenter(emptyList())))
+
+        // Hente ut FOLKEREGISTERIDENT med tom AktørID returnerer null
+        val tomNorskIdentFraAktorId = service.hentIdent(FOLKEREGISTERIDENT, AktoerId(""))
+        assertEquals(null, tomNorskIdentFraAktorId?.id)
+
+        // Hente ut FOLKEREGISTERIDENT med tom AktørID returnerer null
+        val tomNpidIdentFraAktorId = service.hentIdent(FOLKEREGISTERIDENT, AktoerId(""))
+        assertEquals(null, tomNpidIdentFraAktorId?.id)
+
+        // Hente ut NPID med ugyldig AktørID returnerer også null
+        val tomNpidIdentForAktorId = service.hentIdent(FOLKEREGISTERIDENT, AktoerId("987654"))
+        assertEquals(null, tomNpidIdentForAktorId?.id)
     }
 
     @Test