Skip to content

Conversation

mathealk
Copy link
Member

@mathealk mathealk commented Oct 26, 2026 •

Hva er endret?

Lagt til automatiserte tester for frontend og GraphQL-API-et. Frontendtestene dekker komponenter, hooks og Apollo-cache. API-testene kjører resolverne mot en innebygd testdatabase med faste testdata.

GraphQL-oppsettet er delt mellom app.ts og server.ts, slik at resolverne kan testes uten å starte HTTP-serveren eller bruke produksjonsdatabasen. Testskripter, avhengigheter, dokumentasjon og en GitHub Actions-workflow for pull requests er også lagt til.

Ingen endringer i produksjonsdatabasens skjema eller appens tiltenkte funksjonalitet.

Type endring

  • Frontend
  • Backend / GraphQL
  • Database
  • Konfig / verktøy
  • Dokumentasjon
  • Bug / fix
  • Tester

Test endringene

  • Kjør npm run lint for å sjekke kodekvalitet med ESLint.
  • Kjør npm run format:check for å sjekke formatering med Prettier.
  • Kjør npm run build for å kontrollere TypeScript og produksjonsbygget.
  • Kjør npm test for å kjøre automatiserte tester med Vitest.
  • Start appen med npm run dev og test endringene manuelt i appen.

Manuell testing

Ikke relevant – endringene gjelder testoppsett og automatiserte tester, uten endringer i appens funksjonalitet.

Sjekkliste

  • Endringene er testet, eller eventuelle begrensninger er beskrevet.
  • Dokumentasjonen er oppdatert dersom det er nødvendig.
  • Ingen utilsiktede endringer er inkludert.

Closes #59

@mathealk mathealk requested a review from a team as a code owner October 26, 2026 14:06
@mathealk mathealk linked an issue Oct 26, 2026 that may be closed by this pull request
Copy link
Member

@mariehd mariehd left a comment

Choose a reason for hiding this comment

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

Ser bra ut! Testene, lint, build og formatkontrollen passerer hos meg. Men er merge conflict i backend/src/server.ts!

Med ren arbeidsmappe kan du kjøre:

git status
git fetch origin
git switch 59-testoppsett
git pull --ff-only
git merge origin/main

(Pass på at statusfunksjonaliteten fra #15 og x-user-id-context beholdes når serveroppsettet deles mellom app.ts og server.ts. Kjør kontrollene igjen etter konfliktløsningen, og push til samme branch.)

guroklo
guroklo previously approved these changes Oct 30, 2026
Copy link
Member

@guroklo guroklo left a comment

Choose a reason for hiding this comment

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

Nå ser det bra ut!!

mariehd
mariehd previously approved these changes Oct 30, 2026
Copy link
Member

@mariehd mariehd left a comment

Choose a reason for hiding this comment

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

Bra jobba!

@mathealk mathealk dismissed stale reviews from mariehd and guroklo via 6991785 October 30, 2026 14:38
Copy link
Member

@guroklo guroklo left a comment

Choose a reason for hiding this comment

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

Ser bra ut!! God helg 🩷

@guroklo guroklo merged commit 4f60e97 into main Oct 30, 2026
1 check passed
Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[test] Testoppsett og tester av kjernefunksjonalitet
3 participants