Fikse og opprette tester

navikt · Nov 15, 2023 · 0b1db13 · 0b1db13
1 parent 99af4b0
commit 0b1db13
Show file tree

Hide file tree

Showing 2 changed files with 63 additions and 5 deletions.
diff --git a/src/main/kotlin/no/nav/bidrag/behandling/controller/BehandlingController.kt b/src/main/kotlin/no/nav/bidrag/behandling/controller/BehandlingController.kt
@@ -21,6 +21,7 @@ import no.nav.bidrag.behandling.transformers.toLocalDate
 import no.nav.bidrag.behandling.transformers.toRolle
 import no.nav.bidrag.behandling.transformers.toRolleTypeDto
 import no.nav.bidrag.behandling.transformers.toSivilstandDto
+import org.apache.commons.lang3.Validate
 import org.springframework.web.bind.annotation.GetMapping
 import org.springframework.web.bind.annotation.PathVariable
 import org.springframework.web.bind.annotation.PostMapping
@@ -53,7 +54,14 @@ class BehandlingController(private val behandlingService: BehandlingService) {
         @RequestBody(required = true)
         createBehandling: CreateBehandlingRequest,
     ): CreateBehandlingResponse {
-        validereRollerBarn(createBehandling.roller)
+        ingenBarnMedVerkenIdentEllerNavn(createBehandling.roller)
+
+        Validate.isTrue(
+            ingenBarnMedVerkenIdentEllerNavn(createBehandling.roller) &&
+                ingenVoksneUtenIdent(
+                    createBehandling.roller,
+                ),
+        )
 
         val behandling =
             Behandling(
@@ -218,8 +226,12 @@ class BehandlingController(private val behandlingService: BehandlingService) {
         return behandlingService.hentBehandlinger().map { behandlingDto(it.id!!, it) }
     }
 
-    private fun validereRollerBarn(roller: Set<CreateRolleDto>) {
-        roller.filter { r -> r.rolleType == CreateRolleRolleType.BARN && (r.ident == null || r.ident.isBlank()) }
-            .filter { r -> r.navn == null || r.navn.isBlank() }.size == 0
+    private fun ingenBarnMedVerkenIdentEllerNavn(roller: Set<CreateRolleDto>): Boolean {
+        return roller.filter { r -> r.rolleType == CreateRolleRolleType.BARN && r.ident.isNullOrBlank() }
+            .none { r -> r.navn.isNullOrBlank() }
+    }
+
+    private fun ingenVoksneUtenIdent(roller: Set<CreateRolleDto>): Boolean {
+        return roller.filter { r -> r.rolleType != CreateRolleRolleType.BARN && r.ident.isNullOrBlank() }.none()
     }
 }
diff --git a/src/test/kotlin/no/nav/bidrag/behandling/controller/BehandlingControllerTest.kt b/src/test/kotlin/no/nav/bidrag/behandling/controller/BehandlingControllerTest.kt
@@ -38,6 +38,7 @@ data class CreateRolleDtoTest(
     val rolleType: CreateRolleRolleType,
     val ident: String?,
     val opprettetDato: Date?,
+    val navn: String? = null,
 )
 
 @Suppress("NonAsciiCharacters")
@@ -276,7 +277,7 @@ class BehandlingControllerTest() : KontrollerTestRunner() {
     }
 
     @Test
-    fun `skal ikke opprette en behandling med rolle med null ident`() {
+    fun `skal ikke opprette behandling som inkluderer barn uten navn og ident`() {
         // given
         val roller =
             setOf(
@@ -298,6 +299,51 @@ class BehandlingControllerTest() : KontrollerTestRunner() {
         assertEquals(HttpStatus.BAD_REQUEST, responseMedNull.statusCode)
     }
 
+    @Test
+    fun `skal opprette behandling som inkluderer barn med navn men uten ident`() { // given
+        val roller =
+            setOf(
+                CreateRolleDtoTest(CreateRolleRolleType.BARN, null, Date(1), "Ola Dunk"),
+                CreateRolleDtoTest(CreateRolleRolleType.BIDRAGS_MOTTAKER, "123", Date(1)),
+            )
+        val testBehandlingMedNull = createBehandlingRequestTest("sak123", "en12", roller)
+
+        // when
+        val responseMedNull =
+            httpHeaderTestRestTemplate.exchange(
+                "${rootUri()}/behandling",
+                HttpMethod.POST,
+                HttpEntity(testBehandlingMedNull),
+                Void::class.java,
+            )
+
+        // then
+        assertEquals(HttpStatus.OK, responseMedNull.statusCode)
+    }
+
+    @Test
+    fun `skal ikke opprette behandling som inkluderer BP uten ident`() {
+        // given
+        val roller =
+            setOf(
+                CreateRolleDtoTest(CreateRolleRolleType.BARN, "1235", Date(1)),
+                CreateRolleDtoTest(CreateRolleRolleType.BIDRAGS_MOTTAKER, null, Date(1), "Ola Dunk"),
+            )
+        val testBehandlingMedNull = createBehandlingRequestTest("sak123", "en12", roller)
+
+        // when
+        val responseMedNull =
+            httpHeaderTestRestTemplate.exchange(
+                "${rootUri()}/behandling",
+                HttpMethod.POST,
+                HttpEntity(testBehandlingMedNull),
+                Void::class.java,
+            )
+
+        // then
+        assertEquals(HttpStatus.BAD_REQUEST, responseMedNull.statusCode)
+    }
+
     @Ignore
     @Test
     fun `skal ikke opprette en behandling med rolle med blank ident`() {