Skip to content

Commit

Permalink
chore: changing http status return type of avvist
Browse files Browse the repository at this point in the history
  • Loading branch information
jaflaten committed Dec 3, 2024
1 parent 1373a51 commit ff7e714
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,7 @@ class NasjonalOppgaveService(
request: String,
authorization: String,
navEnhet: String,
): ResponseEntity<PapirManuellOppgave> {
): ResponseEntity<NasjonalManuellOppgaveDAO> {
val eksisterendeOppgave = nasjonalOppgaveRepository.findByOppgaveId(oppgaveId)

val avvisningsgrunn = mapper.readValue(request, AvvisSykmeldingRequest::class.java).reason
Expand All @@ -115,8 +115,7 @@ class NasjonalOppgaveService(
)

log.info("Har avvist oppgave med oppgaveId $oppgaveId")
requireNotNull(res)
return ResponseEntity(mapFromDao(res), HttpStatus.OK)
return ResponseEntity(HttpStatus.NO_CONTENT)
} else {
log.info("fant ikke oppgave som skulle avvises")
return ResponseEntity(HttpStatus.NOT_FOUND)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ class NasjonalOppgaveController(
@RequestHeader("Authorization") authorization: String,
@RequestHeader("X-Nav-Enhet") navEnhet: String,
@RequestBody avvisSykmeldingRequest: String,
): ResponseEntity<PapirManuellOppgave> {
): ResponseEntity<NasjonalManuellOppgaveDAO> {
log.info("Forsøker å avvise oppgave med oppgaveId: $oppgaveId")
return nasjonalOppgaveService.avvisOppgave(oppgaveId.toInt(), avvisSykmeldingRequest, authorization, navEnhet)
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -122,9 +122,9 @@ class NasjonalOppgaveServiceTest : IntegrationTest() {

assertTrue(originalOppgave.avvisningsgrunn == null)
val avvistOppgave = nasjonalOppgaveService.avvisOppgave(oppgaveId, request, "auth streng", "enhet")
assertEquals(testDataNasjonalManuellOppgaveDAO(null, "456", oppgaveId).oppgaveId, avvistOppgave.body.oppgaveId)
// assertTrue(avvistOppgave.body?.avvisningsgrunn == "MANGLENDE_DIAGNOSE")
// assertEquals(avvistOppgave.body?.id, originalOppgave.id)
assertEquals(testDataNasjonalManuellOppgaveDAO(null, "456", oppgaveId).oppgaveId, avvistOppgave.body?.oppgaveId ?: 123)
assertTrue(avvistOppgave.body?.avvisningsgrunn == "MANGLENDE_DIAGNOSE")
assertEquals(avvistOppgave.body?.id, originalOppgave.id)

}

Expand All @@ -147,12 +147,14 @@ class NasjonalOppgaveServiceTest : IntegrationTest() {
}

@Test
fun `oppgave isPresent`() = runBlocking {
fun `oppgave blir lagret`() = runBlocking {
val uuid = UUID.randomUUID()
val dao = testDataNasjonalManuellOppgaveDAO(uuid, "123", 123)
val oppgave = nasjonalOppgaveService.lagreOppgave(testDataPapirManuellOppgave())

assertEquals(oppgave.sykmeldingId, dao.sykmeldingId)
val res = nasjonalOppgaveRepository.findBySykmeldingId(oppgave.sykmeldingId)
println(res)
}

fun testDataPapirManuellOppgave(): PapirManuellOppgave {
Expand Down

0 comments on commit ff7e714

Please sign in to comment.