@remit/imap-worker 0.0.86 → 0.0.87

package/src/handlers/empty-trash.test.ts

@@ -1,728 +0,0 @@
-import assert from "node:assert/strict";
-import { beforeEach, describe, it } from "node:test";
-import type { RoleResolution } from "@remit/data-ports/folder-role";
-import { noopLogger } from "@remit/logger-lambda/noop-logger";
-import type { EmptyTrashEvent } from "../events.js";
-import { type EmptyTrashDeps, handleEmptyTrash } from "./empty-trash.js";
-
-interface Call {
-	method: string;
-	args: unknown[];
-}
-
-interface Connection {
-	openBox: (
-		path: string,
-		readOnly?: boolean,
-	) => Promise<{ uidvalidity: number }>;
-	search: (criteria: string[]) => Promise<number[]>;
-	deleteMessages: (uids: number[]) => Promise<void>;
-}
-
-interface LocalMessage {
-	messageId: string;
-	uid: number;
-	status: string;
-	syncStatus?: string;
-	mailboxId?: string;
-	originalMailboxId?: string;
-	originalUid?: number;
-}
-
-type TrashMailbox = { mailboxId: string; fullPath: string };
-
-interface Harness {
-	calls: Call[];
-	account: {
-		accountId: string;
-		accountConfigId: string;
-		deletedAt?: number;
-	} | null;
-	mailbox: {
-		mailboxId: string;
-		fullPath: string;
-		uidValidity: number;
-		cursorState?: string;
-	};
-	mailboxError?: Error;
-	trashResolution: RoleResolution<TrashMailbox>;
-	connection: Connection;
-	localMessages: LocalMessage[];
-	threadMessage: boolean;
-	messagesWithoutListingRow: string[];
-	/** Message ids whose placement another lane has changed under the sweep. */
-	transitionsLost: string[];
-	getConnectionCount: number;
-	disconnectCount: number;
-}
-
-let h: Harness;
-
-const record =
-	(method: string) =>
-	async (...args: unknown[]) => {
-		h.calls.push({ method, args });
-	};
-
-const buildConnection = (): Connection => ({
-	openBox: async () => ({ uidvalidity: 1 }),
-	search: async () => [10, 11],
-	deleteMessages: record(
-		"connection.deleteMessages",
-	) as Connection["deleteMessages"],
-});
-
-const deleting = (messageId: string, uid: number): LocalMessage => ({
-	messageId,
-	uid,
-	status: "deleting",
-});
-
-/** Mid-move into Trash: the folder is written, the uid is still the source's. */
-const movingIntoTrash = (messageId: string, uid: number): LocalMessage => ({
-	messageId,
-	uid,
-	status: "moving",
-	syncStatus: "pending",
-	mailboxId: "trash-mbx",
-	originalMailboxId: "inbox-mbx",
-	originalUid: uid,
-});
-
-/** The same row between a failed attempt and its redelivery. */
-const retryingIntoTrash = (messageId: string, uid: number): LocalMessage => ({
-	...movingIntoTrash(messageId, uid),
-	syncStatus: "failed",
-});
-
-/**
- * A row whose move into Trash SETTLED, on a database written before #1217
- * taught `updateUid` to clear `originalUid`. Two folders count uids
- * independently, so Trash handing back the source's own number is ordinary —
- * and the pair it leaves is indistinguishable from a move still in flight.
- */
-const settledWithStaleOriginalUid = (
-	messageId: string,
-	uid: number,
-): LocalMessage => ({
-	messageId,
-	uid,
-	status: "deleting",
-	syncStatus: "pending",
-	mailboxId: "trash-mbx",
-	originalMailboxId: "inbox-mbx",
-	originalUid: uid,
-});
-
-/** The same row after an empty marked the folder, `status` overwritten. */
-const markedMidMove = (messageId: string, uid: number): LocalMessage => ({
-	...movingIntoTrash(messageId, uid),
-	status: "deleting",
-});
-
-const fresh = (): Harness => ({
-	calls: [],
-	account: { accountId: "acc-1", accountConfigId: "cfg-1" },
-	mailbox: {
-		mailboxId: "trash-mbx",
-		fullPath: "Trash",
-		uidValidity: 1,
-		cursorState: undefined,
-	},
-	trashResolution: {
-		kind: "flagged",
-		mailbox: { mailboxId: "trash-mbx", fullPath: "Trash" },
-	},
-	connection: buildConnection(),
-	localMessages: [deleting("msg-1", 10), deleting("msg-2", 11)],
-	threadMessage: true,
-	messagesWithoutListingRow: [],
-	transitionsLost: [],
-	getConnectionCount: 0,
-	disconnectCount: 0,
-});
-
-// Empty Trash only flips `isDeleted` on the listing row, so its presence is how
-// the revert tells its own marks from a permanent delete's (which removes them
-// up front).
-const listingRow = (messageId: string) =>
-	h.messagesWithoutListingRow.includes(messageId) || !h.threadMessage
-		? null
-		: {
-				accountConfigId: "cfg-1",
-				threadMessageId: `tm-${messageId}`,
-				sentDate: 1_700_000_000_000,
-				mailboxId: "trash-mbx",
-				isRead: false,
-				isDeleted: true,
-				hasStars: false,
-				hasAttachment: false,
-			};
-
-const deps = (): EmptyTrashDeps =>
-	({
-		getClient: async () => ({
-			account: {
-				get: async (accountId: string) => {
-					h.calls.push({ method: "account.get", args: [accountId] });
-					return h.account;
-				},
-			},
-			message: {
-				listAllByMailbox: async () => h.localMessages,
-				delete: record("message.delete"),
-				update: record("message.update"),
-				transitionPlacement: async (
-					messageId: string,
-					expected: unknown,
-					next: unknown,
-				) => {
-					const won = !h.transitionsLost.includes(messageId);
-					h.calls.push({
-						method: "message.transitionPlacement",
-						args: [messageId, expected, next, won],
-					});
-					return won ? { messageId } : undefined;
-				},
-			},
-			threadMessage: {
-				findByMessageId: async (_cfg: string, messageId: string) =>
-					listingRow(messageId),
-				findAllByMessageId: async (_cfg: string, messageId: string) => {
-					const row = listingRow(messageId);
-					return row ? [row] : [];
-				},
-				delete: record("threadMessage.delete"),
-				update: record("threadMessage.update"),
-			},
-			mailbox: {
-				get: async () => {
-					if (h.mailboxError) throw h.mailboxError;
-					return h.mailbox;
-				},
-				update: record("mailbox.update"),
-			},
-			mailboxSpecialUse: {
-				resolveTrashRole: async () => h.trashResolution,
-			},
-			secrets: {},
-		}),
-		buildLifecycleDeps: () => ({}),
-		withOAuthLifecycle: async (
-			_deps: unknown,
-			_account: unknown,
-			_log: unknown,
-			cb: (credentials: unknown) => Promise<void>,
-		) => cb({}),
-		createConnectionScope: () => ({
-			getConnection: async () => {
-				h.getConnectionCount += 1;
-				return h.connection;
-			},
-			disconnect: async () => {
-				h.disconnectCount += 1;
-			},
-		}),
-	}) as unknown as EmptyTrashDeps;
-
-const event: EmptyTrashEvent = {
-	type: "EMPTY_TRASH",
-	schemaVersion: 2,
-	accountId: "acc-1",
-	trashMailboxId: "trash-mbx",
-	trashMailboxPath: "Trash",
-	trashUidValidity: 1,
-} as EmptyTrashEvent;
-
-const called = (method: string): Call[] =>
-	h.calls.filter((c) => c.method === method);
-
-// Only a transition that WON reverted anything; a lost predicate is an offered
-// write that the row refused.
-const revertedMessageIds = (): string[] =>
-	called("message.transitionPlacement")
-		.filter(
-			(c) =>
-				c.args[3] === true &&
-				(c.args[2] as { status?: string; syncStatus?: string }).status ===
-					"active" &&
-				(c.args[2] as { syncStatus?: string }).syncStatus === "synced",
-		)
-		.map((c) => c.args[0] as string);
-
-const undeletedThreadMessageIds = (): string[] =>
-	called("threadMessage.update")
-		.filter((c) => (c.args[2] as { isDeleted?: boolean }).isDeleted === false)
-		.map((c) => c.args[1] as string);
-
-describe("handleEmptyTrash", () => {
-	beforeEach(() => {
-		h = fresh();
-	});
-
-	it("expunges every server uid and both local rows for each trashed message", async () => {
-		await handleEmptyTrash(event, noopLogger, deps());
-
-		assert.deepEqual(called("connection.deleteMessages")[0]?.args, [[10, 11]]);
-		assert.deepEqual(
-			called("message.delete").map((c) => c.args[0]),
-			["msg-1", "msg-2"],
-		);
-		assert.equal(called("threadMessage.delete").length, 2);
-		assert.equal(h.disconnectCount, 1, "the scope is always disconnected");
-	});
-
-	it("keeps the local row for a uid the expunge never covered", async () => {
-		// Mail that reached Trash after the SEARCH — or a move the unordered dev
-		// queue let outrun this event — is still on the server, so deleting its
-		// rows would hide mail the user can still see in another client.
-		h.localMessages = [
-			deleting("msg-1", 10),
-			deleting("msg-2", 11),
-			{ messageId: "msg-late", uid: 12, status: "active" },
-		];
-
-		await handleEmptyTrash(event, noopLogger, deps());
-
-		assert.deepEqual(
-			called("message.delete").map((c) => c.args[0]),
-			["msg-1", "msg-2"],
-		);
-	});
-
-	it("hands every row back when another client emptied the trash first", async () => {
-		// Apple Mail got there first, so the SEARCH is empty and this expunge
-		// covers nothing. Leaving the rows `deleting` hides mail that no longer
-		// exists anywhere, with nothing left to clear the mark.
-		h.connection.search = async () => [];
-
-		await handleEmptyTrash(event, noopLogger, deps());
-
-		assert.equal(called("connection.deleteMessages").length, 0);
-		assert.equal(called("message.delete").length, 0);
-		assert.deepEqual(revertedMessageIds(), ["msg-1", "msg-2"]);
-		assert.deepEqual(undeletedThreadMessageIds(), ["tm-msg-1", "tm-msg-2"]);
-	});
-
-	it("hands back what a partial sweep left when the event is redelivered", async () => {
-		// The first attempt expunged and cleaned up msg-1, then died before
-		// msg-2. On redelivery the server has nothing left to find, and msg-2
-		// would otherwise sit marked for a deletion that will never come.
-		h.localMessages = [deleting("msg-2", 11)];
-		h.connection.search = async () => [];
-
-		await handleEmptyTrash(event, noopLogger, deps());
-
-		assert.equal(called("message.delete").length, 0);
-		assert.deepEqual(revertedMessageIds(), ["msg-2"]);
-	});
-
-	it("leaves a row whose listing rows another operation already removed", async () => {
-		// A permanent delete inside Trash removes its listing rows up front and
-		// marks the Message `deleting`. Reverting it here would resurrect a row
-		// that operation is about to remove.
-		h.localMessages = [deleting("msg-1", 10), deleting("msg-expunging", 12)];
-		h.messagesWithoutListingRow = ["msg-expunging"];
-		h.connection.search = async () => [];
-
-		await handleEmptyTrash(event, noopLogger, deps());
-
-		assert.deepEqual(revertedMessageIds(), ["msg-1"]);
-	});
-
-	it("deletes the message even when it has no thread row", async () => {
-		h.threadMessage = false;
-
-		await handleEmptyTrash(event, noopLogger, deps());
-
-		assert.equal(called("message.delete").length, 2);
-		assert.equal(called("threadMessage.delete").length, 0);
-	});
-
-	it("returns early without connecting when the account is soft-deleted", async () => {
-		h.account = {
-			accountId: "acc-1",
-			accountConfigId: "cfg-1",
-			deletedAt: Date.now(),
-		};
-
-		await handleEmptyTrash(event, noopLogger, deps());
-
-		assert.equal(h.getConnectionCount, 0);
-	});
-
-	it("throws when the account no longer exists", async () => {
-		h.account = null;
-
-		await assert.rejects(
-			handleEmptyTrash(event, noopLogger, deps()),
-			/not found/,
-		);
-	});
-
-	it("acks terminally without connecting when the Trash mailbox was deleted", async () => {
-		h.mailboxError = Object.assign(new Error("Mailbox not found: trash-mbx"), {
-			name: "NotFoundError",
-		});
-
-		await handleEmptyTrash(event, noopLogger, deps());
-
-		assert.equal(h.getConnectionCount, 0);
-		assert.equal(called("message.delete").length, 0);
-	});
-
-	it("abandons and reverts an event minted under an unknown contract", async () => {
-		const unversioned = {
-			type: "EMPTY_TRASH",
-			accountId: "acc-1",
-			trashMailboxId: "trash-mbx",
-			trashMailboxPath: "Trash",
-		} as unknown as EmptyTrashEvent;
-
-		await handleEmptyTrash(unversioned, noopLogger, deps());
-
-		assert.equal(h.getConnectionCount, 0, "no connection is ever opened");
-		assert.equal(called("connection.deleteMessages").length, 0);
-		assert.deepEqual(revertedMessageIds(), ["msg-1", "msg-2"]);
-		assert.deepEqual(undeletedThreadMessageIds(), ["tm-msg-1", "tm-msg-2"]);
-	});
-
-	it("abandons when the Trash role now names a different folder", async () => {
-		h.trashResolution = {
-			kind: "appointed",
-			mailbox: { mailboxId: "other-mbx", fullPath: "INBOX/Bak" },
-		};
-
-		await handleEmptyTrash(event, noopLogger, deps());
-
-		assert.equal(h.getConnectionCount, 0);
-		assert.equal(called("connection.deleteMessages").length, 0);
-		assert.deepEqual(revertedMessageIds(), ["msg-1", "msg-2"]);
-	});
-
-	it("abandons when the Trash role no longer rests on confirmed evidence", async () => {
-		h.trashResolution = {
-			kind: "proposed",
-			mailbox: { mailboxId: "trash-mbx", fullPath: "Trash" },
-		};
-
-		await handleEmptyTrash(event, noopLogger, deps());
-
-		assert.equal(called("connection.deleteMessages").length, 0);
-		assert.deepEqual(revertedMessageIds(), ["msg-1", "msg-2"]);
-	});
-
-	it("refuses the expunge when the served UIDVALIDITY is not the one consented to", async () => {
-		// The path was reused: a third-party client renamed Trash away and made a
-		// fresh one. Same path, different folder, and nobody consented to empty it.
-		h.connection.openBox = async () => ({ uidvalidity: 77 });
-		h.mailbox = { mailboxId: "trash-mbx", fullPath: "Trash", uidValidity: 77 };
-
-		await handleEmptyTrash(event, noopLogger, deps());
-
-		assert.equal(called("connection.deleteMessages").length, 0);
-		assert.deepEqual(revertedMessageIds(), ["msg-1", "msg-2"]);
-		assert.deepEqual(undeletedThreadMessageIds(), ["tm-msg-1", "tm-msg-2"]);
-		assert.equal(h.disconnectCount, 1);
-	});
-
-	it("reverts only the rows this empty marked, never a freshly synced one", async () => {
-		h.localMessages = [
-			deleting("msg-1", 10),
-			{ messageId: "msg-arrived", uid: 12, status: "active" },
-		];
-		h.connection.openBox = async () => ({ uidvalidity: 77 });
-		h.mailbox = { mailboxId: "trash-mbx", fullPath: "Trash", uidValidity: 77 };
-
-		await handleEmptyTrash(event, noopLogger, deps());
-
-		assert.deepEqual(revertedMessageIds(), ["msg-1"]);
-		assert.deepEqual(undeletedThreadMessageIds(), ["tm-msg-1"]);
-	});
-
-	it("reverts the marks when openBox trips a UIDVALIDITY mismatch", async () => {
-		// The event is acked and nothing re-issues it, so leaving the folder
-		// marked `deleting` hides healthy mail until the user notices.
-		h.connection.openBox = async () => ({ uidvalidity: 999 });
-
-		await handleEmptyTrash(event, noopLogger, deps());
-
-		assert.equal(
-			(called("mailbox.update")[0]?.args[2] as { cursorState?: string })
-				?.cursorState,
-			"cursor_invalid",
-		);
-		assert.equal(called("message.delete").length, 0);
-		assert.deepEqual(revertedMessageIds(), ["msg-1", "msg-2"]);
-		assert.equal(h.disconnectCount, 1);
-	});
-
-	it("reverts the marks without connecting when the cursor is rebuilding", async () => {
-		h.mailbox = {
-			mailboxId: "trash-mbx",
-			fullPath: "Trash",
-			uidValidity: 1,
-			cursorState: "rebuilding",
-		};
-
-		await handleEmptyTrash(event, noopLogger, deps());
-
-		assert.equal(h.getConnectionCount, 0);
-		assert.equal(called("message.delete").length, 0);
-		assert.deepEqual(revertedMessageIds(), ["msg-1", "msg-2"]);
-	});
-
-	it("rethrows an unclassified IMAP error so the event is retried", async () => {
-		h.connection.search = async () => {
-			throw new Error("server exploded");
-		};
-
-		await assert.rejects(
-			handleEmptyTrash(event, noopLogger, deps()),
-			/server exploded/,
-		);
-
-		assert.equal(h.disconnectCount, 1);
-	});
-});
-
-/**
- * Issue #1217. The sweep matched every row the Trash listing returned against
- * the expunged uids. A row whose move into Trash has not settled carries the
- * SOURCE folder's uid, so that match answers for whatever Trash holds at that
- * uid — a different message, whose Message and thread rows the sweep then
- * deletes while the mail it named is still sitting in the folder it never left.
- */
-describe("handleEmptyTrash and an unsettled placement", () => {
-	beforeEach(() => {
-		h = fresh();
-	});
-
-	it("deletes no row of a message whose move into Trash has not settled", async () => {
-		h.localMessages = [
-			deleting("msg-1", 10),
-			movingIntoTrash("msg-moving", 10),
-		];
-
-		await handleEmptyTrash(event, noopLogger, deps());
-
-		assert.deepEqual(
-			called("message.delete").map((c) => c.args[0]),
-			["msg-1"],
-			"uid 10 is msg-1's; the in-flight row only borrows it from the inbox",
-		);
-		assert.equal(called("threadMessage.delete").length, 1);
-	});
-
-	it("leaves a row whose move is mid-retry rather than sweeping it", async () => {
-		// `syncStatus: failed` under `status: moving` is an ordinary transient
-		// attempt about to be redelivered (`data-ports/message-settlement.ts`),
-		// not a give-up. Either way the pair is a lie while it stands, and the
-		// uid is somebody else's.
-		h.localMessages = [retryingIntoTrash("msg-retrying", 11)];
-
-		await handleEmptyTrash(event, noopLogger, deps());
-
-		assert.equal(called("message.delete").length, 0);
-		assert.equal(called("threadMessage.delete").length, 0);
-		assert.deepEqual(revertedMessageIds(), []);
-	});
-
-	it("hands back, rather than binds, a mid-move row an earlier mark rewrote", async () => {
-		// Defence in depth for the shape `emptyTrash` used to write: the mark
-		// overwrote `status` and the row reached here reading settled while its
-		// uid still belonged to the inbox. The uid is not bound — but the mark is
-		// cleared, because nothing else is coming to clear it and a row left
-		// `deleting` is hidden from every listing for good.
-		h.localMessages = [deleting("msg-1", 10), markedMidMove("msg-marked", 11)];
-
-		await handleEmptyTrash(event, noopLogger, deps());
-
-		assert.deepEqual(
-			called("message.delete").map((c) => c.args[0]),
-			["msg-1"],
-		);
-		assert.deepEqual(revertedMessageIds(), ["msg-marked"]);
-		assert.deepEqual(undeletedThreadMessageIds(), ["tm-msg-marked"]);
-	});
-
-	it("hands nothing back for a row another lane settled after the sweep read it", async () => {
-		// The hand-back is a transition off the row as this sweep read it
-		// (imap-mutations R3). PLACEMENT_MOVE_PUSH rides a standard queue that
-		// this account's FIFO group does not order, so it can settle a row
-		// mid-sweep; writing `active` + `synced` over that would declare a
-		// mutation settled that nobody confirmed, and take the listing row with
-		// it.
-		h.localMessages = [deleting("msg-1", 10), markedMidMove("msg-raced", 11)];
-		h.transitionsLost = ["msg-raced"];
-
-		await handleEmptyTrash(event, noopLogger, deps());
-
-		assert.deepEqual(
-			called("message.transitionPlacement").map((c) => c.args[0]),
-			["msg-raced"],
-			"the predicate was still offered — the row just lost it",
-		);
-		assert.deepEqual(revertedMessageIds(), []);
-		assert.deepEqual(
-			undeletedThreadMessageIds(),
-			[],
-			"a lost predicate writes nothing at all, listing rows included",
-		);
-	});
-
-	it("predicates the hand-back on the placement the sweep actually read", async () => {
-		h.localMessages = [markedMidMove("msg-marked", 11)];
-
-		await handleEmptyTrash(event, noopLogger, deps());
-
-		const [call] = called("message.transitionPlacement");
-		assert.deepEqual(call?.args[1], {
-			status: "deleting",
-			mailboxId: "trash-mbx",
-			uid: 11,
-		});
-	});
-
-	// Issue #1230. The exclusion used to run after `deleteMessages`, so a row
-	// the sweep refused had already lost its server copy: mail that no longer
-	// exists, visible in Trash, with nothing able to clear it — pressing Empty
-	// Trash again skips it too, because the server no longer lists the uid.
-	it("never expunges a uid whose row it will refuse to remove", async () => {
-		h.localMessages = [
-			deleting("msg-1", 10),
-			settledWithStaleOriginalUid("msg-stale", 11),
-		];
-
-		await handleEmptyTrash(event, noopLogger, deps());
-
-		assert.deepEqual(
-			called("connection.deleteMessages")[0]?.args,
-			[[10]],
-			"uid 11 survives on the server, because its row survives locally",
-		);
-		assert.deepEqual(
-			called("message.delete").map((c) => c.args[0]),
-			["msg-1"],
-		);
-		assert.deepEqual(
-			revertedMessageIds(),
-			["msg-stale"],
-			"and the refused row is handed back rather than left marked",
-		);
-	});
-
-	it("issues no expunge at all when every uid in the folder is refused", async () => {
-		h.localMessages = [
-			settledWithStaleOriginalUid("msg-stale-a", 10),
-			settledWithStaleOriginalUid("msg-stale-b", 11),
-		];
-
-		await handleEmptyTrash(event, noopLogger, deps());
-
-		assert.equal(called("connection.deleteMessages").length, 0);
-		assert.equal(called("message.delete").length, 0);
-	});
-
-	it("sweeps a settled row whose Trash uid matches the one it left behind", async () => {
-		// Two folders count uids independently, so a move can land on the same
-		// number it started from. `updateUid` drops `originalUid` when it settles,
-		// which is what keeps this row apart from one still mid-move — without it
-		// the sweep would expunge the message on the server and then refuse its
-		// rows, leaving mail that no longer exists hidden in Trash for good.
-		h.localMessages = [
-			{
-				messageId: "msg-collided",
-				uid: 10,
-				status: "deleting",
-				syncStatus: "pending",
-				mailboxId: "trash-mbx",
-				originalMailboxId: "inbox-mbx",
-			},
-		];
-
-		await handleEmptyTrash(event, noopLogger, deps());
-
-		assert.deepEqual(
-			called("message.delete").map((c) => c.args[0]),
-			["msg-collided"],
-		);
-		assert.deepEqual(revertedMessageIds(), []);
-	});
-
-	it("sweeps a settled row that reached Trash by a move", async () => {
-		// The confirmed pair: `updateUid` wrote Trash's own uid over the
-		// source's, leaving `originalUid` behind as history. Refusing this would
-		// leave the folder marked for a deletion that already happened.
-		h.localMessages = [
-			{
-				messageId: "msg-settled",
-				uid: 11,
-				status: "deleting",
-				syncStatus: "pending",
-				mailboxId: "trash-mbx",
-				originalMailboxId: "inbox-mbx",
-				originalUid: 42,
-			},
-		];
-
-		await handleEmptyTrash(event, noopLogger, deps());
-
-		assert.deepEqual(
-			called("message.delete").map((c) => c.args[0]),
-			["msg-settled"],
-		);
-	});
-});
-
-describe("handleEmptyTrash — the Trash path comes from its mailbox row", () => {
-	beforeEach(() => {
-		h = fresh();
-	});
-
-	const openedPaths = (): string[] => {
-		const opened: string[] = [];
-		h.connection.openBox = async (path: string) => {
-			opened.push(path);
-			return { uidvalidity: 1 };
-		};
-		return opened;
-	};
-
-	it("runs an empty enqueued with the version 2 payload that carries the folder path", async () => {
-		const opened = openedPaths();
-
-		await handleEmptyTrash(event, noopLogger, deps());
-
-		assert.deepEqual(opened, ["Trash"]);
-		assert.deepEqual(called("connection.deleteMessages")[0]?.args, [[10, 11]]);
-	});
-
-	it("runs an empty enqueued with the pathless payload", async () => {
-		const opened = openedPaths();
-		const pathless: EmptyTrashEvent = {
-			type: "EMPTY_TRASH",
-			schemaVersion: 3,
-			eventId: "evt-1",
-			timestamp: 1,
-			accountId: "acc-1",
-			trashMailboxId: "trash-mbx",
-			trashUidValidity: 1,
-		};
-
-		await handleEmptyTrash(pathless, noopLogger, deps());
-
-		assert.deepEqual(opened, ["Trash"]);
-		assert.deepEqual(called("connection.deleteMessages")[0]?.args, [[10, 11]]);
-	});
-
-	it("empties Trash under its current path when the folder was renamed after enqueue", async () => {
-		const opened = openedPaths();
-		h.mailbox = { ...h.mailbox, fullPath: "Deleted Items" };
-
-		await handleEmptyTrash(event, noopLogger, deps());
-
-		assert.deepEqual(opened, ["Deleted Items"]);
-		assert.deepEqual(called("connection.deleteMessages")[0]?.args, [[10, 11]]);
-	});
-});