-
Notifications
You must be signed in to change notification settings - Fork 0
65 test sett opp ende til ende tester med playwright #81
Merged
birkjr
merged 28 commits into
dev
from
65-test-sett-opp-ende-til-ende-tester-med-playwright
Oct 30, 2026
Merged
65 test sett opp ende til ende tester med playwright #81
birkjr
merged 28 commits into
dev
from
65-test-sett-opp-ende-til-ende-tester-med-playwright
Oct 30, 2026
+3,932
−144
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
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>
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>
birkjr
approved these changes
Oct 30, 2026
There was a problem hiding this 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!
Sign in
to join this conversation on GitHub.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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:e2ekjø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 mednpm run dev.e2e/global-setup.mjslegger inn 60 faste testannonser (ide2e-…), oge2e/global-teardown.mjssletter dem og alt testene har lagt inn. Stopper hvis databasen ikke slutter på_test.BACKEND_URL(uendret uten variabelen).Verdt å se på:
@playwright/tester ny dev-avhengighet, og Chromium må lastes ned mednpx playwright install chromium.Testing
npm run typechecknpm run lintnpm run format:checknpm test(298)npm run test:e2e(4, stabile over tre kjøringer)npm run buildTestene feiler hvis «Tilbake til katalogen» ikke tar med søket, eller hvis sorteringen ikke sendes til serveren (prøvd).
Sjekkliste
docs/er oppdatert hvis det trengs