Merge branch 'main' into dependabot/gradle/org.sonarqube-6.0.0.5145

.github/workflows/deploy-gcp.yml

@@ -30,7 +30,7 @@ jobs:
           java-version: 21
           distribution: 'temurin'
       - name: Setup Gradle
-        uses: gradle/actions/setup-gradle@v3
+        uses: gradle/actions/setup-gradle@v4
       - name: Execute Gradle build
         run: ./gradlew build --scan
       - name: Slack Notification (test failure)

.github/workflows/deploy_feature_branch_gcp.yml

@@ -26,7 +26,7 @@ jobs:
           distribution: temurin
           java-version: 21
       - name: Setup Gradle
-        uses: gradle/actions/setup-gradle@v3
+        uses: gradle/actions/setup-gradle@v4
       - name: Execute Gradle build
         run: ./gradlew assemble --scan
       - name: Push docker image to GAR

.github/workflows/testMedSonar.yml

@@ -35,7 +35,7 @@ jobs:
           java-version: 21
 
       - name: Setup Gradle
-        uses: gradle/actions/setup-gradle@v3
+        uses: gradle/actions/setup-gradle@v4
 
       - name: Cache SonarCloud packages
         uses: actions/cache@v4

build.gradle.kts

@@ -2,23 +2,23 @@
 val spring_boot_version = "3.4.0"
 val common_version = "3.2024.11.26_16.35-432a29107830"
 val dab_common_version = "2024.11.14-10.46.174740baf5c7"
-val poao_tilgang_version = "2024.11.08_14.51-f6a9a0795785"
+val poao_tilgang_version = "2024.11.26_08.36-ad014162ce23"
 val shedlock_version = "6.0.2"
 val _version: String by project
 
 plugins {
     id("java")
-    id("org.jetbrains.kotlin.plugin.spring") version "2.0.21"
+    id("org.jetbrains.kotlin.plugin.spring") version "2.1.0"
     id("application")
     id("maven-publish")
-    kotlin("jvm") version "2.0.21"
+    kotlin("jvm") version "2.1.0"
     id("org.openapi.generator") version "7.10.0"
     id("com.github.davidmc24.gradle.plugin.avro") version "1.9.1"
     id("project-report")
     id("jacoco")
     id("org.sonarqube") version "6.0.0.5145"
     id("org.springframework.boot") version "3.4.0"
-    kotlin("plugin.lombok") version "2.0.21"
+    kotlin("plugin.lombok") version "2.1.0"
     id("io.freefair.lombok") version "8.11"
 }
 
@@ -152,8 +152,8 @@ dependencies {
     implementation("no.nav.common:client:$common_version")
     implementation("no.nav.common:util:$common_version")
     implementation("no.nav.common:types:$common_version")
-    implementation("org.apache.commons:commons-collections4:4.1")
-    implementation("no.nav.tms.varsel:kotlin-builder:2.1.0")
+    implementation("org.apache.commons:commons-collections4:4.4")
+    implementation("no.nav.tms.varsel:kotlin-builder:2.1.1")
     implementation("no.nav.poao.dab:spring-auth:$dab_common_version")
     implementation("no.nav.poao.dab:spring-a2-annotations:$dab_common_version")
 
@@ -173,15 +173,13 @@ dependencies {
     implementation("io.micrometer:micrometer-registry-prometheus-simpleclient")
     implementation("org.flywaydb:flyway-database-postgresql:10.15.2")
     implementation("org.postgresql:postgresql:42.7.4")
-    implementation("com.fasterxml.jackson.dataformat:jackson-dataformat-yaml:2.18.1")
-    // Hvis det ønskes swagger doc, foreslås å bruke springdoc (springdoc-openapi-starter-webmvc-ui - se no.nav.fo.veilarbdialog.rest.SwaggerConfig for eksempelconfig)
-    implementation("io.swagger.core.v3:swagger-annotations:2.2.26")
+    implementation("com.fasterxml.jackson.dataformat:jackson-dataformat-yaml:2.18.2")
 
     // BigQuery
-    implementation(platform("com.google.cloud:libraries-bom:26.50.0"))
+    implementation(platform("com.google.cloud:libraries-bom:26.51.0"))
     implementation("com.google.cloud:google-cloud-bigquery")
 
-    implementation("io.getunleash:unleash-client-java:9.2.5")
+    implementation("io.getunleash:unleash-client-java:9.2.6")
 
     runtimeOnly("org.springframework.boot:spring-boot-devtools")
 
@@ -199,7 +197,7 @@ dependencies {
 //    testImplementation("de.mkammerer.wiremock-junit5:wiremock-junit5:1.1.0")
 //    testImplementation("io.github.ricall.junit5-wiremock:junit5-wiremock:2.0.0")
 
-    testImplementation("org.mockito.kotlin:mockito-kotlin:3.2.0")
+    testImplementation("org.mockito.kotlin:mockito-kotlin:5.4.0")
 
 //spring managed test dependencies
     testImplementation("io.rest-assured:rest-assured")
@@ -208,6 +206,6 @@ dependencies {
     }
     testImplementation("org.springframework.kafka:spring-kafka-test")
     testImplementation("org.mockito:mockito-core")
-    testImplementation("io.zonky.test:embedded-database-spring-test:2.5.1")
+    testImplementation("io.zonky.test:embedded-database-spring-test:2.6.0")
     testImplementation("io.zonky.test:embedded-postgres:2.1.0")
 }

src/main/java/no/nav/veilarbaktivitet/veilarbportefolje/dto/AktivitetTypeDTO.java

@@ -20,6 +20,7 @@ public enum AktivitetTypeDTO {
     static {
         aktivitetskortTypeTilArenaTiltakskode.put(AktivitetskortType.MIDLERTIDIG_LONNSTILSKUDD, "MIDLONTIL");
         aktivitetskortTypeTilArenaTiltakskode.put(AktivitetskortType.VARIG_LONNSTILSKUDD, "VARLONTIL");
+        aktivitetskortTypeTilArenaTiltakskode.put(AktivitetskortType.ARBEIDSTRENING, "ARBTREN");
         aktivitetskortTypeTilArenaTiltakskode.put(AktivitetskortType.INDOPPFAG, "INDOPPFAG");
         aktivitetskortTypeTilArenaTiltakskode.put(AktivitetskortType.ARBFORB, "ARBFORB");
         aktivitetskortTypeTilArenaTiltakskode.put(AktivitetskortType.AVKLARAG, "AVKLARAG");

src/main/kotlin/no/nav/veilarbaktivitet/aktivitetskort/dto/AktivitetskortType.kt

@@ -2,6 +2,7 @@ package no.nav.veilarbaktivitet.aktivitetskort.dto
 /*
       MIDLERTIDIG_LONNSTILSKUDD - Midlertidig lønnstilskudd
       VARIG_LONNSTILSKUDD - Varig lønnstilskudd
+      ARBEIDSTRENING - Arbeidstrening
       INDOPPFAG - Oppfølging
       ARBFORB - Arbeidsforberedende trening (AFT)
       AVKLARAG  - Avklaring
@@ -16,6 +17,7 @@ package no.nav.veilarbaktivitet.aktivitetskort.dto
 enum class AktivitetskortType {
     MIDLERTIDIG_LONNSTILSKUDD,
     VARIG_LONNSTILSKUDD,
+    ARBEIDSTRENING,
     ARENA_TILTAK,
     INDOPPFAG,
     ARBFORB,

src/main/kotlin/no/nav/veilarbaktivitet/aktivitetskort/util/JsonUtil.kt

@@ -16,17 +16,17 @@ object JsonUtil {
             return token == JsonToken.END_OBJECT && level == 0
         }
 
-        parser.use { parser ->
+        parser.use {
             var level = 0
             do {
-                val nextToken = parser.nextToken()
+                val nextToken = it.nextToken()
                 when (nextToken) {
                     JsonToken.END_OBJECT -> { level -= 1 }
                     JsonToken.START_OBJECT -> { level += 1 }
                     JsonToken.VALUE_STRING -> {
-                        val fieldName = parser.currentName
+                        val fieldName = it.currentName()
                         if (propertyName == fieldName) {
-                            result = parser.valueAsString
+                            result = it.valueAsString
                             break
                         }
                     }

src/main/kotlin/no/nav/veilarbaktivitet/arkivering/mapper/AktivitetDtoToArkivPayload.kt

@@ -58,6 +58,7 @@ fun AktivitetData.toArkivTypeTekst(): String {
                 AktivitetskortType.ARENA_TILTAK -> "Tiltak gjennom NAV"
                 AktivitetskortType.MIDLERTIDIG_LONNSTILSKUDD -> "Avtale midlertidig lønnstilskudd"
                 AktivitetskortType.VARIG_LONNSTILSKUDD -> "Avtale varig lønnstilskudd"
+                AktivitetskortType.ARBEIDSTRENING -> "Arbeidstrening"
                 AktivitetskortType.INDOPPFAG -> "Oppfølging"
                 AktivitetskortType.ARBFORB -> "Arbeidsforberedende trening"
                 AktivitetskortType.AVKLARAG -> "Avklaring"

src/main/kotlin/no/nav/veilarbaktivitet/brukernotifikasjon/opprettVarsel/UtgåendeVarsel.kt

@@ -3,6 +3,7 @@ package no.nav.veilarbaktivitet.brukernotifikasjon.opprettVarsel
 import no.nav.veilarbaktivitet.brukernotifikasjon.VarselStatus
 import no.nav.veilarbaktivitet.brukernotifikasjon.VarselType
 import no.nav.veilarbaktivitet.person.Person
+import java.net.URI
 import java.net.URL
 import java.util.*
 
@@ -20,5 +21,5 @@ data class UtgåendeVarsel (
 )
 
 fun createAktivitetLink(aktivitetsplanBasepath: String, aktivitetId: String): URL {
-    return URL(aktivitetsplanBasepath + "/aktivitet/vis/" + aktivitetId)
+    return URI.create("$aktivitetsplanBasepath/aktivitet/vis/$aktivitetId").toURL()
 }

src/main/kotlin/no/nav/veilarbaktivitet/oppfolging/periode/OppfolgingsperiodeUtil.kt

@@ -13,37 +13,37 @@ import kotlin.math.abs
 val log = LoggerFactory.getLogger("no.nav.veilarbaktivitet.oppfolging.periode.OppfolgingsperiodeUtil")!!
 
 
-fun finnOppfolgingsperiodeForArenaAktivitet(oppfolgingsperioder: List<Oppfolgingsperiode>, primærDato: LocalDate?): Oppfolgingsperiode? {
-    return primærDato?.let { oppfolgingsperioder.finnOppfolgingsperiodeForTidspunkt(it.atStartOfDay()) }
+fun finnOppfolgingsperiodeForArenaAktivitet(oppfolgingsperioder: List<Oppfolgingsperiode>, aktivitetOppslagsdato: LocalDate?): Oppfolgingsperiode? {
+    return aktivitetOppslagsdato?.let { oppfolgingsperioder.finnOppfolgingsperiodeForTidspunkt(it.atStartOfDay()) }
 }
 
-fun List<Oppfolgingsperiode>.finnOppfolgingsperiodeForTidspunkt(tidspunkt: LocalDateTime): Oppfolgingsperiode? {
+fun List<Oppfolgingsperiode>.finnOppfolgingsperiodeForTidspunkt(aktivitetOppslagsdato: LocalDateTime): Oppfolgingsperiode? {
     val oppfolgingsperioder = this.sortedByDescending { it.startTid }
     if (oppfolgingsperioder.isEmpty()) {
-        log.info("Arenatiltak finn oppfølgingsperiode - bruker har ingen oppfølgingsperioder - tidspunkt=${tidspunkt}, oppfolgingsperioder=${listOf<OppfolgingPeriodeMinimalDTO>()}")
+        log.info("Arenatiltak finn oppfølgingsperiode - bruker har ingen oppfølgingsperioder - tidspunkt=${aktivitetOppslagsdato}, oppfolgingsperioder=${listOf<OppfolgingPeriodeMinimalDTO>()}")
         return null
     }
 
-    val opprettetTidspunktCZDT = tidspunkt.atZone(ZoneId.systemDefault())
+    val opprettetTidspunktCDT = aktivitetOppslagsdato.atZone(ZoneId.systemDefault())
 
     val match = oppfolgingsperioder
-        .firstOrNull { oppfolgingsperiode -> oppfolgingsperiode.erInnenforPeriode(opprettetTidspunktCZDT) }
+        .firstOrNull { oppfolgingsperiode -> oppfolgingsperiode.erInnenforPeriode(opprettetTidspunktCDT) }
         ?: oppfolgingsperioder
             .filterIndexed { index, oppfolgingsperiode ->
                 val forrigePeriode = oppfolgingsperioder.getOrNull(index + 1)
-                val startetEtterAtForrigePeriodeBleAvsluttet = forrigePeriode?.sluttTid?.let { opprettetTidspunktCZDT.isAfter(it) } ?: false
-                val startetRettFørDennePeriodenBleStartet = oppfolgingsperiode.erInnenforMedEkstraSlack(opprettetTidspunktCZDT)
-                startetEtterAtForrigePeriodeBleAvsluttet || startetRettFørDennePeriodenBleStartet
+                val aktivitetStartetEtterAtForrigePeriodeBleAvsluttet = forrigePeriode?.sluttTid?.let { opprettetTidspunktCDT.isAfter(it) } ?: false
+                val aktivitetStartetRettFørDennePeriodenBleStartet = oppfolgingsperiode.erInnenforMedEkstraSlack(opprettetTidspunktCDT)
+                aktivitetStartetEtterAtForrigePeriodeBleAvsluttet || aktivitetStartetRettFørDennePeriodenBleStartet
             }
-            .minByOrNull { abs(ChronoUnit.MILLIS.between(opprettetTidspunktCZDT, it.startTid)) }
+            .minByOrNull { abs(ChronoUnit.MILLIS.between(opprettetTidspunktCDT, it.startTid)) }
             ?.also { _ ->
-                log.info("Arenatiltak finn oppfølgingsperiode - valgt oppfølgingsperiode som startet etter opprettetdato) - tidspunkt=${tidspunkt}, oppfolgingsperioder=${oppfolgingsperioder}")
+                log.info("Arenatiltak finn oppfølgingsperiode - valgt oppfølgingsperiode som startet etter opprettetdato) - tidspunkt=${aktivitetOppslagsdato}, oppfolgingsperioder=${oppfolgingsperioder}")
             }
 
     return if (match != null) {
         match
     } else {
-        log.info("Arenatiltak finn oppfølgingsperiode - opprettetTidspunkt har ingen god match på oppfølgingsperioder) - tidspunkt=${tidspunkt}, oppfolgingsperioder=${oppfolgingsperioder}")
+        log.info("Arenatiltak finn oppfølgingsperiode - opprettetTidspunkt har ingen god match på oppfølgingsperioder) - tidspunkt=${aktivitetOppslagsdato}, oppfolgingsperioder=${oppfolgingsperioder}")
         null
     }
 }

src/main/kotlin/no/nav/veilarbaktivitet/unleash/FeatureController.kt

@@ -38,6 +38,7 @@ class FeatureController(
             .map { it.value }
             .orElse(generateSessionId(response))
 
+        // TODO Hva med eksternbruker?
         val id = authService.getLoggedInnUser().takeIf { it is NavIdent }?.get()
 
         val unleashContext = UnleashContext(
@@ -50,8 +51,8 @@ class FeatureController(
         try {
             return features.associateWith { unleash.isEnabled(it, unleashContext) }
         } catch (e: Exception) {
-            log.error("Feil under henting av features for unleash", e);
-            throw ResponseStatusException(HttpStatus.INTERNAL_SERVER_ERROR, "Feil under henting av features for unleash");
+            log.error("Feil under henting av features for unleash", e)
+            throw ResponseStatusException(HttpStatus.INTERNAL_SERVER_ERROR, "Feil under henting av features for unleash")
         }
     }
 

src/main/kotlin/no/nav/veilarbaktivitet/veilarbdbutil/VeilarbAktivitetResultSet.kt

@@ -18,12 +18,13 @@ class VeilarbAktivitetResultSet(private val resultSet: ResultSet): ResultSet by
     }
     override fun isWrapperFor( iface: Class<*>): Boolean {
         // TODO Auto-generated method stub
-        return iface != null && iface.isAssignableFrom(this.javaClass)
+        return iface.isAssignableFrom(this.javaClass)
     }
-    override fun <T> unwrap( iface: Class<T>): T {
+    @Suppress("UNCHECKED_CAST")
+    override fun <T> unwrap(iface: Class<T>): T {
         // TODO Auto-generated method stub
         try {
-            if (iface != null && iface.isAssignableFrom(this.javaClass)) {
+            if (iface.isAssignableFrom(this.javaClass)) {
                 return this as T
             }
             throw  java.sql.SQLException("Auto-generated unwrap failed; Revisit implementation")

src/main/resources/db/migration/V9__grant_lesetilgang_til_clousqliamuser.sql

@@ -0,0 +1,11 @@
+DO
+$$
+    BEGIN
+        IF (SELECT exists(SELECT rolname FROM pg_roles WHERE rolname = 'cloudsqliamuser'))
+        THEN
+            GRANT USAGE ON SCHEMA veilarbaktivitet to "cloudsqliamuser";
+            GRANT SELECT ON ALL TABLES IN SCHEMA veilarbaktivitet TO "cloudsqliamuser";
+            ALTER DEFAULT PRIVILEGES IN SCHEMA veilarbaktivitet GRANT SELECT ON TABLES TO "cloudsqliamuser";
+        END IF;
+    END
+$$;

src/main/resources/graphql/schema.graphqls

@@ -145,6 +145,7 @@ enum Sentiment {
 enum AktivitetskortType {
     MIDLERTIDIG_LONNSTILSKUDD
     VARIG_LONNSTILSKUDD
+    ARBEIDSTRENING
     ARENA_TILTAK
     INDOPPFAG
     ARBFORB

src/main/resources/schemas/akaas/Aktivitetskort.V1.shared-definitions.schema.yml

@@ -14,6 +14,7 @@ definitions:
     enum:
       - MIDLERTIDIG_LONNSTILSKUDD
       - VARIG_LONNSTILSKUDD
+      - ARBEIDSTRENING
       - INDOPPFAG
       - ARBFORB
       - AVKLARAG
@@ -28,6 +29,7 @@ definitions:
       Aktivitetskort typer som er tillatt å opprette via tjenesten. Denne enumereringen vil utvides etterhvert.
       MIDLERTIDIG_LONNSTILSKUDD - Midlertidig lønnstilskudd
       VARIG_LONNSTILSKUDD - Varig lønnstilskudd
+      ARBEIDSTRENING - Arbeidstrening
       INDOPPFAG - Oppfølging
       ARBFORB - Arbeidsforberedende trening (AFT)
       AVKLARAG  - Avklaring