Merge pull request #115 from navikt/rydd/inntjeningsperiode

rydd/inntjeningsperiode

Author: gtcno

dp-inntekt-api/src/main/kotlin/no/nav/dagpenger/inntekt/v1/InntjeningsperiodeRoute.kt

@@ -17,26 +17,6 @@ import no.nav.dagpenger.inntekt.v1.models.InntjeningsperiodeResultat
 import java.time.LocalDate
 
 fun Route.opptjeningsperiodeApi(inntektStore: InntektStore) {
-    // todo remmove
-    route("is-samme-inntjeningsperiode") {
-        post {
-            withContext(Dispatchers.IO) {
-                val parametere = call.receive<InntjeningsperiodeParametre>()
-
-                val gammelBeregningsdato = inntektStore.getBeregningsdato(InntektId(parametere.inntektsId))
-
-                val resultat = Opptjeningsperiode(gammelBeregningsdato).sammeOpptjeningsPeriode(
-                    Opptjeningsperiode(
-                        LocalDate.parse(parametere.beregningsdato)
-                    )
-                )
-
-                val response = InntjeningsperiodeResultat(resultat, parametere)
-
-                call.respond(HttpStatusCode.OK, response)
-            }
-        }
-    }
     route("samme-inntjeningsperiode") {
         post {
             withContext(Dispatchers.IO) {

dp-inntekt-api/src/main/kotlin/no/nav/dagpenger/inntekt/v1/models/InntjeningsperiodeParametre.kt

@@ -1,8 +1,6 @@
 package no.nav.dagpenger.inntekt.v1.models
 
 data class InntjeningsperiodeParametre(
-    val aktorId: String?, // todo remove
-    val vedtakId: Int?, // todo remove
     val beregningsdato: String,
     val inntektsId: String
 )

dp-inntekt-api/src/main/kotlin/no/nav/dagpenger/inntekt/v1/models/InntjeningsperiodeResultat.kt

@@ -2,6 +2,4 @@ package no.nav.dagpenger.inntekt.v1.models
 
 class InntjeningsperiodeResultat(
     val sammeInntjeningsPeriode: Boolean,
-    // todo remove
-    val parametere: InntjeningsperiodeParametre? = null
 )

dp-inntekt-api/src/test/kotlin/no/nav/dagpenger/inntekt/v1/OpptjeningsperiodeRouteSpec.kt

@@ -15,41 +15,6 @@ import java.time.LocalDate
 import kotlin.test.assertEquals
 
 class OpptjeningsperiodeRouteSpec {
-    // todo remove
-    @Test
-    fun `Inntjeningsperiode API specification test - Should match json field names and formats`() {
-
-        val inntektStore: InntektStore = mockk()
-
-        every {
-            runBlocking {
-                inntektStore.getBeregningsdato(any())
-            }
-        } returns LocalDate.of(2019, 2, 27)
-
-        withTestApplication(mockInntektApi(inntektStore = inntektStore)) {
-            handleRequest(HttpMethod.Post, "/v1/is-samme-inntjeningsperiode") {
-                addHeader(HttpHeaders.ContentType, "application/json")
-                setBody(
-                    """
-                    {
-                      "aktorId": "1234",
-                      "vedtakId": 5678,
-                      "beregningsdato": "2019-02-27",
-                      "inntektsId": "01ARZ3NDEKTSV4RRFFQ69G5FAV"
-                    }
-                    """.trimIndent()
-                )
-            }.apply {
-                assertEquals(HttpStatusCode.OK, response.status())
-                assertEquals(expectedJson, response.content)
-            }
-        }
-    }
-
-    private val expectedJson =
-        """{"sammeInntjeningsPeriode":true,"parametere":{"aktorId":"1234","vedtakId":5678,"beregningsdato":"2019-02-27","inntektsId":"01ARZ3NDEKTSV4RRFFQ69G5FAV"}}"""
-
     @Test
     fun `Same Inntjeningsperiode API specification test - Should match json field names and formats`() {
         val inntektStore: InntektStore = mockk()