Skip to content

Commit

Permalink
Oppgraderer til token-support 4 (#505)
Browse files Browse the repository at this point in the history
  • Loading branch information
olekvernberg authored Mar 19, 2024
1 parent 4450ca0 commit 820431d
Show file tree
Hide file tree
Showing 8 changed files with 22 additions and 20 deletions.
4 changes: 2 additions & 2 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -37,8 +37,8 @@
<exec-maven-plugin.version>3.1.1</exec-maven-plugin.version>
<kontrakter.version>3.0_20240215101759_cea211f</kontrakter.version>
<eksterne-kontrakter.version>2.0_20240131125409_e3d0f6d</eksterne-kontrakter.version>
<felles.version>2.20240123084817_35f03aa</felles.version>
<nav.security.version>3.2.0</nav.security.version> <!-- Denne burde være samme versjon som i felles -->
<felles.version>3.20240227112118_e55c4be</felles.version>
<nav.security.version>4.1.3</nav.security.version> <!-- Denne burde være samme versjon som i felles -->
<okhttp3.version>4.9.1</okhttp3.version> <!-- overskrever spring sin versjon, blir brukt av mock-oauth2-server -->
<wiremock.version>2.35.1</wiremock.version>
<mockk-jvm.version>1.13.9</mockk-jvm.version>
Expand Down
Original file line number Diff line number Diff line change
@@ -1,13 +1,13 @@
package no.nav.familie.ef.personhendelse.configuration

import com.fasterxml.jackson.module.kotlin.KotlinModule
import no.nav.familie.http.client.RetryOAuth2HttpClient
import no.nav.familie.http.config.RestTemplateAzure
import no.nav.familie.kafka.KafkaErrorHandler
import no.nav.familie.kontrakter.felles.objectMapper
import no.nav.familie.log.filter.LogFilter
import no.nav.familie.log.filter.RequestTimeFilter
import no.nav.security.token.support.client.core.http.OAuth2HttpClient
import no.nav.security.token.support.client.spring.oauth2.DefaultOAuth2HttpClient
import no.nav.security.token.support.client.spring.oauth2.EnableOAuth2Client
import no.nav.security.token.support.spring.api.EnableJwtTokenValidation
import org.slf4j.LoggerFactory
Expand All @@ -20,6 +20,7 @@ import org.springframework.context.annotation.Import
import org.springframework.context.annotation.Primary
import org.springframework.http.converter.json.MappingJackson2HttpMessageConverter
import org.springframework.scheduling.annotation.EnableScheduling
import org.springframework.web.client.RestClient
import org.springframework.web.client.RestTemplate
import java.time.Duration
import java.time.temporal.ChronoUnit
Expand Down Expand Up @@ -80,13 +81,15 @@ class ApplicationConfig {
* pga .setVisibility(PropertyAccessor.SETTER, JsonAutoDetect.Visibility.NONE)
* og [OAuth2AccessTokenResponse] som burde settes med setters, då feltnavn heter noe annet enn feltet i json
*/
@Bean
@Primary
@Bean
fun oAuth2HttpClient(): OAuth2HttpClient {
return DefaultOAuth2HttpClient(
RestTemplateBuilder()
.setConnectTimeout(Duration.of(2, ChronoUnit.SECONDS))
.setReadTimeout(Duration.of(4, ChronoUnit.SECONDS)),
return RetryOAuth2HttpClient(
RestClient.create(
RestTemplateBuilder()
.setConnectTimeout(Duration.of(2, ChronoUnit.SECONDS))
.setReadTimeout(Duration.of(4, ChronoUnit.SECONDS)).build(),
),
)
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ class KontantstøtteVedtakListener(val kontantstøtteVedtakService: Kontantstøt
try {
logger.info("Leser vedtak for kontantstøtte med behandlingId: ${vedtakhendelse.behandlingsId}")
if (erAlleredeHåndtert(vedtakhendelse)) {
logger.info("Har allerede håndtert kontantstøttevedtak med behandlingId=${vedtakhendelse.behandlingsId}")
logger.info("Har allerede håndtert kontantstøttevedtak med behandlingId=${vedtakhendelse.behandlingsId}")
} else {
opprettOppgaveHvisPersonHarLøpendeBarnetilsyn(vedtakhendelse.person.personIdent, vedtakhendelse.behandlingsId)
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ class KontantstøtteVedtakService(
fun lagreKontantstøttehendelse(behandlingId: String) {
kontantstøtteVedtakRepository.lagreKontantstøttevedtak(behandlingId)
}

fun erAlleredeHåndtert(behandlingId: String): Boolean {
return kontantstøtteVedtakRepository.harAlleredeProsessertKontantstøttevedtak(behandlingId)
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ class IntegrasjonstestConfig {
fun oAuth2AccessTokenServiceMock(): OAuth2AccessTokenService {
val tokenMockService = mockk<OAuth2AccessTokenService>()
every { tokenMockService.getAccessToken(any()) }
.returns(OAuth2AccessTokenResponse("Mock-token-response", 60, 60, null))
.returns(OAuth2AccessTokenResponse("Mock-token-response", 60, 60, emptyMap()))
return tokenMockService
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ import io.mockk.every
import io.mockk.just
import io.mockk.mockk
import io.mockk.verify
import java.time.ZonedDateTime
import no.nav.familie.eksterne.kontrakter.BehandlingType
import no.nav.familie.eksterne.kontrakter.BehandlingÅrsak
import no.nav.familie.eksterne.kontrakter.Kategori
Expand All @@ -16,6 +15,7 @@ import org.apache.kafka.clients.consumer.ConsumerRecord
import org.junit.jupiter.api.BeforeEach
import org.junit.jupiter.api.Test
import org.springframework.kafka.support.Acknowledgment
import java.time.ZonedDateTime

class KontantstøtteVedtakListenerTest {
private val kontantstøtteVedtakServiceMock = mockk<KontantstøtteVedtakService>()
Expand All @@ -32,7 +32,7 @@ class KontantstøtteVedtakListenerTest {
kompetanseperioder = listOf(),
funksjonellId = "veri",
behandlingÅrsak = BehandlingÅrsak.SØKNAD,
vilkårResultater = listOf()
vilkårResultater = listOf(),
)

val vedtakDvhAsJson = objectMapper.writeValueAsString(vedtakDvh)
Expand Down Expand Up @@ -69,4 +69,4 @@ class KontantstøtteVedtakListenerTest {
verify(exactly = 1) { kontantstøtteVedtakServiceMock.erAlleredeHåndtert(vedtakDvh.behandlingsId) }
verify(exactly = 1) { kontantstøtteVedtakServiceMock.opprettVurderKonsekvensOppgaveForBarnetilsyn(any(), any()) }
}
}
}
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
package no.nav.familie.ef.personhendelse.kontantstøtte.vedtak

import java.util.UUID
import no.nav.familie.ef.personhendelse.IntegrasjonSpringRunnerTest
import org.assertj.core.api.Assertions.assertThat
import org.junit.jupiter.api.Test
import org.springframework.beans.factory.annotation.Autowired
import java.util.UUID

class KontantstøtteVedtakRepositoryTest : IntegrasjonSpringRunnerTest() {

Expand All @@ -19,12 +19,12 @@ class KontantstøtteVedtakRepositoryTest : IntegrasjonSpringRunnerTest() {

assertThat(
kontantstøtteVedtakRepository.harAlleredeProsessertKontantstøttevedtak(
behandlingAlleredeHåndtert
)
behandlingAlleredeHåndtert,
),
).isTrue()

assertThat(
kontantstøtteVedtakRepository.harAlleredeProsessertKontantstøttevedtak(behandlingIkkeHåndtert)
kontantstøtteVedtakRepository.harAlleredeProsessertKontantstøttevedtak(behandlingIkkeHåndtert),
).isFalse()
}
}
}
1 change: 0 additions & 1 deletion src/test/resources/application-integrasjonstest.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@ no.nav.security.jwt:
issuer.azuread:
discoveryurl: http://localhost:${mock-oauth2-server.port}/azuread/.well-known/openid-configuration
accepted_audience: aud-localhost
cookie_name: localhost-idtoken
spring:
kafka:
bootstrap-servers: http://localhost:9092
Expand Down

0 comments on commit 820431d

Please sign in to comment.