Skip to content

Commit 8126e9a

Browse files
committed
test(provenance): align Copilot query mock with snapshot reader
1 parent 58b781d commit 8126e9a

1 file changed

Lines changed: 17 additions & 7 deletions

File tree

‎apps/sim/lib/copilot/tools/server/table/user-table.test.ts‎

Lines changed: 17 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,8 @@ const {
3434
mockExecuteCopilotFileUseCase,
3535
mockExecuteCopilotWorkflowUseCase,
3636
mockLoadWorkspaceFileContext,
37-
mockLoadTableRowSecretProvenance,
37+
mockCreateTableRowProvenanceReader,
38+
mockExportTableRowProvenance,
3839
mockResolveWorkflowContext,
3940
fakeEnrichment,
4041
} = vi.hoisted(() => ({
@@ -65,7 +66,8 @@ const {
6566
mockExecuteCopilotFileUseCase: vi.fn(),
6667
mockExecuteCopilotWorkflowUseCase: vi.fn(),
6768
mockLoadWorkspaceFileContext: vi.fn(),
68-
mockLoadTableRowSecretProvenance: vi.fn(),
69+
mockCreateTableRowProvenanceReader: vi.fn(),
70+
mockExportTableRowProvenance: vi.fn(),
6971
mockResolveWorkflowContext: vi.fn(),
7072
fakeEnrichment: {
7173
id: 'work-email',
@@ -229,7 +231,12 @@ vi.mock('@/lib/table/rows/secret-provenance', () => ({
229231
Object.keys(data).map((columnId) => [columnId, { version: 1, complete: true, entries: [] }])
230232
),
231233
}),
232-
loadTableRowSecretProvenance: mockLoadTableRowSecretProvenance,
234+
TableRowProvenanceReader: class {
235+
constructor(scope: unknown) {
236+
mockCreateTableRowProvenanceReader(scope)
237+
}
238+
exportProvenance = mockExportTableRowProvenance
239+
},
233240
}))
234241

235242
vi.mock('@/lib/table/jobs/service', () => ({
@@ -271,7 +278,7 @@ import { ResolvedSecretTraceRegistry } from '@/executor/utils/resolved-secret-tr
271278

272279
beforeEach(() => {
273280
mockLoadWorkspaceFileContext.mockResolvedValue({ workspaceId: 'workspace-1' })
274-
mockLoadTableRowSecretProvenance.mockResolvedValue({
281+
mockExportTableRowProvenance.mockReturnValue({
275282
version: 1,
276283
complete: true,
277284
entries: [],
@@ -1223,9 +1230,12 @@ describe('userTableServerTool.query_rows', () => {
12231230
)
12241231

12251232
expect(result.success).toBe(true)
1226-
expect(mockLoadTableRowSecretProvenance).toHaveBeenCalledWith(
1227-
expect.arrayContaining([expect.objectContaining({ id: 'row_1' })]),
1228-
{ userId: 'user-1', workspaceId: 'workspace-1' }
1233+
expect(mockCreateTableRowProvenanceReader).toHaveBeenCalledWith({
1234+
userId: 'user-1',
1235+
workspaceId: 'workspace-1',
1236+
})
1237+
expect(mockQueryRows.mock.calls[0][3]).toEqual(
1238+
expect.objectContaining({ exportProvenance: mockExportTableRowProvenance })
12291239
)
12301240
expect(importProvenance).toHaveBeenCalledWith(
12311241
expect.objectContaining({

0 commit comments

Comments
 (0)