Endringer rundt aksjonspunkt

Author: palfi

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

@@ -5,7 +5,6 @@
 import static no.nav.foreldrepenger.autotest.util.AllureHelper.debugAksjonspunktbekreftelser;
 import static no.nav.foreldrepenger.autotest.util.AllureHelper.debugBehandlingsstatus;
 
-import java.lang.reflect.InvocationTargetException;
 import java.time.LocalDate;
 import java.util.ArrayList;
 import java.util.Arrays;
@@ -39,7 +38,6 @@
 import no.nav.foreldrepenger.autotest.klienter.fpsak.behandlinger.dto.SettBehandlingPaVentDto;
 import no.nav.foreldrepenger.autotest.klienter.fpsak.behandlinger.dto.aksjonspunktbekreftelse.AksjonspunktBekreftelse;
 import no.nav.foreldrepenger.autotest.klienter.fpsak.behandlinger.dto.aksjonspunktbekreftelse.BekreftedeAksjonspunkter;
-import no.nav.foreldrepenger.autotest.klienter.fpsak.behandlinger.dto.aksjonspunktbekreftelse.BekreftelseKode;
 import no.nav.foreldrepenger.autotest.klienter.fpsak.behandlinger.dto.aksjonspunktbekreftelse.FatterVedtakBekreftelse;
 import no.nav.foreldrepenger.autotest.klienter.fpsak.behandlinger.dto.aksjonspunktbekreftelse.avklarfakta.ArbeidInntektsmeldingBekreftelse;
 import no.nav.foreldrepenger.autotest.klienter.fpsak.behandlinger.dto.behandling.Aksjonspunkt;
@@ -294,7 +292,6 @@ public void ventTilBehandlingsstatus(BehandlingStatus forventetStatus) {
     }
 
     protected void refreshBehandling() {
-        LOG.info("Refresh behandling");
         venterPåFerdigProssesseringOgOppdaterBehandling(valgtBehandling);
     }
 
@@ -464,20 +461,9 @@ public Set<AktivitetStatus> hentUnikeBeregningAktivitetStatus() {
                 .collect(Collectors.toSet());
     }
 
-    /**
-     * AKSJONSPUNKT
-     */
-
-    public <T extends AksjonspunktBekreftelse> T hentAksjonspunktbekreftelse(Class<T> type) {
-        var aksjonspunktKode = type.getDeclaredAnnotation(BekreftelseKode.class).kode();
-        LOG.info("Henter aksjonspunktbekreftelse for {} ({})", aksjonspunktKode, type.getSimpleName());
-        try {
-            T bekreftelse = type.getConstructor().newInstance();
-            bekreftelse.oppdaterMedDataFraBehandling(valgtFagsak, valgtBehandling);
-            return bekreftelse;
-        } catch (InstantiationException | IllegalAccessException | InvocationTargetException | NoSuchMethodException e) {
-            throw new RuntimeException(e);
-        }
+    public <T extends AksjonspunktBekreftelse> T hentAksjonspunktbekreftelse(T bekreftelse) {
+        bekreftelse.oppdaterMedDataFraBehandling(valgtFagsak, valgtBehandling);
+        return bekreftelse;
     }
 
     /*
@@ -508,13 +494,13 @@ public boolean harAksjonspunkt(String kode) {
     /*
      * Bekrefte aksjonspunkt bekreftelse
      */
-    public <T extends AksjonspunktBekreftelse> void bekreftAksjonspunktMedDefaultVerdier(Class<T> type) {
+    public <T extends AksjonspunktBekreftelse> void bekreftAksjonspunktMedDefaultVerdier(T type) {
         bekreftAksjonspunkt(hentAksjonspunktbekreftelse(type));
     }
 
     public void bekreftAksjonspunkt(AksjonspunktBekreftelse bekreftelse) {
         bekreftAksjonspunktbekreftelserer(List.of(bekreftelse));
-        LOG.info("Aksjonspunktbekreftelse for {} er sendt inn og AP er løst", bekreftelse.kode());
+        LOG.info("Aksjonspunktbekreftelse for {} er sendt inn og AP er løst", bekreftelse.aksjonspunktKode());
     }
 
     public void bekreftAksjonspunktbekreftelserer(List<AksjonspunktBekreftelse> bekreftelser) {
@@ -539,11 +525,11 @@ public void bekreftAksjonspunktbekreftelserer(List<AksjonspunktBekreftelse> bekr
 
     private void verifsierAtAPErFerdigbehandlet(AksjonspunktBekreftelse bekreftelse) {
         var ap = valgtBehandling.getAksjonspunkt().stream()
-                .filter(aksjonspunkt -> aksjonspunkt.getDefinisjon().equalsIgnoreCase(bekreftelse.kode()))
+                .filter(aksjonspunkt -> aksjonspunkt.getDefinisjon().equalsIgnoreCase(bekreftelse.aksjonspunktKode()))
                 .findFirst()
                 .orElseThrow(); // Vil ikke inntreffe ettersom hentAksjonspunkt() vil alltid bli kalt først.
         if (!ap.getStatus().equalsIgnoreCase("UTFO")) {
-            throw new RuntimeException("AP bekreftelse er sendt inn programatisk for AP [" + bekreftelse.kode() +
+            throw new RuntimeException("AP bekreftelse er sendt inn programatisk for AP [" + bekreftelse.aksjonspunktKode() +
                     "] uten at det løste AP. Forventet status på AP er UTFO, men er [" + ap.getStatus() + "]");
         }
     }
@@ -553,7 +539,7 @@ private void verifsierAtAPErFerdigbehandlet(AksjonspunktBekreftelse bekreftelse)
         for (var kode : avvisteAksjonspunktkoder) {
             var AP = hentAksjonspunkt(kode);
             if (!AP.getStatus().equalsIgnoreCase("OPPR")) {
-                throw new RuntimeException("AP [" + bekreftelse.kode() + "] skal være avvist av beslutter og " +
+                throw new RuntimeException("AP [" + bekreftelse.aksjonspunktKode() + "] skal være avvist av beslutter og " +
                         "opprettet nytt, men har status [" + AP.getStatus() + "]");
             }
         }
@@ -810,7 +796,7 @@ public void fortsettUteninntektsmeldinger() {
                         arbfor.arbeidsgiverIdent(), arbfor.internArbeidsforholdId(), (long) valgtBehandling.versjon))
                 .toList();
         dtoer.forEach(this::lagreArbeidsforholdValg);
-        var ab = hentAksjonspunktbekreftelse(ArbeidInntektsmeldingBekreftelse.class);
+        var ab = hentAksjonspunktbekreftelse(new ArbeidInntektsmeldingBekreftelse());
         bekreftAksjonspunkt(ab);
     }
 

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

@@ -14,13 +14,13 @@ public abstract class VerdikjedeTestBase extends FpsakTestBase {
         if (!revurdering) {
             saksbehandler.ventTilRisikoKlassefiseringsstatus(RisikoklasseType.IKKE_HØY);
         }
-        saksbehandler.bekreftAksjonspunktMedDefaultVerdier(ForeslåVedtakBekreftelse.class);
+        saksbehandler.bekreftAksjonspunkt(new ForeslåVedtakBekreftelse());
 
         beslutter.hentFagsak(saksnummer);
         if (beslutter.harRevurderingBehandling() && revurdering) {
             beslutter.ventPåOgVelgRevurderingBehandling();
         }
-        FatterVedtakBekreftelse bekreftelse = beslutter.hentAksjonspunktbekreftelse(FatterVedtakBekreftelse.class);
+        FatterVedtakBekreftelse bekreftelse = beslutter.hentAksjonspunktbekreftelse(new FatterVedtakBekreftelse());
         bekreftelse.godkjennAksjonspunkter(beslutter.hentAksjonspunktSomSkalTilTotrinnsBehandling());
         if (tilbakekreving) {
             beslutter.bekreftAksjonspunkt(bekreftelse);

src/main/java/no/nav/foreldrepenger/autotest/klienter/JavaHttpKlient.java

@@ -72,13 +72,13 @@ private static HttpResponse<byte[]> sendByteArrayRequest(HttpRequest request) {
 
     private static <T> HttpResponse<T> send(HttpRequest request, HttpResponse.BodyHandler<T> responseHandler) {
         try {
-            var response = sendLogg(request, responseHandler);
+            var response = klient.send(request, responseHandler);
             var antallForsøk = 1;
             while (retryOn5xxFailures(response, antallForsøk)) {
                 LOG.warn("5xx feil mot {} for {}. gang. Prøver på nytt.", request.uri(), antallForsøk);
                 int ventSekunder = Math.min(2000, 1000 * antallForsøk++);
                 sleep(ventSekunder);
-                response = sendLogg(request, responseHandler);
+                response = klient.send(request, responseHandler);
             }
             return response;
         } catch (IOException e) {
@@ -89,13 +89,6 @@ private static <T> HttpResponse<T> send(HttpRequest request, HttpResponse.BodyHa
         }
     }
 
-    private static <T> HttpResponse<T> sendLogg(HttpRequest request, HttpResponse.BodyHandler<T> responseHandler) throws IOException, InterruptedException {
-        LOG.info("Sender request {}", request.uri());
-        var respons = klient.send(request, responseHandler);
-        LOG.info("Response fra {} {}", request.uri(), respons.statusCode());
-        return respons;
-    }
-
     private static <T> boolean retryOn5xxFailures(HttpResponse<T> response, int antallForsøk) {
         return antallForsøk < MAX_RETRY && is5xxStatus(response);
     }

src/main/java/no/nav/foreldrepenger/autotest/klienter/fpsak/behandlinger/dto/aksjonspunktbekreftelse/AksjonspunktBekreftelse.java

@@ -7,26 +7,16 @@
 
 public abstract class AksjonspunktBekreftelse {
 
-    @JsonProperty("@type")
     protected String kode;
     protected String begrunnelse;
 
-    @SuppressWarnings("unused")
-    protected AksjonspunktBekreftelse() {
-        if (null == this.getClass().getAnnotation(BekreftelseKode.class)) {
-            throw new RuntimeException("Kode annotation er ikke satt for " + this.getClass().getTypeName());
-        }
-        kode = this.getClass().getAnnotation(BekreftelseKode.class).kode();
-    }
+    @JsonProperty("@type")
+    public abstract String aksjonspunktKode();
 
     public void oppdaterMedDataFraBehandling(Fagsak fagsak, Behandling behandling) {
 
     }
 
-    public String kode() {
-        return kode;
-    }
-
     public AksjonspunktBekreftelse setBegrunnelse(String begrunnelse) {
         this.begrunnelse = begrunnelse;
         return this;

src/main/java/no/nav/foreldrepenger/autotest/klienter/fpsak/behandlinger/dto/aksjonspunktbekreftelse/AvklarAktiviteterBekreftelse.java

@@ -9,7 +9,6 @@
 import no.nav.foreldrepenger.autotest.klienter.fpsak.fagsak.dto.Fagsak;
 import no.nav.foreldrepenger.common.domain.ArbeidsgiverIdentifikator;
 
-@BekreftelseKode(kode = "5052")
 @JsonIgnoreProperties(ignoreUnknown = true)
 public class AvklarAktiviteterBekreftelse extends AksjonspunktBekreftelse {
 
@@ -47,6 +46,11 @@ private BeregningsaktivitetLagreDto hentBeregningsaktivitetMedOpptjenignsaktivit
                     .orElseThrow(() -> new RuntimeException("Fant ikke beregningsaktivitet med opptjeningsaktivetetstype " + opptjeningsAktivitetType));
     }
 
+    @Override
+    public String aksjonspunktKode() {
+        return "5052";
+    }
+
     @Override
     public void oppdaterMedDataFraBehandling(Fagsak fagsak, Behandling behandling) {
         beregningsaktivitetLagreDtoList = behandling.getBeregningsgrunnlag().getFaktaOmBeregning()

src/main/java/no/nav/foreldrepenger/autotest/klienter/fpsak/behandlinger/dto/aksjonspunktbekreftelse/BekreftelseKode.java

@@ -22,13 +22,13 @@
 import no.nav.foreldrepenger.common.domain.foreldrepenger.fordeling.LukketPeriodeMedVedlegg;
 import no.nav.foreldrepenger.common.domain.foreldrepenger.fordeling.StønadskontoType;
 
-@BekreftelseKode(kode = "5071")
 public class FastsettUttaksperioderManueltBekreftelse extends AksjonspunktBekreftelse {
 
     protected List<UttakResultatPeriode> perioder = new ArrayList<>();
 
-    public FastsettUttaksperioderManueltBekreftelse() {
-        super();
+    @Override
+    public String aksjonspunktKode() {
+        return "5071";
     }
 
     @Override

src/main/java/no/nav/foreldrepenger/autotest/klienter/fpsak/behandlinger/dto/aksjonspunktbekreftelse/FastsettBruttoBeregningsgrunnlagSNBekreftelse.java

@@ -7,13 +7,13 @@
 import no.nav.foreldrepenger.autotest.klienter.fpsak.behandlinger.dto.behandling.uttak.UttakResultatPeriode;
 import no.nav.foreldrepenger.autotest.klienter.fpsak.fagsak.dto.Fagsak;
 
-@BekreftelseKode(kode = "5076")
 public class FastsetteUttakKontrollerOpplysningerOmDødDto extends AksjonspunktBekreftelse {
 
     protected List<UttakResultatPeriode> perioder = new ArrayList<>();
 
-    public FastsetteUttakKontrollerOpplysningerOmDødDto() {
-        super();
+    @Override
+    public String aksjonspunktKode() {
+        return "5076";
     }
 
     @Override

src/main/java/no/nav/foreldrepenger/autotest/klienter/fpsak/behandlinger/dto/aksjonspunktbekreftelse/FastsettUttaksperioderManueltBekreftelse.java

@@ -7,15 +7,10 @@
 import no.nav.foreldrepenger.autotest.klienter.fpsak.behandlinger.dto.behandling.Aksjonspunkt;
 import no.nav.foreldrepenger.autotest.util.AllureHelper;
 
-@BekreftelseKode(kode = "5016")
 public class FatterVedtakBekreftelse extends AksjonspunktBekreftelse {
 
     protected List<AksjonspunktGodkjenningDto> aksjonspunktGodkjenningDtos = new ArrayList<>();
 
-    public FatterVedtakBekreftelse() {
-        super();
-    }
-
     public FatterVedtakBekreftelse godkjennAksjonspunkter(List<Aksjonspunkt> aksjonspunkter) {
         aksjonspunkter.forEach(this::godkjennAksjonspunkt);
         return this;
@@ -63,8 +58,13 @@ public List<String> avvisteAksjonspunkt() {
                 .toList();
     }
 
+    @Override
+    public String aksjonspunktKode() {
+        return "5016";
+    }
+
     public static class AksjonspunktGodkjenningDto {
-        protected String aksjonspunktKode;
+        public String aksjonspunktKode;
         protected List<String> arsaker = new ArrayList<>();
         protected String begrunnelse = null;
         protected boolean godkjent = false;