@serverkgg/bridge 0.1.2 → 0.1.3

package/src/runtime/bridgeServer.test.ts

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, describe, expect, test } from "bun:test";
+import { afterEach, beforeEach, describe, expect, it } from "bun:test";
 import { mkdtemp, rm } from "node:fs/promises";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
@@ -162,7 +162,7 @@ describe("pending dispatch", () => {
 		rejected: null,
 	};
 
-	test("answers pendingList with what the driver lists", async () => {
+	it("answers pendingList with what the driver lists", async () => {
 		const server = createServer({
 			...baseOptions(socket),
 			onPendingList: async () => listed,
@@ -183,7 +183,7 @@ describe("pending dispatch", () => {
 		});
 	});
 
-	test("answers pendingList with the unsupported sentinel when the driver declares no pending module", async () => {
+	it("answers pendingList with the unsupported sentinel when the driver declares no pending module", async () => {
 		const server = createServer({
 			...baseOptions(socket),
 			onPendingList: async () => ({
@@ -216,7 +216,7 @@ describe("pending dispatch", () => {
 		});
 	});
 
-	test("passes the uploaded path to pendingAdd and answers with the refreshed list", async () => {
+	it("passes the uploaded path to pendingAdd and answers with the refreshed list", async () => {
 		const received: string[] = [];
 
 		const server = createServer({
@@ -247,7 +247,7 @@ describe("pending dispatch", () => {
 		});
 	});
 
-	test("refuses a pendingAdd carrying no path", async () => {
+	it("refuses a pendingAdd carrying no path", async () => {
 		const server = createServer({
 			...baseOptions(socket),
 			onPendingAdd: unreachable("pendingAdd"),
@@ -268,7 +268,7 @@ describe("pending dispatch", () => {
 		});
 	});
 
-	test("answers pendingAdd with the rejection the driver reported beside the refreshed list", async () => {
+	it("answers pendingAdd with the rejection the driver reported beside the refreshed list", async () => {
 		const rejected: BridgePendingResult = {
 			...listed,
 			rejected: {
@@ -299,7 +299,7 @@ describe("pending dispatch", () => {
 		});
 	});
 
-	test("names an unknown method so an older agent can degrade", async () => {
+	it("names an unknown method so an older agent can degrade", async () => {
 		const server = createServer(baseOptions(socket));
 
 		const response = await call(socket, {
@@ -367,7 +367,7 @@ describe("setup dispatch", () => {
 		runtime,
 	};
 
-	test("answers setupStatus with what the driver reported", async () => {
+	it("answers setupStatus with what the driver reported", async () => {
 		const server = createServer({
 			...baseOptions(socket),
 			onSetupStatus: () => reported,
@@ -388,7 +388,7 @@ describe("setup dispatch", () => {
 		});
 	});
 
-	test("answers setupStatus with the unsupported sentinel when the driver declares no setup", async () => {
+	it("answers setupStatus with the unsupported sentinel when the driver declares no setup", async () => {
 		const server = createServer({
 			...baseOptions(socket),
 			onSetupStatus: () => unsupportedSetup(),
@@ -412,7 +412,7 @@ describe("setup dispatch", () => {
 		});
 	});
 
-	test("passes the step, action and args of a setupSubmit to the driver", async () => {
+	it("passes the step, action and args of a setupSubmit to the driver", async () => {
 		const received: {
 			step: string;
 			action: string;
@@ -465,7 +465,7 @@ describe("setup dispatch", () => {
 		});
 	});
 
-	test("refuses a setupSubmit carrying no step id", async () => {
+	it("refuses a setupSubmit carrying no step id", async () => {
 		const server = createServer({
 			...baseOptions(socket),
 			onSetupSubmit: unreachable("setupSubmit"),
@@ -487,7 +487,7 @@ describe("setup dispatch", () => {
 		});
 	});
 
-	test("answers a rejected setupSubmit with a failed prompt instead of a failed call", async () => {
+	it("answers a rejected setupSubmit with a failed prompt instead of a failed call", async () => {
 		const server = createServer({
 			...baseOptions(socket),
 			onSetupSubmit: async () => ({
@@ -538,7 +538,7 @@ describe("setup dispatch", () => {
 		});
 	});
 
-	test("answers setupSubmit with the unsupported sentinel when the driver implements no submit", async () => {
+	it("answers setupSubmit with the unsupported sentinel when the driver implements no submit", async () => {
 		const server = createServer({
 			...baseOptions(socket),
 			onSetupSubmit: async () => ({
@@ -605,7 +605,7 @@ describe("error serialisation", () => {
 		return response;
 	};
 
-	test("carries the localized text of a user error beside the message", async () => {
+	it("carries the localized text of a user error beside the message", async () => {
 		expect(
 			await thrown(
 				new BridgeUserError({
@@ -624,7 +624,7 @@ describe("error serialisation", () => {
 		});
 	});
 
-	test("carries the failure code of a failure error and no text", async () => {
+	it("carries the failure code of a failure error and no text", async () => {
 		expect(await thrown(new BridgeFailureError(BridgeFailureCode.CatalogRestricted, "the author blocked it"))).toEqual({
 			id: 1,
 			ok: false,
@@ -633,7 +633,7 @@ describe("error serialisation", () => {
 		});
 	});
 
-	test("marks a runtime refusal unchanged and carries its text", async () => {
+	it("marks a runtime refusal unchanged and carries its text", async () => {
 		expect(
 			await thrown(
 				new BridgeRefusalError('module "worlds" cannot run "delete" right now', {
@@ -653,7 +653,7 @@ describe("error serialisation", () => {
 		});
 	});
 
-	test("marks a runtime refusal without text unchanged and sends no text", async () => {
+	it("marks a runtime refusal without text unchanged and sends no text", async () => {
 		expect(await thrown(new BridgeRefusalError('no module named "worlds"'))).toEqual({
 			id: 1,
 			ok: false,
@@ -662,7 +662,7 @@ describe("error serialisation", () => {
 		});
 	});
 
-	test("carries the message alone for an ordinary error", async () => {
+	it("carries the message alone for an ordinary error", async () => {
 		expect(await thrown(new Error("the file could not be read"))).toEqual({
 			id: 1,
 			ok: false,
@@ -682,7 +682,7 @@ describe("backup ownership dispatch", () => {
 			recursive: true,
 		});
 	});
-	test("carries the lease token through renew and release frames", async () => {
+	it("carries the lease token through renew and release frames", async () => {
 		const socket = join(dir, "bridge.sock");
 		const renewals: (string | undefined)[] = [];
 		const releases: (string | undefined)[] = [];
@@ -746,7 +746,7 @@ describe("error replies", () => {
 		});
 	});
 
-	test("hide a secret value from the error and the customer text", async () => {
+	it("hide a secret value from the error and the customer text", async () => {
 		const server = createServer({
 			...baseOptions(socket),
 			redact: (value) => value.split(SECRET).join("••••"),
@@ -830,7 +830,7 @@ describe("failure logging", () => {
 		return lines;
 	};
 
-	test("logs a refused action as a refusal with its message and no stack", async () => {
+	it("logs a refused action as a refusal with its message and no stack", async () => {
 		expect(
 			await logged(
 				new BridgeUserError({
@@ -849,7 +849,7 @@ describe("failure logging", () => {
 		]);
 	});
 
-	test("logs an ordinary error as a failure with its stack", async () => {
+	it("logs an ordinary error as a failure with its stack", async () => {
 		const [line] = await logged(new Error("the marker could not be written"));
 
 		expect(line?.message).toBe("bridge call failed");
@@ -857,7 +857,7 @@ describe("failure logging", () => {
 		expect(String(line?.fields.stack)).toContain("the marker could not be written");
 	});
 
-	test("logs a refusal with the cause the driver attached", async () => {
+	it("logs a refusal with the cause the driver attached", async () => {
 		expect(
 			await logged(
 				new BridgeUserError(
@@ -907,7 +907,7 @@ describe("activity on the answer frame", () => {
 		subject: "Steve",
 	};
 
-	test("carries the entries an invoke reported beside its result", async () => {
+	it("carries the entries an invoke reported beside its result", async () => {
 		const server = createServer({
 			...baseOptions(socket),
 			onInvoke: async () => ({
@@ -941,7 +941,7 @@ describe("activity on the answer frame", () => {
 		});
 	});
 
-	test("carries the entries a setup submit reported beside its result", async () => {
+	it("carries the entries a setup submit reported beside its result", async () => {
 		const server = createServer({
 			...baseOptions(socket),
 			onSetupSubmit: async () => ({
@@ -970,7 +970,7 @@ describe("activity on the answer frame", () => {
 		});
 	});
 
-	test("carries an empty list when nothing was reported, so the platform knows the runtime reports", async () => {
+	it("carries an empty list when nothing was reported, so the platform knows the runtime reports", async () => {
 		const server = createServer({
 			...baseOptions(socket),
 			onInvoke: async () => ({
@@ -997,7 +997,7 @@ describe("activity on the answer frame", () => {
 		});
 	});
 
-	test("leaves the field off the answers of methods that report no activity", async () => {
+	it("leaves the field off the answers of methods that report no activity", async () => {
 		const server = createServer(baseOptions(socket));
 
 		const response = await call(socket, {
@@ -1020,7 +1020,7 @@ describe("activity on the answer frame", () => {
 		action: "add",
 	};
 
-	test("sends the entries to the events ring once when the caller left before the answer", async () => {
+	it("sends the entries to the events ring once when the caller left before the answer", async () => {
 		const ring: BridgeActivity[][] = [];
 
 		let finish: () => void = () => undefined;
@@ -1068,7 +1068,7 @@ describe("activity on the answer frame", () => {
 		]);
 	});
 
-	test("sends nothing to the ring when the answer reached the caller", async () => {
+	it("sends nothing to the ring when the answer reached the caller", async () => {
 		const ring: BridgeActivity[][] = [];
 
 		const server = createServer({
@@ -1099,7 +1099,7 @@ describe("activity on the answer frame", () => {
 		expect(ring).toEqual([]);
 	});
 
-	test("sends the entries to the ring when the answer cannot be written", async () => {
+	it("sends the entries to the ring when the answer cannot be written", async () => {
 		const ring: BridgeActivity[][] = [];
 
 		const server = createServer({
@@ -1144,7 +1144,7 @@ describe("failure frames", () => {
 		});
 	});
 
-	test("carry the cause in the staff error and keep the customer text friendly", async () => {
+	it("carry the cause in the staff error and keep the customer text friendly", async () => {
 		const SECRET = "tok-1234567890-secret";
 
 		const server = createServer({

package/src/runtime/bridgeSetup.test.ts

@@ -1,4 +1,4 @@
-import { describe, expect, test } from "bun:test";
+import { describe, expect, it } from "bun:test";
 import { BridgeSetupPromptKind, BridgeSetupStepState } from "../core";
 import type { BridgeSetupRuntime } from "../protocol";
 import { failedSetup, normalizeSetupRuntime, setupSummaryOf, unsupportedSetup } from "./bridgeSetup";
@@ -23,11 +23,11 @@ const runtimeWith = (prompt: BridgeSetupRuntime["prompt"]): BridgeSetupRuntime =
 });
 
 describe("the setup summary", () => {
-	test("has nothing to summarise while the driver reported no runtime", () => {
+	it("has nothing to summarise while the driver reported no runtime", () => {
 		expect(setupSummaryOf(null)).toBeNull();
 	});
 
-	test("names the active driver step", () => {
+	it("names the active driver step", () => {
 		expect(
 			setupSummaryOf(
 				runtimeWith({
@@ -41,7 +41,7 @@ describe("the setup summary", () => {
 		});
 	});
 
-	test("names no step once every driver step is done", () => {
+	it("names no step once every driver step is done", () => {
 		expect(
 			setupSummaryOf({
 				steps: [
@@ -58,7 +58,7 @@ describe("the setup summary", () => {
 		});
 	});
 
-	test("asks for attention while a link prompt waits on the customer", () => {
+	it("asks for attention while a link prompt waits on the customer", () => {
 		expect(
 			setupSummaryOf(
 				runtimeWith({
@@ -73,7 +73,7 @@ describe("the setup summary", () => {
 		).toBe(true);
 	});
 
-	test("asks for attention on an action prompt and on a failed one", () => {
+	it("asks for attention on an action prompt and on a failed one", () => {
 		expect(
 			setupSummaryOf(
 				runtimeWith({
@@ -95,7 +95,7 @@ describe("the setup summary", () => {
 		).toBe(true);
 	});
 
-	test("waits quietly while a progress prompt runs", () => {
+	it("waits quietly while a progress prompt runs", () => {
 		expect(
 			setupSummaryOf(
 				runtimeWith({
@@ -110,7 +110,7 @@ describe("the setup summary", () => {
 });
 
 describe("the unsupported sentinel", () => {
-	test("answers a driver that declares no setup", () => {
+	it("answers a driver that declares no setup", () => {
 		expect(unsupportedSetup()).toEqual({
 			supported: false,
 			runtime: null,
@@ -119,7 +119,7 @@ describe("the unsupported sentinel", () => {
 });
 
 describe("a rejected submit", () => {
-	test("turns the user error into a failed prompt keeping the previous actions", () => {
+	it("turns the user error into a failed prompt keeping the previous actions", () => {
 		const result = failedSetup(
 			runtimeWith({
 				kind: BridgeSetupPromptKind.Link,
@@ -164,7 +164,7 @@ describe("a rejected submit", () => {
 		});
 	});
 
-	test("carries no actions over from a prompt that had none", () => {
+	it("carries no actions over from a prompt that had none", () => {
 		const result = failedSetup(
 			runtimeWith({
 				kind: BridgeSetupPromptKind.Wait,
@@ -180,7 +180,7 @@ describe("a rejected submit", () => {
 		});
 	});
 
-	test("stays empty when the driver never reported a runtime", () => {
+	it("stays empty when the driver never reported a runtime", () => {
 		expect(failedSetup(null, text("nothing to fail"))).toEqual({
 			supported: true,
 			runtime: null,
@@ -236,7 +236,7 @@ const link = (
 };
 
 describe("normalizing a reported runtime", () => {
-	test("has nothing to normalize while the driver reported no runtime", () => {
+	it("has nothing to normalize while the driver reported no runtime", () => {
 		expect(
 			normalizeSetupRuntime(null, {
 				declared,
@@ -247,7 +247,7 @@ describe("normalizing a reported runtime", () => {
 		).toBeNull();
 	});
 
-	test("drops a step the driver never declared", () => {
+	it("drops a step the driver never declared", () => {
 		const { normalized, reasons } = normalizing(
 			withSteps([
 				{
@@ -272,7 +272,7 @@ describe("normalizing a reported runtime", () => {
 		]);
 	});
 
-	test("drops the second report of the same step", () => {
+	it("drops the second report of the same step", () => {
 		const { normalized, reasons } = normalizing(
 			withSteps([
 				{
@@ -297,7 +297,7 @@ describe("normalizing a reported runtime", () => {
 		]);
 	});
 
-	test("keeps the first active step and pends the rest", () => {
+	it("keeps the first active step and pends the rest", () => {
 		const { normalized, reasons } = normalizing(
 			withSteps([
 				{
@@ -326,7 +326,7 @@ describe("normalizing a reported runtime", () => {
 		]);
 	});
 
-	test("clamps a percent that left the scale", () => {
+	it("clamps a percent that left the scale", () => {
 		expect(
 			normalizing(
 				runtimeWith({
@@ -361,7 +361,7 @@ describe("normalizing a reported runtime", () => {
 		});
 	});
 
-	test("drops a percent that is not a number", () => {
+	it("drops a percent that is not a number", () => {
 		const { normalized, reasons } = normalizing(
 			runtimeWith({
 				kind: BridgeSetupPromptKind.Progress,
@@ -382,7 +382,7 @@ describe("normalizing a reported runtime", () => {
 		]);
 	});
 
-	test("cuts a detail line to the length the platform takes", () => {
+	it("cuts a detail line to the length the platform takes", () => {
 		const { normalized, reasons } = normalizing(
 			runtimeWith({
 				kind: BridgeSetupPromptKind.Progress,
@@ -403,7 +403,7 @@ describe("normalizing a reported runtime", () => {
 		]);
 	});
 
-	test("drops an empty detail line", () => {
+	it("drops an empty detail line", () => {
 		const { normalized, reasons } = normalizing(
 			runtimeWith({
 				kind: BridgeSetupPromptKind.Progress,
@@ -421,7 +421,7 @@ describe("normalizing a reported runtime", () => {
 		]);
 	});
 
-	test("drops an action whose id is not kebab-case and the duplicate beside it", () => {
+	it("drops an action whose id is not kebab-case and the duplicate beside it", () => {
 		const { normalized, reasons } = normalizing(
 			runtimeWith({
 				kind: BridgeSetupPromptKind.Action,
@@ -459,7 +459,7 @@ describe("normalizing a reported runtime", () => {
 		]);
 	});
 
-	test("keeps only as many actions as the platform renders", () => {
+	it("keeps only as many actions as the platform renders", () => {
 		const { normalized, reasons } = normalizing(
 			runtimeWith({
 				kind: BridgeSetupPromptKind.Failed,
@@ -480,7 +480,7 @@ describe("normalizing a reported runtime", () => {
 		]);
 	});
 
-	test("cuts a long code and drops an empty one", () => {
+	it("cuts a long code and drops an empty one", () => {
 		expect(
 			normalizing(
 				link({
@@ -502,7 +502,7 @@ describe("normalizing a reported runtime", () => {
 		]);
 	});
 
-	test("re-emits an expiry as an iso stamp and drops one it cannot read", () => {
+	it("re-emits an expiry as an iso stamp and drops one it cannot read", () => {
 		expect(
 			normalizing(
 				link({
@@ -524,7 +524,7 @@ describe("normalizing a reported runtime", () => {
 		]);
 	});
 
-	test("turns a link the browser cannot open into a failed prompt keeping its actions", () => {
+	it("turns a link the browser cannot open into a failed prompt keeping its actions", () => {
 		const { normalized, reasons } = normalizing(
 			link({
 				url: "javascript:alert(1)",
@@ -540,7 +540,7 @@ describe("normalizing a reported runtime", () => {
 		expect(normalized?.prompt).toEqual({
 			kind: BridgeSetupPromptKind.Failed,
 			message: {
-				ar: "اللعبة رجّعت رابط دخول ما يشتغل. جرّب مرة ثانية.",
+				ar: "أعادت اللعبة رابط تسجيل دخول غير صالح، جرب مرة ثانية.",
 				en: "The game reported a broken sign-in link. Try again.",
 			},
 			actions: [
@@ -555,7 +555,7 @@ describe("normalizing a reported runtime", () => {
 		]);
 	});
 
-	test("turns a url it cannot parse into a failed prompt", () => {
+	it("turns a url it cannot parse into a failed prompt", () => {
 		expect(
 			normalizing(
 				link({
@@ -565,7 +565,7 @@ describe("normalizing a reported runtime", () => {
 		).toBe(BridgeSetupPromptKind.Failed);
 	});
 
-	test("leaves a healthy runtime exactly as the driver reported it", () => {
+	it("leaves a healthy runtime exactly as the driver reported it", () => {
 		const source = link({});
 		const { normalized, reasons } = normalizing(source);
 

package/src/runtime/bridgeSetup.ts

@@ -68,7 +68,7 @@ export const failedSetup = (runtime: BridgeSetupRuntime | null, text: LocalizedT
 };
 
 const BROKEN_LINK_TEXT: LocalizedText = {
-	ar: "اللعبة رجّعت رابط دخول ما يشتغل. جرّب مرة ثانية.",
+	ar: "أعادت اللعبة رابط تسجيل دخول غير صالح، جرب مرة ثانية.",
 	en: "The game reported a broken sign-in link. Try again.",
 };
 

package/src/runtime/bridgeSetupHandlers.test.ts

@@ -1,4 +1,4 @@
-import { describe, expect, test } from "bun:test";
+import { describe, expect, it } from "bun:test";
 import { tmpdir } from "node:os";
 import type { BridgeContext } from "../context";
 import { BridgeKind, BridgeSetupPromptKind, BridgeSetupStepKind, BridgeSetupStepState, BridgeUserError } from "../core";
@@ -140,7 +140,7 @@ const setupWith = (submit?: BridgeSetupModule["submit"]): BridgeSetupModule => (
 });
 
 describe("the setup handlers", () => {
-	test("answer unsupported for a driver that declares no setup", async () => {
+	it("answer unsupported for a driver that declares no setup", async () => {
 		const { logger } = recorder();
 		const handlers = createSetupHandlers(driverWith(), logger);
 
@@ -155,7 +155,7 @@ describe("the setup handlers", () => {
 		});
 	});
 
-	test("keep nothing for a driver that declares no setup, whatever it reports", () => {
+	it("keep nothing for a driver that declares no setup, whatever it reports", () => {
 		const { logger, warnings } = recorder();
 		const handlers = createSetupHandlers(driverWith(), logger);
 
@@ -165,7 +165,7 @@ describe("the setup handlers", () => {
 		expect(warnings).toEqual([]);
 	});
 
-	test("answer the normalized runtime the driver reported", () => {
+	it("answer the normalized runtime the driver reported", () => {
 		const { logger, warnings } = recorder();
 		const handlers = createSetupHandlers(driverWith(setupWith()), logger);
 
@@ -206,7 +206,7 @@ describe("the setup handlers", () => {
 		]);
 	});
 
-	test("answer unsupported for a setup that exports no submit", async () => {
+	it("answer unsupported for a setup that exports no submit", async () => {
 		const { logger } = recorder();
 		const handlers = createSetupHandlers(driverWith(setupWith()), logger);
 
@@ -216,7 +216,7 @@ describe("the setup handlers", () => {
 		});
 	});
 
-	test("turn a user error into a failed prompt that keeps the previous actions", async () => {
+	it("turn a user error into a failed prompt that keeps the previous actions", async () => {
 		const { logger, warnings } = recorder();
 
 		const handlers = createSetupHandlers(
@@ -254,7 +254,7 @@ describe("the setup handlers", () => {
 		expect(warnings.at(-1)?.message).toBe("setup submit rejected");
 	});
 
-	test("add one rejected entry built from the user error", async () => {
+	it("add one rejected entry built from the user error", async () => {
 		const { logger } = recorder();
 
 		const handlers = createSetupHandlers(
@@ -288,7 +288,7 @@ describe("the setup handlers", () => {
 		]);
 	});
 
-	test("answer what the driver reported while submitting", async () => {
+	it("answer what the driver reported while submitting", async () => {
 		const { logger } = recorder();
 
 		const handlers = createSetupHandlers(
@@ -313,7 +313,7 @@ describe("the setup handlers", () => {
 		]);
 	});
 
-	test("answer the runtime the driver reported while submitting", async () => {
+	it("answer the runtime the driver reported while submitting", async () => {
 		const { logger } = recorder();
 
 		const handlers = createSetupHandlers(
@@ -349,7 +349,7 @@ describe("the setup handlers", () => {
 		});
 	});
 
-	test("let an error the driver cannot explain out", async () => {
+	it("let an error the driver cannot explain out", async () => {
 		const { logger } = recorder();
 
 		const handlers = createSetupHandlers(
@@ -372,7 +372,7 @@ describe("the setup handlers redaction", () => {
 
 	const redact = (value: string) => value.split(SECRET).join("••••");
 
-	test("blanks a secret value in the prompt a driver reports", () => {
+	it("blanks a secret value in the prompt a driver reports", () => {
 		const { logger } = recorder();
 		const handlers = createSetupHandlers(driverWith(setupWith()), logger, redact);
 
@@ -399,7 +399,7 @@ describe("the setup handlers redaction", () => {
 		});
 	});
 
-	test("blanks a secret value in every text of a link prompt", () => {
+	it("blanks a secret value in every text of a link prompt", () => {
 		const { logger } = recorder();
 		const handlers = createSetupHandlers(driverWith(setupWith()), logger, redact);
 		const reported = signIn();
@@ -429,7 +429,7 @@ describe("the setup handlers redaction", () => {
 		});
 	});
 
-	test("blanks a secret value in the text of a rejected submit", async () => {
+	it("blanks a secret value in the text of a rejected submit", async () => {
 		const { logger } = recorder();
 		const handlers = createSetupHandlers(
 			driverWith(

package/src/runtime/bridgeSignal.test.ts

@@ -1,17 +1,17 @@
-import { describe, expect, test } from "bun:test";
+import { describe, expect, it } from "bun:test";
 import { leadsGroup, readProcessGroup, signalGroup } from "./bridgeSignal";
 
 const MISSING_PID = 2 ** 22 - 1;
 
 describe("readProcessGroup", () => {
-	test("reads the process group of this process", async () => {
+	it("reads the process group of this process", async () => {
 		const group = await readProcessGroup(process.pid);
 
 		expect(group).not.toBeNull();
 		expect(group).toBeGreaterThan(0);
 	});
 
-	test("puts a detached child in a group of its own and an attached one in ours", async () => {
+	it("puts a detached child in a group of its own and an attached one in ours", async () => {
 		const detached = Bun.spawn(
 			[
 				"sleep",
@@ -49,13 +49,13 @@ describe("readProcessGroup", () => {
 		}
 	});
 
-	test("answers null for a pid that does not exist", async () => {
+	it("answers null for a pid that does not exist", async () => {
 		expect(await readProcessGroup(MISSING_PID)).toBeNull();
 	});
 });
 
 describe("leadsGroup", () => {
-	test("only a pid equal to its own group leads it", () => {
+	it("only a pid equal to its own group leads it", () => {
 		expect(leadsGroup(1234, 1234)).toBe(true);
 		expect(leadsGroup(1234, 1200)).toBe(false);
 		expect(leadsGroup(1234, null)).toBe(false);
@@ -63,7 +63,7 @@ describe("leadsGroup", () => {
 });
 
 describe("signalGroup", () => {
-	test("signals every process sharing the leader's group", async () => {
+	it("signals every process sharing the leader's group", async () => {
 		const leader = Bun.spawn(
 			[
 				"bash",
@@ -90,7 +90,7 @@ describe("signalGroup", () => {
 		expect(() => process.kill(child, 0)).toThrow();
 	}, 20_000);
 
-	test("refuses to signal a group the pid does not lead", async () => {
+	it("refuses to signal a group the pid does not lead", async () => {
 		const attached = Bun.spawn(
 			[
 				"sleep",