From 566d9fd40d81499ed46bdcd983d71688f32fd163 Mon Sep 17 00:00:00 2001 From: Klara M Helgemo Date: Tue, 10 Sep 2024 10:55:51 +0200 Subject: [PATCH 1/3] Lagt til tiltakshendelser i statustall MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Co-authored-by: Ingrid Fosså --- .../no/nav/pto/veilarbportefolje/domene/Statustall.java | 3 +++ .../opensearch/OpensearchQueryBuilder.java | 3 ++- .../opensearch/domene/StatustallResponse.java | 1 + .../opensearch/OpensearchServiceIntegrationTest.java | 6 +++++- 4 files changed, 11 insertions(+), 2 deletions(-) diff --git a/src/main/java/no/nav/pto/veilarbportefolje/domene/Statustall.java b/src/main/java/no/nav/pto/veilarbportefolje/domene/Statustall.java index 0603788e5e..dad4e4e245 100644 --- a/src/main/java/no/nav/pto/veilarbportefolje/domene/Statustall.java +++ b/src/main/java/no/nav/pto/veilarbportefolje/domene/Statustall.java @@ -33,6 +33,7 @@ public class Statustall { private long fargekategoriE; private long fargekategoriF; private long fargekategoriIngenKategori; + private long tiltakshendelser; public Statustall() { this.totalt = 0; @@ -61,6 +62,7 @@ public Statustall() { this.fargekategoriE = 0; this.fargekategoriF = 0; this.fargekategoriIngenKategori = 0; + this.tiltakshendelser = 0; } public Statustall(StatustallResponse.StatustallAggregation.StatustallFilter.StatustallBuckets buckets, boolean vedtakstottePilotErPa) { @@ -90,5 +92,6 @@ public Statustall(StatustallResponse.StatustallAggregation.StatustallFilter.Stat this.fargekategoriE = buckets.getFargekategoriE().getDoc_count(); this.fargekategoriF = buckets.getFargekategoriF().getDoc_count(); this.fargekategoriIngenKategori = buckets.getFargekategoriIngenKategori().getDoc_count(); + this.tiltakshendelser = buckets.getTiltakshendelser().getDoc_count(); } } \ No newline at end of file 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 78561ba8e1..6f904238c8 100644 --- a/src/main/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchQueryBuilder.java +++ b/src/main/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchQueryBuilder.java @@ -850,7 +850,8 @@ static SearchSourceBuilder byggStatustallQuery(BoolQueryBuilder filtrereVeileder mustMatchQuery(filtrereVeilederOgEnhet, "fargekategoriE", "fargekategori", FargekategoriVerdi.FARGEKATEGORI_E.name()), mustMatchQuery(filtrereVeilederOgEnhet, "fargekategoriF", "fargekategori", FargekategoriVerdi.FARGEKATEGORI_F.name()), mustNotExistFilter(filtrereVeilederOgEnhet, "fargekategoriIngenKategori", "fargekategori"), - mustExistFilter(filtrereVeilederOgEnhet, "mineHuskelapper", "huskelapp") + mustExistFilter(filtrereVeilederOgEnhet, "mineHuskelapper", "huskelapp"), + mustExistFilter(filtrereVeilederOgEnhet, "tiltakshendelser", "tiltakshendelse") }; return new SearchSourceBuilder() diff --git a/src/main/java/no/nav/pto/veilarbportefolje/opensearch/domene/StatustallResponse.java b/src/main/java/no/nav/pto/veilarbportefolje/opensearch/domene/StatustallResponse.java index 65b65c873f..d56356dd95 100644 --- a/src/main/java/no/nav/pto/veilarbportefolje/opensearch/domene/StatustallResponse.java +++ b/src/main/java/no/nav/pto/veilarbportefolje/opensearch/domene/StatustallResponse.java @@ -48,6 +48,7 @@ public static class StatustallBuckets { Bucket fargekategoriE; Bucket fargekategoriF; Bucket fargekategoriIngenKategori; + Bucket tiltakshendelser; } } } 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 3aaa86bfa0..883ae05f4d 100644 --- a/src/test/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchServiceIntegrationTest.java +++ b/src/test/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchServiceIntegrationTest.java @@ -432,7 +432,6 @@ public void skal_hente_ut_brukere_ved_soek_paa_flere_veiledere() { public void skal_hente_riktig_antall_ufordelte_brukere() { List brukere = List.of( - new OppfolgingsBruker() .setAktoer_id(randomAktorId().toString()) .setFnr(randomFnr().get()) @@ -613,6 +612,7 @@ void skal_hente_riktige_statustall_for_veileder() { assertThat(statustall.getFargekategoriE()).isEqualTo(0); assertThat(statustall.getFargekategoriF()).isEqualTo(0); assertThat(statustall.getFargekategoriIngenKategori()).isEqualTo(1); + assertThat(statustall.getTiltakshendelser()).isEqualTo(0); } @Test @@ -3285,10 +3285,14 @@ public void test_filtrering_tiltakshendelser() { null ); List sorterteBrukere = response.getBrukere().stream().sorted(new BrukerComparator()).toList(); +//Sjekk statustall assertThat(response.getAntall()).isEqualTo(2); assertThat(sorterteBrukere.get(0).getFnr()).isEqualTo(bruker2Fnr.toString()); assertThat(sorterteBrukere.get(1).getFnr()).isEqualTo(bruker3Fnr.toString()); + + var statustall = opensearchService.hentStatustallForVeilederPortefolje(TEST_VEILEDER_0, TEST_ENHET); + assertThat(statustall.getTiltakshendelser()).isEqualTo(2); } @Test From 5d9887152264aa5ac472ad96036ec85f219c0e2f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ingrid=20Foss=C3=A5?= Date: Tue, 10 Sep 2024 13:59:24 +0200 Subject: [PATCH 2/3] Bruk isZero i staden for isEqualTo(0) fordi SonarCloud seier det er betre. Co-authored-by: Klara Margrethe Helgemo --- .../OpensearchServiceIntegrationTest.java | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) 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 883ae05f4d..4a59ebbc01 100644 --- a/src/test/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchServiceIntegrationTest.java +++ b/src/test/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchServiceIntegrationTest.java @@ -595,7 +595,7 @@ void skal_hente_riktige_statustall_for_veileder() { pollOpensearchUntil(() -> opensearchTestClient.countDocuments() == liste.size()); var statustall = opensearchService.hentStatustallForVeilederPortefolje(TEST_VEILEDER_0, TEST_ENHET); - assertThat(statustall.getErSykmeldtMedArbeidsgiver()).isEqualTo(0); + assertThat(statustall.getErSykmeldtMedArbeidsgiver()).isZero(); assertThat(statustall.getIavtaltAktivitet()).isEqualTo(1); assertThat(statustall.getIkkeIavtaltAktivitet()).isEqualTo(2); assertThat(statustall.getInaktiveBrukere()).isEqualTo(1); @@ -607,12 +607,12 @@ void skal_hente_riktige_statustall_for_veileder() { assertThat(statustall.getMineHuskelapper()).isEqualTo(1); assertThat(statustall.getFargekategoriA()).isEqualTo(1); assertThat(statustall.getFargekategoriB()).isEqualTo(1); - assertThat(statustall.getFargekategoriC()).isEqualTo(0); - assertThat(statustall.getFargekategoriD()).isEqualTo(0); - assertThat(statustall.getFargekategoriE()).isEqualTo(0); - assertThat(statustall.getFargekategoriF()).isEqualTo(0); + assertThat(statustall.getFargekategoriC()).isZero(); + assertThat(statustall.getFargekategoriD()).isZero(); + assertThat(statustall.getFargekategoriE()).isZero(); + assertThat(statustall.getFargekategoriF()).isZero(); assertThat(statustall.getFargekategoriIngenKategori()).isEqualTo(1); - assertThat(statustall.getTiltakshendelser()).isEqualTo(0); + assertThat(statustall.getTiltakshendelser()).isZero(); } @Test @@ -797,7 +797,7 @@ public void skal_hente_riktige_statustall_for_enhet_naar_veileder_har_alle_bruke ); assertThat(responsMedBrukerinnsyn.getTotalt()).isEqualTo(15); - assertThat(responsUtenBrukerinnsyn.getTotalt()).isEqualTo(0); + assertThat(responsUtenBrukerinnsyn.getTotalt()).isZero(); assertThat(responsMedBrukerinnsyn.getVenterPaSvarFraNAV()).isEqualTo(5); assertThat(responsMedBrukerinnsyn.getUfordelteBrukere()).isEqualTo(5); } @@ -951,7 +951,7 @@ void skal_hente_riktige_statustall_for_enhet() { var statustallForBrukereSomVeilederIkkeHarInnsynsrettPå = opensearchService.hentStatusTallForEnhetPortefolje(TEST_ENHET, BRUKERE_SOM_VEILEDER_IKKE_HAR_INNSYNSRETT_PÅ); assertThat(statustallForBrukereSomVeilederHarInnsynsrettPå.getUfordelteBrukere()).isEqualTo(1); - assertThat(statustallForBrukereSomVeilederIkkeHarInnsynsrettPå.getUfordelteBrukere()).isEqualTo(0); + assertThat(statustallForBrukereSomVeilederIkkeHarInnsynsrettPå.getUfordelteBrukere()).isZero(); } @Test @@ -2795,7 +2795,7 @@ public void test_filtrering_barn_under_18() { } else if (bruker.getFnr().equals(bruker3.getFnr())) { assertThat(bruker.getBarnUnder18AarData().size()).isEqualTo(2); } else if (bruker.getFnr().equals(bruker4.getFnr())) { - assertThat(bruker.getBarnUnder18AarData().size()).isEqualTo(0); + assertThat(bruker.getBarnUnder18AarData().size()).isZero(); } else if (bruker.getFnr().equals(bruker5.getFnr())) { assertThat(bruker.getBarnUnder18AarData().size()).isEqualTo(1); } From 684d1491e335f3026bef1e9cd43445434f9dd327 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ingrid=20Foss=C3=A5?= Date: Tue, 10 Sep 2024 14:03:48 +0200 Subject: [PATCH 3/3] Endringar etter review Co-authored-by: Klara Margrethe Helgemo --- .../java/no/nav/pto/veilarbportefolje/domene/Statustall.java | 2 +- .../opensearch/OpensearchServiceIntegrationTest.java | 3 +-- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/src/main/java/no/nav/pto/veilarbportefolje/domene/Statustall.java b/src/main/java/no/nav/pto/veilarbportefolje/domene/Statustall.java index dad4e4e245..2308ff4549 100644 --- a/src/main/java/no/nav/pto/veilarbportefolje/domene/Statustall.java +++ b/src/main/java/no/nav/pto/veilarbportefolje/domene/Statustall.java @@ -94,4 +94,4 @@ public Statustall(StatustallResponse.StatustallAggregation.StatustallFilter.Stat this.fargekategoriIngenKategori = buckets.getFargekategoriIngenKategori().getDoc_count(); this.tiltakshendelser = buckets.getTiltakshendelser().getDoc_count(); } -} \ No newline at end of file +} 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 4a59ebbc01..d4bcaa85d5 100644 --- a/src/test/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchServiceIntegrationTest.java +++ b/src/test/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchServiceIntegrationTest.java @@ -3224,7 +3224,7 @@ public void test_filtrering_barn_under_18_med_alder_filter_har_tilgang_7() { } @Test - public void test_filtrering_tiltakshendelser() { + public void test_filtrering_og_statustall_tiltakshendelser() { OppfolgingsBruker bruker1 = new OppfolgingsBruker() .setFnr(randomFnr().toString()) .setAktoer_id(randomAktorId().toString()) @@ -3285,7 +3285,6 @@ public void test_filtrering_tiltakshendelser() { null ); List sorterteBrukere = response.getBrukere().stream().sorted(new BrukerComparator()).toList(); -//Sjekk statustall assertThat(response.getAntall()).isEqualTo(2); assertThat(sorterteBrukere.get(0).getFnr()).isEqualTo(bruker2Fnr.toString());