FEATURE: update krav priority by etterlevelse request and merge funct…

…ion to only update krav priority list for etterlevelse dokumentasjon
navikt · Aug 14, 2024 · b6c5e16 · b6c5e16
1 parent 776cb59
commit b6c5e16
Show file tree

Hide file tree

Showing 5 changed files with 66 additions and 2 deletions.
diff --git a/apps/backend/src/main/java/no/nav/data/etterlevelse/etterlevelse/EtterlevelseController.java b/apps/backend/src/main/java/no/nav/data/etterlevelse/etterlevelse/EtterlevelseController.java
@@ -111,8 +111,10 @@ public ResponseEntity<EtterlevelseResponse> createEtterlevelse(@RequestBody Ette
             throw new ValidationException("Tried to create etterlevelse with old architecture");
         }
 
-        var krav = service.save(request);
-        return new ResponseEntity<>(krav.toResponse(), HttpStatus.CREATED);
+        etterlevelseDokumentasjonService.updatePriorityList(request.getEtterlevelseDokumentasjonId(), request.getKravNummer(), request.isPrioritised());
+
+        var etterlevelse = service.save(request);
+        return new ResponseEntity<>(etterlevelse.toResponse(), HttpStatus.CREATED);
     }
 
     @Operation(summary = "Update Etterlevelse")
@@ -128,6 +130,8 @@ public ResponseEntity<EtterlevelseResponse> updateEtterlevelse(@PathVariable UUI
             throw new ValidationException("Tried to create etterlevelse with old architecture");
         }
 
+        etterlevelseDokumentasjonService.updatePriorityList(request.getEtterlevelseDokumentasjonId(), request.getKravNummer(), request.isPrioritised());
+
         var etterlevelse = service.save(request);
         return ResponseEntity.ok(etterlevelse.toResponse());
     }

diff --git a/.../backend/src/main/java/no/nav/data/etterlevelse/etterlevelse/dto/EtterlevelseRequest.java b/.../backend/src/main/java/no/nav/data/etterlevelse/etterlevelse/dto/EtterlevelseRequest.java
@@ -39,6 +39,7 @@ public class EtterlevelseRequest implements RequestElement, KravId {
     private List<SuksesskriterieBegrunnelseRequest> suksesskriterieBegrunnelser;
 
     private Boolean update;
+    private boolean prioritised;
 
     @Override
     public void format() {

diff --git a/.../no/nav/data/etterlevelse/etterlevelseDokumentasjon/EtterlevelseDokumentasjonService.java b/.../no/nav/data/etterlevelse/etterlevelseDokumentasjon/EtterlevelseDokumentasjonService.java
@@ -160,6 +160,27 @@ public EtterlevelseDokumentasjon delete(UUID id) {
         return storage.delete(id);
     }
 
+
+    @Transactional(propagation = Propagation.REQUIRED)
+    public EtterlevelseDokumentasjon updatePriorityList(String etterlevelseDokumentasjonId, int kravNummer, boolean prioritised) {
+        EtterlevelseDokumentasjon etterlevelseDokumentasjon = get(UUID.fromString(etterlevelseDokumentasjonId));
+
+        List<String> priorityList = etterlevelseDokumentasjon.getPrioritertKravNummer();
+        if (prioritised) {
+            if (!priorityList.contains(String.valueOf(kravNummer))) {
+                priorityList.add(String.valueOf(kravNummer));
+            }
+        } else {
+            if (priorityList.contains(String.valueOf(kravNummer))) {
+                priorityList = priorityList.stream().filter(number -> !number.equals(String.valueOf(kravNummer))).toList();
+            }
+        }
+        etterlevelseDokumentasjon.mergePrioritertKravNummer(priorityList);
+        return storage.save(etterlevelseDokumentasjon);
+    }
+
+    ;
+
     public List<EtterlevelseDokumentasjon> getByBehandlingId(List<String> ids) {
         return convertToDomaionObject(etterlevelseDokumentasjonRepo.findByBehandlingIds(ids));
     }

diff --git a/.../no/nav/data/etterlevelse/etterlevelseDokumentasjon/domain/EtterlevelseDokumentasjon.java b/.../no/nav/data/etterlevelse/etterlevelseDokumentasjon/domain/EtterlevelseDokumentasjon.java
@@ -74,6 +74,10 @@ public void merge(EtterlevelseDokumentasjonRequest request) {
         varslingsadresser = copyOf(request.getVarslingsadresser());
     }
 
+    public void mergePrioritertKravNummer(List<String> newPrioritertKravNummer) {
+        prioritertKravNummer = newPrioritertKravNummer;
+    }
+
     public EtterlevelseDokumentasjonResponse toResponse() {
         return EtterlevelseDokumentasjonResponse.builder()
                 .id(id)

diff --git a/.../java/no/nav/data/etterlevelse/etterlevelseDokumentasjon/EtterlevelseDokumentasjonIT.java b/.../java/no/nav/data/etterlevelse/etterlevelseDokumentasjon/EtterlevelseDokumentasjonIT.java
@@ -5,12 +5,15 @@
 import no.nav.data.etterlevelse.codelist.CodelistStub;
 import no.nav.data.etterlevelse.etterlevelse.domain.Etterlevelse;
 import no.nav.data.etterlevelse.etterlevelseDokumentasjon.domain.EtterlevelseDokumentasjon;
+import no.nav.data.etterlevelse.etterlevelseDokumentasjon.dto.EtterlevelseDokumentasjonRequest;
 import no.nav.data.etterlevelse.etterlevelseDokumentasjon.dto.EtterlevelseDokumentasjonResponse;
 import no.nav.data.etterlevelse.etterlevelsemetadata.domain.EtterlevelseMetadata;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.springframework.http.HttpStatus;
 
+import java.util.List;
+
 import static org.assertj.core.api.Assertions.assertThat;
 
 public class EtterlevelseDokumentasjonIT extends IntegrationTestBase {
@@ -70,5 +73,36 @@ void deleteEtterlevelseDokumentasjonAndChildren() {
 
     }
 
+    @Test
+    void shouldOnlyUpdatePrioritertKravNummerWithoutOverwrittingOtherFields() {
+        var etterlevelseDokumentasjon_1 = etterlevelseDokumentasjonService.save(
+                EtterlevelseDokumentasjonRequest.builder()
+                        .title("test dokumentasjon")
+                        .etterlevelseNummer(101)
+                        .knyttetTilVirkemiddel(false)
+                        .virkemiddelId("")
+                        .beskrivelse("")
+                        .forGjenbruk(false)
+                        .teams(List.of(""))
+                        .resources(List.of(""))
+                        .risikoeiere(List.of(""))
+                        .irrelevansFor(List.of(""))
+                        .update(false)
+                        .behandlerPersonopplysninger(true)
+                        .behandlingIds(List.of(""))
+                        .prioritertKravNummer(List.of())
+                        .varslingsadresser(List.of())
+                        .build()
+        );
+
+        etterlevelseDokumentasjon_1.mergePrioritertKravNummer(List.of("test"));
+
+        var updatedEtterlevelseDokumentasjon = etterlevelseDokumentasjonStorageService.save(etterlevelseDokumentasjon_1);
+
+        assertThat(updatedEtterlevelseDokumentasjon.getId()).isEqualTo(etterlevelseDokumentasjon_1.getId());
+        assertThat(updatedEtterlevelseDokumentasjon.getTitle()).isEqualTo(etterlevelseDokumentasjon_1.getTitle());
+        assertThat(updatedEtterlevelseDokumentasjon.getPrioritertKravNummer()).isEqualTo(List.of("test"));
+    }
+
 
 }