Forenkle oppgavetyper (#1254)

* Forenkle oppgaetyper * Fjerne alt obsolete fra oppgavetyper * Deprecations
navikt · Feb 25, 2023 · 00cc14f · 00cc14f
1 parent c12d7dc
commit 00cc14f
Show file tree

Hide file tree

Showing 5 changed files with 35 additions and 36 deletions.
diff --git a/felles/abac/src/main/java/no/nav/vedtak/sikkerhet/abac/ÅpenRessurs.java b/felles/abac/src/main/java/no/nav/vedtak/sikkerhet/abac/ÅpenRessurs.java
@@ -6,11 +6,9 @@
 import java.lang.annotation.RetentionPolicy;
 import java.lang.annotation.Target;
 
-import javax.interceptor.InterceptorBinding;
 import javax.ws.rs.NameBinding;
 
 @Inherited
-@InterceptorBinding
 @Retention(RetentionPolicy.RUNTIME)
 @Target({ ElementType.TYPE, ElementType.METHOD })
 @NameBinding

diff --git a/...ient/src/main/java/no/nav/vedtak/felles/integrasjon/oppgave/v1/AbstractOppgaveKlient.java b/...ient/src/main/java/no/nav/vedtak/felles/integrasjon/oppgave/v1/AbstractOppgaveKlient.java
@@ -2,6 +2,7 @@
 
 import java.net.URI;
 import java.util.List;
+import java.util.Objects;
 
 import javax.ws.rs.core.UriBuilder;
 
@@ -10,11 +11,15 @@
 import no.nav.vedtak.felles.integrasjon.rest.RestConfig;
 import no.nav.vedtak.felles.integrasjon.rest.RestRequest;
 
-//@RestClientConfig(tokenConfig = TokenFlow.ADAPTIVE, endpointProperty = "oppgave.rs.uri", endpointDefault = "http://oppgave.default/api/v1/oppgaver",
-//    scopesProperty = "oppgave.scopes", scopesDefault = "api://prod-fss.oppgavehandtering.oppgave/.default")
+/**
+ * API / Swagger https://oppgave.dev.intern.nav.no
+ *
+ * Bruk TokenFlow ADAPTIVE og scope cluster:oppgavehandtering.oppgave mote /api/v1/oppgaver
+ */
 public abstract class AbstractOppgaveKlient implements Oppgaver {
 
     private static final String STATUSKATEGORI_AAPEN = "AAPEN";
+    private static final String TEMA_FORELDREPENGER = "FOR";
 
     private final RestClient restKlient;
     private final RestConfig restConfig;
@@ -36,18 +41,14 @@ public Oppgave opprettetOppgave(OpprettOppgave oppgave) {
     }
 
     @Override
-    public List<Oppgave> finnAlleOppgaver(String aktørId, String tema, List<String> oppgaveTyper) {
-        var builder = UriBuilder.fromUri(restConfig.endpoint());
-        if (aktørId != null) {
-            builder.queryParam("aktoerId", aktørId);
-        }
-        if (tema != null) {
-            builder.queryParam("tema", tema);
-        }
-        oppgaveTyper.forEach(ot -> builder.queryParam("oppgavetype", ot));
-        var request = RestRequest.newGET(builder.build(), restConfig)
-            .otherCallId(NavHeaders.HEADER_NAV_CORRELATION_ID);
-        return restKlient.send(addCorrelation(request), FinnOppgaveResponse.class).oppgaver();
+    public List<Oppgave> finnÅpneOppgaver(String aktørId, String tema, List<String> oppgaveTyper) {
+        return hentOppgaverFor(aktørId, tema, oppgaveTyper, null, null);
+    }
+
+    @Override
+    public List<Oppgave> finnÅpneOppgaverAvType(Oppgavetype oppgaveType, String aktørId, String enhetsNr, String limit) {
+        Objects.requireNonNull(oppgaveType, "Oppgvetype er påkrevd");
+        return hentOppgaverFor(aktørId, TEMA_FORELDREPENGER, List.of(oppgaveType.getKode()), null, null);
     }
 
     @Override
@@ -56,8 +57,8 @@ public List<Oppgave> finnAlleOppgaver(String aktørId, String tema, List<String>
     }
 
     @Override
-    public List<Oppgave> finnÅpneOppgaver(String aktørId, String tema, List<String> oppgaveTyper) {
-        return hentOppgaverFor(aktørId, tema, oppgaveTyper, null, null);
+    public List<Oppgave> finnÅpneOppgaver(List<String> oppgaveTyper, String aktørId, String enhetsNr, String limit) {
+        return hentOppgaverFor(aktørId, TEMA_FORELDREPENGER, oppgaveTyper, enhetsNr, limit);
     }
 
     private List<Oppgave> hentOppgaverFor(String aktørId, String tema, List<String> oppgaveTyper, String tildeltEnhetsnr, String limit) {

diff --git a/...pgave-rest-klient/src/main/java/no/nav/vedtak/felles/integrasjon/oppgave/v1/Oppgaver.java b/...pgave-rest-klient/src/main/java/no/nav/vedtak/felles/integrasjon/oppgave/v1/Oppgaver.java
@@ -4,17 +4,26 @@
 
 public interface Oppgaver {
 
+    Oppgave opprettetOppgave(OpprettOppgave oppgave);
+
     Oppgave hentOppgave(String oppgaveId);
 
     void feilregistrerOppgave(String oppgaveId);
 
     void ferdigstillOppgave(String oppgaveId);
 
-    List<Oppgave> finnÅpneOppgaver(String aktørId, String tema, List<String> oppgaveTyper) throws Exception;
-    List<Oppgave> finnÅpneOppgaverForEnhet(String tema, List<String> oppgaveTyper, String tildeltEnhetsnr, String limit) throws Exception;
+    @Deprecated(forRemoval = true)
+    List<Oppgave> finnÅpneOppgaver(String aktørId, String tema, List<String> oppgaveTyper);
 
-    List<Oppgave> finnAlleOppgaver(String aktørId, String tema, List<String> oppgaveTyper) throws Exception;
+    // Henter åpne oppgaver med Tema-kode Foreldrepenger for gitt Oppgavetyper.
+    // Øvrige argumenter kan være null,  men hvis det ventes mange oppgaver så sett aktør, enhet eller limit
+    List<Oppgave> finnÅpneOppgaverAvType(Oppgavetype oppgaveType, String aktørId, String enhetsNr, String limit);
 
-    Oppgave opprettetOppgave(OpprettOppgave oppgave);
+    @Deprecated(forRemoval = true)
+    List<Oppgave> finnÅpneOppgaverForEnhet(String tema, List<String> oppgaveTyper, String enhetsNr, String limit);
+
+    // Henter åpne oppgaver med Tema-kode Foreldrepenger for gitte Opgavetyper-kodeverk (kan være tom).
+    // Øvrige argumenter kan være null, men hvis det ventes mange oppgaver så sett aktør, enhet eller limit
+    List<Oppgave> finnÅpneOppgaver(List<String> oppgaveTyper, String aktørId, String enhetsNr, String limit);
 
 }
diff --git a/...ve-rest-klient/src/main/java/no/nav/vedtak/felles/integrasjon/oppgave/v1/Oppgavetype.java b/...ve-rest-klient/src/main/java/no/nav/vedtak/felles/integrasjon/oppgave/v1/Oppgavetype.java
@@ -4,17 +4,13 @@
 
 public enum Oppgavetype {
 
-    BEHANDLE_SAK("BEH_SAK_VL"),
-    REVURDER("RV_VL"),
-    GODKJENNE_VEDTAK("GOD_VED_VL"),
-    REGISTRER_SØKNAD("REG_SOK_VL"),
     JOURNALFØRING("JFR"),
     VURDER_KONSEKVENS_YTELSE("VUR_KONS_YTE"),
-    VURDER_DOKUMENT("VUR_VL"),
+    VURDER_DOKUMENT("VUR"),
+    BEHANDLE_SAK("BEH_SAK"),
     FEILUTBETALING("FEILUTBET"),
     INNHENT_DOK("INNH_DOK"),
-    SETT_UTBETALING_VENT("SETTVENT"),
-    BEHANDLE_SAK_INFOTRYGD("BEH_SAK");
+    SETT_UTBETALING_VENT("SETTVENT");
 
 
     @JsonValue

diff --git a/...est-klient/src/test/java/no/nav/vedtak/felles/integrasjon/oppgave/v1/OppgaveRestTest.java b/...est-klient/src/test/java/no/nav/vedtak/felles/integrasjon/oppgave/v1/OppgaveRestTest.java
@@ -4,16 +4,10 @@
 
 import org.junit.jupiter.api.Test;
 
-import com.fasterxml.jackson.databind.ObjectReader;
-import com.fasterxml.jackson.databind.ObjectWriter;
-
 import no.nav.vedtak.mapper.json.DefaultJsonMapper;
 
 class OppgaveRestTest {
 
-    private static final ObjectWriter WRITER = DefaultJsonMapper.getObjectMapper().writerWithDefaultPrettyPrinter();
-    private static final ObjectReader READER = DefaultJsonMapper.getObjectMapper().reader();
-
     private static final String json = """
         {
           "id": 357736794,
@@ -25,7 +19,7 @@ class OppgaveRestTest {
           "temagruppe":"FMLI",
           "tema":"FOR",
           "behandlingstema":"ab0326",
-          "oppgavetype":"BEH_SAK_VL",
+          "oppgavetype":"BEH_SAK",
           "versjon":1,
           "opprettetAv":"srvengangsstonad",
           "prioritet":"NORM",
@@ -42,5 +36,6 @@ void test_response() throws Exception {
         var deserialized = DefaultJsonMapper.fromJson(json, Oppgave.class);
         assertThat(deserialized).isNotNull();
         assertThat(deserialized.tildeltEnhetsnr()).isEqualTo("1234");
+        assertThat(deserialized.oppgavetype()).isEqualTo(Oppgavetype.BEHANDLE_SAK.getKode());
     }
 }