test fix

Author: gallayl

frontend/src/services/users-service.spec.ts

@@ -247,13 +247,15 @@ describe('UsersService', () => {
       })
     })
 
-    it('should update cache with new user data after update', async () => {
+    it('should invalidate cache after update', async () => {
       const originalUser = createMockUser('[email protected]', ['admin'])
       const updatedUser = createMockUser('[email protected]', ['admin', 'viewer'])
+      const refetchedUser = createMockUser('[email protected]', ['admin', 'viewer'])
       const mockCall = vi
         .fn()
         .mockResolvedValueOnce({ result: originalUser })
         .mockResolvedValueOnce({ result: updatedUser })
+        .mockResolvedValueOnce({ result: refetchedUser })
       const injector = createTestInjector(mockCall)
 
       await usingAsync(injector, async (i) => {
@@ -269,8 +271,8 @@ describe('UsersService', () => {
 
         const result = await service.getUser('[email protected]')
 
-        expect(mockCall).toHaveBeenCalledTimes(2)
-        expect(result).toEqual(updatedUser)
+        expect(mockCall).toHaveBeenCalledTimes(3)
+        expect(result).toEqual(refetchedUser)
       })
     })
 

frontend/src/services/users-service.ts

@@ -59,7 +59,7 @@ export class UsersService {
       url: { id: username },
       body,
     })
-    this.userCache.setObsolete(username)
+    this.userCache.remove(username)
     this.userQueryCache.flushAll()
     return result
   }