@remit/imap-worker 0.0.88 → 0.0.89

package/src/test-helpers/test-world.ts

@@ -5,17 +5,23 @@ import { tmpdir } from "node:os";
 import { join } from "node:path";
 import {
 	_resetForTest,
-	getClient,
 	type RemitClient,
 	setClient,
 } from "@remit/backend/client";
+import { composeSqliteClient } from "@remit/backend/compose-sqlite";
+import { buildSharedDeps } from "@remit/backend/shared-deps";
 import type {
 	AccountItem,
 	MailboxItem,
 	MailboxSpecialUseValue,
 	MessageItem,
 	OutboxMessageItem,
+	QueueProducer,
 } from "@remit/data-ports";
+import {
+	type RecordingQueue,
+	recordingQueue,
+} from "@remit/data-ports/recording-queue";
 import {
 	AccountAuthType,
 	ConnectionState,
@@ -28,13 +34,10 @@ import {
 	eml,
 	FakeImapServer,
 } from "@remit/mailbox-service/test-helpers/fake-imap";
-import {
-	captureQueue,
-	type QueueCapture,
-} from "@remit/mailbox-service/test-helpers/test-queue";
 import { serializeEncryptedPayload } from "@remit/secrets-service";
 import type { createConnectionScopeWithCredentials } from "../connection-scope.js";
 import { syncMessages } from "../handlers/sync-messages.js";
+import { setWorkerQueues } from "../queues.js";
 
 export interface LogLine {
 	level: string;
@@ -93,6 +96,7 @@ export interface SeededMessage {
  */
 export const openStore = async (): Promise<{
 	client: RemitClient;
+	bindWorkerQueues: () => void;
 	world: (accountConfigId?: string) => Promise<World>;
 	close: () => void;
 }> => {
@@ -103,12 +107,38 @@ export const openStore = async (): Promise<{
 	process.env.BETTER_AUTH_URL = "https://mail.example.com";
 	process.env.BETTER_AUTH_SECRET = "a-signing-secret-of-at-least-32-characters";
 	_resetForTest();
-	const client = await getClient();
+	let current = recordingQueue();
+	const queue: QueueProducer<unknown> = {
+		send: (message) => current.send(message),
+		sendBatch: (messages) => current.sendBatch(messages),
+	};
+	const client = await composeSqliteClient(
+		buildSharedDeps({
+			sync: queue,
+			accountFanout: queue,
+			smtp: queue,
+			messageManagement: queue,
+			body: queue,
+		}),
+	);
 	setClient(client);
+	const bindWorkerQueues = (): void =>
+		setWorkerQueues({
+			mailboxes: queue,
+			messages: queue,
+			flags: queue,
+			body: queue,
+			mailboxManagement: queue,
+			messageManagement: queue,
+		});
+	bindWorkerQueues();
 	return {
 		client,
-		world: (accountConfigId = `cfg-${randomUUID()}`) =>
-			createWorld(client, accountConfigId),
+		bindWorkerQueues,
+		world: (accountConfigId = `cfg-${randomUUID()}`) => {
+			current = recordingQueue();
+			return createWorld(client, current, accountConfigId);
+		},
 		close: () => {
 			_resetForTest();
 			rmSync(directory, { recursive: true, force: true });
@@ -135,7 +165,7 @@ export interface World {
 	) => Promise<Array<{ mailboxId: string; uid?: number; isDeleted: boolean }>>;
 	queued: () => Array<Record<string, unknown>>;
 	refuseSends: (error: Error | undefined) => void;
-	queue: QueueCapture;
+	queue: RecordingQueue;
 	draft: (
 		attachment?: Buffer,
 	) => Promise<{ outbox: OutboxMessageItem; attachmentId?: string }>;
@@ -145,10 +175,9 @@ export interface World {
 
 const createWorld = async (
 	client: RemitClient,
+	queue: RecordingQueue,
 	accountConfigId: string,
 ): Promise<World> => {
-	const queue = captureQueue();
-
 	const imap = new FakeImapServer();
 	const logs: LogLine[] = [];
 	const password = await client.secrets.encrypt("app-password");

package/src/processor.test.ts

@@ -1,58 +0,0 @@
-import assert from "node:assert/strict";
-import { describe, it } from "node:test";
-import type { Logger } from "@remit/logger";
-import type { ImapWorkerStopEvent } from "./events.js";
-import { processEvent } from "./processor.js";
-
-interface CapturedLogEntry {
-	args: unknown[];
-}
-
-const createCapturingLogger = (): {
-	log: Logger;
-	infoCalls: CapturedLogEntry[];
-} => {
-	const infoCalls: CapturedLogEntry[] = [];
-	const noop = () => {};
-	const log = {
-		info: (...args: unknown[]) => infoCalls.push({ args }),
-		warn: noop,
-		error: noop,
-		debug: noop,
-		fatal: noop,
-		trace: noop,
-		child: () => log,
-	} as unknown as Logger;
-	return { log, infoCalls };
-};
-
-describe("processEvent — IMAP_WORKER_STOP", () => {
-	it("returns undefined and logs the stop signal — tombstone fence on the account row already halts work", async () => {
-		const event: ImapWorkerStopEvent = {
-			type: "IMAP_WORKER_STOP",
-			accountConfigId: "acfg_alice_replay_safe_test_id",
-			accountId: "acct_alice_replay_safe_test_id",
-		};
-		const { log, infoCalls } = createCapturingLogger();
-
-		const result = await processEvent(event, log);
-
-		assert.equal(result, undefined);
-		const matched = infoCalls.find((c) => {
-			const [meta, msg] = c.args;
-			return (
-				typeof msg === "string" &&
-				msg === "Imap worker stop signal received" &&
-				typeof meta === "object" &&
-				meta !== null &&
-				(meta as { accountConfigId?: string }).accountConfigId ===
-					event.accountConfigId &&
-				(meta as { accountId?: string }).accountId === event.accountId
-			);
-		});
-		assert.ok(
-			matched,
-			"processor must log the cascade-contract stop signal with both ids",
-		);
-	});
-});