Chore: fjerner ekstra kode som skulle fikse alle manglende ytelse typer under avsltting av grunnlag (#2674)

* Rydder ubrukte open api annoteringer.

* Redyserer kode duplisering og fjerner runksjonalitet som ikke lenger trenges.

* Chore: fjerner ekstra kode som skulle fikse alle manglende ytelse typer under avsltting av grunnlag.

Author: mrsladek

domenetjenester/kobling/src/main/java/no/nav/foreldrepenger/abakus/kobling/Kobling.java

@@ -147,10 +147,6 @@ public YtelseType getYtelseType() {
         return ytelseType;
     }
 
-    public void setYtelseType(YtelseType ytelseType) {
-        this.ytelseType = ytelseType;
-    }
-
     public long getVersjon() {
         return this.versjon;
     }

web/src/main/java/no/nav/foreldrepenger/abakus/kobling/AvsluttKoblingTjeneste.java

@@ -5,7 +5,6 @@
 
 import jakarta.enterprise.context.ApplicationScoped;
 import jakarta.inject.Inject;
-import no.nav.abakus.iaygrunnlag.kodeverk.YtelseType;
 import no.nav.foreldrepenger.abakus.iay.InntektArbeidYtelseTjeneste;
 
 @ApplicationScoped
@@ -29,22 +28,16 @@ public AvsluttKoblingTjeneste(KoblingTjeneste koblingTjeneste,
 
     /**
      * Avslutter kobling og sletter inaktive grunnlag. Forutsetter at kobling finnes.
+     *
      * @param koblingReferanse - referanse til kobling
-     * @param ytelseType - ytelseType for å fikse der det mangler ca 2500 koblinger fra 2018 og 2019.
      */
-    public void avsluttKobling(KoblingReferanse koblingReferanse, YtelseType ytelseType) {
+    public void avsluttKobling(KoblingReferanse koblingReferanse) {
         var koblingLås = koblingTjeneste.taSkrivesLås(koblingReferanse);
         var kobling = koblingTjeneste.hentFor(koblingReferanse).orElseThrow();
 
         LOG.info("Starter avslutting av kobling for sak=[{}, {}] med behandling='{}'", kobling.getSaksnummer(), kobling.getYtelseType(),
             kobling.getKoblingReferanse());
 
-        // Midlertidig for å kunne fikse ytelse-type UNDEFINED.
-        // Vi fikser manglende ytelseType på kobling
-        if (YtelseType.UDEFINERT.equals(kobling.getYtelseType()) && ytelseType != null) {
-            kobling.setYtelseType(ytelseType);
-        }
-
         iayTjeneste.slettInaktiveGrunnlagFor(kobling.getKoblingReferanse());
         koblingTjeneste.deaktiver(kobling.getKoblingReferanse());
         koblingTjeneste.oppdaterLåsVersjon(koblingLås);

web/src/main/java/no/nav/foreldrepenger/abakus/kobling/KoblingRestTjeneste.java

@@ -73,7 +73,7 @@ public Response deaktiverKobling(@Valid @NotNull KoblingRestTjeneste.AvsluttKobl
             var kobling = koblingOptional.get();
             if (kobling.erAktiv()) {
                 validerRequest(request, kobling);
-                avsluttKoblingTjeneste.avsluttKobling(kobling.getKoblingReferanse(), request.getYtelseType());
+                avsluttKoblingTjeneste.avsluttKobling(kobling.getKoblingReferanse());
                 return Response.ok().build();
             } else {
                 LOG.info("KOBLING. Kobling er allerede avsluttet for referanse: {}", request.getReferanse());
@@ -90,8 +90,7 @@ private static void validerRequest(AvsluttKoblingRequestAbacDto request, Kobling
             throw new IllegalArgumentException("Prøver å avslutte kobling på feil saksnummer");
         }
 
-        // Noen koblinger fra 2019 og 2020 mangler ytelseType vi slipper de gjennom og oppdatarer ytelseType før avslutning.
-        if (!request.getYtelseType().equals(kobling.getYtelseType()) && !YtelseType.UDEFINERT.equals(kobling.getYtelseType())) {
+        if (!request.getYtelseType().equals(kobling.getYtelseType())) {
             throw new IllegalArgumentException("Prøver å avslutte kobling på feil ytelsetype");
         }
 

web/src/test/java/no/nav/foreldrepenger/abakus/kobling/AvsluttKoblingTjenesteTest.java

@@ -46,49 +46,11 @@ void avslutt_grunnlag_ok() {
         when(koblingTjeneste.hentFor(referanse)).thenReturn(Optional.of(kobling));
 
         // Act
-        avsluttKoblingTjeneste.avsluttKobling(referanse, YtelseType.FORELDREPENGER);
+        avsluttKoblingTjeneste.avsluttKobling(referanse);
 
         // Verify
         verify(iayTjeneste).slettInaktiveGrunnlagFor(referanse);
         verify(koblingTjeneste).deaktiver(referanse);
-
-        verify(kobling, never()).setYtelseType(YtelseType.FORELDREPENGER);
-    }
-
-    @Test
-    void avslutt_grunnlag_oppdater_ytelse_type_mangler_i_request_ok() {
-        var referanse = new KoblingReferanse(UUID.randomUUID());
-        when(kobling.getKoblingReferanse()).thenReturn(referanse);
-        when(kobling.getYtelseType()).thenReturn(YtelseType.UDEFINERT);
-
-        when(koblingTjeneste.hentFor(referanse)).thenReturn(Optional.of(kobling));
-
-        // Act
-        avsluttKoblingTjeneste.avsluttKobling(referanse, null);
-
-        // Verify
-        verify(iayTjeneste).slettInaktiveGrunnlagFor(referanse);
-        verify(koblingTjeneste).deaktiver(referanse);
-
-        verify(kobling, never()).setYtelseType(YtelseType.FORELDREPENGER);
-    }
-
-    @Test
-    void avslutt_grunnlag_oppdater_ytelse_type_ok() {
-        var referanse = new KoblingReferanse(UUID.randomUUID());
-        when(kobling.getKoblingReferanse()).thenReturn(referanse);
-        when(kobling.getYtelseType()).thenReturn(YtelseType.UDEFINERT);
-
-        when(koblingTjeneste.hentFor(referanse)).thenReturn(Optional.of(kobling));
-
-        // Act
-        avsluttKoblingTjeneste.avsluttKobling(referanse, YtelseType.FORELDREPENGER);
-
-        // Verify
-        verify(iayTjeneste).slettInaktiveGrunnlagFor(referanse);
-        verify(koblingTjeneste).deaktiver(referanse);
-
-        verify(kobling).setYtelseType(YtelseType.FORELDREPENGER);
     }
 
     @Test
@@ -98,7 +60,7 @@ void avslutt_grunnlag_exception_ingen_kobling_nok() {
         when(koblingTjeneste.hentFor(referanse)).thenReturn(Optional.empty());
 
         // Act
-        assertThrows(NoSuchElementException.class, () -> avsluttKoblingTjeneste.avsluttKobling(referanse, YtelseType.FORELDREPENGER));
+        assertThrows(NoSuchElementException.class, () -> avsluttKoblingTjeneste.avsluttKobling(referanse));
 
         // Verify
         verify(iayTjeneste, never()).slettInaktiveGrunnlagFor(referanse);

web/src/test/java/no/nav/foreldrepenger/abakus/kobling/KoblingRestTjenesteTest.java

@@ -54,7 +54,7 @@ void sletting_av_kobling_ok() {
 
         // Assert
         assertThat(response.getStatus()).isEqualTo(HttpURLConnection.HTTP_OK);
-        verify(avsluttKobling).avsluttKobling(referanse, ytelseType);
+        verify(avsluttKobling).avsluttKobling(referanse);
     }
 
     @Test
@@ -75,7 +75,7 @@ void sletting_av_kobling_allerede_deaktivert_ok() {
 
         // Assert
         assertThat(response.getStatus()).isEqualTo(HttpURLConnection.HTTP_NO_CONTENT);
-        verify(avsluttKobling, never()).avsluttKobling(referanse, ytelseType);
+        verify(avsluttKobling, never()).avsluttKobling(referanse);
     }
 
     @Test
@@ -91,7 +91,7 @@ void sletting_av_kobling_feil_ytesletype_bad_request_nok() {
 
         // Assert
         assertThat(response.getStatus()).isEqualTo(HttpURLConnection.HTTP_BAD_REQUEST);
-        verify(avsluttKobling, never()).avsluttKobling(any(), any());
+        verify(avsluttKobling, never()).avsluttKobling(any());
     }
 
     @Test
@@ -127,7 +127,7 @@ void sletting_av_kobling_saksnummer_matcher_ikke_nok() {
 
         // Assert
         assertThat(ex.getMessage()).contains("Prøver å avslutte kobling på feil saksnummer");
-        verify(avsluttKobling, never()).avsluttKobling(any(), any());
+        verify(avsluttKobling, never()).avsluttKobling(any());
     }
 
     @Test
@@ -146,7 +146,7 @@ void sletting_av_kobling_ytelse_type_matcher_ikke_nok() {
 
         // Assert
         assertThat(ex.getMessage()).contains("Prøver å avslutte kobling på feil ytelsetype");
-        verify(avsluttKobling, never()).avsluttKobling(any(), any());
+        verify(avsluttKobling, never()).avsluttKobling(any());
     }
 
     @Test
@@ -165,7 +165,7 @@ void sletting_av_kobling_ytelse_type_matcher_ikke_nok() {
 
         // Assert
         assertThat(ex.getMessage()).contains("Prøver å avslutte kobling på feil aktør");
-        verify(avsluttKobling, never()).avsluttKobling(any(), any());
+        verify(avsluttKobling, never()).avsluttKobling(any());
     }
 
     private Kobling opprettKobling(String saksnummer, AktørId aktørId, YtelseType ytelseType, KoblingReferanse referanse) {