Skip to content

Commit efe4066

Browse files
committedSep 11, 2024··
Legg til sortering på tiltakshendelse-tekst også
når eg fyrst er i gong

File tree

4 files changed

+49
-18
lines changed

4 files changed

+49
-18
lines changed
 

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

+2-1
Original file line numberDiff line numberDiff line change
@@ -434,7 +434,7 @@ static SearchSourceBuilder sorterQueryParametere(String sortOrder, String
434434
sorterTiltakshendelseOpprettetDato(searchSourceBuilder, order);
435435
return searchSourceBuilder;
436436
}
437-
if ("ikke_satt".equals(sortField) ) {
437+
if ("ikke_satt".equals(sortField)) {
438438
searchSourceBuilder.sort("aktoer_id", SortOrder.ASC);
439439
return searchSourceBuilder;
440440
}
@@ -480,6 +480,7 @@ static SearchSourceBuilder sorterQueryParametere(String sortOrder, String
480480
case "huskelapp_kommentar" -> searchSourceBuilder.sort("huskelapp.kommentar", order);
481481
case "fargekategori" -> searchSourceBuilder.sort("fargekategori", order);
482482
case "tiltakshendelse_dato_opprettet" -> sorterTiltakshendelseOpprettetDato(searchSourceBuilder, order);
483+
case "tiltakshendelse_tekst" -> searchSourceBuilder.sort("tiltakshendelse.tekst", order);
483484
default -> defaultSort(sortField, searchSourceBuilder, order);
484485
}
485486
addSecondarySort(searchSourceBuilder);

‎src/main/java/no/nav/pto/veilarbportefolje/util/ValideringsRegler.java

+2-1
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,8 @@ public class ValideringsRegler {
7777
"huskelapp",
7878
"fargekategori",
7979
"utdanningOgSituasjonSistEndret",
80-
"tiltakshendelse_dato_opprettet"
80+
"tiltakshendelse_dato_opprettet",
81+
"tiltakshendelse_tekst"
8182
);
8283

8384
public static void sjekkEnhet(String enhet) {

‎src/main/resources/opensearch_settings.json

+10
Original file line numberDiff line numberDiff line change
@@ -259,6 +259,16 @@
259259
"type": "keyword"
260260
}
261261
}
262+
},
263+
"tiltakshendelse": {
264+
"properties": {
265+
"opprettet": {
266+
"type": "date"
267+
},
268+
"tekst": {
269+
"type": "keyword"
270+
}
271+
}
262272
}
263273
}
264274
}

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

+35-16
Original file line numberDiff line numberDiff line change
@@ -3233,6 +3233,7 @@ public void test_filtrering_og_statustall_tiltakshendelser() {
32333233
.setEnhet_id(TEST_ENHET)
32343234
.setTiltakshendelse(null);
32353235

3236+
32363237
Fnr bruker2Fnr = Fnr.of("02020222222");
32373238
UUID bruker2UUID = UUID.randomUUID();
32383239
LocalDateTime bruker2Opprettet = LocalDateTime.now();
@@ -3249,6 +3250,7 @@ public void test_filtrering_og_statustall_tiltakshendelser() {
32493250
.setEnhet_id(TEST_ENHET)
32503251
.setTiltakshendelse(new Tiltakshendelse(bruker2UUID, bruker2Opprettet, bruker2Tekst, bruker2Lenke, bruker2Tiltakstype, bruker2Fnr));
32513252

3253+
32523254
Fnr bruker3Fnr = Fnr.of("03030333333");
32533255
UUID bruker3UUID = UUID.randomUUID();
32543256
LocalDateTime bruker3Opprettet = LocalDateTime.now();
@@ -3265,13 +3267,12 @@ public void test_filtrering_og_statustall_tiltakshendelser() {
32653267
.setEnhet_id(TEST_ENHET)
32663268
.setTiltakshendelse(new Tiltakshendelse(bruker3UUID, bruker3Opprettet, bruker3Tekst, bruker3Lenke, bruker3Tiltakstype, bruker3Fnr));
32673269

3270+
32683271
List<OppfolgingsBruker> brukere = List.of(bruker1, bruker2, bruker3);
32693272

32703273
skrivBrukereTilTestindeks(brukere);
3271-
32723274
pollOpensearchUntil(() -> opensearchTestClient.countDocuments() == brukere.size());
32733275

3274-
32753276
Filtervalg filterValg = new Filtervalg()
32763277
.setFerdigfilterListe(List.of(TILTAKSHENDELSER));
32773278

@@ -3295,11 +3296,11 @@ public void test_filtrering_og_statustall_tiltakshendelser() {
32953296
}
32963297

32973298
@Test
3298-
public void test_sortering_tiltakshendelser() {
3299+
public void test_sortering_tiltakshendelser_opprettet() {
32993300
Fnr bruker1Fnr = Fnr.of("01010111111");
33003301
UUID bruker1UUID = UUID.randomUUID();
33013302
LocalDateTime bruker1Opprettet = LocalDateTime.of(2024, 06, 01, 0, 0);
3302-
String tekst = "Forslag: Endre alt";
3303+
String bruker1tekst = "Dette er noko tekst som startar på D.";
33033304
String lenke = "http.cat/200";
33043305
Tiltakstype tiltakstype = Tiltakstype.ARBFORB;
33053306

@@ -3310,12 +3311,13 @@ public void test_sortering_tiltakshendelser() {
33103311
.setVeileder_id(TEST_VEILEDER_0)
33113312
.setNy_for_veileder(false)
33123313
.setEnhet_id(TEST_ENHET)
3313-
.setTiltakshendelse(new Tiltakshendelse(bruker1UUID, bruker1Opprettet, tekst, lenke, tiltakstype, bruker1Fnr));
3314+
.setTiltakshendelse(new Tiltakshendelse(bruker1UUID, bruker1Opprettet, bruker1tekst, lenke, tiltakstype, bruker1Fnr));
3315+
33143316

33153317
Fnr bruker2Fnr = Fnr.of("02020222222");
33163318
UUID bruker2UUID = UUID.randomUUID();
33173319
LocalDateTime bruker2Opprettet = LocalDateTime.of(2023, 06, 01, 0, 0);
3318-
3320+
String bruker2Tekst = "Akkurat slik startar du ein setning med bokstaven A.";
33193321

33203322
OppfolgingsBruker bruker2 = new OppfolgingsBruker()
33213323
.setFnr(bruker2Fnr.toString())
@@ -3324,11 +3326,13 @@ public void test_sortering_tiltakshendelser() {
33243326
.setVeileder_id(TEST_VEILEDER_0)
33253327
.setNy_for_veileder(false)
33263328
.setEnhet_id(TEST_ENHET)
3327-
.setTiltakshendelse(new Tiltakshendelse(bruker2UUID, bruker2Opprettet, tekst, lenke, tiltakstype, bruker2Fnr));
3329+
.setTiltakshendelse(new Tiltakshendelse(bruker2UUID, bruker2Opprettet, bruker2Tekst, lenke, tiltakstype, bruker2Fnr));
3330+
33283331

33293332
Fnr bruker3Fnr = Fnr.of("03030333333");
33303333
UUID bruker3UUID = UUID.randomUUID();
33313334
LocalDateTime bruker3Opprettet = LocalDateTime.of(2022, 06, 01, 0, 0);
3335+
String bruker3Tekst = "Byrjinga av denne teksten er bokstaven B.";
33323336

33333337
OppfolgingsBruker bruker3 = new OppfolgingsBruker()
33343338
.setFnr(bruker3Fnr.toString())
@@ -3337,15 +3341,14 @@ public void test_sortering_tiltakshendelser() {
33373341
.setVeileder_id(TEST_VEILEDER_0)
33383342
.setNy_for_veileder(false)
33393343
.setEnhet_id(TEST_ENHET)
3340-
.setTiltakshendelse(new Tiltakshendelse(bruker3UUID, bruker3Opprettet, tekst, lenke, tiltakstype, bruker3Fnr));
3344+
.setTiltakshendelse(new Tiltakshendelse(bruker3UUID, bruker3Opprettet, bruker3Tekst, lenke, tiltakstype, bruker3Fnr));
3345+
33413346

33423347
List<OppfolgingsBruker> brukere = List.of(bruker1, bruker2, bruker3);
33433348

33443349
skrivBrukereTilTestindeks(brukere);
3345-
33463350
pollOpensearchUntil(() -> opensearchTestClient.countDocuments() == brukere.size());
33473351

3348-
33493352
Filtervalg filterValg = new Filtervalg()
33503353
.setFerdigfilterListe(List.of(TILTAKSHENDELSER));
33513354

@@ -3366,7 +3369,7 @@ public void test_sortering_tiltakshendelser() {
33663369
assertThat(brukereDefaultRekkefolge.get(2).getFnr()).isEqualTo(bruker1Fnr.toString());
33673370

33683371

3369-
BrukereMedAntall responseEksplisittSortering = opensearchService.hentBrukere(
3372+
BrukereMedAntall responseSortertNyesteDato = opensearchService.hentBrukere(
33703373
TEST_ENHET,
33713374
empty(),
33723375
"descending",
@@ -3375,12 +3378,28 @@ public void test_sortering_tiltakshendelser() {
33753378
null,
33763379
null
33773380
);
3378-
List<Bruker> brukereEksplisittSortert = responseEksplisittSortering.getBrukere();
3381+
List<Bruker> brukereOpprettetSortertPaNyeste = responseSortertNyesteDato.getBrukere();
3382+
3383+
assertThat(responseSortertNyesteDato.getAntall()).isEqualTo(3);
3384+
assertThat(brukereOpprettetSortertPaNyeste.get(0).getFnr()).isEqualTo(bruker1Fnr.toString());
3385+
assertThat(brukereOpprettetSortertPaNyeste.get(1).getFnr()).isEqualTo(bruker2Fnr.toString());
3386+
assertThat(brukereOpprettetSortertPaNyeste.get(2).getFnr()).isEqualTo(bruker3Fnr.toString());
3387+
3388+
BrukereMedAntall responseSortertAlfabetisk = opensearchService.hentBrukere(
3389+
TEST_ENHET,
3390+
empty(),
3391+
"ascending",
3392+
"tiltakshendelse_tekst",
3393+
filterValg,
3394+
null,
3395+
null
3396+
);
3397+
List<Bruker> brukereTekstSortertAlfabetisk = responseSortertAlfabetisk.getBrukere();
33793398

3380-
assertThat(responseEksplisittSortering.getAntall()).isEqualTo(3);
3381-
assertThat(brukereEksplisittSortert.get(0).getFnr()).isEqualTo(bruker1Fnr.toString());
3382-
assertThat(brukereEksplisittSortert.get(1).getFnr()).isEqualTo(bruker2Fnr.toString());
3383-
assertThat(brukereEksplisittSortert.get(2).getFnr()).isEqualTo(bruker3Fnr.toString());
3399+
assertThat(responseSortertAlfabetisk.getAntall()).isEqualTo(3);
3400+
assertThat(brukereTekstSortertAlfabetisk.get(0).getFnr()).isEqualTo(bruker2Fnr.toString());
3401+
assertThat(brukereTekstSortertAlfabetisk.get(1).getFnr()).isEqualTo(bruker3Fnr.toString());
3402+
assertThat(brukereTekstSortertAlfabetisk.get(2).getFnr()).isEqualTo(bruker1Fnr.toString());
33843403
}
33853404

33863405
@Test

0 commit comments

Comments
 (0)
Please sign in to comment.