Skip to content

Commit

Permalink
Fjern bestemmenede fraværsdag (del 2) (#820)
Browse files Browse the repository at this point in the history
  • Loading branch information
@bjerga
bjerga authored Jan 2, 2025
1 parent 11fe5cb commit 221edb3
Show file tree
Hide file tree
Showing 7 changed files with 1 addition and 218 deletions.
17 changes: 0 additions & 17 deletions ...searbeidsgiver/inntektsmelding/berikinntektsmeldingservice/BerikInntektsmeldingService.kt
View file
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,6 @@ import no.nav.helsearbeidsgiver.felles.rapidsrivers.model.Fail
import no.nav.helsearbeidsgiver.felles.rapidsrivers.publish
import no.nav.helsearbeidsgiver.felles.rapidsrivers.service.ServiceMed4Steg
import no.nav.helsearbeidsgiver.felles.utils.Log
import no.nav.helsearbeidsgiver.utils.json.serializer.LocalDateSerializer
import no.nav.helsearbeidsgiver.utils.json.serializer.LocalDateTimeSerializer
import no.nav.helsearbeidsgiver.utils.json.serializer.UuidSerializer
import no.nav.helsearbeidsgiver.utils.json.toJson
Expand All @@ -29,7 +28,6 @@ import no.nav.helsearbeidsgiver.utils.log.logger
import no.nav.helsearbeidsgiver.utils.log.sikkerLogger
import no.nav.helsearbeidsgiver.utils.wrapper.Fnr
import no.nav.helsearbeidsgiver.utils.wrapper.Orgnr
import java.time.LocalDate
import java.time.LocalDateTime
import java.util.UUID

Expand Down Expand Up @@ -58,7 +56,6 @@ data class Steg3(

data class Steg4(
val inntektsmelding: Inntektsmelding,
val bestemmendeFravaersdag: LocalDate,
val erDuplikat: Boolean,
)

Expand Down Expand Up @@ -97,7 +94,6 @@ class BerikInntektsmeldingService(
override fun lesSteg4(melding: Map<Key, JsonElement>): Steg4 =
Steg4(
inntektsmelding = Key.INNTEKTSMELDING.les(Inntektsmelding.serializer(), melding),
bestemmendeFravaersdag = Key.BESTEMMENDE_FRAVAERSDAG.les(LocalDateSerializer, melding),
erDuplikat = Key.ER_DUPLIKAT_IM.les(Boolean.serializer(), melding),
)

Expand Down Expand Up @@ -183,16 +179,6 @@ class BerikInntektsmeldingService(
mottatt = steg0.mottatt,
)

val bestemmendeFravaersdag = utledBestemmendeFravaersdag(steg1.forespoersel, inntektsmelding)

val inntektsdato = inntektsmelding.inntekt?.inntektsdato
if (inntektsdato != null && bestemmendeFravaersdag.isBefore(inntektsdato)) {
"Bestemmende fraværsdag er før inntektsdato. Dette er ikke mulig. Spleis vil trolig spør om ny inntektsmelding.".also {
logger.error(it)
sikkerLogger.error(it)
}
}

rapid
.publish(
key = steg0.skjema.forespoerselId,
Expand All @@ -204,8 +190,6 @@ class BerikInntektsmeldingService(
.plus(
mapOf(
Key.INNTEKTSMELDING to inntektsmelding.toJson(Inntektsmelding.serializer()),
// TODO fjern etter overgangsfase
Key.BESTEMMENDE_FRAVAERSDAG to bestemmendeFravaersdag.toJson(),
Key.INNSENDING_ID to steg0.innsendingId.toJson(Long.serializer()),
),
).toJson(),
Expand All @@ -230,7 +214,6 @@ class BerikInntektsmeldingService(
mapOf(
Key.FORESPOERSEL_ID to steg0.skjema.forespoerselId.toJson(),
Key.INNTEKTSMELDING to steg4.inntektsmelding.toJson(Inntektsmelding.serializer()),
Key.BESTEMMENDE_FRAVAERSDAG to steg4.bestemmendeFravaersdag.toJson(),
Key.INNSENDING_ID to steg0.innsendingId.toJson(Long.serializer()),
).toJson(),
)
Expand Down
30 changes: 0 additions & 30 deletions ...searbeidsgiver/inntektsmelding/berikinntektsmeldingservice/UtledBestemmendeFravaersdag.kt
View file

This file was deleted.

5 changes: 0 additions & 5 deletions ...beidsgiver/inntektsmelding/berikinntektsmeldingservice/BerikInntektsmeldingServiceTest.kt
View file
Original file line number Diff line number Diff line change
Expand Up @@ -30,13 +30,11 @@ import no.nav.helsearbeidsgiver.felles.test.mock.mockInntektsmeldingV1
import no.nav.helsearbeidsgiver.felles.test.mock.mockSkjemaInntektsmelding
import no.nav.helsearbeidsgiver.felles.test.rapidsrivers.message
import no.nav.helsearbeidsgiver.felles.test.rapidsrivers.sendJson
import no.nav.helsearbeidsgiver.utils.json.serializer.LocalDateSerializer
import no.nav.helsearbeidsgiver.utils.json.serializer.UuidSerializer
import no.nav.helsearbeidsgiver.utils.json.serializer.set
import no.nav.helsearbeidsgiver.utils.json.toJson
import no.nav.helsearbeidsgiver.utils.test.date.kl
import no.nav.helsearbeidsgiver.utils.test.date.november
import no.nav.helsearbeidsgiver.utils.test.date.oktober
import no.nav.helsearbeidsgiver.utils.test.wrapper.genererGyldig
import no.nav.helsearbeidsgiver.utils.wrapper.Fnr
import no.nav.helsearbeidsgiver.utils.wrapper.Orgnr
Expand Down Expand Up @@ -101,7 +99,6 @@ class BerikInntektsmeldingServiceTest :

val data = it.lesData()
Key.INNTEKTSMELDING.lesOrNull(Inntektsmelding.serializer(), data) shouldNotBe null
Key.BESTEMMENDE_FRAVAERSDAG.lesOrNull(LocalDateSerializer, data) shouldNotBe null
Key.INNSENDING_ID.lesOrNull(Long.serializer(), data) shouldBe Mock.INNSENDING_ID
}

Expand All @@ -115,7 +112,6 @@ class BerikInntektsmeldingServiceTest :
val data = it.lesData()
Key.FORESPOERSEL_ID.lesOrNull(UuidSerializer, data) shouldBe Mock.skjema.forespoerselId
Key.INNTEKTSMELDING.lesOrNull(Inntektsmelding.serializer(), data) shouldNotBe null
Key.BESTEMMENDE_FRAVAERSDAG.lesOrNull(LocalDateSerializer, data) shouldNotBe null
Key.INNSENDING_ID.lesOrNull(Long.serializer(), data) shouldBe Mock.INNSENDING_ID
}
}
Expand Down Expand Up @@ -204,7 +200,6 @@ private object Mock {
mapOf(
Key.ER_DUPLIKAT_IM to false.toJson(Boolean.serializer()),
Key.INNTEKTSMELDING to mockInntektsmeldingV1().toJson(Inntektsmelding.serializer()),
Key.BESTEMMENDE_FRAVAERSDAG to 20.oktober.toJson(),
),
)
}
154 changes: 0 additions & 154 deletions ...idsgiver/inntektsmelding/berikinntektsmeldingservice/UtledBestemmendeFravaersdagKtTest.kt
View file

This file was deleted.

1 change: 0 additions & 1 deletion apps/felles/src/main/kotlin/no/nav/helsearbeidsgiver/felles/Key.kt
View file
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@ enum class Key : IKey {
// Egendefinerte
ARBEIDSFORHOLD,
ARBEIDSGIVER_FNR,
BESTEMMENDE_FRAVAERSDAG,
DATA,
EKSTERN_INNTEKTSMELDING,
ER_DUPLIKAT_IM,
Expand Down
9 changes: 0 additions & 9 deletions ...o/nav/helsearbeidsgiver/inntektsmelding/integrasjonstest/BerikInntektsmeldingServiceIT.kt
View file
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,6 @@ import no.nav.helsearbeidsgiver.inntektsmelding.integrasjonstest.utils.bjarneBet
import no.nav.helsearbeidsgiver.inntektsmelding.integrasjonstest.utils.maxMekker
import no.nav.helsearbeidsgiver.utils.json.fromJson
import no.nav.helsearbeidsgiver.utils.json.parseJson
import no.nav.helsearbeidsgiver.utils.json.serializer.LocalDateSerializer
import no.nav.helsearbeidsgiver.utils.json.serializer.UuidSerializer
import no.nav.helsearbeidsgiver.utils.json.toJson
import no.nav.helsearbeidsgiver.utils.test.date.august
Expand Down Expand Up @@ -160,10 +159,6 @@ class BerikInntektsmeldingServiceIT : EndToEndTest() {
data[Key.INNTEKTSMELDING]
.shouldNotBeNull()
.fromJson(Inntektsmelding.serializer())

data[Key.BESTEMMENDE_FRAVAERSDAG]
.shouldNotBeNull()
.fromJson(LocalDateSerializer)
}
}

Expand Down Expand Up @@ -297,10 +292,6 @@ class BerikInntektsmeldingServiceIT : EndToEndTest() {
data[Key.INNTEKTSMELDING]
.shouldNotBeNull()
.fromJson(Inntektsmelding.serializer())

data[Key.BESTEMMENDE_FRAVAERSDAG]
.shouldNotBeNull()
.fromJson(LocalDateSerializer)
}
}
}
Expand Down
3 changes: 1 addition & 2 deletions ...c/test/kotlin/no/nav/helsearbeidsgiver/inntektsmelding/joark/TilXmlInntektsmeldingTest.kt
View file
Original file line number Diff line number Diff line change
Expand Up @@ -53,13 +53,12 @@ class TilXmlInntektsmeldingTest {

@Test
fun `skal godta null-verdi i InntektEndringÅrsak`() {
val bestemmendeFravaersdag = 20.oktober
val inntektmeldingUtenAarsak =
mockInntektsmeldingV1().copy(
inntekt =
Inntekt(
beloep = 1.0,
inntektsdato = bestemmendeFravaersdag,
inntektsdato = 20.oktober,
naturalytelser = emptyList(),
endringAarsak = null,
),
Expand Down

0 comments on commit 221edb3

Please sign in to comment.