Sjekk beslutter mot den som foreslo. Fjerne ubrukt event

navikt · Jun 11, 2024 · cb89967 · cb89967
1 parent 57b2043
commit cb89967
Show file tree

Hide file tree

Showing 6 changed files with 14 additions and 138 deletions.
diff --git a/...v/foreldrepenger/tilbakekreving/behandlingskontroll/events/BehandlingStegStatusEvent.java b/...v/foreldrepenger/tilbakekreving/behandlingskontroll/events/BehandlingStegStatusEvent.java
diff --git a/.../no/nav/foreldrepenger/tilbakekreving/behandlingskontroll/impl/BehandlingStegVisitor.java b/.../no/nav/foreldrepenger/tilbakekreving/behandlingskontroll/impl/BehandlingStegVisitor.java
@@ -214,7 +214,6 @@ private void oppdaterBehandlingStegStatus(Behandling behandling,
             if (erForskjellig(førsteStegStatus, nyStegStatus)) {
                 InternalManipulerBehandling.forceOppdaterBehandlingSteg(behandling, stegType, nyStegStatus, BehandlingStegStatus.UTFØRT);
                 behandlingRepository.lagre(behandling, kontekst.getSkriveLås());
-                eventPubliserer.fireEvent(kontekst, stegType, førsteStegStatus, nyStegStatus);
             }
         }
     }

diff --git a/...drepenger/tilbakekreving/behandlingskontroll/impl/BehandlingskontrollEventPubliserer.java b/...drepenger/tilbakekreving/behandlingskontroll/impl/BehandlingskontrollEventPubliserer.java
@@ -5,17 +5,15 @@
 import jakarta.enterprise.context.ApplicationScoped;
 import jakarta.enterprise.event.Event;
 import jakarta.inject.Inject;
+
 import no.nav.foreldrepenger.tilbakekreving.behandlingskontroll.BehandlingskontrollKontekst;
 import no.nav.foreldrepenger.tilbakekreving.behandlingskontroll.events.AksjonspunktStatusEvent;
 import no.nav.foreldrepenger.tilbakekreving.behandlingskontroll.events.BehandlingStatusEvent;
 import no.nav.foreldrepenger.tilbakekreving.behandlingskontroll.events.BehandlingStegOvergangEvent;
-import no.nav.foreldrepenger.tilbakekreving.behandlingskontroll.events.BehandlingStegStatusEvent;
 import no.nav.foreldrepenger.tilbakekreving.behandlingskontroll.events.BehandlingTransisjonEvent;
 import no.nav.foreldrepenger.tilbakekreving.behandlingskontroll.events.BehandlingskontrollEvent;
 import no.nav.foreldrepenger.tilbakekreving.behandlingslager.behandling.BehandlingEvent;
 import no.nav.foreldrepenger.tilbakekreving.behandlingslager.behandling.BehandlingStatus;
-import no.nav.foreldrepenger.tilbakekreving.behandlingslager.behandling.BehandlingStegStatus;
-import no.nav.foreldrepenger.tilbakekreving.behandlingslager.behandling.BehandlingStegType;
 
 
 /**
@@ -53,15 +51,6 @@ public void fireEvent(BehandlingTransisjonEvent event) {
         doFireEvent(event);
     }
 
-    public void fireEvent(BehandlingskontrollKontekst kontekst, BehandlingStegType stegType, BehandlingStegStatus forrigeStatus,
-                          BehandlingStegStatus nyStatus) {
-        if (Objects.equals(forrigeStatus, nyStatus)) {
-            // gjør ingenting
-            return;
-        }
-        doFireEvent(new BehandlingStegStatusEvent(kontekst, stegType, forrigeStatus, nyStatus));
-    }
-
     public void fireEvent(BehandlingskontrollKontekst kontekst, BehandlingStatus gammelStatus, BehandlingStatus nyStatus) {
         if (Objects.equals(gammelStatus, nyStatus)) {
             // gjør ingenting

diff --git a/...enger/tilbakekreving/behandlingskontroll/impl/BehandlingskontrollEventPublisererTest.java b/...enger/tilbakekreving/behandlingskontroll/impl/BehandlingskontrollEventPublisererTest.java
@@ -15,7 +15,6 @@
 import no.nav.foreldrepenger.tilbakekreving.behandlingskontroll.BehandlingStegTilstandSnapshot;
 import no.nav.foreldrepenger.tilbakekreving.behandlingskontroll.BehandlingskontrollKontekst;
 import no.nav.foreldrepenger.tilbakekreving.behandlingskontroll.events.BehandlingStegOvergangEvent;
-import no.nav.foreldrepenger.tilbakekreving.behandlingskontroll.events.BehandlingStegStatusEvent;
 import no.nav.foreldrepenger.tilbakekreving.behandlingskontroll.events.BehandlingskontrollEvent;
 import no.nav.foreldrepenger.tilbakekreving.behandlingskontroll.spi.BehandlingskontrollServiceProvider;
 import no.nav.foreldrepenger.tilbakekreving.behandlingslager.behandling.Behandling;
@@ -100,35 +99,6 @@ void skal_fyre_event_for_behandlingskontroll_startet_stoppet_ved_prosessering()
 
     }
 
-    @Test
-    void skal_fyre_event_for_behandlingskontroll_behandlingsteg_status_endring_ved_prosessering() {
-        // Arrange
-        ScenarioSimple scenario = nyttScenario(STEG_1);
-
-        Behandling behandling = scenario.lagre(repositoryProvider);
-
-        BehandlingskontrollKontekst kontekst = kontrollTjeneste.initBehandlingskontroll(behandling.getId());
-
-        // Act
-        kontrollTjeneste.prosesserBehandling(kontekst);
-
-        // Assert
-
-        BehandlingStegStatusEvent steg1StatusEvent0 = new BehandlingStegStatusEvent(kontekst, STEG_1, null, BehandlingStegStatus.STARTET);
-        BehandlingStegStatusEvent steg1StatusEvent1 = new BehandlingStegStatusEvent(kontekst, STEG_1, BehandlingStegStatus.STARTET, BehandlingStegStatus.UTFØRT);
-        BehandlingStegStatusEvent steg2StatusEvent0 = new BehandlingStegStatusEvent(kontekst, STEG_2, null, BehandlingStegStatus.STARTET);
-        BehandlingStegStatusEvent steg2StatusEvent = new BehandlingStegStatusEvent(kontekst, STEG_2, BehandlingStegStatus.STARTET, BehandlingStegStatus.UTFØRT);
-        BehandlingStegStatusEvent steg3StatusEvent0 = new BehandlingStegStatusEvent(kontekst, STEG_2, null, BehandlingStegStatus.STARTET);
-        BehandlingStegStatusEvent steg3StatusEvent = new BehandlingStegStatusEvent(kontekst, STEG_3, BehandlingStegStatus.STARTET, BehandlingStegStatus.UTFØRT);
-        BehandlingStegStatusEvent steg4StatusEvent0 = new BehandlingStegStatusEvent(kontekst, STEG_4, null, BehandlingStegStatus.STARTET);
-        BehandlingStegStatusEvent steg4StatusEvent = new BehandlingStegStatusEvent(kontekst, STEG_3, BehandlingStegStatus.STARTET, BehandlingStegStatus.UTGANG);
-        TestEventObserver.containsExactly(steg1StatusEvent0, steg1StatusEvent1 //
-                , steg2StatusEvent0, steg2StatusEvent//
-                , steg3StatusEvent0, steg3StatusEvent//
-                , steg4StatusEvent0, steg4StatusEvent//
-        );
-    }
-
     @Test
     void skal_fyre_event_for_behandlingskontroll_tilbakeføring_ved_prosessering() {
         // Arrange

diff --git a/...java/no/nav/foreldrepenger/tilbakekreving/behandlingskontroll/impl/TestEventObserver.java b/...java/no/nav/foreldrepenger/tilbakekreving/behandlingskontroll/impl/TestEventObserver.java
@@ -17,7 +17,6 @@
 import no.nav.foreldrepenger.tilbakekreving.behandlingskontroll.events.AksjonspunktStatusEvent;
 import no.nav.foreldrepenger.tilbakekreving.behandlingskontroll.events.BehandlingStatusEvent;
 import no.nav.foreldrepenger.tilbakekreving.behandlingskontroll.events.BehandlingStegOvergangEvent;
-import no.nav.foreldrepenger.tilbakekreving.behandlingskontroll.events.BehandlingStegStatusEvent;
 import no.nav.foreldrepenger.tilbakekreving.behandlingskontroll.events.BehandlingskontrollEvent;
 import no.nav.foreldrepenger.tilbakekreving.behandlingslager.behandling.BehandlingEvent;
 import no.nav.foreldrepenger.tilbakekreving.behandlingslager.behandling.aksjonspunkt.Aksjonspunkt;
@@ -63,10 +62,6 @@ public void observer(@Observes BehandlingskontrollEvent event) {
         addEvent(event);
     }
 
-    public void observer(@Observes BehandlingStegStatusEvent event) {
-        addEvent(event);
-    }
-
     public static void containsExactly(AksjonspunktDefinisjon[]... ads) {
         List<AksjonspunktStatusEvent> aksjonspunkterEvents = getEvents(AksjonspunktStatusEvent.class);
         Assertions.assertThat(aksjonspunkterEvents).hasSize(ads.length);
@@ -110,16 +105,6 @@ public static void containsExactly(BehandlingStegOvergangEvent... bsoe) {
         }
     }
 
-    public static void containsExactly(BehandlingStegStatusEvent... bsoe) {
-        List<BehandlingStegStatusEvent> behandlingStegStatusEvents = getEvents(BehandlingStegStatusEvent.class);
-        Assertions.assertThat(behandlingStegStatusEvents).hasSize(bsoe.length);
-        for (int i = 0; i < bsoe.length; i++) {
-            BehandlingStegStatusEvent minEvent = bsoe[i];
-            assertThat(behandlingStegStatusEvents.get(i).getForrigeStatus()).as("%s:%s", i, minEvent.getStegType()).isEqualTo(minEvent.getForrigeStatus());
-            assertThat(behandlingStegStatusEvents.get(i).getNyStatus()).as("%s:%s", i, minEvent.getStegType()).isEqualTo(minEvent.getNyStatus());
-        }
-    }
-
     private static String hentKode(Optional<BehandlingStegTilstandSnapshot> behandlingStegTilstand) {
         return behandlingStegTilstand
                 .map(BehandlingStegTilstandSnapshot::getStatus)

diff --git a/...av/foreldrepenger/tilbakekreving/web/app/tjenester/behandling/BehandlingRestTjeneste.java b/...av/foreldrepenger/tilbakekreving/web/app/tjenester/behandling/BehandlingRestTjeneste.java
@@ -7,12 +7,6 @@
 import java.util.Optional;
 import java.util.UUID;
 
-import io.swagger.v3.oas.annotations.Operation;
-import io.swagger.v3.oas.annotations.Parameter;
-import io.swagger.v3.oas.annotations.headers.Header;
-import io.swagger.v3.oas.annotations.media.Content;
-import io.swagger.v3.oas.annotations.media.Schema;
-import io.swagger.v3.oas.annotations.responses.ApiResponse;
 import jakarta.enterprise.context.RequestScoped;
 import jakarta.inject.Inject;
 import jakarta.servlet.http.HttpServletRequest;
@@ -28,6 +22,13 @@
 import jakarta.ws.rs.core.Context;
 import jakarta.ws.rs.core.MediaType;
 import jakarta.ws.rs.core.Response;
+
+import io.swagger.v3.oas.annotations.Operation;
+import io.swagger.v3.oas.annotations.Parameter;
+import io.swagger.v3.oas.annotations.headers.Header;
+import io.swagger.v3.oas.annotations.media.Content;
+import io.swagger.v3.oas.annotations.media.Schema;
+import io.swagger.v3.oas.annotations.responses.ApiResponse;
 import no.nav.foreldrepenger.tilbakekreving.behandling.BehandlingFeil;
 import no.nav.foreldrepenger.tilbakekreving.behandling.dto.BehandlingReferanse;
 import no.nav.foreldrepenger.tilbakekreving.behandling.impl.BehandlendeEnhetTjeneste;
@@ -40,11 +41,13 @@
 import no.nav.foreldrepenger.tilbakekreving.behandling.steg.hentgrunnlag.finn.FinnGrunnlagTask;
 import no.nav.foreldrepenger.tilbakekreving.behandlingskontroll.impl.BehandlingskontrollAsynkTjeneste;
 import no.nav.foreldrepenger.tilbakekreving.behandlingskontroll.task.FortsettBehandlingTask;
+import no.nav.foreldrepenger.tilbakekreving.behandlingslager.BaseEntitet;
 import no.nav.foreldrepenger.tilbakekreving.behandlingslager.aktør.OrganisasjonsEnhet;
 import no.nav.foreldrepenger.tilbakekreving.behandlingslager.behandling.Behandling;
 import no.nav.foreldrepenger.tilbakekreving.behandlingslager.behandling.BehandlingResultatType;
 import no.nav.foreldrepenger.tilbakekreving.behandlingslager.behandling.BehandlingStatus;
 import no.nav.foreldrepenger.tilbakekreving.behandlingslager.behandling.BehandlingType;
+import no.nav.foreldrepenger.tilbakekreving.behandlingslager.behandling.aksjonspunkt.AksjonspunktDefinisjon;
 import no.nav.foreldrepenger.tilbakekreving.behandlingslager.behandling.ekstern.EksternBehandling;
 import no.nav.foreldrepenger.tilbakekreving.behandlingslager.fagsak.FagsakYtelseType;
 import no.nav.foreldrepenger.tilbakekreving.behandlingslager.historikk.HistorikkAktør;
@@ -565,8 +568,10 @@ private BehandlingOperasjonerDto lovligeOperasjoner(Behandling b, boolean finnes
         if (b.erSaksbehandlingAvsluttet()) {
             return BehandlingOperasjonerDto.builder(b.getUuid()).build(); // Skal ikke foreta menyvalg lenger
         } else if (BehandlingStatus.FATTER_VEDTAK.equals(b.getStatus())) {
-            boolean tilgokjenning = b.getAnsvarligSaksbehandler() != null &&
-                !b.getAnsvarligSaksbehandler().equalsIgnoreCase(KontekstHolder.getKontekst().getUid());
+            var sisteSaksbehandler = Optional.ofNullable(b.getAnsvarligSaksbehandler())
+                .or(() -> b.getAksjonspunktMedDefinisjonOptional(AksjonspunktDefinisjon.FORESLÅ_VEDTAK).map(BaseEntitet::getEndretAv));
+            boolean tilgokjenning = sisteSaksbehandler.isPresent() &&
+                !sisteSaksbehandler.get().equalsIgnoreCase(KontekstHolder.getKontekst().getUid());
             return BehandlingOperasjonerDto.builder(b.getUuid()).medTilGodkjenning(tilgokjenning).build();
         } else {
             boolean totrinnRetur = totrinnTjeneste.hentTotrinnsvurderinger(b).stream().anyMatch(tt -> !tt.isGodkjent());