Files
pn-new-crm/tests/integration/document-path-style-download.test.ts
Matt 5422f11747 chore: prettier formatter drift across recent commits
Prettier reformatting on files touched in the wave 11.B sequence —
markdown italics _underscore-style_, single-line conditionals, minor
whitespace fixes. No semantic changes. .env.example reformatting left
unstaged (blocked by pre-commit hook).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
2026-05-11 10:57:37 +02:00

209 lines
7.1 KiB
TypeScript

/**
* Integration test: GET /api/v1/documents/[id]/download/[...slug]
*
* Verifies the slug truth-check and tenancy isolation on the path-style
* download route. The storage backend is swapped for a real FilesystemBackend
* rooted in a tempdir so the byte-streaming path is exercised end-to-end.
*/
import { mkdtemp, rm } from 'node:fs/promises';
import { tmpdir } from 'node:os';
import * as path from 'node:path';
import { afterEach, beforeAll, beforeEach, describe, expect, it, vi } from 'vitest';
import { db } from '@/lib/db';
import { documents, files } from '@/lib/db/schema/documents';
import { user } from '@/lib/db/schema/users';
import { createFolder } from '@/lib/services/document-folders.service';
import { makeMockCtx } from '../helpers/route-tester';
import { makeFullPermissions, makePort } from '../helpers/factories';
let testUserId: string;
beforeAll(async () => {
const [u] = await db.select({ id: user.id }).from(user).limit(1);
testUserId = u!.id;
});
describe('GET /api/v1/documents/[id]/download/[...slug]', () => {
let storageRoot: string;
let backend: import('@/lib/storage/filesystem').FilesystemBackend;
beforeEach(async () => {
storageRoot = await mkdtemp(path.join(tmpdir(), 'pn-doc-dl-'));
const { FilesystemBackend } = await import('@/lib/storage/filesystem');
backend = await FilesystemBackend.create({
root: storageRoot,
proxyHmacSecretEncrypted: null,
});
vi.doMock('@/lib/storage', async () => {
const real = await vi.importActual<typeof import('@/lib/storage')>('@/lib/storage');
return { ...real, getStorageBackend: vi.fn(async () => backend) };
});
});
afterEach(async () => {
vi.doUnmock('@/lib/storage');
await rm(storageRoot, { recursive: true, force: true });
});
async function setupDoc(opts: {
portId: string;
folderId: string | null;
filename: string;
body: Buffer;
storagePath: string;
}) {
await backend.put(opts.storagePath, opts.body, {
contentType: 'application/pdf',
sizeBytes: opts.body.length,
});
const [fileRow] = await db
.insert(files)
.values({
portId: opts.portId,
filename: opts.filename,
originalName: opts.filename,
mimeType: 'application/pdf',
sizeBytes: String(opts.body.length),
storagePath: opts.storagePath,
uploadedBy: testUserId,
})
.returning();
const [docRow] = await db
.insert(documents)
.values({
portId: opts.portId,
documentType: 'other',
title: opts.filename,
createdBy: testUserId,
folderId: opts.folderId,
fileId: fileRow!.id,
})
.returning();
return { fileRow: fileRow!, docRow: docRow! };
}
it('streams the file when the slug matches the current folder path + filename', async () => {
const port = await makePort();
const folder = await createFolder(port.id, testUserId, { name: 'Deals 2026', parentId: null });
const sub = await createFolder(port.id, testUserId, { name: 'Q1', parentId: folder.id });
const body = Buffer.from('hello world');
const { docRow } = await setupDoc({
portId: port.id,
folderId: sub.id,
filename: 'contract.pdf',
body,
storagePath: 'test/contract.pdf',
});
const { downloadHandler } =
await import('@/app/api/v1/documents/[id]/download/[...slug]/handlers');
const ctx = makeMockCtx({ portId: port.id, permissions: makeFullPermissions() });
const req = new Request('http://localhost/api/v1/documents/x/download/whatever') as never;
const res = await downloadHandler(req, ctx, {
id: docRow.id,
slug: ['Deals 2026', 'Q1', 'contract.pdf'],
});
expect(res.status).toBe(200);
const buf = Buffer.from(await res.arrayBuffer());
expect(buf.toString()).toBe('hello world');
expect(res.headers.get('content-type')).toBe('application/pdf');
});
it('404s when the folder-path segments do not match current state', async () => {
const port = await makePort();
const folder = await createFolder(port.id, testUserId, { name: 'Real Folder', parentId: null });
const { docRow } = await setupDoc({
portId: port.id,
folderId: folder.id,
filename: 'spec.pdf',
body: Buffer.from('x'),
storagePath: 'test/spec.pdf',
});
const { downloadHandler } =
await import('@/app/api/v1/documents/[id]/download/[...slug]/handlers');
const ctx = makeMockCtx({ portId: port.id, permissions: makeFullPermissions() });
const req = new Request('http://localhost/x') as never;
const res = await downloadHandler(req, ctx, {
id: docRow.id,
slug: ['Wrong Folder', 'spec.pdf'],
});
expect(res.status).toBe(404);
});
it('404s when the filename segment does not match current state', async () => {
const port = await makePort();
const folder = await createFolder(port.id, testUserId, { name: 'F', parentId: null });
const { docRow } = await setupDoc({
portId: port.id,
folderId: folder.id,
filename: 'real.pdf',
body: Buffer.from('x'),
storagePath: 'test/real.pdf',
});
const { downloadHandler } =
await import('@/app/api/v1/documents/[id]/download/[...slug]/handlers');
const ctx = makeMockCtx({ portId: port.id, permissions: makeFullPermissions() });
const req = new Request('http://localhost/x') as never;
const res = await downloadHandler(req, ctx, {
id: docRow.id,
slug: ['F', 'wrong.pdf'],
});
expect(res.status).toBe(404);
});
it('404s when the document has no attached file (orphaned doc)', async () => {
const port = await makePort();
const [docRow] = await db
.insert(documents)
.values({
portId: port.id,
documentType: 'other',
title: 'No file',
createdBy: testUserId,
folderId: null,
})
.returning();
const { downloadHandler } =
await import('@/app/api/v1/documents/[id]/download/[...slug]/handlers');
const ctx = makeMockCtx({ portId: port.id, permissions: makeFullPermissions() });
const req = new Request('http://localhost/x') as never;
const res = await downloadHandler(req, ctx, {
id: docRow!.id,
slug: ['anything.pdf'],
});
expect(res.status).toBe(404);
});
it('404s when the document belongs to another port (tenant isolation)', async () => {
const portA = await makePort();
const portB = await makePort();
const folder = await createFolder(portA.id, testUserId, {
name: 'Cross',
parentId: null,
});
const { docRow } = await setupDoc({
portId: portA.id,
folderId: folder.id,
filename: 'a.pdf',
body: Buffer.from('x'),
storagePath: 'test/a.pdf',
});
const { downloadHandler } =
await import('@/app/api/v1/documents/[id]/download/[...slug]/handlers');
const ctx = makeMockCtx({ portId: portB.id, permissions: makeFullPermissions() });
const req = new Request('http://localhost/x') as never;
const res = await downloadHandler(req, ctx, {
id: docRow.id,
slug: ['Cross', 'a.pdf'],
});
expect(res.status).toBe(404);
});
});