@remit/web-client 0.0.324 → 0.0.326

package/src/lib/mail-route.test.ts

@@ -1,261 +0,0 @@
-/**
- * Three contracts, each of which has already cost a regression.
- *
- * The list is read off a matched route id, never the parent /mail layout's
- * own pathname: that pathname is "/mail" on every child route, and keying off
- * it routed every mailbox through the brief pane, so the message-row anchors
- * vanished. The location's pathname is the whole address and says which list.
- *
- * The view key of a list equals the view key of anything nested under it. A
- * thread is a child route of the list it was opened from, so a key that moved
- * when the thread opened would tell `lib/search-view.ts` the reader had left
- * the view — and the query they had just typed would be re-seeded from the URL
- * and disappear the moment they opened a hit.
- *
- * And "where is the reader" is answered by the pathname, not by the matches,
- * because the matches lag a navigation by as long as the destination takes to
- * mount — both for "am I the current list" and for which view the search field
- * is searching (#808).
- */
-import assert from "node:assert/strict";
-import { describe, it } from "node:test";
-import {
-	addressQuery,
-	locationIsOnList,
-	locationOpensDetail,
-	MAIL_BRIEF_ROUTE_ID,
-	MAIL_FLAGGED_ROUTE_ID,
-	MAIL_MAILBOX_ROUTE_ID,
-	MAIL_OUTBOX_ROUTE_ID,
-	type MailRouteMatch,
-	mailboxViewKey,
-	mailListRoute,
-	mailViewKey,
-} from "./mail-route.js";
-
-/** Matches as TanStack Router reports them: outermost first, leaf last. */
-const matches = (
-	routeIds: readonly string[],
-	params?: Record<string, string>,
-): MailRouteMatch[] => [
-	{ routeId: "__root__" },
-	{ routeId: "/mail" },
-	...routeIds.map((routeId) => ({ routeId, ...(params ? { params } : {}) })),
-];
-
-const brief = matches([MAIL_BRIEF_ROUTE_ID]);
-const flagged = matches([MAIL_FLAGGED_ROUTE_ID]);
-const outbox = matches([MAIL_OUTBOX_ROUTE_ID]);
-const mailbox = matches([MAIL_MAILBOX_ROUTE_ID], { mailboxId: "inbox-1" });
-
-describe("mailListRoute", () => {
-	it("names each of the four lists", () => {
-		assert.deepEqual(mailListRoute(brief), { list: "brief" });
-		assert.deepEqual(mailListRoute(flagged), { list: "flagged" });
-		assert.deepEqual(mailListRoute(outbox), { list: "outbox" });
-		assert.deepEqual(mailListRoute(mailbox), {
-			list: "mailbox",
-			mailboxId: "inbox-1",
-		});
-		assert.deepEqual(mailListRoute(matches([MAIL_MAILBOX_ROUTE_ID])), {
-			list: "mailbox",
-			mailboxId: undefined,
-		});
-	});
-
-	it("never reads the parent /mail layout as a list", () => {
-		assert.equal(mailListRoute(matches([])), undefined);
-		assert.equal(mailListRoute([{ routeId: "__root__" }]), undefined);
-	});
-
-	it("names the list a thread was opened from, not the thread", () => {
-		assert.deepEqual(
-			mailListRoute(
-				matches([MAIL_BRIEF_ROUTE_ID, `${MAIL_BRIEF_ROUTE_ID}/$threadId`]),
-			),
-			{ list: "brief" },
-		);
-		assert.deepEqual(
-			mailListRoute(
-				matches([MAIL_MAILBOX_ROUTE_ID, `${MAIL_MAILBOX_ROUTE_ID}/$threadId`], {
-					mailboxId: "inbox-1",
-				}),
-			),
-			{ list: "mailbox", mailboxId: "inbox-1" },
-		);
-	});
-});
-
-describe("mailViewKey", () => {
-	it("gives the four lists four distinct keys", () => {
-		const keys = [
-			"/mail/brief",
-			"/mail/flagged",
-			"/mail/outbox",
-			"/mail/inbox-1",
-		].map(mailViewKey);
-		assert.equal(new Set(keys).size, 4);
-	});
-
-	it("distinguishes two mailboxes", () => {
-		assert.notEqual(
-			mailViewKey("/mail/inbox-1"),
-			mailViewKey("/mail/archive-1"),
-		);
-	});
-
-	it("is empty outside the mail shell", () => {
-		assert.equal(mailViewKey("/onboarding"), "");
-	});
-
-	it("is empty on /mail itself, which names no list", () => {
-		assert.equal(mailViewKey("/mail"), "");
-	});
-
-	it("reads the list from the address the router has committed", () => {
-		// The matches lag a navigation by as long as the destination takes to
-		// mount, and the search field follows the address: text typed once this
-		// says the next mailbox is that mailbox's query, not the previous view's.
-		assert.equal(mailViewKey("/mail/inbox-1"), mailboxViewKey("inbox-1"));
-		assert.equal(mailViewKey("/mail/brief"), MAIL_BRIEF_ROUTE_ID);
-		assert.equal(mailViewKey("/mail/flagged"), MAIL_FLAGGED_ROUTE_ID);
-		assert.equal(mailViewKey("/mail/outbox"), MAIL_OUTBOX_ROUTE_ID);
-	});
-
-	// The trap: opening a thread must not read as leaving the list, or the
-	// search field re-seeds and the typed query is gone.
-	it("gives a list and its open thread the same key", () => {
-		for (const list of [
-			"/mail/brief",
-			"/mail/flagged",
-			"/mail/outbox",
-			"/mail/inbox-1",
-		]) {
-			assert.equal(
-				mailViewKey(`${list}/th-1`),
-				mailViewKey(list),
-				`${list} changes view key when a thread opens`,
-			);
-			assert.equal(
-				mailViewKey(`${list}/th-1/msg-1`),
-				mailViewKey(list),
-				`${list} changes view key when a message expands`,
-			);
-		}
-	});
-
-	// Same trap one surface over: compose is a child of the list, and a key that
-	// moved when it opened would wipe the query the reader was mid-search on.
-	it("gives a list and its compose surface the same key", () => {
-		assert.equal(
-			mailViewKey("/mail/brief/compose"),
-			mailViewKey("/mail/brief"),
-		);
-		assert.equal(
-			mailViewKey("/mail/inbox-1/compose/draft-1"),
-			mailViewKey("/mail/inbox-1"),
-		);
-	});
-
-	it("ignores a query string and a fragment", () => {
-		assert.equal(
-			mailViewKey("/mail/brief?q=invoice"),
-			mailViewKey("/mail/brief"),
-		);
-		assert.equal(
-			mailViewKey("/mail/brief#intelligence"),
-			mailViewKey("/mail/brief"),
-		);
-	});
-});
-
-/**
- * The router commits the new location before it swaps the matches, so a list on
- * its way off the screen sees its own matches under the destination's address.
- * Anything that acts on "am I the current list" has to ask the pathname.
- */
-describe("locationIsOnList", () => {
-	it("is true on the list itself", () => {
-		assert.equal(locationIsOnList("/mail/brief", "/mail/brief"), true);
-		assert.equal(locationIsOnList("/mail/inbox-1", "/mail/inbox-1"), true);
-	});
-
-	it("is true for anything the list has open below it", () => {
-		assert.equal(
-			locationIsOnList("/mail/brief/thread-1/message-1", "/mail/brief"),
-			true,
-		);
-		assert.equal(locationIsOnList("/mail/brief/", "/mail/brief"), true);
-		assert.equal(
-			locationIsOnList("/mail/inbox-1/compose/ob-1", "/mail/inbox-1"),
-			true,
-		);
-	});
-
-	it("is false once the address names another list", () => {
-		assert.equal(locationIsOnList("/mail/inbox-1", "/mail/brief"), false);
-		assert.equal(locationIsOnList("/mail/brief", "/mail/inbox-1"), false);
-		assert.equal(locationIsOnList("/mail/archive-1", "/mail/inbox-1"), false);
-		assert.equal(locationIsOnList("/mail/flagged", "/mail/brief"), false);
-	});
-
-	it("is false outside the mail shell", () => {
-		assert.equal(locationIsOnList("/settings/accounts", "/mail/brief"), false);
-		assert.equal(locationIsOnList("/onboarding", "/mail/brief"), false);
-	});
-
-	it("compares whole segments, so a folder may be named after a list", () => {
-		assert.equal(locationIsOnList("/mail/briefing", "/mail/brief"), false);
-		assert.equal(locationIsOnList("/mail/outbox-2024", "/mail/outbox"), false);
-	});
-});
-
-/**
- * What "a thread is open" used to be asked of the query. Everything sharing the
- * single pane with the conversation reads it from the address instead.
- */
-describe("locationOpensDetail", () => {
-	it("is true for a thread and for the message inside it", () => {
-		assert.equal(locationOpensDetail("/mail/brief/thread-1"), true);
-		assert.equal(locationOpensDetail("/mail/brief/thread-1/message-1"), true);
-		assert.equal(locationOpensDetail("/mail/inbox-1/thread-1"), true);
-	});
-
-	it("is true for the compose surface, which the FAB must not sit over", () => {
-		assert.equal(locationOpensDetail("/mail/brief/compose"), true);
-		assert.equal(locationOpensDetail("/mail/inbox-1/compose"), true);
-		assert.equal(locationOpensDetail("/mail/inbox-1/compose/ob-1"), true);
-	});
-
-	it("is false on a bare list, trailing slash and query included", () => {
-		assert.equal(locationOpensDetail("/mail/brief"), false);
-		assert.equal(locationOpensDetail("/mail/brief/"), false);
-		assert.equal(locationOpensDetail("/mail/brief?q=invoice"), false);
-		assert.equal(locationOpensDetail("/mail/inbox-1"), false);
-		assert.equal(locationOpensDetail("/mail"), false);
-	});
-
-	it("is false outside the mail shell", () => {
-		assert.equal(locationOpensDetail("/settings/accounts"), false);
-		assert.equal(locationOpensDetail("/onboarding"), false);
-	});
-});
-
-describe("addressQuery", () => {
-	it("reads the query the address carries", () => {
-		assert.equal(addressQuery({ q: "invoice" }), "invoice");
-	});
-
-	it("is empty where the address carries none", () => {
-		assert.equal(addressQuery({}), "");
-		assert.equal(addressQuery({ wizard: "pick" }), "");
-	});
-
-	it("is empty for anything that is not a query string", () => {
-		// The location's search is parsed, not validated: a hand-edited `?q[]=`
-		// arrives as an array and must not reach the field as one.
-		assert.equal(addressQuery({ q: ["invoice"] }), "");
-		assert.equal(addressQuery(undefined), "");
-		assert.equal(addressQuery(null), "");
-	});
-});

package/src/lib/mail-route.ts

@@ -1,140 +0,0 @@
-/**
- * Which list the router is showing, and whether a location still sits on one.
- *
- * Every mail URL is a list plus, below it, whatever that list has open. The
- * list is a layout route, so it is named by a matched route id: the parent
- * `/mail` match carries the pathname "/mail" on every child, and the deepest
- * match is the thread. A thread opened from the brief and the brief itself
- * resolve to the same list.
- */
-
-/** A matched route, minimal shape needed to identify the list. */
-export interface MailRouteMatch {
-	routeId: string;
-	params?: Record<string, string | undefined>;
-}
-
-/** The list layout route ids. */
-export const MAIL_BRIEF_ROUTE_ID = "/mail/brief" as const;
-export const MAIL_MAILBOX_ROUTE_ID = "/mail/$mailboxId" as const;
-export const MAIL_OUTBOX_ROUTE_ID = "/mail/outbox" as const;
-export const MAIL_FLAGGED_ROUTE_ID = "/mail/flagged" as const;
-
-/**
- * The list a location is browsing. A mailbox route carries its id, which is
- * absent only in the frame before the router has resolved the param.
- */
-export type MailListRoute =
-	| { list: "brief" }
-	| { list: "flagged" }
-	| { list: "outbox" }
-	| { list: "mailbox"; mailboxId: string | undefined };
-
-/**
- * The list a caller is on, flattened to the two values a navigation to it — or
- * a decision about it — needs. Both are primitives, so a caller holding them in
- * a dependency array settles, and `list` is `undefined` outside the mail shell.
- */
-export interface BrowsedList {
-	list: MailListRoute["list"] | undefined;
-	mailboxId: string | undefined;
-}
-
-/**
- * The list layout among the matched routes, or `undefined` outside the mail
- * shell.
- *
- * Reads the shallowest list match, so anything mounted under a list — the open
- * thread, the message, the compose surface — leaves the answer alone.
- */
-export function mailListRoute(
-	matches: readonly MailRouteMatch[],
-): MailListRoute | undefined {
-	for (const match of matches) {
-		if (match.routeId === MAIL_BRIEF_ROUTE_ID) return { list: "brief" };
-		if (match.routeId === MAIL_FLAGGED_ROUTE_ID) return { list: "flagged" };
-		if (match.routeId === MAIL_OUTBOX_ROUTE_ID) return { list: "outbox" };
-		if (match.routeId === MAIL_MAILBOX_ROUTE_ID)
-			return { list: "mailbox", mailboxId: match.params?.mailboxId };
-	}
-	return undefined;
-}
-
-/**
- * Identity of the list view the shell is showing — one mailbox, the brief, the
- * flagged list, or the outbox. Opening a thread is not a view change, so the
- * key of a list and of anything nested under it are equal; `lib/search-view.ts`
- * re-seeds the search field whenever this changes, and a key that moved when a
- * message opened would wipe the query the reader had just typed.
- *
- * Read off the location's pathname, for the reason `locationIsOnList` is: the
- * router commits the address before it swaps the matches, and a route the
- * reader has not visited yet has its component to fetch before it can. The
- * search field follows the address, so text typed once the address says the
- * next mailbox belongs to that mailbox — keying this off the matches instead
- * made those keystrokes read as the previous view's leftovers and dropped them.
- *
- * The whole address, never the /mail match's own pathname, which is "/mail" on
- * every child route.
- */
-export function mailViewKey(pathname: string): string {
-	const segments = pathname.split(/[?#]/)[0].split("/").filter(Boolean);
-	if (segments[0] !== "mail") return "";
-	const list = segments[1];
-	if (!list) return "";
-	if (list === "brief") return MAIL_BRIEF_ROUTE_ID;
-	if (list === "flagged") return MAIL_FLAGGED_ROUTE_ID;
-	if (list === "outbox") return MAIL_OUTBOX_ROUTE_ID;
-	return mailboxViewKey(list);
-}
-
-/**
- * What the address says the query is.
- *
- * Read off the location's own search, never off the matched route's, for the
- * reason `mailViewKey` reads the location's pathname: the router commits the
- * whole address at once and swaps the matches afterwards. Taking the view from
- * one and the query from the other splits a single move in two — the field sees
- * the mailbox it is going to next to the query of the one it is leaving, and
- * re-seeds itself with a query the reader has already navigated away from (#47).
- */
-export function addressQuery(search: unknown): string {
-	if (typeof search !== "object" || search === null) return "";
-	const { q } = search as { q?: unknown };
-	return typeof q === "string" ? q : "";
-}
-
-/** One mailbox's view key. Two mailboxes are two views. */
-export function mailboxViewKey(mailboxId: string): string {
-	return `${MAIL_MAILBOX_ROUTE_ID}:${mailboxId}`;
-}
-
-/**
- * Whether a location sits on a list, or on something the list has open below it.
- *
- * Answers off the pathname rather than the matches on purpose. The router
- * commits the new location before it swaps the matches, so a component leaving
- * the screen sees its own matches under the destination's address for as long as
- * the destination takes to mount. `pathname` is what says where the reader is
- * going; `matches` is what says where they still are.
- *
- * `listPath` is a whole segment: a folder named `briefing` is not the brief.
- */
-export function locationIsOnList(pathname: string, listPath: string): boolean {
-	if (pathname === listPath) return true;
-	return pathname.startsWith(`${listPath}/`);
-}
-
-/**
- * Whether the address names something the list has open below it — a thread, a
- * message, the compose surface. A list on its own, and its bare reading pane,
- * do not.
- *
- * Answers what an open thread used to be asked of the query. Anything sharing
- * the single pane with the conversation reads it from here rather than keeping a
- * second opinion about what is on screen.
- */
-export function locationOpensDetail(pathname: string): boolean {
-	const segments = pathname.split(/[?#]/)[0].split("/").filter(Boolean);
-	return segments[0] === "mail" && segments.length > 2;
-}

package/src/lib/mail-search.ts

@@ -1,69 +0,0 @@
-/**
- * The search params each mail list validates.
- *
- * `q` lives on the parent `/mail` route, but a child's `validateSearch` is
- * authoritative for its own URL, so a child that omits `q` strips it: typing a
- * query on that list does nothing and the query is lost on the next navigation.
- * Every list extends the same base rather than re-declaring the param, so there
- * is one place for it to be right.
- */
-import { z } from "zod";
-
-/** What every list carries, whatever else it carries. */
-const listSearch = z.object({ q: z.string().optional() });
-
-/**
- * The brief's open thread and the message inside it are path segments
- * (`/mail/brief/<thread>/<message>`), so the query carries nothing but the
- * search. An old link's selection params are dropped here, which is what
- * "tolerated and ignored" means.
- */
-export const briefSearchSchema = listSearch.extend({});
-
-/** The same, for `/mail/flagged/<thread>/<message>`. */
-export const flaggedSearchSchema = listSearch.extend({});
-
-/**
- * The outbox's open message is a path segment
- * (`/mail/outbox/draft/<outboxMessageId>`), so the query carries nothing but
- * the search. An old link's selection param is dropped here.
- */
-export const outboxSearchSchema = listSearch.extend({});
-
-/**
- * A folder's open thread and the message inside it are path segments
- * (`/mail/<mailbox>/<thread>/<message>`), so the query carries nothing but the
- * search.
- */
-export const mailboxSearchSchema = listSearch.extend({});
-
-/**
- * `/mail` itself only redirects to the brief, so it accepts what the brief
- * accepts and hands it straight on.
- */
-export const mailIndexSearchSchema = briefSearchSchema;
-
-/** Every list's schema, by the path it validates. */
-export const mailListSearchSchemas = {
-	"/mail/brief": briefSearchSchema,
-	"/mail/flagged": flaggedSearchSchema,
-	"/mail/outbox": outboxSearchSchema,
-	"/mail/$mailboxId": mailboxSearchSchema,
-} as const;
-
-export const composeSeedSearch = z.object({ body: z.string().optional() });
-
-export type ComposeSeed = Required<z.infer<typeof composeSeedSearch>>;
-
-export const withoutComposeSeed = (
-	prev: Record<string, unknown>,
-): Record<string, unknown> => {
-	const { body: _seed, ...rest } = prev;
-	return rest;
-};
-
-export const replySearchSchemas = {
-	brief: briefSearchSchema.extend(composeSeedSearch.shape),
-	flagged: flaggedSearchSchema.extend(composeSeedSearch.shape),
-	mailbox: mailboxSearchSchema.extend(composeSeedSearch.shape),
-} as const;

package/src/lib/mailbox-sync-wait.test.ts

@@ -1,186 +0,0 @@
-/**
- * waitForMailboxSynced — the gate a dependent write (a filter, a move) holds
- * behind while a freshly-created folder is confirmed on the mail server. It
- * resolves only on `synced`, rejects distinctly on `failed` and on timeout, and
- * keeps polling while the row is still `pending` or not yet listed.
- */
-
-import assert from "node:assert/strict";
-import { describe, it } from "node:test";
-import { MailboxSyncStatus } from "@remit/domain-enums";
-import {
-	MAILBOX_SYNC_FAILED_MESSAGE,
-	MAILBOX_SYNC_TIMEOUT_MESSAGE,
-	type MailboxSyncSignal,
-	waitForMailboxSynced,
-} from "./mailbox-sync-wait.js";
-
-const row = (
-	mailboxId: string,
-	syncStatus?: MailboxSyncSignal["syncStatus"],
-	extra: Record<string, unknown> = {},
-): MailboxSyncSignal & Record<string, unknown> => ({
-	mailboxId,
-	syncStatus,
-	...extra,
-});
-
-const noDelay = () => Promise.resolve();
-
-describe("waitForMailboxSynced", () => {
-	it("resolves with the confirmed row once it reaches synced", async () => {
-		const responses = [
-			[row("mbx-1", MailboxSyncStatus.pending)],
-			[row("mbx-1", MailboxSyncStatus.pending)],
-			[row("mbx-1", MailboxSyncStatus.synced, { fullPath: "Server/Receipts" })],
-		];
-		let call = 0;
-		const result = await waitForMailboxSynced({
-			mailboxId: "mbx-1",
-			fetchMailboxes: async () => responses[call++],
-			delay: noDelay,
-		});
-		assert.equal(result.syncStatus, MailboxSyncStatus.synced);
-		assert.equal(
-			(result as Record<string, unknown>).fullPath,
-			"Server/Receipts",
-		);
-		assert.equal(call, 3);
-	});
-
-	it("keeps polling while the row is not yet listed", async () => {
-		const responses = [
-			[] as MailboxSyncSignal[],
-			[row("other", MailboxSyncStatus.synced)],
-			[row("mbx-1", MailboxSyncStatus.synced)],
-		];
-		let call = 0;
-		const result = await waitForMailboxSynced({
-			mailboxId: "mbx-1",
-			fetchMailboxes: async () => responses[call++],
-			delay: noDelay,
-		});
-		assert.equal(result.mailboxId, "mbx-1");
-		assert.equal(call, 3);
-	});
-
-	it("rejects with the failure message when the create is reported failed", async () => {
-		await assert.rejects(
-			waitForMailboxSynced({
-				mailboxId: "mbx-1",
-				fetchMailboxes: async () => [row("mbx-1", MailboxSyncStatus.failed)],
-				delay: noDelay,
-			}),
-			(error: unknown) =>
-				error instanceof Error && error.message === MAILBOX_SYNC_FAILED_MESSAGE,
-		);
-	});
-
-	it("rejects with the timeout message when the row never confirms", async () => {
-		let clock = 0;
-		let fetches = 0;
-		await assert.rejects(
-			waitForMailboxSynced({
-				mailboxId: "mbx-1",
-				fetchMailboxes: async () => {
-					fetches += 1;
-					return [row("mbx-1", MailboxSyncStatus.pending)];
-				},
-				timeoutMs: 30_000,
-				pollIntervalMs: 1_000,
-				now: () => clock,
-				delay: async (ms) => {
-					clock += ms;
-				},
-			}),
-			(error: unknown) =>
-				error instanceof Error &&
-				error.message === MAILBOX_SYNC_TIMEOUT_MESSAGE,
-		);
-		assert.ok(fetches > 1, "polls more than once before timing out");
-	});
-
-	it("does not treat failed as timeout even past the deadline", async () => {
-		let clock = 100_000;
-		await assert.rejects(
-			waitForMailboxSynced({
-				mailboxId: "mbx-1",
-				fetchMailboxes: async () => [row("mbx-1", MailboxSyncStatus.failed)],
-				timeoutMs: 1,
-				now: () => clock++,
-				delay: noDelay,
-			}),
-			(error: unknown) =>
-				error instanceof Error && error.message === MAILBOX_SYNC_FAILED_MESSAGE,
-		);
-	});
-
-	const isAbort = (error: unknown): boolean =>
-		typeof error === "object" &&
-		error !== null &&
-		(error as { name?: unknown }).name === "AbortError";
-
-	it("rejects without polling when the signal is already aborted", async () => {
-		const controller = new AbortController();
-		controller.abort();
-		let fetches = 0;
-		await assert.rejects(
-			waitForMailboxSynced({
-				mailboxId: "mbx-1",
-				signal: controller.signal,
-				fetchMailboxes: async () => {
-					fetches += 1;
-					return [row("mbx-1", MailboxSyncStatus.pending)];
-				},
-				delay: noDelay,
-			}),
-			isAbort,
-		);
-		assert.equal(fetches, 0);
-	});
-
-	it("stops polling and rejects when the signal aborts mid-wait", async () => {
-		const controller = new AbortController();
-		let fetches = 0;
-		await assert.rejects(
-			waitForMailboxSynced({
-				mailboxId: "mbx-1",
-				signal: controller.signal,
-				fetchMailboxes: async () => {
-					fetches += 1;
-					if (fetches === 2) controller.abort();
-					return [row("mbx-1", MailboxSyncStatus.pending)];
-				},
-				delay: noDelay,
-			}),
-			isAbort,
-		);
-		assert.equal(fetches, 2);
-	});
-
-	it("resolves across the real timer delay between polls", async () => {
-		const responses = [
-			[row("mbx-1", MailboxSyncStatus.pending)],
-			[row("mbx-1", MailboxSyncStatus.synced)],
-		];
-		let call = 0;
-		const result = await waitForMailboxSynced({
-			mailboxId: "mbx-1",
-			pollIntervalMs: 1,
-			fetchMailboxes: async () => responses[call++],
-		});
-		assert.equal(result.syncStatus, MailboxSyncStatus.synced);
-	});
-
-	it("aborts an in-progress real timer delay", async () => {
-		const controller = new AbortController();
-		const pending = waitForMailboxSynced({
-			mailboxId: "mbx-1",
-			signal: controller.signal,
-			pollIntervalMs: 10_000,
-			fetchMailboxes: async () => [row("mbx-1", MailboxSyncStatus.pending)],
-		});
-		setTimeout(() => controller.abort(), 5);
-		await assert.rejects(pending, isAbort);
-	});
-});