@remit/web-client 0.0.324 → 0.0.326

package/src/hooks/useUpdateAddressFlags.ts

@@ -1,152 +0,0 @@
-import {
-	addressDetailOperationsUpdateAddressMutation,
-	addressOperationsSearchAddressesQueryKey,
-} from "@remit/api-http-client/@tanstack/react-query.gen.ts";
-import type {
-	AddressOperationsSearchAddressesResponse,
-	RemitImapAddressFlagKey,
-	RemitImapAddressFlags,
-	RemitImapUpdateAddressFlagsInput,
-} from "@remit/api-http-client/types.gen.ts";
-import { useMutation, useQueryClient } from "@tanstack/react-query";
-import { useCallback } from "react";
-import { useErrorBanners } from "@/components/ui/ErrorBannerProvider";
-import { formatErrorDetail } from "@/components/ui/error-banners";
-import { reportFatalError } from "@/lib/fatal-error";
-import { senderAddressSearchQuery } from "@/lib/sender-address";
-
-interface UseUpdateAddressFlagsOptions {
-	addressId: string | undefined;
-	/** The normalizedEmail of the sender — used to key the address search cache. */
-	senderEmail: string | undefined;
-}
-
-interface MutationContext {
-	previous: AddressOperationsSearchAddressesResponse | undefined;
-}
-
-/**
- * Apply a flag update onto an existing AddressFlags object. Each key in
- * `patch` is a flag update object; each key in `clear` is removed outright,
- * after the patch, matching the server's ordering. Mirrors the server-side
- * merge semantics so the optimistic cache matches what the API returns. Typed
- * loosely (the per-key flag value types differ — boolean for most, a category
- * enum for `category`) and re-narrowed at the return.
- */
-function applyFlagPatch(
-	current: RemitImapAddressFlags | undefined,
-	patch: RemitImapUpdateAddressFlagsInput,
-	clear: readonly RemitImapAddressFlagKey[] = [],
-): RemitImapAddressFlags {
-	const next: Record<string, unknown> = { ...(current ?? {}) };
-	for (const [key, update] of Object.entries(patch)) {
-		if (update == null) {
-			delete next[key];
-		} else {
-			// Each flag is `{ value, setAt? }`. Stamp setAt so the optimistic
-			// shape matches what the server writes back.
-			next[key] = { ...update, setAt: update.setAt ?? Date.now() };
-		}
-	}
-	for (const key of clear) {
-		delete next[key];
-	}
-	return next as RemitImapAddressFlags;
-}
-
-/**
- * Optimistic PATCH for per-sender flags (VIP / Mute / Block).
- * Mirrors the `useToggleTrusted` pattern: patch the cached
- * address-search response so the toggle flips instantly, roll back on error,
- * and invalidate on settle to reconcile with the server.
- */
-export function useUpdateAddressFlags({
-	addressId,
-	senderEmail,
-}: UseUpdateAddressFlagsOptions) {
-	const queryClient = useQueryClient();
-	const { pushError } = useErrorBanners();
-
-	const addressCacheKey = addressOperationsSearchAddressesQueryKey({
-		query: senderAddressSearchQuery(senderEmail),
-	});
-
-	const { mutate, isPending } = useMutation({
-		...addressDetailOperationsUpdateAddressMutation(),
-		onMutate: async (vars): Promise<MutationContext> => {
-			const patch = vars.body.flags ?? {};
-			const clear = vars.body.clearFlags ?? [];
-			await queryClient.cancelQueries({ queryKey: addressCacheKey });
-
-			const previous =
-				queryClient.getQueryData<AddressOperationsSearchAddressesResponse>(
-					addressCacheKey,
-				);
-
-			queryClient.setQueryData<AddressOperationsSearchAddressesResponse>(
-				addressCacheKey,
-				(old) => {
-					if (!old) return old;
-					return {
-						...old,
-						items: old.items.map((addr) =>
-							addr.addressId === vars.path.addressId
-								? { ...addr, flags: applyFlagPatch(addr.flags, patch, clear) }
-								: addr,
-						),
-					};
-				},
-			);
-
-			return { previous };
-		},
-		onError: (err, _vars, context) => {
-			if (context?.previous !== undefined) {
-				queryClient.setQueryData(addressCacheKey, context.previous);
-			}
-			pushError({
-				title: "Couldn't update sender preference",
-				detail: formatErrorDetail(err),
-				error: err,
-			});
-		},
-		onSettled: () => {
-			queryClient.invalidateQueries({ queryKey: addressCacheKey });
-		},
-	});
-
-	const submit = useCallback(
-		(body: {
-			flags?: RemitImapUpdateAddressFlagsInput;
-			clearFlags?: RemitImapAddressFlagKey[];
-		}) => {
-			if (!addressId) {
-				reportFatalError(
-					new Error(
-						`Quick action fired but the sender's address record did not resolve (sender: ${senderEmail ?? "unknown"}). This should never happen — the address row exists.`,
-					),
-				);
-				return;
-			}
-			mutate({ path: { addressId }, body });
-		},
-		[addressId, mutate, senderEmail],
-	);
-
-	const updateFlags = useCallback(
-		(flags: RemitImapUpdateAddressFlagsInput) => submit({ flags }),
-		[submit],
-	);
-
-	/**
-	 * Remove flags outright. This is the removal form for every flag whatever
-	 * its value type: `category` holds an enum with no false-equivalent member,
-	 * so `{ value: false }` cannot express "no override" for it.
-	 */
-	const clearFlags = useCallback(
-		(keys: RemitImapAddressFlagKey[]) => submit({ clearFlags: keys }),
-		[submit],
-	);
-
-	return { updateFlags, clearFlags, isPending };
-}

package/src/hooks/useUpdateFilter.ts

@@ -1,56 +0,0 @@
-import {
-	filterDetailOperationsGetFilterQueryKey,
-	filterDetailOperationsUpdateFilterMutation,
-} from "@remit/api-http-client/@tanstack/react-query.gen.ts";
-import type { RemitImapUpdateFilterInput } from "@remit/api-http-client/types.gen.ts";
-import { useMutation, useQueryClient } from "@tanstack/react-query";
-import { useCallback } from "react";
-import { buildFilterListKey } from "./useFilters";
-
-/**
- * Patch an existing filter (RFC 038 D6). The server bumps `ruleChangedAt` only
- * when the body touches a predicate or action field, never on a name-only
- * rename (RFC 034 Decision 3.2) — the caller decides which fields to send. On
- * success both the account's filter list and this filter's detail are
- * invalidated so the settings page and any open editor read the saved rule.
- */
-export const useUpdateFilter = (
-	accountId: string | undefined,
-	filterId: string | undefined,
-) => {
-	const queryClient = useQueryClient();
-	const mutation = useMutation({
-		...filterDetailOperationsUpdateFilterMutation(),
-		onSuccess: () => {
-			if (!accountId) return;
-			queryClient.invalidateQueries({
-				queryKey: buildFilterListKey(accountId),
-			});
-			if (filterId) {
-				queryClient.invalidateQueries({
-					queryKey: filterDetailOperationsGetFilterQueryKey({
-						path: { accountId, filterId },
-					}),
-				});
-			}
-		},
-	});
-	const { mutate } = mutation;
-
-	const updateFilter = useCallback(
-		(body: RemitImapUpdateFilterInput) => {
-			if (!accountId || !filterId) return;
-			mutate({ path: { accountId, filterId }, body });
-		},
-		[accountId, filterId, mutate],
-	);
-
-	return {
-		updateFilter,
-		isPending: mutation.isPending,
-		isSuccess: mutation.isSuccess,
-		isError: mutation.isError,
-		error: mutation.error,
-		reset: mutation.reset,
-	};
-};

package/src/hooks/useVisualViewport.render.test.ts

@@ -1,63 +0,0 @@
-import assert from "node:assert/strict";
-import { afterEach, describe, it } from "node:test";
-import { createElement } from "react";
-import { createDomHarness, type DomHarness } from "@/test-support/dom";
-import { useVisualViewport } from "./useVisualViewport";
-
-let harness: DomHarness | undefined;
-
-function KeyboardProbe() {
-	const { isKeyboardOpen } = useVisualViewport();
-	return createElement("p", null, isKeyboardOpen ? "open" : "closed");
-}
-
-const mount = (): {
-	dom: DomHarness;
-	fullHeight: number;
-	resize: (height: number) => void;
-} => {
-	const fullHeight = window.innerHeight;
-	const viewport = Object.assign(new window.EventTarget(), {
-		height: fullHeight,
-	});
-	Object.defineProperty(window, "visualViewport", {
-		configurable: true,
-		value: viewport,
-	});
-	const dom = createDomHarness();
-	harness = dom;
-	dom.render(createElement(KeyboardProbe));
-	return {
-		dom,
-		fullHeight,
-		resize: (height) => {
-			viewport.height = height;
-			dom.dispatch(viewport, new window.Event("resize"));
-		},
-	};
-};
-
-afterEach(() => {
-	harness?.close();
-	harness = undefined;
-	Reflect.deleteProperty(window, "visualViewport");
-});
-
-describe("the software keyboard", () => {
-	it("reads as open once the visual viewport shrinks past the threshold, and closed again when it restores", () => {
-		const { dom, fullHeight, resize } = mount();
-		assert.equal(dom.text(), "closed");
-
-		resize(fullHeight - 400);
-		assert.equal(dom.text(), "open");
-
-		resize(fullHeight);
-		assert.equal(dom.text(), "closed");
-	});
-
-	it("stays closed when only the address bar collapses", () => {
-		const { dom, fullHeight, resize } = mount();
-		resize(fullHeight - 50);
-		assert.equal(dom.text(), "closed");
-	});
-});

package/src/hooks/useVisualViewport.ts

@@ -1,51 +0,0 @@
-import { useEffect, useState } from "react";
-
-interface VisualViewportState {
-	/** Current visual viewport height in pixels. */
-	viewportHeight: number;
-	/** True when the viewport has shrunk significantly (e.g. software keyboard open). */
-	isKeyboardOpen: boolean;
-}
-
-/** Minimum shrinkage (px) from window.innerHeight to consider the keyboard open. */
-const KEYBOARD_THRESHOLD = 150;
-
-/**
- * Tracks `window.visualViewport` resize events and reports whether the
- * software keyboard is likely open. SSR-safe — returns safe defaults
- * when `window` or `visualViewport` are unavailable.
- */
-export const useVisualViewport = (): VisualViewportState => {
-	const [state, setState] = useState<VisualViewportState>(() => {
-		if (typeof window === "undefined" || !window.visualViewport) {
-			return { viewportHeight: 0, isKeyboardOpen: false };
-		}
-		const h = window.visualViewport.height;
-		return {
-			viewportHeight: h,
-			isKeyboardOpen: window.innerHeight - h > KEYBOARD_THRESHOLD,
-		};
-	});
-
-	useEffect(() => {
-		if (typeof window === "undefined" || !window.visualViewport) return;
-
-		const vv = window.visualViewport;
-
-		const update = () => {
-			const h = vv.height;
-			setState({
-				viewportHeight: h,
-				isKeyboardOpen: window.innerHeight - h > KEYBOARD_THRESHOLD,
-			});
-		};
-
-		// Sync on mount
-		update();
-
-		vv.addEventListener("resize", update);
-		return () => vv.removeEventListener("resize", update);
-	}, []);
-
-	return state;
-};

package/src/hooks/useWarningsAttention.ts

@@ -1,37 +0,0 @@
-import { meOperationsListQuarantineOptions } from "@remit/api-http-client/@tanstack/react-query.gen.ts";
-import type { RemitImapQuarantineListResponse } from "@remit/api-http-client/types.gen.ts";
-import { useQuery } from "@tanstack/react-query";
-import { isUnauthenticated, softErrorStatuses } from "@/lib/error-classifier";
-
-const WARNINGS_LIST_META = softErrorStatuses(401);
-
-export function useWarningsList() {
-	return useQuery({
-		...meOperationsListQuarantineOptions(),
-		meta: WARNINGS_LIST_META,
-	});
-}
-
-export const WARNINGS_ATTENTION = {
-	listed: "Sync warnings",
-	unread: "Warnings could not be read",
-	signedOut: "Session ended",
-} as const;
-
-export function warningsAttentionOf(
-	data: RemitImapQuarantineListResponse | undefined,
-	error: unknown,
-): string | undefined {
-	if (isUnauthenticated(error)) return WARNINGS_ATTENTION.signedOut;
-	if (error) return WARNINGS_ATTENTION.unread;
-	if (!data) return undefined;
-	if ((data.entries?.length ?? 0) > 0 || (data.warnings?.length ?? 0) > 0) {
-		return WARNINGS_ATTENTION.listed;
-	}
-	return undefined;
-}
-
-export function useWarningsAttention(): string | undefined {
-	const list = useWarningsList();
-	return warningsAttentionOf(list.data, list.error);
-}

package/src/index.css

@@ -1,28 +0,0 @@
-/* remit-ui owns @import "tailwindcss", the @theme token block, :root and
-   .dark — importing its token sheet gives the web client the full design
-   system. The @custom-variant dark declaration inside tokens.css covers
-   the .dark ancestor scope used everywhere in this app. */
-@import "@remit/ui/tokens.css";
-
-/* Tailwind v4 content scanning: name every source Tailwind must inspect for
-   utility class usage. Automatic detection is rooted at the Vite root, which
-   for the distributor harness build is a throwaway directory holding only the
-   entry — so nothing here may rely on it. Both this package's own components
-   and the linked workspace UI package are declared explicitly.
-
-   Tests are excluded: they assert on class strings, and a scanned assertion
-   emits the utility it names into the shipped stylesheet. */
-@source "./**/*.{ts,tsx}";
-@source "../../ui/src/**/*.{ts,tsx}";
-@source not "./**/*.test.{ts,tsx}";
-@source not "../../ui/src/**/*.test.{ts,tsx}";
-
-/* Hide scrollbars while maintaining scroll functionality */
-.overflow-y-auto {
-  scrollbar-width: none; /* Firefox */
-  -ms-overflow-style: none; /* IE/Edge */
-}
-
-.overflow-y-auto::-webkit-scrollbar {
-  display: none; /* Chrome, Safari, Opera */
-}

package/src/index.css.test.ts

@@ -1,97 +0,0 @@
-import assert from "node:assert/strict";
-import { readdirSync, readFileSync } from "node:fs";
-import { dirname, relative, resolve, sep } from "node:path";
-import { describe, it } from "node:test";
-import { fileURLToPath } from "node:url";
-
-/**
- * Tailwind v4 roots automatic source detection at the bundler's root. The
- * distributor harness builds from a throwaway root holding only the entry, so
- * nothing this app renders is discovered automatically: every directory whose
- * components carry utility classes has to be named by an `@source` rule. When
- * one is missing the build still succeeds — it just ships a stylesheet with
- * those utilities absent, and the screens using them lay out as unstyled boxes
- * (#57: the settings slide-over covered the whole viewport).
- */
-
-const here = dirname(fileURLToPath(import.meta.url));
-const cssPath = resolve(here, "index.css");
-const css = readFileSync(cssPath, "utf8");
-
-/** Globs Tailwind scans, and globs it is told to skip. */
-const globs = (kind: "include" | "exclude"): string[] =>
-	[...css.matchAll(/@source\s+(not\s+)?"([^"]+)"/g)]
-		.filter(([, negated]) => (kind === "exclude") === Boolean(negated))
-		.map(([, , glob]) => glob);
-
-/** The directory an `@source` glob starts scanning from. */
-const sourceRoots = (): string[] =>
-	globs("include").map((glob) => {
-		const literal = glob.split("*")[0] ?? glob;
-		return resolve(dirname(cssPath), literal);
-	});
-
-/** Every directory below `root` that holds a component file. */
-const componentDirs = (root: string): string[] => {
-	const found: string[] = [];
-	const walk = (dir: string): void => {
-		const entries = readdirSync(dir, { withFileTypes: true });
-		if (entries.some((e) => e.isFile() && e.name.endsWith(".tsx"))) {
-			found.push(dir);
-		}
-		for (const entry of entries) {
-			if (entry.isDirectory() && entry.name !== "node_modules") {
-				walk(resolve(dir, entry.name));
-			}
-		}
-	};
-	walk(root);
-	return found;
-};
-
-const isCovered = (dir: string, roots: string[]): boolean =>
-	roots.some((root) => {
-		const rel = relative(root, dir);
-		return rel === "" || (!rel.startsWith("..") && !rel.startsWith(sep));
-	});
-
-describe("tailwind source coverage (#57)", () => {
-	const roots = sourceRoots();
-
-	it("declares at least one source", () => {
-		assert.ok(roots.length > 0);
-	});
-
-	it("covers every directory in this package that renders components", () => {
-		const uncovered = componentDirs(here).filter((d) => !isCovered(d, roots));
-		assert.deepEqual(
-			uncovered.map((d) => relative(here, d)),
-			[],
-		);
-	});
-
-	it("covers the linked design system package", () => {
-		const ui = resolve(here, "../../ui/src");
-		const uncovered = componentDirs(ui).filter((d) => !isCovered(d, roots));
-		assert.deepEqual(
-			uncovered.map((d) => relative(ui, d)),
-			[],
-		);
-	});
-
-	/**
-	 * A test asserting on a class string is a class string in a scanned file, so
-	 * without this every utility a test names is emitted into the shipped CSS.
-	 */
-	it("excludes test files from every source it scans", () => {
-		const uncovered = globs("include").filter(
-			(glob) =>
-				!globs("exclude").some(
-					(excluded) =>
-						excluded ===
-						glob.replace(/\/\*\*\/\*\.\{ts,tsx\}$/, "/**/*.test.{ts,tsx}"),
-				),
-		);
-		assert.deepEqual(uncovered, []);
-	});
-});

package/src/kit-stylesheets.test.ts

@@ -1,64 +0,0 @@
-import assert from "node:assert/strict";
-import { readdirSync } from "node:fs";
-import { dirname, join, relative, resolve } from "node:path";
-import { describe, it } from "node:test";
-import { fileURLToPath } from "node:url";
-import {
-	AUTH_COMPOSITIONS,
-	bundleAppGraph,
-	graphPath,
-} from "@/test-support/test-bundle-app";
-
-/**
- * A stylesheet the kit ships beside a component has to reach the app on its
- * own. #1062: `CalendarGrid`'s structural sheet was a subpath every consuming
- * app had to `@import` by hand, this one never did, and three releases drew the
- * week and day views as an unstyled stack with no events in them — while
- * Storybook, the render tests and the server render all read correctly, because
- * each of those pulled the sheet in itself.
- *
- * The kit's token sheet is the exception the app asks for by name: one theme,
- * loaded whether or not anything renders. Every other sheet is structural,
- * belongs to the component that cannot draw without it, and is checked here
- * against the bundler's own module graph walked from the app entry — so a sheet
- * no component imports fails the build instead of a deployment.
- */
-
-const here = dirname(fileURLToPath(import.meta.url));
-const kitSrc = resolve(here, "../../ui/src");
-
-/** The kit's theme sheet, which `src/index.css` imports directly. */
-const THEME_SHEET = join(kitSrc, "tokens.css");
-
-const structuralSheets = (dir: string): string[] => {
-	const found: string[] = [];
-	for (const entry of readdirSync(dir, { withFileTypes: true })) {
-		const path = join(dir, entry.name);
-		if (entry.isDirectory()) {
-			if (entry.name !== "node_modules") found.push(...structuralSheets(path));
-			continue;
-		}
-		if (entry.name.endsWith(".css") && path !== THEME_SHEET) found.push(path);
-	}
-	return found.sort();
-};
-
-describe("kit stylesheets (#1062)", () => {
-	it("walks every structural sheet @remit/ui ships from the app entry", async () => {
-		const sheets = structuralSheets(kitSrc);
-		assert.ok(
-			sheets.length > 0,
-			"the kit ships no structural sheet, so this asserts nothing — name the sheet it should follow, or drop it",
-		);
-
-		const { inputs } = await bundleAppGraph(AUTH_COMPOSITIONS.betterAuth);
-		const reached = new Set(inputs.map(graphPath));
-		const orphaned = sheets.filter((sheet) => !reached.has(sheet));
-
-		assert.deepEqual(
-			orphaned.map((sheet) => relative(kitSrc, sheet)),
-			[],
-			"a structural sheet no component imports never reaches the deployed app",
-		);
-	});
-});

package/src/lib/account-order.test.ts

@@ -1,74 +0,0 @@
-import assert from "node:assert";
-import { describe, test } from "node:test";
-import type { RemitImapAccountResponse } from "@remit/api-http-client/types.gen.ts";
-import { compareAccounts, sortAccountsByCreatedAt } from "./account-order.js";
-
-const account = (
-	overrides: Partial<RemitImapAccountResponse> &
-		Pick<RemitImapAccountResponse, "accountId" | "createdAt">,
-): RemitImapAccountResponse =>
-	({
-		accountConfigId: "cfg-1",
-		username: "user",
-		email: "user@example.com",
-		imapHost: "imap.example.com",
-		imapPort: 993,
-		imapTls: true,
-		imapStartTls: false,
-		smtpHost: "smtp.example.com",
-		smtpPort: 587,
-		smtpTls: false,
-		smtpStartTls: true,
-		smtpUsername: "user",
-		isActive: true,
-		updatedAt: overrides.createdAt,
-		...overrides,
-	}) as RemitImapAccountResponse;
-
-describe("compareAccounts", () => {
-	test("oldest createdAt sorts first", () => {
-		const a = account({ accountId: "z", createdAt: 1000 });
-		const b = account({ accountId: "a", createdAt: 2000 });
-		assert.ok(compareAccounts(a, b) < 0);
-		assert.ok(compareAccounts(b, a) > 0);
-	});
-
-	test("ties on createdAt are broken by accountId ascending", () => {
-		const a = account({ accountId: "a", createdAt: 1000 });
-		const b = account({ accountId: "b", createdAt: 1000 });
-		assert.ok(compareAccounts(a, b) < 0);
-		assert.ok(compareAccounts(b, a) > 0);
-	});
-});
-
-describe("sortAccountsByCreatedAt", () => {
-	test("returns a new array sorted oldest-first", () => {
-		const input = [
-			account({ accountId: "c", createdAt: 3000 }),
-			account({ accountId: "a", createdAt: 1000 }),
-			account({ accountId: "b", createdAt: 2000 }),
-		];
-		const sorted = sortAccountsByCreatedAt(input);
-		assert.deepStrictEqual(
-			sorted.map((a) => a.accountId),
-			["a", "b", "c"],
-		);
-		assert.deepStrictEqual(
-			input.map((a) => a.accountId),
-			["c", "a", "b"],
-		);
-	});
-
-	test("ties on createdAt sort by accountId", () => {
-		const input = [
-			account({ accountId: "z", createdAt: 1000 }),
-			account({ accountId: "a", createdAt: 1000 }),
-			account({ accountId: "m", createdAt: 1000 }),
-		];
-		const sorted = sortAccountsByCreatedAt(input);
-		assert.deepStrictEqual(
-			sorted.map((a) => a.accountId),
-			["a", "m", "z"],
-		);
-	});
-});

package/src/lib/account-order.ts

@@ -1,13 +0,0 @@
-import type { RemitImapAccountResponse } from "@remit/api-http-client/types.gen.ts";
-
-export const compareAccounts = (
-	a: RemitImapAccountResponse,
-	b: RemitImapAccountResponse,
-): number => {
-	if (a.createdAt !== b.createdAt) return a.createdAt - b.createdAt;
-	return a.accountId.localeCompare(b.accountId);
-};
-
-export const sortAccountsByCreatedAt = <T extends RemitImapAccountResponse>(
-	accounts: readonly T[],
-): T[] => [...accounts].sort(compareAccounts);

package/src/lib/adjacent-message.test.ts

@@ -1,34 +0,0 @@
-import assert from "node:assert";
-import { describe, test } from "node:test";
-import { adjacentMessageId } from "./adjacent-message.js";
-
-describe("adjacentMessageId", () => {
-	const ids = ["a", "b", "c"];
-
-	test("next returns the following id", () => {
-		assert.strictEqual(adjacentMessageId(ids, "a", "next"), "b");
-		assert.strictEqual(adjacentMessageId(ids, "b", "next"), "c");
-	});
-
-	test("previous returns the preceding id", () => {
-		assert.strictEqual(adjacentMessageId(ids, "c", "previous"), "b");
-		assert.strictEqual(adjacentMessageId(ids, "b", "previous"), "a");
-	});
-
-	test("returns null at the end of the list (no wrap)", () => {
-		assert.strictEqual(adjacentMessageId(ids, "c", "next"), null);
-	});
-
-	test("returns null at the start of the list (no wrap)", () => {
-		assert.strictEqual(adjacentMessageId(ids, "a", "previous"), null);
-	});
-
-	test("returns null when current id is not in the list", () => {
-		assert.strictEqual(adjacentMessageId(ids, "z", "next"), null);
-		assert.strictEqual(adjacentMessageId(ids, "z", "previous"), null);
-	});
-
-	test("returns null when there is no current id", () => {
-		assert.strictEqual(adjacentMessageId(ids, undefined, "next"), null);
-	});
-});