Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Skal kunne bruke personident ved opprettelse av oppgaver for å unngå … #1147

Merged
merged 1 commit into from
Aug 1, 2024
Merged

Skal kunne bruke personident ved opprettelse av oppgaver for å unngå … #1147

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 0 additions & 6 deletions src/main/kotlin/no/nav/familie/ef/mottak/integration/IntegrasjonerClient.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,6 @@ import no.nav.familie.kontrakter.felles.oppgave.Oppgave
import no.nav.familie.kontrakter.felles.oppgave.OppgaveResponse
import no.nav.familie.kontrakter.felles.oppgave.Oppgavetype
import no.nav.familie.kontrakter.felles.oppgave.OpprettOppgaveRequest
import no.nav.familie.kontrakter.felles.personopplysning.Ident
import org.springframework.beans.factory.annotation.Qualifier
import org.springframework.retry.annotation.Backoff
import org.springframework.retry.annotation.Retryable
Expand Down Expand Up @@ -214,11 +213,6 @@ class IntegrasjonerClient(
return response.getDataOrThrow()["saksnummer"].toString()
}

fun hentAktørId(personident: String): String {
val response = postForEntity<Ressurs<MutableMap<*, *>>>(aktørUri, Ident(personident))
return response.getDataOrThrow()["aktørId"].toString()
}

fun hentIdentForAktørId(aktørId: String): String {
val response = postForEntity<Ressurs<MutableMap<*, *>>>(identFraAktørUri, aktørId)
return response.getDataOrThrow()["personIdent"].toString()
Expand Down
9 changes: 2 additions & 7 deletions src/main/kotlin/no/nav/familie/ef/mottak/mapper/OpprettOppgaveMapper.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
package no.nav.familie.ef.mottak.mapper

import no.nav.familie.ef.mottak.integration.IntegrasjonerClient
import no.nav.familie.kontrakter.felles.BrukerIdType
import no.nav.familie.kontrakter.felles.Tema
import no.nav.familie.kontrakter.felles.journalpost.Journalpost
Expand All @@ -17,9 +16,7 @@ import java.time.LocalDate
import java.time.LocalDateTime

@Component
class OpprettOppgaveMapper(
private val integrasjonerClient: IntegrasjonerClient,
) {
class OpprettOppgaveMapper {
val logger: Logger = LoggerFactory.getLogger(javaClass)

fun toJournalføringsoppgave(
Expand Down Expand Up @@ -58,9 +55,7 @@ class OpprettOppgaveMapper(
}

return when (journalpost.bruker!!.type) {
BrukerIdType.FNR -> {
OppgaveIdentV2(ident = integrasjonerClient.hentAktørId(journalpost.bruker!!.id), gruppe = IdentGruppe.AKTOERID)
}
BrukerIdType.FNR -> OppgaveIdentV2(ident = journalpost.bruker!!.id, gruppe = IdentGruppe.FOLKEREGISTERIDENT)
BrukerIdType.ORGNR -> OppgaveIdentV2(ident = journalpost.bruker!!.id, gruppe = IdentGruppe.ORGNR)
BrukerIdType.AKTOERID -> OppgaveIdentV2(ident = journalpost.bruker!!.id, gruppe = IdentGruppe.AKTOERID)
}
Expand Down
3 changes: 1 addition & 2 deletions src/test/kotlin/no/nav/familie/ef/mottak/mapper/OpprettOppgaveMapperTest.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,13 +1,12 @@
package no.nav.familie.ef.mottak.mapper

import io.mockk.mockk
import org.assertj.core.api.Assertions.assertThat
import org.junit.jupiter.api.Test
import java.time.LocalDate
import java.time.LocalDateTime

internal class OpprettOppgaveMapperTest {
private val mapper = OpprettOppgaveMapper(mockk())
private val mapper = OpprettOppgaveMapper()

@Test
internal fun `skal sette frist til 1 dag for journalføringsoppgave opprettet før kl 12`() {
Expand Down
2 changes: 0 additions & 2 deletions src/test/kotlin/no/nav/familie/ef/mottak/mockapi/MockConfiguration.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,5 @@ class MockConfiguration {
override fun lagOppgave(opprettOppgaveRequest: OpprettOppgaveRequest): OppgaveResponse = OppgaveResponse(1)

override fun hentSaksnummer(journalPostId: String): String = "sak1"

override fun hentAktørId(personident: String): String = "aktørId"
}
}
3 changes: 1 addition & 2 deletions src/test/kotlin/no/nav/familie/ef/mottak/service/OppgaveServiceTest.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ import kotlin.test.assertEquals
internal class OppgaveServiceTest {
private val integrasjonerClient: IntegrasjonerClient = mockk()
private val søknadService: SøknadService = mockk()
private val opprettOppgaveMapper = spyk(OpprettOppgaveMapper(integrasjonerClient))
private val opprettOppgaveMapper = spyk(OpprettOppgaveMapper())
private val ettersendingService = mockk<EttersendingService>()
private val featureToggleService = mockk<FeatureToggleService>()
private val cacheManager = ConcurrentMapCacheManager()
Expand All @@ -80,7 +80,6 @@ internal class OppgaveServiceTest {

@BeforeEach
fun setUp() {
every { integrasjonerClient.hentAktørId(any()) } returns Testdata.randomAktørId()
every { integrasjonerClient.hentIdentForAktørId(any()) } returns Testdata.randomFnr()
every { integrasjonerClient.finnBehandlendeEnhetForPersonMedRelasjoner(any()) } returns
listOf(
Expand Down
Loading