Rename and refactor mapping strategy classes

#deploy-skattekort-service Renamed `TrekktypeMappingStrategy` to `ForskuddstrekkMappingStrategy` for better clarity. Updated variable names inside the class to align with the new name. Modified `Forskuddstrekk` class by adding `AllArgsConstructor` and changing `SuperBuilder` to `Builder`.
navikt · Aug 9, 2024 · 0e61095 · 0e61095
1 parent c4b30bd
commit 0e61095
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 20 deletions.
diff --git a/...rt-service/src/main/java/no/nav/skattekortservice/mapper/ArbeidsgiverMappingStrategy.java b/...rt-service/src/main/java/no/nav/skattekortservice/mapper/ArbeidsgiverMappingStrategy.java
@@ -40,16 +40,6 @@ public void mapAtoB(SkattekortResponsIntermediate.Skattekortmelding source, Skat
                 })
                 .register();
 
-//        factory.classMap(SkattekortResponsIntermediate.Forskuddstrekk.class, Forskuddstrekk.class)
-//                .customize(new CustomMapper<>() {
-//                    @Override
-//                    public void mapAtoB(SkattekortResponsIntermediate.Forskuddstrekk source, Forskuddstrekk target, MappingContext context) {
-//
-//                        target.setTrekkode(Trekkode.valueOf(source.getTrekkode().getValue()));
-//                    }
-//                })
-//                .register();
-
         factory.classMap(SkattekortResponsIntermediate.Frikort.class, Frikort.class)
                 .customize(new CustomMapper<>() {
                     @Override

diff --git a/...vice/mapper/TrekktypeMappingStrategy.java → ...mapper/ForskuddstrekkMappingStrategy.java b/...vice/mapper/TrekktypeMappingStrategy.java → ...mapper/ForskuddstrekkMappingStrategy.java
@@ -16,7 +16,7 @@
 import static java.util.Objects.nonNull;
 
 @Component
-public class TrekktypeMappingStrategy implements MappingStrategy {
+public class ForskuddstrekkMappingStrategy implements MappingStrategy {
     @Override
     public void register(MapperFactory factory) {
 
@@ -26,15 +26,15 @@ public void register(MapperFactory factory) {
                     public void mapAtoB(Skattekort skattekort, no.skatteetaten.fastsetting.formueinntekt.forskudd.skattekorttilarbeidsgiver.v3.Skattekort skattekort2, MappingContext context) {
 
                         skattekort2.getForskuddstrekk().addAll(skattekort.getForskuddstrekk().stream()
-                                .map(trekktype -> {
-                                    if (nonNull(trekktype.getFrikort())) {
-                                        return mapperFacade.map(trekktype.getFrikort(),
+                                .map(forskuddstrekk -> {
+                                    if (nonNull(forskuddstrekk.getFrikort())) {
+                                        return mapperFacade.map(forskuddstrekk.getFrikort(),
                                                 no.skatteetaten.fastsetting.formueinntekt.forskudd.skattekorttilarbeidsgiver.v3.Frikort.class);
-                                    } else if (nonNull(trekktype.getTrekktabell())) {
-                                        return mapperFacade.map(trekktype.getTrekktabell(),
+                                    } else if (nonNull(forskuddstrekk.getTrekktabell())) {
+                                        return mapperFacade.map(forskuddstrekk.getTrekktabell(),
                                                 no.skatteetaten.fastsetting.formueinntekt.forskudd.skattekorttilarbeidsgiver.v3.Trekktabell.class);
-                                    } else if (nonNull(trekktype.getTrekkprosent())) {
-                                        return mapperFacade.map(trekktype.getTrekkprosent(),
+                                    } else if (nonNull(forskuddstrekk.getTrekkprosent())) {
+                                        return mapperFacade.map(forskuddstrekk.getTrekkprosent(),
                                                 no.skatteetaten.fastsetting.formueinntekt.forskudd.skattekorttilarbeidsgiver.v3.Trekkprosent.class);
                                     } else {
                                         return null;

diff --git a/...er-objects/src/main/java/no/nav/testnav/libs/dto/skattekortservice/v1/Forskuddstrekk.java b/...er-objects/src/main/java/no/nav/testnav/libs/dto/skattekortservice/v1/Forskuddstrekk.java
@@ -1,16 +1,18 @@
 package no.nav.testnav.libs.dto.skattekortservice.v1;
 
 import com.fasterxml.jackson.annotation.JsonIgnore;
+import lombok.AllArgsConstructor;
+import lombok.Builder;
 import lombok.Data;
 import lombok.NoArgsConstructor;
-import lombok.experimental.SuperBuilder;
 
 import java.util.Arrays;
 import java.util.Objects;
 
 @Data
-@SuperBuilder
+@Builder
 @NoArgsConstructor
+@AllArgsConstructor
 public class Forskuddstrekk {
 
     private Frikort frikort;