@remit/imap-worker 0.0.85 → 0.0.87

package/src/handlers/message-move.test.ts

@@ -1,38 +1,16 @@
 import assert from "node:assert";
-import { afterEach, before, beforeEach, describe, it, mock } from "node:test";
-import { getClient, type RemitClient, setClient } from "@remit/backend/client";
-import type { AccountItem, ThreadMessageItem } from "@remit/data-ports";
-import type { Logger } from "@remit/logger-lambda";
-import type { IImapConnection } from "@remit/mailbox-service";
-import type { MessageMoveEvent } from "../events.js";
-import { withOAuthLifecycle } from "../with-oauth-lifecycle.js";
-import { buildLifecycleDeps } from "../with-oauth-lifecycle-deps.js";
+import { describe, it } from "node:test";
+import type { ThreadMessageItem } from "@remit/data-ports";
+import {
+	eml,
+	FakeImapServer,
+} from "@remit/mailbox-service/test-helpers/fake-imap";
 import {
 	buildThreadMessageMoveUpdate,
-	emitMoveResync,
-	getMessageMoveMaxAttempts,
-	handleMessageMove,
-	MESSAGE_MOVE_MAX_ATTEMPTS,
-	type MessageMoveDeps,
-	moveThenResync,
 	probePausedPlacement,
 	searchMailboxForHighestMessageIdUid,
 } from "./message-move.js";
 
-const silentLogger = (() => {
-	const noop = () => {};
-	const log = {
-		info: noop,
-		warn: noop,
-		error: noop,
-		debug: noop,
-		fatal: noop,
-		trace: noop,
-		child: () => log,
-	} as unknown as Logger;
-	return log;
-})();
-
 const sourceMailboxId = "source-mailbox-id-aaaaaaaaa";
 const destinationMailboxId = "destination-mailbox-aaaaa";
 
@@ -80,39 +58,6 @@ describe("buildThreadMessageMoveUpdate", () => {
 		);
 	});
 
-	it("composites.mailboxId is the CURRENT (source) mailboxId, not the destination", () => {
-		const args = buildThreadMessageMoveUpdate(
-			baseThreadMessage,
-			42,
-			destinationMailboxId,
-		);
-
-		assert.strictEqual(
-			args.composites.mailboxId,
-			sourceMailboxId,
-			"composites.mailboxId must be the CURRENT source mailbox; passing the destination breaks the conditional check",
-		);
-		assert.notStrictEqual(
-			args.composites.mailboxId,
-			destinationMailboxId,
-			"composites.mailboxId must NOT match the new destinationMailboxId",
-		);
-	});
-
-	it("composites.isDeleted is the CURRENT value, not the new false", () => {
-		const args = buildThreadMessageMoveUpdate(
-			{ ...baseThreadMessage, isDeleted: true },
-			42,
-			destinationMailboxId,
-		);
-
-		assert.strictEqual(
-			args.composites.isDeleted,
-			true,
-			"composites.isDeleted must be the CURRENT value (true here), not the new value",
-		);
-	});
-
 	it("composites mirrors every CURRENT indexed attribute on the threadMessage", () => {
 		const tm = {
 			sentDate: 1700000000123,
@@ -136,868 +81,136 @@ describe("buildThreadMessageMoveUpdate", () => {
 	});
 });
 
-const accountId = "alice-account-aaaaaaaaaa";
-
-describe("emitMoveResync (#1031)", () => {
-	// A move shifts a message between two folders; both folders' counts must
-	// refresh from IMAP via the existing per-folder SYNC_MESSAGES sync. Counts
-	// are never mutated locally — only re-read downstream.
-
-	it("emits SYNC_MESSAGES for both the source and destination folders", async () => {
-		const emit = mock.fn(async () => undefined);
-
-		await emitMoveResync(emit, {
-			accountId,
-			sourceMailboxId,
-			destinationMailboxId,
-		});
-
-		assert.equal(emit.mock.calls.length, 2);
-		assert.deepEqual(emit.mock.calls[0].arguments, [
-			{ type: "SYNC_MESSAGES", accountId, mailboxId: sourceMailboxId },
-		]);
-		assert.deepEqual(emit.mock.calls[1].arguments, [
-			{ type: "SYNC_MESSAGES", accountId, mailboxId: destinationMailboxId },
-		]);
-	});
-});
-
-describe("moveThenResync (#1031)", () => {
-	it("runs the resync after the move resolves", async () => {
-		const order: string[] = [];
-		const performMove = mock.fn(async () => {
-			order.push("move");
-		});
-		const resync = mock.fn(async () => {
-			order.push("resync");
-		});
-
-		await moveThenResync(performMove, resync);
-
-		assert.deepEqual(order, ["move", "resync"]);
-		assert.equal(resync.mock.calls.length, 1);
-	});
-
-	it("does not resync when the move fails, and propagates the error", async () => {
-		const performMove = mock.fn(async () => {
-			throw new Error("IMAP move failed");
-		});
-		const resync = mock.fn(async () => undefined);
-
-		await assert.rejects(
-			() => moveThenResync(performMove, resync),
-			/IMAP move failed/,
-		);
-
-		assert.equal(resync.mock.calls.length, 0);
-	});
-});
-
-describe("getMessageMoveMaxAttempts — env-derived threshold (#655)", () => {
-	it("parses the injected env var", () => {
-		assert.equal(
-			getMessageMoveMaxAttempts({ MESSAGE_MOVE_MAX_ATTEMPTS: "3" }),
-			3,
-		);
-		assert.equal(
-			getMessageMoveMaxAttempts({ MESSAGE_MOVE_MAX_ATTEMPTS: "5" }),
-			5,
-		);
-	});
-
-	it("defaults to the message queue's own maxReceiveCount when unset", () => {
-		assert.equal(getMessageMoveMaxAttempts({}), 3);
-	});
-
-	it("defaults on a non-numeric or non-positive value", () => {
-		assert.equal(
-			getMessageMoveMaxAttempts({ MESSAGE_MOVE_MAX_ATTEMPTS: "nope" }),
-			3,
-		);
-		assert.equal(
-			getMessageMoveMaxAttempts({ MESSAGE_MOVE_MAX_ATTEMPTS: "0" }),
-			3,
-		);
-	});
-
-	it("MESSAGE_MOVE_MAX_ATTEMPTS is a concrete, positive number at module load", () => {
-		assert.ok(MESSAGE_MOVE_MAX_ATTEMPTS > 0);
-	});
-});
-
-describe("handleMessageMove — the move's own pending state gates every attempt", () => {
-	const acctId = "mm-acc-zzz";
-
-	const cappedAccount = (): AccountItem =>
-		({
-			accountId: acctId,
-			accountConfigId: "mm-cfg-zzz",
-			connectionState: "authenticated",
-			username: "mm@imap.example.com",
-			imapHost: "imap.example.com",
-			imapPort: 993,
-			imapTls: true,
-			passwordHash: JSON.stringify({
-				encryptedDek: "",
-				encryptedData: "",
-				iv: "",
-				authTag: "",
-			}),
-		}) as unknown as AccountItem;
-
-	const event: MessageMoveEvent = {
-		type: "MESSAGE_MOVE",
-		accountId: acctId,
-		accountConfigId: "mm-cfg-zzz",
-		messageId: "mm-msg-zzz",
-		sourceMailboxId: "mm-src-zzz",
-		sourceMailboxPath: "INBOX",
-		destinationMailboxId: "mm-dst-zzz",
-		destinationMailboxPath: "Archive",
-		uid: 10,
-		eventId: "mm-evt-zzz",
-		timestamp: 1700000000000,
-	} as MessageMoveEvent;
-
-	const pendingRow = () => ({
-		messageId: "mm-msg-zzz",
-		mailboxId: "mm-dst-zzz",
-		uid: 10,
-		status: "moving",
-		syncStatus: "pending",
-	});
-
-	// The client is supplied by injection (`setClient`), so these tests register
-	// the repositories they then mock rather than reaching a composition.
-	before(() => {
-		setClient({
-			account: { get: async () => undefined },
-			mailbox: { get: async () => undefined },
-			message: {
-				get: async () => [],
-				update: async () => undefined,
-				updateUid: async () => undefined,
-				transitionPlacement: async () => undefined,
-			},
-			threadMessage: {
-				findAllByMessageId: async () => [],
-				findByMessageId: async () => undefined,
-				update: async () => undefined,
-			},
-			secrets: { decrypt: async () => undefined },
-		} as unknown as RemitClient);
-	});
-
-	let emitted: unknown[] = [];
-	let connectCount = 0;
-	let logLines: { level: string; fields: Record<string, unknown> }[] = [];
-
-	// The level a give-up is logged at is behaviour: a routine pause must not
-	// page anyone, and a placement the server never settled must.
-	const capturingLogger = (): Logger => {
-		const at =
-			(level: string) =>
-			(fields: Record<string, unknown>): void => {
-				logLines.push({ level, fields });
-			};
-		const log = {
-			info: at("info"),
-			warn: at("warn"),
-			error: at("error"),
-			debug: at("debug"),
-			fatal: at("fatal"),
-			trace: at("trace"),
-			child: () => log,
-		} as unknown as Logger;
-		return log;
-	};
-
-	// One folder's Message-ID SEARCH answers, keyed by the box last opened.
-	// `openBox` on this handle is deliberately unguarded: the paused settle asks
-	// on the identity axis, which a UIDVALIDITY change leaves intact.
-	const holdingConnection = (
-		holdings: Record<string, number[]>,
-	): IImapConnection => {
-		let opened = "";
-		return {
-			openBox: async (path: string) => {
-				opened = path;
-				return {} as never;
-			},
-			search: async () => holdings[opened] ?? [],
-		} as unknown as IImapConnection;
-	};
-
-	const moveDeps = (connection?: IImapConnection): MessageMoveDeps => ({
-		getClient,
-		buildLifecycleDeps,
-		withOAuthLifecycle,
-		createConnectionScope: () => ({
-			getConnection: async () => {
-				connectCount += 1;
-				if (!connection) throw new Error("this case must not connect");
-				return connection;
-			},
-			disconnect: async () => undefined,
-		}),
-		emitEvent: (async (event: unknown) => {
-			emitted.push(event);
-		}) as MessageMoveDeps["emitEvent"],
-	});
-
-	const pausedSource = () => ({
-		mailboxId: "mm-src-zzz",
-		uidValidity: 1,
-		cursorState: "rebuilding",
-	});
-
-	beforeEach(() => {
-		emitted = [];
-		connectCount = 0;
-		logLines = [];
-	});
-
-	afterEach(() => mock.restoreAll());
-
-	it("acks without connecting when the source mailbox was deleted", async () => {
-		const client = await getClient();
-		mock.method(client.account, "get", async () => cappedAccount());
-		mock.method(client.secrets, "decrypt", async () => "fake-password");
-		mock.method(client.message, "get", async () => [pendingRow()]);
-		mock.method(client.mailbox, "get", async () => {
-			throw Object.assign(new Error("Mailbox not found: mm-src-zzz"), {
-				name: "NotFoundError",
-			});
-		});
-		const updateUid = mock.method(client.message, "updateUid", async () => {});
-
-		// Must resolve, not reject — a deleted source folder makes the move moot.
-		await handleMessageMove(event, silentLogger);
-
-		assert.equal(
-			updateUid.mock.calls.length,
-			0,
-			"a deleted mailbox never reaches the IMAP move",
-		);
-	});
-
-	// Without this gate a redelivery of an already-confirmed move re-runs the
-	// MOVE against a UID the source no longer holds, and on exhaustion the
-	// terminal resolver reads the source's honest "gone" as grounds to delete a
-	// row that is correct and settled.
-	it("acks without connecting when the move already settled", async () => {
-		const client = await getClient();
-		mock.method(client.account, "get", async () => cappedAccount());
-		mock.method(client.secrets, "decrypt", async () => "fake-password");
-		mock.method(client.message, "get", async () => [
-			{ ...pendingRow(), uid: 4711, status: "active", syncStatus: "synced" },
-		]);
-		const mailboxGet = mock.method(client.mailbox, "get", async () => {
-			throw new Error("a settled move must never resolve a mailbox");
-		});
-		const update = mock.method(client.message, "update", async () => {});
-
-		await handleMessageMove(event, silentLogger, MESSAGE_MOVE_MAX_ATTEMPTS);
-
-		assert.equal(mailboxGet.mock.calls.length, 0);
-		assert.equal(
-			update.mock.calls.length,
-			0,
-			"a settled row is never written again",
-		);
-	});
-
-	const arrangePausedMove = async (
-		options: { probeable: boolean } = { probeable: true },
-	): Promise<{
-		transitionCalls: unknown[][];
-		updateUidCalls: unknown[][];
-		threadUpdateCalls: unknown[][];
-	}> => {
-		const client = await getClient();
-		mock.method(client.account, "get", async () => cappedAccount());
-		mock.method(client.secrets, "decrypt", async () => "fake-password");
-		mock.method(client.message, "get", async () => [
-			{
-				...pendingRow(),
-				messageIdHeader: options.probeable ? "<moved@example.com>" : undefined,
-			},
-		]);
-		mock.method(
-			client.mailbox,
-			"get",
-			async (_accountId: string, id: string) =>
-				id === "mm-src-zzz"
-					? { ...pausedSource(), fullPath: "INBOX" }
-					: {
-							mailboxId: id,
-							uidValidity: 1,
-							cursorState: "normal",
-							fullPath: "Archive",
-						},
-		);
-		const transitionCalls: unknown[][] = [];
-		mock.method(
-			client.message,
-			"transitionPlacement",
-			async (...args: unknown[]) => {
-				transitionCalls.push(args);
-				return { messageId: "mm-msg-zzz" };
-			},
-		);
-		const updateUidCalls: unknown[][] = [];
-		mock.method(client.message, "updateUid", async (...args: unknown[]) => {
-			updateUidCalls.push(args);
-		});
-		const threadRow = {
-			...baseThreadMessage,
-			accountConfigId: "mm-cfg-zzz",
-			threadMessageId: "mm-tm-zzz",
-			mailboxId: "mm-dst-zzz",
-		};
-		mock.method(client.threadMessage, "findAllByMessageId", async () => [
-			threadRow,
-		]);
-		mock.method(client.threadMessage, "findByMessageId", async () => threadRow);
-		const threadUpdateCalls: unknown[][] = [];
-		mock.method(client.threadMessage, "update", async (...args: unknown[]) => {
-			threadUpdateCalls.push(args);
-		});
-		return { transitionCalls, updateUidCalls, threadUpdateCalls };
-	};
-
-	// Issue #1203. Acking a paused cursor left the row `moving` with `mailboxId`
-	// on the destination and `uid` on the source. Nothing re-enqueues a
-	// MESSAGE_MOVE, and the cursor rebuild matches rows by
-	// `(accountConfigId, mailboxId)`, so a row naming the destination sits in
-	// neither folder's set: `placementBindingOf` answered `in_flight` for good
-	// and the message became unmovable and undeletable.
-	it("hands the row back to its source on a first delivery, without connecting", async () => {
-		const { transitionCalls, threadUpdateCalls } = await arrangePausedMove();
-
-		await handleMessageMove(event, silentLogger, 1, moveDeps());
-
-		assert.equal(
-			connectCount,
-			0,
-			"a first delivery has provably issued no MOVE, so it needs no answer from the server",
-		);
-		assert.deepEqual(
-			transitionCalls[0],
-			[
-				"mm-msg-zzz",
-				{ status: ["moving", "deleting"] },
-				{
-					mailboxId: "mm-src-zzz",
-					uid: 10,
-					status: "active",
-					syncStatus: "synced",
-					abandonedMutation: "none",
-				},
-			],
-			"the row goes back to the source pair, which is the set the rebuild adjudicates",
-		);
-		assert.equal(
-			(threadUpdateCalls[0]?.[2] as { mailboxId?: string })?.mailboxId,
-			"mm-src-zzz",
-			"the listing row follows the message back to its source folder",
-		);
-	});
-
-	// A paused settle is a settle, so both folders re-read their counts from
-	// IMAP — the same resync every other terminal verdict in this handler runs.
-	// Without it the repair depended entirely on someone else arming the
-	// mailbox's rebuild.
-	it("resyncs both folders after a paused move settles", async () => {
-		await arrangePausedMove();
-
-		await handleMessageMove(event, silentLogger, 1, moveDeps());
-
-		assert.deepEqual(emitted, [
-			{ type: "SYNC_MESSAGES", accountId: acctId, mailboxId: "mm-src-zzz" },
-			{ type: "SYNC_MESSAGES", accountId: acctId, mailboxId: "mm-dst-zzz" },
-		]);
-	});
-
-	// Issue #1203, the redelivery half, and the mail-loss edge in it. INBOX was
-	// recreated (UIDVALIDITY bumped, which is why the cursor is paused), the
-	// MOVE never ran, and INBOX honestly answers "no" for the Message-ID because
-	// it no longer holds anything it used to. Archive holds an older copy of the
-	// same Message-ID at uid 40 — a sieve `fileinto` + `keep`, a resend — which
-	// `deriveMessageId` folds into this one local row. Settling `moved` on 40
-	// binds the row to that copy, and the next permanent delete expunges it:
-	// Archive is not paused, so no rebuild is ever coming to adjudicate there.
-	it("never settles a redelivered paused move as moved off a destination sighting", async () => {
-		const { transitionCalls, updateUidCalls } = await arrangePausedMove();
-
-		await handleMessageMove(
-			event,
-			capturingLogger(),
-			2,
-			moveDeps(holdingConnection({ INBOX: [], Archive: [40] })),
-		);
-
-		assert.equal(
-			updateUidCalls.length,
-			0,
-			"a sighting off a source whose uid axis has moved is not proof this move ran",
-		);
-		// The source pair is the set its own cursor rebuild walks and adjudicates
-		// by Message-ID, so handing the row back there is the reconcile path.
-		assert.deepEqual(transitionCalls[0]?.[2], {
-			mailboxId: "mm-src-zzz",
-			uid: 10,
-			status: "active",
-			syncStatus: "abandoned",
-			abandonedMutation: "move",
-		});
-		assert.equal(
-			logLines.filter(
-				(line) =>
-					line.fields.alert === "message_move_paused_placement_unproven",
-			).length,
-			1,
-			"a placement the server never settled is an operator's to adjudicate",
-		);
-	});
-
-	// The taxonomy `message-copy.ts` already applies: an evidence-free
-	// redelivery is broken, not settled. With no Message-ID header neither
-	// folder was asked, and silence is not an answer to write `synced` on.
-	it("settles a redelivered paused move broken when neither folder can be asked", async () => {
-		const { transitionCalls } = await arrangePausedMove({
-			probeable: false,
-		});
-
-		await handleMessageMove(
-			event,
-			capturingLogger(),
-			2,
-			moveDeps(holdingConnection({})),
-		);
-
-		assert.equal(
-			(transitionCalls[0]?.[2] as { syncStatus?: string })?.syncStatus,
-			"abandoned",
-		);
-		assert.equal(
-			logLines.filter(
-				(line) =>
-					line.fields.alert === "message_move_paused_placement_unproven",
-			).length,
-			1,
-		);
-	});
-
-	// The user's move was silently dropped. Nothing throws, nothing alerts and
-	// the row reads settled, so the only trace it ever leaves is this metric —
-	// the counterpart of copy's `message_copy_not_landed`.
-	it("counts a paused move the server was never told about", async () => {
-		await arrangePausedMove();
+const _accountId = "alice-account-aaaaaaaaaa";
 
-		await handleMessageMove(event, capturingLogger(), 1, moveDeps());
-
-		const dropped = logLines.filter(
-			(line) => line.fields.metric === "message_move_dropped_on_pause",
-		);
-		assert.equal(dropped.length, 1);
-		assert.equal(dropped[0]?.level, "warn");
-	});
-
-	// Issue #1122 on the same path. The destination already held an older copy
-	// of this Message-ID and the MOVE never ran, so the destination hit is not
-	// this message. The source is asked first for exactly that reason, and its
-	// answer ends it.
-	it("never binds a redelivered paused move to an older copy while the source still answers", async () => {
-		const { transitionCalls, updateUidCalls } = await arrangePausedMove();
-
-		await handleMessageMove(
-			event,
-			silentLogger,
-			2,
-			moveDeps(holdingConnection({ INBOX: [4], Archive: [12] })),
-		);
-
-		assert.equal(updateUidCalls.length, 0);
-		assert.deepEqual(transitionCalls[0]?.[2], {
-			mailboxId: "mm-src-zzz",
-			uid: 10,
-			status: "active",
-			syncStatus: "synced",
-			abandonedMutation: "none",
-		});
-	});
-
-	const arrangeLiveMove = async (
-		folders: Record<string, string>,
-	): Promise<{
-		opened: string[];
-		movedTo: string[];
-		updateUidCalls: unknown[][];
-		transitionCalls: unknown[][];
-		connection: IImapConnection;
-	}> => {
-		const client = await getClient();
-		mock.method(client.account, "get", async () => cappedAccount());
-		mock.method(client.secrets, "decrypt", async () => "fake-password");
-		mock.method(client.message, "get", async () => [pendingRow()]);
-		mock.method(
-			client.mailbox,
-			"get",
-			async (_accountId: string, id: string) => {
-				const fullPath = folders[id];
-				if (fullPath === undefined) {
-					throw Object.assign(new Error(`Mailbox not found: ${id}`), {
-						name: "NotFoundError",
-					});
-				}
-				return {
-					mailboxId: id,
-					uidValidity: 1,
-					cursorState: "normal",
-					fullPath,
-				};
-			},
-		);
-		const updateUidCalls: unknown[][] = [];
-		mock.method(client.message, "updateUid", async (...args: unknown[]) => {
-			updateUidCalls.push(args);
-		});
-		const transitionCalls: unknown[][] = [];
-		mock.method(
-			client.message,
-			"transitionPlacement",
-			async (...args: unknown[]) => {
-				transitionCalls.push(args);
-				return { messageId: "mm-msg-zzz" };
-			},
-		);
-		const opened: string[] = [];
-		const movedTo: string[] = [];
-		const connection = {
-			openBox: async (path: string) => {
-				opened.push(path);
-				return { uidvalidity: 1 } as never;
-			},
-			moveMessages: async (_uids: number[], destination: string) => {
-				movedTo.push(destination);
-				return { uidMap: new Map([[10, 44]]) };
-			},
-		} as unknown as IImapConnection;
-		return { opened, movedTo, updateUidCalls, transitionCalls, connection };
+describe("probePausedPlacement — which folder a paused mutation left the message in (#1203)", () => {
+	const HEADER = "<paused@example.com>";
+
+	const serverHolding = (holdings: Record<string, number[]>) => {
+		const server = new FakeImapServer();
+		server.addFolder("INBOX");
+		server.addFolder("Archive");
+		for (const [path, uids] of Object.entries(holdings)) {
+			for (const uid of uids)
+				server.deliver(path, eml({ "Message-ID": HEADER }), { uid: uid });
+		}
+		return server;
 	};
 
-	it("runs a move enqueued with the unversioned payload that carries folder paths", async () => {
-		const { movedTo, updateUidCalls, connection } = await arrangeLiveMove({
-			"mm-src-zzz": "INBOX",
-			"mm-dst-zzz": "Archive",
-		});
-
-		await handleMessageMove(event, silentLogger, 1, moveDeps(connection));
-
-		assert.deepEqual(movedTo, ["Archive"]);
-		assert.deepEqual(updateUidCalls[0], ["mm-msg-zzz", 44, "mm-dst-zzz"]);
-	});
-
-	it("runs a move enqueued with the pathless payload", async () => {
-		const { opened, movedTo, connection } = await arrangeLiveMove({
-			"mm-src-zzz": "INBOX",
-			"mm-dst-zzz": "Archive",
-		});
-		const pathless: MessageMoveEvent = {
-			type: "MESSAGE_MOVE",
-			schemaVersion: 3,
-			accountId: acctId,
-			messageId: "mm-msg-zzz",
-			sourceMailboxId: "mm-src-zzz",
-			destinationMailboxId: "mm-dst-zzz",
-			uid: 10,
-			eventId: "mm-evt-zzz",
-			timestamp: 1700000000000,
-		};
-
-		await handleMessageMove(pathless, silentLogger, 1, moveDeps(connection));
+	const opened = (server: FakeImapServer): string[] =>
+		server.log
+			.filter((line) => /^(SELECT|EXAMINE) /.test(line))
+			.map((line) => line.split(" ")[1] ?? "");
 
-		assert.deepEqual(opened, ["INBOX"]);
-		assert.deepEqual(movedTo, ["Archive"]);
-	});
-
-	it("moves under the destination's current path when the folder was renamed after enqueue", async () => {
-		const { opened, movedTo, updateUidCalls, connection } =
-			await arrangeLiveMove({
-				"mm-src-zzz": "INBOX",
-				"mm-dst-zzz": "Records",
-			});
-
-		await handleMessageMove(event, silentLogger, 1, moveDeps(connection));
-
-		assert.deepEqual(opened, ["INBOX"]);
-		assert.deepEqual(
-			movedTo,
-			["Records"],
-			"the payload still says Archive; the row says where the folder is now",
-		);
-		assert.deepEqual(updateUidCalls[0], ["mm-msg-zzz", 44, "mm-dst-zzz"]);
-	});
-
-	it("opens the source under its current path when it was renamed after enqueue", async () => {
-		const { opened, movedTo, connection } = await arrangeLiveMove({
-			"mm-src-zzz": "Old/Inbox-renamed",
-			"mm-dst-zzz": "Archive",
-		});
-
-		await handleMessageMove(event, silentLogger, 1, moveDeps(connection));
-
-		assert.deepEqual(
-			opened,
-			["Old/Inbox-renamed"],
-			"the payload still says INBOX; the row says where the folder is now",
-		);
-		assert.deepEqual(movedTo, ["Archive"]);
-	});
-
-	it("hands the row back to its source without connecting when the destination mailbox was deleted", async () => {
-		const { transitionCalls, updateUidCalls } = await arrangeLiveMove({
-			"mm-src-zzz": "INBOX",
+	const probe = (
+		server: FakeImapServer,
+		input: Partial<Parameters<typeof probePausedPlacement>[1]> = {},
+	) =>
+		probePausedPlacement(server.connection(), {
+			messageIdHeader: HEADER,
+			sourceMailboxPath: "INBOX",
+			destinationMailboxPath: "Archive",
+			...input,
 		});
 
-		await handleMessageMove(event, capturingLogger(), 1, moveDeps());
-
-		assert.equal(connectCount, 0);
-		assert.equal(updateUidCalls.length, 0);
-		assert.deepEqual(transitionCalls[0]?.[2], {
-			mailboxId: "mm-src-zzz",
-			uid: 10,
-			status: "active",
-			syncStatus: "synced",
-			abandonedMutation: "none",
-		});
-		assert.equal(
-			logLines.filter(
-				(line) => line.fields.metric === "message_move_destination_gone",
-			).length,
-			1,
-		);
-		assert.deepEqual(emitted, [
-			{ type: "SYNC_MESSAGES", accountId: acctId, mailboxId: "mm-src-zzz" },
-		]);
-	});
-
-	it("acks without connecting when the message row is already gone", async () => {
-		const client = await getClient();
-		mock.method(client.account, "get", async () => cappedAccount());
-		mock.method(client.secrets, "decrypt", async () => "fake-password");
-		mock.method(client.message, "get", async () => []);
-		const mailboxGet = mock.method(client.mailbox, "get", async () => {
-			throw new Error("a missing row must never resolve a mailbox");
-		});
-
-		await handleMessageMove(event, silentLogger, MESSAGE_MOVE_MAX_ATTEMPTS);
-
-		assert.equal(mailboxGet.mock.calls.length, 0);
-	});
-});
-
-describe("probePausedPlacement — which folder a paused mutation left the message in (#1203)", () => {
-	const buildConnection = (
-		holdings: Record<string, number[]>,
-	): { connection: IImapConnection; opened: string[] } => {
-		const opened: string[] = [];
-		let current = "";
-		return {
-			opened,
-			connection: {
-				openBox: async (path: string) => {
-					opened.push(path);
-					current = path;
-					return {} as never;
-				},
-				search: async () => holdings[current] ?? [],
-			} as unknown as IImapConnection,
-		};
-	};
-
-	const probe = (holdings: Record<string, number[]>) => {
-		const { connection, opened } = buildConnection(holdings);
-		return {
-			opened,
-			result: probePausedPlacement(connection, {
-				messageIdHeader: "<paused@example.com>",
-				sourceMailboxPath: "INBOX",
-				destinationMailboxPath: "Archive",
-			}),
-		};
-	};
-
 	it("stops at the source and never asks the destination when the source still holds it", async () => {
-		const { opened, result } = probe({ INBOX: [4], Archive: [12] });
+		const server = serverHolding({ INBOX: [4], Archive: [12] });
 
-		assert.deepStrictEqual(await result, { kind: "at-source" });
+		assert.deepStrictEqual(await probe(server), { kind: "at-source" });
 		assert.deepStrictEqual(
-			opened,
+			opened(server),
 			["INBOX"],
 			"a source hit ends it: with the mutation unrun, every destination hit is an older copy (#1122)",
 		);
 	});
 
 	it("answers the destination's highest matching uid once the source has let go", async () => {
-		const { result } = probe({ INBOX: [], Archive: [12, 77] });
+		const server = serverHolding({ Archive: [12, 77] });
 
-		assert.deepStrictEqual(await result, { kind: "at-destination", uid: 77 });
+		assert.deepStrictEqual(await probe(server), {
+			kind: "at-destination",
+			uid: 77,
+		});
 	});
 
 	it("answers gone when neither folder holds it", async () => {
-		assert.deepStrictEqual(await probe({}).result, { kind: "gone" });
+		assert.deepStrictEqual(await probe(serverHolding({})), { kind: "gone" });
 	});
 
 	it("answers gone, not at-source, when there is no destination to ask", async () => {
-		const { connection, opened } = buildConnection({ INBOX: [] });
+		const server = serverHolding({});
 
 		assert.deepStrictEqual(
-			await probePausedPlacement(connection, {
-				messageIdHeader: "<paused@example.com>",
-				sourceMailboxPath: "INBOX",
-				destinationMailboxPath: undefined,
-			}),
+			await probe(server, { destinationMailboxPath: undefined }),
 			{ kind: "gone" },
 		);
-		assert.deepStrictEqual(opened, ["INBOX"]);
+		assert.deepStrictEqual(opened(server), ["INBOX"]);
 	});
 
 	// A row with no Message-ID header cannot be asked about, and the server's
 	// silence is not an answer. `unprobeable` keeps that distinct from `gone`.
 	it("asks nothing at all when the row carries no Message-ID header", async () => {
-		const { connection, opened } = buildConnection({ INBOX: [4] });
+		const server = serverHolding({ INBOX: [4] });
 
 		assert.deepStrictEqual(
-			await probePausedPlacement(connection, {
-				messageIdHeader: undefined,
-				sourceMailboxPath: "INBOX",
-				destinationMailboxPath: "Archive",
-			}),
-			{ kind: "unprobeable" },
+			await probe(server, { messageIdHeader: undefined }),
+			{
+				kind: "unprobeable",
+			},
 		);
-		assert.deepStrictEqual(opened, []);
+		assert.deepStrictEqual(server.log, []);
 	});
 });
 
 describe("searchMailboxForHighestMessageIdUid — the probe that binds a move to a UID (#912)", () => {
-	const isMessageIdCriterion = (
-		criterion: unknown,
-	): criterion is [string, string, string] =>
-		Array.isArray(criterion) &&
-		criterion.length === 3 &&
-		typeof criterion[0] === "string" &&
-		criterion[0].toUpperCase() === "HEADER" &&
-		typeof criterion[1] === "string" &&
-		criterion[1].toLowerCase() === "message-id" &&
-		typeof criterion[2] === "string";
-
-	const buildDestination = (
-		messages: Array<{ uid: number; messageIdHeader: string }>,
-	): IImapConnection => {
-		const searchAll = () => messages.map((row) => row.uid);
-		return {
-			openBox: async () => ({}) as never,
-			search: async (criteria: unknown[]) => {
-				const criterion = criteria.find(isMessageIdCriterion);
-				if (criterion === undefined) return searchAll();
-				return messages
-					.filter((row) => row.messageIdHeader === criterion[2])
-					.map((row) => row.uid);
-			},
-		} as unknown as IImapConnection;
+	const archiveHolding = (messages: Array<[uid: number, header: string]>) => {
+		const server = new FakeImapServer();
+		server.addFolder("Archive");
+		for (const [uid, header] of messages)
+			server.deliver("Archive", eml({ "Message-ID": header }), { uid: uid });
+		return server;
 	};
 
+	const search = (server: FakeImapServer, header: string) =>
+		searchMailboxForHighestMessageIdUid(server.connection(), "Archive", header);
+
 	it("asks the folder by Message-ID rather than by an interpolated string", async () => {
-		const sent: unknown[][] = [];
-		const destination = {
-			openBox: async () => ({}) as never,
-			search: async (criteria: unknown[]) => {
-				sent.push(criteria);
-				return [];
-			},
-		} as unknown as IImapConnection;
+		const server = archiveHolding([]);
+		const hostile = '<a"b@example.com>\r\nUID 1';
 
-		await searchMailboxForHighestMessageIdUid(
-			destination,
-			"Archive",
-			'<a"b@example.com>\r\nUID 1',
-		);
+		await search(server, hostile);
 
-		assert.deepStrictEqual(sent, [
-			[["HEADER", "Message-ID", '<a"b@example.com>\r\nUID 1']],
+		assert.deepStrictEqual(server.commands("SEARCH"), [
+			`SEARCH Archive ${JSON.stringify([["HEADER", "Message-ID", hostile]])}`,
 		]);
 	});
 
 	it("answers null, not another message's UID, when the folder holds no such message", async () => {
-		const destination = buildDestination([
-			{ uid: 11, messageIdHeader: "<stranger-a@example.com>" },
-			{ uid: 12, messageIdHeader: "<stranger-b@example.com>" },
+		const server = archiveHolding([
+			[11, "<stranger-a@example.com>"],
+			[12, "<stranger-b@example.com>"],
 		]);
 
-		assert.strictEqual(
-			await searchMailboxForHighestMessageIdUid(
-				destination,
-				"Archive",
-				"<moved@example.com>",
-			),
-			null,
-		);
+		assert.strictEqual(await search(server, "<moved@example.com>"), null);
 	});
 
 	it("answers the UID of the message carrying that Message-ID", async () => {
-		const destination = buildDestination([
-			{ uid: 11, messageIdHeader: "<stranger-a@example.com>" },
-			{ uid: 12, messageIdHeader: "<moved@example.com>" },
-		]);
-
-		assert.strictEqual(
-			await searchMailboxForHighestMessageIdUid(
-				destination,
-				"Archive",
-				"<moved@example.com>",
-			),
-			12,
-		);
-	});
-
-	// Issue #1122. The folder already held an older copy of this Message-ID — a
-	// sieve `fileinto` + `keep`, a resend, an earlier copy of the same message —
-	// and the uid this probe hands back settles the row the fresh copy lives in.
-	// Answering 12 binds that row to mail this operation never touched and
-	// leaves the copy that just arrived unreachable by any later delete.
-	it("answers the newest copy's UID when the folder already held an older copy of it", async () => {
-		const destination = buildDestination([
-			{ uid: 12, messageIdHeader: "<moved@example.com>" },
-			{ uid: 40, messageIdHeader: "<moved@example.com>" },
+		const server = archiveHolding([
+			[11, "<stranger-a@example.com>"],
+			[12, "<moved@example.com>"],
 		]);
 
-		assert.strictEqual(
-			await searchMailboxForHighestMessageIdUid(
-				destination,
-				"Archive",
-				"<moved@example.com>",
-			),
-			40,
-		);
+		assert.strictEqual(await search(server, "<moved@example.com>"), 12);
 	});
 
 	// RFC 3501 does not order a SEARCH response, so the fresh copy is the
 	// highest uid returned rather than the last one returned.
 	it("takes the highest match whatever order the server lists them in", async () => {
-		const destination = buildDestination([
-			{ uid: 40, messageIdHeader: "<moved@example.com>" },
-			{ uid: 12, messageIdHeader: "<moved@example.com>" },
+		const server = archiveHolding([
+			[40, "<moved@example.com>"],
+			[12, "<moved@example.com>"],
 		]);
 
-		assert.strictEqual(
-			await searchMailboxForHighestMessageIdUid(
-				destination,
-				"Archive",
-				"<moved@example.com>",
-			),
-			40,
-		);
+		assert.strictEqual(await search(server, "<moved@example.com>"), 40);
 	});
 });