Skip to content

Commit b9060e1

Browse files
committed
fix(files): mock servable downloads in hydration tests
1 parent a98f272 commit b9060e1

1 file changed

Lines changed: 25 additions & 18 deletions

File tree

apps/sim/lib/uploads/utils/user-file-base64.server.test.ts

Lines changed: 25 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -9,24 +9,26 @@ import {
99
} from '@/lib/uploads/utils/user-file-base64.server'
1010
import type { UserFile } from '@/executor/types'
1111

12-
const { mockDownloadFile, mockRedis, mockVerifyFileAccess } = vi.hoisted(() => {
13-
const mockRedis = {
14-
get: vi.fn(),
15-
set: vi.fn(),
16-
hget: vi.fn(),
17-
hset: vi.fn(),
18-
hgetall: vi.fn(),
19-
expire: vi.fn(),
20-
scan: vi.fn(),
21-
del: vi.fn(),
22-
eval: vi.fn(),
23-
}
24-
return {
25-
mockDownloadFile: vi.fn(),
26-
mockRedis,
27-
mockVerifyFileAccess: vi.fn(),
28-
}
29-
})
12+
const { mockDownloadFile, mockDownloadServableFileFromStorage, mockRedis, mockVerifyFileAccess } =
13+
vi.hoisted(() => {
14+
const mockRedis = {
15+
get: vi.fn(),
16+
set: vi.fn(),
17+
hget: vi.fn(),
18+
hset: vi.fn(),
19+
hgetall: vi.fn(),
20+
expire: vi.fn(),
21+
scan: vi.fn(),
22+
del: vi.fn(),
23+
eval: vi.fn(),
24+
}
25+
return {
26+
mockDownloadFile: vi.fn(),
27+
mockDownloadServableFileFromStorage: vi.fn(),
28+
mockRedis,
29+
mockVerifyFileAccess: vi.fn(),
30+
}
31+
})
3032

3133
const mockGetRedisClient = redisConfigMockFns.mockGetRedisClient
3234

@@ -44,6 +46,7 @@ vi.mock('@/lib/uploads/contexts/execution/execution-file-manager', () => ({
4446

4547
vi.mock('@/lib/uploads/utils/file-utils.server', () => ({
4648
downloadFileFromStorage: mockDownloadFile,
49+
downloadServableFileFromStorage: mockDownloadServableFileFromStorage,
4750
}))
4851

4952
vi.mock('@/app/api/files/authorization', () => ({
@@ -64,6 +67,10 @@ describe('hydrateUserFilesWithBase64', () => {
6467
mockRedis.del.mockResolvedValue(1)
6568
mockRedis.eval.mockResolvedValue([1, 'ok', 0, 0])
6669
mockVerifyFileAccess.mockResolvedValue(true)
70+
mockDownloadServableFileFromStorage.mockImplementation(async (file: unknown) => ({
71+
buffer: await mockDownloadFile(file),
72+
contentType: 'application/octet-stream',
73+
}))
6774
})
6875

6976
it('strips existing base64 when it exceeds maxBytes', async () => {

0 commit comments

Comments
 (0)