Skip to content

Commit 020f06e

Browse files
espenwaagaespenwaaga
authored
TFP-5554: Venting på historikkinslag basere seg nå på tittel/skjermle… (#1477)
* TFP-5554: Venting på historikkinslag basere seg nå på tittel/skjermlenke og ikke type * HistorikkType ikke typer
1 parent 3578176 commit 020f06e

File tree

16 files changed

+118
-210
lines changed

16 files changed

+118
-210
lines changed

‎src/main/java/no/nav/foreldrepenger/autotest/aktoerer/innsender/DokumentInnsendingHjelper.java‎

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,6 @@
11
package no.nav.foreldrepenger.autotest.aktoerer.innsender;
22

33
import static no.nav.foreldrepenger.autotest.klienter.fpsak.historikk.dto.HistorikkAktør.ARBEIDSGIVER;
4-
import static no.nav.foreldrepenger.autotest.klienter.fpsak.historikk.dto.HistorikkAktør.SØKER;
5-
import static no.nav.foreldrepenger.autotest.klienter.fpsak.historikk.dto.HistorikkinnslagType.BEH_STARTET;
6-
import static no.nav.foreldrepenger.autotest.klienter.fpsak.historikk.dto.HistorikkinnslagType.VEDLEGG_MOTTATT;
74

85
import java.time.LocalDateTime;
96
import java.time.temporal.ChronoUnit;
@@ -15,6 +12,7 @@
1512
import no.nav.foreldrepenger.autotest.klienter.fpsak.fagsak.dto.Fagsak;
1613
import no.nav.foreldrepenger.autotest.klienter.fpsak.historikk.HistorikkFpsakKlient;
1714
import no.nav.foreldrepenger.autotest.klienter.fpsak.historikk.dto.HistorikkInnslag;
15+
import no.nav.foreldrepenger.autotest.klienter.fpsak.historikk.dto.HistorikkType;
1816
import no.nav.foreldrepenger.autotest.klienter.vtp.journalpost.JournalforingKlient;
1917
import no.nav.foreldrepenger.autotest.klienter.vtp.sikkerhet.azure.SaksbehandlerRolle;
2018
import no.nav.foreldrepenger.autotest.util.vent.Vent;
@@ -65,8 +63,7 @@ protected Saksnummer ventTilAlleInntekstmeldingeneErMottatt(Fødselsnummer fnr,
6563
return 0;
6664
}
6765
return (int) historikkKlient.hentHistorikk(saksnummer).stream()
68-
.filter(h -> VEDLEGG_MOTTATT.equals(h.type()))
69-
.filter(h -> ARBEIDSGIVER.equals(h.aktoer()))
66+
.filter(h -> h.erAvTypen(ARBEIDSGIVER, HistorikkType.VEDLEGG_MOTTATT))
7067
.count();
7168
}
7269

@@ -89,7 +86,7 @@ protected Saksnummer ventTilFagsakOgBehandlingErOpprettet(Fødselsnummer fnr, Lo
8986
for (var fagsak : fagsakKlient.søk(fnr)) {
9087
var saksnummer = fagsak.saksnummer();
9188
for (var h : historikkinnslagEtterSkjæringstidspunkt(saksnummer, skjæringstidpunktRundetNed)) {
92-
if (erSøknadMottatt(h)) {
89+
if (h.erSøknadMottatt()) {
9390
return saksnummer;
9491
}
9592
}
@@ -129,9 +126,7 @@ protected Saksnummer ventTilFagsakOgBehandlingErOpprettet(Fødselsnummer fnr, Lo
129126
.toList();
130127
}
131128

132-
private boolean erSøknadMottatt(HistorikkInnslag h) {
133-
return BEH_STARTET.equals(h.type()) && SØKER.equals(h.aktoer());
134-
}
129+
135130

136131
protected int antallEksistrendeFagsakerPåSøker(Fødselsnummer fnr) {
137132
return fagsakKlient.søk(fnr).size();

‎src/main/java/no/nav/foreldrepenger/autotest/aktoerer/saksbehandler/fpsak/Saksbehandler.java‎

Lines changed: 18 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77

88
import java.time.LocalDate;
99
import java.util.ArrayList;
10+
import java.util.Arrays;
1011
import java.util.Comparator;
1112
import java.util.List;
1213
import java.util.Objects;
@@ -54,7 +55,7 @@
5455
import no.nav.foreldrepenger.autotest.klienter.fpsak.fagsak.dto.FagsakStatus;
5556
import no.nav.foreldrepenger.autotest.klienter.fpsak.historikk.HistorikkFpsakKlient;
5657
import no.nav.foreldrepenger.autotest.klienter.fpsak.historikk.dto.HistorikkInnslag;
57-
import no.nav.foreldrepenger.autotest.klienter.fpsak.historikk.dto.HistorikkinnslagType;
58+
import no.nav.foreldrepenger.autotest.klienter.fpsak.historikk.dto.HistorikkType;
5859
import no.nav.foreldrepenger.autotest.klienter.fpsak.prosesstask.ProsesstaskFpsakKlient;
5960
import no.nav.foreldrepenger.autotest.klienter.vtp.saf.SafKlient;
6061
import no.nav.foreldrepenger.autotest.klienter.vtp.sikkerhet.azure.SaksbehandlerRolle;
@@ -68,8 +69,6 @@
6869
public class Saksbehandler {
6970
private final Logger LOG = LoggerFactory.getLogger(Saksbehandler.class);
7071

71-
private static final Set<HistorikkinnslagType> GJENOPPTATT = Set.of(HistorikkinnslagType.BEH_GJEN, HistorikkinnslagType.BEH_MAN_GJEN);
72-
7372
public Fagsak valgtFagsak;
7473
public Behandling valgtBehandling;
7574
public List<Behandling> behandlinger;
@@ -375,19 +374,19 @@ private void populateBehandling(Behandling behandling) {
375374
@Step("Setter behandling på vent")
376375
public void settBehandlingPåVent(LocalDate frist, Venteårsak årsak) {
377376
behandlingerKlient.settPaVent(new SettBehandlingPaVentDto(valgtBehandling, frist, årsak));
378-
ventTilHistorikkinnslag(HistorikkinnslagType.BEH_VENT);
377+
ventTilHistorikkinnslag(HistorikkType.BEH_VENT);
379378
}
380379

381380
@Step("Gjenopptar Behandling")
382381
public void gjenopptaBehandling() {
383382
valgtBehandling = behandlingerKlient.gjenoppta(new BehandlingIdVersjonDto(valgtBehandling));
384-
ventTilHistorikkinnslag(GJENOPPTATT);
383+
ventTilHistorikkinnslag(HistorikkType.BEH_GJEN, HistorikkType.BEH_MAN_GJEN);
385384
}
386385

387386
@Step("Henlegger behandling")
388387
public void henleggBehandling(BehandlingResultatType årsak) {
389388
behandlingerKlient.henlegg(new BehandlingHenlegg(valgtBehandling.uuid, valgtBehandling.versjon, årsak, "Henlagt"));
390-
ventTilHistorikkinnslag(HistorikkinnslagType.AVBRUTT_BEH);
389+
ventTilHistorikkinnslag(HistorikkType.AVBRUTT_BEH);
391390
}
392391

393392
/*
@@ -553,8 +552,9 @@ public void ventTilAvsluttetBehandling() {
553552
* 2) Behandlingen er ikke tatt av vent enda og vi venter på at behandlingen GJENOPPRETTET
554553
* Venter da til den er gjenopprettet, for så og vente på potensiell prosessering.
555554
*/
556-
if (hentHistorikkinnslagPåBehandling().stream().anyMatch(h -> h.type().equals(HistorikkinnslagType.BEH_VENT))) {
557-
Vent.på(() -> hentHistorikkinnslagPåBehandling().stream().anyMatch(h -> GJENOPPTATT.contains(h.type())),
555+
if (hentHistorikkinnslagPåBehandling().stream().anyMatch(h -> h.erAvTypen(HistorikkType.BEH_VENT))) {
556+
Vent.på(() -> hentHistorikkinnslagPåBehandling().stream().anyMatch(h -> h.erAvTypen(
557+
HistorikkType.BEH_GJEN, HistorikkType.BEH_MAN_GJEN)),
558558
"Behandlingen er på vent og er ikke blitt gjenopptatt!");
559559
}
560560

@@ -591,49 +591,39 @@ public void ventTilAvsluttetBehandlingOgDetOpprettesTilbakekreving() {
591591
.toList();
592592
}
593593

594-
public boolean harHistorikkinnslagPåBehandling(HistorikkinnslagType type) {
595-
return harHistorikkinnslagPåBehandling(Set.of(type), valgtBehandling.uuid);
596-
}
597-
598-
public boolean harHistorikkinnslagPåBehandling(Set<HistorikkinnslagType> type) {
599-
return harHistorikkinnslagPåBehandling(type, valgtBehandling.uuid);
594+
public boolean harHistorikkinnslagPåBehandling(HistorikkType... type) {
595+
return harHistorikkinnslagPåBehandling(valgtBehandling.uuid, type);
600596
}
601597

602-
private boolean harHistorikkinnslagPåBehandling(Set<HistorikkinnslagType> type, UUID behandlingsId) {
603-
if (type.contains(HistorikkinnslagType.VEDLEGG_MOTTATT) || type.contains(HistorikkinnslagType.REVURD_OPPR)) {
598+
private boolean harHistorikkinnslagPåBehandling(UUID behandlingsId, HistorikkType... type) {
599+
if (Arrays.stream(type).anyMatch(t -> Set.of(HistorikkType.VEDLEGG_MOTTATT, HistorikkType.REVURD_OPPR).contains(t))) {
604600
behandlingsId = null;
605601
}
606602
return hentHistorikkinnslagPåBehandling(behandlingsId).stream()
607-
.anyMatch(innslag -> type.contains(innslag.type()));
603+
.anyMatch(innslag -> innslag.erAvTypen(type));
608604
}
609605

610-
public HistorikkInnslag hentHistorikkinnslagAvType(HistorikkinnslagType type) {
606+
public HistorikkInnslag hentHistorikkinnslagAvType(HistorikkType type) {
611607
return hentHistorikkinnslagAvType(type, valgtBehandling.uuid, valgtBehandling.id);
612608
}
613609

614-
public HistorikkInnslag hentHistorikkinnslagAvType(HistorikkinnslagType type, UUID behandlingsUuid, int behandlingId) {
610+
public HistorikkInnslag hentHistorikkinnslagAvType(HistorikkType type, UUID behandlingsUuid, int behandlingId) {
615611
venterPåFerdigProssesseringOgOppdaterBehandling(behandlingsUuid, behandlingId);
616-
if (List.of(HistorikkinnslagType.VEDLEGG_MOTTATT, HistorikkinnslagType.REVURD_OPPR).contains(type)) {
612+
if (Set.of(HistorikkType.VEDLEGG_MOTTATT, HistorikkType.REVURD_OPPR).contains(type)) {
617613
behandlingsUuid = null;
618614
}
619615
return hentHistorikkinnslagPåBehandling(behandlingsUuid).stream()
620-
.filter(innslag -> type.equals(innslag.type()))
616+
.filter(innslag -> innslag.erAvTypen(type))
621617
.findFirst()
622618
.orElse(null);
623619
}
624620

625-
public void ventTilHistorikkinnslag(HistorikkinnslagType type) {
621+
public void ventTilHistorikkinnslag(HistorikkType... type) {
626622
Vent.på(() -> harHistorikkinnslagPåBehandling(type),
627623
() -> "Saken hadde ikke historikkinslag " + type + "\nHistorikkInnslag:"
628624
+ String.join("\t\n", String.valueOf(hentHistorikkinnslagPåBehandling())));
629625
}
630626

631-
public void ventTilHistorikkinnslag(Set<HistorikkinnslagType> typer) {
632-
Vent.på(() -> harHistorikkinnslagPåBehandling(typer),
633-
() -> "Saken hadde ikke historikkinslag " + typer + "\nHistorikkInnslag:"
634-
+ String.join("\t\n", String.valueOf(hentHistorikkinnslagPåBehandling())));
635-
}
636-
637627
/*
638628
* Vilkar
639629
*/

‎src/main/java/no/nav/foreldrepenger/autotest/aktoerer/saksbehandler/fptilbake/TilbakekrevingSaksbehandler.java‎

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121
import no.nav.foreldrepenger.autotest.klienter.fpsak.behandlinger.dto.behandling.Aksjonspunkt;
2222
import no.nav.foreldrepenger.autotest.klienter.fpsak.behandlinger.dto.behandling.Behandling;
2323
import no.nav.foreldrepenger.autotest.klienter.fpsak.historikk.dto.HistorikkInnslag;
24-
import no.nav.foreldrepenger.autotest.klienter.fpsak.historikk.dto.HistorikkinnslagType;
24+
import no.nav.foreldrepenger.autotest.klienter.fpsak.historikk.dto.HistorikkType;
2525
import no.nav.foreldrepenger.autotest.klienter.fptilbake.behandlinger.BehandlingFptilbakeKlient;
2626
import no.nav.foreldrepenger.autotest.klienter.fptilbake.behandlinger.dto.BehandlingIdBasicDto;
2727
import no.nav.foreldrepenger.autotest.klienter.fptilbake.behandlinger.dto.BehandlingOpprett;
@@ -51,9 +51,6 @@ public class TilbakekrevingSaksbehandler {
5151

5252
private final Logger LOG = LoggerFactory.getLogger(TilbakekrevingSaksbehandler.class);
5353

54-
private static final Set<HistorikkinnslagType> GJENOPPTATT = Set.of(HistorikkinnslagType.BEH_GJEN, HistorikkinnslagType.BEH_MAN_GJEN);
55-
56-
public List<Behandling> behandlingList;
5754
public Behandling valgtBehandling;
5855
public Saksnummer saksnummer;
5956

@@ -287,8 +284,9 @@ public void ventTilAvsluttetBehandling() {
287284
* 2) Behandlingen er ikke tatt av vent enda og vi venter på at behandlingen GJENOPPRETTET
288285
* Venter da til den er gjenopprettet, for så og vente på potensiell prosessering.
289286
*/
290-
if (hentHistorikkinnslagPåBehandling().stream().anyMatch(h -> h.type().equals(HistorikkinnslagType.BEH_VENT))) {
291-
Vent.på(() -> hentHistorikkinnslagPåBehandling().stream().anyMatch(h -> GJENOPPTATT.contains(h.type())),
287+
if (hentHistorikkinnslagPåBehandling().stream().anyMatch(h -> h.erAvTypen(HistorikkType.BEH_VENT))) {
288+
Vent.på(() -> hentHistorikkinnslagPåBehandling().stream().anyMatch(h -> h.erAvTypen(
289+
HistorikkType.BEH_GJEN, HistorikkType.BEH_MAN_GJEN)),
292290
"Behandlingen er på vent og er ikke blitt gjenopptatt!");
293291
}
294292

‎src/main/java/no/nav/foreldrepenger/autotest/base/VerdikjedeTestBase.java‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
import no.nav.foreldrepenger.autotest.klienter.fpsak.behandlinger.dto.aksjonspunktbekreftelse.FatterVedtakBekreftelse;
44
import no.nav.foreldrepenger.autotest.klienter.fpsak.behandlinger.dto.aksjonspunktbekreftelse.ForeslåVedtakBekreftelse;
5-
import no.nav.foreldrepenger.autotest.klienter.fpsak.historikk.dto.HistorikkinnslagType;
5+
import no.nav.foreldrepenger.autotest.klienter.fpsak.historikk.dto.HistorikkType;
66
import no.nav.foreldrepenger.common.domain.Saksnummer;
77
import no.nav.foreldrepenger.kontrakter.risk.kodeverk.RisikoklasseType;
88

@@ -28,8 +28,8 @@ public abstract class VerdikjedeTestBase extends FpsakTestBase {
2828
} else {
2929
beslutter.fattVedtakOgVentTilAvsluttetBehandling(bekreftelse);
3030
}
31-
if (saksbehandler.harHistorikkinnslagPåBehandling(HistorikkinnslagType.BREV_BESTILT)) {
32-
saksbehandler.ventTilHistorikkinnslag(HistorikkinnslagType.BREV_SENT);
31+
if (saksbehandler.harHistorikkinnslagPåBehandling(HistorikkType.BREV_BESTILT)) {
32+
saksbehandler.ventTilHistorikkinnslag(HistorikkType.BREV_SENT);
3333
}
3434
}
3535

‎src/main/java/no/nav/foreldrepenger/autotest/klienter/fpsak/historikk/dto/HistorikkInnslag.java‎

Lines changed: 31 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,39 @@
11
package no.nav.foreldrepenger.autotest.klienter.fpsak.historikk.dto;
22

33
import java.time.LocalDateTime;
4+
import java.util.Arrays;
45
import java.util.List;
56
import java.util.UUID;
67

7-
import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
8+
public record HistorikkInnslag(UUID behandlingUuid,
9+
HistorikkAktørDto aktør,
10+
String skjermlenke,
11+
String tittel,
12+
LocalDateTime opprettetTidspunkt,
13+
List<HistorikkInnslagDokumentLinkDto> dokumenter,
14+
List<String> body) {
815

9-
@JsonIgnoreProperties(ignoreUnknown = true)
10-
public record HistorikkInnslag(
11-
UUID behandlingUuid,
12-
HistorikkinnslagType type,
13-
HistorikkAktør aktoer,
14-
String kjoenn,
15-
LocalDateTime opprettetTidspunkt,
16-
List<HistorikkInnslagDokumentLinkDto> dokumentLinks) {
16+
public record HistorikkAktørDto(HistorikkAktør type, String ident) {}
17+
18+
19+
public boolean erSøknadMottatt() {
20+
return erAvTypen(HistorikkType.BEH_STARTET) && HistorikkAktør.SØKER.equals(aktør.type);
21+
}
22+
23+
public boolean erAvTypen(HistorikkType... type) {
24+
return erAvTypen(null, type);
25+
}
26+
27+
public boolean erAvTypen(HistorikkAktør aktør, HistorikkType... type) {
28+
if (aktør != null && !aktør.equals(this.aktør().type)) {
29+
return false;
30+
}
31+
32+
return Arrays.stream(type).anyMatch(this::harTittelEllerSkjemlenkeTilsvarendeType);
33+
}
34+
35+
private boolean harTittelEllerSkjemlenkeTilsvarendeType(HistorikkType t) {
36+
return t.tittel().equals(tittel()) || (t.skjermlenke() != null && t.skjermlenke().equals(skjermlenke()));
37+
}
1738
}
39+

‎src/main/java/no/nav/foreldrepenger/autotest/klienter/fpsak/historikk/dto/HistorikkType.java‎

Lines changed: 36 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,36 @@
1+
package no.nav.foreldrepenger.autotest.klienter.fpsak.historikk.dto;
2+
3+
public enum HistorikkType {
4+
BREV_SENT("Brev sendt"),
5+
BREV_BESTILT("Brev bestilt"),
6+
BEH_STARTET("Behandling startet"),
7+
BEH_MAN_GJEN("Gjenoppta behandling"),
8+
BEH_GJEN("Behandling gjenopptatt"),
9+
BEH_VENT("Behandling på vent"),
10+
VEDLEGG_MOTTATT("Vedlegg mottatt"),
11+
REVURD_OPPR("Revurdering opprettet"),
12+
SPOLT_TILBAKE("Behandlingen er flyttet"),
13+
AVBRUTT_BEH("Behandling er henlagt"),
14+
VEDTAK_FATTET("Vedtak fattet", "VEDTAK");
15+
16+
private final String tittel;
17+
private final String skjermlenke;
18+
19+
HistorikkType(String tittel) {
20+
this(tittel, null);
21+
}
22+
23+
HistorikkType(String tittel, String skjermlenke) {
24+
this.tittel = tittel;
25+
this.skjermlenke = skjermlenke;
26+
}
27+
28+
public String tittel() {
29+
return tittel;
30+
}
31+
32+
public String skjermlenke() {
33+
return skjermlenke;
34+
}
35+
36+
}

‎src/main/java/no/nav/foreldrepenger/autotest/klienter/fpsak/historikk/dto/HistorikkinnslagType.java‎

Lines changed: 0 additions & 68 deletions
This file was deleted.

‎src/main/java/no/nav/foreldrepenger/autotest/util/AllureHelper.java‎

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111
import no.nav.foreldrepenger.autotest.klienter.fpsak.behandlinger.dto.aksjonspunktbekreftelse.AksjonspunktBekreftelse;
1212
import no.nav.foreldrepenger.autotest.klienter.fpsak.behandlinger.dto.behandling.Aksjonspunkt;
1313
import no.nav.foreldrepenger.autotest.klienter.fpsak.behandlinger.dto.behandling.Behandling;
14-
import no.nav.foreldrepenger.autotest.klienter.fpsak.historikk.dto.HistorikkInnslag;
1514

1615
public final class AllureHelper {
1716

@@ -45,15 +44,6 @@ public static void debugFritekst(String fritekst) {
4544
// Dette legger til et punkt i allure rapporten som matcher det som står i @Step
4645
}
4746

48-
public static void debugLoggHistorikkinnslag(List<HistorikkInnslag> historikkInnslagList) {
49-
StringBuilder sb = new StringBuilder();
50-
sb.append("Historikkinnslag\n");
51-
for (HistorikkInnslag historikkInnslag : historikkInnslagList) {
52-
sb.append("\t").append("{").append(historikkInnslag.type()).append("}");
53-
}
54-
loggHistorikkinnslag(sb.toString());
55-
}
56-
5747
@Step("Informasjon om historikkinnslag: {historikkinnslag}")
5848
private static void loggHistorikkinnslag(String historikkinnslag) {
5949
// Dette legger til et punkt i allure rapporten som matcher det som står i @Step

0 commit comments

Comments
 (0)