youngkingdom/dify
1
0
Fork 0
mirror of https://github.com/langgenius/dify.git synced 2026-05-04 01:18:05 +08:00
Code Issues Packages Projects Releases Wiki Activity

Revert "refactor!: replace Zustand global store with TanStack Query for systemFeatures"

Browse Source 
This reverts commit 806ece9a67.
This commit is contained in:
yyh
2026-02-01 19:06:45 +08:00
parent 806ece9a67
commit e2913d9ee1
102 changed files with 411 additions and 296 deletions
Download Patch File Download Diff File Expand all files Collapse all files

44
web/utils/setup-status.spec.ts
View File

@ -1,14 +1,14 @@
import type { SetupStatusResponse } from '@/models/common'
import { consoleClient } from '@/service/client'
import { fetchSetupStatus } from '@/service/common'
import { fetchSetupStatusWithCache } from './setup-status'
vi.mock('@/service/client', () => ({
consoleClient: {
setupStatus: vi.fn(),
},
vi.mock('@/service/common', () => ({
fetchSetupStatus: vi.fn(),
}))
const mockSetupStatus = vi.mocked(consoleClient.setupStatus)
const mockFetchSetupStatus = vi.mocked(fetchSetupStatus)
describe('setup-status utilities', () => {
beforeEach(() => {
@ -24,7 +24,7 @@ describe('setup-status utilities', () => {
const result = await fetchSetupStatusWithCache()
expect(result).toEqual({ step: 'finished' })
expect(mockSetupStatus).not.toHaveBeenCalled()
expect(mockFetchSetupStatus).not.toHaveBeenCalled()
})
it('should not modify localStorage when returning cached value', async () => {
@ -39,22 +39,22 @@ describe('setup-status utilities', () => {
describe('when cache does not exist', () => {
it('should call API and cache finished status', async () => {
const apiResponse: SetupStatusResponse = { step: 'finished' }
mockSetupStatus.mockResolvedValue(apiResponse)
mockFetchSetupStatus.mockResolvedValue(apiResponse)
const result = await fetchSetupStatusWithCache()
expect(mockSetupStatus).toHaveBeenCalledTimes(1)
expect(mockFetchSetupStatus).toHaveBeenCalledTimes(1)
expect(result).toEqual(apiResponse)
expect(localStorage.getItem('setup_status')).toBe('finished')
})
it('should call API and remove cache when not finished', async () => {
const apiResponse: SetupStatusResponse = { step: 'not_started' }
mockSetupStatus.mockResolvedValue(apiResponse)
mockFetchSetupStatus.mockResolvedValue(apiResponse)
const result = await fetchSetupStatusWithCache()
expect(mockSetupStatus).toHaveBeenCalledTimes(1)
expect(mockFetchSetupStatus).toHaveBeenCalledTimes(1)
expect(result).toEqual(apiResponse)
expect(localStorage.getItem('setup_status')).toBeNull()
})
@ -62,7 +62,7 @@ describe('setup-status utilities', () => {
it('should clear stale cache when API returns not_started', async () => {
localStorage.setItem('setup_status', 'some_invalid_value')
const apiResponse: SetupStatusResponse = { step: 'not_started' }
mockSetupStatus.mockResolvedValue(apiResponse)
mockFetchSetupStatus.mockResolvedValue(apiResponse)
const result = await fetchSetupStatusWithCache()
@ -75,44 +75,44 @@ describe('setup-status utilities', () => {
it('should call API when cache value is empty string', async () => {
localStorage.setItem('setup_status', '')
const apiResponse: SetupStatusResponse = { step: 'finished' }
mockSetupStatus.mockResolvedValue(apiResponse)
mockFetchSetupStatus.mockResolvedValue(apiResponse)
const result = await fetchSetupStatusWithCache()
expect(mockSetupStatus).toHaveBeenCalledTimes(1)
expect(mockFetchSetupStatus).toHaveBeenCalledTimes(1)
expect(result).toEqual(apiResponse)
})
it('should call API when cache value is not "finished"', async () => {
localStorage.setItem('setup_status', 'not_started')
const apiResponse: SetupStatusResponse = { step: 'finished' }
mockSetupStatus.mockResolvedValue(apiResponse)
mockFetchSetupStatus.mockResolvedValue(apiResponse)
const result = await fetchSetupStatusWithCache()
expect(mockSetupStatus).toHaveBeenCalledTimes(1)
expect(mockFetchSetupStatus).toHaveBeenCalledTimes(1)
expect(result).toEqual(apiResponse)
})
it('should call API when localStorage key does not exist', async () => {
const apiResponse: SetupStatusResponse = { step: 'finished' }
mockSetupStatus.mockResolvedValue(apiResponse)
mockFetchSetupStatus.mockResolvedValue(apiResponse)
const result = await fetchSetupStatusWithCache()
expect(mockSetupStatus).toHaveBeenCalledTimes(1)
expect(mockFetchSetupStatus).toHaveBeenCalledTimes(1)
expect(result).toEqual(apiResponse)
})
})
describe('API response handling', () => {
it('should preserve setup_at from API response', async () => {
const setupDate = '2024-01-01T00:00:00.000Z'
const setupDate = new Date('2024-01-01')
const apiResponse: SetupStatusResponse = {
step: 'finished',
setup_at: setupDate,
}
mockSetupStatus.mockResolvedValue(apiResponse)
mockFetchSetupStatus.mockResolvedValue(apiResponse)
const result = await fetchSetupStatusWithCache()
@ -122,13 +122,13 @@ describe('setup-status utilities', () => {
it('should propagate API errors', async () => {
const apiError = new Error('Network error')
mockSetupStatus.mockRejectedValue(apiError)
mockFetchSetupStatus.mockRejectedValue(apiError)
await expect(fetchSetupStatusWithCache()).rejects.toThrow('Network error')
})
it('should not update cache when API call fails', async () => {
mockSetupStatus.mockRejectedValue(new Error('API error'))
mockFetchSetupStatus.mockRejectedValue(new Error('API error'))
await expect(fetchSetupStatusWithCache()).rejects.toThrow()

4
web/utils/setup-status.ts
View File

@ -1,5 +1,5 @@
import type { SetupStatusResponse } from '@/models/common'
import { consoleClient } from '@/service/client'
import { fetchSetupStatus } from '@/service/common'
const SETUP_STATUS_KEY = 'setup_status'
@ -10,7 +10,7 @@ export const fetchSetupStatusWithCache = async (): Promise<SetupStatusResponse>
if (isSetupStatusCached())
return { step: 'finished' }
const status = await consoleClient.setupStatus()
const status = await fetchSetupStatus()
if (status.step === 'finished')
localStorage.setItem(SETUP_STATUS_KEY, 'finished')