From d6724d78970680ae8c3d300b2a9edf3c00b1f6bb Mon Sep 17 00:00:00 2001 From: Torben Sorensen Date: Mon, 15 Dec 2025 21:12:27 -0800 Subject: [PATCH] Refactor: Enhance findUserByRefreshToken tests to include rowCount for improved query validation --- src/services/db/user.db.test.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/services/db/user.db.test.ts b/src/services/db/user.db.test.ts index 5407a97d..eaffc9c0 100644 --- a/src/services/db/user.db.test.ts +++ b/src/services/db/user.db.test.ts @@ -373,13 +373,13 @@ describe('User DB Service', () => { describe('findUserByRefreshToken', () => { it('should query for a user by their refresh token', async () => { - mockPoolInstance.query.mockResolvedValue({ rows: [{ user_id: '123' }] }); + mockPoolInstance.query.mockResolvedValue({ rows: [{ user_id: '123' }], rowCount: 1 }); await userRepo.findUserByRefreshToken('a-token', mockLogger); expect(mockPoolInstance.query).toHaveBeenCalledWith(expect.stringContaining('WHERE refresh_token = $1'), ['a-token']); }); it('should throw NotFoundError if token is not found', async () => { - mockPoolInstance.query.mockResolvedValue({ rows: [] }); + mockPoolInstance.query.mockResolvedValue({ rows: [], rowCount: 0 }); await expect(userRepo.findUserByRefreshToken('a-token', mockLogger)).rejects.toThrow(NotFoundError); await expect(userRepo.findUserByRefreshToken('a-token', mockLogger)).rejects.toThrow('User not found for the given refresh token.'); });