feat. trenger ikke lengre å hoppe over dubletter av deltaker id

Nå støtter arena-agreement-migration -tabellen å ha flere deltakere på ett tiltak. Derfor er det ikke noe behov for å ha egen logikk for å hoppe over disse - de kan bare ignoreres.
navikt · Dec 12, 2024 · a37aee8 · a37aee8
1 parent 753fc3c
commit a37aee8
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 36 deletions.
diff --git a/.../java/no/nav/tag/tiltaksgjennomforing/arena/models/migration/ArenaAgreementAggregate.java b/.../java/no/nav/tag/tiltaksgjennomforing/arena/models/migration/ArenaAgreementAggregate.java
@@ -73,12 +73,8 @@ public Optional<LocalDate> findSluttdato() {
             .findFirst();
     }
 
-    public boolean isTiltakDublett() {
-        return isDublett(eksternIdTiltak);
-    }
-
-    public boolean isDeltakerDublett() {
-        return isDublett(eksternIdDeltaker);
+    public boolean isDublett() {
+        return isDublett(eksternIdTiltak) || isDublett(eksternIdDeltaker);
     }
 
     public Optional<UUID> getEksternIdAsUuid() throws IllegalArgumentException {

diff --git a/...ain/java/no/nav/tag/tiltaksgjennomforing/arena/models/migration/ArenaMigrationAction.java b/...ain/java/no/nav/tag/tiltaksgjennomforing/arena/models/migration/ArenaMigrationAction.java
@@ -8,7 +8,6 @@
 import java.time.LocalDate;
 
 public enum ArenaMigrationAction {
-    HOPP_OVER,
     GJENOPPRETT,
     OPPRETT,
     OPPDATER,
@@ -21,14 +20,9 @@ public static ArenaMigrationAction map(ArenaAgreementAggregate agreementAggregat
         boolean isSluttdatoInTheFuture = agreementAggregate.findSluttdato()
             .map(sluttdato -> sluttdato.isAfter(LocalDate.now())).orElse(false);
 
-        if (agreementAggregate.isDeltakerDublett()) {
-            return HOPP_OVER;
-        }
-
-        if (agreementAggregate.isTiltakDublett()) {
+        if (agreementAggregate.isDublett()) {
             return IGNORER;
         }
-
         return switch (deltakerstatuskode) {
             case GJENN, TILBUD -> isSluttdatoInTheFuture ? OPPRETT : IGNORER;
             case null, default -> IGNORER;
@@ -46,11 +40,7 @@ public static ArenaMigrationAction map(
         boolean isSluttdatoIFremtiden = agreementAggregate.findSluttdato()
             .map(sluttdato -> sluttdato.isAfter(LocalDate.now())).orElse(false);
 
-        if (agreementAggregate.isDeltakerDublett()) {
-            return HOPP_OVER;
-        }
-
-        if (agreementAggregate.isTiltakDublett()) {
+        if (agreementAggregate.isDublett()) {
             return IGNORER;
         }
 

diff --git a/...a/no/nav/tag/tiltaksgjennomforing/arena/models/migration/ArenaMigrationProcessResult.java b/...a/no/nav/tag/tiltaksgjennomforing/arena/models/migration/ArenaMigrationProcessResult.java
@@ -3,7 +3,6 @@
 import no.nav.tag.tiltaksgjennomforing.avtale.Avtale;
 
 public sealed interface ArenaMigrationProcessResult {
-    record Skip(ArenaMigrationAction action) implements ArenaMigrationProcessResult {}
     record Ignored(ArenaMigrationAction action) implements ArenaMigrationProcessResult {}
     record Completed(ArenaMigrationAction action, Avtale avtale) implements ArenaMigrationProcessResult {}
 }
diff --git a/...n/java/no/nav/tag/tiltaksgjennomforing/arena/service/ArenaAgreementProcessingService.java b/...n/java/no/nav/tag/tiltaksgjennomforing/arena/service/ArenaAgreementProcessingService.java
@@ -116,9 +116,6 @@ public void process(ArenaAgreementAggregate agreementAggregate) {
                         eksternId,
                         null
                     );
-                case ArenaMigrationProcessResult.Skip skip -> {
-                    // Gjør ingenting
-                }
             }
         } catch(Exception e) {
             log.error("Feil ved prossesering av avtale fra Arena", e);
@@ -158,20 +155,14 @@ private void saveMigrationStatus(
     private ArenaMigrationProcessResult updateAvtale(Avtale avtale, ArenaAgreementAggregate agreementAggregate) {
         ArenaMigrationAction action = ArenaMigrationAction.map(avtale, agreementAggregate);
         switch (action) {
-            case HOPP_OVER -> {
-                log.info(
-                    "Avtale har deltaker som er duplett i Arena. Hopper over avtalen. Den vil sannsynligvis bli oppdatert senere."
-                );
-                return new ArenaMigrationProcessResult.Skip(action);
-            }
             case IGNORER -> {
                 log.info(
                     "Avtale med id {} og status {} har tiltakstatus {}, deltakerstatus {} og tiltaket er {} i Arena. Ignorerer avtalen.",
                     avtale.getId(),
                     avtale.getStatus(),
                     agreementAggregate.getTiltakstatuskode(),
                     agreementAggregate.getDeltakerstatuskode(),
-                    agreementAggregate.isTiltakDublett() ? "dublett" : "ikke dublett"
+                    agreementAggregate.isDublett() ? "dublett" : "ikke dublett"
                 );
                 return new ArenaMigrationProcessResult.Ignored(action);
             }
@@ -224,13 +215,6 @@ private ArenaMigrationProcessResult updateAvtale(Avtale avtale, ArenaAgreementAg
     private ArenaMigrationProcessResult createAvtale(ArenaAgreementAggregate agreementAggregate) {
         ArenaMigrationAction action = ArenaMigrationAction.map(agreementAggregate);
 
-        if (ArenaMigrationAction.HOPP_OVER == action) {
-            log.info(
-                "Avtale har deltaker som er duplett i Arena. Hopper over avtalen. Den vil sannsynligvis bli oppdatert senere."
-            );
-            return new ArenaMigrationProcessResult.Skip(action);
-        }
-
         if (ArenaMigrationAction.IGNORER == action) {
             log.info(
                 "Avtale har tiltaksstatus {}, deltakerstatus {}, sluttdato {} og er {} i Arena. Ignorerer avtalen.",