Skip to content

Commit

Permalink
oppgraderte avhengigheter + litt hamcrest->assertj og sånt
Browse files Browse the repository at this point in the history
  • Loading branch information
@jankroken-nav
jankroken-nav committed Sep 29, 2024
1 parent fdf18e3 commit dc16272
Show file tree
Hide file tree
Showing 4 changed files with 55 additions and 45 deletions.
17 changes: 9 additions & 8 deletions build.gradle.kts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -5,9 +5,9 @@ val domeneVersion = "1.0.70"
val azureAdClient = "0.0.7"
val jacksonVersion = "2.17.1"
val logbackEncoderVersion = "8.0"
val postgresqlVersion = "42.7.3"
val flywayCoreVersion = "9.22.3" // 10.x krever nyere postgres-database
val springKafkaTestVersion = "3.2.2"
val postgresqlVersion = "42.7.4"
val flywayCoreVersion = "10.18.2"
val springKafkaTestVersion = "3.2.4"
val springCloudContractVersion = "4.0.4"
val testcontainersVersion = "1.20.1"
val mockkVersion = "1.13.12"
Expand All @@ -16,22 +16,22 @@ val jsonAssertVersion = "1.5.3"
val wiremockVersion = "3.9.1"
val mockitoVersion = "5.4.0"
val unleashVersion = "9.2.4"
val navTokenSupportVersion = "5.0.1"
val navTokenSupportVersion = "5.0.5"
val hibernateValidatorVersion = "8.0.1.Final"
val junit5Version = "5.10.3"


val snakeYamlVersion = "2.2"
val snappyJavaVersion = "1.1.10.5"
val httpClient5Version = "5.3.1"
val snakeYamlVersion = "2.3"
val snappyJavaVersion = "1.1.10.7"
val httpClient5Version = "5.3.1" // TODO: 5.4 feiler med NoClassDefFoundError
val httpClientVersion = "4.5.14" // deprecated, men brukes av

plugins {
val kotlinVersion = "2.0.0"
id("org.jetbrains.kotlin.jvm") version kotlinVersion
id("org.jetbrains.kotlin.plugin.spring") version kotlinVersion
id("org.jetbrains.kotlin.plugin.jpa") version kotlinVersion
id("org.springframework.boot") version "3.3.2"
id("org.springframework.boot") version "3.3.4"
id("com.github.ben-manes.versions") version "0.51.0"
}

Expand Down Expand Up @@ -78,6 +78,7 @@ dependencies {
// DB
implementation("org.postgresql:postgresql:$postgresqlVersion")
implementation("org.flywaydb:flyway-core:$flywayCoreVersion")
implementation("org.flywaydb:flyway-database-postgresql:$flywayCoreVersion")

// transitive dependency overrides
implementation("org.yaml:snakeyaml:$snakeYamlVersion")
Expand Down
2 changes: 0 additions & 2 deletions .../opptjening/omsorgsopptjening/bestem/pensjonsopptjening/person/external/pdl/PdlService.kt
View file
Original file line number Diff line number Diff line change
Expand Up @@ -36,8 +36,6 @@ internal class PdlService(
}
}

internal data class PdlFnr(val fnr: String, val gjeldende: Boolean)

internal class PdlException(pdlError: PdlError?) : RuntimeException(pdlError?.message ?: "Unknown error from PDL") {
val code: PdlErrorCode? = pdlError?.extensions?.code
}
Expand Down
75 changes: 42 additions & 33 deletions ...ening/omsorgsopptjening/bestem/pensjonsopptjening/oppgave/model/OppgaveopprettelseTest.kt
View file
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,7 @@ import no.nav.pensjon.opptjening.omsorgsopptjening.felles.domene.kafka.Rådata
import no.nav.pensjon.opptjening.omsorgsopptjening.felles.domene.kafka.messages.domene.Kilde
import no.nav.pensjon.opptjening.omsorgsopptjening.felles.domene.kafka.messages.domene.Landstilknytning
import no.nav.pensjon.opptjening.omsorgsopptjening.felles.domene.kafka.messages.domene.Omsorgstype
import org.junit.jupiter.api.Assertions.assertEquals
import org.junit.jupiter.api.Assertions.assertFalse
import org.assertj.core.api.Assertions.assertThat
import org.junit.jupiter.api.Test
import org.junit.jupiter.api.extension.RegisterExtension
import org.mockito.BDDMockito.willAnswer
Expand Down Expand Up @@ -95,9 +94,9 @@ class OppgaveopprettelseTest : SpringContextTest.NoKafka() {
)

handler.process()!!.first().single().also { behandling ->
assertFalse(behandling.erInnvilget())
assertEquals(emptyList<Oppgave>(), oppgaveRepo.findForMelding(behandling.meldingId))
assertEquals(emptyList<Oppgave>(), oppgaveRepo.findForBehandling(behandling.id))
assertThat(behandling.erInnvilget()).isFalse()
assertThat(oppgaveRepo.findForMelding(behandling.meldingId)).isEmpty()
assertThat(oppgaveRepo.findForBehandling(behandling.id)).isEmpty()
}

repo.lagre(
Expand Down Expand Up @@ -144,11 +143,13 @@ class OppgaveopprettelseTest : SpringContextTest.NoKafka() {
)

handler.process()!!.first().single().also { behandling ->
assertFalse(behandling.erInnvilget())
assertThat(behandling.erInnvilget()).isFalse()
oppgaveRepo.findForMelding(behandling.meldingId).single().also { oppgave ->
assertEquals(oppgave, oppgaveRepo.findForBehandling(behandling.id).single())
assertEquals(
oppgave.detaljer, OppgaveDetaljer.MottakerOgTekst(
assertThat(oppgaveRepo.findForBehandling(behandling.id)).containsOnly(oppgave)
assertThat(
oppgave.detaljer
).isEqualTo(
OppgaveDetaljer.MottakerOgTekst(
oppgavemottaker = "04010012797",
oppgavetekst = setOf("""Godskr. omsorgspoeng, flere mottakere: Flere personer har mottatt barnetrygd samme år for barnet under 6 år med fnr 07081812345. Den bruker som oppgaven gjelder mottok barnetrygd i minst seks måneder, og hadde barnetrygd i desember måned. Bruker med fnr 12345678910 mottok også barnetrygd for 6 måneder i samme år. Vurder hvem som skal ha omsorgspoengene.""")
)
Expand Down Expand Up @@ -215,11 +216,13 @@ class OppgaveopprettelseTest : SpringContextTest.NoKafka() {
)

handler.process()!!.first().single().also { behandling ->
assertFalse(behandling.erInnvilget())
assertThat(behandling.erInnvilget()).isFalse()
oppgaveRepo.findForMelding(behandling.meldingId).single().also { oppgave ->
assertEquals(oppgave, oppgaveRepo.findForBehandling(behandling.id).single())
assertEquals(
oppgave.detaljer, OppgaveDetaljer.MottakerOgTekst(
assertThat(oppgaveRepo.findForBehandling(behandling.id)).containsOnly(oppgave)
assertThat(
oppgave.detaljer
).isEqualTo(
OppgaveDetaljer.MottakerOgTekst(
oppgavemottaker = "12345678910",
oppgavetekst = setOf("""Godskr. omsorgspoeng, flere mottakere: Flere personer har mottatt barnetrygd samme år for barnet under 6 år med fnr 07081812345. Den bruker som oppgaven gjelder mottok barnetrygd i minst seks måneder, og hadde barnetrygd i desember måned. Bruker med fnr 04010012797 mottok også barnetrygd for 6 måneder i samme år. Vurder hvem som skal ha omsorgspoengene.""")
)
Expand Down Expand Up @@ -295,12 +298,14 @@ class OppgaveopprettelseTest : SpringContextTest.NoKafka() {
)

handler.process()!!.first().also { behandlinger ->
assertEquals(2, behandlinger.antallBehandlinger())
assertThat(behandlinger.antallBehandlinger()).isEqualTo(2)
oppgaveRepo.findForMelding(behandlinger.alle()[0].meldingId).single().also { oppgave ->
assertEquals(emptyList<Oppgave>(), oppgaveRepo.findForBehandling(behandlinger.finnBehandlingsId()[0]))
assertEquals(oppgave, oppgaveRepo.findForBehandling(behandlinger.finnBehandlingsId()[1]).single())
assertEquals(
oppgave.detaljer, OppgaveDetaljer.MottakerOgTekst(
assertThat(oppgaveRepo.findForBehandling(behandlinger.finnBehandlingsId()[0])).isEmpty()
assertThat(oppgaveRepo.findForBehandling(behandlinger.finnBehandlingsId()[1])).containsOnly(oppgave)
assertThat(
oppgave.detaljer
).isEqualTo(
OppgaveDetaljer.MottakerOgTekst(
oppgavemottaker = "04010012797",
oppgavetekst = setOf("""Godskr. omsorgspoeng, flere mottakere: Flere personer som har mottatt barnetrygd samme år for barnet med fnr 01052012345 i barnets fødselsår. Vurder hvem som skal ha omsorgspoengene.""")
)
Expand Down Expand Up @@ -358,11 +363,13 @@ class OppgaveopprettelseTest : SpringContextTest.NoKafka() {
)

handler.process()!!.first().single().also { behandling ->
assertFalse(behandling.erInnvilget())
assertThat(behandling.erInnvilget()).isFalse()
oppgaveRepo.findForMelding(behandling.meldingId).single().also { oppgave ->
assertEquals(oppgave, oppgaveRepo.findForBehandling(behandling.id).single())
assertEquals(
oppgave.detaljer, OppgaveDetaljer.MottakerOgTekst(
assertThat(oppgaveRepo.findForBehandling(behandling.id)).containsOnly(oppgave)
assertThat(
oppgave.detaljer
).isEqualTo(
OppgaveDetaljer.MottakerOgTekst(
oppgavemottaker = "12345678910",
oppgavetekst = setOf("""Godskr. omsorgspoeng, flere mottakere: Flere personer har mottatt barnetrygd samme år for barnet under 6 år med fnr 07081812345. Den bruker som oppgaven gjelder mottok barnetrygd i minst seks måneder, og hadde barnetrygd i desember måned. Bruker med fnr 04010012797 mottok også barnetrygd for 6 måneder i samme år. Vurder hvem som skal ha omsorgspoengene.""")
)
Expand Down Expand Up @@ -413,9 +420,9 @@ class OppgaveopprettelseTest : SpringContextTest.NoKafka() {
),
)
handler.process()!!.first().single().also { behandling ->
assertFalse(behandling.erInnvilget())
assertEquals(emptyList<Oppgave>(), oppgaveRepo.findForMelding(behandling.meldingId))
assertEquals(emptyList<Oppgave>(), oppgaveRepo.findForBehandling(behandling.id))
assertThat(behandling.erInnvilget()).isFalse()
assertThat(oppgaveRepo.findForMelding(behandling.meldingId)).isEmpty()
assertThat(oppgaveRepo.findForBehandling(behandling.id)).isEmpty()
}
}

Expand Down Expand Up @@ -486,23 +493,25 @@ class OppgaveopprettelseTest : SpringContextTest.NoKafka() {
)

handler.process()!!.first().let { result ->
assertEquals(2, result.antallBehandlinger())
assertThat(result.antallBehandlinger()).isEqualTo(2)
result.alle().first().also { behandling ->
assertFalse(behandling.erInnvilget())
assertThat(behandling.erInnvilget()).isFalse()
oppgaveRepo.findForMelding(behandling.meldingId).single().also { oppgave ->
assertEquals(oppgave, oppgaveRepo.findForBehandling(behandling.id).single())
assertEquals(
oppgave.detaljer, OppgaveDetaljer.MottakerOgTekst(
assertThat(oppgaveRepo.findForBehandling(behandling.id)).containsOnly(oppgave)
assertThat(
oppgave.detaljer
).isEqualTo(
OppgaveDetaljer.MottakerOgTekst(
oppgavemottaker = "12345678910",
oppgavetekst = setOf("""Godskr. omsorgspoeng, flere mottakere: Flere personer har mottatt barnetrygd samme år for barnet under 6 år med fnr 07081812345. Den bruker som oppgaven gjelder mottok barnetrygd i minst seks måneder, og hadde barnetrygd i desember måned. Bruker med fnr 04010012797 mottok også barnetrygd for 6 måneder i samme år. Vurder hvem som skal ha omsorgspoengene.""")
)
)
}
}
result.alle().last().also { behandling ->
assertFalse(behandling.erInnvilget())
assertEquals(emptyList<Oppgave>(), oppgaveRepo.findForBehandling(behandling.id))
assertEquals(1, oppgaveRepo.findForMelding(behandling.meldingId).count())
assertThat(behandling.erInnvilget()).isFalse()
assertThat(oppgaveRepo.findForBehandling(behandling.id)).isEmpty()
assertThat(oppgaveRepo.findForMelding(behandling.meldingId)).hasSize(1)
}
}
}
Expand Down
6 changes: 4 additions & 2 deletions ...tjening/omsorgsopptjening/bestem/pensjonsopptjening/person/external/pdl/PdlServiceTest.kt
View file
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,8 @@ import kotlin.test.assertNotNull

internal class PdlServiceTest : SpringContextTest.NoKafka() {

private val UUID_REGEX = "^[0-9a-fA-F]{8}\\b-[0-9a-fA-F]{4}\\b-[0-9a-fA-F]{4}\\b-[0-9a-fA-F]{4}\\b-[0-9a-fA-F]{12}\$"

@Autowired
lateinit var pdlService: PdlService

Expand Down Expand Up @@ -101,11 +103,11 @@ internal class PdlServiceTest : SpringContextTest.NoKafka() {
.withHeader("Tema", WireMock.equalTo("PEN"))
.withHeader(
"x-correlation-id",
WireMock.matching("^[0-9a-fA-F]{8}\\b-[0-9a-fA-F]{4}\\b-[0-9a-fA-F]{4}\\b-[0-9a-fA-F]{4}\\b-[0-9a-fA-F]{12}\$")
WireMock.matching(UUID_REGEX)
)
.withHeader(
"Nav-Call-Id",
WireMock.matching("^[0-9a-fA-F]{8}\\b-[0-9a-fA-F]{4}\\b-[0-9a-fA-F]{4}\\b-[0-9a-fA-F]{4}\\b-[0-9a-fA-F]{12}\$")
WireMock.matching(UUID_REGEX)
)
)
}
Expand Down

0 comments on commit dc16272

Please sign in to comment.