Merge pull request #71 from navikt/MELDEPLIKT-330

Meldeplikt 330
navikt · Jul 5, 2022 · 94e44fc · 94e44fc
2 parents 1436816 + 2799ea3
commit 94e44fc
Show file tree

Hide file tree

Showing 4 changed files with 51 additions and 18 deletions.
diff --git a/src/main/kotlin/no/nav/meldeplikt/meldekortservice/config/Environment.kt b/src/main/kotlin/no/nav/meldeplikt/meldekortservice/config/Environment.kt
@@ -13,7 +13,6 @@ var DUMMY_TOKEN: String = JWT.create()
     .toString()
 
 data class Environment(
-    val ameldingUrl: URL = URL(getEnvVar("AMELDING_URI", "$DUMMY_URL/path")),
     val ordsUrl: URL = URL(getEnvVar("ORDS_URI", DUMMY_URL)),
     val ordsClientId: String = getEnvVar("CLIENT_ID", "cLiEnTiD"),
     val ordsClientSecret: String = getEnvVar("CLIENT_SECRET", "cLiEnTsEcReT"),
@@ -69,10 +68,10 @@ fun getEnvVar(varName: String, defaultValue: String? = null): String {
 }
 
 fun removeTrailingSlash(s: String): String {
-    if (s.endsWith("/")) {
-        return s.substring(0, s.length - 1);
+    return if (s.endsWith("/")) {
+        s.substring(0, s.length - 1)
     } else {
-        return s
+        s
     }
 }
 

diff --git a/src/main/kotlin/no/nav/meldeplikt/meldekortservice/mapper/KontrollertTypeMapper.kt b/src/main/kotlin/no/nav/meldeplikt/meldekortservice/mapper/KontrollertTypeMapper.kt
@@ -9,7 +9,7 @@ class KontrollertTypeMapper {
 
     // Vi ignorerer kontrollstatus her
     fun mapKontrollResponseToKontrollertType(message: KontrollResponse): MeldekortKontrollertType {
-        var kontroll = MeldekortKontrollertType()
+        val kontroll = MeldekortKontrollertType()
         kontroll.meldekortId = message.meldekortId
         kontroll.arsakskoder = trekkutArsakskoder(message)
         kontroll.meldekortDager = trekkutMeldekortDager(message)
@@ -21,9 +21,9 @@ class KontrollertTypeMapper {
     }
 
     private fun trekkutArsakskoder(message: KontrollResponse): MeldekortKontrollertType.Arsakskoder {
-        var uttrekk = MeldekortKontrollertType.Arsakskoder()
+        val uttrekk = MeldekortKontrollertType.Arsakskoder()
         message.feilListe.forEach {
-            var item = ArsakskodeType()
+            val item = ArsakskodeType()
             item.kode = it.kode
             item.tekst = it.tekst
             uttrekk.arsakskode.add(item)
@@ -32,9 +32,9 @@ class KontrollertTypeMapper {
     }
 
     private fun trekkutMeldekortDager(message: KontrollResponse): MeldekortKontrollertType.MeldekortDager {
-        var uttrekk = MeldekortKontrollertType.MeldekortDager()
+        val uttrekk = MeldekortKontrollertType.MeldekortDager()
         message.feilListe.forEach {
-            var item = MeldekortDagKontrollertType()
+            val item = MeldekortDagKontrollertType()
             item.dag = it.dag!!
             uttrekk.meldekortDag.add(item)
         }

diff --git a/src/test/kotlin/no/nav/meldeplikt/meldekortservice/mapper/KontrollertTypeMapperTest.kt b/src/test/kotlin/no/nav/meldeplikt/meldekortservice/mapper/KontrollertTypeMapperTest.kt
@@ -10,21 +10,38 @@ class KontrollertTypeMapperTest {
     private val kontrollertTypeMapper = KontrollertTypeMapper()
 
     @Test
-    fun testMapKontrollResponseToKontrollertType() {
-        var feilListe = mutableListOf<KontrollFeil>()
+    fun tmapKontrollResponseToKontrollertTypeSkalReturnereOk() {
+        val feilListe = mutableListOf<KontrollFeil>()
+
+        val fra = KontrollResponse(87576,  "OK", feilListe)
+        val til: MeldekortKontrollertType = kontrollertTypeMapper.mapKontrollResponseToKontrollertType(fra)
+
+        assert(til.meldekortId == fra.meldekortId)
+        assert(til.status == "OK")
+        assert(til.meldekortDager.meldekortDag.size == fra.feilListe.size)
+        assert(til.arsakskoder.arsakskode.size == fra.feilListe.size)
+    }
+
+    @Test
+    fun mapKontrollResponseToKontrollertTypeSkalReturnereFeil() {
+        val feilListe = mutableListOf<KontrollFeil>()
         val feil1 = KontrollFeil("X05", "Set sail for fail", 1)
         val feil2 = KontrollFeil("S09", "On the failboat", 3)
         feilListe.add(feil1)
         feilListe.add(feil2)
 
-        var fra = KontrollResponse(87576,  "OK", feilListe)
-        var til: MeldekortKontrollertType = kontrollertTypeMapper.mapKontrollResponseToKontrollertType(fra)
+        val fra = KontrollResponse(87576,  "OK", feilListe)
+        val til: MeldekortKontrollertType = kontrollertTypeMapper.mapKontrollResponseToKontrollertType(fra)
 
-        assert(til.arsakskoder.arsakskode.size == fra.feilListe.size)
         assert(til.meldekortId == fra.meldekortId)
         assert(til.status == "FEIL")
         assert(til.meldekortDager.meldekortDag.size == fra.feilListe.size)
-        assert(til.arsakskoder.arsakskode[0].kode == feil1.kode)
+        assert(til.meldekortDager.meldekortDag[0].dag == feil1.dag)
         assert(til.meldekortDager.meldekortDag[1].dag == feil2.dag)
+        assert(til.arsakskoder.arsakskode.size == fra.feilListe.size)
+        assert(til.arsakskoder.arsakskode[0].kode == feil1.kode)
+        assert(til.arsakskoder.arsakskode[0].tekst == feil1.tekst)
+        assert(til.arsakskoder.arsakskode[1].kode == feil2.kode)
+        assert(til.arsakskoder.arsakskode[1].tekst == feil2.tekst)
     }
 }
diff --git a/src/test/kotlin/no/nav/meldeplikt/meldekortservice/service/KontrollServiceTest.kt b/src/test/kotlin/no/nav/meldeplikt/meldekortservice/service/KontrollServiceTest.kt
@@ -12,6 +12,7 @@ import io.mockk.mockk
 import kotlinx.coroutines.runBlocking
 import no.nav.meldeplikt.meldekortservice.model.meldekortdetaljer.kontroll.FravaerInn
 import no.nav.meldeplikt.meldekortservice.model.meldekortdetaljer.kontroll.Meldekortkontroll
+import no.nav.meldeplikt.meldekortservice.model.meldekortdetaljer.kontroll.response.KontrollFeil
 import no.nav.meldeplikt.meldekortservice.model.meldekortdetaljer.kontroll.response.KontrollResponse
 import no.nav.meldeplikt.meldekortservice.utils.defaultObjectMapper
 import no.nav.meldeplikt.meldekortservice.utils.objectMapper
@@ -23,8 +24,24 @@ import kotlin.test.assertEquals
 class KontrollServiceTest {
 
     @Test
-    fun kontroller() {
+    fun kontrollerSkalReturnereOk() {
         val kontrollResponse = KontrollResponse(meldekortId = 123, kontrollStatus = "OK")
+
+        test(kontrollResponse, "OK")
+    }
+
+    @Test
+    fun kontrollerSkalReturnereFeil() {
+        val kontrollResponse = KontrollResponse(
+            meldekortId = 123, kontrollStatus = "OK", feilListe = listOf(
+                KontrollFeil(kode = "", tekst = "", dag = 1)
+            )
+        )
+
+        test(kontrollResponse, "FEIL")
+    }
+
+    private fun test(kontrollResponse: KontrollResponse, forventetStatus: String) {
         val meldekortkontroll = Meldekortkontroll(
             meldekortId = 123,
             fnr = "11111111111",
@@ -83,8 +100,8 @@ class KontrollServiceTest {
 
         runBlocking {
             val actualResponse = kontrollService.kontroller(meldekortkontroll)
-            assertEquals(actualResponse.meldekortId, kontrollResponse.meldekortId)
-            assertEquals(actualResponse.status, kontrollResponse.kontrollStatus)
+            assertEquals(kontrollResponse.meldekortId, actualResponse.meldekortId)
+            assertEquals(forventetStatus, actualResponse.status)
         }
     }