remote/navidrome
1
0
Fork 0
mirror of https://github.com/navidrome/navidrome.git synced 2026-10-28 14:17:10 +01:00
Code Issues Projects Releases Packages Wiki Activity

feat(library): add refresh functionality on LibrarySelector close

Browse source 
Signed-off-by: Deluan <deluan@navidrome.org>
This commit is contained in:
Deluan 2025-06-26 11:52:57 -04:00
commit 5ecad64a3f
2 changed files with 8 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

4
ui/src/common/LibrarySelector.jsx
View file

@ -1,6 +1,6 @@
import React, { useState, useEffect } from 'react'
import { useDispatch, useSelector } from 'react-redux'
import { useDataProvider, useTranslate } from 'react-admin'
import { useDataProvider, useTranslate, useRefresh } from 'react-admin'
import {
Box,
Chip,
@ -71,6 +71,7 @@ const LibrarySelector = () => {
const dispatch = useDispatch()
const dataProvider = useDataProvider()
const translate = useTranslate()
const refresh = useRefresh()
const [anchorEl, setAnchorEl] = useState(null)
const [open, setOpen] = useState(false)
@ -113,6 +114,7 @@ const LibrarySelector = () => {
const handleClose = () => {
setOpen(false)
refresh()
}
const handleLibraryToggle = (libraryId) => {

5
ui/src/common/LibrarySelector.test.jsx
View file

@ -10,6 +10,7 @@ const mockDataProvider = {
getOne: vi.fn(),
}
const mockIdentity = { username: 'testuser' }
const mockRefresh = vi.fn()
const mockTranslate = vi.fn((key, options = {}) => {
const translations = {
'menu.librarySelector.allLibraries': `All Libraries (${options.count || 0})`,
@ -29,6 +30,7 @@ vi.mock('react-admin', () => ({
useDataProvider: () => mockDataProvider,
useGetIdentity: () => ({ identity: mockIdentity }),
useTranslate: () => mockTranslate,
useRefresh: () => mockRefresh,
}))
// Mock Material-UI components
@ -458,6 +460,9 @@ describe('LibrarySelector', () => {
await waitFor(() => {
expect(screen.queryByTestId('popper')).not.toBeInTheDocument()
})
// Should trigger refresh when closing
expect(mockRefresh).toHaveBeenCalledTimes(1)
})
it('should load user libraries on mount', async () => {