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.

Sign up for GitHub

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

Oppgrader nav-pakker #1095

Merged
merged 9 commits into from
Mar 1, 2024
Merged

Oppgrader nav-pakker #1095

Show file tree
Hide file tree
Changes from 6 commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
594a2ca
Påpek at node 20 ikke funker
kristeine Feb 29, 2024
43c0915
Ikke anta at lint-staged er globalt installert
kristeine Feb 29, 2024
506bbb9
Oppgrader familie-felles-frontend-pakker
kristeine Feb 29, 2024
98421a6
Oppgrader nav-ds-pakker
kristeine Feb 29, 2024
08b82ce
Endre syntaks for test assertion for bedre feilmelding
kristeine Feb 29, 2024
b7b20b5
Håndter at Datepicker i v6 rendrer kalender til dom selv om den er sk…
kristeine Feb 29, 2024
8e6c7b9
Fjern test-id som jeg ikke endte opp med å bruke
kristeine Feb 29, 2024
0acd2b2
Merge branch 'main' into chore/oppgrader-nav-pakker
kristeine Feb 29, 2024
428978b
Sett date-fns til siste release
kristeine Mar 1, 2024
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
2 changes: 1 addition & 1 deletion .husky/pre-commit
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
#!/usr/bin/env sh
. "$(dirname -- "$0")/_/husky.sh"

lint-staged
npx lint-staged
yarn run typecheck
2 changes: 1 addition & 1 deletion README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
Frontend - søknad for barnetrygd.

## Avhengigheter
1. Node versjon >=18.13.0
1. Node versjon >=18.13.0, <20


## Logg inn på https://npm.pkg.github.com
Expand Down
19 changes: 9 additions & 10 deletions package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -92,15 +92,14 @@
"dependencies": {
"@formatjs/intl-datetimeformat": "^6.12.0",
"@formatjs/intl-numberformat": "^8.9.0",
"@navikt/aksel-icons": "^5.11.4",
"@navikt/ds-css": "^5.11.4",
"@navikt/ds-react": "^5.11.4",
"@navikt/familie-form-elements": "^13.0.0",
"@navikt/familie-http": "^6.0.2",
"@navikt/familie-logging": "^6.0.0",
"@navikt/familie-skjema": "^7.0.8",
"@navikt/familie-sprakvelger": "^10.0.1",
"@navikt/familie-typer": "^8.0.0",
"@navikt/aksel-icons": "^6.1.0",
"@navikt/ds-css": "^6.1.0",
"@navikt/ds-react": "^6.1.0",
"@navikt/familie-form-elements": "^15.0.0",
"@navikt/familie-logging": "^7.0.0",
"@navikt/familie-skjema": "^8.0.8",
"@navikt/familie-sprakvelger": "^12.0.0",
"@navikt/familie-typer": "^8.0.2",
"@navikt/fnrvalidator": "^1.3.3",
"@navikt/nav-dekoratoren-moduler": "^1.6.4",
"@sentry/react": "^7.101.1",
Expand All @@ -111,7 +110,7 @@
"constate": "^3.3.2",
"cookie-parser": "^1.4.6",
"cors": "^2.8.5",
"date-fns": "^2.30.0",
"date-fns": "^3.0.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

Det finnes en nyere versjon (3.3.1) enn dette også, skulle vi bare ha oppgradert dit med en gang?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ja, ser at det er den som ble installert også. Kan endre til det!

"dotenv": "^16.3.1",
"express": "^4.18.2",
"http-proxy-middleware": "^2.0.6",
Expand Down
20 changes: 14 additions & 6 deletions src/frontend/components/Felleskomponenter/Datovelger/Datovelger.test.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -59,9 +59,14 @@ describe('Datovelger', () => {
visFeilmeldinger: true,
});

const { getAllByRole, container } = render(
const { getAllByRole } = render(
<TestProvidere>
<Datovelger felt={current.fraOgMed} skjema={skjemaMock} label={'test-fra-og-med'} />
<Datovelger
felt={current.fraOgMed}
skjema={skjemaMock}
label={'test-fra-og-med'}
data-testid="test-fra-og-med"
/>
<Datovelger
felt={current.tilOgMed}
tilhørendeFraOgMedFelt={current.fraOgMed}
Expand All @@ -71,11 +76,14 @@ describe('Datovelger', () => {
</TestProvidere>
);

const tilOgMedÅpneknapp = getAllByRole('button')[1];
const [, tilOgMedÅpneknapp] = getAllByRole('button');
Copy link
Contributor

@idaame idaame Mar 1, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

Av nysgjerrighet: Hvordan funker denne måten å skrive det på/hva betyr det? [, tilOgMedÅpneknapp]

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Dette er array destructuring, der jeg ser bort fra første (nullte) element. Det er mest digg når du skal lage konstanter ut fra flere elementer i et array, f.eks. const [idag, imorgen] = listeMedDager, men jeg synes det også er fint som her når vi vil hoppe over første element. Liker det litt bedre enn å lese indekslesing etter et funksjonskall 😇

Men det krever at det er lesbart for andre, da. Så om det ikke gir mening trenger vi ikke å gjøre det sånn

Se MDN: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Destructuring_assignment

act(() => tilOgMedÅpneknapp.click());
const forrigeDag = container.querySelector('[aria-label="torsdag 9"]');

expect(forrigeDag?.getAttribute('disabled')).not.toBeNull();
const [tilOgMedKalender] = getAllByRole('dialog');
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Tidligere gikk vi ut fra at container.querySelector('[aria-label="torsdag 9"]'); kun ville finne datoen i kalender for til-og-med, men nå som Datepicker rendrer kalenderen til DOM uansett om den vises for brukeren vil det første treffet være fra fra-og-med-kalenderen, der datoen er tillatt å velge.

Skriver om til at vi henter ut til-og-med-kalenderen ved å se etter dialog-elementer som er tilgjengelige for brukeren (i dette caset er fra-og-med-kalenderen aria-hidden og treffer derfor ikke på getAllByRole('dialog'))


const forrigeDag = tilOgMedKalender.querySelector('[aria-label="torsdag 9"]');

expect(forrigeDag).toBeDisabled();
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Dette gir en bedre feilmelding:

expect(element).toBeDisabled()
    Received element is not disabled:
      <button aria-label="torsdag 9" class="rdp-button_reset rdp-button rdp-day" name="day" role="gridcell" tabindex="-1" type="button" />

Heller enn før:

expect(received).not.toBeNull()
    Received: null

Copy link
Contributor

Choose a reason for hiding this comment

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

🙌

});

it('kan avgrenses frem i tid', () => {
Expand Down Expand Up @@ -114,6 +122,6 @@ describe('Datovelger', () => {

act(() => datovelgerÅpneKnapp.click());
const nesteDag = container.querySelector('[aria-label="lørdag 11"]');
expect(nesteDag?.getAttribute('disabled')).not.toBeNull();
expect(nesteDag).toBeDisabled();
});
});
Loading
Loading