Skip to content

Test/tour details - #44

Merged
ellagabari merged 4 commits into
mainfrom
test/TourDetails
Oct 6, 2026
Merged

ellagabari merged 4 commits into
mainfrom
test/TourDetails

Conversation

@Yasminjama

Copy link
Copy Markdown
Collaborator

Sammanfattning

Lägger till komponenttester för TourDetail-vyn och fixar två buggar som testerna avslöjade: mätpunkter utan höjd räknades som 0 m (vilket gav fel antal höjdmeter) samt vyn kraschade när API:et svarade med fel (t.ex. 500). Testerna skrevs och committades före fixen och var röda mot den gamla vyn.

Ändringar

  • Ny TourDetail.test.ts med två tester (Vue Testing Library, fetch och vue-router mockade):
    • mätpunkt utan höjd hoppas över i höjdmeterberäkningen och visas som "–" i listan (regressionstest för skuld #N i docs/debt.md)
    • felmeddelande visas när servern svarar 500
  • TourDetail.vue portad till typescript (lang = ts) med typer från @utpost/shared
  • TourDetail.vue använder elevationGain från lib/tours i stället för egen uträkning
  • TourDetail.vue kontrollerar response.ok och visar felet med role="alert"
  • TourDetail.vue visar "–" i stället för "null" för saknad höjd eller puls
  • Ny typ Tour i shared/src/types.ts

Testning

Beskriv hur ändringarna testades, eller skriv varför testning inte behövdes.

  • 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 merged commit 6845ee1 into main Oct 6, 2026
2 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