Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

User infov3 #392

Merged
merged 5 commits into from
Dec 13, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
@@ -1,10 +1,8 @@
package no.nav.arbeidsgiver.min_side.models

import com.fasterxml.jackson.annotation.JsonProperty

data class Organisasjon(
@field:JsonProperty("Name") var name: String,
@field:JsonProperty("ParentOrganizationNumber") var parentOrganizationNumber: String? = null,
@field:JsonProperty("OrganizationNumber") var organizationNumber: String,
@field:JsonProperty("OrganizationForm") var organizationForm: String,
var name: String,
var parentOrganizationNumber: String? = null,
var organizationNumber: String,
var organizationForm: String,
)
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,14 @@ class DigisyfoService(
val antallSykmeldte: Int,
)

data class VirksomhetOgAntallSykmeldteV3(
val orgnr: String,
val navn: String,
val organisasjonsform: String,
val antallSykmeldte: Int,
val underenheter: List<VirksomhetOgAntallSykmeldteV3>
)

fun hentVirksomheterOgSykmeldte(fnr: String): Collection<VirksomhetOgAntallSykmeldte> {
val underenheter = digisyfoRepository.virksomheterOgSykmeldte(fnr)
.flatMap { hentUnderenhet(it) }
Expand All @@ -36,6 +44,44 @@ class DigisyfoService(
return resultat.distinctBy { it.organisasjon.organizationNumber }
}

fun hentVirksomheterOgSykmeldteV3(fnr: String): List<VirksomhetOgAntallSykmeldteV3> {
val underenheter = digisyfoRepository.virksomheterOgSykmeldte(fnr)
.flatMap { hentUnderenhet(it) }

val alleOverenheter = underenheter
.mapNotNull { it.organisasjon.parentOrganizationNumber }
.toSet()
.flatMap { hentOverenhet(it) }
.distinctBy { it.organisasjon.organizationNumber }

//TODO når v2 er borte kan vi kanskje bygge treet mens vi henter overenheter?

val alleOrganisasjoner = underenheter + alleOverenheter

fun byggHierarki(org: VirksomhetOgAntallSykmeldte): VirksomhetOgAntallSykmeldteV3 {
val direkteUnderenheter = alleOrganisasjoner.filter {
it.organisasjon.parentOrganizationNumber == org.organisasjon.organizationNumber
}
return VirksomhetOgAntallSykmeldteV3(
orgnr = org.organisasjon.organizationNumber,
navn = org.organisasjon.name,
organisasjonsform = org.organisasjon.organizationForm,
antallSykmeldte = org.antallSykmeldte,
underenheter = direkteUnderenheter.map(::byggHierarki)
)
}

meterRegistry.counter(
"msa.digisyfo.tilgang",
"virksomheter",
underenheter.size.toString()
).increment()

return alleOverenheter
.filter { it.organisasjon.parentOrganizationNumber == null }
.map(::byggHierarki)
}

private fun hentForfedre(org: Organisasjon, orgs: MutableList<Organisasjon> = mutableListOf()): List<Organisasjon> {
if (org.parentOrganizationNumber == null) return orgs

Expand Down
Original file line number Diff line number Diff line change
@@ -1,12 +1,12 @@
package no.nav.arbeidsgiver.min_side.userinfo

import com.fasterxml.jackson.annotation.JsonProperty
import kotlinx.coroutines.async
import kotlinx.coroutines.supervisorScope
import no.nav.arbeidsgiver.min_side.controller.AuthenticatedUserHolder
import no.nav.arbeidsgiver.min_side.services.altinn.AltinnService
import no.nav.arbeidsgiver.min_side.services.altinn.AltinnTilganger
import no.nav.arbeidsgiver.min_side.services.digisyfo.DigisyfoService
import no.nav.arbeidsgiver.min_side.services.digisyfo.DigisyfoService.VirksomhetOgAntallSykmeldte
import no.nav.arbeidsgiver.min_side.services.tiltak.RefusjonStatusService
import org.springframework.web.bind.annotation.GetMapping
import org.springframework.web.bind.annotation.RestController
Expand All @@ -18,6 +18,26 @@ class UserInfoController(
private val refusjonStatusService: RefusjonStatusService,
private val authenticatedUserHolder: AuthenticatedUserHolder,
) {
@GetMapping("/api/userInfo/v3")
suspend fun getUserInfoV3() = supervisorScope {
val tilganger = async {
runCatching {
altinnService.hentAltinnTilganger()
}
}

val syfoVirksomheter = async {
runCatching {
digisyfoService.hentVirksomheterOgSykmeldteV3(authenticatedUserHolder.fnr)
}
}
val refusjoner = async {
runCatching {
refusjonStatusService.statusoversikt(authenticatedUserHolder.fnr)
}
}
UserInfoResponsV3.from(tilganger.await(), syfoVirksomheter.await(), refusjoner.await())
}

@GetMapping("/api/userInfo/v2")
suspend fun getUserInfo() = supervisorScope {
Expand Down Expand Up @@ -48,13 +68,25 @@ data class UserInfoRespons(
val digisyfoError: Boolean,
val organisasjoner: List<AltinnTilganger.AltinnTilgang>,
val tilganger: Map<String, Collection<String>>,
val digisyfoOrganisasjoner: Collection<VirksomhetOgAntallSykmeldte>,
val digisyfoOrganisasjoner: Collection<VirksomhetOgAntallSykmeldteV2>,
val refusjoner: List<RefusjonStatusService.Statusoversikt>,
) {
companion object {
data class OrganisasjonV2(
@field:JsonProperty("Name") var name: String,
@field:JsonProperty("ParentOrganizationNumber") var parentOrganizationNumber: String? = null,
@field:JsonProperty("OrganizationNumber") var organizationNumber: String,
@field:JsonProperty("OrganizationForm") var organizationForm: String,
)

data class VirksomhetOgAntallSykmeldteV2(
val organisasjon: OrganisasjonV2,
val antallSykmeldte: Int,
)

fun from(
tilgangerResult: Result<AltinnTilganger>,
syfoResult: Result<Collection<VirksomhetOgAntallSykmeldte>>,
syfoResult: Result<Collection<DigisyfoService.VirksomhetOgAntallSykmeldte>>,
refusjonerResult: Result<List<RefusjonStatusService.Statusoversikt>>
) = UserInfoRespons(
digisyfoError = syfoResult.isFailure,
Expand All @@ -69,6 +101,58 @@ data class UserInfoRespons(
onFailure = { emptyList() }
),

digisyfoOrganisasjoner = syfoResult.fold(
onSuccess = { it.map{virksomhetOgAntallSykmeldte -> VirksomhetOgAntallSykmeldteV2(
antallSykmeldte = virksomhetOgAntallSykmeldte.antallSykmeldte,
organisasjon = OrganisasjonV2(
name = virksomhetOgAntallSykmeldte.organisasjon.name,
parentOrganizationNumber = virksomhetOgAntallSykmeldte.organisasjon.parentOrganizationNumber,
organizationNumber = virksomhetOgAntallSykmeldte.organisasjon.organizationNumber,
organizationForm = virksomhetOgAntallSykmeldte.organisasjon.organizationForm
))
} },
onFailure = { emptyList() }
),

refusjoner = refusjonerResult.fold(
onSuccess = { it },
onFailure = { emptyList() }
),

tilganger = tilgangerResult.fold(
onSuccess = { it.tilgangTilOrgNr },
onFailure = { emptyMap() }
),
)
}
}

data class UserInfoResponsV3(
val altinnError: Boolean,
val digisyfoError: Boolean,
val organisasjoner: List<AltinnTilganger.AltinnTilgang>,
val tilganger: Map<String, Collection<String>>,
val digisyfoOrganisasjoner: List<DigisyfoService.VirksomhetOgAntallSykmeldteV3>,
val refusjoner: List<RefusjonStatusService.Statusoversikt>,
) {
companion object {
fun from(
tilgangerResult: Result<AltinnTilganger>,
syfoResult: Result<List<DigisyfoService.VirksomhetOgAntallSykmeldteV3>>,
refusjonerResult: Result<List<RefusjonStatusService.Statusoversikt>>
) = UserInfoResponsV3(
digisyfoError = syfoResult.isFailure,

altinnError = tilgangerResult.fold(
onSuccess = { it.isError || refusjonerResult.isFailure },
onFailure = { true }
),

organisasjoner = tilgangerResult.fold(
onSuccess = { it.hierarki },
onFailure = { emptyList() }
),

digisyfoOrganisasjoner = syfoResult.fold(
onSuccess = { it },
onFailure = { emptyList() }
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -108,6 +108,22 @@ class DigisyfoServiceTest {
)
}

@Test
fun `nestede rettigheter hentVirksomheterOgSykmeldteV3`() {
Mockito.`when`(digisyfoRepository.virksomheterOgSykmeldte("42")).thenReturn(
listOf(
DigisyfoRepository.Virksomhetsinfo("3000", 2),
DigisyfoRepository.Virksomhetsinfo("301", 1),
DigisyfoRepository.Virksomhetsinfo("20", 1),
DigisyfoRepository.Virksomhetsinfo("10", 1),
DigisyfoRepository.Virksomhetsinfo("11", 1),
)
)

val result = digisyfoService.hentVirksomheterOgSykmeldteV3("42")
assertThat(result).isEqualTo(digisyfoVirksomheterHieraki)
}


private fun mkUnderenhet(orgnr: String, parentOrgnr: String) =
Organisasjon(
Expand All @@ -123,4 +139,82 @@ class DigisyfoServiceTest {
organizationNumber = orgnr,
organizationForm = "AS",
)
}
}

val digisyfoVirksomheterHieraki = listOf(
DigisyfoService.VirksomhetOgAntallSykmeldteV3(
orgnr = "3",
navn = "overenhet",
organisasjonsform = "AS",
antallSykmeldte = 0,
underenheter = listOf(
DigisyfoService.VirksomhetOgAntallSykmeldteV3(
orgnr = "30",
navn = "underenhet",
organisasjonsform = "BEDR",
antallSykmeldte = 0,
underenheter = listOf(
DigisyfoService.VirksomhetOgAntallSykmeldteV3(
orgnr = "301",
navn = "underenhet",
organisasjonsform = "BEDR",
antallSykmeldte = 1,
underenheter = listOf()
),
DigisyfoService.VirksomhetOgAntallSykmeldteV3(
orgnr = "300",
navn = "underenhet",
organisasjonsform = "BEDR",
antallSykmeldte = 0,
underenheter = listOf(
DigisyfoService.VirksomhetOgAntallSykmeldteV3(
orgnr = "3000",
navn = "underenhet",
organisasjonsform = "BEDR",
antallSykmeldte = 2,
underenheter = listOf()
)
)
)
)
)
)
),
DigisyfoService.VirksomhetOgAntallSykmeldteV3(
orgnr = "2",
navn = "overenhet",
organisasjonsform = "AS",
antallSykmeldte = 0,
underenheter = listOf(
DigisyfoService.VirksomhetOgAntallSykmeldteV3(
orgnr = "20",
navn = "underenhet",
organisasjonsform = "BEDR",
antallSykmeldte = 1,
underenheter = listOf()
)
)
),
DigisyfoService.VirksomhetOgAntallSykmeldteV3(
orgnr = "1",
navn = "overenhet",
organisasjonsform = "AS",
antallSykmeldte = 0,
underenheter = listOf(
DigisyfoService.VirksomhetOgAntallSykmeldteV3(
orgnr = "10",
navn = "underenhet",
organisasjonsform = "BEDR",
antallSykmeldte = 1,
underenheter = listOf()
),
DigisyfoService.VirksomhetOgAntallSykmeldteV3(
orgnr = "11",
navn = "underenhet",
organisasjonsform = "BEDR",
antallSykmeldte = 1,
underenheter = listOf()
)
)
)
)
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@ import org.springframework.boot.test.mock.mockito.MockBean
import org.springframework.context.annotation.Bean
import org.springframework.context.annotation.Import
import org.springframework.security.oauth2.jwt.JwtDecoder
import org.springframework.security.test.web.servlet.request.SecurityMockMvcRequestPostProcessors.anonymous
import org.springframework.test.web.servlet.MockMvc
import org.springframework.test.web.servlet.get
import org.springframework.web.filter.CharacterEncodingFilter
Expand Down Expand Up @@ -110,6 +109,40 @@ class UserInfoControllerTest {
),
)
)
`when`(digisyfoService.hentVirksomheterOgSykmeldteV3("42")).thenReturn(
listOf(
DigisyfoService.VirksomhetOgAntallSykmeldteV3(
navn = "overenhet",
orgnr = "1",
organisasjonsform = "AS",
antallSykmeldte = 0,
underenheter = listOf(
DigisyfoService.VirksomhetOgAntallSykmeldteV3(
navn = "underenhet",
orgnr = "10",
organisasjonsform = "BEDR",
antallSykmeldte = 0,
underenheter = listOf(),
),
)
),
DigisyfoService.VirksomhetOgAntallSykmeldteV3(
navn = "overenhet",
orgnr = "2",
organisasjonsform = "AS",
antallSykmeldte = 0,
underenheter = listOf(
DigisyfoService.VirksomhetOgAntallSykmeldteV3(
navn = "underenhet",
orgnr = "20",
organisasjonsform = "BEDR",
antallSykmeldte = 1,
underenheter = listOf(),
),
)
)
)
)
`when`(refusjonStatusService.statusoversikt("42")).thenReturn(
listOf(
RefusjonStatusService.Statusoversikt(
Expand Down Expand Up @@ -227,14 +260,23 @@ class UserInfoControllerTest {
)
}
}
}

@Test
fun `returnerer http 401 for ikke innlogget bruker`() {
mockMvc.get("/api/userInfo/v1") {
with(anonymous())
}.andExpect {
status { isUnauthorized() }
mockMvc.get("/api/userInfo/v3") {
with(jwtWithPid("42"))
}.asyncDispatch().andExpect {
status { isOk() }
content {
jsonPath("$.organisasjoner.length()") { value(1) }
}
}

mockMvc.get("/api/userInfo/v3") {
with(jwtWithPid("42"))
}.asyncDispatch().andExpect {
status { isOk() }
content {
jsonPath("$.digisyfoOrganisasjoner.length()") { value(2) }
}
}
}

Expand Down
Loading