diff --git a/src/main/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/NasjonalOppgaveService.kt b/src/main/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/NasjonalOppgaveService.kt index c1303f18..eef6f1a8 100644 --- a/src/main/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/NasjonalOppgaveService.kt +++ b/src/main/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/NasjonalOppgaveService.kt @@ -67,10 +67,6 @@ class NasjonalOppgaveService( return updated } - fun isValidOppgaveId(oppgaveId: String): Boolean { - val regex = Regex("^\\d{9}$|^[a-zA-Z0-9]{1,20}$") - return oppgaveId.matches(regex) - } fun findByOppgaveId(oppgaveId: String): NasjonalManuellOppgaveDAO? { if(!isValidOppgaveId(oppgaveId)) @@ -269,3 +265,8 @@ private fun getLoggingMeta(sykmeldingId: String, oppgave: NasjonalManuellOppgave ) } } + +fun isValidOppgaveId(oppgaveId: String): Boolean { + val regex = Regex("^\\d{9}$|^[a-zA-Z0-9]{1,20}$") + return oppgaveId.matches(regex) +} \ No newline at end of file diff --git a/src/main/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/api/SmregistreringClient.kt b/src/main/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/api/SmregistreringClient.kt index 3b401e1e..cf33605f 100644 --- a/src/main/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/api/SmregistreringClient.kt +++ b/src/main/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/api/SmregistreringClient.kt @@ -5,6 +5,7 @@ import no.nav.sykdig.digitalisering.papirsykmelding.api.model.AvvisSykmeldingReq import no.nav.sykdig.digitalisering.papirsykmelding.api.model.PapirManuellOppgave import no.nav.sykdig.digitalisering.papirsykmelding.api.model.SmRegistreringManuell import no.nav.sykdig.digitalisering.papirsykmelding.api.model.Sykmelder +import no.nav.sykdig.digitalisering.papirsykmelding.isValidOppgaveId import org.springframework.beans.factory.annotation.Value import org.springframework.http.HttpEntity import org.springframework.http.HttpHeaders @@ -57,6 +58,8 @@ class SmregistreringClient( authorization: String, oppgaveId: String, ): ResponseEntity { + if(!isValidOppgaveId(oppgaveId)) + throw IllegalArgumentException("Invalid oppgaveId does not contain only alphanumerical characters. oppgaveId: $oppgaveId") val headers = HttpHeaders() headers.contentType = MediaType.APPLICATION_JSON headers.setBearerAuth(removeBearerPrefix(authorization)) diff --git a/src/test/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/NasjonalOppgaveServiceTest.kt b/src/test/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/NasjonalOppgaveServiceTest.kt index f45a54bf..8f52a7cb 100644 --- a/src/test/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/NasjonalOppgaveServiceTest.kt +++ b/src/test/kotlin/no/nav/sykdig/digitalisering/papirsykmelding/NasjonalOppgaveServiceTest.kt @@ -35,6 +35,8 @@ import org.springframework.beans.factory.annotation.Autowired import org.springframework.beans.factory.annotation.Qualifier import org.springframework.boot.test.mock.mockito.MockBean import org.springframework.http.HttpMethod +import org.springframework.http.HttpStatus +import org.springframework.http.HttpStatusCode import org.springframework.http.MediaType import org.springframework.security.core.context.ReactiveSecurityContextHolder import org.springframework.security.core.context.SecurityContextHolder @@ -131,9 +133,7 @@ class NasjonalOppgaveServiceTest : IntegrationTest() { assertTrue(originalOppgave.avvisningsgrunn == null) val avvistOppgave = nasjonalOppgaveService.avvisOppgave(oppgaveId, request, "enhet") - assertEquals(testDataNasjonalManuellOppgaveDAO(null, "456", oppgaveId).oppgaveId, avvistOppgave.body?.oppgaveId ?: 123) - assertTrue(avvistOppgave.body?.avvisningsgrunn == "MANGLENDE_DIAGNOSE") - assertEquals(avvistOppgave.body?.id, originalOppgave.id) + assertEquals(avvistOppgave.statusCode, HttpStatus.NO_CONTENT) }