Skip to content

Conversation

kfmelen
Copy link
Member

@kfmelen kfmelen commented Nov 10, 2026 •

Summary

Expand test coverage for core user journeys and GraphQL behavior.

  • Test login, account creation, validation and failed authentication.
  • Test review creation and deletion, including count updates and logged-out behavior.
  • Test filtering, sorting, pagination and loading additional results without duplicates.
  • Test adding and removing favorites across pages.
  • Add keyboard and mobile viewport checks for the filter menu.
  • Use controlled fixtures and a disposable database for integration tests.

Validation

  • 17 component tests passed.
  • 16 API and end-to-end tests passed.

Closes #79

@kfmelen kfmelen added the tests Anything to do with tests label Nov 10, 2026
@kfmelen kfmelen added this to the Delivery four milestone Nov 10, 2026
@kfmelen kfmelen self-assigned this Nov 10, 2026
@kfmelen kfmelen requested a review from a team as a code owner November 10, 2026 11:41
@kfmelen kfmelen linked an issue Nov 10, 2026 that may be closed by this pull request
@kfmelen kfmelen requested a review from martiasu November 10, 2026 11:46
Copy link
Member

@martiasu martiasu left a comment

Choose a reason for hiding this comment

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

Lgtm!

@kfmelen kfmelen merged commit 97e05db into main Nov 10, 2026
@kfmelen kfmelen deleted the 79-make-more-tests branch November 10, 2026 11:59
Sign in to join this conversation on GitHub.
Labels
tests Anything to do with tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make more tests
2 participants