From 1262a3764bacf6e3df5d9c1d88428e06c0b79774 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Bj=C3=B8rnar=20Askevold?= <43171873+baskevold@users.noreply.github.com> Date: Wed, 21 Feb 2024 16:49:27 +0100 Subject: [PATCH] =?UTF-8?q?Fjerner=20valg=20av=20beskyttelsetype=20i=20fro?= =?UTF-8?q?ntend.=20Hvis=20sendt=20med=20i=20filter,=20kj=C3=B8res=20sp?= =?UTF-8?q?=C3=B8rring=20som=20om=20ordin=C3=A6r=20er=20valgt.=20Endret=20?= =?UTF-8?q?tester=20frem=20til=20lansert.=20Kan=20gjeninnf=C3=B8res=20i=20?= =?UTF-8?q?query=20ved=20at=20backend=20selv=20setter=20den=20basert=20p?= =?UTF-8?q?=C3=A5=20disk.=20til=20avd.leder=20som=20eier/oppretter=20k?= =?UTF-8?q?=C3=B8en=20(#1921)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../kodeverk/BeskyttelseType.kt | 1 - .../query/db/SqlOppgaveQuery.kt | 3 +-- .../nyoppgavestyring/pep/PepCacheServiceTest.kt | 17 +++++++++-------- .../nyoppgavestyring/query/OppgaveQueryTest.kt | 7 +++++-- 4 files changed, 15 insertions(+), 13 deletions(-) diff --git a/src/main/kotlin/no/nav/k9/los/nyoppgavestyring/kodeverk/BeskyttelseType.kt b/src/main/kotlin/no/nav/k9/los/nyoppgavestyring/kodeverk/BeskyttelseType.kt index b2b536094..2af841adf 100644 --- a/src/main/kotlin/no/nav/k9/los/nyoppgavestyring/kodeverk/BeskyttelseType.kt +++ b/src/main/kotlin/no/nav/k9/los/nyoppgavestyring/kodeverk/BeskyttelseType.kt @@ -3,7 +3,6 @@ package no.nav.k9.los.nyoppgavestyring.kodeverk import com.fasterxml.jackson.annotation.JsonCreator enum class BeskyttelseType(val kode: String, val beskrivelse: String) { - KODE6("KODE6", "Kode 6"), KODE7("KODE7", "Kode 7"), ORDINÆR("ORDINÆR", "Vanlig"); diff --git a/src/main/kotlin/no/nav/k9/los/nyoppgavestyring/query/db/SqlOppgaveQuery.kt b/src/main/kotlin/no/nav/k9/los/nyoppgavestyring/query/db/SqlOppgaveQuery.kt index e4b27ba8e..08a8fa68d 100644 --- a/src/main/kotlin/no/nav/k9/los/nyoppgavestyring/query/db/SqlOppgaveQuery.kt +++ b/src/main/kotlin/no/nav/k9/los/nyoppgavestyring/query/db/SqlOppgaveQuery.kt @@ -103,9 +103,8 @@ class SqlOppgaveQuery( } "beskyttelse" -> { when(feltverdi) { - BeskyttelseType.KODE6.kode -> query += "${combineOperator.sql} opc.kode6 is true " BeskyttelseType.KODE7.kode -> query += "${combineOperator.sql} opc.kode7 is true " - BeskyttelseType.ORDINÆR.kode -> { + else -> { query += "${combineOperator.sql} opc.kode6 is not true AND opc.kode7 is not true " } } diff --git a/src/test/kotlin/no/nav/k9/los/nyoppgavestyring/pep/PepCacheServiceTest.kt b/src/test/kotlin/no/nav/k9/los/nyoppgavestyring/pep/PepCacheServiceTest.kt index d9164cde0..c93ccfe3a 100644 --- a/src/test/kotlin/no/nav/k9/los/nyoppgavestyring/pep/PepCacheServiceTest.kt +++ b/src/test/kotlin/no/nav/k9/los/nyoppgavestyring/pep/PepCacheServiceTest.kt @@ -165,18 +165,19 @@ class PepCacheServiceTest : KoinTest, AbstractPostgresTest() { val eksternId = UUID.randomUUID().toString() k9sakEventHandler.prosesser(lagBehandlingprosessEventMedStatus(eksternId, saksnummer)) - assertThat(hentOppgaverMedSikkerhetsklassifisering(oppgaveQueryService, eksternId, BeskyttelseType.ORDINÆR)).isNotEmpty() + assertThat(hentOppgaverMedSikkerhetsklassifisering(oppgaveQueryService, eksternId, BeskyttelseType.ORDINÆR.kode)).isNotEmpty() assertThat(hentOppgaverMedSikkerhetsklassifisering(oppgaveQueryService, eksternId)).isNotEmpty() - assertThat(hentOppgaverMedSikkerhetsklassifisering(oppgaveQueryService, eksternId, BeskyttelseType.KODE6)).isEmpty() + assertThat(hentOppgaverMedSikkerhetsklassifisering(oppgaveQueryService, eksternId, "KODE6")).isNotEmpty() // Samme som ordinær frem til håndtert. Kan f.eks gjeninnføres som egen interntype som ikke eksponeres ut - verify(exactly = 2) { runBlocking { pepClient.harTilgangTilLesSak(eq(saksnummer), any()) } } + verify(exactly = 3) { runBlocking { pepClient.harTilgangTilLesSak(eq(saksnummer), any()) } } gjørSakKode6(saksnummer) ventPåAntallForsøk(10, "Pepcache") { pepRepository.hent("K9", eksternId)?.kode6 == true } - assertThat(hentOppgaverMedSikkerhetsklassifisering(oppgaveQueryService, eksternId, BeskyttelseType.ORDINÆR)).isEmpty() - assertThat(hentOppgaverMedSikkerhetsklassifisering(oppgaveQueryService, eksternId)).isNotEmpty() // Alle beskyttelsetyper er inkludert hvis ikke beksyttelsetype er spesifisert - assertThat(hentOppgaverMedSikkerhetsklassifisering(oppgaveQueryService, eksternId, BeskyttelseType.KODE6)).isNotEmpty() + assertThat(hentOppgaverMedSikkerhetsklassifisering(oppgaveQueryService, eksternId, BeskyttelseType.ORDINÆR.kode)).isEmpty() + assertThat(hentOppgaverMedSikkerhetsklassifisering(oppgaveQueryService, eksternId)).isNotEmpty() // Alle beskyttelsetyper er inkludert hvis ikke beskyttelsetype er spesifisert + assertThat(hentOppgaverMedSikkerhetsklassifisering(oppgaveQueryService, eksternId, "KODE6")).isEmpty() // Samme som ordinær frem til håndtert + job.cancel() verify(exactly = 4) { runBlocking { pepClient.harTilgangTilLesSak(eq(saksnummer), any()) } } @@ -200,7 +201,7 @@ class PepCacheServiceTest : KoinTest, AbstractPostgresTest() { private fun hentOppgaverMedSikkerhetsklassifisering( oppgaveQueryService: OppgaveQueryService, eksternId: String, - vararg sikkerhetsklassifiseringtyper: BeskyttelseType + vararg sikkerhetsklassifiseringtyper: String ): List { val transactionalManager = get() return transactionalManager.transaction { tx -> @@ -210,7 +211,7 @@ class PepCacheServiceTest : KoinTest, AbstractPostgresTest() { område = null, kode = FeltType.BESKYTTELSE.eksternId, operator = "IN", - verdi = sikkerhetsklassifiseringtyper.map { it.kode } + verdi = sikkerhetsklassifiseringtyper.toList() ), FeltverdiOppgavefilter(område = "K9", kode = FeltType.BEHANDLINGUUID.eksternId, diff --git a/src/test/kotlin/no/nav/k9/los/nyoppgavestyring/query/OppgaveQueryTest.kt b/src/test/kotlin/no/nav/k9/los/nyoppgavestyring/query/OppgaveQueryTest.kt index afe054190..b59597024 100644 --- a/src/test/kotlin/no/nav/k9/los/nyoppgavestyring/query/OppgaveQueryTest.kt +++ b/src/test/kotlin/no/nav/k9/los/nyoppgavestyring/query/OppgaveQueryTest.kt @@ -24,6 +24,7 @@ import no.nav.k9.los.nyoppgavestyring.query.dto.query.CombineOppgavefilter import no.nav.k9.los.nyoppgavestyring.query.dto.query.FeltverdiOppgavefilter import no.nav.k9.los.nyoppgavestyring.query.dto.query.OppgaveQuery import no.nav.k9.los.nyoppgavestyring.visningoguttrekk.OppgaveRepository +import org.junit.jupiter.api.Disabled import org.junit.jupiter.api.Test import org.koin.test.get import org.slf4j.Logger @@ -293,6 +294,7 @@ class OppgaveQueryTest : AbstractK9LosIntegrationTest() { } @Test + @Disabled fun `Resultat skal kun inneholde kode6-oppgaver når filtre er satt til kode6`() { val eksternId = lagOppgave(kode6 = true) @@ -301,7 +303,7 @@ class OppgaveQueryTest : AbstractK9LosIntegrationTest() { val oppgaveQueryRepository = OppgaveQueryRepository(dataSource, mockk()) val query = OppgaveQuery(listOf( byggFilterK9(FeltType.BEHANDLINGUUID, FeltverdiOperator.EQUALS, eksternId), - byggGenereltFilter(FeltType.BESKYTTELSE, FeltverdiOperator.IN, BeskyttelseType.KODE6.kode) + byggGenereltFilter(FeltType.BESKYTTELSE, FeltverdiOperator.IN, "KODE6") )) assertThat(oppgaveQueryRepository.query(query)).isNotEmpty() @@ -324,6 +326,7 @@ class OppgaveQueryTest : AbstractK9LosIntegrationTest() { } @Test + @Disabled fun `Resultat skal ikke inneholde kode7 eller ordinære oppgaver når filtre er satt til kode6 oppgaver`() { val eksternId7 = lagOppgave(kode7 = true) val eksternIdOrdinær = lagOppgave() @@ -333,7 +336,7 @@ class OppgaveQueryTest : AbstractK9LosIntegrationTest() { val oppgaveQueryRepository = OppgaveQueryRepository(dataSource, mockk()) val query = OppgaveQuery(listOf( byggFilterK9(FeltType.BEHANDLINGUUID, FeltverdiOperator.EQUALS, eksternId7, eksternIdOrdinær), - byggGenereltFilter(FeltType.BESKYTTELSE, FeltverdiOperator.IN, BeskyttelseType.KODE6.kode) + byggGenereltFilter(FeltType.BESKYTTELSE, FeltverdiOperator.IN, "KODE6") )) assertThat(oppgaveQueryRepository.query(query)).isEmpty()