Refactor: Update AaregClientTest and AaregConsumerTest to enhance mock behavior and improve test setup #deploy-test-dolly-backend

Author: krharum

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

@@ -16,7 +16,6 @@
 import no.nav.testnav.libs.dto.aareg.v1.OrdinaerArbeidsavtale;
 import no.nav.testnav.libs.dto.aareg.v1.Organisasjon;
 import no.nav.testnav.libs.dto.aareg.v1.Person;
-import no.nav.testnav.libs.securitycore.domain.AccessToken;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.ExtendWith;
@@ -57,9 +56,6 @@ class AaregClientTest {
     @Mock
     private MapperFacade mapperFacade;
 
-    @Mock
-    private AccessToken accessToken;
-
     @Mock
     private Bruker bruker;
 
@@ -83,6 +79,7 @@ void setup() {
     private static ArbeidsforholdRespons buildArbeidsforhold(boolean isOrgnummer) {
 
         return ArbeidsforholdRespons.builder()
+                .miljo(ENV)
                 .eksisterendeArbeidsforhold(singletonList(
                         Arbeidsforhold.builder()
                                 .arbeidstaker(Person.builder()
@@ -140,6 +137,8 @@ void gjenopprettArbeidsforhold_tidligereArbeidsforholdFinnesAktoerPerson_returns
                         .build()));
         when(mapperFacade.mapAsList(anyList(), eq(Arbeidsforhold.class)))
                 .thenReturn(buildArbeidsforhold(false).getEksisterendeArbeidsforhold());
+        when(mapperFacade.map(any(), eq(Arbeidsforhold.class), any()))
+                .thenReturn(buildArbeidsforhold(false).getEksisterendeArbeidsforhold().getFirst());
 
         StepVerifier.create(aaregClient.gjenopprett(request,
                                 DollyPerson.builder().ident(IDENT)
@@ -175,6 +174,8 @@ void gjenopprettArbeidsforhold_tidligereArbeidsforholdFinnesAktoerOrganisasjon_r
                         .build()));
         when(mapperFacade.mapAsList(anyList(), eq(Arbeidsforhold.class)))
                 .thenReturn(buildArbeidsforhold(true).getEksisterendeArbeidsforhold());
+        when(mapperFacade.map(any(), eq(Arbeidsforhold.class), any()))
+                .thenReturn(buildArbeidsforhold(true).getEksisterendeArbeidsforhold().getFirst());
 
         StepVerifier.create(aaregClient.gjenopprett(request, DollyPerson.builder().ident(IDENT)
                                 .bruker(bruker)

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

@@ -9,11 +9,15 @@
 import no.nav.testnav.libs.dto.aareg.v1.Organisasjon;
 import no.nav.testnav.libs.dto.aareg.v1.Person;
 import no.nav.testnav.libs.securitycore.domain.AccessToken;
+import no.nav.testnav.libs.securitycore.domain.ServerProperties;
+import no.nav.testnav.libs.standalone.servletsecurity.exchange.TokenExchange;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
+import org.mockito.ArgumentMatchers;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.cloud.contract.wiremock.AutoConfigureWireMock;
 import org.springframework.test.context.bean.override.mockito.MockitoBean;
+import reactor.core.publisher.Mono;
 
 import java.util.List;
 
@@ -28,6 +32,7 @@
 import static java.util.Collections.emptyList;
 import static java.util.Collections.singletonList;
 import static org.assertj.core.api.AssertionsForClassTypes.assertThat;
+import static org.mockito.Mockito.when;
 
 @DollySpringBootTest
 @AutoConfigureWireMock(port = 0)
@@ -40,7 +45,7 @@ class AaregConsumerTest {
 
     @MockitoBean
     @SuppressWarnings("unused")
-    private AccessToken accessToken;
+    private TokenExchange tokenService;
 
     @Autowired
     private AaregConsumer aaregConsumer;
@@ -55,7 +60,7 @@ private static String asJsonString(final Object object) throws JsonProcessingExc
 
     @BeforeEach
     void setUp() {
-
+        when(tokenService.exchange(ArgumentMatchers.any(ServerProperties.class))).thenReturn(Mono.just(new AccessToken("token")));
         opprettRequest = Arbeidsforhold.builder()
                 .arbeidstaker(Person.builder()
                         .offentligIdent(IDENT)