Refactor 'foedsel' to 'foedselsdato' in PdlPerson

Renamed the 'foedsel' field to 'foedselsdato' across multiple files and classes. This change updates both the JSON structure and Java classes to reflect the new field name for consistency and clarity. Removed deprecated birth-related fields and adjusted corresponding tests.
navikt · Sep 2, 2024 · d46a088 · d46a088
1 parent a9335bc
commit d46a088
Show file tree

Hide file tree

Showing 15 changed files with 992 additions and 52 deletions.
diff --git a/...java/no/nav/testnav/apps/syntvedtakshistorikkservice/consumer/response/pdl/PdlPerson.java b/...java/no/nav/testnav/apps/syntvedtakshistorikkservice/consumer/response/pdl/PdlPerson.java
@@ -61,7 +61,7 @@ public static class HentPerson {
 
         private List<Boadresse> bostedsadresse;
         private List<Navn> navn;
-        private List<Foedsel> foedsel;
+        private List<Foedselsdato> foedselsdato;
         private List<Doedsfall> doedsfall;
 
         public List<Boadresse> getBostedsadresse() {
@@ -78,11 +78,11 @@ public List<Navn> getNavn() {
             return navn;
         }
 
-        public List<Foedsel> getFoedsel() {
-            if (isNull(foedsel)) {
-                foedsel = new ArrayList<>();
+        public List<Foedselsdato> getFoedselsdato() {
+            if (isNull(foedselsdato)) {
+                foedselsdato = new ArrayList<>();
             }
-            return foedsel;
+            return foedselsdato;
         }
 
         public List<Doedsfall> getDoedsfall() {
@@ -140,13 +140,10 @@ public static class Navn {
     @lombok.Data
     @NoArgsConstructor
     @AllArgsConstructor
-    public static class Foedsel {
+    public static class Foedselsdato {
 
         private String foedselsaar;
         private LocalDate foedselsdato;
-        private String foedeland;
-        private String foedested;
-        private String foedekommune;
         private Folkeregistermetadata folkeregistermetadata;
         private Metadata metadata;
     }

diff --git a/...rc/main/java/no/nav/testnav/apps/syntvedtakshistorikkservice/service/ArenaAapService.java b/...rc/main/java/no/nav/testnav/apps/syntvedtakshistorikkservice/service/ArenaAapService.java
@@ -85,7 +85,7 @@ public void opprettVedtakUngUfoer(
             String miljoe,
             List<RettighetRequest> rettigheter
     ) {
-        var foedselsdato = person.getFoedsel().getFoedselsdato();
+        var foedselsdato = person.getFoedselsdato().getFoedselsdato();
         var ungUfoer = historikk.getUngUfoer();
         if (nonNull(ungUfoer) && !ungUfoer.isEmpty()) {
             for (var vedtak : ungUfoer) {

diff --git a/...n/java/no/nav/testnav/apps/syntvedtakshistorikkservice/service/ArenaDagpengerService.java b/...n/java/no/nav/testnav/apps/syntvedtakshistorikkservice/service/ArenaDagpengerService.java
@@ -70,7 +70,7 @@ public Map<String, List<DagpengerResponseDTO>> registrerArenaBrukereMedDagpenger
 
         Map<String, List<DagpengerResponseDTO>> responses = new HashMap<>();
         for (var ident : utvalgteIdenter) {
-            var foedselsdato = ident.getFoedsel().getFoedselsdato();
+            var foedselsdato = ident.getFoedselsdato().getFoedselsdato();
 
             var minDate = foedselsdato.plusYears(18).isAfter(MINIMUM_DATE) ? foedselsdato.plusYears(18) : MINIMUM_DATE;
 
@@ -113,7 +113,7 @@ private List<DagpengerResponseDTO> sendDagpenger(String ident, String miljoe, Lo
         var soknadResponse = arenaForvalterService.opprettMottaDagpengesoknad(soknadRequest);
 
         if (soknadResponse.getFeiledeDagpenger().isEmpty() && !soknadResponse.getNyeDagpenger().isEmpty() && nonNull(vedtakdato)) {
-            var vedtakRequest = getDagpengevedtakRequest(ident, miljoe, vedtakdato, rettighetKode, soknadResponse.getNyeDagpenger().get(0));
+            var vedtakRequest = getDagpengevedtakRequest(ident, miljoe, vedtakdato, rettighetKode, soknadResponse.getNyeDagpenger().getFirst());
             if (isNull(vedtakRequest)) {
                 return Collections.singletonList(soknadResponse);
             }

diff --git a/...e/src/main/java/no/nav/testnav/apps/syntvedtakshistorikkservice/service/IdentService.java b/...e/src/main/java/no/nav/testnav/apps/syntvedtakshistorikkservice/service/IdentService.java
@@ -137,11 +137,11 @@ private boolean validBarn(List<String> barn, LocalDate tidligsteDatoBarnetillegg
 
     private boolean under18VedTidspunkt(PdlPersonBolk.PersonBolk personBolk, LocalDate tidspunkt) {
         var person = personBolk.getPerson();
-        if (nonNull(person) && nonNull(person.getFoedsel()) && !person.getFoedsel().isEmpty()){
-            var foedselsdato = person.getFoedsel().get(0).getFoedselsdato();
+        if (nonNull(person) && nonNull(person.getFoedselsdato()) && !person.getFoedselsdato().isEmpty()){
+            var foedselsdato = person.getFoedselsdato().getFirst().getFoedselsdato();
 
             if(nonNull(person.getDoedsfall()) && !person.getDoedsfall().isEmpty()){
-                var doedsdato = person.getDoedsfall().get(0).getDoedsdato();
+                var doedsdato = person.getDoedsfall().getFirst().getDoedsdato();
                 if (doedsdato.isBefore(tidspunkt.plusDays(1))) return false;
             }
             var alder = Math.toIntExact(ChronoUnit.YEARS.between(foedselsdato, tidspunkt));
@@ -160,19 +160,19 @@ public Kontoinfo getIdentMedKontoinformasjon() {
 
         return Kontoinfo.builder()
                 .fnr(ident.getIdent())
-                .fornavn(navnInfo.isEmpty() ? "" : navnInfo.get(0).getFornavn())
-                .mellomnavn(navnInfo.isEmpty() || isNull(navnInfo.get(0).getMellomnavn()) ? "" : navnInfo.get(0).getMellomnavn())
-                .etternavn(navnInfo.isEmpty() ? "" : navnInfo.get(0).getEtternavn())
+                .fornavn(navnInfo.isEmpty() ? "" : navnInfo.getFirst().getFornavn())
+                .mellomnavn(navnInfo.isEmpty() || isNull(navnInfo.getFirst().getMellomnavn()) ? "" : navnInfo.getFirst().getMellomnavn())
+                .etternavn(navnInfo.isEmpty() ? "" : navnInfo.getFirst().getEtternavn())
                 .kontonummer(ident.getKontonummer())
                 .adresseLinje1(getAdresseLinje(boadresseInfo))
-                .postnr(boadresseInfo.isEmpty() ? "" : boadresseInfo.get(0).getVegadresse().getPostnummer())
+                .postnr(boadresseInfo.isEmpty() ? "" : boadresseInfo.getFirst().getVegadresse().getPostnummer())
                 .landkode("NO")
                 .build();
     }
 
     private String getAdresseLinje(List<PdlPerson.Boadresse> boadresse) {
-        if (boadresse.isEmpty() || isNull(boadresse.get(0).getVegadresse())) return "";
-        var vegadresse = boadresse.get(0).getVegadresse();
+        if (boadresse.isEmpty() || isNull(boadresse.getFirst().getVegadresse())) return "";
+        var vegadresse = boadresse.getFirst().getVegadresse();
         var husbokstav = isNull(vegadresse.getHusbokstav()) ? "" : vegadresse.getHusbokstav();
         return vegadresse.getAdressenavn() + " " + vegadresse.getHusnummer() + husbokstav;
     }

diff --git a/...src/main/java/no/nav/testnav/apps/syntvedtakshistorikkservice/service/PensjonService.java b/...src/main/java/no/nav/testnav/apps/syntvedtakshistorikkservice/service/PensjonService.java
@@ -37,7 +37,7 @@ private boolean opprettPersonIPopp(
     ) {
         var opprettPersonStatus = pensjonTestdataFacadeConsumer.opprettPerson(PensjonTestdataPerson.builder()
                 .bostedsland("NOR")
-                .fodselsDato(person.getFoedsel().getFoedselsdato())
+                .fodselsDato(person.getFoedselsdato().getFoedselsdato())
                 .miljoer(Collections.singletonList(miljoe))
                 .fnr(person.getIdent())
                 .build());

diff --git a/apps/synt-vedtakshistorikk-service/src/main/resources/application-dev.yml b/apps/synt-vedtakshistorikk-service/src/main/resources/application-dev.yml
@@ -1,8 +1,3 @@
-azure:
-  app:
-    client:
-      id: ${client_id}
-      secret: ${client_secret}
 
 batch:
   miljoe: q2

diff --git a/apps/synt-vedtakshistorikk-service/src/main/resources/pdlperson/graphql.config.yml b/apps/synt-vedtakshistorikk-service/src/main/resources/pdlperson/graphql.config.yml
@@ -0,0 +1,8 @@
+schema: pdl-api-schema.graphql
+extensions:
+  endpoints:
+    PDL GraphQL Endpoint:
+      url: https://pdl-api.dev.intern.nav.no/graphql
+      headers:
+        user-agent: ''
+      introspect: false