@remit/drizzle-service 0.0.103 → 0.0.105

package/src/repos/i4-account-config.test.ts

@@ -1,128 +1,4 @@
-import assert from "node:assert";
-import { after, before, describe, test } from "node:test";
-import { createSqliteRepositories } from "../sqlite-repositories.js";
-import { createTestDb, randomId, type TestDb } from "../test-db.js";
-import type { AccountConfigRepo } from "./i4-account-config.js";
+import { openSqliteTestStores } from "../test-stores.js";
+import { accountConfigRepoSuite } from "./i4-account-config.suite.js";
 
-describe("AccountConfigRepo", () => {
-	let db: TestDb;
-	let close: () => Promise<void>;
-	let repo: AccountConfigRepo;
-
-	before(async () => {
-		({ db, close } = await createTestDb());
-		repo = createSqliteRepositories({
-			app: db,
-			mail: db,
-			search: db,
-		}).accountConfig;
-	});
-
-	after(async () => {
-		await close();
-	});
-
-	test("create and get", async () => {
-		const userId = randomId();
-		const cfg = await repo.create({ userId, name: "My Config" });
-
-		assert.ok(cfg.accountConfigId);
-		assert.equal(cfg.userId, userId);
-		assert.equal(cfg.name, "My Config");
-		assert.equal(cfg.state, "active");
-
-		const fetched = await repo.get(cfg.accountConfigId);
-		assert.equal(fetched.accountConfigId, cfg.accountConfigId);
-
-		await repo.delete(cfg.accountConfigId);
-	});
-
-	test("batchGet: WHERE id = ANY($1)", async () => {
-		const userId = randomId();
-		const c1 = await repo.create({ userId });
-		const c2 = await repo.create({ userId });
-
-		const results = await repo.get([c1.accountConfigId, c2.accountConfigId]);
-		assert.equal(results.length, 2);
-
-		await repo.deleteMany([c1.accountConfigId, c2.accountConfigId]);
-	});
-
-	test("batchGet empty array returns []", async () => {
-		const results = await repo.get([]);
-		assert.deepEqual(results, []);
-	});
-
-	test("get throws for missing config", async () => {
-		await assert.rejects(repo.get(randomId()), /not found/i);
-	});
-
-	test("describe assembles collection (accountConfig + account + address)", async () => {
-		const userId = randomId();
-		const cfg = await repo.create({ userId });
-
-		const repos = createSqliteRepositories({ app: db, mail: db, search: db });
-		const accountRepo = repos.account;
-		const addrRepo = repos.address;
-
-		await accountRepo.create({
-			accountConfigId: cfg.accountConfigId,
-			username: "u",
-			email: "u@test.com",
-			isActive: true,
-			imapHost: "imap.test.com",
-			imapPort: 993,
-			imapTls: true,
-			imapStartTls: false,
-			connectionState: "not_authenticated",
-		});
-
-		await addrRepo.createAddress({
-			addressId: randomId(),
-			accountConfigId: cfg.accountConfigId,
-			localPart: "u",
-			domain: "test.com",
-			normalizedEmail: "u@test.com",
-			normalizedCompound: "u@test.com:u",
-		});
-
-		const desc = await repo.describe(cfg.accountConfigId);
-		assert.equal(desc.accountConfig.length, 1);
-		assert.equal(desc.account.length, 1);
-		assert.equal(desc.address.length, 1);
-
-		await repo.delete(cfg.accountConfigId);
-	});
-
-	test("listAll returns all configs", async () => {
-		const userId = randomId();
-		const c1 = await repo.create({ userId });
-		const c2 = await repo.create({ userId });
-
-		const all = await repo.listAll();
-		const ids = new Set(all.map((c) => c.accountConfigId));
-		assert.ok(ids.has(c1.accountConfigId));
-		assert.ok(ids.has(c2.accountConfigId));
-
-		await repo.deleteMany([c1.accountConfigId, c2.accountConfigId]);
-	});
-
-	describe("continuation token rejection (#172)", () => {
-		for (const [label, token] of [
-			["an unparseable", "not-a-cursor"],
-			["a bare number", Buffer.from("123").toString("base64url")],
-			["a JSON array", Buffer.from("[1,2]").toString("base64url")],
-		] as const) {
-			test(`${label} token is rejected as a 400`, async () => {
-				await assert.rejects(
-					() => repo.listByUser(randomId(), { continuationToken: token }),
-					(error: unknown) => {
-						assert.equal((error as { statusCode?: number }).statusCode, 400);
-						assert.equal((error as Error).name, "BadRequestError");
-						return true;
-					},
-				);
-			});
-		}
-	});
-});
+accountConfigRepoSuite(openSqliteTestStores);

package/src/repos/i4-account-export-request.d1.test.ts

@@ -0,0 +1,4 @@
+import { openD1TestStores } from "../test-d1.js";
+import { accountExportRequestRepoSuite } from "./i4-account-export-request.suite.js";
+
+accountExportRequestRepoSuite(openD1TestStores);

package/src/repos/i4-account-export-request.suite.ts

@@ -0,0 +1,44 @@
+import assert from "node:assert/strict";
+import { after, before, describe, test } from "node:test";
+import { randomId } from "../id.js";
+import type { OpenTestStores } from "../test-stores.js";
+import { AccountExportRequestRepo } from "./i4-account-export-request.js";
+
+export const accountExportRequestRepoSuite = (open: OpenTestStores): void => {
+	describe("AccountExportRequestRepo", () => {
+		let close: () => Promise<void>;
+		let repo: AccountExportRequestRepo;
+
+		before(async () => {
+			const opened = await open();
+			close = opened.close;
+			repo = new AccountExportRequestRepo(opened.stores.app.db);
+		});
+
+		after(async () => {
+			await close();
+		});
+
+		describe("continuation token rejection (#172)", () => {
+			for (const [label, token] of [
+				["an unparseable", "not-a-cursor"],
+				["a bare number", Buffer.from("123").toString("base64url")],
+				["a JSON array", Buffer.from("[1,2]").toString("base64url")],
+			] as const) {
+				test(`${label} token is rejected as a 400`, async () => {
+					await assert.rejects(
+						() =>
+							repo.listByAccountConfig(randomId(), {
+								continuationToken: token,
+							}),
+						(error: unknown) => {
+							assert.equal((error as { statusCode?: number }).statusCode, 400);
+							assert.equal((error as Error).name, "BadRequestError");
+							return true;
+						},
+					);
+				});
+			}
+		});
+	});
+};

package/src/repos/i4-account-export-request.test.ts

@@ -1,41 +1,4 @@
-import assert from "node:assert/strict";
-import { after, before, describe, test } from "node:test";
-import { createTestDb, randomId, type TestDb } from "../test-db.js";
-import { AccountExportRequestRepo } from "./i4-account-export-request.js";
+import { openSqliteTestStores } from "../test-stores.js";
+import { accountExportRequestRepoSuite } from "./i4-account-export-request.suite.js";
 
-describe("AccountExportRequestRepo", () => {
-	let db: TestDb;
-	let close: () => Promise<void>;
-	let repo: AccountExportRequestRepo;
-
-	before(async () => {
-		({ db, close } = await createTestDb());
-		repo = new AccountExportRequestRepo(db as never);
-	});
-
-	after(async () => {
-		await close();
-	});
-
-	describe("continuation token rejection (#172)", () => {
-		for (const [label, token] of [
-			["an unparseable", "not-a-cursor"],
-			["a bare number", Buffer.from("123").toString("base64url")],
-			["a JSON array", Buffer.from("[1,2]").toString("base64url")],
-		] as const) {
-			test(`${label} token is rejected as a 400`, async () => {
-				await assert.rejects(
-					() =>
-						repo.listByAccountConfig(randomId(), {
-							continuationToken: token,
-						}),
-					(error: unknown) => {
-						assert.equal((error as { statusCode?: number }).statusCode, 400);
-						assert.equal((error as Error).name, "BadRequestError");
-						return true;
-					},
-				);
-			});
-		}
-	});
-});
+accountExportRequestRepoSuite(openSqliteTestStores);

package/src/repos/i4-account-setting.d1.test.ts

@@ -0,0 +1,4 @@
+import { openD1TestStores } from "../test-d1.js";
+import { accountSettingRepoSuite } from "./i4-account-setting.suite.js";
+
+accountSettingRepoSuite(openD1TestStores);

package/src/repos/i4-account-setting.suite.ts

@@ -0,0 +1,100 @@
+import assert from "node:assert";
+import { after, before, describe, test } from "node:test";
+import { randomId } from "../id.js";
+import type { OpenTestStores } from "../test-stores.js";
+import { AccountSettingRepo } from "./i4-account-setting.js";
+
+export const accountSettingRepoSuite = (open: OpenTestStores): void => {
+	describe("AccountSettingRepo", () => {
+		let close: () => Promise<void>;
+		let repo: AccountSettingRepo;
+
+		before(async () => {
+			const opened = await open();
+			close = opened.close;
+			repo = new AccountSettingRepo(opened.stores.app.db);
+		});
+
+		after(async () => {
+			await close();
+		});
+
+		test("upsert and get", async () => {
+			const accountConfigId = randomId();
+			await repo.upsert({
+				accountConfigId,
+				name: "Theme",
+				value: { kind: "String", value: "dark" },
+			});
+
+			const setting = await repo.get(accountConfigId, "Theme");
+			assert.ok(setting);
+			assert.equal(setting.name, "Theme");
+			assert.deepEqual(setting.value, { kind: "String", value: "dark" });
+
+			await repo.delete(accountConfigId, "Theme");
+		});
+
+		test("upsert is idempotent (overwrites)", async () => {
+			const accountConfigId = randomId();
+			await repo.upsert({
+				accountConfigId,
+				name: "Density",
+				value: { kind: "String", value: "compact" },
+			});
+			await repo.upsert({
+				accountConfigId,
+				name: "Density",
+				value: { kind: "String", value: "comfortable" },
+			});
+
+			const setting = await repo.get(accountConfigId, "Density");
+			assert.deepEqual(setting?.value, {
+				kind: "String",
+				value: "comfortable",
+			});
+
+			await repo.delete(accountConfigId, "Density");
+		});
+
+		test("get returns null when setting absent", async () => {
+			const setting = await repo.get(randomId(), "Theme");
+			assert.equal(setting, null);
+		});
+
+		test("listByAccountConfig returns all settings ordered by name", async () => {
+			const accountConfigId = randomId();
+			await repo.upsert({
+				accountConfigId,
+				name: "Theme",
+				value: { kind: "String", value: "dark" },
+			});
+			await repo.upsert({
+				accountConfigId,
+				name: "Density",
+				value: { kind: "String", value: "compact" },
+			});
+
+			const settings = await repo.listByAccountConfig(accountConfigId);
+			assert.equal(settings.length, 2);
+			assert.equal(settings[0].name, "Density");
+			assert.equal(settings[1].name, "Theme");
+
+			await repo.delete(accountConfigId, "Theme");
+			await repo.delete(accountConfigId, "Density");
+		});
+
+		test("delete removes the setting", async () => {
+			const accountConfigId = randomId();
+			await repo.upsert({
+				accountConfigId,
+				name: "Theme",
+				value: { kind: "Boolean", value: true },
+			});
+			await repo.delete(accountConfigId, "Theme");
+
+			const setting = await repo.get(accountConfigId, "Theme");
+			assert.equal(setting, null);
+		});
+	});
+};

package/src/repos/i4-account-setting.test.ts

@@ -1,94 +1,4 @@
-import assert from "node:assert";
-import { after, before, describe, test } from "node:test";
-import { createTestDb, randomId, type TestDb } from "../test-db.js";
-import { AccountSettingRepo } from "./i4-account-setting.js";
+import { openSqliteTestStores } from "../test-stores.js";
+import { accountSettingRepoSuite } from "./i4-account-setting.suite.js";
 
-describe("AccountSettingRepo", () => {
-	let db: TestDb;
-	let close: () => Promise<void>;
-	let repo: AccountSettingRepo;
-
-	before(async () => {
-		({ db, close } = await createTestDb());
-		repo = new AccountSettingRepo(db as never);
-	});
-
-	after(async () => {
-		await close();
-	});
-
-	test("upsert and get", async () => {
-		const accountConfigId = randomId();
-		await repo.upsert({
-			accountConfigId,
-			name: "Theme",
-			value: { kind: "String", value: "dark" },
-		});
-
-		const setting = await repo.get(accountConfigId, "Theme");
-		assert.ok(setting);
-		assert.equal(setting.name, "Theme");
-		assert.deepEqual(setting.value, { kind: "String", value: "dark" });
-
-		await repo.delete(accountConfigId, "Theme");
-	});
-
-	test("upsert is idempotent (overwrites)", async () => {
-		const accountConfigId = randomId();
-		await repo.upsert({
-			accountConfigId,
-			name: "Density",
-			value: { kind: "String", value: "compact" },
-		});
-		await repo.upsert({
-			accountConfigId,
-			name: "Density",
-			value: { kind: "String", value: "comfortable" },
-		});
-
-		const setting = await repo.get(accountConfigId, "Density");
-		assert.deepEqual(setting?.value, { kind: "String", value: "comfortable" });
-
-		await repo.delete(accountConfigId, "Density");
-	});
-
-	test("get returns null when setting absent", async () => {
-		const setting = await repo.get(randomId(), "Theme");
-		assert.equal(setting, null);
-	});
-
-	test("listByAccountConfig returns all settings ordered by name", async () => {
-		const accountConfigId = randomId();
-		await repo.upsert({
-			accountConfigId,
-			name: "Theme",
-			value: { kind: "String", value: "dark" },
-		});
-		await repo.upsert({
-			accountConfigId,
-			name: "Density",
-			value: { kind: "String", value: "compact" },
-		});
-
-		const settings = await repo.listByAccountConfig(accountConfigId);
-		assert.equal(settings.length, 2);
-		assert.equal(settings[0].name, "Density");
-		assert.equal(settings[1].name, "Theme");
-
-		await repo.delete(accountConfigId, "Theme");
-		await repo.delete(accountConfigId, "Density");
-	});
-
-	test("delete removes the setting", async () => {
-		const accountConfigId = randomId();
-		await repo.upsert({
-			accountConfigId,
-			name: "Theme",
-			value: { kind: "Boolean", value: true },
-		});
-		await repo.delete(accountConfigId, "Theme");
-
-		const setting = await repo.get(accountConfigId, "Theme");
-		assert.equal(setting, null);
-	});
-});
+accountSettingRepoSuite(openSqliteTestStores);

package/src/repos/i4-account.test.ts

@@ -1,5 +1,6 @@
 import assert from "node:assert";
 import { after, before, describe, test } from "node:test";
+import { BETTER_SQLITE3_LIMITS } from "../bound-parameters.js";
 import { createSqliteRepositories } from "../sqlite-repositories.js";
 import { createTestDb, randomId, type TestDb } from "../test-db.js";
 import type { AccountRepo } from "./i4-account.js";
@@ -26,11 +27,14 @@ describe("AccountRepo", () => {
 
 	before(async () => {
 		({ db, close } = await createTestDb());
-		repo = createSqliteRepositories({
-			app: db,
-			mail: db,
-			search: db,
-		}).account;
+		repo = createSqliteRepositories(
+			{
+				app: db,
+				mail: db,
+				search: db,
+			},
+			BETTER_SQLITE3_LIMITS,
+		).account;
 	});
 
 	after(async () => {
@@ -164,11 +168,14 @@ describe("AccountRepo", () => {
 		const accountConfigId = randomId();
 		const account = await repo.create(makeAccountInput(accountConfigId));
 
-		const mailboxRepo = createSqliteRepositories({
-			app: db,
-			mail: db,
-			search: db,
-		}).mailbox;
+		const mailboxRepo = createSqliteRepositories(
+			{
+				app: db,
+				mail: db,
+				search: db,
+			},
+			BETTER_SQLITE3_LIMITS,
+		).mailbox;
 		mailboxCreated(
 			await mailboxRepo.create({
 				accountId: account.accountId,

package/src/repos/i4-address-junk-move.sqlite.test.ts

@@ -1,6 +1,7 @@
 import assert from "node:assert/strict";
 import { after, before, beforeEach, describe, test } from "node:test";
 import type Database from "better-sqlite3";
+import { BETTER_SQLITE3_LIMITS } from "../bound-parameters.js";
 import { createSqliteRepositories } from "../sqlite-repositories.js";
 import { createTestDb, type TestDb } from "../test-db.js";
 import { AddressRepo } from "./i4-address.js";
@@ -125,11 +126,14 @@ describe("reconciling one message's addresses at the moment it moves", () => {
 	before(async () => {
 		({ db, sqlite, close } = await createTestDb());
 		repo = new AddressRepo(db as never);
-		specialUse = createSqliteRepositories({
-			app: db,
-			mail: db,
-			search: db,
-		}).mailboxSpecialUse;
+		specialUse = createSqliteRepositories(
+			{
+				app: db,
+				mail: db,
+				search: db,
+			},
+			BETTER_SQLITE3_LIMITS,
+		).mailboxSpecialUse;
 	});
 
 	/**

package/src/repos/i4-mailbox-special-use.sqlite.test.ts

@@ -6,6 +6,7 @@ import {
 	meetsTrashAssurance,
 } from "@remit/data-ports/folder-role";
 import { CanonicalMailboxRole } from "@remit/domain-enums";
+import { BETTER_SQLITE3_LIMITS } from "../bound-parameters.js";
 import {
 	accountSettingTable,
 	accountTable,
@@ -57,11 +58,14 @@ describe("MailboxSpecialUseRepo role lookups (sqlite)", () => {
 		});
 		close = testDb.close;
 		db = testDb.db;
-		const repos = createSqliteRepositories({
-			app: testDb.db,
-			mail: testDb.db,
-			search: testDb.db,
-		});
+		const repos = createSqliteRepositories(
+			{
+				app: testDb.db,
+				mail: testDb.db,
+				search: testDb.db,
+			},
+			BETTER_SQLITE3_LIMITS,
+		);
 		repo = repos.mailboxSpecialUse;
 		mailboxes = repos.mailbox;
 		accountSettings = repos.accountSetting;

package/src/repos/i4-mailbox.sqlite.test.ts

@@ -4,6 +4,7 @@ import { after, before, describe, test } from "node:test";
 import { MailboxSyncStatus } from "@remit/domain-enums";
 import Database from "better-sqlite3";
 import { drizzle } from "drizzle-orm/better-sqlite3";
+import { BETTER_SQLITE3_LIMITS } from "../bound-parameters.js";
 import { mailboxTable } from "../schema.js";
 import { createSqliteRepositories } from "../sqlite-repositories.js";
 import { createSqliteTestDb } from "../test-db-sqlite.js";
@@ -41,11 +42,14 @@ describe("MailboxRepo (sqlite)", () => {
 	before(async () => {
 		const testDb = await createSqliteTestDb({ mailbox: mailboxTable });
 		close = testDb.close;
-		repo = createSqliteRepositories({
-			app: testDb.db,
-			mail: testDb.db,
-			search: testDb.db,
-		}).mailbox;
+		repo = createSqliteRepositories(
+			{
+				app: testDb.db,
+				mail: testDb.db,
+				search: testDb.db,
+			},
+			BETTER_SQLITE3_LIMITS,
+		).mailbox;
 	});
 
 	after(async () => {
@@ -211,11 +215,14 @@ describe("MailboxRepo (sqlite, shipped migrations)", () => {
 		const sqlite = new Database(":memory:");
 		applyMigrationSet(sqlite, "mail");
 		const db = drizzle(sqlite, { schema: { mailbox: mailboxTable } });
-		repo = createSqliteRepositories({
-			app: db,
-			mail: db,
-			search: db,
-		}).mailbox;
+		repo = createSqliteRepositories(
+			{
+				app: db,
+				mail: db,
+				search: db,
+			},
+			BETTER_SQLITE3_LIMITS,
+		).mailbox;
 		close = async () => {
 			sqlite.close();
 		};

package/src/repos/i4-mailbox.test.ts

@@ -3,6 +3,7 @@ import { after, before, describe, test } from "node:test";
 import { MailboxSyncStatus } from "@remit/domain-enums";
 import { eq } from "drizzle-orm";
 import { syncBatch } from "../batch.js";
+import { BETTER_SQLITE3_LIMITS } from "../bound-parameters.js";
 import {
 	envelopeId as deriveEnvelopeId,
 	rootBodyPartId as deriveRootBodyPartId,
@@ -54,11 +55,14 @@ describe("MailboxRepo", () => {
 
 	before(async () => {
 		({ db, close } = await createTestDb());
-		repo = createSqliteRepositories({
-			app: db,
-			mail: db,
-			search: db,
-		}).mailbox;
+		repo = createSqliteRepositories(
+			{
+				app: db,
+				mail: db,
+				search: db,
+			},
+			BETTER_SQLITE3_LIMITS,
+		).mailbox;
 	});
 
 	after(async () => {
@@ -543,11 +547,14 @@ describe("MailboxRepo", () => {
 	describe("deleteMailboxWithMail — the folder's mail goes with it (D8)", () => {
 		const seedMessage = async (mailboxId: string, uid: number) => {
 			const messageId = randomId();
-			await createSqliteRepositories({
-				app: db,
-				mail: db,
-				search: db,
-			}).message.create({
+			await createSqliteRepositories(
+				{
+					app: db,
+					mail: db,
+					search: db,
+				},
+				BETTER_SQLITE3_LIMITS,
+			).message.create({
 				messageId,
 				mailboxId,
 				uid,

package/src/repos/i4-outbox-attachment.test.ts

@@ -1,5 +1,6 @@
 import assert from "node:assert";
 import { after, before, describe, test } from "node:test";
+import { BETTER_SQLITE3_LIMITS } from "../bound-parameters.js";
 import { createSqliteRepositories } from "../sqlite-repositories.js";
 import { createTestDb, randomId } from "../test-db.js";
 import type { OutboxAttachmentRepo } from "./i4-outbox-attachment.js";
@@ -28,11 +29,14 @@ const input = (
 
 before(async () => {
 	harness = await createTestDb();
-	repo = createSqliteRepositories({
-		app: harness.db,
-		mail: harness.db,
-		search: harness.db,
-	}).outboxAttachment;
+	repo = createSqliteRepositories(
+		{
+			app: harness.db,
+			mail: harness.db,
+			search: harness.db,
+		},
+		BETTER_SQLITE3_LIMITS,
+	).outboxAttachment;
 });
 
 after(async () => {

package/src/repos/label.conformance.d1.test.ts

@@ -0,0 +1,20 @@
+import { labelRepositoryConformance } from "@remit/data-ports/conformance";
+import { NotFoundError } from "../error.js";
+import { randomId } from "../id.js";
+import { openD1TestStores } from "../test-d1.js";
+import { LabelRepo } from "./label.js";
+
+let close: (() => Promise<void>) | undefined;
+
+labelRepositoryConformance({
+	async createRepository() {
+		const opened = await openD1TestStores();
+		close = opened.close;
+		return new LabelRepo(opened.stores.app.db);
+	},
+	teardown: async () => {
+		await close?.();
+	},
+	makeId: () => randomId(),
+	isNotFoundError: (error) => error instanceof NotFoundError,
+});