Skip to content

Commit 379e865

Browse files
committed
timeout justeringer
1 parent 7d619fa commit 379e865

File tree

3 files changed

+23
-18
lines changed

3 files changed

+23
-18
lines changed

src/main/java/no/nav/foreldrepenger/autotest/aktoerer/innsyn/Innsyn.java

Lines changed: 12 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,8 @@
1515

1616
public class Innsyn {
1717
private static final Logger LOG = LoggerFactory.getLogger(Innsyn.class);
18-
private static final int PROGRESSIV_VENTETID = 3000;
18+
private static final int API_PROGRESSIV_VENTETID = 3000;
19+
private static final int API_TIMEOUT_SEKUNDER = 30;
1920

2021
private final InnsynKlient innsynKlient;
2122
private final Fødselsnummer fnr;
@@ -41,8 +42,8 @@ private Saker hentSaker() {
4142
return null;
4243
}
4344
return sak.get();
44-
}, () -> "Finner ikke sak med tilstand " + behandlingTilstand + " for sak " + saksnummer, PROGRESSIV_VENTETID,
45-
"åpen es behandling med tilstand " + behandlingTilstand);
45+
}, () -> "Finner ikke sak med tilstand " + behandlingTilstand + " for sak " + saksnummer, API_PROGRESSIV_VENTETID,
46+
"åpen es behandling med tilstand " + behandlingTilstand, API_TIMEOUT_SEKUNDER);
4647
}
4748

4849
public SvpSak hentSvpSakMedÅpenBehandlingTilstand(Saksnummer saksnummer, BehandlingTilstand behandlingTilstand) {
@@ -56,8 +57,8 @@ private Saker hentSaker() {
5657
return null;
5758
}
5859
return sak.get();
59-
}, () -> "Finner ikke sak med tilstand " + behandlingTilstand + " for sak " + saksnummer, PROGRESSIV_VENTETID,
60-
"åpen svp behandling med tilstand " + behandlingTilstand);
60+
}, () -> "Finner ikke sak med tilstand " + behandlingTilstand + " for sak " + saksnummer, API_PROGRESSIV_VENTETID,
61+
"åpen svp behandling med tilstand " + behandlingTilstand, API_TIMEOUT_SEKUNDER);
6162
}
6263

6364
public EsSak hentEsSakUtenÅpenBehandling(Saksnummer saksnummer) {
@@ -71,8 +72,8 @@ private Saker hentSaker() {
7172
return null;
7273
}
7374
return sak.get();
74-
}, () -> "Finner ikke sak uten åpen behandling for sak " + saksnummer, PROGRESSIV_VENTETID,
75-
"es sak uten åpen behandling");
75+
}, () -> "Finner ikke sak uten åpen behandling for sak " + saksnummer, API_PROGRESSIV_VENTETID,
76+
"es sak uten åpen behandling", API_TIMEOUT_SEKUNDER);
7677
}
7778

7879
public SvpSak hentSvpSakUtenÅpenBehandling(Saksnummer saksnummer) {
@@ -86,8 +87,8 @@ private Saker hentSaker() {
8687
return null;
8788
}
8889
return sak.get();
89-
}, () -> "Finner ikke sak uten åpen behandling for sak " + saksnummer, PROGRESSIV_VENTETID,
90-
"svp sak uten åpen behandling");
90+
}, () -> "Finner ikke sak uten åpen behandling for sak " + saksnummer, API_PROGRESSIV_VENTETID,
91+
"svp sak uten åpen behandling", API_TIMEOUT_SEKUNDER);
9192
}
9293

9394
public FpSak hentFpSakUtenÅpenBehandling(Saksnummer saksnummer) {
@@ -101,7 +102,7 @@ private Saker hentSaker() {
101102
return null;
102103
}
103104
return sak.get();
104-
}, () -> "Finner ikke sak uten åpen behandling for sak " + saksnummer, PROGRESSIV_VENTETID,
105-
"fp sak uten åpen behandling");
105+
}, () -> "Finner ikke sak uten åpen behandling for sak " + saksnummer, API_PROGRESSIV_VENTETID,
106+
"fp sak uten åpen behandling", API_TIMEOUT_SEKUNDER);
106107
}
107108
}

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -345,7 +345,7 @@ private Behandling ventTilBehandlingErFerdigProsessertOgReturner(UUID behandling
345345
.append("\n");
346346
}
347347
return "Behandling status var ikke klar men har ikke feilet\n" + prosessTaskList;
348-
}, "behandling er ferdig prosessert");
348+
}, 50, "behandling er ferdig prosessert", 40);
349349
}
350350

351351
private List<ProsessTaskDataDto> hentProsesstaskerForBehandling(int behandlingId) {

src/main/java/no/nav/foreldrepenger/autotest/util/vent/Vent.java

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -29,14 +29,18 @@ public static void til(Callable<Boolean> callable, Callable<String> errorMessage
2929
}
3030

3131
public static <T> T (Callable<T> callable, Callable<String> errorMessageProducer, String loggForklaring) {
32-
return (callable, errorMessageProducer, 50, loggForklaring);
32+
return (callable, errorMessageProducer, 50, loggForklaring, TIMEOUT_SEKUNDER);
3333
}
3434

35-
public static <T> T (Callable<T> callable, Callable<String> errorMessageProducer, int progressivVentetid, String loggForklaring) {
35+
public static <T> T (Callable<T> callable,
36+
Callable<String> errorMessageProducer,
37+
int progressivVentetid,
38+
String loggForklaring,
39+
int timeoutSekunder) {
3640
LOG.info("Venter på {}", loggForklaring);
3741
var start = LocalDateTime.now();
38-
var end = start.plusSeconds(TIMEOUT_SEKUNDER);
39-
var advarsel = start.plusSeconds((int) (TIMEOUT_SEKUNDER * 0.75));
42+
var end = start.plusSeconds(timeoutSekunder);
43+
var advarsel = start.plusSeconds((int) (timeoutSekunder * 0.75));
4044
var logget = false;
4145

4246
long progressivVentetidMs = progressivVentetid;
@@ -48,11 +52,11 @@ public static void til(Callable<Boolean> callable, Callable<String> errorMessage
4852
if (!logget && now.isAfter(advarsel)) {
4953
logget = true;
5054
var ste = getCallerCallerClassName();
51-
LOG.warn("Async venting av {} har tatt mer enn 75% av timeout på {} sekunder!", ste, TIMEOUT_SEKUNDER);
55+
LOG.warn("Async venting av {} har tatt mer enn 75% av timeout på {} sekunder!", ste, timeoutSekunder);
5256
}
5357
if (now.isAfter(end)) {
5458
throw new IllegalStateException(
55-
"Async venting timet ut etter " + TIMEOUT_SEKUNDER + " sekunder fordi: " + errorMessageProducer.call());
59+
"Async venting timet ut etter " + timeoutSekunder + " sekunder fordi: " + errorMessageProducer.call());
5660
}
5761
sleep(progressivVentetidMs);
5862
progressivVentetidMs = Math.min(750, 2 * progressivVentetidMs);

0 commit comments

Comments
 (0)