From 066ac056d9f55552b5a1342331fc7d2d29909a8c Mon Sep 17 00:00:00 2001 From: Ole Christian Kvernberg Date: Tue, 2 Jan 2024 09:16:32 +0100 Subject: [PATCH 1/3] =?UTF-8?q?Venter=20med=20=C3=A5=20hente=20inntekt=20f?= =?UTF-8?q?or=20forrige=20=C3=A5r=20til=20det=20er=20klart?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- src/main/kotlin/no/nav/familie/ef/sak/sigrun/SigrunService.kt | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/main/kotlin/no/nav/familie/ef/sak/sigrun/SigrunService.kt b/src/main/kotlin/no/nav/familie/ef/sak/sigrun/SigrunService.kt index d96cae3858..459b38afc1 100644 --- a/src/main/kotlin/no/nav/familie/ef/sak/sigrun/SigrunService.kt +++ b/src/main/kotlin/no/nav/familie/ef/sak/sigrun/SigrunService.kt @@ -15,7 +15,8 @@ class SigrunService(val sigrunClient: SigrunClient, val fagsakPersonService: Fag fun hentInntektForAlleÅrMedInntekt(fagsakPersonId: UUID): List { val aktivIdent = fagsakPersonService.hentAktivIdent(fagsakPersonId) - val inntektsår = (YearMonth.now().year - 1) downTo 2017 + val tilOgMedÅr = if (YearMonth.now().month.value < 6) YearMonth.now().year - 2 else YearMonth.now().year - 1 + val inntektsår = tilOgMedÅr downTo 2017 val pensjonsgivendeInntektList = inntektsår.map { sigrunClient.hentPensjonsgivendeInntekt(aktivIdent, it).mapTilPensjonsgivendeInntektVisning(it) From fdc4cc4fb5c4bf510f8bbd48d81d1a279c5f0d48 Mon Sep 17 00:00:00 2001 From: Ole Christian Kvernberg Date: Tue, 2 Jan 2024 09:43:14 +0100 Subject: [PATCH 2/3] Testfixer --- .../ef/sak/beregning/VedtakServiceTest.kt | 4 +- .../ef/sak/sigrun/SigrunServiceTest.kt | 60 +++++++------------ 2 files changed, 25 insertions(+), 39 deletions(-) diff --git a/src/test/kotlin/no/nav/familie/ef/sak/beregning/VedtakServiceTest.kt b/src/test/kotlin/no/nav/familie/ef/sak/beregning/VedtakServiceTest.kt index 2757f8de11..2f76ff97dc 100644 --- a/src/test/kotlin/no/nav/familie/ef/sak/beregning/VedtakServiceTest.kt +++ b/src/test/kotlin/no/nav/familie/ef/sak/beregning/VedtakServiceTest.kt @@ -225,8 +225,8 @@ internal class VedtakServiceTest : OppslagSpringRunnerTest() { @Test internal fun `hentForventetInntektForVedtakOgDato - forskjellig inntekt mellom to måneder`() { - val vedtakFraOgMedDato = LocalDate.of(2022, 1, 1) - val vedtakTilOgMedDato = LocalDate.of(2023, 12, 31) + val vedtakFraOgMedDato = LocalDate.of(YearMonth.now().year-1, 1, 1) + val vedtakTilOgMedDato = LocalDate.of(YearMonth.now().year, 12, 31) val inntektsperiodeToMånederTilbake = Inntektsperiode( diff --git a/src/test/kotlin/no/nav/familie/ef/sak/sigrun/SigrunServiceTest.kt b/src/test/kotlin/no/nav/familie/ef/sak/sigrun/SigrunServiceTest.kt index 4d83d5e2af..e317730420 100644 --- a/src/test/kotlin/no/nav/familie/ef/sak/sigrun/SigrunServiceTest.kt +++ b/src/test/kotlin/no/nav/familie/ef/sak/sigrun/SigrunServiceTest.kt @@ -24,51 +24,37 @@ internal class SigrunServiceTest { @BeforeEach fun setup() { - every { fagsakPersonService.hentAktivIdent(any()) } returns "123" - every { sigrunClient.hentPensjonsgivendeInntekt(any(), 2022) } returns PensjonsgivendeInntektResponse( - "123", - 2022, - pensjonsgivendeInntekt = listOf( - pensjonsgivendeInntektForSkatteordning(Skatteordning.FASTLAND), - pensjonsgivendeInntektForSkatteordning(Skatteordning.SVALBARD, 325_000, 20_000), - ), - ) - every { sigrunClient.hentPensjonsgivendeInntekt(any(), 2021) } returns PensjonsgivendeInntektResponse( - "123", - 2021, - pensjonsgivendeInntekt = listOf( - pensjonsgivendeInntektForSkatteordning(Skatteordning.FASTLAND), - pensjonsgivendeInntektForSkatteordning(Skatteordning.SVALBARD), - ), - ) - every { sigrunClient.hentPensjonsgivendeInntekt(any(), 2020) } returns PensjonsgivendeInntektResponse( - "123", - 2020, - pensjonsgivendeInntekt = listOf( - pensjonsgivendeInntektForSkatteordning(Skatteordning.FASTLAND), - pensjonsgivendeInntektForSkatteordning(Skatteordning.SVALBARD), - ), - ) + every { fagsakPersonService.hentAktivIdent(any()) } returns "123" + val tilOgMedÅr = if (YearMonth.now().month.value < 6) YearMonth.now().year - 2 else YearMonth.now().year - 1 - every { sigrunClient.hentPensjonsgivendeInntekt(any(), 2019) } returns PensjonsgivendeInntektResponse("123", 2019, listOf()) - every { sigrunClient.hentPensjonsgivendeInntekt(any(), 2018) } returns PensjonsgivendeInntektResponse("123", 2018, listOf()) + every { sigrunClient.hentPensjonsgivendeInntekt("123", any()) } answers { + PensjonsgivendeInntektResponse( + "123", + secondArg(), + pensjonsgivendeInntekt = listOf( + pensjonsgivendeInntektForSkatteordning(Skatteordning.FASTLAND), + pensjonsgivendeInntektForSkatteordning(Skatteordning.SVALBARD), + ), + ) + } - every { sigrunClient.hentPensjonsgivendeInntekt(any(), 2017) } returns PensjonsgivendeInntektResponse( - "123", - 2017, - pensjonsgivendeInntekt = listOf( - pensjonsgivendeInntektForSkatteordning(Skatteordning.FASTLAND), - pensjonsgivendeInntektForSkatteordning(Skatteordning.SVALBARD), - ), - ) + every { sigrunClient.hentPensjonsgivendeInntekt("123", tilOgMedÅr) } answers { + PensjonsgivendeInntektResponse( + "123", + 2022, + pensjonsgivendeInntekt = listOf( + pensjonsgivendeInntektForSkatteordning(Skatteordning.FASTLAND), + pensjonsgivendeInntektForSkatteordning(Skatteordning.SVALBARD, 325_000, 20_000), + ), + ) + } } @Test - fun `hent inntekt siste fem år med svalbard inntekt`() { + fun `hent inntekt tilbake til 2017 med svalbard inntekt`() { val fagsakId = UUID.randomUUID() val pensjonsgivendeInntektVisning = sigrunService.hentInntektForAlleÅrMedInntekt(fagsakId) - assertThat(pensjonsgivendeInntektVisning.first().inntektsår).isEqualTo(YearMonth.now().year - 1) assertThat(pensjonsgivendeInntektVisning.first().næring).isEqualTo(250_000) assertThat(pensjonsgivendeInntektVisning.first().person).isEqualTo(100_000) assertThat(pensjonsgivendeInntektVisning.first().svalbard?.næring).isEqualTo(70_000) From 575301f5a031102d623a43d299c1a168dd70397b Mon Sep 17 00:00:00 2001 From: Ole Christian Kvernberg Date: Tue, 2 Jan 2024 09:45:21 +0100 Subject: [PATCH 3/3] Linterendringer --- .../kotlin/no/nav/familie/ef/sak/beregning/VedtakServiceTest.kt | 2 +- .../kotlin/no/nav/familie/ef/sak/sigrun/SigrunServiceTest.kt | 1 - 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/src/test/kotlin/no/nav/familie/ef/sak/beregning/VedtakServiceTest.kt b/src/test/kotlin/no/nav/familie/ef/sak/beregning/VedtakServiceTest.kt index 2f76ff97dc..2156be6d9e 100644 --- a/src/test/kotlin/no/nav/familie/ef/sak/beregning/VedtakServiceTest.kt +++ b/src/test/kotlin/no/nav/familie/ef/sak/beregning/VedtakServiceTest.kt @@ -225,7 +225,7 @@ internal class VedtakServiceTest : OppslagSpringRunnerTest() { @Test internal fun `hentForventetInntektForVedtakOgDato - forskjellig inntekt mellom to måneder`() { - val vedtakFraOgMedDato = LocalDate.of(YearMonth.now().year-1, 1, 1) + val vedtakFraOgMedDato = LocalDate.of(YearMonth.now().year - 1, 1, 1) val vedtakTilOgMedDato = LocalDate.of(YearMonth.now().year, 12, 31) val inntektsperiodeToMånederTilbake = diff --git a/src/test/kotlin/no/nav/familie/ef/sak/sigrun/SigrunServiceTest.kt b/src/test/kotlin/no/nav/familie/ef/sak/sigrun/SigrunServiceTest.kt index e317730420..a4c7b9e387 100644 --- a/src/test/kotlin/no/nav/familie/ef/sak/sigrun/SigrunServiceTest.kt +++ b/src/test/kotlin/no/nav/familie/ef/sak/sigrun/SigrunServiceTest.kt @@ -24,7 +24,6 @@ internal class SigrunServiceTest { @BeforeEach fun setup() { - every { fagsakPersonService.hentAktivIdent(any()) } returns "123" val tilOgMedÅr = if (YearMonth.now().month.value < 6) YearMonth.now().year - 2 else YearMonth.now().year - 1