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

Økt aldersgrense (PEK-868) Magicbox -> Main #1714

Open
wants to merge 78 commits into
base: main
Choose a base branch
from
Open

Økt aldersgrense (PEK-868) Magicbox -> Main #1714

wants to merge 78 commits into from

Conversation

mlp73
Copy link
Collaborator

@mlp73 mlp73 commented Jan 20, 2025

Tar i bruk dynamiske verdier for normert pensjonsalder og nedre aldersgrense som er levert gjennom /person endepunktet.
Beskrivelse for hvert punkt på Jira saken PEK-868

Lucas8448 and others added 30 commits December 17, 2024 09:40
…ector med direkte tilgang til ubetinget uttaksalder
@mlp73 mlp73 marked this pull request as ready for review February 12, 2025 07:55
@@ -81,7 +85,7 @@ describe('TidligstMuligUttaksalder', () => {
).toBeInTheDocument()
expect(
screen.getByText(
'Aldersgrensene vil øke gradvis fra 1964-kullet med én til to måneder per årskull,',
'Aldersgrensene vil øke gradvis fra 1964-kullet med én til to måneder per årskull.',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mulig at dette ikke har en påvirkning på sanity, men vi burde være oppmerksomme når vi henter tekst og ikke id pga. sanity

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ref dialog på Slack skriver jeg svaret her og :)
Disse tekstene kommer ikke enda fra Sanity, derfor tester vi selve teksten. Vi kommer til å skrive assertions som heller går på testid og ikke på selve innholdet som det er f.eks gjort i Utenlandsopphold.test.tsx.

Dette tar vi fortløpende når vi flytter tekstene ifbm https://jira.adeo.no/browse/PEK-972

screen.getByText('Du kan tidligst ta ut alderspensjon ved 67 år.', {
exact: false,
})
screen.getByText(
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Forrige kommentar gjelder her også


const stepArrays = isLoependeVedtakEndring(
getLoependeVedtakResponse.data as LoependeVedtak
)
? stegvisningOrderEndring
: stegvisningOrder

// Bruker med uføretrygd, som svarer ja til afp, og som er under 62 kan se steget
// Bruker med uføretrygd, som svarer ja til afp, og som er under nedre aldersgrense kan se steget
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Er det riktig å skrive "nedre aldersgrense"?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ja. de begrepene som ble definert er:

  • nedre aldersgrense
  • normert pensjonsalder

export const isAlderOverAnnenAlder = (alder: Alder, alder2: Alder) => {
if (alder.aar > alder2.aar) {
export const isAlderOverAnnenAlder = (
storsteAlder: Alder,
Copy link
Collaborator Author

@mlp73 mlp73 Feb 13, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

stoersteAlder istedenfor storsteAlder


debugger

if (ufoeregrad < 100) {
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Logikken som ble refaktoret bør gjenspeile forrige logikk:

  • Har brukeren 100% uføretrygd skal det returneres normert pensjonsalder (ser OK ut)
  • Har brukeren 0 eller gradert uføretrygd skal det returneres brukerens alder + 1md. Hvis brukeren er yngre enn nedre aldersgrense er det nedre aldersgrense som gjelder. (ser feil ut)

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Logikken ble forsåvidt flyttet over til getBrukerensAlderPlus1Maaned i alder.ts ser jeg. Så det der er en dobbelimplementasjon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants