|
| 1 | +/** |
| 2 | + * @vitest-environment node |
| 3 | + */ |
| 4 | +import { hybridAuthMockFns } from '@sim/testing' |
| 5 | +import { NextRequest } from 'next/server' |
| 6 | +import { beforeEach, describe, expect, it, vi } from 'vitest' |
| 7 | + |
| 8 | +const { |
| 9 | + mockCheckAccess, |
| 10 | + mockDeleteTable, |
| 11 | + mockGetTableById, |
| 12 | + mockMoveTableToFolder, |
| 13 | + mockRenameTable, |
| 14 | + mockUpdateTableLocks, |
| 15 | + mockFindActiveFolder, |
| 16 | + mockGetLimits, |
| 17 | +} = vi.hoisted(() => ({ |
| 18 | + mockCheckAccess: vi.fn(), |
| 19 | + mockDeleteTable: vi.fn(), |
| 20 | + mockGetTableById: vi.fn(), |
| 21 | + mockMoveTableToFolder: vi.fn(), |
| 22 | + mockRenameTable: vi.fn(), |
| 23 | + mockUpdateTableLocks: vi.fn(), |
| 24 | + mockFindActiveFolder: vi.fn(), |
| 25 | + mockGetLimits: vi.fn(), |
| 26 | +})) |
| 27 | + |
| 28 | +vi.mock('@/lib/table', () => ({ |
| 29 | + deleteTable: mockDeleteTable, |
| 30 | + getTableById: mockGetTableById, |
| 31 | + moveTableToFolder: mockMoveTableToFolder, |
| 32 | + renameTable: mockRenameTable, |
| 33 | + updateTableLocks: mockUpdateTableLocks, |
| 34 | + TableConflictError: class extends Error {}, |
| 35 | +})) |
| 36 | +vi.mock('@/lib/table/billing', () => ({ getWorkspaceTableLimits: mockGetLimits })) |
| 37 | +vi.mock('@/lib/folders/queries', () => ({ findActiveFolder: mockFindActiveFolder })) |
| 38 | +vi.mock('@/lib/core/config/feature-flags', () => ({ isFeatureEnabled: vi.fn() })) |
| 39 | +vi.mock('@/lib/posthog/server', () => ({ captureServerEvent: vi.fn() })) |
| 40 | +vi.mock('@/lib/workspaces/permissions/utils', () => ({ |
| 41 | + getWorkspaceWithOwner: vi.fn(), |
| 42 | + getUserEntityPermissions: vi.fn(), |
| 43 | +})) |
| 44 | +vi.mock('@/app/api/table/utils', () => ({ |
| 45 | + accessError: () => new Response('denied', { status: 403 }), |
| 46 | + checkAccess: mockCheckAccess, |
| 47 | + normalizeColumn: (column: unknown) => column, |
| 48 | + tableLockErrorResponse: () => null, |
| 49 | +})) |
| 50 | + |
| 51 | +import { PATCH } from '@/app/api/table/[tableId]/route' |
| 52 | + |
| 53 | +const TABLE = { |
| 54 | + id: 'tbl_1', |
| 55 | + name: 'people', |
| 56 | + workspaceId: 'workspace-1', |
| 57 | + folderId: null as string | null, |
| 58 | + schema: { columns: [] }, |
| 59 | + locks: { |
| 60 | + schemaLocked: false, |
| 61 | + insertLocked: false, |
| 62 | + updateLocked: false, |
| 63 | + deleteLocked: false, |
| 64 | + }, |
| 65 | +} |
| 66 | + |
| 67 | +function patchRequest(body: unknown): NextRequest { |
| 68 | + return new NextRequest('http://localhost:3000/api/table/tbl_1', { |
| 69 | + method: 'PATCH', |
| 70 | + headers: { 'content-type': 'application/json' }, |
| 71 | + body: JSON.stringify(body), |
| 72 | + }) |
| 73 | +} |
| 74 | + |
| 75 | +const routeContext = { params: Promise.resolve({ tableId: 'tbl_1' }) } |
| 76 | + |
| 77 | +describe('PATCH /api/table/[tableId] folder moves', () => { |
| 78 | + beforeEach(() => { |
| 79 | + vi.clearAllMocks() |
| 80 | + hybridAuthMockFns.mockCheckSessionOrInternalAuth.mockResolvedValue({ |
| 81 | + success: true, |
| 82 | + userId: 'user-1', |
| 83 | + authType: 'session', |
| 84 | + }) |
| 85 | + mockCheckAccess.mockResolvedValue({ ok: true, table: TABLE }) |
| 86 | + mockGetTableById.mockResolvedValue({ ...TABLE, folderId: 'folder-1' }) |
| 87 | + mockFindActiveFolder.mockResolvedValue({ id: 'folder-1' }) |
| 88 | + }) |
| 89 | + |
| 90 | + it('moves the table into a folder in the same workspace and tree', async () => { |
| 91 | + const response = await PATCH( |
| 92 | + patchRequest({ workspaceId: 'workspace-1', folderId: 'folder-1' }), |
| 93 | + routeContext |
| 94 | + ) |
| 95 | + |
| 96 | + expect(response.status).toBe(200) |
| 97 | + expect(mockFindActiveFolder).toHaveBeenCalledWith('folder-1', 'workspace-1', 'table') |
| 98 | + expect(mockMoveTableToFolder).toHaveBeenCalledWith( |
| 99 | + 'tbl_1', |
| 100 | + 'folder-1', |
| 101 | + expect.any(String), |
| 102 | + 'user-1' |
| 103 | + ) |
| 104 | + }) |
| 105 | + |
| 106 | + it('moves the table to the workspace root on an explicit null, with no folder lookup', async () => { |
| 107 | + mockGetTableById.mockResolvedValue({ ...TABLE, folderId: null }) |
| 108 | + |
| 109 | + const response = await PATCH( |
| 110 | + patchRequest({ workspaceId: 'workspace-1', folderId: null }), |
| 111 | + routeContext |
| 112 | + ) |
| 113 | + |
| 114 | + expect(response.status).toBe(200) |
| 115 | + expect(mockFindActiveFolder).not.toHaveBeenCalled() |
| 116 | + expect(mockMoveTableToFolder).toHaveBeenCalledWith('tbl_1', null, expect.any(String), 'user-1') |
| 117 | + }) |
| 118 | + |
| 119 | + it('leaves placement untouched when folderId is omitted', async () => { |
| 120 | + await PATCH(patchRequest({ workspaceId: 'workspace-1', name: 'renamed' }), routeContext) |
| 121 | + |
| 122 | + expect(mockRenameTable).toHaveBeenCalled() |
| 123 | + expect(mockMoveTableToFolder).not.toHaveBeenCalled() |
| 124 | + }) |
| 125 | + |
| 126 | + it('rejects a folder from another workspace or resource tree without writing', async () => { |
| 127 | + mockFindActiveFolder.mockResolvedValue(null) |
| 128 | + |
| 129 | + const response = await PATCH( |
| 130 | + patchRequest({ workspaceId: 'workspace-1', folderId: 'kb-folder' }), |
| 131 | + routeContext |
| 132 | + ) |
| 133 | + |
| 134 | + expect(response.status).toBe(404) |
| 135 | + expect(mockMoveTableToFolder).not.toHaveBeenCalled() |
| 136 | + }) |
| 137 | + |
| 138 | + it('rejects a body with no name, folder, or lock changes', async () => { |
| 139 | + const response = await PATCH(patchRequest({ workspaceId: 'workspace-1' }), routeContext) |
| 140 | + |
| 141 | + expect(response.status).toBe(400) |
| 142 | + expect(mockMoveTableToFolder).not.toHaveBeenCalled() |
| 143 | + expect(mockRenameTable).not.toHaveBeenCalled() |
| 144 | + }) |
| 145 | +}) |
0 commit comments