Skip to content

Commit

Permalink
Oppdaterer til spring 3.4.1 (#1020)
Browse files Browse the repository at this point in the history
Favrokort:
https://favro.com/organization/98c34fb974ce445eac854de0/1844bbac3b6605eacc8f5543?card=NAV-23886

Samme feil som i ba-sak med optimistic lock.
Dette oppstår i testene våres, og jeg tenker egentlig at det skyldes
feil oppsett av tester. Justerer derfor på testene for å se om det løser
det, og at vi kan hoppe over til 3.4.1 i ks-sak.

Tester OK. Deployes manuelt til preprod for testing av ordinære løp.

---------

Co-authored-by: Magnus Tønnessen <[email protected]>
  • Loading branch information
UyQuangNguyen and MagnusTonnessen authored Jan 16, 2025
1 parent effd7ce commit 5a95a9c
Show file tree
Hide file tree
Showing 9 changed files with 16 additions and 39 deletions.
8 changes: 4 additions & 4 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
<parent>
<groupId>org.springframework.boot</groupId>
<artifactId>spring-boot-starter-parent</artifactId>
<version>3.3.6</version> <!-- Skal samsvare med spring-boot-dependencies -->
<version>3.4.1</version> <!-- Skal samsvare med spring-boot-dependencies -->
<relativePath/> <!-- lookup parent from repository -->
</parent>

Expand All @@ -29,12 +29,12 @@
<kotlinx.version>1.9.0</kotlinx.version>

<!-- Spring -->
<spring-boot-dependencies.version>3.3.6</spring-boot-dependencies.version> <!-- Skal samsvare med spring-boot-starter-parent -->
<springdoc.version>2.6.0</springdoc.version>
<spring-boot-dependencies.version>3.4.1</spring-boot-dependencies.version> <!-- Skal samsvare med spring-boot-starter-parent -->
<springdoc.version>2.8.3</springdoc.version>

<!-- Nav felles -->
<felles.version>3.20241121151626_836e7ca</felles.version>
<nav-token-client.version>5.0.11</nav-token-client.version>
<nav-token-client.version>5.0.14</nav-token-client.version>
<nav-foedselsnummer.version>1.0-SNAPSHOT.6</nav-foedselsnummer.version>

<!-- Nav Familie-->
Expand Down
12 changes: 1 addition & 11 deletions src/test/common/TestdataGenerator.kt
Original file line number Diff line number Diff line change
Expand Up @@ -245,21 +245,11 @@ fun nesteVedtakId(): Long {
return gjeldendeVedtakId
}

fun nesteBehandlingId(): Long {
gjeldendeBehandlingId += ID_INKREMENT
return gjeldendeBehandlingId
}

fun nestePersonId(): Long {
gjeldendePersonId += ID_INKREMENT
return gjeldendePersonId
}

fun nesteUtvidetVedtaksperiodeId(): Long {
gjeldendeUtvidetVedtaksperiodeId += ID_INKREMENT
return gjeldendeUtvidetVedtaksperiodeId
}

fun lagLogg(
behandlingId: Long,
id: Long = 1L,
Expand Down Expand Up @@ -289,7 +279,7 @@ fun lagBehandling(
resultat: Behandlingsresultat = Behandlingsresultat.IKKE_VURDERT,
aktiv: Boolean = true,
status: BehandlingStatus = BehandlingStatus.UTREDES,
id: Long = nesteBehandlingId(),
id: Long = 0L,
endretTidspunkt: LocalDateTime = LocalDateTime.now(),
lagBehandlingStegTilstander: (behandling: Behandling) -> Set<BehandlingStegTilstand> = {
setOf(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ class StegServiceUnitTest {
@InjectMockKs
private lateinit var stegService: StegService

private val behandling = lagBehandling(opprettetÅrsak = BehandlingÅrsak.SØKNAD)
private val behandling = lagBehandling(id = 1, opprettetÅrsak = BehandlingÅrsak.SØKNAD)

@Test
fun `settBehandlingstegTilstandPåVent - skal sette årsak frist og status på behandlingstegtilstand til nåværende behandlingssteg`() {
Expand Down Expand Up @@ -172,7 +172,7 @@ class StegServiceUnitTest {
@Test
fun `skal iverksette mot oppdrag hvis det er endring i ordinære andeler`() {
// Arrange
val forrigeBehandling = lagBehandling(fagsak = behandling.fagsak, resultat = Behandlingsresultat.INNVILGET)
val forrigeBehandling = lagBehandling(id = 0, fagsak = behandling.fagsak, resultat = Behandlingsresultat.INNVILGET)
val forrigeBehandlingAndeler =
listOf(
lagAndelTilkjentYtelse(
Expand Down Expand Up @@ -215,7 +215,7 @@ class StegServiceUnitTest {
@Test
fun `skal iverksette mot oppdrag hvis det er endring i overgangsordningandeler`() {
// Arrange
val forrigeBehandling = lagBehandling(fagsak = behandling.fagsak, resultat = Behandlingsresultat.INNVILGET)
val forrigeBehandling = lagBehandling(id = 0, fagsak = behandling.fagsak, resultat = Behandlingsresultat.INNVILGET)
val forrigeBehandlingAndeler =
listOf(
lagAndelTilkjentYtelse(
Expand Down Expand Up @@ -274,7 +274,7 @@ class StegServiceUnitTest {
@Test
fun `skal iverksette mot oppdrag hvis det mangler andeler i ny behandling`() {
// Arrange
val forrigeBehandling = lagBehandling(fagsak = behandling.fagsak, resultat = Behandlingsresultat.INNVILGET)
val forrigeBehandling = lagBehandling(id = 0, fagsak = behandling.fagsak, resultat = Behandlingsresultat.INNVILGET)
val forrigeBehandlingAndeler =
listOf(
lagAndelTilkjentYtelse(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -364,7 +364,7 @@ internal class VedtaksperiodeServiceTest {
andelerTilkjentYtelseOgEndreteUtbetalingerService.finnAndelerTilkjentYtelseMedEndreteUtbetalinger(behandling.id)
} returns listOf(AndelTilkjentYtelseMedEndreteUtbetalinger(andelTilkjentYtelse, emptyList()))

val revurdering = lagBehandling()
val revurdering = lagBehandling(id = 2)
val andelTilkjentYtelseForRevurdering =
lagAndelTilkjentYtelse(
tilkjentYtelse = lagInitieltTilkjentYtelse(revurdering),
Expand Down Expand Up @@ -421,7 +421,7 @@ internal class VedtaksperiodeServiceTest {
AndelTilkjentYtelseMedEndreteUtbetalinger(andelTilkjentYtelse2, emptyList()),
)

val revurdering = lagBehandling()
val revurdering = lagBehandling(id = 2)
val andelTilkjentYtelseForRevurdering1 =
lagAndelTilkjentYtelse(
tilkjentYtelse = lagInitieltTilkjentYtelse(revurdering),
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ class VilkårsvurderingStegTest {

private val fagsak = lagFagsak(søker)

private val behandling = lagBehandling(fagsak, opprettetÅrsak = BehandlingÅrsak.SØKNAD)
private val behandling = lagBehandling(id = 1, fagsak = fagsak, opprettetÅrsak = BehandlingÅrsak.SØKNAD)

@BeforeEach
fun init() {
Expand Down Expand Up @@ -630,7 +630,7 @@ class VilkårsvurderingStegTest {
barnPersonResultat.setSortedVilkårResultater(vilkårResultaterForBarn)
vilkårsvurderingForSøker.personResultater = setOf(søkerPersonResultat, barnPersonResultat)

val forrigeBehandling = lagBehandling(opprettetÅrsak = BehandlingÅrsak.SØKNAD)
val forrigeBehandling = lagBehandling(id = 0, opprettetÅrsak = BehandlingÅrsak.SØKNAD)
val forrigeVilkårsvurderingForSøker = Vilkårsvurdering(behandling = behandling)
val søkerPersonResultatIForrigeVilkårsvurdering =
PersonResultat(vilkårsvurdering = forrigeVilkårsvurderingForSøker, aktør = søker)
Expand Down Expand Up @@ -723,7 +723,7 @@ class VilkårsvurderingStegTest {
barnPersonResultat.setSortedVilkårResultater(vilkårResultaterForBarn)
vilkårsvurderingForSøker.personResultater = setOf(søkerPersonResultat, barnPersonResultat)

val forrigeBehandling = lagBehandling(opprettetÅrsak = BehandlingÅrsak.SØKNAD)
val forrigeBehandling = lagBehandling(id = 0, opprettetÅrsak = BehandlingÅrsak.SØKNAD)
val forrigeVilkårsvurderingForSøker = Vilkårsvurdering(behandling = behandling)
val søkerPersonResultatIForrigeVilkårsvurdering =
PersonResultat(vilkårsvurdering = forrigeVilkårsvurderingForSøker, aktør = søker)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ class TilkjentYtelseValideringServiceTest {
private val barn1 = randomAktør()
private val barn2 = randomAktør()
private val barn3MedUtbetalinger = randomAktør()
private val behandling = lagBehandling(opprettetÅrsak = BehandlingÅrsak.SØKNAD)
private val behandling = lagBehandling(id = 1, opprettetÅrsak = BehandlingÅrsak.SØKNAD)
private val personopplysningGrunnlag =
lagPersonopplysningGrunnlag(
behandling.id,
Expand Down Expand Up @@ -162,7 +162,7 @@ class TilkjentYtelseValideringServiceTest {
),
)

val forrigeBehandling = lagBehandling(opprettetÅrsak = BehandlingÅrsak.SØKNAD)
val forrigeBehandling = lagBehandling(id = 0, opprettetÅrsak = BehandlingÅrsak.SØKNAD)

val forrigeTilkjentYtelse =
TilkjentYtelse(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -133,23 +133,20 @@ class OvergangsordningAndelControllerTest : OppslagSpringRunnerTest() {
val gamleOvergangsordningAndeler =
listOf(
OvergangsordningAndel(
id = 0,
behandlingId = behandling.id,
person = barn1,
antallTimer = BigDecimal(20),
fom = barn1.fødselsdato.plusMonths(20).toYearMonth(),
tom = barn1.fødselsdato.plusMonths(21).toYearMonth(),
),
OvergangsordningAndel(
id = 1,
behandlingId = behandling.id,
person = barn1,
antallTimer = BigDecimal.ZERO,
fom = barn1.fødselsdato.plusMonths(22).toYearMonth(),
tom = barn1.fødselsdato.plusMonths(23).toYearMonth(),
),
OvergangsordningAndel(
id = 2,
behandlingId = behandling.id,
person = barn2,
fom = barn2.fødselsdato.plusMonths(20).toYearMonth(),
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@ class KorrigertEtterbetalingRepositoryTest(
fun `finnAktivtKorrigeringPåBehandling skal returnere null dersom det ikke eksisterer en aktiv etterbetaling korrigering på behandling`() {
val inaktivKorrigertEtterbetaling =
KorrigertEtterbetaling(
id = 10000001,
årsak = KorrigertEtterbetalingÅrsak.REFUSJON_FRA_ANDRE_MYNDIGHETER,
begrunnelse = "Test på inaktiv korrigering",
beløp = 1000,
Expand All @@ -44,7 +43,6 @@ class KorrigertEtterbetalingRepositoryTest(
fun `finnAktivtKorrigeringPåBehandling skal returnere aktiv korrigering på behandling dersom det finnes`() {
val aktivKorrigertEtterbetaling =
KorrigertEtterbetaling(
id = 10000002,
årsak = KorrigertEtterbetalingÅrsak.REFUSJON_FRA_ANDRE_MYNDIGHETER,
begrunnelse = "Test på aktiv korrigering",
beløp = 1000,
Expand All @@ -65,7 +63,6 @@ class KorrigertEtterbetalingRepositoryTest(
fun `Det skal kastes DataIntegrityViolationException dersom det forsøkes å lagre aktivt korrigering når det allerede finnes en`() {
val aktivKorrigertEtterbetaling =
KorrigertEtterbetaling(
id = 10000007,
årsak = KorrigertEtterbetalingÅrsak.REFUSJON_FRA_ANDRE_MYNDIGHETER,
begrunnelse = "Test på aktiv korrigering",
beløp = 1000,
Expand All @@ -75,7 +72,6 @@ class KorrigertEtterbetalingRepositoryTest(

val aktivKorrigertEtterbetaling2 =
KorrigertEtterbetaling(
id = 10000008,
årsak = KorrigertEtterbetalingÅrsak.REFUSJON_FRA_ANDRE_MYNDIGHETER,
begrunnelse = "Test på aktiv korrigering",
beløp = 1000,
Expand All @@ -94,7 +90,6 @@ class KorrigertEtterbetalingRepositoryTest(
fun `hentAlleKorrigeringPåBehandling skal returnere alle KorrigertEtterbetaling på behandling`() {
val aktivKorrigertEtterbetaling =
KorrigertEtterbetaling(
id = 10000003,
årsak = KorrigertEtterbetalingÅrsak.REFUSJON_FRA_ANDRE_MYNDIGHETER,
begrunnelse = "1",
beløp = 1000,
Expand All @@ -104,7 +99,6 @@ class KorrigertEtterbetalingRepositoryTest(

val inaktivKorrigertEtterbetaling =
KorrigertEtterbetaling(
id = 10000004,
årsak = KorrigertEtterbetalingÅrsak.REFUSJON_FRA_ANDRE_MYNDIGHETER,
begrunnelse = "2",
beløp = 1000,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@ class KorrigertVedtakRepositoryTest(
fun `finnAktivtKorrigertVedtakPåBehandling skal returnere null dersom det ikke eksisterer en aktiv korrigering av vedtak på behandling`() {
val inaktivKorrigertVedtak =
KorrigertVedtak(
id = 10000001,
vedtaksdato = LocalDate.now().minusDays(6),
begrunnelse = "Test på inaktiv korrigering",
behandling = behandling,
Expand All @@ -43,7 +42,6 @@ class KorrigertVedtakRepositoryTest(
fun `finnAktivtKorrigertVedtakPåBehandling skal returnere aktiv korrigert vedtak når det eksisterer en aktiv korrigering av vedtak på behandling`() {
val aktivKorrigertVedtak =
KorrigertVedtak(
id = 10000001,
vedtaksdato = LocalDate.now().minusDays(6),
begrunnelse = "Test på aktiv korrigering",
behandling = behandling,
Expand All @@ -65,7 +63,6 @@ class KorrigertVedtakRepositoryTest(
fun `Det skal kastes DataIntegrityViolationException dersom det forsøkes å lagre aktivt korrigert vedtak når det allerede finnes en`() {
val aktivKorrigertVedtak1 =
KorrigertVedtak(
id = 10000007,
begrunnelse = "Test på aktiv korrigering",
vedtaksdato = LocalDate.now().minusDays(6),
behandling = behandling,
Expand All @@ -74,7 +71,6 @@ class KorrigertVedtakRepositoryTest(

val aktivKorrigertVedtak2 =
KorrigertVedtak(
id = 10000008,
begrunnelse = "Test på aktiv korrigering",
vedtaksdato = LocalDate.now().minusDays(3),
behandling = behandling,
Expand Down

0 comments on commit 5a95a9c

Please sign in to comment.