Skip to content

Commit

Permalink
Merge pull request #130 from navikt/dev
Browse files Browse the repository at this point in the history 
Tolk behov
  • Loading branch information
@dragutin-nav
dragutin-nav authored Jul 4, 2022
2 parents ae5c9a4 + 4778019 commit 0e72bfe
Show file tree
Hide file tree
Showing 6 changed files with 32 additions and 25 deletions.
2 changes: 1 addition & 1 deletion pom.xml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -112,7 +112,7 @@
<dependency>
<groupId>org.projectlombok</groupId>
<artifactId>lombok</artifactId>
<version>1.18.22</version>
<version>1.18.24</version>
</dependency>

<!-- Common -->
Expand Down
11 changes: 4 additions & 7 deletions src/main/java/no/nav/pto/veilarbfilter/domene/PortefoljeFilter.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -92,11 +92,8 @@ public class PortefoljeFilter {
@JsonSetter(nulls = Nulls.AS_EMPTY)
private List<String> foedeland = emptyList();

private Boolean talespraaktolk;

private Boolean tegnspraaktolk;

private String tolkBehovSpraak = "";
@JsonSetter(nulls = Nulls.AS_EMPTY)
private List<String> tolkebehov = emptyList();;

@JsonIgnore
public Boolean isNotEmpty() {
Expand All @@ -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());
}

}
34 changes: 22 additions & 12 deletions src/main/java/no/nav/pto/veilarbfilter/service/MetricsReporter.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -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();
Expand All @@ -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();
}
Expand All @@ -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);

Expand Down
6 changes: 3 additions & 3 deletions src/test/java/no/nav/pto/veilarbfilter/domene/PortefoljeFilterTest.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -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);
Expand All @@ -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);
}
Expand Down
2 changes: 1 addition & 1 deletion src/test/java/no/nav/pto/veilarbfilter/rest/MineLagredeFilterTest.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -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() {
Expand Down
2 changes: 1 addition & 1 deletion src/test/java/no/nav/pto/veilarbfilter/rest/VeilederGruppeTest.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -214,6 +214,6 @@ private NyttFilterModel getRandomNyttFilter(List<String> veiledersList) {
public PortefoljeFilter getRandomPortefoljeFilter(List<String> 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());
}
}

0 comments on commit 0e72bfe

Please sign in to comment.