Nytt oppsett for kall mot nytt medl endepunkt (#726)

* nytt medl-endepunkt, ident fra header og parametere flyttet inn i request body

* endrer junit tester til å mocke kall med riktig oppsett mot riktig medl endepunkt

* fikser på request body

* Setter contentType til json i medl kall

* fikser request body til riktig format

* fikser riktig request body i mocking i tester

* Fjerner ubrukt dato parameter i medl

* Fjerner ubrukte imports

---------

Co-authored-by: Stian S. Douzette <[email protected]>

Author: StianSDouzette

src/main/kotlin/no/nav/medlemskap/clients/medl/MedlClient.kt

@@ -24,18 +24,21 @@ class MedlClient(
 
     private val logger = KotlinLogging.logger { }
 
-    suspend fun hentMedlemskapsunntak(ident: String, callId: String, fraOgMed: LocalDate? = null, tilOgMed: LocalDate? = null): List<MedlMedlemskapsunntak> {
+    suspend fun hentMedlemskapsunntak(
+        ident: String,
+        callId: String
+    ): List<MedlMedlemskapsunntak> {
         val token = azureAdClient.hentToken(configuration.register.medlScope)
         return runCatching {
+            val medlQuery = "{\"personident\": \"${ident}\"}"
             runWithRetryAndMetrics<List<MedlMedlemskapsunntak>>("Medl", "MedlemskapsunntakV1", retry) {
-                httpClient.get() {
-                    url("$baseUrl/api/v1/medlemskapsunntak")
+                httpClient.post() {
+                    url("$baseUrl/rest/v1/periode/soek")
                     header(HttpHeaders.Authorization, "Bearer ${token.token}")
                     header(HttpHeaders.Accept, ContentType.Application.Json)
+                    header(HttpHeaders.ContentType, ContentType.Application.Json)
                     header("Nav-Call-Id", callId)
-                    header("Nav-Personident", ident)
-                    fraOgMed?.let { parameter("fraOgMed", fraOgMed.tilIsoFormat()) }
-                    tilOgMed?.let { parameter("tilOgMed", tilOgMed.tilIsoFormat()) }
+                    setBody(medlQuery)
                 }.body()
             }
         }.fold(
@@ -52,6 +55,7 @@ class MedlClient(
                             throw error
                         }
                     }
+
                     else -> throw error
                 }
             }
@@ -68,4 +72,4 @@ class MedlClient(
             header(HttpHeaders.Accept, ContentType.Application.Json)
         }
     }
-}
+}

src/main/kotlin/no/nav/medlemskap/services/medl/MedlService.kt

@@ -5,6 +5,6 @@ import java.time.LocalDate
 
 class MedlService(private val medlClient: MedlClient) {
 
-    suspend fun hentMedlemskapsunntak(ident: String, callId: String, fraOgMed: LocalDate? = null, tilOgMed: LocalDate? = null) =
-        mapMedlemskapResultat(medlClient.hentMedlemskapsunntak(ident, callId, fraOgMed, tilOgMed))
+    suspend fun hentMedlemskapsunntak(ident: String, callId: String) =
+        mapMedlemskapResultat(medlClient.hentMedlemskapsunntak(ident, callId))
 }

src/test/kotlin/no/nav/medlemskap/clients/medl/medlClientTest.kt

@@ -60,7 +60,7 @@ class medlClientTest {
         )
 
         val client = createMedlClient(azureAdClient)
-        val response = runBlocking { client.hentMedlemskapsunntak("10109000398", callId, LocalDate.of(2010, 1, 1), LocalDate.of(2016, 1, 1)) }
+        val response = runBlocking { client.hentMedlemskapsunntak("10109000398", callId) }
 
         Assertions.assertEquals("Full", response[0].dekning)
         Assertions.assertEquals(100064642, response[0].unntakId)
@@ -108,7 +108,7 @@ class medlClientTest {
         val client = createMedlClient(azureAdClient)
 
         Assertions.assertThrows(ServerResponseException::class.java) {
-            runBlocking { client.hentMedlemskapsunntak("10109000398", callId, LocalDate.of(2010, 1, 1), LocalDate.of(2016, 1, 1)) }
+            runBlocking { client.hentMedlemskapsunntak("10109000398", callId) }
         }
     }
 
@@ -131,7 +131,7 @@ class medlClientTest {
         val client = createMedlClient(azureAdClient)
 
         Assertions.assertThrows(ClientRequestException::class.java) {
-            runBlocking { client.hentMedlemskapsunntak("10109000398", callId, LocalDate.of(2010, 1, 1), LocalDate.of(2016, 1, 1)) }
+            runBlocking { client.hentMedlemskapsunntak("10109000398", callId) }
         }
     }
 
@@ -151,7 +151,7 @@ class medlClientTest {
         )
 
         val client = createMedlClient(azureAdClient)
-        val response = runBlocking { client.hentMedlemskapsunntak("10109000398", callId, LocalDate.of(2010, 1, 1), LocalDate.of(2016, 1, 1)) }
+        val response = runBlocking { client.hentMedlemskapsunntak("10109000398", callId) }
 
         Assertions.assertEquals(0, response.size)
     }
@@ -160,12 +160,11 @@ class medlClientTest {
         return MedlClient(server.baseUrl(), azureAdClient, config, cioHttpClient)
     }
 
-    private val queryMapping: MappingBuilder = WireMock.get(WireMock.urlPathEqualTo("/api/v1/medlemskapsunntak"))
+    private val medlQuery = "{\"personident\": \"10109000398\"}"
+    private val queryMapping: MappingBuilder = WireMock.post(WireMock.urlPathEqualTo("/rest/v1/periode/soek"))
         .withHeader(HttpHeaders.Authorization, equalTo("Bearer dummytoken"))
-        .withHeader("Nav-Personident", equalTo("10109000398"))
         .withHeader("Nav-Call-Id", equalTo("12345"))
-        .withQueryParam("fraOgMed", equalTo("2010-01-01"))
-        .withQueryParam("fraOgMed", equalTo("2010-01-01"))
+        .withRequestBody(equalTo(medlQuery))
 
     private val medlResponse =
         """