[PR #2069] [MERGED] refactor: use functional approach for FE tests #2043

Closed
opened 2026-02-26 03:33:07 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/koel/koel/pull/2069
Author: @phanan
Created: 8/25/2025
Status: Merged
Merged: 8/25/2025
Merged by: @phanan

Base: masterHead: test-functional


📝 Commits (1)

  • 8f565fc refactor: use functional approach for FE tests

📊 Changes

243 files changed (+7895 additions, -7996 deletions)

View changed files

📝 resources/assets/js/__tests__/TestHarness.ts (+53 -27)
📝 resources/assets/js/components/album/AlbumCard.spec.ts (+82 -83)
📝 resources/assets/js/components/album/AlbumContextMenu.spec.ts (+80 -80)
📝 resources/assets/js/components/album/AlbumInfo.spec.ts (+27 -28)
📝 resources/assets/js/components/album/AlbumListSorter.spec.ts (+23 -23)
📝 resources/assets/js/components/album/AlbumTrackList.spec.ts (+19 -20)
📝 resources/assets/js/components/album/AlbumTrackListItem.spec.ts (+26 -27)
📝 resources/assets/js/components/album/EditAlbumForm.spec.ts (+27 -28)
📝 resources/assets/js/components/album/__snapshots__/AlbumCard.spec.ts.snap (+2 -2)
📝 resources/assets/js/components/album/__snapshots__/AlbumContextMenu.spec.ts.snap (+1 -1)
📝 resources/assets/js/components/album/__snapshots__/AlbumTrackListItem.spec.ts.snap (+1 -1)
📝 resources/assets/js/components/artist/ArtistCard.spec.ts (+55 -56)
📝 resources/assets/js/components/artist/ArtistContextMenu.spec.ts (+70 -70)
📝 resources/assets/js/components/artist/ArtistInfo.spec.ts (+24 -25)
📝 resources/assets/js/components/artist/ArtistListSorter.spec.ts (+19 -19)
📝 resources/assets/js/components/artist/__snapshots__/ArtistCard.spec.ts.snap (+2 -2)
📝 resources/assets/js/components/artist/__snapshots__/ArtistContextMenu.spec.ts.snap (+1 -1)
📝 resources/assets/js/components/auth/ForgotPasswordForm.spec.ts (+18 -18)
📝 resources/assets/js/components/auth/LoginForm.spec.ts (+50 -50)
📝 resources/assets/js/components/auth/ResetPasswordForm.spec.ts (+19 -19)

...and 80 more files

📄 Description

No description provided


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/koel/koel/pull/2069 **Author:** [@phanan](https://github.com/phanan) **Created:** 8/25/2025 **Status:** ✅ Merged **Merged:** 8/25/2025 **Merged by:** [@phanan](https://github.com/phanan) **Base:** `master` ← **Head:** `test-functional` --- ### 📝 Commits (1) - [`8f565fc`](https://github.com/koel/koel/commit/8f565fc0f3b34d526e08dd17b6bfd61cb52ef0f0) refactor: use functional approach for FE tests ### 📊 Changes **243 files changed** (+7895 additions, -7996 deletions) <details> <summary>View changed files</summary> 📝 `resources/assets/js/__tests__/TestHarness.ts` (+53 -27) 📝 `resources/assets/js/components/album/AlbumCard.spec.ts` (+82 -83) 📝 `resources/assets/js/components/album/AlbumContextMenu.spec.ts` (+80 -80) 📝 `resources/assets/js/components/album/AlbumInfo.spec.ts` (+27 -28) 📝 `resources/assets/js/components/album/AlbumListSorter.spec.ts` (+23 -23) 📝 `resources/assets/js/components/album/AlbumTrackList.spec.ts` (+19 -20) 📝 `resources/assets/js/components/album/AlbumTrackListItem.spec.ts` (+26 -27) 📝 `resources/assets/js/components/album/EditAlbumForm.spec.ts` (+27 -28) 📝 `resources/assets/js/components/album/__snapshots__/AlbumCard.spec.ts.snap` (+2 -2) 📝 `resources/assets/js/components/album/__snapshots__/AlbumContextMenu.spec.ts.snap` (+1 -1) 📝 `resources/assets/js/components/album/__snapshots__/AlbumTrackListItem.spec.ts.snap` (+1 -1) 📝 `resources/assets/js/components/artist/ArtistCard.spec.ts` (+55 -56) 📝 `resources/assets/js/components/artist/ArtistContextMenu.spec.ts` (+70 -70) 📝 `resources/assets/js/components/artist/ArtistInfo.spec.ts` (+24 -25) 📝 `resources/assets/js/components/artist/ArtistListSorter.spec.ts` (+19 -19) 📝 `resources/assets/js/components/artist/__snapshots__/ArtistCard.spec.ts.snap` (+2 -2) 📝 `resources/assets/js/components/artist/__snapshots__/ArtistContextMenu.spec.ts.snap` (+1 -1) 📝 `resources/assets/js/components/auth/ForgotPasswordForm.spec.ts` (+18 -18) 📝 `resources/assets/js/components/auth/LoginForm.spec.ts` (+50 -50) 📝 `resources/assets/js/components/auth/ResetPasswordForm.spec.ts` (+19 -19) _...and 80 more files_ </details> ### 📄 Description _No description provided_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 03:33:07 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/koel-koel#2043
No description provided.