diff --git a/pom.xml b/pom.xml index f339cd2a..be41c11b 100644 --- a/pom.xml +++ b/pom.xml @@ -112,7 +112,7 @@ org.projectlombok lombok - 1.18.22 + 1.18.24 diff --git a/src/main/java/no/nav/pto/veilarbfilter/domene/PortefoljeFilter.java b/src/main/java/no/nav/pto/veilarbfilter/domene/PortefoljeFilter.java index a1a337e5..4fea970a 100644 --- a/src/main/java/no/nav/pto/veilarbfilter/domene/PortefoljeFilter.java +++ b/src/main/java/no/nav/pto/veilarbfilter/domene/PortefoljeFilter.java @@ -92,11 +92,8 @@ public class PortefoljeFilter { @JsonSetter(nulls = Nulls.AS_EMPTY) private List foedeland = emptyList(); - private Boolean talespraaktolk; - - private Boolean tegnspraaktolk; - - private String tolkBehovSpraak = ""; + @JsonSetter(nulls = Nulls.AS_EMPTY) + private List tolkebehov = emptyList();; @JsonIgnore public Boolean isNotEmpty() { @@ -123,8 +120,8 @@ public Boolean isNotEmpty() { (sisteEndringKategori != null && !sisteEndringKategori.isEmpty()) || (aktiviteterForenklet != null && !aktiviteterForenklet.isEmpty()) || aktiviteter != null || - !landgruppe.isEmpty() || !foedeland.isEmpty() || !tolkBehovSpraak.isEmpty() || - talespraaktolk != null || tegnspraaktolk != null; + (landgruppe != null && !landgruppe.isEmpty()) || (foedeland != null && !foedeland.isEmpty()) || + (tolkebehov != null && !tolkebehov.isEmpty()); } } \ No newline at end of file diff --git a/src/main/java/no/nav/pto/veilarbfilter/service/MetricsReporter.java b/src/main/java/no/nav/pto/veilarbfilter/service/MetricsReporter.java index f2ff90ed..125db864 100644 --- a/src/main/java/no/nav/pto/veilarbfilter/service/MetricsReporter.java +++ b/src/main/java/no/nav/pto/veilarbfilter/service/MetricsReporter.java @@ -67,31 +67,31 @@ public void reportLagradeFilter() { antallFiltre++; } } - if (!filterValg.getAlder().isEmpty()) { + if (filterValg.getAlder() != null && !filterValg.getAlder().isEmpty()) { metrikk.addTagToReport("alder", "1"); addValuesAsTags(metrikk, filterValg.getAlder()); antallFiltre += filterValg.getAlder().size(); } - if (!filterValg.getFerdigfilterListe().isEmpty()) { + if (filterValg.getFerdigfilterListe() != null && !filterValg.getFerdigfilterListe().isEmpty()) { metrikk.addTagToReport("ferdigfilterListe", "1"); addValuesAsTags(metrikk, filterValg.getFerdigfilterListe()); antallFiltre += filterValg.getFerdigfilterListe().size(); } - if (!filterValg.getFodselsdagIMnd().isEmpty()) { + if (filterValg.getFodselsdagIMnd() != null && !filterValg.getFodselsdagIMnd().isEmpty()) { metrikk.addTagToReport("fodselsdagIMnd", "1"); antallFiltre += filterValg.getFodselsdagIMnd().size(); } - if (!filterValg.getFormidlingsgruppe().isEmpty()) { + if (filterValg.getFormidlingsgruppe() != null && !filterValg.getFormidlingsgruppe().isEmpty()) { metrikk.addTagToReport("formidlingsgruppe", "1"); addValuesAsTags(metrikk, filterValg.getFormidlingsgruppe()); antallFiltre += filterValg.getFormidlingsgruppe().size(); } - if (!filterValg.getHovedmal().isEmpty()) { + if (filterValg.getHovedmal() != null && !filterValg.getHovedmal().isEmpty()) { metrikk.addTagToReport("hovedmal", "1"); addValuesAsTags(metrikk, filterValg.getHovedmal()); antallFiltre += filterValg.getHovedmal().size(); } - if (!filterValg.getInnsatsgruppe().isEmpty()) { + if (filterValg.getInnsatsgruppe() != null && !filterValg.getInnsatsgruppe().isEmpty()) { metrikk.addTagToReport("innsatsgruppe", "1"); addValuesAsTags(metrikk, filterValg.getInnsatsgruppe()); antallFiltre += filterValg.getInnsatsgruppe().size(); @@ -101,31 +101,31 @@ public void reportLagradeFilter() { metrikk.addTagToReport(filterValg.getKjonn(), "1"); antallFiltre++; } - if (!filterValg.getManuellBrukerStatus().isEmpty()) { + if (filterValg.getManuellBrukerStatus() != null && !filterValg.getManuellBrukerStatus().isEmpty()) { metrikk.addTagToReport("manuellBrukerStatus", "1"); addValuesAsTags(metrikk, filterValg.getManuellBrukerStatus()); antallFiltre += filterValg.getManuellBrukerStatus().size(); } - if (!filterValg.getRettighetsgruppe().isEmpty()) { + if (filterValg.getRettighetsgruppe() != null && !filterValg.getRettighetsgruppe().isEmpty()) { metrikk.addTagToReport("rettighetsgruppe", "1"); addValuesAsTags(metrikk, filterValg.getRettighetsgruppe()); antallFiltre += filterValg.getRettighetsgruppe().size(); } - if (!filterValg.getServicegruppe().isEmpty()) { + if (filterValg.getServicegruppe() != null && !filterValg.getServicegruppe().isEmpty()) { metrikk.addTagToReport("servicegruppe", "1"); addValuesAsTags(metrikk, filterValg.getServicegruppe()); antallFiltre += filterValg.getServicegruppe().size(); } - if (!filterValg.getTiltakstyper().isEmpty()) { + if (filterValg.getTiltakstyper() != null && !filterValg.getTiltakstyper().isEmpty()) { metrikk.addTagToReport("tiltakstyper", "1"); addValuesAsTags(metrikk, filterValg.getTiltakstyper()); antallFiltre += filterValg.getTiltakstyper().size(); } - if (!filterValg.getVeilederNavnQuery().isEmpty()) { + if (filterValg.getVeilederNavnQuery() != null && !filterValg.getVeilederNavnQuery().isEmpty()) { metrikk.addTagToReport("veilederNavnQuery", "1"); antallFiltre++; } - if (!filterValg.getVeiledere().isEmpty()) { + if (filterValg.getVeiledere() != null && !filterValg.getVeiledere().isEmpty()) { metrikk.addTagToReport("veiledere", "1"); antallFiltre += filterValg.getVeiledere().size(); } @@ -148,6 +148,16 @@ public void reportLagradeFilter() { metrikk.addTagToReport("arbeidslisteKategori", "1"); antallFiltre += filterValg.getArbeidslisteKategori().size(); } + if (filterValg.getLandgruppe() != null && !filterValg.getLandgruppe().isEmpty()) { + metrikk.addTagToReport("landgruppe", "1"); + addValuesAsTags(metrikk, filterValg.getLandgruppe()); + antallFiltre += filterValg.getLandgruppe().size(); + } + if (filterValg.getTolkebehov() != null && !filterValg.getTolkebehov().isEmpty()) { + metrikk.addTagToReport("tolkbehov", "1"); + addValuesAsTags(metrikk, filterValg.getTolkebehov()); + antallFiltre += filterValg.getTolkebehov().size(); + } metrikk.addFieldToReport("antallFiltre", antallFiltre); metricsClient.report(metrikk); diff --git a/src/test/java/no/nav/pto/veilarbfilter/domene/PortefoljeFilterTest.java b/src/test/java/no/nav/pto/veilarbfilter/domene/PortefoljeFilterTest.java index ef9f5124..01aa67ea 100644 --- a/src/test/java/no/nav/pto/veilarbfilter/domene/PortefoljeFilterTest.java +++ b/src/test/java/no/nav/pto/veilarbfilter/domene/PortefoljeFilterTest.java @@ -98,7 +98,7 @@ public void testDeserializationAndSettingDefaultValue() throws JsonProcessingExc @Test public void testSerializationOfEmptyFilter() throws JsonProcessingException { String correctOutput = """ - {"aktiviteter":null,"aktiviteterForenklet":[],"alder":[],"arbeidslisteKategori":[],"cvJobbprofil":"","ferdigfilterListe":[],"fodselsdagIMnd":[],"foedeland":[],"formidlingsgruppe":[],"hovedmal":[],"innsatsgruppe":[],"kjonn":"","landgruppe":[],"manuellBrukerStatus":[],"navnEllerFnrQuery":"","registreringstype":[],"rettighetsgruppe":[],"servicegruppe":[],"sisteEndringKategori":[],"talespraaktolk":null,"tegnspraaktolk":null,"tiltakstyper":[],"tolkBehovSpraak":"","ulesteEndringer":"","utdanning":[],"utdanningBestatt":[],"utdanningGodkjent":[],"veilederNavnQuery":"","veiledere":[],"ytelse":""}"""; + {"aktiviteter":null,"aktiviteterForenklet":[],"alder":[],"arbeidslisteKategori":[],"cvJobbprofil":"","ferdigfilterListe":[],"fodselsdagIMnd":[],"foedeland":[],"formidlingsgruppe":[],"hovedmal":[],"innsatsgruppe":[],"kjonn":"","landgruppe":[],"manuellBrukerStatus":[],"navnEllerFnrQuery":"","registreringstype":[],"rettighetsgruppe":[],"servicegruppe":[],"sisteEndringKategori":[],"tiltakstyper":[],"tolkebehov":[],"ulesteEndringer":"","utdanning":[],"utdanningBestatt":[],"utdanningGodkjent":[],"veilederNavnQuery":"","veiledere":[],"ytelse":""}"""; PortefoljeFilter portefoljeFilter = new PortefoljeFilter(); String jsonString = objectMapper.writeValueAsString(portefoljeFilter); Assertions.assertEquals(jsonString, correctOutput); @@ -107,10 +107,10 @@ public void testSerializationOfEmptyFilter() throws JsonProcessingException { @Test public void testSerializationOfVeiledere() throws JsonProcessingException { String correctOutput = """ - {"aktiviteter":null,"aktiviteterForenklet":null,"alder":null,"arbeidslisteKategori":null,"cvJobbprofil":null,"ferdigfilterListe":null,"fodselsdagIMnd":null,"foedeland":null,"formidlingsgruppe":null,"hovedmal":null,"innsatsgruppe":null,"kjonn":null,"landgruppe":null,"manuellBrukerStatus":null,"navnEllerFnrQuery":null,"registreringstype":null,"rettighetsgruppe":null,"servicegruppe":null,"sisteEndringKategori":null,"talespraaktolk":null,"tegnspraaktolk":null,"tiltakstyper":null,"tolkBehovSpraak":null,"ulesteEndringer":null,"utdanning":null,"utdanningBestatt":null,"utdanningGodkjent":null,"veilederNavnQuery":null,"veiledere":["A123","B123"],"ytelse":null}"""; + {"aktiviteter":null,"aktiviteterForenklet":null,"alder":null,"arbeidslisteKategori":null,"cvJobbprofil":null,"ferdigfilterListe":null,"fodselsdagIMnd":null,"foedeland":null,"formidlingsgruppe":null,"hovedmal":null,"innsatsgruppe":null,"kjonn":null,"landgruppe":null,"manuellBrukerStatus":null,"navnEllerFnrQuery":null,"registreringstype":null,"rettighetsgruppe":null,"servicegruppe":null,"sisteEndringKategori":null,"tiltakstyper":null,"tolkebehov":null,"ulesteEndringer":null,"utdanning":null,"utdanningBestatt":null,"utdanningGodkjent":null,"veilederNavnQuery":null,"veiledere":["A123","B123"],"ytelse":null}"""; PortefoljeFilter portefoljeFilter = new PortefoljeFilter(null, null, null, null, null, null, null, null, null, null, null, null, null, null, List.of("A123", "B123"), null, null, null, null, null, null, null, null, null, null, - null, null, null, null, null); + null, null, null); String jsonString = objectMapper.writeValueAsString(portefoljeFilter); Assertions.assertEquals(jsonString, correctOutput); } diff --git a/src/test/java/no/nav/pto/veilarbfilter/rest/MineLagredeFilterTest.java b/src/test/java/no/nav/pto/veilarbfilter/rest/MineLagredeFilterTest.java index b8fe701d..7c28a265 100644 --- a/src/test/java/no/nav/pto/veilarbfilter/rest/MineLagredeFilterTest.java +++ b/src/test/java/no/nav/pto/veilarbfilter/rest/MineLagredeFilterTest.java @@ -415,7 +415,7 @@ public PortefoljeFilter getRandomPortefoljeFilter() { emptyList(), emptyList(), emptyList(), emptyList(), kjonnVelg.get(random.nextInt(1)), emptyList(), String.valueOf(rndChar()), emptyList(), emptyList(), emptyList(), "", emptyList(), "", emptyList(), "", emptyList(), emptyList(), emptyList(), emptyList(), emptyList(), "", emptyList(), emptyList(), emptyList(), - null, null, ""); + emptyList()); } private static char rndChar() { diff --git a/src/test/java/no/nav/pto/veilarbfilter/rest/VeilederGruppeTest.java b/src/test/java/no/nav/pto/veilarbfilter/rest/VeilederGruppeTest.java index d4c1553c..68f501a6 100644 --- a/src/test/java/no/nav/pto/veilarbfilter/rest/VeilederGruppeTest.java +++ b/src/test/java/no/nav/pto/veilarbfilter/rest/VeilederGruppeTest.java @@ -214,6 +214,6 @@ private NyttFilterModel getRandomNyttFilter(List veiledersList) { public PortefoljeFilter getRandomPortefoljeFilter(List veiledersList) { return new PortefoljeFilter(null, emptyList(), emptyList(), emptyList(), emptyList(), emptyList(), emptyList(), "", emptyList(), "", emptyList(), emptyList(), emptyList(), "", veiledersList, "", emptyList(), "", emptyList(), - emptyList(), emptyList(), emptyList(), emptyList(), "", emptyList(), emptyList(), emptyList(), null, null, ""); + emptyList(), emptyList(), emptyList(), emptyList(), "", emptyList(), emptyList(), emptyList(), emptyList()); } }