d25d482dc2
Publish CLI Package / publish-npm (push) Waiting to run
Publish Python SDK / publish-pypi (push) Waiting to run
Publish TypeScript SDK / publish-npm (push) Waiting to run
CI / Migrate Dev DB (push) Has been skipped
CI / Detect Version (push) Has been cancelled
CI / Migrate DB (push) Has been cancelled
CI / Build Dev ECR (./docker/app.Dockerfile, ECR_APP) (push) Has been cancelled
CI / Build Dev ECR (./docker/db.Dockerfile, ECR_MIGRATIONS) (push) Has been cancelled
CI / Build Dev ECR (./docker/pii.Dockerfile, ECR_PII) (push) Has been cancelled
CI / Build Dev ECR (./docker/realtime.Dockerfile, ECR_REALTIME) (push) Has been cancelled
CI / Deploy Trigger.dev (Dev) (push) Has been cancelled
CI / Build AMD64 (./docker/app.Dockerfile, ECR_APP, ghcr.io/simstudioai/simstudio) (push) Has been cancelled
CI / Build AMD64 (./docker/db.Dockerfile, ECR_MIGRATIONS, ghcr.io/simstudioai/migrations) (push) Has been cancelled
CI / Build AMD64 (./docker/pii.Dockerfile, ECR_PII, ghcr.io/simstudioai/pii) (push) Has been cancelled
CI / Build AMD64 (./docker/realtime.Dockerfile, ECR_REALTIME, ghcr.io/simstudioai/realtime) (push) Has been cancelled
CI / Build ARM64 (GHCR Only) (./docker/app.Dockerfile, ghcr.io/simstudioai/simstudio) (push) Has been cancelled
CI / Build ARM64 (GHCR Only) (./docker/db.Dockerfile, ghcr.io/simstudioai/migrations) (push) Has been cancelled
CI / Build ARM64 (GHCR Only) (./docker/pii.Dockerfile, ghcr.io/simstudioai/pii) (push) Has been cancelled
CI / Build ARM64 (GHCR Only) (./docker/realtime.Dockerfile, ghcr.io/simstudioai/realtime) (push) Has been cancelled
CI / Create GHCR Manifests (ghcr.io/simstudioai/migrations) (push) Has been cancelled
CI / Create GHCR Manifests (ghcr.io/simstudioai/pii) (push) Has been cancelled
CI / Create GHCR Manifests (ghcr.io/simstudioai/realtime) (push) Has been cancelled
CI / Create GHCR Manifests (ghcr.io/simstudioai/simstudio) (push) Has been cancelled
CI / Check Docs Changes (push) Has been cancelled
CI / Process Docs (push) Has been cancelled
CI / Create GitHub Release (push) Has been cancelled
CI / Test and Build (push) Has been cancelled
471 lines
15 KiB
TypeScript
471 lines
15 KiB
TypeScript
/**
|
|
* @vitest-environment node
|
|
*/
|
|
import { beforeEach, describe, expect, it, vi } from 'vitest'
|
|
|
|
const { queryClient, cacheStore } = vi.hoisted(() => {
|
|
const cache = new Map<string, unknown>()
|
|
return {
|
|
cacheStore: cache,
|
|
queryClient: {
|
|
cancelQueries: vi.fn().mockResolvedValue(undefined),
|
|
invalidateQueries: vi.fn().mockResolvedValue(undefined),
|
|
getQueryData: vi.fn((key: readonly unknown[]) => cache.get(JSON.stringify(key))),
|
|
setQueryData: vi.fn((key: readonly unknown[], updater: unknown) => {
|
|
const k = JSON.stringify(key)
|
|
const prev = cache.get(k)
|
|
const next =
|
|
typeof updater === 'function' ? (updater as (p: unknown) => unknown)(prev) : updater
|
|
cache.set(k, next)
|
|
return next
|
|
}),
|
|
getQueriesData: vi.fn((opts: { queryKey: readonly unknown[] }) => {
|
|
const prefix = JSON.stringify(opts.queryKey).slice(0, -1)
|
|
return [...cache.entries()]
|
|
.filter(([k]) => k.startsWith(prefix))
|
|
.map(([k, v]) => [JSON.parse(k), v])
|
|
}),
|
|
removeQueries: vi.fn(),
|
|
},
|
|
}
|
|
})
|
|
|
|
vi.mock('@tanstack/react-query', () => ({
|
|
keepPreviousData: {},
|
|
infiniteQueryOptions: (opts: unknown) => opts,
|
|
useQuery: vi.fn(),
|
|
useInfiniteQuery: vi.fn(),
|
|
useQueryClient: vi.fn(() => queryClient),
|
|
useMutation: vi.fn((options) => options),
|
|
}))
|
|
|
|
vi.mock('@/lib/api/client/request', () => ({
|
|
requestJson: vi.fn(),
|
|
}))
|
|
|
|
vi.mock('@/lib/api/client/errors', () => ({
|
|
isValidationError: vi.fn(() => false),
|
|
}))
|
|
|
|
vi.mock('@/lib/api/contracts/tables', () => ({
|
|
addTableColumnContract: {},
|
|
addWorkflowGroupContract: {},
|
|
batchCreateTableRowsContract: {},
|
|
batchUpdateTableRowsContract: {},
|
|
cancelTableRunsContract: {},
|
|
createTableContract: {},
|
|
createTableRowContract: {},
|
|
deleteTableColumnContract: {},
|
|
deleteTableContract: {},
|
|
deleteTableRowContract: {},
|
|
deleteTableRowsContract: {},
|
|
deleteWorkflowGroupContract: {},
|
|
getTableContract: {},
|
|
importCsvContract: {},
|
|
listTableRowsContract: {},
|
|
listTablesContract: {},
|
|
renameTableContract: {},
|
|
restoreTableContract: {},
|
|
runWorkflowGroupContract: {},
|
|
updateTableColumnContract: {},
|
|
updateTableMetadataContract: {},
|
|
updateTableRowContract: {},
|
|
updateWorkflowGroupContract: {},
|
|
uploadCsvContract: {},
|
|
}))
|
|
|
|
vi.mock('@/app/workspace/providers/socket-provider', () => ({
|
|
useSocket: vi.fn(() => ({ socket: null })),
|
|
}))
|
|
|
|
vi.mock('@sim/emcn', () => ({
|
|
toast: { error: vi.fn(), success: vi.fn() },
|
|
}))
|
|
|
|
import {
|
|
tableRowsInfiniteOptions,
|
|
tableRowsParamsKey,
|
|
useDeleteColumn,
|
|
useRestoreTable,
|
|
useUpdateColumn,
|
|
} from '@/hooks/queries/tables'
|
|
import { tableKeys } from '@/hooks/queries/utils/table-keys'
|
|
|
|
const TABLE_ID = 'tbl-1'
|
|
const WORKSPACE_ID = 'ws-1'
|
|
|
|
function setCache(key: readonly unknown[], value: unknown) {
|
|
cacheStore.set(JSON.stringify(key), value)
|
|
}
|
|
|
|
function getCache<T>(key: readonly unknown[]): T | undefined {
|
|
return cacheStore.get(JSON.stringify(key)) as T | undefined
|
|
}
|
|
|
|
beforeEach(() => {
|
|
cacheStore.clear()
|
|
vi.clearAllMocks()
|
|
})
|
|
|
|
describe('useDeleteColumn optimistic update', () => {
|
|
it('removes column from schema cache, strips its width, and clears it from row data', async () => {
|
|
setCache(tableKeys.detail(TABLE_ID), {
|
|
id: TABLE_ID,
|
|
schema: {
|
|
columns: [
|
|
{ name: 'name', type: 'string' },
|
|
{ name: 'age', type: 'number' },
|
|
],
|
|
},
|
|
metadata: {
|
|
columnWidths: { name: 200, age: 100 },
|
|
},
|
|
})
|
|
setCache(tableKeys.rowsRoot(TABLE_ID), {
|
|
rows: [
|
|
{ id: 'r1', data: { name: 'a', age: 1 } },
|
|
{ id: 'r2', data: { name: 'b', age: 2 } },
|
|
],
|
|
totalCount: 2,
|
|
})
|
|
|
|
const hook = useDeleteColumn({ workspaceId: WORKSPACE_ID, tableId: TABLE_ID })
|
|
const ctx = await hook.onMutate?.('age')
|
|
|
|
const detail = getCache<{
|
|
schema: { columns: Array<{ name: string }> }
|
|
metadata: { columnWidths: Record<string, number> }
|
|
}>(tableKeys.detail(TABLE_ID))
|
|
expect(detail?.schema.columns.map((c) => c.name)).toEqual(['name'])
|
|
expect(detail?.metadata.columnWidths).toEqual({ name: 200 })
|
|
|
|
const rows = getCache<{ rows: Array<{ data: Record<string, unknown> }> }>(
|
|
tableKeys.rowsRoot(TABLE_ID)
|
|
)
|
|
expect(rows?.rows.every((r) => !('age' in r.data))).toBe(true)
|
|
expect(rows?.rows[0]?.data).toEqual({ name: 'a' })
|
|
|
|
expect(ctx?.previousDetail).toBeDefined()
|
|
expect(ctx?.rowSnapshots?.length).toBeGreaterThan(0)
|
|
})
|
|
|
|
it('rolls back schema and rows on error using snapshots', async () => {
|
|
const originalDetail = {
|
|
id: TABLE_ID,
|
|
schema: { columns: [{ name: 'name' }, { name: 'age' }] },
|
|
metadata: { columnWidths: { name: 200, age: 100 } },
|
|
}
|
|
const originalRows = {
|
|
rows: [{ id: 'r1', data: { name: 'a', age: 1 } }],
|
|
totalCount: 1,
|
|
}
|
|
setCache(tableKeys.detail(TABLE_ID), originalDetail)
|
|
setCache(tableKeys.rowsRoot(TABLE_ID), originalRows)
|
|
|
|
const hook = useDeleteColumn({ workspaceId: WORKSPACE_ID, tableId: TABLE_ID })
|
|
const ctx = await hook.onMutate?.('age')
|
|
|
|
expect(getCache(tableKeys.detail(TABLE_ID))).not.toEqual(originalDetail)
|
|
|
|
hook.onError?.(new Error('boom'), 'age', ctx)
|
|
|
|
expect(getCache(tableKeys.detail(TABLE_ID))).toEqual(originalDetail)
|
|
expect(getCache(tableKeys.rowsRoot(TABLE_ID))).toEqual(originalRows)
|
|
})
|
|
|
|
it('invalidates schema, rows, and lists in onSettled', () => {
|
|
const hook = useDeleteColumn({ workspaceId: WORKSPACE_ID, tableId: TABLE_ID })
|
|
hook.onSettled?.(undefined, null, 'age', undefined)
|
|
|
|
const calls = queryClient.invalidateQueries.mock.calls.map((c) => c[0]?.queryKey)
|
|
expect(calls).toEqual(
|
|
expect.arrayContaining([
|
|
tableKeys.detail(TABLE_ID),
|
|
tableKeys.rowsRoot(TABLE_ID),
|
|
tableKeys.lists(),
|
|
])
|
|
)
|
|
})
|
|
})
|
|
|
|
describe('useUpdateColumn optimistic update', () => {
|
|
it('writes the column update to the schema cache and rolls back on error', async () => {
|
|
const original = {
|
|
id: TABLE_ID,
|
|
schema: {
|
|
columns: [
|
|
{ name: 'name', type: 'string' },
|
|
{ name: 'age', type: 'string' },
|
|
],
|
|
},
|
|
}
|
|
setCache(tableKeys.detail(TABLE_ID), original)
|
|
|
|
const hook = useUpdateColumn({ workspaceId: WORKSPACE_ID, tableId: TABLE_ID })
|
|
const ctx = await hook.onMutate?.({ columnName: 'age', updates: { type: 'number' } })
|
|
|
|
const after = getCache<{ schema: { columns: Array<{ name: string; type: string }> } }>(
|
|
tableKeys.detail(TABLE_ID)
|
|
)
|
|
expect(after?.schema.columns.find((c) => c.name === 'age')?.type).toBe('number')
|
|
|
|
hook.onError?.(new Error('boom'), { columnName: 'age', updates: { type: 'number' } }, ctx)
|
|
|
|
expect(getCache(tableKeys.detail(TABLE_ID))).toEqual(original)
|
|
})
|
|
|
|
it('renames metadata-only: patches the column name + stamps id, leaves row data untouched', async () => {
|
|
setCache(tableKeys.detail(TABLE_ID), {
|
|
id: TABLE_ID,
|
|
schema: { columns: [{ name: 'age', type: 'number' }] },
|
|
})
|
|
setCache(tableKeys.rowsRoot(TABLE_ID), {
|
|
rows: [
|
|
{ id: 'r1', data: { age: 30 } },
|
|
{ id: 'r2', data: { age: 40 } },
|
|
],
|
|
totalCount: 2,
|
|
})
|
|
|
|
const hook = useUpdateColumn({ workspaceId: WORKSPACE_ID, tableId: TABLE_ID })
|
|
await hook.onMutate?.({ columnName: 'age', updates: { name: 'years' } })
|
|
|
|
// Row data is id-keyed; a rename never moves it. The stored key (`age`)
|
|
// becomes the column's stamped id, so cells stay reachable via getColumnId.
|
|
const rows = getCache<{ rows: Array<{ data: Record<string, unknown> }> }>(
|
|
tableKeys.rowsRoot(TABLE_ID)
|
|
)
|
|
expect(rows?.rows[0]?.data).toEqual({ age: 30 })
|
|
expect(rows?.rows[1]?.data).toEqual({ age: 40 })
|
|
|
|
const detail = getCache<{ schema: { columns: Array<{ id?: string; name: string }> } }>(
|
|
tableKeys.detail(TABLE_ID)
|
|
)
|
|
expect(detail?.schema.columns[0]).toMatchObject({ id: 'age', name: 'years' })
|
|
})
|
|
})
|
|
|
|
describe('useRestoreTable cache invalidation', () => {
|
|
it('primes the table detail cache and clears stale rows for the restored table', () => {
|
|
const hook = useRestoreTable()
|
|
const table = {
|
|
id: TABLE_ID,
|
|
name: 'Restored table',
|
|
schema: { columns: [{ name: 'name', type: 'string' }] },
|
|
rowCount: 1,
|
|
maxRows: 100,
|
|
workspaceId: WORKSPACE_ID,
|
|
createdBy: 'user-1',
|
|
archivedAt: null,
|
|
createdAt: new Date(),
|
|
updatedAt: new Date(),
|
|
}
|
|
|
|
hook.onSuccess?.({ success: true, data: { table } }, TABLE_ID, undefined)
|
|
|
|
expect(getCache(tableKeys.detail(TABLE_ID))).toEqual(table)
|
|
expect(queryClient.removeQueries).toHaveBeenCalledWith({
|
|
queryKey: tableKeys.rowsRoot(TABLE_ID),
|
|
})
|
|
})
|
|
|
|
it('invalidates lists, table detail, and row data for the restored table', () => {
|
|
const hook = useRestoreTable()
|
|
hook.onSettled?.(undefined, null, TABLE_ID, undefined)
|
|
|
|
const calls = queryClient.invalidateQueries.mock.calls.map((c) => c[0]?.queryKey)
|
|
expect(calls).toEqual(
|
|
expect.arrayContaining([
|
|
tableKeys.lists(),
|
|
tableKeys.detail(TABLE_ID),
|
|
tableKeys.rowsRoot(TABLE_ID),
|
|
])
|
|
)
|
|
})
|
|
})
|
|
|
|
describe('useDeleteColumn case-insensitive row cleanup', () => {
|
|
it('strips the row data key even when stored casing differs from the requested name', async () => {
|
|
setCache(tableKeys.detail(TABLE_ID), {
|
|
id: TABLE_ID,
|
|
schema: { columns: [{ name: 'Age', type: 'number' }] },
|
|
})
|
|
setCache(tableKeys.rowsRoot(TABLE_ID), {
|
|
rows: [{ id: 'r1', data: { Age: 30, name: 'a' } }],
|
|
totalCount: 1,
|
|
})
|
|
|
|
const hook = useDeleteColumn({ workspaceId: WORKSPACE_ID, tableId: TABLE_ID })
|
|
await hook.onMutate?.('age')
|
|
|
|
const rows = getCache<{ rows: Array<{ data: Record<string, unknown> }> }>(
|
|
tableKeys.rowsRoot(TABLE_ID)
|
|
)
|
|
expect(rows?.rows[0]?.data).toEqual({ name: 'a' })
|
|
})
|
|
})
|
|
|
|
describe('tableRowsParamsKey', () => {
|
|
it('produces the same key for identical params', () => {
|
|
const k1 = tableRowsParamsKey({ pageSize: 1000, filter: null, sort: null })
|
|
const k2 = tableRowsParamsKey({ pageSize: 1000, filter: null, sort: null })
|
|
expect(k1).toBe(k2)
|
|
})
|
|
|
|
it('treats undefined filter and sort as null', () => {
|
|
const withUndefined = tableRowsParamsKey({ pageSize: 1000, filter: undefined, sort: undefined })
|
|
const withNull = tableRowsParamsKey({ pageSize: 1000, filter: null, sort: null })
|
|
expect(withUndefined).toBe(withNull)
|
|
})
|
|
|
|
it('produces different keys for different filters', () => {
|
|
const k1 = tableRowsParamsKey({ pageSize: 1000, filter: null, sort: null })
|
|
const k2 = tableRowsParamsKey({
|
|
pageSize: 1000,
|
|
filter: { column: 'name', operator: 'eq', value: 'Alice' } as never,
|
|
sort: null,
|
|
})
|
|
expect(k1).not.toBe(k2)
|
|
})
|
|
|
|
it('produces different keys for different page sizes', () => {
|
|
const k1 = tableRowsParamsKey({ pageSize: 1000, filter: null, sort: null })
|
|
const k2 = tableRowsParamsKey({ pageSize: 500, filter: null, sort: null })
|
|
expect(k1).not.toBe(k2)
|
|
})
|
|
|
|
it('produces different keys for different sorts', () => {
|
|
const k1 = tableRowsParamsKey({ pageSize: 1000, filter: null, sort: null })
|
|
const k2 = tableRowsParamsKey({
|
|
pageSize: 1000,
|
|
filter: null,
|
|
sort: { column: 'name', direction: 'asc' } as never,
|
|
})
|
|
expect(k1).not.toBe(k2)
|
|
})
|
|
})
|
|
|
|
describe('tableRowsInfiniteOptions', () => {
|
|
const PAGE_SIZE = 1000
|
|
|
|
interface PageFixture {
|
|
rows: Array<{ id: string; orderKey?: string }>
|
|
totalCount: number | null
|
|
}
|
|
|
|
function makeOpts(pageSize = PAGE_SIZE, sort: unknown = null) {
|
|
return tableRowsInfiniteOptions({
|
|
workspaceId: WORKSPACE_ID,
|
|
tableId: TABLE_ID,
|
|
pageSize,
|
|
filter: null,
|
|
sort: sort as never,
|
|
}) as {
|
|
queryKey: readonly unknown[]
|
|
getNextPageParam: (
|
|
lastPage: PageFixture,
|
|
allPages: PageFixture[],
|
|
lastPageParam: unknown
|
|
) => number | { orderKey: string; id: string } | undefined
|
|
}
|
|
}
|
|
|
|
function makePage(count: number, totalCount: number | null, startAt = 0, withOrderKey = false) {
|
|
return {
|
|
rows: Array.from({ length: count }, (_, i) => ({
|
|
id: `r${startAt + i}`,
|
|
...(withOrderKey ? { orderKey: `a${startAt + i}` } : {}),
|
|
})),
|
|
totalCount,
|
|
}
|
|
}
|
|
|
|
function next(
|
|
opts: ReturnType<typeof makeOpts>,
|
|
pages: PageFixture[],
|
|
lastPageParam: unknown = 0
|
|
) {
|
|
return opts.getNextPageParam(pages[pages.length - 1], pages, lastPageParam)
|
|
}
|
|
|
|
it('getNextPageParam terminates when the count is covered by a partial page', () => {
|
|
const opts = makeOpts()
|
|
expect(next(opts, [makePage(500, 500)])).toBeUndefined()
|
|
})
|
|
|
|
it('getNextPageParam terminates on an empty page', () => {
|
|
const opts = makeOpts()
|
|
expect(next(opts, [makePage(1000, null), makePage(0, null, 1000)])).toBeUndefined()
|
|
})
|
|
|
|
it('getNextPageParam continues past a short page when the count says more rows exist', () => {
|
|
// The regression the termination rule exists for: a page shorter than the
|
|
// requested size (e.g. a byte-cut page) must not be read as end-of-table.
|
|
const opts = makeOpts()
|
|
expect(next(opts, [makePage(36, 100)])).toBe(36)
|
|
})
|
|
|
|
it('getNextPageParam terminates a full page when the count is covered', () => {
|
|
const opts = makeOpts()
|
|
expect(next(opts, [makePage(PAGE_SIZE, PAGE_SIZE)])).toBeUndefined()
|
|
})
|
|
|
|
it('getNextPageParam returns next offset for a full page with an unknown count', () => {
|
|
const opts = makeOpts()
|
|
expect(next(opts, [makePage(PAGE_SIZE, null)])).toBe(PAGE_SIZE)
|
|
})
|
|
|
|
it('getNextPageParam advances correctly across three pages', () => {
|
|
const opts = makeOpts()
|
|
const p0 = makePage(PAGE_SIZE, 2200)
|
|
const p1 = makePage(PAGE_SIZE, null, 1000)
|
|
const p2 = makePage(200, null, 2000)
|
|
|
|
expect(next(opts, [p0])).toBe(1000)
|
|
expect(next(opts, [p0, p1], 1000)).toBe(2000)
|
|
expect(next(opts, [p0, p1, p2], 2000)).toBeUndefined()
|
|
})
|
|
|
|
it('getNextPageParam returns a keyset cursor when rows carry orderKey and there is no sort', () => {
|
|
const opts = makeOpts()
|
|
const pages = [makePage(PAGE_SIZE, 2000, 0, true)]
|
|
expect(next(opts, pages)).toEqual({
|
|
orderKey: `a${PAGE_SIZE - 1}`,
|
|
id: `r${PAGE_SIZE - 1}`,
|
|
})
|
|
})
|
|
|
|
it('getNextPageParam falls back to offset for sorted views even with orderKey present', () => {
|
|
const opts = makeOpts(PAGE_SIZE, { column: 'name', direction: 'asc' })
|
|
const p0 = makePage(PAGE_SIZE, 3000, 0, true)
|
|
const p1 = makePage(PAGE_SIZE, null, 1000, true)
|
|
expect(next(opts, [p0])).toBe(PAGE_SIZE)
|
|
expect(next(opts, [p0, p1], PAGE_SIZE)).toBe(PAGE_SIZE * 2)
|
|
})
|
|
|
|
it('queryKey includes the result of tableRowsParamsKey', () => {
|
|
const paramsKey = tableRowsParamsKey({ pageSize: PAGE_SIZE, filter: null, sort: null })
|
|
const opts = makeOpts(PAGE_SIZE)
|
|
// queryKey is a tuple; one element must be exactly the paramsKey string
|
|
expect(opts.queryKey).toContain(paramsKey)
|
|
})
|
|
|
|
it('queryKey differs when filter changes', () => {
|
|
const opts1 = tableRowsInfiniteOptions({
|
|
workspaceId: WORKSPACE_ID,
|
|
tableId: TABLE_ID,
|
|
pageSize: PAGE_SIZE,
|
|
filter: null,
|
|
sort: null,
|
|
}) as { queryKey: readonly unknown[] }
|
|
const opts2 = tableRowsInfiniteOptions({
|
|
workspaceId: WORKSPACE_ID,
|
|
tableId: TABLE_ID,
|
|
pageSize: PAGE_SIZE,
|
|
filter: { column: 'name', operator: 'eq', value: 'Alice' } as never,
|
|
sort: null,
|
|
}) as { queryKey: readonly unknown[] }
|
|
expect(JSON.stringify(opts1.queryKey)).not.toBe(JSON.stringify(opts2.queryKey))
|
|
})
|
|
})
|