Comfy-Org/ComfyUI_frontend
1
0
Fork 0
mirror of https://github.com/Comfy-Org/ComfyUI_frontend.git synced 2026-02-05 15:40:10 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix newUserService test after console.log removal

Browse Source 
Update test to verify new user status without relying on console.log
This commit is contained in:
bymyself
2025-07-04 16:22:11 -07:00
parent df15572562
commit 548d38fe4b
1 changed files with 11 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
tests-ui/tests/services/newUserService.test.ts
View File

@@ -323,9 +323,7 @@ describe('newUserService', () => {
expect(mockSettingStore.set).toHaveBeenCalledTimes(1)
})
it('should log new user status', async () => {
const consoleSpy = vi.spyOn(console, 'log').mockImplementation(() => {})
it('should correctly determine new user status', async () => {
mockSettingStore.settingValues = {}
mockSettingStore.get.mockImplementation((key: string) => {
if (key === 'Comfy.TutorialCompleted') return undefined
@@ -333,12 +331,19 @@ describe('newUserService', () => {
})
mockLocalStorage.getItem.mockReturnValue(null)
// Before initialization, isNewUser should return null
expect(service.isNewUser()).toBeNull()
await service.initializeIfNewUser(mockSettingStore)
expect(consoleSpy).toHaveBeenCalledWith(
'New user status determined: true'
// After initialization, isNewUser should return true for a new user
expect(service.isNewUser()).toBe(true)
// Should set the installed version for new users
expect(mockSettingStore.set).toHaveBeenCalledWith(
'Comfy.InstalledVersion',
expect.any(String)
)
consoleSpy.mockRestore()
})
})