From 890b8ccb26bbd80784e8885097ac49fc7ff240f7 Mon Sep 17 00:00:00 2001 From: erikb Date: Tue, 21 Jan 2025 09:30:40 +0100 Subject: [PATCH 1/9] legg til inngar i inbound for team dab --- .nais/application/application-config-dev.yaml | 3 +++ .nais/application/application-config-prod.yaml | 3 +++ 2 files changed, 6 insertions(+) diff --git a/.nais/application/application-config-dev.yaml b/.nais/application/application-config-dev.yaml index 7ba7b76098..c0b6c6ba6b 100644 --- a/.nais/application/application-config-dev.yaml +++ b/.nais/application/application-config-dev.yaml @@ -69,6 +69,9 @@ spec: - application: azure-token-generator namespace: aura cluster: dev-gcp + - application: inngar + namespace: dab + cluster: dev-gcp outbound: external: - host: team-obo-unleash-api.nav.cloud.nais.io diff --git a/.nais/application/application-config-prod.yaml b/.nais/application/application-config-prod.yaml index 844076b4ec..1abcdf80f9 100644 --- a/.nais/application/application-config-prod.yaml +++ b/.nais/application/application-config-prod.yaml @@ -68,6 +68,9 @@ spec: - application: pto-admin namespace: pto cluster: prod-fss + - application: inngar + namespace: dab + cluster: prod-gcp outbound: external: - host: team-obo-unleash-api.nav.cloud.nais.io From fe224fe73035222c7ffd48dc4d3e625c6f6aa153 Mon Sep 17 00:00:00 2001 From: Klara M Helgemo Date: Wed, 22 Jan 2025 15:27:07 +0100 Subject: [PATCH 2/9] Fjerner feature toggle stopp_opensearch_indeksering --- .../pto/veilarbportefolje/config/FeatureToggle.java | 6 ------ .../persononinfo/PdlBrukerdataKafkaService.java | 10 +--------- .../BarnUnder18AarTest/BarnUnder18AarKafkaTest.java | 6 +----- .../BarnUnder18AarMisterForeldreansvarTest.java | 9 ++------- .../persononinfo/PDLPersonBarnTest.java | 4 ++-- .../persononinfo/PdlBrukerdataKafkaServiceTest.java | 3 +-- 6 files changed, 7 insertions(+), 31 deletions(-) diff --git a/src/main/java/no/nav/pto/veilarbportefolje/config/FeatureToggle.java b/src/main/java/no/nav/pto/veilarbportefolje/config/FeatureToggle.java index 7385cf7518..10bfcf1934 100644 --- a/src/main/java/no/nav/pto/veilarbportefolje/config/FeatureToggle.java +++ b/src/main/java/no/nav/pto/veilarbportefolje/config/FeatureToggle.java @@ -13,19 +13,13 @@ private FeatureToggle() { public static final String KAFKA_AIVEN_CONSUMERS_STOP = "veilarbportefolje.kafka_aiven_consumers_stop"; public static final String KAFKA_SISTE_14A_STOP = "veilarbportefolje.kafka_siste_14a_stop"; public static final String BRUK_FILTER_FOR_BRUKERINNSYN_TILGANGER = "veilarbportefolje.bruk_filter_for_brukerinnsyn_tilganger"; - public static final String STOPP_OPENSEARCH_INDEKSERING = "veilarbportefolje.stopp_opensearch_indeksering"; public static final String STOPP_KONSUMERING_FRA_PORTEFOLJE_HENDELSESFILTER_TOPIC = "veilarbportefolje.stopp_konsumering_fra_portefolje_hendelsesfilter_topic"; public static boolean brukAvAliasIndeksering(DefaultUnleash defaultUnleash) { return defaultUnleash.isEnabled(FeatureToggle.ALIAS_INDEKSERING); } - public static boolean brukFilterForBrukerinnsynTilganger(DefaultUnleash defaultUnleash) { return defaultUnleash.isEnabled(FeatureToggle.BRUK_FILTER_FOR_BRUKERINNSYN_TILGANGER); } - - public static boolean stoppOpensearchIndeksering(DefaultUnleash defaultUnleash) { - return defaultUnleash.isEnabled(FeatureToggle.STOPP_OPENSEARCH_INDEKSERING); - } } diff --git a/src/main/java/no/nav/pto/veilarbportefolje/persononinfo/PdlBrukerdataKafkaService.java b/src/main/java/no/nav/pto/veilarbportefolje/persononinfo/PdlBrukerdataKafkaService.java index 4c38eb423f..872bdde9ea 100644 --- a/src/main/java/no/nav/pto/veilarbportefolje/persononinfo/PdlBrukerdataKafkaService.java +++ b/src/main/java/no/nav/pto/veilarbportefolje/persononinfo/PdlBrukerdataKafkaService.java @@ -1,12 +1,10 @@ package no.nav.pto.veilarbportefolje.persononinfo; -import io.getunleash.DefaultUnleash; import lombok.RequiredArgsConstructor; import lombok.SneakyThrows; import lombok.extern.slf4j.Slf4j; import no.nav.common.types.identer.AktorId; import no.nav.common.types.identer.Fnr; -import no.nav.pto.veilarbportefolje.config.FeatureToggle; import no.nav.pto.veilarbportefolje.kafka.KafkaCommonNonKeyedConsumerService; import no.nav.pto.veilarbportefolje.opensearch.OpensearchIndexer; import no.nav.pto.veilarbportefolje.opensearch.OpensearchIndexerV2; @@ -39,7 +37,6 @@ public class PdlBrukerdataKafkaService extends KafkaCommonNonKeyedConsumerServic private final BarnUnder18AarService barnUnder18AarService; private final OpensearchIndexer opensearchIndexer; private final OpensearchIndexerV2 opensearchIndexerV2; - private final DefaultUnleash defaultUnleash; @Override @SneakyThrows @@ -60,14 +57,11 @@ public void behandleKafkaMeldingLogikk(PdlDokument pdlDokument) { if (pdlIdentRepository.harAktorIdUnderOppfolging(aktorIder)) { AktorId aktivAktorId = hentAktivAktor(pdlIdenter); secureLog.info("Det oppsto en PDL endring aktoer: {}", aktivAktorId); - handterIdentEndring(pdlIdenter); handterBrukerDataEndring(pdlDokument.getHentPerson(), pdlIdenter); - if (!FeatureToggle.stoppOpensearchIndeksering(defaultUnleash)) { - oppdaterOpensearch(aktivAktorId, pdlIdenter); - } + oppdaterOpensearch(aktivAktorId, pdlIdenter); } if (barnUnder18AarService.erFnrBarnAvForelderUnderOppfolging(fnrs)) { @@ -77,7 +71,6 @@ public void behandleKafkaMeldingLogikk(PdlDokument pdlDokument) { handterBarnEndring(pdlDokument.getHentPerson(), pdlIdenter); - if (!FeatureToggle.stoppOpensearchIndeksering(defaultUnleash)) { List foreldreTilBarn = barnUnder18AarService.finnForeldreTilBarn(aktivtFnr); foreldreTilBarn.forEach(fnrForelder -> { @@ -89,7 +82,6 @@ public void behandleKafkaMeldingLogikk(PdlDokument pdlDokument) { } } ); - } } } diff --git a/src/test/java/no/nav/pto/veilarbportefolje/persononinfo/BarnUnder18AarTest/BarnUnder18AarKafkaTest.java b/src/test/java/no/nav/pto/veilarbportefolje/persononinfo/BarnUnder18AarTest/BarnUnder18AarKafkaTest.java index aeb2533151..b660a90f1d 100644 --- a/src/test/java/no/nav/pto/veilarbportefolje/persononinfo/BarnUnder18AarTest/BarnUnder18AarKafkaTest.java +++ b/src/test/java/no/nav/pto/veilarbportefolje/persononinfo/BarnUnder18AarTest/BarnUnder18AarKafkaTest.java @@ -3,7 +3,6 @@ import com.fasterxml.jackson.core.JsonProcessingException; import com.fasterxml.jackson.databind.ObjectMapper; import com.github.tomakehurst.wiremock.WireMockServer; -import io.getunleash.DefaultUnleash; import no.nav.common.client.pdl.PdlClientImpl; import no.nav.common.types.identer.AktorId; import no.nav.common.types.identer.Fnr; @@ -65,8 +64,6 @@ public class BarnUnder18AarKafkaTest { private final String pdlDokumentBarn1MedDiskresjonskodeAsString = readFileAsJsonString("/PDL_Files/pdl_dokument_barn1_med_diskresjonskode.json", getClass()); private final JdbcTemplate db; - @MockBean - private DefaultUnleash defaultUnleash; private final WireMockServer server = new WireMockServer(); @@ -107,8 +104,7 @@ public void setup() { new BrukerServiceV2(this.pdlIdentRepository, this.oppfolgingsbrukerRepositoryV3, this.oppfolgingRepositoryV2), this.barnUnder18AarService, opensearchIndexer, - opensearchIndexerV2, - defaultUnleash + opensearchIndexerV2 ); } diff --git a/src/test/java/no/nav/pto/veilarbportefolje/persononinfo/BarnUnder18AarTest/BarnUnder18AarMisterForeldreansvarTest.java b/src/test/java/no/nav/pto/veilarbportefolje/persononinfo/BarnUnder18AarTest/BarnUnder18AarMisterForeldreansvarTest.java index 676d97a861..5ab88f8e94 100644 --- a/src/test/java/no/nav/pto/veilarbportefolje/persononinfo/BarnUnder18AarTest/BarnUnder18AarMisterForeldreansvarTest.java +++ b/src/test/java/no/nav/pto/veilarbportefolje/persononinfo/BarnUnder18AarTest/BarnUnder18AarMisterForeldreansvarTest.java @@ -3,7 +3,6 @@ import com.fasterxml.jackson.core.JsonProcessingException; import com.fasterxml.jackson.databind.ObjectMapper; import com.github.tomakehurst.wiremock.WireMockServer; -import io.getunleash.DefaultUnleash; import no.nav.common.client.pdl.PdlClientImpl; import no.nav.common.types.identer.AktorId; import no.nav.common.types.identer.Fnr; @@ -65,9 +64,6 @@ public class BarnUnder18AarMisterForeldreansvarTest { private final String pdlDokumentIngenBarn = readFileAsJsonString("/PDL_Files/pdl_dokument_ingen_barn.json", getClass()); private final JdbcTemplate db; - @MockBean - private DefaultUnleash defaultUnleash; - private final WireMockServer server = new WireMockServer(); @@ -94,7 +90,7 @@ public void setup() { server.start(); - PdlPortefoljeClient pdlPortefoljeClient = new PdlPortefoljeClient(new PdlClientImpl("http://localhost:" + server.port(), () -> "SYSTEM_TOKEN")); + PdlPortefoljeClient pdlPortefoljeClient = new PdlPortefoljeClient(new PdlClientImpl("http://localhost:" + server.port(), () -> "SYSTEM_TOKEN", "B555")); barnUnder18AarService = new BarnUnder18AarService(barnUnder18AarRepository, pdlPortefoljeClient); @@ -108,8 +104,7 @@ public void setup() { new BrukerServiceV2(this.pdlIdentRepository, this.oppfolgingsbrukerRepositoryV3, this.oppfolgingRepositoryV2), this.barnUnder18AarService, opensearchIndexer, - opensearchIndexerV2, - defaultUnleash + opensearchIndexerV2 ); } diff --git a/src/test/java/no/nav/pto/veilarbportefolje/persononinfo/PDLPersonBarnTest.java b/src/test/java/no/nav/pto/veilarbportefolje/persononinfo/PDLPersonBarnTest.java index 0873ed8317..d061ddaaa6 100644 --- a/src/test/java/no/nav/pto/veilarbportefolje/persononinfo/PDLPersonBarnTest.java +++ b/src/test/java/no/nav/pto/veilarbportefolje/persononinfo/PDLPersonBarnTest.java @@ -13,7 +13,7 @@ import org.junit.jupiter.api.AfterEach; import org.junit.jupiter.api.BeforeEach; import org.junit.jupiter.api.Test; -import org.springframework.beans.factory.annotation.Autowired; +import org.springframework.boot.test.mock.mockito.MockBean; import org.springframework.jdbc.core.JdbcTemplate; import java.time.LocalDate; @@ -33,7 +33,7 @@ public class PDLPersonBarnTest { private final JdbcTemplate db; private final PdlPersonRepository pdlPersonRepository; - @Autowired + @MockBean private PdlPortefoljeClient mockedPdlPortefoljeClient; private PdlService pdlService; diff --git a/src/test/java/no/nav/pto/veilarbportefolje/persononinfo/PdlBrukerdataKafkaServiceTest.java b/src/test/java/no/nav/pto/veilarbportefolje/persononinfo/PdlBrukerdataKafkaServiceTest.java index bc82409885..23e6819f4f 100644 --- a/src/test/java/no/nav/pto/veilarbportefolje/persononinfo/PdlBrukerdataKafkaServiceTest.java +++ b/src/test/java/no/nav/pto/veilarbportefolje/persononinfo/PdlBrukerdataKafkaServiceTest.java @@ -75,8 +75,7 @@ public PdlBrukerdataKafkaServiceTest() throws JsonProcessingException { new BrukerServiceV2(pdlIdentRepository, oppfolgingsbrukerRepositoryV3, oppfolgingRepositoryV2), barnUnder18AarService, opensearchIndexer, - opensearchIndexerV2, - mockUnleash + opensearchIndexerV2 ); } From 47ba407e4103fdd6c637fb7698a9e74a8cacf754 Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Tue, 21 Jan 2025 14:28:09 +0000 Subject: [PATCH 3/9] Bump org.flywaydb:flyway-core from 10.21.0 to 11.2.0 Bumps [org.flywaydb:flyway-core](https://github.com/flyway/flyway) from 10.21.0 to 11.2.0. - [Release notes](https://github.com/flyway/flyway/releases) - [Commits](https://github.com/flyway/flyway/compare/flyway-10.21.0...flyway-11.2.0) --- updated-dependencies: - dependency-name: org.flywaydb:flyway-core dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] --- pom.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pom.xml b/pom.xml index 38a2f2d476..3fdb1d6c4c 100644 --- a/pom.xml +++ b/pom.xml @@ -296,7 +296,7 @@ org.flywaydb flyway-core - 10.21.0 + 11.2.0 compile From ff74e3b5d2085f68d2c9afcd487162b23cf35bde Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Tue, 21 Jan 2025 14:28:06 +0000 Subject: [PATCH 4/9] Bump org.flywaydb:flyway-database-postgresql from 10.21.0 to 11.2.0 Bumps org.flywaydb:flyway-database-postgresql from 10.21.0 to 11.2.0. --- updated-dependencies: - dependency-name: org.flywaydb:flyway-database-postgresql dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] --- pom.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pom.xml b/pom.xml index 3fdb1d6c4c..e5249d3ddd 100644 --- a/pom.xml +++ b/pom.xml @@ -302,7 +302,7 @@ org.flywaydb flyway-database-postgresql - 10.21.0 + 11.2.0 org.postgresql From cd7345637571cf1b5b65c1afa9b03e38157bd6bc Mon Sep 17 00:00:00 2001 From: Sondre Larsen Ovrid Date: Wed, 22 Jan 2025 20:55:49 +0100 Subject: [PATCH 5/9] =?UTF-8?q?Ta=20i=20bruk=20Java=20Switch=20Expressions?= =?UTF-8?q?=20for=20=C3=A5=20f=C3=A5=20exhaustiveness-sjekk=20fr=C3=A5=20k?= =?UTF-8?q?ompilatoren?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../opensearch/OpensearchQueryBuilder.java | 227 +++++++++++++----- 1 file changed, 173 insertions(+), 54 deletions(-) diff --git a/src/main/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchQueryBuilder.java b/src/main/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchQueryBuilder.java index ee9a0cd867..8efcaf701b 100644 --- a/src/main/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchQueryBuilder.java +++ b/src/main/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchQueryBuilder.java @@ -411,11 +411,11 @@ static void leggTilManuelleFilter(BoolQueryBuilder queryBuilder, Filtervalg filt BoolQueryBuilder subQuery = boolQuery(); if (valgteGjeldendeVedtak14aFilter.contains(GjeldendeVedtak14aFilter.HAR_14A_VEDTAK) - && !valgteGjeldendeVedtak14aFilter.contains(GjeldendeVedtak14aFilter.HAR_IKKE_14A_VEDTAK)) { + && !valgteGjeldendeVedtak14aFilter.contains(GjeldendeVedtak14aFilter.HAR_IKKE_14A_VEDTAK)) { subQuery.must(existsQuery("gjeldendeVedtak14a")); queryBuilder.must(subQuery); } else if (valgteGjeldendeVedtak14aFilter.contains(GjeldendeVedtak14aFilter.HAR_IKKE_14A_VEDTAK) - && !valgteGjeldendeVedtak14aFilter.contains(GjeldendeVedtak14aFilter.HAR_14A_VEDTAK)) { + && !valgteGjeldendeVedtak14aFilter.contains(GjeldendeVedtak14aFilter.HAR_14A_VEDTAK)) { subQuery.mustNot(existsQuery("gjeldendeVedtak14a")); queryBuilder.must(subQuery); } @@ -442,6 +442,15 @@ static List byggAktivitetFilterQuery(Filtervalg filtervalg, Bo ).collect(toList()); } + /** + * + * @param sortOrder + * @param sortField + * @param searchSourceBuilder + * @param filtervalg + * @param brukerinnsynTilganger + * @return + */ static SearchSourceBuilder sorterQueryParametere(String sortOrder, String sortField, SearchSourceBuilder searchSourceBuilder, Filtervalg filtervalg, BrukerinnsynTilganger brukerinnsynTilganger) { SortOrder order = "ascending".equals(sortOrder) ? SortOrder.ASC : SortOrder.DESC; @@ -449,73 +458,183 @@ static SearchSourceBuilder sorterQueryParametere(String sortOrder, String /* På sikt (tm) skal vi typesikre sortField slik at vi får Sorteringsfelt her, gjerne allereie på Controller-nivå. I denne omgangen lagar eg berre enumen for sorteringsfelta. 2024-11-28, Ingrid. */ Sorteringsfelt sorteringsfelt = Sorteringsfelt.nameFromValue(sortField); - switch (sorteringsfelt) { - case IKKE_SATT -> brukStandardsorteringBasertPaValgteFilter(filtervalg, searchSourceBuilder); - case VALGTE_AKTIVITETER -> sorterValgteAktiviteter(filtervalg, searchSourceBuilder, order); - case MOTER_MED_NAV_IDAG -> searchSourceBuilder.sort("alle_aktiviteter_mote_startdato", order); - case MOTESTATUS -> searchSourceBuilder.sort("aktivitet_mote_startdato", order); + SearchSourceBuilder hovedsortering = switch (sorteringsfelt) { + case IKKE_SATT -> { + brukStandardsorteringBasertPaValgteFilter(filtervalg, searchSourceBuilder); + yield searchSourceBuilder; + } + case VALGTE_AKTIVITETER -> { + sorterValgteAktiviteter(filtervalg, searchSourceBuilder, order); + yield searchSourceBuilder; + } + case MOTER_MED_NAV_IDAG -> { + searchSourceBuilder.sort("alle_aktiviteter_mote_startdato", order); + yield searchSourceBuilder; + } + case MOTESTATUS -> { + searchSourceBuilder.sort("aktivitet_mote_startdato", order); + yield searchSourceBuilder; + } case I_AVTALT_AKTIVITET -> { FieldSortBuilder builder = new FieldSortBuilder("aktivitet_utlopsdatoer") .order(order) .sortMode(MIN); searchSourceBuilder.sort(builder); + yield searchSourceBuilder; } - case FODSELSNUMMER -> searchSourceBuilder.sort("fnr.raw", order); - case UTLOPTE_AKTIVITETER -> searchSourceBuilder.sort("nyesteutlopteaktivitet", order); - case ARBEIDSLISTE_FRIST -> searchSourceBuilder.sort("arbeidsliste_frist", order); - case AAP_TYPE -> searchSourceBuilder.sort("ytelse", order); - case AAP_VURDERINGSFRIST -> sorterAapVurderingsfrist(searchSourceBuilder, order, filtervalg); - case AAP_RETTIGHETSPERIODE -> sorterAapRettighetsPeriode(searchSourceBuilder, order); - case GJELDENDE_VEDTAK_14A_INNSATSGRUPPE -> - searchSourceBuilder.sort("gjeldendeVedtak14a.innsatsgruppe", order); - case GJELDENDE_VEDTAK_14A_HOVEDMAL -> searchSourceBuilder.sort("gjeldendeVedtak14a.hovedmal", order); - case GJELDENDE_VEDTAK_14A_VEDTAKSDATO -> sorterGjeldendeVedtak14aVedtaksdato(searchSourceBuilder, order); - case UTKAST_14A_STATUS -> searchSourceBuilder.sort("utkast_14a_status", order); - case ARBEIDSLISTE_KATEGORI -> searchSourceBuilder.sort("arbeidsliste_kategori", order); - case SISTE_ENDRING_DATO -> sorterSisteEndringTidspunkt(searchSourceBuilder, order, filtervalg); - case ARBEIDSLISTE_OVERSKRIFT -> sorterArbeidslisteOverskrift(searchSourceBuilder, order); - case FODELAND -> sorterFodeland(searchSourceBuilder, order); - case STATSBORGERSKAP -> sorterStatsborgerskap(searchSourceBuilder, order); - case STATSBORGERSKAP_GYLDIG_FRA -> sorterStatsborgerskapGyldigFra(searchSourceBuilder, order); - case TOLKESPRAK -> sorterTolkeSpraak(filtervalg, searchSourceBuilder, order); - case TOLKEBEHOV_SIST_OPPDATERT -> searchSourceBuilder.sort("tolkBehovSistOppdatert", order); - case ENSLIGE_FORSORGERE_UTLOP_YTELSE -> sorterEnsligeForsorgereUtlopsDato(searchSourceBuilder, order); - case ENSLIGE_FORSORGERE_VEDTAKSPERIODETYPE -> - sorterEnsligeForsorgereVedtaksPeriode(searchSourceBuilder, order); - case ENSLIGE_FORSORGERE_AKTIVITETSPLIKT -> - sorterEnsligeForsorgereAktivitetsPlikt(searchSourceBuilder, order); - case ENSLIGE_FORSORGERE_OM_BARNET -> sorterEnsligeForsorgereOmBarnet(searchSourceBuilder, order); - case BARN_UNDER_18_AR -> sorterBarnUnder18(searchSourceBuilder, order, brukerinnsynTilganger, filtervalg); - case BRUKERS_SITUASJON_SIST_ENDRET -> searchSourceBuilder.sort("brukers_situasjon_sist_endret", order); - case UTDANNING_OG_SITUASJON_SIST_ENDRET -> - searchSourceBuilder.sort("utdanning_og_situasjon_sist_endret", order); - case HUSKELAPP_FRIST -> sorterHuskelappFrist(searchSourceBuilder, order); - case HUSKELAPP -> sorterHuskelappEksistere(searchSourceBuilder, order); - case HUSKELAPP_KOMMENTAR -> searchSourceBuilder.sort("huskelapp.kommentar", order); - case FARGEKATEGORI -> searchSourceBuilder.sort("fargekategori", order); - case TILTAKSHENDELSE_DATO_OPPRETTET -> sorterTiltakshendelseOpprettetDato(searchSourceBuilder, order); - case TILTAKSHENDELSE_TEKST -> searchSourceBuilder.sort("tiltakshendelse.tekst", order); - case UTGATT_VARSEL_DATO -> sorterUtgattVarselHendelseDato(searchSourceBuilder, order); - default -> defaultSort(sorteringsfelt, searchSourceBuilder, order); - } - addSecondarySort(searchSourceBuilder); - return searchSourceBuilder; + case FODSELSNUMMER -> { + searchSourceBuilder.sort("fnr.raw", order); + yield searchSourceBuilder; + } + case UTLOPTE_AKTIVITETER -> { + searchSourceBuilder.sort("nyesteutlopteaktivitet", order); + yield searchSourceBuilder; + } + case ARBEIDSLISTE_FRIST -> { + searchSourceBuilder.sort("arbeidsliste_frist", order); + yield searchSourceBuilder; + } + case AAP_TYPE -> { + searchSourceBuilder.sort("ytelse", order); + yield searchSourceBuilder; + } + case AAP_VURDERINGSFRIST -> { + sorterAapVurderingsfrist(searchSourceBuilder, order, filtervalg); + yield searchSourceBuilder; + } + case AAP_RETTIGHETSPERIODE -> { + sorterAapRettighetsPeriode(searchSourceBuilder, order); + yield searchSourceBuilder; + } + case GJELDENDE_VEDTAK_14A_INNSATSGRUPPE -> { + searchSourceBuilder.sort("gjeldendeVedtak14a.innsatsgruppe", order); + yield searchSourceBuilder; + } + case GJELDENDE_VEDTAK_14A_HOVEDMAL -> { + searchSourceBuilder.sort("gjeldendeVedtak14a.hovedmal", order); + yield searchSourceBuilder; + } + case GJELDENDE_VEDTAK_14A_VEDTAKSDATO -> { + sorterGjeldendeVedtak14aVedtaksdato(searchSourceBuilder, order); + yield searchSourceBuilder; + } + case UTKAST_14A_STATUS -> { + searchSourceBuilder.sort("utkast_14a_status", order); + yield searchSourceBuilder; + } + case ARBEIDSLISTE_KATEGORI -> { + searchSourceBuilder.sort("arbeidsliste_kategori", order); + yield searchSourceBuilder; + } + case SISTE_ENDRING_DATO -> { + sorterSisteEndringTidspunkt(searchSourceBuilder, order, filtervalg); + yield searchSourceBuilder; + } + case ARBEIDSLISTE_OVERSKRIFT -> { + sorterArbeidslisteOverskrift(searchSourceBuilder, order); + yield searchSourceBuilder; + } + case FODELAND -> { + sorterFodeland(searchSourceBuilder, order); + yield searchSourceBuilder; + } + case STATSBORGERSKAP -> { + sorterStatsborgerskap(searchSourceBuilder, order); + yield searchSourceBuilder; + } + case STATSBORGERSKAP_GYLDIG_FRA -> { + sorterStatsborgerskapGyldigFra(searchSourceBuilder, order); + yield searchSourceBuilder; + } + case TOLKESPRAK -> { + sorterTolkeSpraak(filtervalg, searchSourceBuilder, order); + yield searchSourceBuilder; + } + case TOLKEBEHOV_SIST_OPPDATERT -> { + searchSourceBuilder.sort("tolkBehovSistOppdatert", order); + yield searchSourceBuilder; + } + case ENSLIGE_FORSORGERE_UTLOP_YTELSE -> { + sorterEnsligeForsorgereUtlopsDato(searchSourceBuilder, order); + yield searchSourceBuilder; + } + case ENSLIGE_FORSORGERE_VEDTAKSPERIODETYPE -> { + sorterEnsligeForsorgereVedtaksPeriode(searchSourceBuilder, order); + yield searchSourceBuilder; + } + case ENSLIGE_FORSORGERE_AKTIVITETSPLIKT -> { + sorterEnsligeForsorgereAktivitetsPlikt(searchSourceBuilder, order); + yield searchSourceBuilder; + } + case ENSLIGE_FORSORGERE_OM_BARNET -> { + sorterEnsligeForsorgereOmBarnet(searchSourceBuilder, order); + yield searchSourceBuilder; + } + case BARN_UNDER_18_AR -> { + sorterBarnUnder18(searchSourceBuilder, order, brukerinnsynTilganger, filtervalg); + yield searchSourceBuilder; + } + case BRUKERS_SITUASJON_SIST_ENDRET -> { + searchSourceBuilder.sort("brukers_situasjon_sist_endret", order); + yield searchSourceBuilder; + } + case UTDANNING_OG_SITUASJON_SIST_ENDRET -> { + searchSourceBuilder.sort("utdanning_og_situasjon_sist_endret", order); + yield searchSourceBuilder; + } + case HUSKELAPP_FRIST -> { + sorterHuskelappFrist(searchSourceBuilder, order); + yield searchSourceBuilder; + } + case HUSKELAPP -> { + sorterHuskelappEksistere(searchSourceBuilder, order); + yield searchSourceBuilder; + } + case HUSKELAPP_KOMMENTAR -> { + searchSourceBuilder.sort("huskelapp.kommentar", order); + yield searchSourceBuilder; + } + case FARGEKATEGORI -> { + searchSourceBuilder.sort("fargekategori", order); + yield searchSourceBuilder; + } + case TILTAKSHENDELSE_DATO_OPPRETTET -> { + sorterTiltakshendelseOpprettetDato(searchSourceBuilder, order); + yield searchSourceBuilder; + } + case TILTAKSHENDELSE_TEKST -> { + searchSourceBuilder.sort("tiltakshendelse.tekst", order); + yield searchSourceBuilder; + } + case UTGATT_VARSEL_DATO -> { + sorterUtgattVarselHendelseDato(searchSourceBuilder, order); + yield searchSourceBuilder; + } + case ETTERNAVN, CV_SVARFRIST, AAP_MAXTID_UKE, AAP_UNNTAK_UKER_IGJEN, VENTER_PA_SVAR_FRA_NAV, + VENTER_PA_SVAR_FRA_BRUKER, STARTDATO_FOR_AVTALT_AKTIVITET, NESTE_STARTDATO_FOR_AVTALT_AKTIVITET, + FORRIGE_DATO_FOR_AVTALT_AKTIVITET, UTKAST_14A_STATUS_ENDRET, UTKAST_14A_ANSVARLIG_VEILEDER, + BOSTED_KOMMUNE, BOSTED_BYDEL, BOSTED_SIST_OPPDATERT, OPPFOLGING_STARTET, UTLOPSDATO, VEILEDER_IDENT, + DAGPENGER_UTLOP_UKE, DAGPENGER_PERM_UTLOP_UKE -> { + defaultSort(sorteringsfelt, searchSourceBuilder, order); + yield searchSourceBuilder; + } + }; + addSecondarySort(hovedsortering); + return hovedsortering; } private static void brukStandardsorteringBasertPaValgteFilter(Filtervalg filtervalg, SearchSourceBuilder searchSourceBuilder) { boolean filtrertPaTiltakshendelse = filtervalg.ferdigfilterListe != null && filtervalg.ferdigfilterListe.contains(Brukerstatus.TILTAKSHENDELSER); boolean filtrertPaUtgatteVarsel = filtervalg.ferdigfilterListe != null && filtervalg.ferdigfilterListe.contains(Brukerstatus.UTGATTE_VARSEL); boolean filtrertPaEtGjeldendeVedtak14aFilter = filtervalg.gjeldendeVedtak14a.contains("HAR_14A_VEDTAK") || - (filtervalg.innsatsgruppeGjeldendeVedtak14a != null && !filtervalg.innsatsgruppeGjeldendeVedtak14a.isEmpty()) || - (filtervalg.hovedmalGjeldendeVedtak14a != null && !filtervalg.hovedmalGjeldendeVedtak14a.isEmpty()); + (filtervalg.innsatsgruppeGjeldendeVedtak14a != null && !filtervalg.innsatsgruppeGjeldendeVedtak14a.isEmpty()) || + (filtervalg.hovedmalGjeldendeVedtak14a != null && !filtervalg.hovedmalGjeldendeVedtak14a.isEmpty()); if (filtrertPaTiltakshendelse) { sorterTiltakshendelseOpprettetDato(searchSourceBuilder, SortOrder.ASC); - } - else if (filtrertPaUtgatteVarsel) { + } else if (filtrertPaUtgatteVarsel) { sorterUtgattVarselHendelseDato(searchSourceBuilder, SortOrder.ASC); - } - else if (filtrertPaEtGjeldendeVedtak14aFilter) { + } else if (filtrertPaEtGjeldendeVedtak14aFilter) { sorterGjeldendeVedtak14aVedtaksdato(searchSourceBuilder, SortOrder.ASC); } else { searchSourceBuilder.sort("aktoer_id", SortOrder.ASC); @@ -929,7 +1048,7 @@ private static void byggUlestEndringsFilter(List sisteEndringKategori, B BoolQueryBuilder orQuery = boolQuery(); relvanteKategorier.forEach(kategori -> orQuery.should( - QueryBuilders.matchQuery("siste_endringer." + kategori + ".er_sett", "N") + matchQuery("siste_endringer." + kategori + ".er_sett", "N") )); queryBuilder.must(orQuery); } From da7cab4baba5d9f5954956a8b97f543f0c32324f Mon Sep 17 00:00:00 2001 From: Sondre Larsen Ovrid Date: Wed, 22 Jan 2025 21:06:19 +0100 Subject: [PATCH 6/9] Legg til kommentar om koplinga mellom sorteringsfelt, OpenSearch og "OppfolgingsBruker.java" --- .../opensearch/OpensearchQueryBuilder.java | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/src/main/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchQueryBuilder.java b/src/main/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchQueryBuilder.java index 8efcaf701b..c4e33f508a 100644 --- a/src/main/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchQueryBuilder.java +++ b/src/main/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchQueryBuilder.java @@ -9,6 +9,7 @@ import no.nav.pto.veilarbportefolje.domene.filtervalg.UtdanningGodkjentSvar; import no.nav.pto.veilarbportefolje.domene.filtervalg.UtdanningSvar; import no.nav.pto.veilarbportefolje.fargekategori.FargekategoriVerdi; +import no.nav.pto.veilarbportefolje.opensearch.domene.OppfolgingsBruker; import no.nav.pto.veilarbportefolje.persononinfo.domene.Adressebeskyttelse; import no.nav.pto.veilarbportefolje.sisteendring.SisteEndringsKategori; import no.nav.pto.veilarbportefolje.util.ValideringsRegler; @@ -443,13 +444,14 @@ static List byggAktivitetFilterQuery(Filtervalg filtervalg, Bo } /** + * Tar i mot en {@link SearchSourceBuilder} og utvider denne med sorteringsqueryer basert på valgt sorteringsrekkefølge, + * sorteringsfelt og valgte filter. * - * @param sortOrder - * @param sortField - * @param searchSourceBuilder - * @param filtervalg - * @param brukerinnsynTilganger - * @return + * Merk: {@link Sorteringsfelt} er en enum som representerer lovlige sorteringsfelter slik frontend har definert dem. + * Disse mappes til felter i OpenSearch, dvs. for enkelte verdier av {@link Sorteringsfelt} kan det være at feltet + * det faktisk sorteres på heter noe annet i OpenSearch. Siden {@link OppfolgingsBruker} er "fasiten" for hvilke felter + * som er lov å sortere på i OpenSearch er det viktig at vi ikke legger til nye sorteringsfelter i {@link Sorteringsfelt} + * uten å sørge for at disse også er tilgjengelige i {@link OppfolgingsBruker}. */ static SearchSourceBuilder sorterQueryParametere(String sortOrder, String sortField, SearchSourceBuilder searchSourceBuilder, Filtervalg filtervalg, BrukerinnsynTilganger brukerinnsynTilganger) { From a32e253bf08b53571ca1556616f5a23222cdeed9 Mon Sep 17 00:00:00 2001 From: Sondre Larsen Ovrid Date: Wed, 22 Jan 2025 21:09:48 +0100 Subject: [PATCH 7/9] =?UTF-8?q?Fjern=20un=C3=B8dvendig=20validering=20og?= =?UTF-8?q?=20legg=20til=20kommentarar=20om=20exhaustiveness-check?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../opensearch/OpensearchQueryBuilder.java | 24 ++++++------------- 1 file changed, 7 insertions(+), 17 deletions(-) diff --git a/src/main/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchQueryBuilder.java b/src/main/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchQueryBuilder.java index c4e33f508a..85dd62f015 100644 --- a/src/main/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchQueryBuilder.java +++ b/src/main/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchQueryBuilder.java @@ -12,7 +12,6 @@ import no.nav.pto.veilarbportefolje.opensearch.domene.OppfolgingsBruker; import no.nav.pto.veilarbportefolje.persononinfo.domene.Adressebeskyttelse; import no.nav.pto.veilarbportefolje.sisteendring.SisteEndringsKategori; -import no.nav.pto.veilarbportefolje.util.ValideringsRegler; import org.apache.lucene.search.join.ScoreMode; import org.opensearch.index.query.*; import org.opensearch.script.Script; @@ -446,7 +445,7 @@ static List byggAktivitetFilterQuery(Filtervalg filtervalg, Bo /** * Tar i mot en {@link SearchSourceBuilder} og utvider denne med sorteringsqueryer basert på valgt sorteringsrekkefølge, * sorteringsfelt og valgte filter. - * + *

* Merk: {@link Sorteringsfelt} er en enum som representerer lovlige sorteringsfelter slik frontend har definert dem. * Disse mappes til felter i OpenSearch, dvs. for enkelte verdier av {@link Sorteringsfelt} kan det være at feltet * det faktisk sorteres på heter noe annet i OpenSearch. Siden {@link OppfolgingsBruker} er "fasiten" for hvilke felter @@ -460,6 +459,9 @@ static SearchSourceBuilder sorterQueryParametere(String sortOrder, String /* På sikt (tm) skal vi typesikre sortField slik at vi får Sorteringsfelt her, gjerne allereie på Controller-nivå. I denne omgangen lagar eg berre enumen for sorteringsfelta. 2024-11-28, Ingrid. */ Sorteringsfelt sorteringsfelt = Sorteringsfelt.nameFromValue(sortField); + // Vi må assigne til en ny variabel for at kompilatoren sin exhaustiveness-check skal slå inn. + // Dette er strengt tatt ikke nødvendig da vi bare kunne returnert searchSourceBuilder direkte, men da ville vi + // mistet exhaustiveness-checken. SearchSourceBuilder hovedsortering = switch (sorteringsfelt) { case IKKE_SATT -> { brukStandardsorteringBasertPaValgteFilter(filtervalg, searchSourceBuilder); @@ -612,12 +614,14 @@ static SearchSourceBuilder sorterQueryParametere(String sortOrder, String sorterUtgattVarselHendelseDato(searchSourceBuilder, order); yield searchSourceBuilder; } + // Vi har eksplisitt latt være å definere en "default" case i switch-en for å tvinge oss selv til å håndtere + // alle sorteringsfeltene (exhaustivness check som gjøres av kompilatoren). Så i praksis er dette default-tilfellet. case ETTERNAVN, CV_SVARFRIST, AAP_MAXTID_UKE, AAP_UNNTAK_UKER_IGJEN, VENTER_PA_SVAR_FRA_NAV, VENTER_PA_SVAR_FRA_BRUKER, STARTDATO_FOR_AVTALT_AKTIVITET, NESTE_STARTDATO_FOR_AVTALT_AKTIVITET, FORRIGE_DATO_FOR_AVTALT_AKTIVITET, UTKAST_14A_STATUS_ENDRET, UTKAST_14A_ANSVARLIG_VEILEDER, BOSTED_KOMMUNE, BOSTED_BYDEL, BOSTED_SIST_OPPDATERT, OPPFOLGING_STARTET, UTLOPSDATO, VEILEDER_IDENT, DAGPENGER_UTLOP_UKE, DAGPENGER_PERM_UTLOP_UKE -> { - defaultSort(sorteringsfelt, searchSourceBuilder, order); + searchSourceBuilder.sort(sorteringsfelt.sorteringsverdi, order); yield searchSourceBuilder; } }; @@ -1066,20 +1070,6 @@ private static void byggSisteEndringFilter(List sisteEndringKategori, Bo queryBuilder.must(existsQuery("siste_endringer." + sisteEndringKategori.get(0))); } - /** - * Sorter alfabetisk på OpenSearch-feltet som er likt filterverdien. - * Eksempel der det fungerer: "etternavn" (filter) og "etternavn" (OpenSearch) - * Eksempel der det ikkje fungerer: "aap_type" (filter) og "ytelse" (OpenSearch), eller - * "gjeldende_vedtak_14a_innsatsgruppe" (filter) og "gjeldendeVedtak14a.innsatsgruppe" (OpenSearch) - */ - private static void defaultSort(Sorteringsfelt sortField, SearchSourceBuilder searchSourceBuilder, SortOrder order) { - if (ValideringsRegler.sortFields.contains(sortField.sorteringsverdi)) { - searchSourceBuilder.sort(sortField.sorteringsverdi, order); - } else { - throw new IllegalStateException(); - } - } - private static void sorterEnsligeForsorgereUtlopsDato(SearchSourceBuilder builder, SortOrder order) { String expresion = """ if (doc.containsKey('enslige_forsorgere_overgangsstonad.utlopsDato') && !doc['enslige_forsorgere_overgangsstonad.utlopsDato'].empty) { From 091c1af5a00544e15b486cc7e351a6c7f8bf6f1d Mon Sep 17 00:00:00 2001 From: Sondre Larsen Ovrid Date: Wed, 22 Jan 2025 21:39:14 +0100 Subject: [PATCH 8/9] =?UTF-8?q?Konverter=20til=20validert=20type=20s=C3=A5?= =?UTF-8?q?=20tidleg=20som=20mogleg=20og=20rydd=20i/forenkle=20koden?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../controller/EnhetController.java | 6 +- .../controller/VeilederController.java | 6 +- .../domene/Sorteringsfelt.java | 6 +- .../domene/Sorteringsrekkefolge.java | 30 ++ .../opensearch/OpensearchQueryBuilder.java | 88 +++-- .../opensearch/OpensearchService.java | 6 +- .../util/ValideringsRegler.java | 88 +---- .../AktiviteterOpensearchIntegrasjonTest.java | 16 +- .../LonnstilskuddUtAvArenaTest.java | 28 +- .../ArbeidslistaSorteringOpensearchTest.java | 17 +- .../EnsligeForsorgereServiceTest.java | 12 +- .../HendelseIntegrationTest.kt | 41 +- .../OpensearchQueryBuilderTest.java | 23 +- .../OpensearchServiceIntegrationTest.java | 373 +++++++++--------- .../SisteEndringIntegrationTest.java | 76 ++-- 15 files changed, 371 insertions(+), 445 deletions(-) create mode 100644 src/main/java/no/nav/pto/veilarbportefolje/domene/Sorteringsrekkefolge.java diff --git a/src/main/java/no/nav/pto/veilarbportefolje/controller/EnhetController.java b/src/main/java/no/nav/pto/veilarbportefolje/controller/EnhetController.java index 6122d47a8d..35c5393fb9 100644 --- a/src/main/java/no/nav/pto/veilarbportefolje/controller/EnhetController.java +++ b/src/main/java/no/nav/pto/veilarbportefolje/controller/EnhetController.java @@ -61,12 +61,12 @@ public Portefolje hentPortefoljeForEnhet( @RequestBody Filtervalg filtervalg) { ValideringsRegler.sjekkEnhet(enhet); - ValideringsRegler.sjekkSortering(sortDirection, sortField); - ValideringsRegler.sjekkFiltervalg(filtervalg); + Sorteringsrekkefolge validertSorteringsrekkefolge = ValideringsRegler.sjekkSorteringsrekkefolge(sortDirection); + Sorteringsfelt validertSorteringsfelt = ValideringsRegler.sjekkSorteringsfelt(sortField); authService.innloggetVeilederHarTilgangTilOppfolging(); authService.innloggetVeilederHarTilgangTilEnhet(enhet); - BrukereMedAntall brukereMedAntall = opensearchService.hentBrukere(enhet, Optional.empty(), sortDirection, sortField, filtervalg, fra, antall); + BrukereMedAntall brukereMedAntall = opensearchService.hentBrukere(enhet, Optional.empty(), validertSorteringsrekkefolge, validertSorteringsfelt, filtervalg, fra, antall); List sensurerteBrukereSublist = authService.sensurerBrukere(brukereMedAntall.getBrukere()); return PortefoljeUtils.buildPortefolje(brukereMedAntall.getAntall(), diff --git a/src/main/java/no/nav/pto/veilarbportefolje/controller/VeilederController.java b/src/main/java/no/nav/pto/veilarbportefolje/controller/VeilederController.java index bf96bb87e6..c0f3f05ca5 100644 --- a/src/main/java/no/nav/pto/veilarbportefolje/controller/VeilederController.java +++ b/src/main/java/no/nav/pto/veilarbportefolje/controller/VeilederController.java @@ -44,12 +44,12 @@ public Portefolje hentPortefoljeForVeileder( ValideringsRegler.sjekkVeilederIdent(veilederIdent, false); ValideringsRegler.sjekkEnhet(enhet); - ValideringsRegler.sjekkSortering(sortDirection, sortField); - ValideringsRegler.sjekkFiltervalg(filtervalg); + Sorteringsrekkefolge validertSorteringsrekkefolge = ValideringsRegler.sjekkSorteringsrekkefolge(sortDirection); + Sorteringsfelt validertSorteringsfelt = ValideringsRegler.sjekkSorteringsfelt(sortField); authService.innloggetVeilederHarTilgangTilOppfolging(); authService.innloggetVeilederHarTilgangTilEnhet(enhet); - BrukereMedAntall brukereMedAntall = opensearchService.hentBrukere(enhet, Optional.of(veilederIdent), sortDirection, sortField, filtervalg, fra, antall); + BrukereMedAntall brukereMedAntall = opensearchService.hentBrukere(enhet, Optional.of(veilederIdent), validertSorteringsrekkefolge, validertSorteringsfelt, filtervalg, fra, antall); List sensurerteBrukereSublist = authService.sensurerBrukere(brukereMedAntall.getBrukere()); return PortefoljeUtils.buildPortefolje(brukereMedAntall.getAntall(), diff --git a/src/main/java/no/nav/pto/veilarbportefolje/domene/Sorteringsfelt.java b/src/main/java/no/nav/pto/veilarbportefolje/domene/Sorteringsfelt.java index d7862ff79b..4d765c8d5a 100644 --- a/src/main/java/no/nav/pto/veilarbportefolje/domene/Sorteringsfelt.java +++ b/src/main/java/no/nav/pto/veilarbportefolje/domene/Sorteringsfelt.java @@ -101,13 +101,13 @@ public enum Sorteringsfelt { this.sorteringsverdi = sorteringsverdi; } - public static Sorteringsfelt nameFromValue(String value) { + public static Sorteringsfelt toSorteringsfelt(String sorteringsverdi) { for (Sorteringsfelt sorteringsfelt : values()) { - if (sorteringsfelt.sorteringsverdi.equals(value)) { + if (sorteringsfelt.sorteringsverdi.equals(sorteringsverdi)) { return sorteringsfelt; } } - throw new IllegalArgumentException("Ugyldig verdi for enum: " + value ); + throw new IllegalArgumentException("Ugyldig verdi for enum: " + sorteringsverdi); } @Override diff --git a/src/main/java/no/nav/pto/veilarbportefolje/domene/Sorteringsrekkefolge.java b/src/main/java/no/nav/pto/veilarbportefolje/domene/Sorteringsrekkefolge.java new file mode 100644 index 0000000000..ebec025c5e --- /dev/null +++ b/src/main/java/no/nav/pto/veilarbportefolje/domene/Sorteringsrekkefolge.java @@ -0,0 +1,30 @@ +package no.nav.pto.veilarbportefolje.domene; + +public enum Sorteringsrekkefolge { + IKKE_SATT("ikke_satt"), + STIGENDE("ascending"), + SYNKENDE("descending"); + + /** + * Verdien som blir sendt mellom frontend og backend + */ + public final String sorteringsverdi; + + Sorteringsrekkefolge(String sorteringsverdi) { + this.sorteringsverdi = sorteringsverdi; + } + + public static Sorteringsrekkefolge toSorteringsrekkefolge(String sorteringsverdi) { + for (Sorteringsrekkefolge sorteringsrekkefolge : values()) { + if (sorteringsrekkefolge.sorteringsverdi.equals(sorteringsverdi)) { + return sorteringsrekkefolge; + } + } + throw new IllegalArgumentException("Ugyldig verdi for enum: " + sorteringsverdi); + } + + @Override + public String toString() { + return this.name() + " (" + this.sorteringsverdi + ")"; + } +} diff --git a/src/main/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchQueryBuilder.java b/src/main/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchQueryBuilder.java index 85dd62f015..3577c5cf9d 100644 --- a/src/main/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchQueryBuilder.java +++ b/src/main/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchQueryBuilder.java @@ -452,12 +452,14 @@ static List byggAktivitetFilterQuery(Filtervalg filtervalg, Bo * som er lov å sortere på i OpenSearch er det viktig at vi ikke legger til nye sorteringsfelter i {@link Sorteringsfelt} * uten å sørge for at disse også er tilgjengelige i {@link OppfolgingsBruker}. */ - static SearchSourceBuilder sorterQueryParametere(String sortOrder, String - sortField, SearchSourceBuilder searchSourceBuilder, Filtervalg filtervalg, BrukerinnsynTilganger brukerinnsynTilganger) { - SortOrder order = "ascending".equals(sortOrder) ? SortOrder.ASC : SortOrder.DESC; - - /* På sikt (tm) skal vi typesikre sortField slik at vi får Sorteringsfelt her, gjerne allereie på Controller-nivå. I denne omgangen lagar eg berre enumen for sorteringsfelta. 2024-11-28, Ingrid. */ - Sorteringsfelt sorteringsfelt = Sorteringsfelt.nameFromValue(sortField); + static SearchSourceBuilder sorterQueryParametere( + Sorteringsrekkefolge sorteringsrekkefolge, + Sorteringsfelt sorteringsfelt, + SearchSourceBuilder searchSourceBuilder, + Filtervalg filtervalg, + BrukerinnsynTilganger brukerinnsynTilganger + ) { + SortOrder sorteringsrekkefolgeOpenSearch = Sorteringsrekkefolge.STIGENDE.equals(sorteringsrekkefolge) ? SortOrder.ASC : SortOrder.DESC; // Vi må assigne til en ny variabel for at kompilatoren sin exhaustiveness-check skal slå inn. // Dette er strengt tatt ikke nødvendig da vi bare kunne returnert searchSourceBuilder direkte, men da ville vi @@ -468,150 +470,150 @@ static SearchSourceBuilder sorterQueryParametere(String sortOrder, String yield searchSourceBuilder; } case VALGTE_AKTIVITETER -> { - sorterValgteAktiviteter(filtervalg, searchSourceBuilder, order); + sorterValgteAktiviteter(filtervalg, searchSourceBuilder, sorteringsrekkefolgeOpenSearch); yield searchSourceBuilder; } case MOTER_MED_NAV_IDAG -> { - searchSourceBuilder.sort("alle_aktiviteter_mote_startdato", order); + searchSourceBuilder.sort("alle_aktiviteter_mote_startdato", sorteringsrekkefolgeOpenSearch); yield searchSourceBuilder; } case MOTESTATUS -> { - searchSourceBuilder.sort("aktivitet_mote_startdato", order); + searchSourceBuilder.sort("aktivitet_mote_startdato", sorteringsrekkefolgeOpenSearch); yield searchSourceBuilder; } case I_AVTALT_AKTIVITET -> { FieldSortBuilder builder = new FieldSortBuilder("aktivitet_utlopsdatoer") - .order(order) + .order(sorteringsrekkefolgeOpenSearch) .sortMode(MIN); searchSourceBuilder.sort(builder); yield searchSourceBuilder; } case FODSELSNUMMER -> { - searchSourceBuilder.sort("fnr.raw", order); + searchSourceBuilder.sort("fnr.raw", sorteringsrekkefolgeOpenSearch); yield searchSourceBuilder; } case UTLOPTE_AKTIVITETER -> { - searchSourceBuilder.sort("nyesteutlopteaktivitet", order); + searchSourceBuilder.sort("nyesteutlopteaktivitet", sorteringsrekkefolgeOpenSearch); yield searchSourceBuilder; } case ARBEIDSLISTE_FRIST -> { - searchSourceBuilder.sort("arbeidsliste_frist", order); + searchSourceBuilder.sort("arbeidsliste_frist", sorteringsrekkefolgeOpenSearch); yield searchSourceBuilder; } case AAP_TYPE -> { - searchSourceBuilder.sort("ytelse", order); + searchSourceBuilder.sort("ytelse", sorteringsrekkefolgeOpenSearch); yield searchSourceBuilder; } case AAP_VURDERINGSFRIST -> { - sorterAapVurderingsfrist(searchSourceBuilder, order, filtervalg); + sorterAapVurderingsfrist(searchSourceBuilder, sorteringsrekkefolgeOpenSearch, filtervalg); yield searchSourceBuilder; } case AAP_RETTIGHETSPERIODE -> { - sorterAapRettighetsPeriode(searchSourceBuilder, order); + sorterAapRettighetsPeriode(searchSourceBuilder, sorteringsrekkefolgeOpenSearch); yield searchSourceBuilder; } case GJELDENDE_VEDTAK_14A_INNSATSGRUPPE -> { - searchSourceBuilder.sort("gjeldendeVedtak14a.innsatsgruppe", order); + searchSourceBuilder.sort("gjeldendeVedtak14a.innsatsgruppe", sorteringsrekkefolgeOpenSearch); yield searchSourceBuilder; } case GJELDENDE_VEDTAK_14A_HOVEDMAL -> { - searchSourceBuilder.sort("gjeldendeVedtak14a.hovedmal", order); + searchSourceBuilder.sort("gjeldendeVedtak14a.hovedmal", sorteringsrekkefolgeOpenSearch); yield searchSourceBuilder; } case GJELDENDE_VEDTAK_14A_VEDTAKSDATO -> { - sorterGjeldendeVedtak14aVedtaksdato(searchSourceBuilder, order); + sorterGjeldendeVedtak14aVedtaksdato(searchSourceBuilder, sorteringsrekkefolgeOpenSearch); yield searchSourceBuilder; } case UTKAST_14A_STATUS -> { - searchSourceBuilder.sort("utkast_14a_status", order); + searchSourceBuilder.sort("utkast_14a_status", sorteringsrekkefolgeOpenSearch); yield searchSourceBuilder; } case ARBEIDSLISTE_KATEGORI -> { - searchSourceBuilder.sort("arbeidsliste_kategori", order); + searchSourceBuilder.sort("arbeidsliste_kategori", sorteringsrekkefolgeOpenSearch); yield searchSourceBuilder; } case SISTE_ENDRING_DATO -> { - sorterSisteEndringTidspunkt(searchSourceBuilder, order, filtervalg); + sorterSisteEndringTidspunkt(searchSourceBuilder, sorteringsrekkefolgeOpenSearch, filtervalg); yield searchSourceBuilder; } case ARBEIDSLISTE_OVERSKRIFT -> { - sorterArbeidslisteOverskrift(searchSourceBuilder, order); + sorterArbeidslisteOverskrift(searchSourceBuilder, sorteringsrekkefolgeOpenSearch); yield searchSourceBuilder; } case FODELAND -> { - sorterFodeland(searchSourceBuilder, order); + sorterFodeland(searchSourceBuilder, sorteringsrekkefolgeOpenSearch); yield searchSourceBuilder; } case STATSBORGERSKAP -> { - sorterStatsborgerskap(searchSourceBuilder, order); + sorterStatsborgerskap(searchSourceBuilder, sorteringsrekkefolgeOpenSearch); yield searchSourceBuilder; } case STATSBORGERSKAP_GYLDIG_FRA -> { - sorterStatsborgerskapGyldigFra(searchSourceBuilder, order); + sorterStatsborgerskapGyldigFra(searchSourceBuilder, sorteringsrekkefolgeOpenSearch); yield searchSourceBuilder; } case TOLKESPRAK -> { - sorterTolkeSpraak(filtervalg, searchSourceBuilder, order); + sorterTolkeSpraak(filtervalg, searchSourceBuilder, sorteringsrekkefolgeOpenSearch); yield searchSourceBuilder; } case TOLKEBEHOV_SIST_OPPDATERT -> { - searchSourceBuilder.sort("tolkBehovSistOppdatert", order); + searchSourceBuilder.sort("tolkBehovSistOppdatert", sorteringsrekkefolgeOpenSearch); yield searchSourceBuilder; } case ENSLIGE_FORSORGERE_UTLOP_YTELSE -> { - sorterEnsligeForsorgereUtlopsDato(searchSourceBuilder, order); + sorterEnsligeForsorgereUtlopsDato(searchSourceBuilder, sorteringsrekkefolgeOpenSearch); yield searchSourceBuilder; } case ENSLIGE_FORSORGERE_VEDTAKSPERIODETYPE -> { - sorterEnsligeForsorgereVedtaksPeriode(searchSourceBuilder, order); + sorterEnsligeForsorgereVedtaksPeriode(searchSourceBuilder, sorteringsrekkefolgeOpenSearch); yield searchSourceBuilder; } case ENSLIGE_FORSORGERE_AKTIVITETSPLIKT -> { - sorterEnsligeForsorgereAktivitetsPlikt(searchSourceBuilder, order); + sorterEnsligeForsorgereAktivitetsPlikt(searchSourceBuilder, sorteringsrekkefolgeOpenSearch); yield searchSourceBuilder; } case ENSLIGE_FORSORGERE_OM_BARNET -> { - sorterEnsligeForsorgereOmBarnet(searchSourceBuilder, order); + sorterEnsligeForsorgereOmBarnet(searchSourceBuilder, sorteringsrekkefolgeOpenSearch); yield searchSourceBuilder; } case BARN_UNDER_18_AR -> { - sorterBarnUnder18(searchSourceBuilder, order, brukerinnsynTilganger, filtervalg); + sorterBarnUnder18(searchSourceBuilder, sorteringsrekkefolgeOpenSearch, brukerinnsynTilganger, filtervalg); yield searchSourceBuilder; } case BRUKERS_SITUASJON_SIST_ENDRET -> { - searchSourceBuilder.sort("brukers_situasjon_sist_endret", order); + searchSourceBuilder.sort("brukers_situasjon_sist_endret", sorteringsrekkefolgeOpenSearch); yield searchSourceBuilder; } case UTDANNING_OG_SITUASJON_SIST_ENDRET -> { - searchSourceBuilder.sort("utdanning_og_situasjon_sist_endret", order); + searchSourceBuilder.sort("utdanning_og_situasjon_sist_endret", sorteringsrekkefolgeOpenSearch); yield searchSourceBuilder; } case HUSKELAPP_FRIST -> { - sorterHuskelappFrist(searchSourceBuilder, order); + sorterHuskelappFrist(searchSourceBuilder, sorteringsrekkefolgeOpenSearch); yield searchSourceBuilder; } case HUSKELAPP -> { - sorterHuskelappEksistere(searchSourceBuilder, order); + sorterHuskelappEksistere(searchSourceBuilder, sorteringsrekkefolgeOpenSearch); yield searchSourceBuilder; } case HUSKELAPP_KOMMENTAR -> { - searchSourceBuilder.sort("huskelapp.kommentar", order); + searchSourceBuilder.sort("huskelapp.kommentar", sorteringsrekkefolgeOpenSearch); yield searchSourceBuilder; } case FARGEKATEGORI -> { - searchSourceBuilder.sort("fargekategori", order); + searchSourceBuilder.sort("fargekategori", sorteringsrekkefolgeOpenSearch); yield searchSourceBuilder; } case TILTAKSHENDELSE_DATO_OPPRETTET -> { - sorterTiltakshendelseOpprettetDato(searchSourceBuilder, order); + sorterTiltakshendelseOpprettetDato(searchSourceBuilder, sorteringsrekkefolgeOpenSearch); yield searchSourceBuilder; } case TILTAKSHENDELSE_TEKST -> { - searchSourceBuilder.sort("tiltakshendelse.tekst", order); + searchSourceBuilder.sort("tiltakshendelse.tekst", sorteringsrekkefolgeOpenSearch); yield searchSourceBuilder; } case UTGATT_VARSEL_DATO -> { - sorterUtgattVarselHendelseDato(searchSourceBuilder, order); + sorterUtgattVarselHendelseDato(searchSourceBuilder, sorteringsrekkefolgeOpenSearch); yield searchSourceBuilder; } // Vi har eksplisitt latt være å definere en "default" case i switch-en for å tvinge oss selv til å håndtere @@ -621,7 +623,7 @@ static SearchSourceBuilder sorterQueryParametere(String sortOrder, String FORRIGE_DATO_FOR_AVTALT_AKTIVITET, UTKAST_14A_STATUS_ENDRET, UTKAST_14A_ANSVARLIG_VEILEDER, BOSTED_KOMMUNE, BOSTED_BYDEL, BOSTED_SIST_OPPDATERT, OPPFOLGING_STARTET, UTLOPSDATO, VEILEDER_IDENT, DAGPENGER_UTLOP_UKE, DAGPENGER_PERM_UTLOP_UKE -> { - searchSourceBuilder.sort(sorteringsfelt.sorteringsverdi, order); + searchSourceBuilder.sort(sorteringsfelt.sorteringsverdi, sorteringsrekkefolgeOpenSearch); yield searchSourceBuilder; } }; diff --git a/src/main/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchService.java b/src/main/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchService.java index 58675b073b..c4a48211ec 100644 --- a/src/main/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchService.java +++ b/src/main/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchService.java @@ -50,8 +50,8 @@ public class OpensearchService { public BrukereMedAntall hentBrukere( String enhetId, Optional veilederIdent, - String sortOrder, - String sortField, + Sorteringsrekkefolge sorteringsrekkefolge, + Sorteringsfelt sorteringsfelt, Filtervalg filtervalg, Integer fra, Integer antall @@ -102,7 +102,7 @@ public BrukereMedAntall hentBrukere( leggTilBrukerinnsynTilgangFilter(boolQuery, authService.hentVeilederBrukerInnsynTilganger(), BRUKERE_SOM_VEILEDER_HAR_INNSYNSRETT_PÅ); } - sorterQueryParametere(sortOrder, sortField, searchSourceBuilder, filtervalg, authService.hentVeilederBrukerInnsynTilganger()); + sorterQueryParametere(sorteringsrekkefolge, sorteringsfelt, searchSourceBuilder, filtervalg, authService.hentVeilederBrukerInnsynTilganger()); OpensearchResponse response = search(searchSourceBuilder, indexName.getValue(), OpensearchResponse.class); int totalHits = response.hits().getTotal().getValue(); diff --git a/src/main/java/no/nav/pto/veilarbportefolje/util/ValideringsRegler.java b/src/main/java/no/nav/pto/veilarbportefolje/util/ValideringsRegler.java index cd21eea5c5..cdb265ece5 100644 --- a/src/main/java/no/nav/pto/veilarbportefolje/util/ValideringsRegler.java +++ b/src/main/java/no/nav/pto/veilarbportefolje/util/ValideringsRegler.java @@ -7,7 +7,8 @@ import no.nav.pto.veilarbportefolje.arbeidsliste.ArbeidslisteDTO; import no.nav.pto.veilarbportefolje.arbeidsliste.v1.ArbeidslisteRequest; import no.nav.pto.veilarbportefolje.arbeidsliste.v2.ArbeidslisteV2Request; -import no.nav.pto.veilarbportefolje.domene.Filtervalg; +import no.nav.pto.veilarbportefolje.domene.Sorteringsfelt; +import no.nav.pto.veilarbportefolje.domene.Sorteringsrekkefolge; import org.springframework.http.HttpStatus; import org.springframework.web.server.ResponseStatusException; @@ -16,97 +17,36 @@ import java.time.LocalDate; import java.util.ArrayList; import java.util.List; -import java.util.function.Supplier; import static io.vavr.control.Validation.invalid; import static io.vavr.control.Validation.valid; import static java.lang.String.format; -import static java.util.Arrays.asList; import static no.nav.common.utils.StringUtils.nullOrEmpty; -import static no.nav.pto.veilarbportefolje.domene.Sorteringsfelt.*; public class ValideringsRegler { - private static List sortDirs = asList("ikke_satt", "ascending", "descending"); - public static List sortFields = asList( - IKKE_SATT.sorteringsverdi, - VALGTE_AKTIVITETER.sorteringsverdi, - ETTERNAVN.sorteringsverdi, - FODSELSNUMMER.sorteringsverdi, - UTLOPSDATO.sorteringsverdi, - DAGPENGER_UTLOP_UKE.sorteringsverdi, - DAGPENGER_PERM_UTLOP_UKE.sorteringsverdi, - AAP_TYPE.sorteringsverdi, - AAP_VURDERINGSFRIST.sorteringsverdi, - AAP_MAXTID_UKE.sorteringsverdi, - AAP_UNNTAK_UKER_IGJEN.sorteringsverdi, - ARBEIDSLISTE_FRIST.sorteringsverdi, - VENTER_PA_SVAR_FRA_NAV.sorteringsverdi, - VENTER_PA_SVAR_FRA_BRUKER.sorteringsverdi, - UTLOPTE_AKTIVITETER.sorteringsverdi, - STARTDATO_FOR_AVTALT_AKTIVITET.sorteringsverdi, - NESTE_STARTDATO_FOR_AVTALT_AKTIVITET.sorteringsverdi, - FORRIGE_DATO_FOR_AVTALT_AKTIVITET.sorteringsverdi, - I_AVTALT_AKTIVITET.sorteringsverdi, - AAP_RETTIGHETSPERIODE.sorteringsverdi, - MOTER_MED_NAV_IDAG.sorteringsverdi, - MOTESTATUS.sorteringsverdi, - OPPFOLGING_STARTET.sorteringsverdi, - VEILEDER_IDENT.sorteringsverdi, - GJELDENDE_VEDTAK_14A_INNSATSGRUPPE.sorteringsverdi, - GJELDENDE_VEDTAK_14A_HOVEDMAL.sorteringsverdi, - GJELDENDE_VEDTAK_14A_VEDTAKSDATO.sorteringsverdi, - UTKAST_14A_STATUS.sorteringsverdi, - UTKAST_14A_ANSVARLIG_VEILEDER.sorteringsverdi, - UTKAST_14A_STATUS_ENDRET.sorteringsverdi, - ARBEIDSLISTE_KATEGORI.sorteringsverdi, - ARBEIDSLISTE_OVERSKRIFT.sorteringsverdi, - SISTE_ENDRING_DATO.sorteringsverdi, - FODELAND.sorteringsverdi, - STATSBORGERSKAP.sorteringsverdi, - STATSBORGERSKAP_GYLDIG_FRA.sorteringsverdi, - TOLKESPRAK.sorteringsverdi, - TOLKEBEHOV_SIST_OPPDATERT.sorteringsverdi, - BOSTED_KOMMUNE.sorteringsverdi, - BOSTED_BYDEL.sorteringsverdi, - BOSTED_SIST_OPPDATERT.sorteringsverdi, - CV_SVARFRIST.sorteringsverdi, - ENSLIGE_FORSORGERE_UTLOP_YTELSE.sorteringsverdi, - ENSLIGE_FORSORGERE_VEDTAKSPERIODETYPE.sorteringsverdi, - ENSLIGE_FORSORGERE_AKTIVITETSPLIKT.sorteringsverdi, - ENSLIGE_FORSORGERE_OM_BARNET.sorteringsverdi, - BARN_UNDER_18_AR.sorteringsverdi, - BRUKERS_SITUASJON_SIST_ENDRET.sorteringsverdi, - HUSKELAPP_FRIST.sorteringsverdi, - HUSKELAPP_KOMMENTAR.sorteringsverdi, - HUSKELAPP.sorteringsverdi, - FARGEKATEGORI.sorteringsverdi, - UTDANNING_OG_SITUASJON_SIST_ENDRET.sorteringsverdi, - TILTAKSHENDELSE_DATO_OPPRETTET.sorteringsverdi, - TILTAKSHENDELSE_TEKST.sorteringsverdi, - UTGATT_VARSEL_DATO.sorteringsverdi - ); - public static void sjekkEnhet(String enhet) { test("enhet", enhet, enhet.matches("\\d{4}")); } - public static void sjekkVeilederIdent(String veilederIdent, boolean optional) { test("veilederident", veilederIdent, optional || veilederIdent.matches("[A-Z]\\d{6}")); } - public static void sjekkFiltervalg(Filtervalg filtervalg) { - test("filtervalg", filtervalg, filtervalg::valider); - } - - public static void sjekkSortering(String sortDirection, String sortField) { - test("sortDirection", sortDirection, sortDirs.contains(sortDirection)); - test("sortField", sortField, sortFields.contains(sortField)); + public static Sorteringsfelt sjekkSorteringsfelt(String sorteringsFelt) { + try { + return Sorteringsfelt.toSorteringsfelt(sorteringsFelt); + } catch (Exception e) { + throw new ResponseStatusException(HttpStatus.BAD_REQUEST, format("%s er ikke et gyldig sorteringsfelt", sorteringsFelt)); + } } - private static void test(String navn, Object data, Supplier matches) { - test(navn, data, matches.get()); + public static Sorteringsrekkefolge sjekkSorteringsrekkefolge(String sorteringsRekkefolge) { + try { + return Sorteringsrekkefolge.toSorteringsrekkefolge(sorteringsRekkefolge); + } catch (Exception e) { + throw new ResponseStatusException(HttpStatus.BAD_REQUEST, format("%s er ikke en gyldig sorteringsrekkefølge", sorteringsRekkefolge)); + } } private static void test(String navn, Object data, boolean matches) { diff --git a/src/test/java/no/nav/pto/veilarbportefolje/aktiviteter/AktiviteterOpensearchIntegrasjonTest.java b/src/test/java/no/nav/pto/veilarbportefolje/aktiviteter/AktiviteterOpensearchIntegrasjonTest.java index 7a2b75d3a8..0e2101039c 100644 --- a/src/test/java/no/nav/pto/veilarbportefolje/aktiviteter/AktiviteterOpensearchIntegrasjonTest.java +++ b/src/test/java/no/nav/pto/veilarbportefolje/aktiviteter/AktiviteterOpensearchIntegrasjonTest.java @@ -80,8 +80,8 @@ public void lasteroppeikkelagreteaktiviteteter() { BrukereMedAntall responseBrukere = opensearchService.hentBrukere( navKontor.getValue(), empty(), - "asc", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, new Filtervalg().setFerdigfilterListe(List.of(I_AKTIVITET)), null, null); @@ -114,8 +114,8 @@ public void lasteroppaktivitetStillingFraNAV() { BrukereMedAntall responseBrukere = opensearchService.hentBrukere( navKontor.getValue(), empty(), - "asc", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, new Filtervalg().setNavnEllerFnrQuery(fodselsnummer.toString()).setFerdigfilterListe(new ArrayList<>()), null, null); @@ -165,8 +165,8 @@ public void filtrerBrukerePaStillingFraNAV() { BrukereMedAntall responseBrukere = opensearchService.hentBrukere( navKontor.getValue(), empty(), - "asc", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, new Filtervalg().setFerdigfilterListe(List.of(I_AKTIVITET)), null, null); @@ -179,8 +179,8 @@ public void filtrerBrukerePaStillingFraNAV() { BrukereMedAntall responseBrukere = opensearchService.hentBrukere( navKontor.getValue(), empty(), - "asc", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, new Filtervalg().setStillingFraNavFilter(List.of(StillingFraNAVFilter.CV_KAN_DELES_STATUS_JA)).setFerdigfilterListe(new ArrayList<>()), null, null); diff --git a/src/test/java/no/nav/pto/veilarbportefolje/aktiviteter/LonnstilskuddUtAvArenaTest.java b/src/test/java/no/nav/pto/veilarbportefolje/aktiviteter/LonnstilskuddUtAvArenaTest.java index dd4a92e58f..81f56651ef 100644 --- a/src/test/java/no/nav/pto/veilarbportefolje/aktiviteter/LonnstilskuddUtAvArenaTest.java +++ b/src/test/java/no/nav/pto/veilarbportefolje/aktiviteter/LonnstilskuddUtAvArenaTest.java @@ -184,8 +184,8 @@ public void skal_ikke_indeksere_bruker_med_tiltaksaktivitet_data_naar_status_er_ BrukereMedAntall response1 = opensearchService.hentBrukere( navKontor.getValue(), empty(), - "asc", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, new Filtervalg().setFerdigfilterListe(List.of()).setTiltakstyper(List.of("MIDLONTIL")), null, null); @@ -195,8 +195,8 @@ public void skal_ikke_indeksere_bruker_med_tiltaksaktivitet_data_naar_status_er_ BrukereMedAntall response2 = opensearchService.hentBrukere( navKontor.getValue(), empty(), - "asc", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, new Filtervalg().setFerdigfilterListe(List.of()).setTiltakstyper(List.of("VARLONTIL")), null, null); @@ -291,8 +291,8 @@ public void skal_indeksere_bruker_naar_vi_faar_lonnstilskudd_fra_DAB() { BrukereMedAntall response1 = opensearchService.hentBrukere( navKontor.getValue(), empty(), - "asc", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, new Filtervalg().setFerdigfilterListe(List.of()).setTiltakstyper(List.of("MIDLONTIL")), null, null); @@ -302,8 +302,8 @@ public void skal_indeksere_bruker_naar_vi_faar_lonnstilskudd_fra_DAB() { BrukereMedAntall response2 = opensearchService.hentBrukere( navKontor.getValue(), empty(), - "asc", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, new Filtervalg().setFerdigfilterListe(List.of()).setTiltakstyper(List.of("VARLONTIL")), null, null); @@ -343,8 +343,8 @@ public void skal_lese_melding_og_indeksere_bruker_naar_fraDato_og_tilDato_er_nul BrukereMedAntall response1 = opensearchService.hentBrukere( navKontor.getValue(), empty(), - "asc", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, new Filtervalg().setFerdigfilterListe(List.of()).setTiltakstyper(List.of("MIDLONTIL")), null, null); @@ -642,8 +642,8 @@ public void verifiserAtKafkameldingMedSammeAktivitetIdMenNyVersjonOverskriverGam BrukereMedAntall responseBrukereMIDLONTIL = opensearchService.hentBrukere( navKontor.getValue(), empty(), - "asc", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, new Filtervalg().setFerdigfilterListe(List.of()).setTiltakstyper(List.of("MIDLONTIL")), null, null); @@ -651,8 +651,8 @@ public void verifiserAtKafkameldingMedSammeAktivitetIdMenNyVersjonOverskriverGam BrukereMedAntall responseBrukereLONNTILS = opensearchService.hentBrukere( navKontor.getValue(), empty(), - "asc", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, new Filtervalg().setFerdigfilterListe(List.of()).setTiltakstyper(List.of("VARLONTIL")), null, null); diff --git a/src/test/java/no/nav/pto/veilarbportefolje/arbeidsliste/ArbeidslistaSorteringOpensearchTest.java b/src/test/java/no/nav/pto/veilarbportefolje/arbeidsliste/ArbeidslistaSorteringOpensearchTest.java index 89575a708d..c83cb510a4 100644 --- a/src/test/java/no/nav/pto/veilarbportefolje/arbeidsliste/ArbeidslistaSorteringOpensearchTest.java +++ b/src/test/java/no/nav/pto/veilarbportefolje/arbeidsliste/ArbeidslistaSorteringOpensearchTest.java @@ -2,10 +2,7 @@ import no.nav.common.types.identer.AktorId; import no.nav.common.types.identer.Fnr; -import no.nav.pto.veilarbportefolje.domene.BrukereMedAntall; -import no.nav.pto.veilarbportefolje.domene.Brukerstatus; -import no.nav.pto.veilarbportefolje.domene.Filtervalg; -import no.nav.pto.veilarbportefolje.domene.Sorteringsfelt; +import no.nav.pto.veilarbportefolje.domene.*; import no.nav.pto.veilarbportefolje.domene.value.NavKontor; import no.nav.pto.veilarbportefolje.domene.value.VeilederId; import no.nav.pto.veilarbportefolje.opensearch.OpensearchService; @@ -76,8 +73,8 @@ public void sisteendring_sortering() { final BrukereMedAntall brukereMedAntall = opensearchService.hentBrukere( enhetId.getValue(), empty(), - "ascending", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, getArbeidslisteFilter(), null, null); @@ -88,8 +85,8 @@ public void sisteendring_sortering() { var sortertResponsAscending = opensearchService.hentBrukere( enhetId.getValue(), empty(), - "ascending", - Sorteringsfelt.ARBEIDSLISTE_OVERSKRIFT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.ARBEIDSLISTE_OVERSKRIFT, getArbeidslisteFilter(), null, null); @@ -97,8 +94,8 @@ public void sisteendring_sortering() { var sortertResponsDescending = opensearchService.hentBrukere( enhetId.getValue(), empty(), - "desc", - Sorteringsfelt.ARBEIDSLISTE_OVERSKRIFT.sorteringsverdi, + Sorteringsrekkefolge.SYNKENDE, + Sorteringsfelt.ARBEIDSLISTE_OVERSKRIFT, getArbeidslisteFilter(), null, null); diff --git a/src/test/java/no/nav/pto/veilarbportefolje/ensligforsorger/EnsligeForsorgereServiceTest.java b/src/test/java/no/nav/pto/veilarbportefolje/ensligforsorger/EnsligeForsorgereServiceTest.java index 27450f70fc..8c32051fe2 100644 --- a/src/test/java/no/nav/pto/veilarbportefolje/ensligforsorger/EnsligeForsorgereServiceTest.java +++ b/src/test/java/no/nav/pto/veilarbportefolje/ensligforsorger/EnsligeForsorgereServiceTest.java @@ -73,8 +73,8 @@ public void testNyOvergangsstonadForBrukerIndex() { BrukereMedAntall responseBrukere = opensearchService.hentBrukere( navKontor.toString(), empty(), - "asc", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, filtervalg, null, null); @@ -113,8 +113,8 @@ public void testAvsluttetOvergangsstonadForBrukerIndex() { BrukereMedAntall responseBrukere = opensearchService.hentBrukere( navKontor.toString(), empty(), - "asc", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, filtervalg, null, null); @@ -173,8 +173,8 @@ public void test_filtrering_enslige_forsorgere() { BrukereMedAntall response = opensearchService.hentBrukere( navKontor.toString(), empty(), - "ascending", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, filterValg, null, null diff --git a/src/test/java/no/nav/pto/veilarbportefolje/hendelsesfilter/HendelseIntegrationTest.kt b/src/test/java/no/nav/pto/veilarbportefolje/hendelsesfilter/HendelseIntegrationTest.kt index 4bc6b7e907..04f4443316 100644 --- a/src/test/java/no/nav/pto/veilarbportefolje/hendelsesfilter/HendelseIntegrationTest.kt +++ b/src/test/java/no/nav/pto/veilarbportefolje/hendelsesfilter/HendelseIntegrationTest.kt @@ -5,6 +5,7 @@ import no.nav.pto.veilarbportefolje.database.PostgresTable.HENDELSE import no.nav.pto.veilarbportefolje.domene.Bruker import no.nav.pto.veilarbportefolje.domene.Filtervalg import no.nav.pto.veilarbportefolje.domene.Sorteringsfelt +import no.nav.pto.veilarbportefolje.domene.Sorteringsrekkefolge import no.nav.pto.veilarbportefolje.opensearch.OpensearchService import no.nav.pto.veilarbportefolje.util.EndToEndTest import no.nav.pto.veilarbportefolje.util.OpensearchTestClient.pollOpensearchUntil @@ -54,8 +55,8 @@ class HendelseIntegrationTest( val brukerFraRespons: Bruker = opensearchService.hentBrukere( brukerOppfolgingsEnhet.value, Optional.empty(), - "asc", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, Filtervalg().setFerdigfilterListe(emptyList()), null, null @@ -83,8 +84,8 @@ class HendelseIntegrationTest( val brukerFraRespons: Bruker = opensearchService.hentBrukere( brukerOppfolgingsEnhet.value, Optional.empty(), - "asc", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, Filtervalg().setFerdigfilterListe(emptyList()), null, null @@ -116,8 +117,8 @@ class HendelseIntegrationTest( val brukerFraRespons: Bruker = opensearchService.hentBrukere( brukerOppfolgingsEnhet.value, Optional.empty(), - "asc", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, Filtervalg().setFerdigfilterListe(emptyList()), null, null @@ -162,8 +163,8 @@ class HendelseIntegrationTest( val brukerFraRespons: Bruker = opensearchService.hentBrukere( brukerOppfolgingsEnhet.value, Optional.empty(), - "asc", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, Filtervalg().setFerdigfilterListe(emptyList()), null, null @@ -205,8 +206,8 @@ class HendelseIntegrationTest( val brukerFraRespons: Bruker = opensearchService.hentBrukere( brukerOppfolgingsEnhet.value, Optional.empty(), - "asc", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, Filtervalg().setFerdigfilterListe(emptyList()), null, null @@ -249,8 +250,8 @@ class HendelseIntegrationTest( val brukerFraRespons: Bruker = opensearchService.hentBrukere( brukerOppfolgingsEnhet.value, Optional.empty(), - "asc", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, Filtervalg().setFerdigfilterListe(emptyList()), null, null @@ -301,8 +302,8 @@ class HendelseIntegrationTest( val brukerFraRespons: Bruker = opensearchService.hentBrukere( brukerOppfolgingsEnhet.value, Optional.empty(), - "asc", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, Filtervalg().setFerdigfilterListe(emptyList()), null, null @@ -338,8 +339,8 @@ class HendelseIntegrationTest( val brukerFraRespons: Bruker = opensearchService.hentBrukere( brukerOppfolgingsEnhet.value, Optional.empty(), - "asc", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, Filtervalg().setFerdigfilterListe(emptyList()), null, null @@ -384,8 +385,8 @@ class HendelseIntegrationTest( val brukerFraRespons: Bruker = opensearchService.hentBrukere( brukerOppfolgingsEnhet.value, Optional.empty(), - "asc", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, Filtervalg().setFerdigfilterListe(emptyList()), null, null @@ -440,8 +441,8 @@ class HendelseIntegrationTest( val brukerFraRespons: Bruker = opensearchService.hentBrukere( brukerOppfolgingsEnhet.value, Optional.empty(), - "asc", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, Filtervalg().setFerdigfilterListe(emptyList()), null, null diff --git a/src/test/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchQueryBuilderTest.java b/src/test/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchQueryBuilderTest.java index 7fa51dad4b..a2357700a5 100644 --- a/src/test/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchQueryBuilderTest.java +++ b/src/test/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchQueryBuilderTest.java @@ -4,6 +4,7 @@ import no.nav.pto.veilarbportefolje.domene.AktivitetFiltervalg; import no.nav.pto.veilarbportefolje.domene.Filtervalg; import no.nav.pto.veilarbportefolje.domene.Sorteringsfelt; +import no.nav.pto.veilarbportefolje.domene.Sorteringsrekkefolge; import org.junit.BeforeClass; import org.junit.Test; import org.opensearch.index.query.BoolQueryBuilder; @@ -14,15 +15,9 @@ import static no.nav.pto.veilarbportefolje.domene.AktivitetFiltervalg.JA; import static no.nav.pto.veilarbportefolje.domene.AktivitetFiltervalg.NEI; -import static no.nav.pto.veilarbportefolje.opensearch.OpensearchQueryBuilder.byggAktivitetFilterQuery; -import static no.nav.pto.veilarbportefolje.opensearch.OpensearchQueryBuilder.byggAlderQuery; -import static no.nav.pto.veilarbportefolje.opensearch.OpensearchQueryBuilder.byggPortefoljestorrelserQuery; -import static no.nav.pto.veilarbportefolje.opensearch.OpensearchQueryBuilder.byggVeilederPaaEnhetScript; -import static no.nav.pto.veilarbportefolje.opensearch.OpensearchQueryBuilder.sorterQueryParametere; -import static no.nav.pto.veilarbportefolje.opensearch.OpensearchQueryBuilder.sorterValgteAktiviteter; +import static no.nav.pto.veilarbportefolje.opensearch.OpensearchQueryBuilder.*; import static no.nav.pto.veilarbportefolje.util.TestUtil.readFileAsJsonString; import static org.assertj.core.api.AssertionsForClassTypes.assertThat; -import static org.assertj.core.api.AssertionsForClassTypes.fail; import static org.opensearch.index.query.QueryBuilders.boolQuery; import static org.opensearch.search.sort.SortOrder.ASC; @@ -36,23 +31,11 @@ public static void beforeAll() { @Test public void skal_sortere_etternavn_paa_etternavn_feltet() { - var searchSourceBuilder = sorterQueryParametere("asc", Sorteringsfelt.ETTERNAVN.sorteringsverdi, new SearchSourceBuilder(), new Filtervalg(), new BrukerinnsynTilganger(true, true, true)); + var searchSourceBuilder = sorterQueryParametere(Sorteringsrekkefolge.STIGENDE, Sorteringsfelt.ETTERNAVN, new SearchSourceBuilder(), new Filtervalg(), new BrukerinnsynTilganger(true, true, true)); var fieldName = searchSourceBuilder.sorts().get(0).toString(); assertThat(fieldName).contains(Sorteringsfelt.ETTERNAVN.sorteringsverdi); } - @Test - public void skal_ikke_sortere_pa_ugyldig_sorteringsverdi() { - String ugyldigSorteringsverdi = "Dette kommer sannsynligvis aldri til å bli en gyldig filterverdi"; - - try { - sorterQueryParametere("asc", ugyldigSorteringsverdi, new SearchSourceBuilder(), new Filtervalg(), new BrukerinnsynTilganger(true, true, true)); - fail("Ugyldig input ble godtatt og brukt til sortering"); - } catch (Exception e) { - assertThat(e).isNotNull(); - } - } - @Test public void skal_bygge_riktig_filtrer_paa_veileder_script() { String actualScript = byggVeilederPaaEnhetScript(List.of("Z000000", "Z000001", "Z000002")); diff --git a/src/test/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchServiceIntegrationTest.java b/src/test/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchServiceIntegrationTest.java index a9b7a80c16..73d871af77 100644 --- a/src/test/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchServiceIntegrationTest.java +++ b/src/test/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchServiceIntegrationTest.java @@ -33,7 +33,6 @@ import no.nav.pto.veilarbportefolje.util.EndToEndTest; import no.nav.pto.veilarbportefolje.vedtakstotte.Hovedmal; import no.nav.pto.veilarbportefolje.vedtakstotte.Innsatsgruppe; -import org.assertj.core.api.AssertionsForClassTypes; import org.junit.jupiter.api.BeforeEach; import org.junit.jupiter.api.Test; import org.opensearch.search.builder.SearchSourceBuilder; @@ -333,8 +332,8 @@ public void skal_kun_hente_ut_brukere_under_oppfolging() { BrukereMedAntall brukereMedAntall = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ikke_satt", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.IKKE_SATT, + Sorteringsfelt.IKKE_SATT, new Filtervalg(), null, null @@ -371,8 +370,8 @@ public void skal_sette_brukere_med_veileder_fra_annen_enhet_til_ufordelt() { var response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ikke_satt", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.IKKE_SATT, + Sorteringsfelt.IKKE_SATT, filtervalg, null, null @@ -430,8 +429,8 @@ public void skal_hente_ut_brukere_ved_soek_paa_flere_veiledere() { var response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ikke_satt", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.IKKE_SATT, + Sorteringsfelt.IKKE_SATT, filtervalg, null, null); @@ -475,8 +474,8 @@ public void skal_hente_riktig_antall_ufordelte_brukere() { var response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ikke_satt", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.IKKE_SATT, + Sorteringsfelt.IKKE_SATT, filtervalg, null, null); @@ -1000,8 +999,8 @@ void skal_sortere_brukere_pa_arbeidslisteikon() { BrukereMedAntall brukereMedAntall = opensearchService.hentBrukere( TEST_ENHET, Optional.empty(), - "desc", - Sorteringsfelt.ARBEIDSLISTE_KATEGORI.sorteringsverdi, + Sorteringsrekkefolge.SYNKENDE, + Sorteringsfelt.ARBEIDSLISTE_KATEGORI, new Filtervalg(), null, null @@ -1060,8 +1059,8 @@ void skal_sortere_brukere_pa_aktivteter() { BrukereMedAntall brukereMedAntall = opensearchService.hentBrukere( TEST_ENHET, Optional.empty(), - "desc", - Sorteringsfelt.VALGTE_AKTIVITETER.sorteringsverdi, + Sorteringsrekkefolge.SYNKENDE, + Sorteringsfelt.VALGTE_AKTIVITETER, filtervalg1, null, null @@ -1069,8 +1068,8 @@ void skal_sortere_brukere_pa_aktivteter() { BrukereMedAntall brukereMedAntall2 = opensearchService.hentBrukere( TEST_ENHET, Optional.empty(), - "desc", - Sorteringsfelt.VALGTE_AKTIVITETER.sorteringsverdi, + Sorteringsrekkefolge.SYNKENDE, + Sorteringsfelt.VALGTE_AKTIVITETER, filtervalg2, null, null @@ -1124,8 +1123,8 @@ void skal_hente_brukere_som_trenger_vurdering_og_er_ny_for_enhet() { var response = opensearchService.hentBrukere( TEST_ENHET, Optional.empty(), - "ikke_satt", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.IKKE_SATT, + Sorteringsfelt.IKKE_SATT, new Filtervalg().setFerdigfilterListe(ferdigFiltere), null, null @@ -1160,8 +1159,8 @@ void skal_ikke_kunne_hente_brukere_veileder_ikke_har_tilgang_til() { var response = opensearchService.hentBrukere( TEST_ENHET, Optional.of(TEST_VEILEDER_0), - "ikke_satt", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.IKKE_SATT, + Sorteringsfelt.IKKE_SATT, new Filtervalg(), null, null @@ -1199,8 +1198,8 @@ void skal_anse_bruker_som_ufordelt_om_bruker_har_veileder_som_ikke_har_tilgang_t var response = opensearchService.hentBrukere( TEST_ENHET, Optional.of(LITE_PRIVILEGERT_VEILEDER), - "ikke_satt", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.IKKE_SATT, + Sorteringsfelt.IKKE_SATT, new Filtervalg().setFerdigfilterListe(List.of(Brukerstatus.UFORDELTE_BRUKERE)), null, null @@ -1244,8 +1243,8 @@ void skal_anse_bruker_som_ufordelt_om_bruker_har_veileder_som_ikke_har_tilgang_t var response = opensearchService.hentBrukere( TEST_ENHET, Optional.of(TEST_VEILEDER_0), - "ikke_satt", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.IKKE_SATT, + Sorteringsfelt.IKKE_SATT, filterValg, null, null @@ -1286,8 +1285,8 @@ void skal_anse_bruker_som_ufordelt_om_bruker_har_veileder_som_ikke_har_tilgang_t var response = opensearchService.hentBrukere( TEST_ENHET, Optional.of(TEST_VEILEDER_0), - "ikke_satt", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.IKKE_SATT, + Sorteringsfelt.IKKE_SATT, filterValg, null, null @@ -1328,8 +1327,8 @@ void skal_anse_bruker_som_ufordelt_om_bruker_har_veileder_som_ikke_har_tilgang_t var response = opensearchService.hentBrukere( TEST_ENHET, Optional.of(TEST_VEILEDER_0), - "ikke_satt", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.IKKE_SATT, + Sorteringsfelt.IKKE_SATT, filterValg, null, null @@ -1393,8 +1392,8 @@ void skal_anse_bruker_som_ufordelt_om_bruker_har_veileder_som_ikke_har_tilgang_t var response = opensearchService.hentBrukere( TEST_ENHET, Optional.of(TEST_VEILEDER_0), - "ikke_satt", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.IKKE_SATT, + Sorteringsfelt.IKKE_SATT, filterValg, null, null @@ -1444,8 +1443,8 @@ void skal_anse_bruker_som_ufordelt_om_bruker_har_veileder_som_ikke_har_tilgang_t var response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ikke_satt", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.IKKE_SATT, + Sorteringsfelt.IKKE_SATT, filterValg, null, null @@ -1493,8 +1492,8 @@ void skal_anse_bruker_som_ufordelt_om_bruker_har_veileder_som_ikke_har_tilgang_t var response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ikke_satt", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.IKKE_SATT, + Sorteringsfelt.IKKE_SATT, filterValg, null, null @@ -1544,8 +1543,8 @@ void skal_hente_ut_alle_brukere_med_tiltak() { var response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ikke_satt", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.IKKE_SATT, + Sorteringsfelt.IKKE_SATT, filterValg, null, null @@ -1596,8 +1595,8 @@ void skal_hente_ut_alle_brukere_som_ikke_har_tiltak() { var response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ikke_satt", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.IKKE_SATT, + Sorteringsfelt.IKKE_SATT, filterValg, null, null @@ -1671,8 +1670,8 @@ public void skal_hente_alle_brukere_som_har_vedtak() { var response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.UTKAST_14A_ANSVARLIG_VEILEDER.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.UTKAST_14A_ANSVARLIG_VEILEDER, filterValg, null, null @@ -1746,8 +1745,8 @@ public void skal_hente_alle_brukere_som_har_tolkbehov() { var response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, filterValg, null, null @@ -1765,8 +1764,8 @@ public void skal_hente_alle_brukere_som_har_tolkbehov() { response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, filterValg, null, null @@ -1781,8 +1780,8 @@ public void skal_hente_alle_brukere_som_har_tolkbehov() { response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, filterValg, null, null @@ -1801,8 +1800,8 @@ public void skal_hente_alle_brukere_som_har_tolkbehov() { response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, filterValg, null, null @@ -1818,8 +1817,8 @@ public void skal_hente_alle_brukere_som_har_tolkbehov() { response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, filterValg, null, null @@ -1899,8 +1898,8 @@ public void skal_hente_alle_brukere_fra_landgruppe() { var response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, filterValg, null, null @@ -1918,8 +1917,8 @@ public void skal_hente_alle_brukere_fra_landgruppe() { response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, filterValg, null, null @@ -1935,8 +1934,8 @@ public void skal_hente_alle_brukere_fra_landgruppe() { response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, filterValg, null, null @@ -1951,8 +1950,8 @@ public void skal_hente_alle_brukere_fra_landgruppe() { response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, filterValg, null, null @@ -2042,8 +2041,8 @@ public void test_sortering_landgruppe() { BrukereMedAntall response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.FODELAND.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.FODELAND, filterValg, null, null @@ -2060,8 +2059,8 @@ public void test_sortering_landgruppe() { response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "descending", - Sorteringsfelt.FODELAND.sorteringsverdi, + Sorteringsrekkefolge.SYNKENDE, + Sorteringsfelt.FODELAND, filterValg, null, null @@ -2078,8 +2077,8 @@ public void test_sortering_landgruppe() { response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.STATSBORGERSKAP.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.STATSBORGERSKAP, filterValg, null, null @@ -2154,8 +2153,8 @@ public void skal_hente_alle_brukere_med_bosted() { var response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, filterValg, null, null @@ -2172,8 +2171,8 @@ public void skal_hente_alle_brukere_med_bosted() { response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, filterValg, null, null @@ -2189,8 +2188,8 @@ public void skal_hente_alle_brukere_med_bosted() { response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, filterValg, null, null @@ -2262,8 +2261,8 @@ public void test_sortering_bostedkommune() { BrukereMedAntall response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.BOSTED_KOMMUNE.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.BOSTED_KOMMUNE, filterValg, null, null @@ -2279,8 +2278,8 @@ public void test_sortering_bostedkommune() { response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "descending", - Sorteringsfelt.BOSTED_BYDEL.sorteringsverdi, + Sorteringsrekkefolge.SYNKENDE, + Sorteringsfelt.BOSTED_BYDEL, filterValg, null, null @@ -2345,8 +2344,8 @@ public void test_sortering_bostedkommune() { var response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, filterValg, null, null @@ -2408,8 +2407,8 @@ public void test_sortering_bostedkommune() { var response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, filterValg, null, null @@ -2469,8 +2468,8 @@ public void test_sortering_bostedkommune() { var response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, filterValg, null, null @@ -2518,8 +2517,8 @@ void skal_ikke_automatisk_sortere_nye_brukere_paa_top() { var response = opensearchService.hentBrukere( TEST_ENHET, Optional.of(TEST_VEILEDER_0), - "descending", - Sorteringsfelt.ETTERNAVN.sorteringsverdi, + Sorteringsrekkefolge.SYNKENDE, + Sorteringsfelt.ETTERNAVN, filterValg, null, null @@ -2566,8 +2565,8 @@ void skal_ikke_automatisk_sortere_ufordelte_brukere_paa_top() { var response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "descending", - Sorteringsfelt.ETTERNAVN.sorteringsverdi, + Sorteringsrekkefolge.SYNKENDE, + Sorteringsfelt.ETTERNAVN, filterValg, null, null @@ -2640,8 +2639,8 @@ public void test_sortering_enslige_forsorgere() { BrukereMedAntall response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.ENSLIGE_FORSORGERE_UTLOP_YTELSE.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.ENSLIGE_FORSORGERE_UTLOP_YTELSE, filterValg, null, null @@ -2657,8 +2656,8 @@ public void test_sortering_enslige_forsorgere() { response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.ENSLIGE_FORSORGERE_OM_BARNET.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.ENSLIGE_FORSORGERE_OM_BARNET, filterValg, null, null @@ -2674,8 +2673,8 @@ public void test_sortering_enslige_forsorgere() { response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.ENSLIGE_FORSORGERE_AKTIVITETSPLIKT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.ENSLIGE_FORSORGERE_AKTIVITETSPLIKT, filterValg, null, null @@ -2690,8 +2689,8 @@ public void test_sortering_enslige_forsorgere() { response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.ENSLIGE_FORSORGERE_VEDTAKSPERIODETYPE.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.ENSLIGE_FORSORGERE_VEDTAKSPERIODETYPE, filterValg, null, null @@ -2773,8 +2772,8 @@ public void test_filtrering_barn_under_18() { BrukereMedAntall response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, filterValg, null, null @@ -2829,8 +2828,8 @@ public void test_filtrering_barn_under_18_ingen_tilganger() { BrukereMedAntall response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, filterValg, null, null @@ -2883,8 +2882,8 @@ public void test_filtrering_barn_under_18_tilgang_6() { BrukereMedAntall response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, filterValg, null, null @@ -2949,8 +2948,8 @@ public void test_filtrering_barn_under_18_tilgang_7() { BrukereMedAntall response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, filterValg, null, null @@ -3013,8 +3012,8 @@ public void test_filtrering_barn_under_18_tilgang_6_7() { BrukereMedAntall response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, filterValg, null, null @@ -3122,8 +3121,8 @@ public void test_filtrering_barn_under_18_med_alder_filter_ikke_tilgang_6_eller_ BrukereMedAntall response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.BARN_UNDER_18_AR.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.BARN_UNDER_18_AR, filterValg, null, null @@ -3206,8 +3205,8 @@ public void test_filtrering_barn_under_18_med_alder_filter_har_tilgang_7() { BrukereMedAntall response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.BARN_UNDER_18_AR.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.BARN_UNDER_18_AR, filterValg, null, null @@ -3275,8 +3274,8 @@ public void test_filtrering_og_statustall_tiltakshendelser() { BrukereMedAntall response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, filterValg, null, null @@ -3345,8 +3344,8 @@ public void test_sortering_tiltakshendelser() { BrukereMedAntall responseDefaultSortering = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ikke_satt", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.IKKE_SATT, + Sorteringsfelt.IKKE_SATT, filterValg, null, null @@ -3361,8 +3360,8 @@ public void test_sortering_tiltakshendelser() { BrukereMedAntall responseSortertNyesteDato = opensearchService.hentBrukere( TEST_ENHET, empty(), - "descending", - Sorteringsfelt.TILTAKSHENDELSE_DATO_OPPRETTET.sorteringsverdi, + Sorteringsrekkefolge.SYNKENDE, + Sorteringsfelt.TILTAKSHENDELSE_DATO_OPPRETTET, filterValg, null, null @@ -3377,8 +3376,8 @@ public void test_sortering_tiltakshendelser() { BrukereMedAntall responseSortertAlfabetisk = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.TILTAKSHENDELSE_TEKST.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.TILTAKSHENDELSE_TEKST, filterValg, null, null @@ -3439,8 +3438,8 @@ public void test_filtrering_og_statustall_utgatte_varsel() { BrukereMedAntall response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, filterValg, null, null @@ -3510,8 +3509,8 @@ void skal_sortere_pa_hendelsesdato_som_standard_ved_filtrering_pa_utgatt_varsel( BrukereMedAntall response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ikke_satt", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.IKKE_SATT, + Sorteringsfelt.IKKE_SATT, filtervalg, null, null @@ -3576,8 +3575,8 @@ void skal_kunne_sortere_pa_hendelsesdato_pa_utgatt_varsel() { BrukereMedAntall response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.UTGATT_VARSEL_DATO.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.UTGATT_VARSEL_DATO, filtervalg, null, null @@ -3729,8 +3728,8 @@ public void test_sorting_barn_under_18_veileder_tilgang_6_7() { BrukereMedAntall response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.BARN_UNDER_18_AR.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.BARN_UNDER_18_AR, filterValg, null, null @@ -3883,8 +3882,8 @@ public void test_sorting_barn_under_18_veileder_ikke_tilgang_6_7() { BrukereMedAntall response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.BARN_UNDER_18_AR.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.BARN_UNDER_18_AR, filterValg, null, null @@ -4033,8 +4032,8 @@ public void test_sorting_barn_under_18_veileder_tilgang_6_ikke_7() { BrukereMedAntall response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.BARN_UNDER_18_AR.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.BARN_UNDER_18_AR, filterValg, null, null @@ -4122,8 +4121,8 @@ public void test_sortering_AAP() { BrukereMedAntall response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.AAP_VURDERINGSFRIST.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.AAP_VURDERINGSFRIST, filterValg, null, null @@ -4141,8 +4140,8 @@ public void test_sortering_AAP() { response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.AAP_VURDERINGSFRIST.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.AAP_VURDERINGSFRIST, filterValg, null, null @@ -4221,8 +4220,8 @@ public void test_sortering_huskelapp() { BrukereMedAntall response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.HUSKELAPP_KOMMENTAR.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.HUSKELAPP_KOMMENTAR, filterValg, null, null @@ -4238,8 +4237,8 @@ public void test_sortering_huskelapp() { response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "descending", - Sorteringsfelt.HUSKELAPP_FRIST.sorteringsverdi, + Sorteringsrekkefolge.SYNKENDE, + Sorteringsfelt.HUSKELAPP_FRIST, filterValg, null, null @@ -4320,8 +4319,8 @@ public void test_filtering_og_sortering_fargekategori() { BrukereMedAntall response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, filterValg, null, null @@ -4337,8 +4336,8 @@ public void test_filtering_og_sortering_fargekategori() { response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, filterValg, null, null @@ -4354,8 +4353,8 @@ public void test_filtering_og_sortering_fargekategori() { response = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.FARGEKATEGORI.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.FARGEKATEGORI, filterValg, null, null @@ -4412,8 +4411,8 @@ void skal_indeksere_arbeidsliste_data_riktig() { BrukereMedAntall respons = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, new Filtervalg().setFerdigfilterListe(emptyList()), null, null @@ -4465,8 +4464,8 @@ public void skal_hente_brukere_med_gjeldendeVedtak14a() { BrukereMedAntall respons = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, new Filtervalg().setFerdigfilterListe(emptyList()).setGjeldendeVedtak14a(List.of("HAR_14A_VEDTAK")), null, null @@ -4523,8 +4522,8 @@ public void skal_hente_brukere_uten_gjeldendeVedtak14a() { BrukereMedAntall respons = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, new Filtervalg().setFerdigfilterListe(emptyList()).setGjeldendeVedtak14a(List.of("HAR_IKKE_14A_VEDTAK")), null, null @@ -4578,8 +4577,8 @@ public void skal_hente_brukere_med_og_uten_gjeldendeVedtak14a() { BrukereMedAntall respons = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, new Filtervalg().setFerdigfilterListe(emptyList()).setGjeldendeVedtak14a(List.of("HAR_14A_VEDTAK", "HAR_IKKE_14A_VEDTAK")), null, null @@ -4649,8 +4648,8 @@ public void sorter_pa_vedtaksdato_som_standard_ved_filtrering_pa_alle_gjeldendeV BrukereMedAntall responsFiltrertGjeldendeVedtak = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ikke_satt", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.IKKE_SATT, + Sorteringsfelt.IKKE_SATT, filtrertHarGjeldendeVedtak, null, null @@ -4671,8 +4670,8 @@ public void sorter_pa_vedtaksdato_som_standard_ved_filtrering_pa_alle_gjeldendeV BrukereMedAntall responsFiltrertInnsatsgruppe = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ikke_satt", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.IKKE_SATT, + Sorteringsfelt.IKKE_SATT, filtrertInnsatsgruppe, null, null @@ -4696,8 +4695,8 @@ public void sorter_pa_vedtaksdato_som_standard_ved_filtrering_pa_alle_gjeldendeV BrukereMedAntall responsFiltrertHovedmal = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ikke_satt", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.IKKE_SATT, + Sorteringsfelt.IKKE_SATT, filtrertHovedmal, null, null @@ -4776,8 +4775,8 @@ public void skal_kunne_sortere_brukere_med_og_uten_gjeldendeVedtak14a_pa_14a_kol BrukereMedAntall responsInnsatsgruppeStigende = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.GJELDENDE_VEDTAK_14A_INNSATSGRUPPE.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.GJELDENDE_VEDTAK_14A_INNSATSGRUPPE, filtervalg, null, null @@ -4792,8 +4791,8 @@ public void skal_kunne_sortere_brukere_med_og_uten_gjeldendeVedtak14a_pa_14a_kol BrukereMedAntall responsInnsatsgruppeSynkende = opensearchService.hentBrukere( TEST_ENHET, empty(), - "descending", - Sorteringsfelt.GJELDENDE_VEDTAK_14A_INNSATSGRUPPE.sorteringsverdi, + Sorteringsrekkefolge.SYNKENDE, + Sorteringsfelt.GJELDENDE_VEDTAK_14A_INNSATSGRUPPE, filtervalg, null, null @@ -4808,8 +4807,8 @@ public void skal_kunne_sortere_brukere_med_og_uten_gjeldendeVedtak14a_pa_14a_kol BrukereMedAntall responsHovedmalStigende = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.GJELDENDE_VEDTAK_14A_HOVEDMAL.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.GJELDENDE_VEDTAK_14A_HOVEDMAL, filtervalg, null, null @@ -4824,8 +4823,8 @@ public void skal_kunne_sortere_brukere_med_og_uten_gjeldendeVedtak14a_pa_14a_kol BrukereMedAntall responsVedtaksdatoStigende = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.GJELDENDE_VEDTAK_14A_VEDTAKSDATO.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.GJELDENDE_VEDTAK_14A_VEDTAKSDATO, filtervalg, null, null @@ -4895,8 +4894,8 @@ public void skal_hente_brukere_med_innsatsgruppeGjeldendeVedtak14a() { BrukereMedAntall respons = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.GJELDENDE_VEDTAK_14A_INNSATSGRUPPE.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.GJELDENDE_VEDTAK_14A_INNSATSGRUPPE, filtervalg, null, null @@ -4967,8 +4966,8 @@ public void skal_hente_brukere_med_hovedmalGjeldendeVedtak14a() { BrukereMedAntall respons = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.GJELDENDE_VEDTAK_14A_HOVEDMAL.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.GJELDENDE_VEDTAK_14A_HOVEDMAL, filtervalg, null, null @@ -5025,8 +5024,8 @@ public void skal_sortere_pa_aktorid_som_standard_om_ikke_sorteringsfelt_er_valgt BrukereMedAntall respons = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ikke_satt", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.IKKE_SATT, + Sorteringsfelt.IKKE_SATT, new Filtervalg().setFerdigfilterListe(emptyList()), null, null @@ -5037,26 +5036,6 @@ public void skal_sortere_pa_aktorid_som_standard_om_ikke_sorteringsfelt_er_valgt assertEquals(respons.getBrukere().get(2).getAktoerid(), bruker1.getAktoer_id()); } - @Test - public void skal_ikke_kunne_sortere_pa_ugyldig_sorteringsverdi() { - String ugyldigSorteringsverdi = "Dette kommer sannsynligvis aldri til å bli en gyldig filterverdi"; - - try { - opensearchService.hentBrukere( - TEST_ENHET, - empty(), - "ascending", - ugyldigSorteringsverdi, - new Filtervalg().setFerdigfilterListe(emptyList()), - null, - null - ); - fail("Ugyldig input ble godtatt og brukt til sortering"); - } catch (Exception e) { - AssertionsForClassTypes.assertThat(e).isNotNull(); - } - } - @Test public void skal_kunne_sortere_pa_alle_gyldige_sorteringsverdier() { Sorteringsfelt[] alleSorteringsfelt = Sorteringsfelt.values(); @@ -5067,8 +5046,8 @@ public void skal_kunne_sortere_pa_alle_gyldige_sorteringsverdier() { opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - sorteringsfelt.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + sorteringsfelt, new Filtervalg().setFerdigfilterListe(emptyList()), null, null @@ -5100,8 +5079,8 @@ void skal_indeksere_hendelse_data_riktig_for_utgatt_varsel() { BrukereMedAntall respons = opensearchService.hentBrukere( TEST_ENHET, empty(), - "ascending", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, new Filtervalg().setFerdigfilterListe(emptyList()), null, null @@ -5120,8 +5099,8 @@ private BrukereMedAntall sorterBrukerePaStandardsorteringenAktorid(OpensearchSer return osService.hentBrukere( TEST_ENHET, empty(), - "ikke_satt", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.IKKE_SATT, + Sorteringsfelt.IKKE_SATT, new Filtervalg().setFerdigfilterListe(emptyList()), null, null @@ -5156,8 +5135,8 @@ private static BrukereMedAntall loggInnVeilederOgHentEnhetPortefolje(OpensearchS () -> opensearchService.hentBrukere( TEST_ENHET, empty(), - "ikke_satt", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.IKKE_SATT, + Sorteringsfelt.IKKE_SATT, new Filtervalg(), null, null @@ -5171,8 +5150,8 @@ private static BrukereMedAntall loggInnVeilederOgHentVeilederPortefolje(Opensear () -> opensearchService.hentBrukere( TEST_ENHET, Optional.of(veilederId), - "ikke_satt", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.IKKE_SATT, + Sorteringsfelt.IKKE_SATT, new Filtervalg(), null, null diff --git a/src/test/java/no/nav/pto/veilarbportefolje/sisteendring/SisteEndringIntegrationTest.java b/src/test/java/no/nav/pto/veilarbportefolje/sisteendring/SisteEndringIntegrationTest.java index 176cc75031..f039930d6d 100644 --- a/src/test/java/no/nav/pto/veilarbportefolje/sisteendring/SisteEndringIntegrationTest.java +++ b/src/test/java/no/nav/pto/veilarbportefolje/sisteendring/SisteEndringIntegrationTest.java @@ -9,6 +9,7 @@ import no.nav.pto.veilarbportefolje.domene.BrukereMedAntall; import no.nav.pto.veilarbportefolje.domene.Filtervalg; import no.nav.pto.veilarbportefolje.domene.Sorteringsfelt; +import no.nav.pto.veilarbportefolje.domene.Sorteringsrekkefolge; import no.nav.pto.veilarbportefolje.domene.value.NavKontor; import no.nav.pto.veilarbportefolje.domene.value.VeilederId; import no.nav.pto.veilarbportefolje.mal.MalEndringKafkaDTO; @@ -28,18 +29,11 @@ import org.springframework.jdbc.core.JdbcTemplate; import java.time.ZonedDateTime; -import java.util.ArrayList; -import java.util.List; -import java.util.Map; -import java.util.Optional; -import java.util.Random; +import java.util.*; import java.util.concurrent.TimeUnit; import static java.util.Optional.empty; -import static no.nav.pto.veilarbportefolje.sisteendring.SisteEndringsKategori.FULLFORT_EGEN; -import static no.nav.pto.veilarbportefolje.sisteendring.SisteEndringsKategori.FULLFORT_IJOBB; -import static no.nav.pto.veilarbportefolje.sisteendring.SisteEndringsKategori.MAL; -import static no.nav.pto.veilarbportefolje.sisteendring.SisteEndringsKategori.NY_IJOBB; +import static no.nav.pto.veilarbportefolje.sisteendring.SisteEndringsKategori.*; import static no.nav.pto.veilarbportefolje.util.OpensearchTestClient.pollOpensearchUntil; import static no.nav.pto.veilarbportefolje.util.TestDataUtils.randomAktorId; import static org.assertj.core.api.AssertionsForClassTypes.assertThat; @@ -158,8 +152,8 @@ public void sisteendring_filtrering() { final BrukereMedAntall brukereMedAntall = opensearchService.hentBrukere( testEnhet.getValue(), empty(), - "asc", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, getFiltervalg(FULLFORT_IJOBB), null, null); @@ -170,8 +164,8 @@ public void sisteendring_filtrering() { var responseBrukere = opensearchService.hentBrukere( testEnhet.getValue(), empty(), - "asc", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, getFiltervalg(FULLFORT_IJOBB), null, null); @@ -191,8 +185,8 @@ public void sisteendring_ulestfilter_skalIkkeKrasjeVedNull() { final BrukereMedAntall brukereMedAntall = opensearchService.hentBrukere( testEnhet.getValue(), empty(), - "asc", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, new Filtervalg(), null, null); @@ -203,8 +197,8 @@ public void sisteendring_ulestfilter_skalIkkeKrasjeVedNull() { var responseBrukere = opensearchService.hentBrukere( testEnhet.getValue(), empty(), - "asc", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, getFiltervalg(FULLFORT_IJOBB, true), null, null); @@ -226,8 +220,8 @@ public void sisteendring_ulestfilter() { final BrukereMedAntall brukereMedAntall = opensearchService.hentBrukere( testEnhet.getValue(), empty(), - "asc", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, new Filtervalg(), null, null); @@ -248,8 +242,8 @@ public void sisteendring_ulestfilter() { final BrukereMedAntall brukereMedAntall = opensearchService.hentBrukere( testEnhet.getValue(), empty(), - "asc", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, getFiltervalg(NY_IJOBB), null, null); @@ -262,8 +256,8 @@ public void sisteendring_ulestfilter() { final BrukereMedAntall brukereMedAntall = opensearchService.hentBrukere( testEnhet.getValue(), empty(), - "asc", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, getFiltervalg(FULLFORT_IJOBB, true), null, null); @@ -273,8 +267,8 @@ public void sisteendring_ulestfilter() { var responseBrukere1 = opensearchService.hentBrukere( testEnhet.getValue(), empty(), - "asc", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, getFiltervalg(NY_IJOBB, true), null, null); @@ -284,8 +278,8 @@ public void sisteendring_ulestfilter() { var responseBrukere2 = opensearchService.hentBrukere( testEnhet.getValue(), empty(), - "asc", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, getFiltervalg(FULLFORT_IJOBB, true), null, null); @@ -314,8 +308,8 @@ public void sisteendring_sortering() { final BrukereMedAntall brukereMedAntall = opensearchService.hentBrukere( testEnhet.getValue(), empty(), - "asc", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, new Filtervalg(), null, null); @@ -347,8 +341,8 @@ public void sisteendring_sortering() { final BrukereMedAntall brukereMedAntall = opensearchService.hentBrukere( testEnhet.getValue(), empty(), - "ascending", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, getFiltervalg(FULLFORT_IJOBB), null, null); @@ -360,8 +354,8 @@ public void sisteendring_sortering() { final BrukereMedAntall brukereMedAntall = opensearchService.hentBrukere( testEnhet.getValue(), empty(), - "ascending", - Sorteringsfelt.IKKE_SATT.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.IKKE_SATT, getFiltervalg(FULLFORT_EGEN), null, null); @@ -372,8 +366,8 @@ public void sisteendring_sortering() { var responseSortertFULLFORT_IJOBB = opensearchService.hentBrukere( testEnhet.getValue(), empty(), - "descending", - Sorteringsfelt.SISTE_ENDRING_DATO.sorteringsverdi, + Sorteringsrekkefolge.SYNKENDE, + Sorteringsfelt.SISTE_ENDRING_DATO, getFiltervalg(FULLFORT_IJOBB), null, null); @@ -385,8 +379,8 @@ public void sisteendring_sortering() { var responseSortertFULLFORT_EGEN = opensearchService.hentBrukere( testEnhet.getValue(), empty(), - "ascending", - Sorteringsfelt.SISTE_ENDRING_DATO.sorteringsverdi, + Sorteringsrekkefolge.STIGENDE, + Sorteringsfelt.SISTE_ENDRING_DATO, getFiltervalg(FULLFORT_EGEN), null, null); @@ -399,8 +393,8 @@ public void sisteendring_sortering() { var responseSortertTomRes1 = opensearchService.hentBrukere( testEnhet.getValue(), empty(), - "descending", - Sorteringsfelt.SISTE_ENDRING_DATO.sorteringsverdi, + Sorteringsrekkefolge.SYNKENDE, + Sorteringsfelt.SISTE_ENDRING_DATO, getFiltervalg(NY_IJOBB), null, null); @@ -413,8 +407,8 @@ public void sisteendring_filterPaFlereEndringerSkalKasteError() { () -> opensearchService.hentBrukere( testEnhet.getValue(), empty(), - "descending", - Sorteringsfelt.SISTE_ENDRING_DATO.sorteringsverdi, + Sorteringsrekkefolge.SYNKENDE, + Sorteringsfelt.SISTE_ENDRING_DATO, getFiltervalg(FULLFORT_IJOBB, FULLFORT_EGEN), null, null)); From c34238a011d89f58ec05d01e2e34530da377091b Mon Sep 17 00:00:00 2001 From: Sondre Larsen Ovrid Date: Thu, 23 Jan 2025 19:50:46 +0100 Subject: [PATCH 9/9] Fiks test som feila pga. "NullPointerException" --- .../persononinfo/PdlBrukerdataKafkaServiceTest.java | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/src/test/java/no/nav/pto/veilarbportefolje/persononinfo/PdlBrukerdataKafkaServiceTest.java b/src/test/java/no/nav/pto/veilarbportefolje/persononinfo/PdlBrukerdataKafkaServiceTest.java index 23e6819f4f..8ee8eb8a27 100644 --- a/src/test/java/no/nav/pto/veilarbportefolje/persononinfo/PdlBrukerdataKafkaServiceTest.java +++ b/src/test/java/no/nav/pto/veilarbportefolje/persononinfo/PdlBrukerdataKafkaServiceTest.java @@ -34,11 +34,6 @@ public class PdlBrukerdataKafkaServiceTest extends EndToEndTest { private static PdlBrukerdataKafkaService pdlBrukerdataKafkaService; - @MockBean - private static OpensearchIndexer opensearchIndexer; - @MockBean - private static OpensearchIndexerV2 opensearchIndexerV2; - private static JdbcTemplate db; public PdlBrukerdataKafkaServiceTest() throws JsonProcessingException { @@ -74,8 +69,8 @@ public PdlBrukerdataKafkaServiceTest() throws JsonProcessingException { , pdlIdentRepository, new BrukerServiceV2(pdlIdentRepository, oppfolgingsbrukerRepositoryV3, oppfolgingRepositoryV2), barnUnder18AarService, - opensearchIndexer, - opensearchIndexerV2 + Mockito.mock(OpensearchIndexer.class), + Mockito.mock(OpensearchIndexerV2.class) ); }