Skip to content

Added some tests, and made some minor UI improvements. #45

Jump to bottom
Merged
merged 2 commits into from
Sep 16, 2026
Merged

Added some tests, and made some minor UI improvements. #45

merged 2 commits into from
Sep 16, 2026
+1,588 −59

Conversation

kfmelen
Copy link
Member

@kfmelen kfmelen commented Sep 16, 2026 •
edited
Loading

What changed:

  • Added Vitest and React Testing Library setup
  • Added 7 component tests across 5 test files
  • Added mocked tests for filters, favorites, and preview playback
  • Added a Searchbar snapshot test
  • Added a SpotifyAPI test
  • Adjusted the detail-page glow positioning and size
  • Made minor UI improvements
  • Updated the preview button to show the play icon and “Preview” text together

Closes #42

@kfmelen kfmelen self-assigned this Sep 16, 2026
@kfmelen kfmelen requested a review from a team as a code owner September 16, 2026 12:27
@kfmelen kfmelen linked an issue Sep 16, 2026 that may be closed by this pull request
Write tests for the project #42
Closed
martiasu
martiasu approved these changes Sep 16, 2026
View reviewed changes
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!

@martiasu martiasu merged commit 7f87a26 into main Sep 16, 2026
@kfmelen kfmelen deleted the 42-write-tests-for-the-project branch September 16, 2026 12:37
Sign in to join this conversation on GitHub.
Reviewers

@martiasu martiasu martiasu approved these changes

@tobiaheg tobiaheg Awaiting requested review from tobiaheg

Assignees

@kfmelen kfmelen

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Write tests for the project
2 participants
@kfmelen @martiasu