@remit/web-client 0.0.200 → 0.0.201

package/src/components/mail/SelectionWizardHost.job-status.test.ts

@@ -1,50 +0,0 @@
-import assert from "node:assert/strict";
-import { readFileSync } from "node:fs";
-import { dirname, resolve } from "node:path";
-import { describe, it } from "node:test";
-import { fileURLToPath } from "node:url";
-
-/**
- * A back-apply whose status could not be read is still running on the server
- * (#526), so the run screen's affordance looks at that job again rather than
- * queuing a second pass over the same mail.
- *
- * The host wires routing, history and several data hooks together, so — as with
- * this package's other component-level rules (see `SelectionWizardHost.run-exit.test.ts`)
- * — the wiring is read off the source. What it decides is proven where it can be
- * run: `./organize-run-state.test.ts` for the ending the screen shows, and
- * `../../hooks/useOrganizeJob.render.test.ts` for the poll it re-issues.
- */
-
-const here = dirname(fileURLToPath(import.meta.url));
-const source = readFileSync(resolve(here, "SelectionWizardHost.tsx"), "utf8");
-
-const retryBody = (): string => {
-	const body = source.match(
-		/const retry = \(\): void => \{([\s\S]*?)\n\t\};/,
-	)?.[1];
-	assert.ok(body, "the run screen has no retry");
-	return body;
-};
-
-describe("retrying from an unknown status", () => {
-	it("looks at the job it already has", () => {
-		const body = retryBody();
-		const unknown = body.indexOf('run.state === "statusUnknown"');
-		assert.ok(unknown >= 0, "retry does not recognise an unknown status");
-		assert.match(body.slice(unknown), /organizeJob\.refreshStatus\(\)/);
-	});
-
-	it("decides that before it can reach anything that queues a second job", () => {
-		const body = retryBody();
-		assert.ok(
-			body.indexOf('run.state === "statusUnknown"') < body.indexOf("startJob("),
-			"a second back-apply can be queued over a job that is still running",
-		);
-	});
-
-	it("reads what the job is doing rather than that something failed", () => {
-		assert.doesNotMatch(source, /organizeJob\.isError/);
-		assert.match(source, /organizeRunState\(\{/);
-	});
-});

package/src/components/mail/SelectionWizardHost.run-exit.test.ts

@@ -1,83 +0,0 @@
-import assert from "node:assert/strict";
-import { readFileSync } from "node:fs";
-import { dirname, resolve } from "node:path";
-import { describe, it } from "node:test";
-import { fileURLToPath } from "node:url";
-
-/**
- * Every way off the run screen is the same movement, and none of them ends the
- * run the screen promised would keep going (#521). Ending it is a control of
- * its own, and whichever way the user leaves, the run reports where it got to.
- *
- * The host wires routing, history and several data hooks together, so — as with
- * this package's other component-level rules (see `MessageList.selection.test.ts`)
- * — the wiring is read off the source. What the exits then do is proven where it
- * can be run: `../../hooks/escalated-run-lifetime.render.test.ts` for the run's
- * lifetime, and `@remit/ui`'s `selection-wizard.render.test.ts` for the controls.
- */
-
-const here = dirname(fileURLToPath(import.meta.url));
-const source = readFileSync(resolve(here, "SelectionWizardHost.tsx"), "utf8");
-
-describe("leaving the run screen", () => {
-	it("routes the header's X and back arrow through the same dismiss the footer uses", () => {
-		assert.match(source, /onExit=\{current === "run" \? dismiss : cancel\}/);
-	});
-
-	it("routes hardware back through that dismiss too", () => {
-		assert.match(
-			source,
-			/if \(action !== "BACK"\) return false;\s*\n\s*dismiss\(\);/,
-		);
-	});
-
-	it("leaves the run alone on the way out", () => {
-		const dismissBody = source.match(
-			/const dismiss = useCallback\(\(\) => \{([\s\S]*?)\}, \[/,
-		)?.[1];
-		assert.ok(dismissBody, "the run screen has no dismiss");
-		assert.doesNotMatch(dismissBody, /stop/i);
-	});
-});
-
-describe("ending the run", () => {
-	it("is a control of its own, offered only while a run is in flight", () => {
-		assert.match(source, /onCancelRun: runInFlight \? stopRun : undefined/);
-		assert.match(
-			source,
-			/const runInFlight =\s*bulkRun !== undefined &&\s*bulkRun\.outcome === undefined &&\s*bulkRun\.failureReason === undefined;/,
-		);
-	});
-
-	it("stops whichever runner is paging — the list's or the wizard's own", () => {
-		const stopBody = source.match(
-			/const stopRun = useCallback\(\(\) => \{([\s\S]*?)\}, \[/,
-		)?.[1];
-		assert.ok(stopBody, "there is no way to stop a run");
-		assert.match(stopBody, /escalated\.stop\(\)/);
-		assert.match(stopBody, /stopBulk\(\)/);
-	});
-});
-
-/**
- * Both runners report, not just the predicate one: a ticked select-all runs on
- * the wizard's own runner, and the same screen makes the same promise over it.
- */
-describe("stating how a run ended once the screen is gone", () => {
-	it("marks the walk-away where it happens rather than reading it back off the URL", () => {
-		assert.match(source, /walkedAway\.current = true;\s*\n\s*closeWizard\(/);
-	});
-
-	for (const runner of ["runBulk", "runEscalated"]) {
-		it(`reports the run ${runner} drove`, () => {
-			const body =
-				source.match(
-					new RegExp(
-						`const ${runner} = useCallback\\(([\\s\\S]*?)\\n\\t\\}, \\[`,
-					),
-				)?.[1] ?? "";
-			assert.match(body, /if \(walkedAway\.current\)/);
-			assert.match(body, /onRunEnded\?\.\(action\.kind, /);
-		});
-	}
-});

package/src/components/mail/make-filter-entry.test.ts

@@ -1,63 +0,0 @@
-import assert from "node:assert/strict";
-import { readdirSync, readFileSync } from "node:fs";
-import { dirname, resolve } from "node:path";
-import { describe, it } from "node:test";
-import { fileURLToPath } from "node:url";
-
-/**
- * "Make this a filter" is the wizard's second entry (#477 clause 1.8), and the
- * step it pushes has to be answered on the view that offered it.
- *
- * The affordance is built once by `MailListHeader` and handed down as
- * `makeFilterSlot`, so any surface can put it on screen — but the wizard is
- * mounted per surface, and a surface that renders the row without one leaves
- * the press on a URL nothing answers. That is the dead button clause 1.7 exists
- * to prevent, and it is invisible in review: the row and the host are two files
- * apart, and the affordance keeps working everywhere else.
- *
- * So the pairing is the assertion. A surface that renders the row names where
- * its step is answered, and that file is checked to mount a host. A fourth
- * surface fails here until it does the same.
- */
-
-const here = dirname(fileURLToPath(import.meta.url));
-
-/** How the chrome's slot is read — `MailListHeader` writes it, everyone else reads it. */
-const RENDERS_SLOT = /\.makeFilterSlot/;
-const MOUNTS_HOST = /<SelectionWizardHost/;
-
-/** Surface that renders the affordance → the file whose composition answers the step. */
-const ANSWERED_BY: Record<string, string> = {
-	"DailyBrief.tsx": "DailyBrief.tsx",
-	"MessageList.tsx": "MessageList.tsx",
-	// The starred list's bar is shared with the brief's, so its host is mounted
-	// by the view rather than beside the bar.
-	"ThreadListInteraction.tsx": "FlaggedList.tsx",
-};
-
-const read = (file: string): string =>
-	readFileSync(resolve(here, file), "utf8");
-
-const surfacesRenderingTheAffordance = (): string[] =>
-	readdirSync(here)
-		.filter((file) => file.endsWith(".tsx"))
-		.filter((file) => RENDERS_SLOT.test(read(file)));
-
-describe("the make-filter entry", () => {
-	it("is rendered only by surfaces that say where its step is answered", () => {
-		assert.deepEqual(
-			surfacesRenderingTheAffordance().sort(),
-			Object.keys(ANSWERED_BY).sort(),
-		);
-	});
-
-	it("is answered by a mounted wizard on every one of them", () => {
-		for (const [surface, host] of Object.entries(ANSWERED_BY)) {
-			assert.match(
-				read(host),
-				MOUNTS_HOST,
-				`${surface} is answered by ${host}`,
-			);
-		}
-	});
-});

package/src/components/mail/selection-verbs.test.ts

@@ -1,285 +0,0 @@
-import assert from "node:assert/strict";
-import { readdirSync, readFileSync } from "node:fs";
-import { dirname, resolve } from "node:path";
-import { describe, it } from "node:test";
-import { fileURLToPath } from "node:url";
-
-/**
- * One invariant, over every surface: a verb aimed at a selection reaches the
- * mail server only through the wizard, whose review screen names what it covers
- * (#477 1.4 and its FAQ, #508).
- *
- * There are two ways to ask for one — the selection bar and the keyboard — and
- * three surfaces that raise a bar. Checked here rather than by a spec per
- * surface, because what has repeatedly gone wrong is not a broken route but a
- * missing one: a fourth surface, or a sixth verb, quietly keeping a direct
- * handler while every other route was moved. That is invisible in review and
- * indistinguishable from correct until someone selects 3,412 messages and
- * presses a key.
- *
- * These read the source rather than rendering, for the reason
- * `MessageList.selection.test.ts` gives: these components wire the DOM, the
- * router and several data hooks together, and the rule is about which function
- * a prop names.
- */
-
-const here = dirname(fileURLToPath(import.meta.url));
-const read = (file: string): string =>
-	readFileSync(resolve(here, file), "utf8");
-
-/** Every verb the bar can carry — the props `SelectionTopBar` accepts. */
-const BAR_VERB_PROPS = [
-	"onDelete",
-	"onMove",
-	"onOrganize",
-	"onJunk",
-	"onMarkRead",
-] as const;
-
-/**
- * How a surface is allowed to answer a verb prop. `startWizard`/`wizard.start`/
- * `startSelectionVerb` all open the wizard on the verb; `organizeSelection` and
- * `startFromSearch` open it on the search entry, which is the same wizard
- * reached by its other door (#477 1.8). Anything else is a route around the
- * review screen.
- */
-const OPENS_THE_WIZARD =
-	/^(startWizard|wizard\.start|startSelectionVerb|organizeSelection|startFromSearch)\b/;
-
-const surfaceFiles = (): string[] =>
-	readdirSync(here)
-		.filter((file) => file.endsWith(".tsx"))
-		.filter((file) => /<SelectionTopBar/.test(read(file)));
-
-/**
- * The bar element as written, to its own closing tag. Found by matching tags
- * rather than by stopping at the first `/>` on a line: a nested self-closing
- * element matches that too, so the scan would end early and silently skip
- * whatever a surface happened to write after it.
- */
-const barMarkup = (source: string): string => {
-	const start = source.indexOf("<SelectionTopBar");
-	if (start === -1) return "";
-	let depth = 0;
-	for (let i = start; i < source.length; i++) {
-		if (source[i] === "<" && /[A-Za-z]/.test(source[i + 1] ?? "")) depth++;
-		else if (source.startsWith("/>", i)) {
-			depth--;
-			if (depth === 0) return source.slice(start, i + 2);
-			i++;
-		} else if (source.startsWith("</", i)) depth--;
-	}
-	return "";
-};
-
-/** The body of `const <name> = useCallback(…)`, or of an inline arrow value. */
-const bracedFrom = (source: string, open: number): string => {
-	let depth = 0;
-	for (let i = open; i < source.length; i++) {
-		if (source[i] === "{") depth++;
-		else if (source[i] === "}") {
-			depth--;
-			if (depth === 0) return source.slice(open, i + 1);
-		}
-	}
-	return source.slice(open);
-};
-
-const namedCallbackBody = (source: string, name: string): string => {
-	const at = source.indexOf(`const ${name} = useCallback(`);
-	if (at === -1) return "";
-	const open = source.indexOf("{", at);
-	return open === -1 ? "" : bracedFrom(source, open);
-};
-
-/** The triage handlers a pane wires into the keyboard layer, by name. */
-const declaredHandlers = (source: string): string[] => {
-	const at = source.indexOf("handlers: {");
-	if (at === -1) return [];
-	return Array.from(
-		bracedFrom(source, source.indexOf("{", at)).matchAll(/\n\t{3}(\w+): /g),
-		(match) => match[1],
-	);
-};
-
-/**
- * What a pane does for one keyboard verb: the handler it wired into the triage
- * layer, plus any callback that handler defers to for its target. A handler
- * that offers the list the press and then falls through to a helper picking the
- * selection is exactly as wrong as one picking it inline, so both are read.
- */
-const paneVerbBody = (source: string, handler: string): string => {
-	const at = source.search(new RegExp(`\\n\\t{3}${handler}: `));
-	if (at === -1) return "";
-	const value =
-		source.slice(at).match(new RegExp(`${handler}: (.*)`))?.[1] ?? "";
-	const body = value.startsWith("(")
-		? bracedFrom(source, source.indexOf("{", at))
-		: namedCallbackBody(source, value.replace(/[,\s].*$/, ""));
-	const helpers = Array.from(body.matchAll(/\b(\w+MessageIds)\(\)/g), (match) =>
-		namedCallbackBody(source, match[1]),
-	);
-	return [body, ...helpers].join("\n");
-};
-
-/**
- * What a verb prop is wired to, with the whitespace, the guard a verb is
- * offered behind, and the arrow taken off — what is left is the call itself.
- */
-const verbHandler = (bar: string, prop: string): string | undefined => {
-	const assigned = bar.match(new RegExp(`${prop}=\\{([\\s\\S]*?)\\}\\n`))?.[1];
-	if (assigned === undefined) return undefined;
-	return assigned
-		.replace(/\s+/g, " ")
-		.replace(/^.*? \? /, "")
-		.replace(/ : undefined$/, "")
-		.replace(/^\(\)\s*=>\s*/, "")
-		.trim();
-};
-
-describe("every verb on a selection bar opens the wizard", () => {
-	it("covers the surfaces that raise one", () => {
-		assert.deepEqual(surfaceFiles().sort(), [
-			"DailyBrief.tsx",
-			"MessageList.tsx",
-			"ThreadListInteraction.tsx",
-		]);
-	});
-
-	for (const file of surfaceFiles()) {
-		it(`${file} routes each verb it offers through the wizard`, () => {
-			const bar = barMarkup(read(file));
-			assert.notEqual(bar, "", `${file} renders a SelectionTopBar`);
-			const offered = BAR_VERB_PROPS.filter((prop) => bar.includes(`${prop}=`));
-			assert.ok(offered.length > 0, `${file} offers at least one verb`);
-			for (const prop of offered) {
-				const handler = verbHandler(bar, prop);
-				assert.ok(handler, `${file}'s ${prop} is wired to something`);
-				assert.match(
-					handler,
-					OPENS_THE_WIZARD,
-					`${file}'s ${prop} must open the wizard, not run the verb`,
-				);
-			}
-		});
-	}
-});
-
-/**
- * The keyboard's half. A list offers the pane one seam — `requestVerb` — and
- * claims the press whenever it has a selection, so a pane's own handler only
- * ever sees a verb aimed at the bare cursor. A pane that runs a verb over
- * `selectedIds` without offering the list the press first is the hole this
- * binds shut.
- */
-describe("every keyboard verb over a selection goes through the list", () => {
-	const LISTS = ["MessageList.tsx", "ThreadListInteraction.tsx"];
-	const PANES = ["MailboxPane.tsx", "BriefPane.tsx", "FlaggedPane.tsx"];
-	/**
-	 * The triage handlers that can be aimed at a selection, each with the wizard
-	 * verb it has to route to. Star is a knowing carve-out: it is not one of the
-	 * five verbs the wizard walks, it is not on `SelectionTopBar`, and it sets a
-	 * flag on mail already in front of the user that the same key unsets — so
-	 * there is nothing for a review screen to name. `LabelApplyTrigger` applies
-	 * labels to the selection directly for the same reason, and because labels
-	 * are out of scope for the epic that put the wizard in front of the rest
-	 * (#477 §6). Mute and block are per-sender and reach no selection at all.
-	 */
-	const PANE_VERB_ROUTE: Record<string, string> = {
-		delete: "delete",
-		toggleRead: "markRead",
-		markJunk: "junk",
-	};
-
-	/**
-	 * How a list may answer a verb aimed at the bare cursor: with the shared
-	 * decision, whose rules are in `../../lib/list-verb-request.test.ts`, or
-	 * inline. Either way only Delete is the list's.
-	 */
-	const CURSOR_DELETE_ONLY = /listVerbRequest\(\{|if \(verb !== "delete"/;
-
-	for (const file of LISTS) {
-		it(`${file} publishes the one seam and claims a selection`, () => {
-			const source = read(file);
-			assert.match(source, /requestVerb,/);
-			assert.match(
-				source,
-				CURSOR_DELETE_ONLY,
-				"only delete is the list's over a bare cursor",
-			);
-			assert.doesNotMatch(
-				source,
-				/requestDelete:/,
-				"one seam, so no verb can have a second one",
-			);
-		});
-
-		it(`${file} stands its keyboard down while the wizard is up`, () => {
-			assert.match(
-				read(file),
-				/blocksKeyboard: confirmOpen \|\| wizard/,
-				"a shortcut must not act behind the screen already asking",
-			);
-		});
-	}
-
-	// Only the handlers a pane really declares get a case, so a pane that never
-	// wired one cannot pass by skipping — the roll-call below is what notices a
-	// handler nobody routed.
-	for (const file of PANES) {
-		for (const handler of declaredHandlers(read(file))) {
-			const verb = PANE_VERB_ROUTE[handler];
-			if (verb === undefined) continue;
-
-			it(`${file}'s ${handler} offers the list ${verb} first`, () => {
-				const body = paneVerbBody(read(file), handler);
-				assert.notEqual(body, "", `${handler}'s body was found`);
-				assert.match(
-					body,
-					new RegExp(`requestVerb\\("${verb}"\\)`),
-					`${handler} must hand ${verb} to the list before running it`,
-				);
-			});
-
-			it(`${file}'s ${handler} aims at the cursor once the list declines`, () => {
-				const body = paneVerbBody(read(file), handler);
-				assert.notEqual(body, "", `${handler}'s body was found`);
-				// The list declines only when its commands are gone, which an
-				// escalated selection outlives — so a fallback that could still see
-				// a selection would run the verb over a stale snapshot of rows that
-				// are no longer loaded, with no wizard in front of it.
-				assert.doesNotMatch(
-					body,
-					/selectedIds/i,
-					`${handler} falls back to the focused row, never to a selection`,
-				);
-			});
-		}
-	}
-
-	it("checks the routed handler on every pane that declares one", () => {
-		// The roll-call. Each pair above is generated from what a pane declares, so
-		// this is what says the generation reached everything it should have — a
-		// handler that stops being declared, or a body the reader stops finding,
-		// shows up here as a missing pair rather than as a case that passed by
-		// doing nothing.
-		const checked = PANES.flatMap((file) =>
-			declaredHandlers(read(file))
-				.filter((handler) => handler in PANE_VERB_ROUTE)
-				.map((handler) => `${file}:${handler}`),
-		);
-		assert.deepEqual(checked.sort(), [
-			"BriefPane.tsx:delete",
-			"BriefPane.tsx:toggleRead",
-			"FlaggedPane.tsx:delete",
-			"FlaggedPane.tsx:toggleRead",
-			"MailboxPane.tsx:delete",
-			"MailboxPane.tsx:markJunk",
-			"MailboxPane.tsx:toggleRead",
-		]);
-		for (const entry of checked) {
-			const [file, handler] = entry.split(":");
-			assert.notEqual(paneVerbBody(read(file), handler), "", entry);
-		}
-	});
-});