Skip to content

Conversation

sebasti
Copy link
Member

@sebasti sebasti commented Oct 29, 2026

Issue

Closes #65

Bygger på #64, som bygger på #63 og #62. Skal merges etter dem. Til da viser PR-en også commitene derfra.

Endringer

  • npm run test:e2e kjører Playwright (playwright.config.ts). Den starter backenden mot testdatabasen på port 3101 (e2e/start-backend.mjs) og Vite på 5175, så den ikke kolliderer med npm run dev.
  • e2e/global-setup.mjs legger inn 60 faste testannonser (id e2e-…), og e2e/global-teardown.mjs sletter dem og alt testene har lagt inn. Stopper hvis databasen ikke slutter på _test.
  • Fire tester:
    • søk, kategori, sortering, blaing, åpne annonse og tilbake (lenke og nettleserens tilbakeknapp) med alt bevart
    • søk uten treff
    • ny annonse som finnes igjen i katalogen
    • vurdering som er lagret etter ny lasting av siden
  • Testene finner elementer med rolle og etikett, ikke CSS-klasser.
  • Vite-proxyen kan peke på en annen backend med BACKEND_URL (uendret uten variabelen).

Verdt å se på: @playwright/test er ny dev-avhengighet, og Chromium må lastes ned med npx playwright install chromium.

Testing

  • npm run typecheck
  • npm run lint
  • npm run format:check
  • npm test (298)
  • npm run test:e2e (4, stabile over tre kjøringer)
  • npm run build

Testene feiler hvis «Tilbake til katalogen» ikke tar med søket, eller hvis sorteringen ikke sendes til serveren (prøvd).

Sjekkliste

  • README eller docs/ er oppdatert hvis det trengs

Sebastian Ingebrigtsen and others added 28 commits October 29, 2026 22:01
Co-Authored-By: andretg <andretg@stud.ntnu.no>
Co-Authored-By: andretg <andretg@stud.ntnu.no>
Co-Authored-By: andretg <andretg@stud.ntnu.no>
Co-Authored-By: andretg <andretg@stud.ntnu.no>
Co-Authored-By: andretg <andretg@stud.ntnu.no>
…ser» forsvinner

Co-Authored-By: andretg <andretg@stud.ntnu.no>
Co-Authored-By: andretg <andretg@stud.ntnu.no>
Co-Authored-By: birkjr <birkjr@stud.ntnu.no>
Co-Authored-By: birkjr <birkjr@stud.ntnu.no>
…_URL

Co-Authored-By: tordtst <tordtst@stud.ntnu.no>
Co-Authored-By: tordtst <tordtst@stud.ntnu.no>
…alogen

Co-Authored-By: tordtst <tordtst@stud.ntnu.no>
Co-Authored-By: tordtst <tordtst@stud.ntnu.no>
Co-Authored-By: tordtst <tordtst@stud.ntnu.no>
@sebasti sebasti requested a review from a team as a code owner October 29, 2026 21:57
Copy link
Member

@birkjr birkjr left a comment

Choose a reason for hiding this comment

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

Veldig bra å få på plass ende-til-ende-tester! Liker spesielt at testene dekker hele flyten fra søk og filtrering til å opprette annonser, og at de bruker en egen testdatabase. Fint at oppsettet også er dokumentert, så det blir enkelt å kjøre testene videre. Ser bra ut!

@birkjr birkjr merged commit 904f1bd into dev Oct 30, 2026
@tordtst tordtst deleted the 65-test-sett-opp-ende-til-ende-tester-med-playwright branch October 30, 2026 13:26
Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants