Skip to content

Commit 28e5289

Browse files
slovridslovrid
committed
Endre namn frå "siste14aVedtak" til "gjeldendeVedtak14a" ifm. nytt filter
1 parent 06aad12 commit 28e5289

11 files changed

+52
-63
lines changed

‎src/main/java/no/nav/pto/veilarbportefolje/domene/Bruker.java

+3-4
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,7 @@
1111
import no.nav.pto.veilarbportefolje.persononinfo.barnUnder18Aar.BarnUnder18AarData;
1212
import no.nav.pto.veilarbportefolje.persononinfo.domene.Adressebeskyttelse;
1313
import no.nav.pto.veilarbportefolje.siste14aVedtak.Avvik14aVedtak;
14-
import no.nav.pto.veilarbportefolje.siste14aVedtak.Siste14aVedtak;
15-
import no.nav.pto.veilarbportefolje.siste14aVedtak.Siste14aVedtakForBruker;
14+
import no.nav.pto.veilarbportefolje.siste14aVedtak.GjeldendeVedtak14a;
1615
import no.nav.pto.veilarbportefolje.util.OppfolgingUtils;
1716

1817
import java.sql.Timestamp;
@@ -115,7 +114,7 @@ public class Bruker {
115114
String fargekategoriEnhetId;
116115

117116
TiltakshendelseForBruker tiltakshendelse;
118-
Siste14aVedtak siste14aVedtak;
117+
GjeldendeVedtak14a gjeldendeVedtak14a;
119118

120119
public static Bruker of(OppfolgingsBruker bruker, boolean ufordelt, boolean erVedtakstottePilotPa) {
121120

@@ -214,7 +213,7 @@ public static Bruker of(OppfolgingsBruker bruker, boolean ufordelt, boolean erVe
214213
.setFargekategori(bruker.getFargekategori())
215214
.setFargekategoriEnhetId(bruker.getFargekategori_enhetId())
216215
.setTiltakshendelse(TiltakshendelseForBruker.of(bruker.getTiltakshendelse()))
217-
.setSiste14aVedtak(bruker.getSiste14aVedtak());
216+
.setGjeldendeVedtak14a(bruker.getGjeldendeVedtak14a());
218217
}
219218

220219
public void kalkulerNesteUtlopsdatoAvValgtAktivitetFornklet(List<String> aktiviteterForenklet) {

‎src/main/java/no/nav/pto/veilarbportefolje/domene/Filtervalg.java

+7-8
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@
33
import lombok.Data;
44
import lombok.experimental.Accessors;
55
import no.nav.pto.veilarbportefolje.arbeidssoeker.v2.JobbSituasjonBeskrivelse;
6-
import no.nav.pto.veilarbportefolje.domene.filtervalg.DinSituasjonSvar;
76
import no.nav.pto.veilarbportefolje.domene.filtervalg.UtdanningBestattSvar;
87
import no.nav.pto.veilarbportefolje.domene.filtervalg.UtdanningGodkjentSvar;
98
import no.nav.pto.veilarbportefolje.domene.filtervalg.UtdanningSvar;
@@ -56,7 +55,7 @@ public class Filtervalg {
5655
public List<Avvik14aVedtak> avvik14aVedtak;
5756
public List<EnsligeForsorgere> ensligeForsorgere;
5857
public List<String> fargekategorier = new ArrayList<>();
59-
public List<String> siste14aVedtak = new ArrayList<>();
58+
public List<String> gjeldendeVedtak14a = new ArrayList<>();
6059

6160
public boolean harAktiveFilter() {
6261
return harFerdigFilter() ||
@@ -93,11 +92,11 @@ public boolean harAktiveFilter() {
9392
harAvvik14aVedtakFilter() ||
9493
harEnsligeForsorgereFilter() ||
9594
harFargeKategoriFilter() ||
96-
harSiste14aVedtakFilter();
95+
harGjeldendeVedtak14aFilter();
9796
}
9897

99-
public boolean harSiste14aVedtakFilter() {
100-
return siste14aVedtak != null && !siste14aVedtak.isEmpty();
98+
public boolean harGjeldendeVedtak14aFilter() {
99+
return gjeldendeVedtak14a != null && !gjeldendeVedtak14a.isEmpty();
101100
}
102101

103102
public boolean harEnsligeForsorgereFilter() {
@@ -237,12 +236,12 @@ public boolean valider() {
237236
.map(Filtervalg::erGyldigAldersSpenn)
238237
.reduce(true, and());
239238

240-
Boolean siste14aVedtakOk = siste14aVedtak
239+
Boolean gjeldendeVedtak14aOk = gjeldendeVedtak14a
241240
.stream()
242-
.map(Siste14aVedtakFilter::contains)
241+
.map(GjeldendeVedtak14aFilter::contains)
243242
.reduce(true, and());
244243

245-
return alderOk && fodselsdatoOk && veiledereOk && utdanningOK && sisteEndringOK && barnAlderOk && siste14aVedtakOk;
244+
return alderOk && fodselsdatoOk && veiledereOk && utdanningOK && sisteEndringOK && barnAlderOk && gjeldendeVedtak14aOk;
246245
}
247246

248247
private BinaryOperator<Boolean> and() {

‎src/main/java/no/nav/pto/veilarbportefolje/domene/Siste14aVedtakFilter.java ‎src/main/java/no/nav/pto/veilarbportefolje/domene/GjeldendeVedtak14aFilter.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -3,13 +3,13 @@
33
import lombok.extern.slf4j.Slf4j;
44

55
@Slf4j
6-
public enum Siste14aVedtakFilter {
6+
public enum GjeldendeVedtak14aFilter {
77
HAR_14A_VEDTAK,
88
HAR_IKKE_14A_VEDTAK;
99

1010
public static boolean contains(String value) {
1111
try {
12-
Siste14aVedtakFilter.valueOf(value);
12+
GjeldendeVedtak14aFilter.valueOf(value);
1313
return true;
1414
} catch (IllegalArgumentException e) {
1515
log.warn("Kunne ikke instanstiere enum - fikk ugyldig verdi.");

‎src/main/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchIndexerV2.java

+3-4
Original file line numberDiff line numberDiff line change
@@ -377,18 +377,17 @@ public void slettTiltakshendelse(AktorId aktorId) {
377377
}
378378

379379
@SneakyThrows
380-
public void updateSiste14aVedtak(Siste14aVedtakForBruker siste14AVedtakForBruker, AktorId aktorId) {
380+
public void updateGjeldendeVedtak14a(Siste14aVedtakForBruker siste14AVedtakForBruker, AktorId aktorId) {
381381
final XContentBuilder content = jsonBuilder()
382382
.startObject()
383-
.startObject("siste14aVedtak")
383+
.startObject("gjeldendeVedtak14a")
384384
.field("innsatsgruppe", siste14AVedtakForBruker.getInnsatsgruppe())
385385
.field("hovedmal", siste14AVedtakForBruker.getHovedmal())
386386
.field("fattetDato", siste14AVedtakForBruker.getFattetDato())
387-
.field("fraArena", siste14AVedtakForBruker.isFraArena())
388387
.endObject()
389388
.endObject();
390389

391-
update(aktorId, content, format("Oppdaterte siste14aVedtak for aktorId: %s", aktorId.get()));
390+
update(aktorId, content, format("Oppdaterte gjeldendeVedtak14a for aktorId: %s", aktorId.get()));
392391
}
393392

394393
private void update(AktorId aktoerId, XContentBuilder content, String logInfo) throws IOException {

‎src/main/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchQueryBuilder.java

+8-9
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,6 @@
2828
import java.util.List;
2929
import java.util.Map;
3030
import java.util.StringJoiner;
31-
import java.util.stream.Collectors;
3231

3332
import static java.lang.Integer.parseInt;
3433
import static java.lang.String.format;
@@ -406,17 +405,17 @@ static void leggTilManuelleFilter(BoolQueryBuilder queryBuilder, Filtervalg filt
406405
queryBuilder.must(brukerensUtdanningSubQuery);
407406
}
408407

409-
if (filtervalg.harSiste14aVedtakFilter()) {
410-
List<Siste14aVedtakFilter> valgteSiste14aVedtakFilter = filtervalg.siste14aVedtak.stream().map(Siste14aVedtakFilter::valueOf).toList();
408+
if (filtervalg.harGjeldendeVedtak14aFilter()) {
409+
List<GjeldendeVedtak14aFilter> valgteGjeldendeVedtak14aFilter = filtervalg.gjeldendeVedtak14a.stream().map(GjeldendeVedtak14aFilter::valueOf).toList();
411410
BoolQueryBuilder subQuery = boolQuery();
412411

413-
if (valgteSiste14aVedtakFilter.contains(Siste14aVedtakFilter.HAR_14A_VEDTAK)
414-
&& !valgteSiste14aVedtakFilter.contains(Siste14aVedtakFilter.HAR_IKKE_14A_VEDTAK)) {
415-
subQuery.must(existsQuery("siste14aVedtak"));
412+
if (valgteGjeldendeVedtak14aFilter.contains(GjeldendeVedtak14aFilter.HAR_14A_VEDTAK)
413+
&& !valgteGjeldendeVedtak14aFilter.contains(GjeldendeVedtak14aFilter.HAR_IKKE_14A_VEDTAK)) {
414+
subQuery.must(existsQuery("gjeldendeVedtak14a"));
416415
queryBuilder.must(subQuery);
417-
} else if (valgteSiste14aVedtakFilter.contains(Siste14aVedtakFilter.HAR_IKKE_14A_VEDTAK)
418-
&& !valgteSiste14aVedtakFilter.contains(Siste14aVedtakFilter.HAR_14A_VEDTAK)) {
419-
subQuery.mustNot(existsQuery("siste14aVedtak"));
416+
} else if (valgteGjeldendeVedtak14aFilter.contains(GjeldendeVedtak14aFilter.HAR_IKKE_14A_VEDTAK)
417+
&& !valgteGjeldendeVedtak14aFilter.contains(GjeldendeVedtak14aFilter.HAR_14A_VEDTAK)) {
418+
subQuery.mustNot(existsQuery("gjeldendeVedtak14a"));
420419
queryBuilder.must(subQuery);
421420
}
422421
}

‎src/main/java/no/nav/pto/veilarbportefolje/opensearch/domene/OppfolgingsBruker.java

+2-3
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,7 @@
77
import no.nav.pto.veilarbportefolje.domene.Statsborgerskap;
88
import no.nav.pto.veilarbportefolje.persononinfo.barnUnder18Aar.BarnUnder18AarData;
99
import no.nav.pto.veilarbportefolje.siste14aVedtak.Avvik14aVedtak;
10-
import no.nav.pto.veilarbportefolje.siste14aVedtak.Siste14aVedtak;
11-
import no.nav.pto.veilarbportefolje.siste14aVedtak.Siste14aVedtakForBruker;
10+
import no.nav.pto.veilarbportefolje.siste14aVedtak.GjeldendeVedtak14a;
1211
import no.nav.pto.veilarbportefolje.tiltakshendelse.domain.Tiltakshendelse;
1312

1413
import java.time.LocalDate;
@@ -134,5 +133,5 @@ public class OppfolgingsBruker {
134133
String fargekategori_enhetId;
135134
Tiltakshendelse tiltakshendelse;
136135

137-
Siste14aVedtak siste14aVedtak;
136+
GjeldendeVedtak14a gjeldendeVedtak14a;
138137
}

‎src/main/java/no/nav/pto/veilarbportefolje/postgres/PostgresOpensearchMapper.java

+2-3
Original file line numberDiff line numberDiff line change
@@ -233,11 +233,10 @@ public void flettInnSiste14aVedtak(List<OppfolgingsBruker> brukere) {
233233
);
234234
brukere.forEach(bruker -> {
235235
Optional<Siste14aVedtakForBruker> maybeSiste14aVedtakForBruker = Optional.ofNullable(aktorIdSiste14aVedtakMap.get(AktorId.of(bruker.getAktoer_id())));
236-
bruker.setSiste14aVedtak(maybeSiste14aVedtakForBruker.map(siste14aVedtakForBruker -> new Siste14aVedtak(
236+
bruker.setGjeldendeVedtak14a(maybeSiste14aVedtakForBruker.map(siste14aVedtakForBruker -> new GjeldendeVedtak14a(
237237
siste14aVedtakForBruker.getInnsatsgruppe(),
238238
siste14aVedtakForBruker.getHovedmal(),
239-
siste14aVedtakForBruker.getFattetDato(),
240-
siste14aVedtakForBruker.isFraArena()
239+
siste14aVedtakForBruker.getFattetDato()
241240
)).orElse(null));
242241
});
243242
}

‎src/main/java/no/nav/pto/veilarbportefolje/siste14aVedtak/Siste14aVedtak.java ‎src/main/java/no/nav/pto/veilarbportefolje/siste14aVedtak/GjeldendeVedtak14a.java

+2-3
Original file line numberDiff line numberDiff line change
@@ -5,10 +5,9 @@
55

66
import java.time.ZonedDateTime;
77

8-
public record Siste14aVedtak(
8+
public record GjeldendeVedtak14a(
99
Innsatsgruppe innsatsgruppe,
1010
Hovedmal hovedmal,
11-
ZonedDateTime fattetDato,
12-
boolean fraArena
11+
ZonedDateTime fattetDato
1312
) {
1413
}

‎src/main/java/no/nav/pto/veilarbportefolje/siste14aVedtak/Siste14aVedtakService.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ public void lagreSiste14aVedtak(Siste14aVedtakForBruker siste14AVedtakForBruker)
2828
if (pdlIdentRepository.erBrukerUnderOppfolging(siste14AVedtakForBruker.aktorId.get())) {
2929
IdenterForBruker identer = pdlIdentRepository.hentIdenterForBruker(siste14AVedtakForBruker.aktorId.get());
3030
siste14aVedtakRepository.upsert(siste14AVedtakForBruker, identer);
31-
opensearchIndexerV2.updateSiste14aVedtak(siste14AVedtakForBruker, siste14AVedtakForBruker.getAktorId());
31+
opensearchIndexerV2.updateGjeldendeVedtak14a(siste14AVedtakForBruker, siste14AVedtakForBruker.getAktorId());
3232
}
3333
}
3434

‎src/test/java/no/nav/pto/veilarbportefolje/opensearch/OpensearchServiceIntegrationTest.java

+14-17
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222
import no.nav.pto.veilarbportefolje.persononinfo.barnUnder18Aar.BarnUnder18AarData;
2323
import no.nav.pto.veilarbportefolje.persononinfo.domene.Adressebeskyttelse;
2424
import no.nav.pto.veilarbportefolje.siste14aVedtak.Avvik14aVedtak;
25-
import no.nav.pto.veilarbportefolje.siste14aVedtak.Siste14aVedtak;
25+
import no.nav.pto.veilarbportefolje.siste14aVedtak.GjeldendeVedtak14a;
2626
import no.nav.pto.veilarbportefolje.siste14aVedtak.Siste14aVedtakForBruker;
2727
import no.nav.pto.veilarbportefolje.tiltakshendelse.domain.Tiltakshendelse;
2828
import no.nav.pto.veilarbportefolje.tiltakshendelse.domain.Tiltakstype;
@@ -4263,11 +4263,10 @@ public void skal_hente_brukere_med_siste14avedtak() {
42634263
.setAktoer_id(brukerMedSiste14aVedtakAktorId.get())
42644264
.setEnhet_id(TEST_ENHET)
42654265
.setOppfolging(true)
4266-
.setSiste14aVedtak(new Siste14aVedtak(
4266+
.setGjeldendeVedtak14a(new GjeldendeVedtak14a(
42674267
siste14aVedtakForBruker.getInnsatsgruppe(),
42684268
siste14aVedtakForBruker.getHovedmal(),
4269-
siste14aVedtakForBruker.getFattetDato(),
4270-
siste14aVedtakForBruker.isFraArena()
4269+
siste14aVedtakForBruker.getFattetDato()
42714270
)),
42724271
new OppfolgingsBruker()
42734272
.setFnr(brukerUtenSiste14aVedtakFnr.get())
@@ -4283,20 +4282,19 @@ public void skal_hente_brukere_med_siste14avedtak() {
42834282
empty(),
42844283
"ascending",
42854284
"ikke_satt",
4286-
new Filtervalg().setFerdigfilterListe(emptyList()).setSiste14aVedtak(List.of("HAR_14A_VEDTAK")),
4285+
new Filtervalg().setFerdigfilterListe(emptyList()).setGjeldendeVedtak14a(List.of("HAR_14A_VEDTAK")),
42874286
null,
42884287
null
42894288
);
42904289
assertThat(respons.getAntall()).isEqualTo(1);
42914290
Bruker brukerFraOpenSearch = respons.getBrukere().getFirst();
42924291
assertThat(brukerFraOpenSearch.getFnr()).isEqualTo(brukerMedSiste14aVedtakFnr.get());
42934292
assertThat(brukerFraOpenSearch.getAktoerid()).isEqualTo(brukerMedSiste14aVedtakAktorId.get());
4294-
Siste14aVedtak brukerFraOpenSearchSiste14aVedtak = brukerFraOpenSearch.getSiste14aVedtak();
4295-
assertThat(brukerFraOpenSearchSiste14aVedtak).isNotNull();
4296-
assertThat(brukerFraOpenSearchSiste14aVedtak.innsatsgruppe()).isEqualTo(innsatsgruppe);
4297-
assertThat(brukerFraOpenSearchSiste14aVedtak.hovedmal()).isEqualTo(hovedmal);
4298-
assertThat(brukerFraOpenSearchSiste14aVedtak.fattetDato()).isEqualTo(fattetDato.toOffsetDateTime().toZonedDateTime());
4299-
assertThat(brukerFraOpenSearchSiste14aVedtak.fraArena()).isEqualTo(fraArena);
4293+
GjeldendeVedtak14a brukerFraOpenSearchGjeldendeVedtak14a = brukerFraOpenSearch.getGjeldendeVedtak14a();
4294+
assertThat(brukerFraOpenSearchGjeldendeVedtak14a).isNotNull();
4295+
assertThat(brukerFraOpenSearchGjeldendeVedtak14a.innsatsgruppe()).isEqualTo(innsatsgruppe);
4296+
assertThat(brukerFraOpenSearchGjeldendeVedtak14a.hovedmal()).isEqualTo(hovedmal);
4297+
assertThat(brukerFraOpenSearchGjeldendeVedtak14a.fattetDato()).isEqualTo(fattetDato.toOffsetDateTime().toZonedDateTime());
43004298
}
43014299

43024300
@Test
@@ -4323,11 +4321,10 @@ public void skal_hente_brukere_uten_siste14avedtak() {
43234321
.setAktoer_id(brukerMedSiste14aVedtakAktorId.get())
43244322
.setEnhet_id(TEST_ENHET)
43254323
.setOppfolging(true)
4326-
.setSiste14aVedtak(new Siste14aVedtak(
4324+
.setGjeldendeVedtak14a(new GjeldendeVedtak14a(
43274325
siste14aVedtakForBruker.getInnsatsgruppe(),
43284326
siste14aVedtakForBruker.getHovedmal(),
4329-
siste14aVedtakForBruker.getFattetDato(),
4330-
siste14aVedtakForBruker.isFraArena()
4327+
siste14aVedtakForBruker.getFattetDato()
43314328
)),
43324329
new OppfolgingsBruker()
43334330
.setFnr(brukerUtenSiste14aVedtakFnr.get())
@@ -4343,16 +4340,16 @@ public void skal_hente_brukere_uten_siste14avedtak() {
43434340
empty(),
43444341
"ascending",
43454342
"ikke_satt",
4346-
new Filtervalg().setFerdigfilterListe(emptyList()).setSiste14aVedtak(List.of("HAR_IKKE_14A_VEDTAK")),
4343+
new Filtervalg().setFerdigfilterListe(emptyList()).setGjeldendeVedtak14a(List.of("HAR_IKKE_14A_VEDTAK")),
43474344
null,
43484345
null
43494346
);
43504347
assertThat(respons.getAntall()).isEqualTo(1);
43514348
Bruker brukerFraOpenSearch = respons.getBrukere().getFirst();
43524349
assertThat(brukerFraOpenSearch.getFnr()).isEqualTo(brukerUtenSiste14aVedtakFnr.get());
43534350
assertThat(brukerFraOpenSearch.getAktoerid()).isEqualTo(brukerUtenSiste14aVedtakAktorId.get());
4354-
Siste14aVedtak brukerFraOpenSearchSiste14aVedtak = brukerFraOpenSearch.getSiste14aVedtak();
4355-
assertThat(brukerFraOpenSearchSiste14aVedtak).isNull();
4351+
GjeldendeVedtak14a brukerFraOpenSearchGjeldendeVedtak14a = brukerFraOpenSearch.getGjeldendeVedtak14a();
4352+
assertThat(brukerFraOpenSearchGjeldendeVedtak14a).isNull();
43564353
}
43574354

43584355
private boolean veilederExistsInResponse(String veilederId, BrukereMedAntall brukere) {

‎src/test/java/no/nav/pto/veilarbportefolje/oppfolging/OppfolgingStartetOgAvsluttetServiceTest.java

+8-9
Original file line numberDiff line numberDiff line change
@@ -169,21 +169,20 @@ public void cleanup() {
169169
oppfolgingPeriodeService.behandleKafkaMeldingLogikk(genererStartetOppfolgingsperiode(aktorId));
170170

171171
IdenterForBruker identerForBruker = pdlIdentRepository.hentIdenterForBruker(aktorId.get());
172-
Optional<Siste14aVedtakForBruker> siste14aVedtak = siste14aVedtakRepository.hentSiste14aVedtak(identerForBruker);
173-
assertThat(siste14aVedtak).isNotEmpty();
174-
assertThat(siste14aVedtak).isEqualTo(Optional.of(Siste14aVedtakForBruker.fraApiDto(siste14aVedtakApiDto, aktorId)));
172+
Optional<Siste14aVedtakForBruker> siste14aVedtakForBruker = siste14aVedtakRepository.hentSiste14aVedtak(identerForBruker);
173+
assertThat(siste14aVedtakForBruker).isNotEmpty();
174+
assertThat(siste14aVedtakForBruker).isEqualTo(Optional.of(Siste14aVedtakForBruker.fraApiDto(siste14aVedtakApiDto, aktorId)));
175175

176-
Siste14aVedtak siste14aVedtakFraOpenSearch = opensearchTestClient.hentBrukerFraOpensearch(aktorId).getSiste14aVedtak();
177-
Siste14aVedtak forventetSiste14aVedtak = new Siste14aVedtak(
176+
GjeldendeVedtak14a gjeldendeVedtak14aFraOpenSearch = opensearchTestClient.hentBrukerFraOpensearch(aktorId).getGjeldendeVedtak14a();
177+
GjeldendeVedtak14a forventetGjeldendeVedtak14a = new GjeldendeVedtak14a(
178178
siste14aVedtakApiDto.getInnsatsgruppe(),
179179
siste14aVedtakApiDto.getHovedmal(),
180180
// Vi må kvitte oss med ZoneId siden dates lagret i OpenSearch ikke får med dette (kun tidssone).
181181
// Derfor gjør vi denne toOffsetDataTime().toZonedDateTime() "hacken".
182-
siste14aVedtakApiDto.getFattetDato().toOffsetDateTime().toZonedDateTime(),
183-
siste14aVedtakApiDto.isFraArena()
182+
siste14aVedtakApiDto.getFattetDato().toOffsetDateTime().toZonedDateTime()
184183
);
185-
assertThat(siste14aVedtakFraOpenSearch).isNotNull();
186-
assertThat(siste14aVedtakFraOpenSearch).isEqualTo(forventetSiste14aVedtak);
184+
assertThat(gjeldendeVedtak14aFraOpenSearch).isNotNull();
185+
assertThat(gjeldendeVedtak14aFraOpenSearch).isEqualTo(forventetGjeldendeVedtak14a);
187186
}
188187

189188
@Test

0 commit comments

Comments
 (0)