Skip to content

Commit

Permalink
Fjern utdatert kode mot frontend (#698)
Browse files Browse the repository at this point in the history
* Fjern utdatert kode mot frontend

* Fjern utdatert felt for skjæringstidspunkt
  • Loading branch information
bjerga authored Oct 9, 2024
1 parent f58b628 commit fadc300
Show file tree
Hide file tree
Showing 12 changed files with 8 additions and 33 deletions.
1 change: 0 additions & 1 deletion api/http/innsending.http
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,6 @@ Content-Type: application/json
"utbetalerHeleEllerDeler": false
},
"bekreftOpplysninger": true,
"behandlingsdager": [],
"årsakInnsending": "NY"
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,18 +19,12 @@ data class HentForespoerselResponse(
val innsenderNavn: String,
val identitetsnummer: String,
val orgnrUnderenhet: String,
@Deprecated("fjern når det ikke lenger brukes i frontend")
val skjaeringstidspunkt: LocalDate?,
val fravaersperioder: List<Periode>,
val egenmeldingsperioder: List<Periode>,
val bestemmendeFravaersdag: LocalDate,
val eksternBestemmendeFravaersdag: LocalDate?,
val bruttoinntekt: Double?,
val tidligereinntekter: List<InntektPerMaaned>,
@Deprecated("fjern når det ikke lenger brukes i frontend")
val behandlingsperiode: Periode?,
@Deprecated("fjern når det ikke lenger brukes i frontend")
val behandlingsdager: List<LocalDate>,
val forespurtData: ForespurtData?,
val erBesvart: Boolean,
val feilReport: FeilReport? = null,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -103,15 +103,12 @@ private fun HentForespoerselResultat.toResponse(): HentForespoerselResponse {
orgNavn = orgNavn,
identitetsnummer = forespoersel.fnr,
orgnrUnderenhet = forespoersel.orgnr,
skjaeringstidspunkt = forespoersel.eksternBestemmendeFravaersdag(),
fravaersperioder = forespoersel.sykmeldingsperioder,
egenmeldingsperioder = forespoersel.egenmeldingsperioder,
bestemmendeFravaersdag = forespoersel.forslagBestemmendeFravaersdag(),
eksternBestemmendeFravaersdag = forespoersel.eksternBestemmendeFravaersdag(),
bruttoinntekt = inntekt?.gjennomsnitt(),
tidligereinntekter = inntekt?.maanedOversikt.orEmpty(),
behandlingsperiode = null,
behandlingsdager = emptyList(),
forespurtData = forespoersel.forespurtData,
erBesvart = forespoersel.erBesvart,
feilReport =
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -39,8 +39,7 @@ fun Route.innsending(
val producer = InnsendingProducer(rapid)
val redisPoller = RedisStore(redisConnection, RedisPrefix.Innsending).let(::RedisPoller)

// TODO ubrukt path param satt til optional. fjern i frontend, så her.
post(Routes.INNSENDING + "/{forespoerselId?}") {
post(Routes.INNSENDING) {
Metrics.innsendingEndpoint.recordTime(Route::innsending) {
val transaksjonId = UUID.randomUUID()

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ class AuthorizationTest : ApiTest() {
Routes.HENT_FORESPOERSEL to ::postUtenAuth,
Routes.INNTEKT to ::postUtenAuth,
Routes.INNTEKT_SELVBESTEMT to ::postUtenAuth,
Routes.INNSENDING + "/0" to ::postUtenAuth,
Routes.INNSENDING to ::postUtenAuth,
Routes.SELVBESTEMT_INNTEKTSMELDING to ::postUtenAuth,
Routes.SELVBESTEMT_INNTEKTSMELDING + "/0" to ::getUtenAuth,
Routes.KVITTERING to ::getUtenAuth,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -229,15 +229,12 @@ private object Mock {
"innsenderNavn": "Arbeidsgiver",
"identitetsnummer": "${forespoersel.fnr}",
"orgnrUnderenhet": "${forespoersel.orgnr}",
"skjaeringstidspunkt": ${forespoersel.eksternBestemmendeFravaersdag().jsonStrOrNull()},
"fravaersperioder": [${forespoersel.sykmeldingsperioder.joinToString(transform = Periode::hardcodedJson)}],
"egenmeldingsperioder": [${forespoersel.egenmeldingsperioder.joinToString(transform = Periode::hardcodedJson)}],
"bestemmendeFravaersdag": "${forespoersel.forslagBestemmendeFravaersdag()}",
"eksternBestemmendeFravaersdag": ${forespoersel.eksternBestemmendeFravaersdag().jsonStrOrNull()},
"bruttoinntekt": ${inntekt.gjennomsnitt()},
"tidligereinntekter": [${inntekt.maanedOversikt.joinToString(transform = InntektPerMaaned::hardcodedJson)}],
"behandlingsperiode": null,
"behandlingsdager": [],
"forespurtData": ${forespoersel.forespurtData.hardcodedJson()},
"erBesvart": ${forespoersel.erBesvart},
"success": {
Expand All @@ -246,15 +243,12 @@ private object Mock {
"innsenderNavn": "Arbeidsgiver",
"identitetsnummer": "${forespoersel.fnr}",
"orgnrUnderenhet": "${forespoersel.orgnr}",
"skjaeringstidspunkt": ${forespoersel.eksternBestemmendeFravaersdag().jsonStrOrNull()},
"fravaersperioder": [${forespoersel.sykmeldingsperioder.joinToString(transform = Periode::hardcodedJson)}],
"egenmeldingsperioder": [${forespoersel.egenmeldingsperioder.joinToString(transform = Periode::hardcodedJson)}],
"bestemmendeFravaersdag": "${forespoersel.forslagBestemmendeFravaersdag()}",
"eksternBestemmendeFravaersdag": ${forespoersel.eksternBestemmendeFravaersdag().jsonStrOrNull()},
"bruttoinntekt": ${inntekt.gjennomsnitt()},
"tidligereinntekter": [${inntekt.maanedOversikt.joinToString(transform = InntektPerMaaned::hardcodedJson)}],
"behandlingsperiode": null,
"behandlingsdager": [],
"forespurtData": ${forespoersel.forespurtData.hardcodedJson()},
"erBesvart": ${forespoersel.erBesvart}
}
Expand All @@ -269,15 +263,12 @@ private object Mock {
"innsenderNavn": "Arbeidsgiver",
"identitetsnummer": "${forespoersel.fnr}",
"orgnrUnderenhet": "${forespoersel.orgnr}",
"skjaeringstidspunkt": ${forespoersel.eksternBestemmendeFravaersdag().jsonStrOrNull()},
"fravaersperioder": [${forespoersel.sykmeldingsperioder.joinToString(transform = Periode::hardcodedJson)}],
"egenmeldingsperioder": [${forespoersel.egenmeldingsperioder.joinToString(transform = Periode::hardcodedJson)}],
"bestemmendeFravaersdag": "${forespoersel.forslagBestemmendeFravaersdag()}",
"eksternBestemmendeFravaersdag": ${forespoersel.eksternBestemmendeFravaersdag().jsonStrOrNull()},
"bruttoinntekt": ${inntekt.gjennomsnitt()},
"tidligereinntekter": [${inntekt.maanedOversikt.joinToString(transform = InntektPerMaaned::hardcodedJson)}],
"behandlingsperiode": null,
"behandlingsdager": [],
"forespurtData": ${mockForespurtDataMedForrigeInntekt().hardcodedJson()},
"erBesvart": ${forespoersel.erBesvart},
"success": {
Expand All @@ -286,15 +277,12 @@ private object Mock {
"innsenderNavn": "Arbeidsgiver",
"identitetsnummer": "${forespoersel.fnr}",
"orgnrUnderenhet": "${forespoersel.orgnr}",
"skjaeringstidspunkt": ${forespoersel.eksternBestemmendeFravaersdag().jsonStrOrNull()},
"fravaersperioder": [${forespoersel.sykmeldingsperioder.joinToString(transform = Periode::hardcodedJson)}],
"egenmeldingsperioder": [${forespoersel.egenmeldingsperioder.joinToString(transform = Periode::hardcodedJson)}],
"bestemmendeFravaersdag": "${forespoersel.forslagBestemmendeFravaersdag()}",
"eksternBestemmendeFravaersdag": ${forespoersel.eksternBestemmendeFravaersdag().jsonStrOrNull()},
"bruttoinntekt": ${inntekt.gjennomsnitt()},
"tidligereinntekter": [${inntekt.maanedOversikt.joinToString(transform = InntektPerMaaned::hardcodedJson)}],
"behandlingsperiode": null,
"behandlingsdager": [],
"forespurtData": ${mockForespurtDataMedForrigeInntekt().hardcodedJson()},
"erBesvart": ${forespoersel.erBesvart}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,10 +23,9 @@ import no.nav.helsearbeidsgiver.utils.json.toJsonStr
import org.junit.jupiter.api.Assertions.assertEquals
import org.junit.jupiter.api.BeforeEach
import org.junit.jupiter.api.Test
import java.util.UUID

class InnsendingRouteKtTest : ApiTest() {
private val path = Routes.PREFIX + Routes.INNSENDING + "/${UUID.randomUUID()}"
private val path = Routes.PREFIX + Routes.INNSENDING

@BeforeEach
fun setup() {
Expand Down
1 change: 0 additions & 1 deletion api/src/test/resources/innsendingrequest.json
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,6 @@
"utbetalerHeleEllerDeler": false
},
"bekreftOpplysninger": true,
"behandlingsdager": [],
"årsakInnsending": "NY",
"telefonnummer": "004612345678"
}
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ class LagreEksternImRiver(
)
}

override fun LagreEksternImMelding.haandter(json: Map<Key, JsonElement>): Map<Key, JsonElement>? {
override fun LagreEksternImMelding.haandter(json: Map<Key, JsonElement>): Map<Key, JsonElement> {
imRepo.lagreEksternInntektsmelding(forespoerselId, eksternInntektsmelding)

"Lagret ekstern inntektsmelding med arkiv referanse ${eksternInntektsmelding.arkivreferanse} i database.".also {
Expand All @@ -66,7 +66,7 @@ class LagreEksternImRiver(
override fun LagreEksternImMelding.haandterFeil(
json: Map<Key, JsonElement>,
error: Throwable,
): Map<Key, JsonElement>? {
): Map<Key, JsonElement> {
val fail =
Fail(
feilmelding = "Klarte ikke lagre ekstern inntektsmelding i database.",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ class ForespoerselBesvartFraSimbaRiver : ObjectRiver<BesvartSimbaMelding>() {
)
}

override fun BesvartSimbaMelding.haandter(json: Map<Key, JsonElement>): Map<Key, JsonElement>? {
override fun BesvartSimbaMelding.haandter(json: Map<Key, JsonElement>): Map<Key, JsonElement> {
Metrics.forespoerslerBesvartFraSimba.inc()

return notifikasjonHentIdMelding(transaksjonId, forespoerselId)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ class ForespoerselBesvartFraSpleisRiver(
spinnInntektsmeldingId = Pri.Key.SPINN_INNTEKTSMELDING_ID.lesOrNull(UuidSerializer, json),
)

override fun BesvartSpleisMelding.haandter(json: Map<Pri.Key, JsonElement>): Map<Key, JsonElement>? {
override fun BesvartSpleisMelding.haandter(json: Map<Pri.Key, JsonElement>): Map<Key, JsonElement> {
logger.info("Mottok melding på pri-topic om ${Pri.NotisType.FORESPOERSEL_BESVART}.")
sikkerLogger.info("Mottok melding på pri-topic:\n${json.toPretty()}")

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ class ForespoerselMottattRiver : PriObjectRiver<Melding>() {
fnr = Pri.Key.FNR.les(Fnr.serializer(), json),
)

override fun Melding.haandter(json: Map<Pri.Key, JsonElement>): Map<Key, JsonElement>? {
override fun Melding.haandter(json: Map<Pri.Key, JsonElement>): Map<Key, JsonElement> {
logger.info("Mottok melding på pri-topic om ${Pri.NotisType.FORESPØRSEL_MOTTATT}.")
sikkerLogger.info("Mottok melding på pri-topic:\n${json.toPretty()}")

Expand Down

0 comments on commit fadc300

Please sign in to comment.