Merge pull request #78 from navikt/mp-396-korrigering

Endret navn på variabel og slettet unødvendig sjekk på null
navikt · Aug 29, 2022 · 1e2e5db · 1e2e5db
2 parents 31c4413 + e89d229
commit 1e2e5db
Show file tree

Hide file tree

Showing 2 changed files with 39 additions and 10 deletions.
diff --git a/src/main/kotlin/no/nav/meldeplikt/meldekortservice/service/ArenaOrdsService.kt b/src/main/kotlin/no/nav/meldeplikt/meldekortservice/service/ArenaOrdsService.kt
@@ -66,16 +66,20 @@ class ArenaOrdsService(
     }
 
     suspend fun kopierMeldekort(meldekortId: Long): Long {
-        val nyMeldekortId = ordsClient.post<String>("${env.ordsUrl}$ARENA_ORDS_KOPIER_MELDEKORT") {
-            setupOrdsRequest(meldekortId)
-        }
-        if (nyMeldekortId != null)
-        {
-            defaultLog.info("Meldekort med id " + mapFraXml(nyMeldekortId, KopierMeldekortResponse::class.java).meldekortId + " er opprettet for korrigering. Kopiert fra meldekort med id " + meldekortId)
-        } else {
-            defaultLog.warn("Feil ved opprettelse av meldekort for korrigering! Meldekort med id " + meldekortId + " har ikke blitt kopiert.")
+        try {
+            val responseMedNyMeldekortId = ordsClient.post<String>("${env.ordsUrl}$ARENA_ORDS_KOPIER_MELDEKORT") {
+                setupOrdsRequest(meldekortId)
+            }
+
+            val nyMeldekortId = mapFraXml(responseMedNyMeldekortId, KopierMeldekortResponse::class.java).meldekortId
+            defaultLog.info("Meldekort med id $nyMeldekortId er opprettet for korrigering. Kopiert fra meldekort med id $meldekortId")
+            return nyMeldekortId
+
+        } catch (e: Exception) {
+            defaultLog.warn("Feil ved opprettelse av meldekort for korrigering! Meldekort med id $meldekortId har ikke blitt kopiert.", e)
         }
-        return mapFraXml(nyMeldekortId, KopierMeldekortResponse::class.java).meldekortId
+
+        return 0
     }
 
     private fun HttpRequestBuilder.setupOrdsRequest(meldekortId: Long? = null) {

diff --git a/src/test/kotlin/no/nav/meldeplikt/meldekortservice/service/ArenaOrdsServiceTest.kt b/src/test/kotlin/no/nav/meldeplikt/meldekortservice/service/ArenaOrdsServiceTest.kt
@@ -183,4 +183,29 @@ class ArenaOrdsServiceTest {
         }
     }
 
-}
+    @Test
+    fun `test kopierMeldekort returns 0 hvis ikke OK`() {
+        val xmlString = """NOT XML STRING"""
+        val client = HttpClient(MockEngine) {
+            engine {
+                addHandler { request ->
+                    request.method shouldBe HttpMethod.Post
+                    request.headers["Authorization"] shouldNotBe null
+                    request.headers["Authorization"] shouldStartWith "Bearer $DUMMY_TOKEN"
+                    request.headers["meldekortId"] shouldBe "123"
+                    request.url.toString() shouldBe "https://dummyurl.nav.no/api/v1/meldeplikt/meldekort/kopi"
+                    respondOk(
+                        xmlString
+                    )
+                }
+            }
+        }
+        val arenaOrdsService = ArenaOrdsService(client)
+
+        runBlocking {
+            val actualResponse = arenaOrdsService.kopierMeldekort(123)
+
+            assertEquals(0, actualResponse)
+        }
+    }
+}