svarer med 200 og null fremfor 404 ved manglende sak/søknad

Author: di0nys1us

app/src/main/kotlin/no/nav/hjelpemidler/soknad/db/sak/SakApi.kt

@@ -1,26 +1,26 @@
 package no.nav.hjelpemidler.soknad.db.sak
 
-import io.ktor.http.HttpStatusCode
+import io.github.oshai.kotlinlogging.KotlinLogging
 import io.ktor.server.application.call
 import io.ktor.server.resources.get
-import io.ktor.server.response.respond
+import io.ktor.server.response.respondNullable
 import io.ktor.server.routing.Route
 import no.nav.hjelpemidler.behovsmeldingsmodell.sak.HotsakSakId
-import no.nav.hjelpemidler.soknad.db.exception.feilmelding
 import no.nav.hjelpemidler.soknad.db.store.Transaction
 
-fun Route.sakApi(
-    transaction: Transaction,
-) {
+private val logg = KotlinLogging.logger {}
+
+fun Route.sakApi(transaction: Transaction) {
     /**
      * NB! Kun saker fra Hotsak har en entydig sakId.
      */
     get<Saker.SakId> {
         val sakId = HotsakSakId(it.sakId)
-        val sak = transaction {
-            hotsakStore.finnSak(sakId)
-        } ?: return@get call.feilmelding(HttpStatusCode.NotFound, "Fant ikke sak med sakId: $sakId")
-        call.respond(sak)
+        val sak = transaction { hotsakStore.finnSak(sakId) }
+        if (sak == null) {
+            logg.info { "Fant ikke sak med sakId: $sakId" }
+        }
+        call.respondNullable(sak)
     }
 
     /**
@@ -31,7 +31,10 @@ fun Route.sakApi(
         val søknad = transaction {
             val sak = hotsakStore.finnSak(sakId) ?: return@transaction null
             søknadStore.finnSøknad(sak.søknadId, it.inkluderData)
-        } ?: return@get call.feilmelding(HttpStatusCode.NotFound, "Fant ikke søknad for sakId: $sakId")
-        call.respond(søknad)
+        }
+        if (søknad == null) {
+            logg.info { "Fant ikke søknad for sakId: $sakId" }
+        }
+        call.respondNullable(søknad)
     }
 }

app/src/main/kotlin/no/nav/hjelpemidler/soknad/db/soknad/SøknadApi.kt

@@ -8,6 +8,7 @@ import io.ktor.server.resources.get
 import io.ktor.server.resources.post
 import io.ktor.server.resources.put
 import io.ktor.server.response.respond
+import io.ktor.server.response.respondNullable
 import io.ktor.server.routing.Route
 import no.nav.hjelpemidler.behovsmeldingsmodell.BehovsmeldingStatus
 import no.nav.hjelpemidler.behovsmeldingsmodell.sak.Sakstilknytning
@@ -25,10 +26,12 @@ fun Route.søknadApi(
     val søknadService = serviceContext.søknadService
 
     get<Søknader.SøknadId> {
-        val søknad = transaction {
-            søknadStore.finnSøknad(it.søknadId, it.inkluderData)
-        } ?: return@get call.feilmelding(HttpStatusCode.NotFound, "Fant ikke søknad med søknadId: ${it.søknadId}")
-        call.respond(HttpStatusCode.OK, søknad)
+        val søknadId = it.søknadId
+        val søknad = transaction { søknadStore.finnSøknad(søknadId, it.inkluderData) }
+        if (søknad == null) {
+            logg.info { "Fant ikke søknad med søknadId: $søknadId" }
+        }
+        call.respondNullable(HttpStatusCode.OK, søknad)
     }
 
     put<Søknader.SøknadId.Journalpost> {

app/src/test/kotlin/no/nav/hjelpemidler/soknad/db/soknad/SøknadApiTest.kt

@@ -1,54 +1,63 @@
 package no.nav.hjelpemidler.soknad.db.soknad
 
 import io.kotest.matchers.nulls.shouldBeNull
-import io.kotest.matchers.should
+import io.kotest.matchers.nulls.shouldNotBeNull
 import io.kotest.matchers.shouldBe
 import io.ktor.client.plugins.resources.put
 import io.ktor.client.request.setBody
 import io.ktor.http.HttpStatusCode
 import no.nav.hjelpemidler.soknad.db.test.expect
 import no.nav.hjelpemidler.soknad.db.test.testApplication
+import java.util.UUID
 import kotlin.test.Test
 
 class SøknadApiTest {
     @Test
     fun `Skal hente søknad`() = testApplication {
         val søknadId = lagreSøknad().soknadId
-        val søknad = hentSøknad(søknadId, true)
-        søknad.søknadId shouldBe søknadId
+        finnSøknad(søknadId, true).shouldNotBeNull {
+            this.søknadId shouldBe søknadId
+        }
+    }
+
+    @Test
+    fun `Skal hente søknad som ikke finnes`() = testApplication {
+        val søknadId = UUID.randomUUID()
+        val søknad = finnSøknad(søknadId, true)
+        søknad shouldBe null
     }
 
     @Test
     fun `Skal oppdatere journalpostId`() = testApplication {
         val søknadId = lagreSøknad().soknadId
-        hentSøknad(søknadId).should {
-            it.journalpostId.shouldBeNull()
+        finnSøknad(søknadId).shouldNotBeNull {
+            journalpostId.shouldBeNull()
         }
         val journalpostId = "102030"
         client
             .put(Søknader.SøknadId.Journalpost(søknadId)) {
                 setBody(mapOf("journalpostId" to journalpostId))
             }
             .expect(HttpStatusCode.OK, 1)
-        hentSøknad(søknadId).should {
-            it.journalpostId shouldBe journalpostId
+        finnSøknad(søknadId).shouldNotBeNull {
+            journalpostId shouldBe journalpostId
         }
     }
 
     @Test
     fun `Skal oppdatere oppgaveId`() = testApplication {
         val søknadId = lagreSøknad().soknadId
-        hentSøknad(søknadId).should {
-            it.oppgaveId.shouldBeNull()
+        finnSøknad(søknadId).shouldNotBeNull {
+            oppgaveId.shouldBeNull()
         }
         val oppgaveId = "302010"
         client
             .put(Søknader.SøknadId.Oppgave(søknadId)) {
                 setBody(mapOf("oppgaveId" to oppgaveId))
             }
             .expect(HttpStatusCode.OK, 1)
-        hentSøknad(søknadId).should {
-            it.oppgaveId shouldBe oppgaveId
+        finnSøknad(søknadId).shouldNotBeNull {
+            oppgaveId shouldBe oppgaveId
         }
     }
 }

app/src/test/kotlin/no/nav/hjelpemidler/soknad/db/test/TestContext.kt

@@ -93,10 +93,10 @@ class TestContext(
         return søknad
     }
 
-    suspend fun hentSøknad(søknadId: SøknadId, inkluderData: Boolean = false): Søknad {
+    suspend fun finnSøknad(søknadId: SøknadId, inkluderData: Boolean = false): Søknad? {
         val response = client.get(Søknader.SøknadId(søknadId, inkluderData))
         response shouldHaveStatus HttpStatusCode.OK
-        return response.body()
+        return response.body<Søknad?>()
     }
 
     // fixme -> ny url