Fjern overflødige felt dersom man ikke ber om AGP

navikt · Oct 10, 2023 · 00982ad · 00982ad
1 parent 37c2002
commit 00982ad
Show file tree

Hide file tree

Showing 4 changed files with 56 additions and 10 deletions.
diff --git a/...rc/main/kotlin/no/nav/helsearbeidsgiver/inntektsmelding/api/innsending/InnsendingRoute.kt b/...rc/main/kotlin/no/nav/helsearbeidsgiver/inntektsmelding/api/innsending/InnsendingRoute.kt
@@ -73,6 +73,23 @@ fun RouteExtra.innsendingRoute() {
                                 it
                             }
                         }
+                        .let {
+                            // TODO gjør denne sjekken ved opprettelse
+                            if (it.forespurtData?.contains("arbeidsgiverperiode") == false) {
+                                if (it.fullLønnIArbeidsgiverPerioden != null) {
+                                    "Frontend sender med ${InnsendingRequest::fullLønnIArbeidsgiverPerioden.name} når man ikke ber om AGP."
+                                        .also { feilmelding ->
+                                            logger.error(feilmelding)
+                                            sikkerLogger.error(feilmelding)
+                                        }
+                                }
+                                it.copy(
+                                    fullLønnIArbeidsgiverPerioden = null
+                                )
+                            } else {
+                                it
+                            }
+                        }
 
                     "Mottok innsending med forespørselId: $forespoerselId".let {
                         logger.info(it)

diff --git a/...main/kotlin/no/nav/helsearbeidsgiver/inntektsmelding/api/innsending/InnsendingValidate.kt b/...main/kotlin/no/nav/helsearbeidsgiver/inntektsmelding/api/innsending/InnsendingValidate.kt
@@ -16,7 +16,6 @@ import org.valiktor.functions.isGreaterThanOrEqualTo
 import org.valiktor.functions.isLessThan
 import org.valiktor.functions.isNotEmpty
 import org.valiktor.functions.isNotNull
-import org.valiktor.functions.isNull
 import org.valiktor.functions.isTrue
 import org.valiktor.functions.validate
 import org.valiktor.functions.validateForEach
@@ -36,9 +35,6 @@ fun InnsendingRequest.validate() {
                     validate(FullLonnIArbeidsgiverPerioden::utbetalt).isLessThan(1_000_000.0.toBigDecimal())
                 }
             }
-        } else {
-            // skal ikke komme i delvis im - gir ikke mening - og da bør vi heller ikke ta det imot og lagre det!!
-            validate(InnsendingRequest::fullLønnIArbeidsgiverPerioden).isNull()
         }
         // Den ansatte
         validate(InnsendingRequest::orgnrUnderenhet).isNotNull()

diff --git a/...t/kotlin/no/nav/helsearbeidsgiver/inntektsmelding/api/innsending/InnsendingRouteKtTest.kt b/...t/kotlin/no/nav/helsearbeidsgiver/inntektsmelding/api/innsending/InnsendingRouteKtTest.kt
@@ -2,6 +2,7 @@
 
 package no.nav.helsearbeidsgiver.inntektsmelding.api.innsending
 
+import io.kotest.matchers.nulls.shouldBeNull
 import io.kotest.matchers.shouldBe
 import io.ktor.client.statement.bodyAsText
 import io.ktor.http.HttpStatusCode
@@ -211,6 +212,42 @@ class InnsendingRouteKtTest : ApiTest() {
         }
     }
 
+    @Test
+    fun `fjern ugyldige verdier dersom man ikke spør om AGP`() = testApi {
+        mockTilgang(Tilgang.HAR_TILGANG)
+
+        val mockClientId = UUID.randomUUID()
+
+        // Må bare returnere noe
+        coEvery { anyConstructed<RedisPoller>().hent(mockClientId, any(), any()) } returns JsonNull
+
+        val request = GYLDIG_INNSENDING_REQUEST.copy(
+            forespurtData = listOf("inntekt", "refusjon"),
+            fullLønnIArbeidsgiverPerioden = FullLonnIArbeidsgiverPerioden(
+                utbetalerFullLønn = false,
+                begrunnelse = BegrunnelseIngenEllerRedusertUtbetalingKode.FRAVAER_UTEN_GYLDIG_GRUNN,
+                utbetalt = 222_333.0.toBigDecimal()
+            )
+        )
+            .let(Jackson::toJson).parseJson()
+
+        mockConstructor(InnsendingProducer::class) {
+            every { anyConstructed<InnsendingProducer>().publish(any(), any(), any()) } returns mockClientId
+
+            post(path, request)
+
+            verifySequence {
+                anyConstructed<InnsendingProducer>().publish(
+                    forespoerselId = any(),
+                    request = withArg {
+                        it.fullLønnIArbeidsgiverPerioden.shouldBeNull()
+                    },
+                    arbeidsgiverFnr = any()
+                )
+            }
+        }
+    }
+
     private object Mock {
         val forespoerselId: UUID = UUID.randomUUID()
     }

diff --git a/...otlin/no/nav/helsearbeidsgiver/inntektsmelding/api/innsending/InnsendingValidateKtTest.kt b/...otlin/no/nav/helsearbeidsgiver/inntektsmelding/api/innsending/InnsendingValidateKtTest.kt
@@ -40,11 +40,11 @@ class InnsendingValidateKtTest : FunSpec({
     val maksNaturalBeloep = 1_000_000.0.toBigDecimal()
     val negativtBeloep = (-0.1).toBigDecimal()
 
-    test("skal akseptere gyldig") {
+    test("godtar fullstendig innsending") {
         GYLDIG_INNSENDING_REQUEST.validate()
     }
 
-    test("skal tillate at refusjon i arbeidsgiverperioden ikke settes (ved delvis innsending)") {
+    test("godtar delvis innsending") {
         DELVIS_INNSENDING_REQUEST.validate()
     }
 
@@ -105,10 +105,6 @@ class InnsendingValidateKtTest : FunSpec({
         ).validate()
     }
 
-    test("skal godta delvis innsending") {
-        DELVIS_INNSENDING_REQUEST.validate()
-    }
-
     test("skal gi feilmelding når orgnummer er ugyldig") {
         shouldThrowExactly<ConstraintViolationException> {
             GYLDIG_INNSENDING_REQUEST.copy(orgnrUnderenhet = "").validate()