bvandeusen/minstrel
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 38 Wiki Activity

fix(web): update PlayerBar test for /now-playing cover link
test-web / test (push) Successful in 30s
Details

Browse Source 
CI on a2466b7d caught it: the cover/title area now links to
/now-playing per the NowPlaying landing, but PlayerBar.test.ts
still asserted /open album/i + href /albums/xyz. Rewritten test
asserts /open now playing/i + href /now-playing, with a comment
explaining the Spotify/YouTube Music pattern (album reachable via
the album-name link inside NowPlaying).
This commit is contained in:
Bryan Van Deusen
2026-06-01 11:32:46 -04:00
parent a2466b7d9a
commit bbc1036f77
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
web/src/lib/components/PlayerBar.test.ts
+5 -3
View File
@@ -92,14 +92,16 @@ beforeEach(() => {
afterEach(() => vi.clearAllMocks());
describe('PlayerBar', () => {
test('renders title, linked artist, linked cover', () => {
test('renders title, artist, and cover linked to /now-playing', () => {
render(PlayerBar);
expect(compact().getByText('So What')).toBeInTheDocument();
// The artist label sits under the title in compact (no link wrapper);
// look for it as plain text.
expect(compact().getByText('Miles Davis')).toBeInTheDocument();
const cover = compact().getByRole('link', { name: /open album/i });
expect(cover).toHaveAttribute('href', '/albums/xyz');
// Cover + title area links to /now-playing (Spotify/YouTube Music
// pattern). Album reachable via the album-name link inside NowPlaying.
const cover = compact().getByRole('link', { name: /open now playing/i });
expect(cover).toHaveAttribute('href', '/now-playing');
});
test('play button click calls togglePlay', async () => {