Skip to content

Test/tours view - #46

Merged
Yasminjama merged 3 commits into
mainfrom
test/ToursView
Oct 7, 2026
Merged

Yasminjama merged 3 commits into
mainfrom
test/ToursView

Conversation

@ellagabari

Copy link
Copy Markdown
Collaborator

Sammanfattning

Den här PR:n lägger till två tester för ToursView.vue och uppdaterar README så att den stämmer med projektets nuvarande struktur och kommandon.

Ändringar

  • Ny testfil client/src/views/ToursView.test.ts med två tester:
    • Visar turens uppgifter (titel med länk till /tours/:id, författare, guide, längd i km och antal bilder).
    • Visar felmeddelandet "Kunde inte ladda turer." när fetch misslyckas.
  • README uppdaterad:
    • shared/ tillagd i strukturen.
    • typecheck tillagd bland kommandona.
    • Node-kravet tillagt

Testning

Testerna kördes lokalt med npm run test --workspace=client. Alla 9 tester i klienten gick igenom, inklusive de två nya. README-ändringarna är dokumentation och behövde ingen testning.

  • Testat lokalt
  • Körda tester: npm test (om projektet har tester)
  • Kontrollerat relevanta manuella flöden

Checklista

  • Koden följer projektets befintliga struktur och stil
  • Onödiga debug-utskrifter och kommentarer är borttagna
  • Dokumentation har uppdaterats om det behövs
  • PR:n är redo för review

Review

  • Minst en teammedlem har reviewat ändringarna
  • Review-kommentarer är hanterade
  • PR:n är godkänd innan merge

@ellagabari ellagabari reopened this Oct 6, 2026
@Yasminjama
Yasminjama merged commit 0ba115c into main Oct 7, 2026
4 checks passed
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.

2 participants