From adc83573c5c4823c368d94736cff8783b29e6da8 Mon Sep 17 00:00:00 2001 From: Emil Albrektsson <61695840+albrektsson@users.noreply.github.com> Date: Tue, 10 Oct 2023 12:33:40 +0200 Subject: [PATCH] Ryddet opp i gosys dto, fjernet k9sak metode som ikke er i bruk lengre (#1016) --- .../gosys/OpprettOppgaveRequest.kt | 14 -------- .../integrasjoner/k9sak/K9SakService.kt | 8 ----- .../integrasjoner/k9sak/K9SakServiceImpl.kt | 33 ------------------- .../rest/eksternt/k9sak/LokalK9SakService.kt | 11 ------- .../rest/eksternt/k9sak/TestK9SakService.kt | 13 +------- .../no/nav/k9punsj/wiremock/GosysMocks.kt | 3 -- 6 files changed, 1 insertion(+), 81 deletions(-) diff --git a/src/main/kotlin/no/nav/k9punsj/integrasjoner/gosys/OpprettOppgaveRequest.kt b/src/main/kotlin/no/nav/k9punsj/integrasjoner/gosys/OpprettOppgaveRequest.kt index b537ea536..9417b4be3 100644 --- a/src/main/kotlin/no/nav/k9punsj/integrasjoner/gosys/OpprettOppgaveRequest.kt +++ b/src/main/kotlin/no/nav/k9punsj/integrasjoner/gosys/OpprettOppgaveRequest.kt @@ -12,27 +12,13 @@ internal data class OpprettOppgaveRequest( private val gjelder: Gjelder ) { - val prioritet = "NORM" - val aktivDato: LocalDate = LocalDate.now(ZoneId.of("Europe/Oslo")) - val fristFerdigstillelse: LocalDate = aktivDato.treVirkerdagerFrem() val tema = "OMS" - val oppgavetype = "JFR" - val behandlingstema = gjelder.behandlingstema?.kodeverksverdi - val behandlingstype = gjelder.behandlingstype?.kodeverksverdi init { check(gjelder.aktiv) { "Gjelderkategorien $gjelder er ikke aktiv." } } - - private companion object { - private fun LocalDate.treVirkerdagerFrem() = when (dayOfWeek) { - DayOfWeek.FRIDAY -> plusDays(5) - DayOfWeek.SATURDAY -> plusDays(4) - else -> plusDays(3) - } - } } internal data class PatchOppgaveRequest( diff --git a/src/main/kotlin/no/nav/k9punsj/integrasjoner/k9sak/K9SakService.kt b/src/main/kotlin/no/nav/k9punsj/integrasjoner/k9sak/K9SakService.kt index f2ffa88a6..ab6a65051 100644 --- a/src/main/kotlin/no/nav/k9punsj/integrasjoner/k9sak/K9SakService.kt +++ b/src/main/kotlin/no/nav/k9punsj/integrasjoner/k9sak/K9SakService.kt @@ -5,7 +5,6 @@ import no.nav.k9.søknad.Søknad import no.nav.k9punsj.felles.FagsakYtelseType import no.nav.k9punsj.felles.dto.ArbeidsgiverMedArbeidsforholdId import no.nav.k9punsj.felles.dto.PeriodeDto -import no.nav.k9punsj.felles.dto.SaksnummerDto import no.nav.k9punsj.felles.dto.SøknadEntitet interface K9SakService { @@ -40,13 +39,6 @@ interface K9SakService { fagsakYtelseType: FagsakYtelseType ): Pair - suspend fun hentSisteSaksnummerForPeriode( - fagsakYtelseType: FagsakYtelseType, - periode: PeriodeDto?, - søker: String, - pleietrengende: String? - ): Pair - suspend fun sendInnSoeknad( soknad: Søknad, journalpostId: String, diff --git a/src/main/kotlin/no/nav/k9punsj/integrasjoner/k9sak/K9SakServiceImpl.kt b/src/main/kotlin/no/nav/k9punsj/integrasjoner/k9sak/K9SakServiceImpl.kt index 3fba9ebf3..3b3c3d5d8 100644 --- a/src/main/kotlin/no/nav/k9punsj/integrasjoner/k9sak/K9SakServiceImpl.kt +++ b/src/main/kotlin/no/nav/k9punsj/integrasjoner/k9sak/K9SakServiceImpl.kt @@ -23,7 +23,6 @@ import no.nav.k9punsj.integrasjoner.k9sak.K9SakServiceImpl.Urls.finnFagsak import no.nav.k9punsj.integrasjoner.k9sak.K9SakServiceImpl.Urls.hentIntektsmeldingerUrl import no.nav.k9punsj.integrasjoner.k9sak.K9SakServiceImpl.Urls.hentPerioderUrl import no.nav.k9punsj.integrasjoner.k9sak.K9SakServiceImpl.Urls.sendInnSøknadUrl -import no.nav.k9punsj.integrasjoner.k9sak.K9SakServiceImpl.Urls.sokFagsaker import no.nav.k9punsj.integrasjoner.k9sak.K9SakServiceImpl.Urls.sokFagsakerUrl import no.nav.k9punsj.omsorgspengeraleneomsorg.tilOmsAOvisning import no.nav.k9punsj.omsorgspengerkronisksyktbarn.tilOmsKSBvisning @@ -232,38 +231,6 @@ class K9SakServiceImpl( } - override suspend fun hentSisteSaksnummerForPeriode( - fagsakYtelseType: no.nav.k9punsj.felles.FagsakYtelseType, - periode: PeriodeDto?, - søker: String, - pleietrengende: String? - ): Pair { - val hentSaksnummerForPeriodeDto = HentSaksnummerForPeriodeDto( - ytelseType = FagsakYtelseType.fraKode(fagsakYtelseType.kode), - bruker = søker, - pleietrengende = listOfNotNull(pleietrengende).ifEmpty { null }, - periode = periode - ) - - val body = kotlin.runCatching { objectMapper().writeValueAsString(hentSaksnummerForPeriodeDto) }.getOrNull() - ?: return Pair(null, "Feilet serialisering") - - val (response, feil) = httpPost(body, sokFagsaker) - return try { - if (response == null) { - return Pair(null, feil!!) - } - val saksnummer = response.fagsaker() - .filterNot { it.gyldigPeriode?.fom == null } - .sortedBy { it.gyldigPeriode!!.fom } - .first() - .saksnummer - Pair(SaksnummerDto(saksnummer), null) - } catch (e: Exception) { - Pair(null, "Feilet deserialisering $e") - } - } - override suspend fun sendInnSoeknad( soknad: Søknad, journalpostId: String, diff --git a/src/test/kotlin/no/nav/k9punsj/rest/eksternt/k9sak/LokalK9SakService.kt b/src/test/kotlin/no/nav/k9punsj/rest/eksternt/k9sak/LokalK9SakService.kt index 6fda0b4bc..3983982de 100644 --- a/src/test/kotlin/no/nav/k9punsj/rest/eksternt/k9sak/LokalK9SakService.kt +++ b/src/test/kotlin/no/nav/k9punsj/rest/eksternt/k9sak/LokalK9SakService.kt @@ -6,10 +6,8 @@ import no.nav.k9punsj.LokalProfil import no.nav.k9punsj.felles.FagsakYtelseType import no.nav.k9punsj.felles.dto.ArbeidsgiverMedArbeidsforholdId import no.nav.k9punsj.felles.dto.PeriodeDto -import no.nav.k9punsj.felles.dto.SaksnummerDto import no.nav.k9punsj.felles.dto.SøknadEntitet import no.nav.k9punsj.integrasjoner.k9sak.Fagsak -import no.nav.k9punsj.integrasjoner.k9sak.HentK9SaksnummerGrunnlag import no.nav.k9punsj.integrasjoner.k9sak.K9SakService import no.nav.k9punsj.util.MockUtil.erFødtI import org.springframework.stereotype.Component @@ -80,15 +78,6 @@ class LokalK9SakService : K9SakService { return Pair("ABC123", null) } - override suspend fun hentSisteSaksnummerForPeriode( - fagsakYtelseType: FagsakYtelseType, - periode: PeriodeDto?, - søker: String, - pleietrengende: String? - ): Pair { - return Pair(SaksnummerDto("ABC123"), null) - } - override suspend fun sendInnSoeknad( soknad: Søknad, journalpostId: String, diff --git a/src/test/kotlin/no/nav/k9punsj/rest/eksternt/k9sak/TestK9SakService.kt b/src/test/kotlin/no/nav/k9punsj/rest/eksternt/k9sak/TestK9SakService.kt index 21765e5e0..4be4a7893 100644 --- a/src/test/kotlin/no/nav/k9punsj/rest/eksternt/k9sak/TestK9SakService.kt +++ b/src/test/kotlin/no/nav/k9punsj/rest/eksternt/k9sak/TestK9SakService.kt @@ -6,10 +6,8 @@ import no.nav.k9punsj.TestProfil import no.nav.k9punsj.felles.FagsakYtelseType import no.nav.k9punsj.felles.dto.ArbeidsgiverMedArbeidsforholdId import no.nav.k9punsj.felles.dto.PeriodeDto -import no.nav.k9punsj.felles.dto.SaksnummerDto import no.nav.k9punsj.felles.dto.SøknadEntitet import no.nav.k9punsj.integrasjoner.k9sak.Fagsak -import no.nav.k9punsj.integrasjoner.k9sak.HentK9SaksnummerGrunnlag import no.nav.k9punsj.integrasjoner.k9sak.K9SakService import org.springframework.stereotype.Component import java.time.LocalDate @@ -89,20 +87,11 @@ internal class TestK9SakService : K9SakService { override suspend fun hentEllerOpprettSaksnummer( søknadEntitet: SøknadEntitet, - fagsakYtelseType: no.nav.k9punsj.felles.FagsakYtelseType + fagsakYtelseType: FagsakYtelseType ): Pair { return Pair("ABC123", null) } - override suspend fun hentSisteSaksnummerForPeriode( - fagsakYtelseType: FagsakYtelseType, - periode: PeriodeDto?, - søker: String, - pleietrengende: String? - ): Pair { - return Pair(SaksnummerDto("ABC123"), null) - } - override suspend fun sendInnSoeknad( soknad: Søknad, journalpostId: String, diff --git a/src/test/kotlin/no/nav/k9punsj/wiremock/GosysMocks.kt b/src/test/kotlin/no/nav/k9punsj/wiremock/GosysMocks.kt index 9d7233be3..517edabdf 100644 --- a/src/test/kotlin/no/nav/k9punsj/wiremock/GosysMocks.kt +++ b/src/test/kotlin/no/nav/k9punsj/wiremock/GosysMocks.kt @@ -16,10 +16,7 @@ private fun WireMockServer.stubOpprettOppgave(): WireMockServer { .withHeader("Nav-Consumer-Id", WireMock.equalTo("k9-punsj")) .withRequestBody(WireMock.matchingJsonPath("$.aktoerId")) .withRequestBody(WireMock.matchingJsonPath("$.journalpostId")) - .withRequestBody(WireMock.matchingJsonPath("$.fristFerdigstillelse")) - .withRequestBody(WireMock.matchingJsonPath("$.prioritet", WireMock.equalTo("NORM"))) .withRequestBody(WireMock.matchingJsonPath("$.tema", WireMock.equalTo("OMS"))) - .withRequestBody(WireMock.matchingJsonPath("$.oppgavetype", WireMock.equalTo("JFR"))) .withRequestBody(WireMock.matchingJsonPath("$.gjelder", WireMock.absent())) .willReturn( WireMock.aResponse()