Skip to content

Commit

Permalink
T - Oppdatering av tester for hentsak
Browse files Browse the repository at this point in the history
  • Loading branch information
dskarpas committed Sep 27, 2024
1 parent 11dc02b commit e83eeef
Show file tree
Hide file tree
Showing 12 changed files with 82 additions and 38 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@ import no.nav.eessi.pensjon.eux.model.document.SedDokumentfiler
import no.nav.eessi.pensjon.eux.model.document.SedVedlegg
import no.nav.eessi.pensjon.gcp.GcpStorageService
import no.nav.eessi.pensjon.integrasjonstest.saksflyt.JournalforingTestBase
import no.nav.eessi.pensjon.journalforing.HentSakService
import no.nav.eessi.pensjon.journalforing.OpprettJournalpostRequest
import no.nav.eessi.pensjon.journalforing.VurderBrukerInfo
import no.nav.eessi.pensjon.journalforing.etterlatte.EtterlatteService
Expand Down Expand Up @@ -88,8 +89,8 @@ internal class ConfigRestTemplateTest {
@MockkBean(relaxed = true)
private lateinit var etterlatteService: EtterlatteService

// @MockkBean(relaxed = true)
// private lateinit var hentSakService: HentSakService
@MockkBean(relaxed = true)
private lateinit var hentSakService: HentSakService

@MockkBean(relaxed = true)
private lateinit var safClient: SafClient
Expand Down Expand Up @@ -120,7 +121,7 @@ internal class ConfigRestTemplateTest {
every { gcpStorageService.gjennyFinnes(any())} returns false
every { gcpStorageService.journalFinnes(any())} returns false
every { gcpStorageService.arkiverteSakerForRinaId(any(), any()) } returns emptyList()
// every { hentSakService.hentSak("147666") } returns mockk(relaxed = true)
every { hentSakService.hentSak("147666") } returns mockk(relaxed = true)
}


Expand Down Expand Up @@ -153,6 +154,10 @@ internal class ConfigRestTemplateTest {
@Primary
fun euxKlientLab(): EuxKlientLib = EuxKlientLib(euxOAuthRestTemplate())

@Bean
@Primary
fun hentSakService(): HentSakService = mockk(relaxed = true)

@Bean
@Primary
fun euxOAuthRestTemplate(): RestTemplate {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ import no.nav.eessi.pensjon.EessiPensjonJournalforingTestApplication
import no.nav.eessi.pensjon.eux.klient.EuxKlientLib
import no.nav.eessi.pensjon.eux.model.buc.Buc
import no.nav.eessi.pensjon.gcp.GcpStorageService
import no.nav.eessi.pensjon.journalforing.HentSakService
import no.nav.eessi.pensjon.journalforing.VurderBrukerInfo
import no.nav.eessi.pensjon.journalforing.saf.SafClient
import no.nav.eessi.pensjon.utils.toJson
Expand Down Expand Up @@ -36,6 +37,9 @@ internal class SedSendtIntegrationTest : IntegrasjonsBase() {
@Autowired
private lateinit var gcpStorageService: GcpStorageService

@Autowired
private lateinit var hentSakService: HentSakService

init {
if (System.getProperty("mockServerport") == null) {
mockServer = ClientAndServer(Configuration().apply {
Expand All @@ -54,7 +58,7 @@ internal class SedSendtIntegrationTest : IntegrasjonsBase() {
justRun { gcpStorageService.lagreJournalpostDetaljer(any(), any(), any(), any(), any()) }
every { gcpStorageService.hentFraJournal(any()) } returns null
every { gcpStorageService.arkiverteSakerForRinaId(any(), any()) } returns emptyList()
// every { hentSakService.hentSak(any(), any(), any(), any()) } returns mockk(relaxed = true)
every { gcpStorageService.hentFraGjenny(any()) } returns null
}

@TestConfiguration
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -144,7 +144,16 @@ internal open class JournalforingTestBase {
}
private val statistikkPublisher = StatistikkPublisher(automatiseringHandlerKafka)

protected val gcpStorageService : GcpStorageService = mockk(relaxed = true)
protected val gcpStorageService : GcpStorageService = mockk<GcpStorageService>()

@BeforeEach
fun setup() {
ReflectionTestUtils.setField(kravHandler, "kravTopic", "kravTopic")
journalforingService.nameSpace = "test"
every { gcpStorageService.journalFinnes(any()) } returns false
every { gcpStorageService.gjennyFinnes(any()) } returns false
every { gcpStorageService.hentFraGjenny(any()) } returns null
}

val hentSakService = HentSakService(etterlatteService, gcpStorageService)
val journalforingService: JournalforingService = JournalforingService(
Expand Down Expand Up @@ -188,14 +197,6 @@ internal open class JournalforingTestBase {
navansattKlient = navansattKlient
)


@BeforeEach
fun setup() {
ReflectionTestUtils.setField(kravHandler, "kravTopic", "kravTopic")
journalforingService.nameSpace = "test"
every { gcpStorageService.gjennyFinnes(any()) } returns false
}

@AfterEach
fun after() {
clearAllMocks()
Expand Down Expand Up @@ -229,6 +230,9 @@ internal open class JournalforingTestBase {

every { personService.harAdressebeskyttelse(any()) } returns harAdressebeskyttelse
every { navansattKlient.navAnsattMedEnhetsInfo(any(), any()) } returns null
every { gcpStorageService.journalFinnes(any()) } returns false
every { gcpStorageService.gjennyFinnes(any()) } returns false
every { gcpStorageService.hentFraGjenny(any()) } returns null

sakId?.let {
every { etterlatteService.hentGjennySak(any()) } returns mockHentGjennySak(it)
Expand Down Expand Up @@ -358,6 +362,10 @@ internal open class JournalforingTestBase {
every { navansattKlient.navAnsattMedEnhetsInfo(any(), any()) } returns null
every { journalpostKlient.oppdaterDistribusjonsinfo(any()) } returns Unit
every { etterlatteService.hentGjennySak(any()) } returns mockHentGjennySak(sakId)
every { gcpStorageService.journalFinnes(any()) } returns false
every { gcpStorageService.gjennyFinnes(any()) } returns false
every { gcpStorageService.hentFraGjenny(any()) } returns null


val (journalpost, _) = initJournalPostRequestSlot(true)

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@ import no.nav.eessi.pensjon.utils.mapJsonToAny
import org.junit.jupiter.api.*
import org.junit.jupiter.params.ParameterizedTest
import org.junit.jupiter.params.provider.EnumSource
import org.springframework.test.util.ReflectionTestUtils
import java.util.*

@DisplayName("P_BUC_02 – IntegrationTest")
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -32,11 +32,10 @@ import no.nav.eessi.pensjon.personoppslag.pdl.model.NorskIdent
import no.nav.eessi.pensjon.shared.person.Fodselsnummer
import no.nav.eessi.pensjon.utils.mapJsonToAny
import no.nav.eessi.pensjon.utils.toJson
import org.junit.jupiter.api.Assertions
import org.junit.jupiter.api.*
import org.junit.jupiter.api.Assertions.assertEquals
import org.junit.jupiter.api.DisplayName
import org.junit.jupiter.api.Nested
import org.junit.jupiter.api.Test
import org.springframework.test.context.event.annotation.BeforeTestClass
import org.springframework.test.util.ReflectionTestUtils

@DisplayName("P_BUC_05 - IntegrationTest")
internal class PBuc05IntegrationTest : JournalforingTestBase() {
Expand Down Expand Up @@ -1242,6 +1241,7 @@ internal class PBuc05IntegrationTest : JournalforingTestBase() {
@Nested
@DisplayName("Inngående - Scenario 3")
inner class Scenario3Inngaende {

@Test
fun `Manglende eller feil FNR-DNR - to personer angitt - etterlatte mangler fnr medfører bruk av sokPerson`() {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -372,6 +372,7 @@ internal class PBuc10IntegrationTest : JournalforingTestBase() {
@Test
fun `Krav om gjenlevendeytelse - flere sakstyper i retur - sakid finnes i sed - maskinell journalføring`() {
val bestemsak = BestemSakResponse(null, listOf(sakInformasjon(), sakInformasjon("123456", SakType.UFOREP)))
every { gcpStorageService.hentFraGjenny(any()) } returns null

testRunnerBarn(
FNR_VOKSEN_UNDER_62,
Expand All @@ -394,6 +395,7 @@ internal class PBuc10IntegrationTest : JournalforingTestBase() {
@Test
fun `Krav om gjenlevendeytelse - flere sakstyper i retur - ingen saker - id og fordeling`() {
val bestemsak = null
every { gcpStorageService.hentFraGjenny(any()) } returns null

testRunnerBarn(
FNR_VOKSEN_UNDER_62,
Expand Down Expand Up @@ -433,6 +435,7 @@ internal class PBuc10IntegrationTest : JournalforingTestBase() {

@Test
fun `mangler som fører til manuell oppgave - etterlatteytelser`() {

testRunnerBarnUtenOppgave(FNR_VOKSEN_2, null, krav = GJENLEV, alleDocs = allDocuemtActions, relasjonAvod = RelasjonTilAvdod.EKTEFELLE, hendelseType = SENDT) {
assertEquals(PENSJON, it.tema)
//TODO: Kan vi finne en smartere måte å finne behandlingstema for tilfeller der person ikke er identifiserbar og SED har pesysSakid i seg
Expand Down Expand Up @@ -894,6 +897,9 @@ internal class PBuc10IntegrationTest : JournalforingTestBase() {
?: SED.generateSedToClass(createSedPensjon(SedType.P15000, fnrVoksen, eessiSaknr = sakId, gjenlevendeFnr = fnrBarn, krav = krav, pdlPerson = sokPerson , relasjon = relasjonAvod))

initCommonMocks(sed, alleDocs)
every { gcpStorageService.journalFinnes(any()) } returns false
every { gcpStorageService.gjennyFinnes(any()) } returns false
every { gcpStorageService.hentFraGjenny(any()) } returns null

every { personService.hentPerson(NorskIdent(fnrVoksen)) } returns createBrukerWith(fnrVoksen, "Mamma forsørger", "Etternavn", land, aktorId = AKTOER_ID)
every { navansattKlient.navAnsattMedEnhetsInfo(any(), any()) } returns null
Expand Down Expand Up @@ -1064,6 +1070,9 @@ internal class PBuc10IntegrationTest : JournalforingTestBase() {
if (benyttSokPerson) {
every { personService.sokPerson(any()) } returns setOf(IdentInformasjon(fnrBarn, IdentGruppe.FOLKEREGISTERIDENT), IdentInformasjon("BLÆ", IdentGruppe.AKTORID))
}
every { gcpStorageService.journalFinnes(any()) } returns false
every { gcpStorageService.gjennyFinnes(any()) } returns false
every { gcpStorageService.hentFraGjenny(any()) } returns null

every { personService.hentPerson(NorskIdent(fnrVoksen)) } returns createBrukerWith(fnrVoksen, "Mamma forsørger", "Etternavn", land, aktorId = AKTOER_ID)
every { personService.hentPerson(NorskIdent(fnrBarn)) } returns mockBarn
Expand Down Expand Up @@ -1117,6 +1126,9 @@ internal class PBuc10IntegrationTest : JournalforingTestBase() {
) {
val sed = SED.generateSedToClass<P15000>(createSedPensjon(SedType.P15000, fnrVoksen, eessiSaknr = sakId, gjenlevendeFnr = fnrVoksenSoker, krav = krav, relasjon = relasjonAvod))
initCommonMocks(sed, alleDocs)
every { gcpStorageService.hentFraGjenny(any()) } returns null
every { gcpStorageService.journalFinnes(any()) } returns false
every { gcpStorageService.gjennyFinnes(any()) } returns false

every { personService.hentPerson(NorskIdent(fnrVoksen)) } returns createBrukerWith(fnrVoksen, "Voksen ", "Forsikret", land, aktorId = AKTOER_ID)
sakId?.let {
Expand Down Expand Up @@ -1188,6 +1200,9 @@ internal class PBuc10IntegrationTest : JournalforingTestBase() {

every { navansattKlient.navAnsattMedEnhetsInfo(any(), any()) } returns null
every { journalpostKlient.oppdaterDistribusjonsinfo(any()) } returns Unit
every { gcpStorageService.journalFinnes(any()) } returns false
every { gcpStorageService.gjennyFinnes(any()) } returns false
every { gcpStorageService.hentFraGjenny(any()) } returns null

sakId?.let {
every { etterlatteService.hentGjennySak(any()) } returns mockHentGjennySak(it)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ abstract class JournalforingServiceBase {
val pdfService = mockk<PDFService>()
val oppgaveHandler = mockk<OppgaveHandler>(relaxed = true)
val kravHandeler = mockk<KravInitialiseringsHandler>()
val gcpStorageService = mockk<GcpStorageService>(relaxed = true)
val gcpStorageService = mockk<GcpStorageService>()
val kravService = KravInitialiseringsService(kravHandeler)
val etterlatteService = mockk<EtterlatteService>()
val hentSakService = HentSakService(etterlatteService, gcpStorageService)
Expand Down Expand Up @@ -67,14 +67,16 @@ abstract class JournalforingServiceBase {

//MOCK RESPONSES
every { pdfService.hentDokumenterOgVedlegg(any(), any(), any()) } returns Pair("Supported Documents", emptyList())
every { gcpStorageService.journalFinnes(any()) } returns false
// every { gcpStorageService.journalFinnes(any()) } returns false
every { gcpStorageService.gjennyFinnes(any()) } returns false
val opprettJournalPostResponse = OpprettJournalPostResponse(
journalpostId = "12345",
journalstatus = "EKSPEDERT",
melding = "",
journalpostferdigstilt = false,
)
every { etterlatteService.hentGjennySak(any()) } returns JournalforingTestBase.mockHentGjennySak("123456789")
// every { etterlatteService.hentGjennySak(any()) } returns JournalforingTestBase.mockHentGjennySak("123456789")
// every { gcpStorageService.hentFraGjenny(any()) } returns null

every { journalpostKlient.opprettJournalpost(capture(opprettJournalpostRequestCapturingSlot), any(), null) } returns opprettJournalPostResponse
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,14 +23,11 @@ class JournalforingServiceHentSakTest : JournalforingServiceBase() {
val gjennySakJson = """{ "sakId": "sakId123", "sakType": "EY"}""".trimIndent()
val fnr = Fodselsnummer.fra(FodselsnummerGenerator.generateFnrForTest(20))


every { gcpStorageService.gjennyFinnes(euxCaseId) } returns true
every { gcpStorageService.hentFraGjenny(euxCaseId) } returns gjennySakJson

val result = hentSakService.hentSak(euxCaseId, identifisertPersonFnr = fnr)

assertEquals(Sak("FAGSAK", "sakId123", "EY"), result)
verify { gcpStorageService.gjennyFinnes(euxCaseId) }
verify { gcpStorageService.hentFraGjenny(euxCaseId) }
}

Expand All @@ -39,12 +36,10 @@ class JournalforingServiceHentSakTest : JournalforingServiceBase() {
val euxCaseId = "123"
val gjennySakJson = """{ "sakId": "sakId123", "sakType": "EY"}""".trimIndent()

every { gcpStorageService.gjennyFinnes(euxCaseId) } returns true
every { gcpStorageService.hentFraGjenny(euxCaseId) } returns gjennySakJson

assertNull(hentSakService.hentSak(euxCaseId))

verify (exactly = 0) { gcpStorageService.gjennyFinnes(euxCaseId) }
verify (exactly = 0) { gcpStorageService.hentFraGjenny(euxCaseId) }
}
@Test
Expand All @@ -53,13 +48,13 @@ class JournalforingServiceHentSakTest : JournalforingServiceBase() {
val sakIdFraSed = "12131132"
val fnr = Fodselsnummer.fra(FodselsnummerGenerator.generateFnrForTest(20))

every { gcpStorageService.gjennyFinnes(euxCaseId) } returns false
every { gcpStorageService.hentFraGjenny(euxCaseId) } returns null
every { etterlatteService.hentGjennySak(eq(sakIdFraSed)) } returns JournalforingTestBase.mockHentGjennySakMedError()

every { etterlatteService.hentGjennySak(eq("123456789")) } returns JournalforingTestBase.mockHentGjennySakMedError()
val result = hentSakService.hentSak(euxCaseId, sakIdFraSed, identifisertPersonFnr = fnr)

assertEquals(Sak("FAGSAK", sakIdFraSed, "PP01"), result)
verify { gcpStorageService.gjennyFinnes(euxCaseId) }
verify { gcpStorageService.hentFraGjenny(euxCaseId) }
}

@Test
Expand All @@ -68,25 +63,25 @@ class JournalforingServiceHentSakTest : JournalforingServiceBase() {
val sakInformasjon = SakInformasjon("12131223", SakType.GJENLEV, LOPENDE)
val fnr = Fodselsnummer.fra(FodselsnummerGenerator.generateFnrForTest(20))

every { gcpStorageService.gjennyFinnes(euxCaseId) } returns false
every { gcpStorageService.hentFraGjenny(euxCaseId) } returns null

val result = hentSakService.hentSak(euxCaseId, sakInformasjon = sakInformasjon, identifisertPersonFnr = fnr)

assertEquals(Sak("FAGSAK", sakInformasjon.sakId!!, "PP01"), result)
verify { gcpStorageService.gjennyFinnes(euxCaseId) }
verify { gcpStorageService.hentFraGjenny(euxCaseId) }
}

@Test
fun `hentSak skal gi null when naar alt mangler `() {
fun `hentSak skal gi null naar alt mangler `() {
val euxCaseId = "123"
val fnr = Fodselsnummer.fra(FodselsnummerGenerator.generateFnrForTest(20))

every { gcpStorageService.gjennyFinnes(euxCaseId) } returns false
every { gcpStorageService.hentFraGjenny(euxCaseId) } returns null

val result = hentSakService.hentSak(euxCaseId, identifisertPersonFnr = fnr)

assertNull(result)
verify { gcpStorageService.gjennyFinnes(euxCaseId) }
verify { gcpStorageService.hentFraGjenny(euxCaseId) }
}

@Test
Expand All @@ -95,7 +90,7 @@ class JournalforingServiceHentSakTest : JournalforingServiceBase() {
val sakInformasjon = SakInformasjon("00000000000", SakType.GJENLEV, LOPENDE)
val fnr = Fodselsnummer.fra(FodselsnummerGenerator.generateFnrForTest(20))

every { gcpStorageService.gjennyFinnes(euxCaseId) } returns false
every { gcpStorageService.hentFraGjenny(euxCaseId) } returns null

val result = hentSakService.hentSak(euxCaseId, sakInformasjon = sakInformasjon, identifisertPersonFnr = fnr)

Expand All @@ -108,7 +103,9 @@ class JournalforingServiceHentSakTest : JournalforingServiceBase() {
val sakIdFraSed = "000000000000"
val fnr = Fodselsnummer.fra(FodselsnummerGenerator.generateFnrForTest(20))

every { gcpStorageService.gjennyFinnes(euxCaseId) } returns false
every { etterlatteService.hentGjennySak(eq(sakIdFraSed)) } returns JournalforingTestBase.mockHentGjennySakMedError()
every { gcpStorageService.hentFraGjenny(euxCaseId) } returns null

val result = hentSakService.hentSak(euxCaseId, sakIdFraSed, identifisertPersonFnr = fnr)

assertNull(result)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,7 @@ internal class JournalforingServiceMedJournalpostTest : JournalforingServiceBase
any()
)
} returns mockk(relaxed = true)
every { gcpStorageService.hentFraGjenny(any()) } returns null


journalforingService.journalfor(
Expand Down Expand Up @@ -82,7 +83,7 @@ internal class JournalforingServiceMedJournalpostTest : JournalforingServiceBase

val forsoekFerdigstillSlot = slot<Boolean>()
every { journalpostKlient.opprettJournalpost(any(), capture(forsoekFerdigstillSlot), any()) } returns mockk(relaxed = true)

every { gcpStorageService.hentFraGjenny(any()) } returns null
journalforingService.journalfor(
sedHendelse,
HendelseType.SENDT,
Expand Down Expand Up @@ -172,6 +173,8 @@ internal class JournalforingServiceMedJournalpostTest : JournalforingServiceBase
val capturedMelding = slot<OppgaveMelding>()
justRun { oppgaveHandler.opprettOppgaveMeldingPaaKafkaTopic(capture(capturedMelding)) }
justRun { kravHandeler.putKravInitMeldingPaaKafka(any()) }
every { gcpStorageService.hentFraGjenny(any()) } returns null

journalforingService.journalfor(
sedHendelse,
HendelseType.MOTTATT,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@ import no.nav.eessi.pensjon.personoppslag.pdl.model.SEDPersonRelasjon
import no.nav.eessi.pensjon.shared.person.Fodselsnummer
import no.nav.eessi.pensjon.utils.mapJsonToAny
import org.junit.jupiter.api.Assertions.assertEquals
import org.junit.jupiter.api.BeforeEach
import org.junit.jupiter.api.Test
import org.junit.jupiter.params.ParameterizedTest
import org.junit.jupiter.params.provider.CsvSource
Expand All @@ -47,6 +48,11 @@ internal class JournalforingServiceTest : JournalforingServiceBase() {
private val STERK_BUSK = Fodselsnummer.fra("12011577847")!!
}

@BeforeEach
fun setupClass() {
every { gcpStorageService.hentFraGjenny(any()) } returns null
}

@Test
fun `Sendt gyldig Sed R004 på R_BUC_02`() {
val hendelse = javaClass.getResource("/eux/hendelser/R_BUC_02_R004.json")!!.readText()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -164,6 +164,7 @@ internal class SedSendtJournalforingMedNavansattTest {
justRun { journalpostKlient.oppdaterDistribusjonsinfo(any()) }
justRun { gcpStorageService.lagreJournalpostDetaljer(any(), any(), any(), any(), any()) }
every { etterlatteService.hentGjennySak(eq("1234")) } returns JournalforingTestBase.mockHentGjennySak("123")
every { gcpStorageService.hentFraGjenny(any()) } returns null

val opprettJournalPostResponse = OpprettJournalPostResponse(
journalpostId = "12345",
Expand Down
Loading

0 comments on commit e83eeef

Please sign in to comment.