Refactor AaregUtility and Arbeidsavtale: enhance equality checks and improve data comparison logic #deploy-test-dolly-backend

Author: krharum

apps/dolly-backend/src/main/java/no/nav/dolly/bestilling/aareg/util/AaregUtility.java

@@ -35,13 +35,13 @@ public static ArbeidsforholdEksistens doEksistenssjekk(ArbeidsforholdRespons res
         return ArbeidsforholdEksistens.builder()
                 .nyeArbeidsforhold(request.stream()
                         .filter(arbeidsforhold -> response.getEksisterendeArbeidsforhold().stream()
-                                        .noneMatch(response1 ->
-                                                isEqualArbeidsforhold(response1, arbeidsforhold)) ||
+                                .noneMatch(response1 ->
+                                        isEqualArbeidsforhold(response1, arbeidsforhold)) ||
                                 isNotTrue(arbeidsforhold.getIsOppdatering()) && isOpprettEndre)
                         .toList())
                 .eksisterendeArbeidsforhold(request.stream()
                         .filter(arbeidsforhold -> response.getEksisterendeArbeidsforhold().stream()
-                                        .anyMatch(response1 -> isEqualArbeidsforhold(response1, arbeidsforhold))
+                                .anyMatch(response1 -> isEqualArbeidsforhold(response1, arbeidsforhold))
                                 && (isTrue(arbeidsforhold.getIsOppdatering()) || !isOpprettEndre))
                         .toList())
                 .ubestemmeligArbeidsforhold(request.stream()
@@ -91,8 +91,10 @@ private static boolean isArbeidsgiverOrganisasjonAlike(Arbeidsforhold arbeidsfor
 
     private static boolean isArbeidsforholdAlike(Arbeidsforhold arbeidsforhold1, Arbeidsforhold arbeidsforhold2) {
 
-        return arbeidsforhold1.getArbeidsavtaler().stream()
-                .allMatch(arbeidsavtale -> arbeidsforhold2.getArbeidsavtaler().stream()
-                .anyMatch(arbeidsavtale::equals));
+        return (isBlank(arbeidsforhold1.getArbeidsforholdId()) ||
+                isBlank(arbeidsforhold2.getArbeidsforholdId())) &&
+                arbeidsforhold1.getArbeidsavtaler().stream()
+                        .allMatch(arbeidsavtale -> arbeidsforhold2.getArbeidsavtaler().stream()
+                                .anyMatch(arbeidsavtale::equals));
     }
 }

apps/dolly-backend/src/test/java/no/nav/dolly/bestilling/aareg/util/AaaregUtilityTest.java

@@ -33,10 +33,11 @@ private static Arbeidsforhold getArbeidsforhold(String arbeidsforholdId, Boolean
     }
 
     @ParameterizedTest
-    @CsvSource({"1, false, 1, 0, 1",
+    @CsvSource(value = {"1, false, 1, 0, 1",
             "1, true, 1, 1, 0",
             "2, false, 1, 0, 1",
-            "2, true, 1, 0, 1"})
+            "2, true, 1, 0, 1",
+            "null, true, 1, 1, 0"}, nullValues = "null")
     void arbeidsforholdEksistens_eksisterIkke(String reqArbForholdId, Boolean isOppdatering, String respArbForholdId,
                                               int antallEksisterende, int antallNye) {