Skip to content

Commit

Permalink
Hent inntekter lenger frem fix (#110)
Browse files Browse the repository at this point in the history
* CookieTokenFilter for lokal utvikling

Siste oppgradering av token-validation-spring (v4)
fjernet støtte for tokens som ligger i en cookie.
Det var vår måte å "logge inn" på i lokal utvikling.

Ved å redusere "presedensen" til valideringsfilteret
og sette inn et "CookieTokenFilter" foran, kan vi
manuelt sette inn et token i auth-header som fanges
opp av valideringsfilteret senere.

* lagt til nye funksjoner for unntak om inntekter to mnd frem

* fjernet dobble funksjoner

---------

Co-authored-by: Odd Andreas Sørsæther <[email protected]>
  • Loading branch information
MagnusRom and Oddsor authored Mar 18, 2024
1 parent b704ec5 commit 953d1df
Show file tree
Hide file tree
Showing 6 changed files with 16 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -223,9 +223,9 @@ data class InnloggetSaksbehandler(
return refusjon
}

fun merkForUnntakOmInntekterToMånederFrem(id: String, merking: Int) {
fun merkForUnntakOmInntekterFremITid(id: String, merking: Int) {
val refusjon = finnRefusjon(id)
refusjon.merkForUnntakOmInntekterToMånederFrem(merking, this)
refusjon.merkForUnntakOmInntekterFremITid(merking, this)
refusjonRepository.save(refusjon)
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -363,7 +363,7 @@ class Refusjon(
return id.hashCode()
}

fun merkForUnntakOmInntekterToMånederFrem(merking: Int, utførtAv: InnloggetBruker) {
fun merkForUnntakOmInntekterFremITid(merking: Int, utførtAv: InnloggetBruker) {
krevStatus(RefusjonStatus.FOR_TIDLIG, RefusjonStatus.KLAR_FOR_INNSENDING)
if(merking == 1 && hentInntekterLengerFrem != null) {
throw FeilkodeException(Feilkode.HAR_ALLERDE_UNNTAK_OM_INNTEKTER_1_MND_FREM)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -59,9 +59,16 @@ class SaksbehandlerRefusjonController(
@PostMapping("/{id}/merk-for-unntak-om-inntekter-to-mnd-frem")
fun merkForUnntakOmInntekterToMånederFrem(@PathVariable id: String, @RequestBody request: MerkForUnntakOmInntekterToMånederFremRequest) {
val saksbehandler = innloggetBrukerService.hentInnloggetSaksbehandler()
saksbehandler.merkForUnntakOmInntekterToMånederFrem(id, request.merking)
saksbehandler.merkForUnntakOmInntekterFremITid(id, request.merking)
}


@PostMapping("/{id}/merk-for-unntak-om-inntekter-frem-i-tid")
fun merkForUnntakOmInntekterFremITid(@PathVariable id: String, @RequestBody request: MerkForUnntakOmInntekterToMånederFremRequest) {
val saksbehandler = innloggetBrukerService.hentInnloggetSaksbehandler()
saksbehandler.merkForUnntakOmInntekterFremITid(id, request.merking)
}


@PostMapping("reberegn-dry/{id}")
fun reberegnDryRun(@PathVariable id: String, @RequestBody request: ReberegnRequest): Beregning {
val saksbehandler = innloggetBrukerService.hentInnloggetSaksbehandler()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@ import org.springframework.web.filter.OncePerRequestFilter
@Order(Ordered.HIGHEST_PRECEDENCE)
class CookieTokenFilter: OncePerRequestFilter() {
override fun doFilterInternal(request: HttpServletRequest, response: HttpServletResponse, filterChain: FilterChain) {

val aadToken = request.cookies?.find { it.name == "aad-token" }?.value
val tokenxToken = request.cookies?.find { it.name == "tokenx-token" }?.value

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -371,7 +371,7 @@ class RefusjonServiceTest(
}

Now.fixedDate(LocalDate.now().plusDays(1))
refusjon.merkForUnntakOmInntekterToMånederFrem(2, innloggetSaksbehandler)
refusjon.merkForUnntakOmInntekterFremITid(2, innloggetSaksbehandler)
refusjonService.gjørInntektsoppslag(refusjon, innloggetArbeidsgiver)
verify {
inntektskomponentService.hentInntekter(tilskuddMelding.deltakerFnr, tilskuddMelding.bedriftNr, tilskuddMelding.tilskuddFom, tilskuddMelding.tilskuddTom.plusMonths(2))
Expand Down Expand Up @@ -462,7 +462,7 @@ class RefusjonServiceTest(
inntektskomponentService.hentInntekter(tilskuddMelding.deltakerFnr, tilskuddMelding.bedriftNr, tilskuddMelding.tilskuddFom, tilskuddMelding.tilskuddTom.plusMonths(0))
}
Now.fixedDate(LocalDate.now().plusDays(1))
refusjon.merkForUnntakOmInntekterToMånederFrem(2, innloggetSaksbehandler)
refusjon.merkForUnntakOmInntekterFremITid(2, innloggetSaksbehandler)
refusjonService.gjørInntektsoppslag(refusjon, innloggetArbeidsgiver)
verify {
inntektskomponentService.hentInntekter(tilskuddMelding.deltakerFnr, tilskuddMelding.bedriftNr, tilskuddMelding.tilskuddFom, tilskuddMelding.tilskuddTom.plusMonths(2))
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -408,7 +408,7 @@ internal class RefusjonTest {
@Test
internal fun `merk refusjon for henting av inntekt frem skal ikke gå når den allerde er merket med unntak om 2 måneder av saksbehandler`() {
val refusjon = enRefusjon().medInntektsgrunnlag().medBedriftKontonummer()
refusjon.merkForUnntakOmInntekterToMånederFrem(2, innloggetVeileder)
refusjon.merkForUnntakOmInntekterFremITid(2, innloggetVeileder)
assertFeilkode(Feilkode.HAR_ALLERDE_UNNTAK_OM_INNTEKTER_2_MND_FREM) {
refusjon.merkForHentInntekterFrem(
true,
Expand Down

0 comments on commit 953d1df

Please sign in to comment.