@serverkgg/bridge 0.1.1 → 0.1.3

package/src/runtime/bridgeDriver.test.ts

@@ -1,4 +1,4 @@
-import { describe, expect, test } from "bun:test";
+import { describe, expect, it } from "bun:test";
 import { BridgeKind, BridgeSetupStepKind, PortJoin, PortProtocol } from "../core";
 import { GuideOpenTab } from "../guides";
 import type { ManifestPort, ManifestPortActivation } from "../manifest";
@@ -275,23 +275,44 @@ const driverWithTabs = (tabs: number, sections: number): BridgeDriver => ({
 });
 
 describe("panel caps", () => {
-	test("takes the tab and section counts the agent mirror accepts", () => {
+	it("rejects a collection row action named after a read action", () => {
+		const driver = driverWithTabs(1, 1);
+
+		driver.modules = {
+			...driver.modules,
+			worlds: {
+				kind: BridgeKind.Collection,
+				list: async () => [],
+				actions: {
+					preview: async () => {
+						return;
+					},
+				},
+			},
+		};
+
+		expect(() => validateDriver(driver)).toThrow(
+			'module "worlds" declares action "preview", which is reserved for reads',
+		);
+	});
+
+	it("takes the tab and section counts the agent mirror accepts", () => {
 		expect(() => validateDriver(driverWithTabs(8, 8))).not.toThrow();
 	});
 
-	test("rejects a ninth tab the agent mirror would drop", () => {
+	it("rejects a ninth tab the agent mirror would drop", () => {
 		expect(() => validateDriver(driverWithTabs(9, 1))).toThrow(
 			"the panel declares 9 tabs — the platform takes at most 8",
 		);
 	});
 
-	test("rejects a ninth section the agent mirror would drop", () => {
+	it("rejects a ninth section the agent mirror would drop", () => {
 		expect(() => validateDriver(driverWithTabs(1, 9))).toThrow(
 			'tab "tab-0" declares 9 sections — the platform takes at most 8',
 		);
 	});
 
-	test("rejects a form carrying more fields than the agent mirror accepts", () => {
+	it("rejects a form carrying more fields than the agent mirror accepts", () => {
 		const driver = driverWithTabs(1, 1);
 		const [tab] = driver.panel?.tabs ?? [];
 		const [section] = tab?.sections ?? [];
@@ -307,7 +328,7 @@ describe("panel caps", () => {
 		expect(() => validateDriver(driver)).toThrow("declares 33 fields — the platform takes at most 32");
 	});
 
-	test("rejects a table carrying more columns than the agent mirror accepts", () => {
+	it("rejects a table carrying more columns than the agent mirror accepts", () => {
 		const driver = driverWithTabs(1, 1);
 		const [tab] = driver.panel?.tabs ?? [];
 
@@ -339,7 +360,7 @@ describe("panel caps", () => {
 		expect(() => validateDriver(driver)).toThrow("declares 9 columns — the platform takes at most 8");
 	});
 
-	test("rejects a label longer than the agent mirror accepts", () => {
+	it("rejects a label longer than the agent mirror accepts", () => {
 		const driver = driverWithTabs(1, 1);
 		const [tab] = driver.panel?.tabs ?? [];
 
@@ -356,7 +377,7 @@ describe("panel caps", () => {
 });
 
 describe("actions sections", () => {
-	test("accepts an action the module implements", () => {
+	it("accepts an action the module implements", () => {
 		expect(() =>
 			validateDriver(
 				driverWithActions([
@@ -372,7 +393,7 @@ describe("actions sections", () => {
 		).not.toThrow();
 	});
 
-	test("rejects an action the module does not implement", () => {
+	it("rejects an action the module does not implement", () => {
 		expect(() =>
 			validateDriver(
 				driverWithActions([
@@ -385,11 +406,11 @@ describe("actions sections", () => {
 		).toThrow('declares action "shout", which module "broadcast" does not implement');
 	});
 
-	test("rejects a section with no actions", () => {
+	it("rejects a section with no actions", () => {
 		expect(() => validateDriver(driverWithActions([]))).toThrow("declares no actions");
 	});
 
-	test("rejects an action carrying more fields than one prompt fits", () => {
+	it("rejects an action carrying more fields than one prompt fits", () => {
 		expect(() =>
 			validateDriver(
 				driverWithActions([
@@ -410,7 +431,7 @@ describe("actions sections", () => {
 		).toThrow('action "say" declares 6 fields — at most 5 fit one prompt');
 	});
 
-	test("rejects a section pointing at a module that is not an actions module", () => {
+	it("rejects a section pointing at a module that is not an actions module", () => {
 		const driver = driverWithActions([
 			{
 				id: "say",
@@ -428,7 +449,7 @@ describe("actions sections", () => {
 		expect(() => validateDriver(driver)).toThrow('references "broadcast", which is not an actions module');
 	});
 
-	test("checks the fields a row action declares", () => {
+	it("checks the fields a row action declares", () => {
 		const driver = driverWithActions([
 			{
 				id: "say",
@@ -489,7 +510,7 @@ describe("actions sections", () => {
 });
 
 describe("visibleWhen", () => {
-	test("accepts a rule that matches on values", () => {
+	it("accepts a rule that matches on values", () => {
 		expect(() =>
 			validateDriver(
 				driverWith({
@@ -502,7 +523,7 @@ describe("visibleWhen", () => {
 		).not.toThrow();
 	});
 
-	test("accepts a rule that matches on emptiness", () => {
+	it("accepts a rule that matches on emptiness", () => {
 		expect(() =>
 			validateDriver(
 				driverWith({
@@ -522,7 +543,7 @@ describe("visibleWhen", () => {
 		).not.toThrow();
 	});
 
-	test("accepts a rule that narrows values inside a presence check", () => {
+	it("accepts a rule that narrows values inside a presence check", () => {
 		expect(() =>
 			validateDriver(
 				driverWith({
@@ -536,7 +557,7 @@ describe("visibleWhen", () => {
 		).not.toThrow();
 	});
 
-	test("rejects a rule that declares neither values nor empty", () => {
+	it("rejects a rule that declares neither values nor empty", () => {
 		expect(() =>
 			validateDriver(
 				driverWith({
@@ -546,7 +567,7 @@ describe("visibleWhen", () => {
 		).toThrow('is visible when "MODPACK" matches nothing');
 	});
 
-	test("rejects an empty values list", () => {
+	it("rejects an empty values list", () => {
 		expect(() =>
 			validateDriver(
 				driverWith({
@@ -557,7 +578,7 @@ describe("visibleWhen", () => {
 		).toThrow('is visible when "MODPACK" matches nothing');
 	});
 
-	test("checks a field rule as well as a section rule", () => {
+	it("checks a field rule as well as a section rule", () => {
 		const driver = driverWith({
 			variable: "MODPACK",
 			empty: true,
@@ -584,11 +605,11 @@ describe("visibleWhen", () => {
 });
 
 describe("form preview", () => {
-	test("accepts a variables form pointing at a preview module", () => {
+	it("accepts a variables form pointing at a preview module", () => {
 		expect(() => validateDriver(driverWithPreview())).not.toThrow();
 	});
 
-	test("rejects a preview ref to a module that is not a preview module", () => {
+	it("rejects a preview ref to a module that is not a preview module", () => {
 		const driver = driverWithPreview();
 
 		driver.modules = {
@@ -601,7 +622,7 @@ describe("form preview", () => {
 		expect(() => validateDriver(driver)).toThrow('references "transition", which is not a preview module');
 	});
 
-	test("rejects a preview on a form that writes settings", () => {
+	it("rejects a preview on a form that writes settings", () => {
 		const driver = driverWithPreview();
 		const section = formSectionOf(driver);
 
@@ -624,7 +645,7 @@ describe("form preview", () => {
 });
 
 describe("validatePortActivation", () => {
-	test("accepts a variable a variables form or a catalog section declares", () => {
+	it("accepts a variable a variables form or a catalog section declares", () => {
 		expect(() =>
 			validatePortActivation(
 				[
@@ -646,7 +667,7 @@ describe("validatePortActivation", () => {
 		).not.toThrow();
 	});
 
-	test("rejects a variable nothing in the panel declares", () => {
+	it("rejects a variable nothing in the panel declares", () => {
 		expect(() =>
 			validatePortActivation(
 				[
@@ -783,11 +804,11 @@ const clearingFormOf = (driver: BridgeDriver) => {
 };
 
 describe("detail sections", () => {
-	test("accepts a detail section on a detail module", () => {
+	it("accepts a detail section on a detail module", () => {
 		expect(() => validateDriver(withNoop(driverWithDetail()))).not.toThrow();
 	});
 
-	test("rejects a section pointing at a module that is not a detail module", () => {
+	it("rejects a section pointing at a module that is not a detail module", () => {
 		const driver = withNoop(driverWithDetail());
 
 		driver.modules = {
@@ -801,7 +822,7 @@ describe("detail sections", () => {
 		expect(() => validateDriver(driver)).toThrow('references "modpackStatus", which is not a detail module');
 	});
 
-	test("rejects an action the module does not implement", () => {
+	it("rejects an action the module does not implement", () => {
 		const driver = withNoop(driverWithDetail());
 
 		detailSectionOf(driver).actions = [
@@ -816,7 +837,7 @@ describe("detail sections", () => {
 		);
 	});
 
-	test("rejects a download action, since a detail has no rows to serve", () => {
+	it("rejects a download action, since a detail has no rows to serve", () => {
 		const driver = withNoop(driverWithDetail());
 
 		detailSectionOf(driver).actions = [
@@ -830,7 +851,7 @@ describe("detail sections", () => {
 		expect(() => validateDriver(driver)).toThrow('declares action "update" as a download');
 	});
 
-	test("rejects a related tab the panel does not declare", () => {
+	it("rejects a related tab the panel does not declare", () => {
 		const driver = withNoop(driverWithDetail());
 
 		detailSectionOf(driver).related = {
@@ -841,7 +862,7 @@ describe("detail sections", () => {
 		expect(() => validateDriver(driver)).toThrow('relates to tab "worlds", which the panel does not declare');
 	});
 
-	test("counts a detail variable as declared for port activation", () => {
+	it("counts a detail variable as declared for port activation", () => {
 		expect(() =>
 			validatePortActivation(
 				[
@@ -859,11 +880,11 @@ describe("detail sections", () => {
 });
 
 describe("form clears", () => {
-	test("accepts a clear of a variable a detail section declares", () => {
+	it("accepts a clear of a variable a detail section declares", () => {
 		expect(() => validateDriver(withNoop(driverWithDetail()))).not.toThrow();
 	});
 
-	test("rejects a clear of a variable nothing declares", () => {
+	it("rejects a clear of a variable nothing declares", () => {
 		const driver = withNoop(driverWithDetail());
 
 		clearingFormOf(driver).clears = [
@@ -878,7 +899,7 @@ describe("form clears", () => {
 		);
 	});
 
-	test("rejects a clear of one of the form's own fields", () => {
+	it("rejects a clear of one of the form's own fields", () => {
 		const driver = withNoop(driverWithDetail());
 
 		clearingFormOf(driver).clears = [
@@ -891,7 +912,7 @@ describe("form clears", () => {
 		expect(() => validateDriver(driver)).toThrow('clears "MC_VERSION", which is one of its own fields');
 	});
 
-	test("rejects clears on a form that writes settings", () => {
+	it("rejects clears on a form that writes settings", () => {
 		const driver = withNoop(driverWithDetail());
 		const form = clearingFormOf(driver);
 
@@ -969,16 +990,16 @@ describe("table uploads", () => {
 		mode,
 	});
 
-	test("accepts an upload that declares no mode", () => {
+	it("accepts an upload that declares no mode", () => {
 		expect(() => validateDriver(driverWithUpload(upload()))).not.toThrow();
 	});
 
-	test("accepts both declared upload modes", () => {
+	it("accepts both declared upload modes", () => {
 		expect(() => validateDriver(driverWithUpload(upload(BridgeUploadMode.Archive)))).not.toThrow();
 		expect(() => validateDriver(driverWithUpload(upload(BridgeUploadMode.File)))).not.toThrow();
 	});
 
-	test("rejects an upload mode the platform does not know", () => {
+	it("rejects an upload mode the platform does not know", () => {
 		const section = upload("stream" as BridgeUploadMode);
 
 		expect(() => validateDriver(driverWithUpload(section))).toThrow('declares an unknown upload mode "stream"');
@@ -1015,7 +1036,7 @@ describe("the pending module", () => {
 		...overrides,
 	});
 
-	test("accepts a driver that declares no pending module", () => {
+	it("accepts a driver that declares no pending module", () => {
 		expect(() =>
 			validateDriver({
 				install: {
@@ -1034,11 +1055,11 @@ describe("the pending module", () => {
 		).not.toThrow();
 	});
 
-	test("accepts a pending module declaring a staging path and one extension", () => {
+	it("accepts a pending module declaring a staging path and one extension", () => {
 		expect(() => validateDriver(driverWithPending(pendingWith({})))).not.toThrow();
 	});
 
-	test("rejects a pending module carrying another kind", () => {
+	it("rejects a pending module carrying another kind", () => {
 		const pending = pendingWith({
 			kind: BridgeKind.Collection as BridgeKind.Pending,
 		});
@@ -1048,7 +1069,7 @@ describe("the pending module", () => {
 		);
 	});
 
-	test("rejects a pending module with no staging path", () => {
+	it("rejects a pending module with no staging path", () => {
 		expect(() =>
 			validateDriver(
 				driverWithPending(
@@ -1060,7 +1081,7 @@ describe("the pending module", () => {
 		).toThrow("the pending module declares no staging path");
 	});
 
-	test("rejects a staging path longer than the platform takes", () => {
+	it("rejects a staging path longer than the platform takes", () => {
 		expect(() =>
 			validateDriver(
 				driverWithPending(
@@ -1072,7 +1093,7 @@ describe("the pending module", () => {
 		).toThrow("the pending module staging path is 129 characters — the platform takes at most 128");
 	});
 
-	test("rejects a pending module declaring no extensions", () => {
+	it("rejects a pending module declaring no extensions", () => {
 		expect(() =>
 			validateDriver(
 				driverWithPending(
@@ -1084,7 +1105,7 @@ describe("the pending module", () => {
 		).toThrow("the pending module declares no extensions");
 	});
 
-	test("rejects more extensions than the platform takes", () => {
+	it("rejects more extensions than the platform takes", () => {
 		expect(() =>
 			validateDriver(
 				driverWithPending(
@@ -1098,7 +1119,7 @@ describe("the pending module", () => {
 		).toThrow("the pending module declares 9 extensions — the platform takes at most 8");
 	});
 
-	test("rejects an extension longer than the platform takes", () => {
+	it("rejects an extension longer than the platform takes", () => {
 		expect(() =>
 			validateDriver(
 				driverWithPending(
@@ -1112,7 +1133,7 @@ describe("the pending module", () => {
 		).toThrow('the pending module extension "xxxxxxxxxxxxxxxxx" is 17 characters — the platform takes at most 16');
 	});
 
-	test("reserves the pending module id for the singleton", () => {
+	it("reserves the pending module id for the singleton", () => {
 		expect(() =>
 			validateDriver({
 				install: {
@@ -1224,7 +1245,7 @@ const driverWithFields = (fields: BridgeField[]): BridgeDriver => ({
 });
 
 describe("the setup singleton", () => {
-	test("takes a driver step with both locales", () => {
+	it("takes a driver step with both locales", () => {
 		expect(() =>
 			validateDriver(
 				driverWithSetup(
@@ -1236,11 +1257,11 @@ describe("the setup singleton", () => {
 		).not.toThrow();
 	});
 
-	test("rejects a setup that declares no steps", () => {
+	it("rejects a setup that declares no steps", () => {
 		expect(() => validateDriver(driverWithSetup(setupWith([])))).toThrow("the setup module declares no steps");
 	});
 
-	test("rejects more steps than the platform takes", () => {
+	it("rejects more steps than the platform takes", () => {
 		expect(() =>
 			validateDriver(
 				driverWithSetup(
@@ -1254,7 +1275,7 @@ describe("the setup singleton", () => {
 		).toThrow("the setup module declares 9 steps — the platform takes at most 8");
 	});
 
-	test("rejects a step id that is not kebab-case", () => {
+	it("rejects a step id that is not kebab-case", () => {
 		expect(() =>
 			validateDriver(
 				driverWithSetup(
@@ -1266,7 +1287,7 @@ describe("the setup singleton", () => {
 		).toThrow('the setup module step id "signIn" must be kebab-case');
 	});
 
-	test("rejects the same step id twice", () => {
+	it("rejects the same step id twice", () => {
 		expect(() =>
 			validateDriver(
 				driverWithSetup(
@@ -1279,7 +1300,7 @@ describe("the setup singleton", () => {
 		).toThrow('the setup module declares step "sign-in" more than once');
 	});
 
-	test("rejects a title that is missing a locale", () => {
+	it("rejects a title that is missing a locale", () => {
 		expect(() =>
 			validateDriver(
 				driverWithSetup(
@@ -1298,7 +1319,7 @@ describe("the setup singleton", () => {
 		).toThrow('the setup module step "sign-in" title needs both arabic and english text');
 	});
 
-	test("takes a form step that names an existing form section and its own fields", () => {
+	it("takes a form step that names an existing form section and its own fields", () => {
 		expect(() =>
 			validateDriver(
 				driverWithSetup(
@@ -1319,7 +1340,7 @@ describe("the setup singleton", () => {
 		).not.toThrow();
 	});
 
-	test("rejects a form step that names a section the panel does not declare", () => {
+	it("rejects a form step that names a section the panel does not declare", () => {
 		expect(() =>
 			validateDriver(
 				driverWithSetup(
@@ -1339,7 +1360,7 @@ describe("the setup singleton", () => {
 		);
 	});
 
-	test("rejects a form step that names a field the section does not declare", () => {
+	it("rejects a form step that names a field the section does not declare", () => {
 		expect(() =>
 			validateDriver(
 				driverWithSetup(
@@ -1360,7 +1381,7 @@ describe("the setup singleton", () => {
 		).toThrow('the setup module step "name" names field "PASSWORD", which section "server" does not declare');
 	});
 
-	test("takes an open step that names an existing panel tab", () => {
+	it("takes an open step that names an existing panel tab", () => {
 		expect(() =>
 			validateDriver(
 				driverWithSetup(
@@ -1380,7 +1401,7 @@ describe("the setup singleton", () => {
 		).not.toThrow();
 	});
 
-	test("rejects an open step that names a panel tab the panel does not declare", () => {
+	it("rejects an open step that names a panel tab the panel does not declare", () => {
 		expect(() =>
 			validateDriver(
 				driverWithSetup(
@@ -1400,7 +1421,7 @@ describe("the setup singleton", () => {
 		).toThrow('the setup module step "tune" opens panel tab "world", which the panel does not declare');
 	});
 
-	test("takes an open step that leaves the panel altogether", () => {
+	it("takes an open step that leaves the panel altogether", () => {
 		expect(() =>
 			validateDriver(
 				driverWithSetup(
@@ -1419,7 +1440,7 @@ describe("the setup singleton", () => {
 		).not.toThrow();
 	});
 
-	test("takes a form step over a secret field", () => {
+	it("takes a form step over a secret field", () => {
 		expect(() =>
 			validateDriver(
 				driverWithSetup(
@@ -1440,7 +1461,7 @@ describe("the setup singleton", () => {
 		).not.toThrow();
 	});
 
-	test("rejects a driver step while the setup exports no submit", () => {
+	it("rejects a driver step while the setup exports no submit", () => {
 		expect(() =>
 			validateDriver(
 				driverWithSetup(
@@ -1452,7 +1473,7 @@ describe("the setup singleton", () => {
 		).toThrow("the setup module declares a driver step but exports no submit");
 	});
 
-	test("takes a setup with no submit while every step is answered elsewhere", () => {
+	it("takes a setup with no submit while every step is answered elsewhere", () => {
 		expect(() =>
 			validateDriver(
 				driverWithSetup(
@@ -1471,7 +1492,7 @@ describe("the setup singleton", () => {
 		).not.toThrow();
 	});
 
-	test("reserves the setup module id for the singleton", () => {
+	it("reserves the setup module id for the singleton", () => {
 		expect(() =>
 			validateDriver({
 				install: {
@@ -1498,7 +1519,7 @@ describe("the setup singleton", () => {
 });
 
 describe("field patterns", () => {
-	test("takes a pattern on a text field and on a secret one", () => {
+	it("takes a pattern on a text field and on a secret one", () => {
 		expect(() =>
 			validateDriver(
 				driverWithFields([
@@ -1506,8 +1527,8 @@ describe("field patterns", () => {
 						key: "LICENSE_KEY",
 						control: BridgeControl.Secret,
 						label: text("Licence key"),
-						pattern: "^cfxk_\\w{1,60}_\\w{1,20}$",
-						patternHint: text("A licence key starts with cfxk_"),
+						pattern: "^key_\\w{1,60}_\\w{1,20}$",
+						patternHint: text("A licence key starts with key_"),
 					},
 					{
 						key: "SERVER_NAME",
@@ -1520,7 +1541,7 @@ describe("field patterns", () => {
 		).not.toThrow();
 	});
 
-	test("rejects a pattern that is not a regular expression", () => {
+	it("rejects a pattern that is not a regular expression", () => {
 		expect(() =>
 			validateDriver(
 				driverWithFields([
@@ -1528,14 +1549,14 @@ describe("field patterns", () => {
 						key: "LICENSE_KEY",
 						control: BridgeControl.Text,
 						label: text("Licence key"),
-						pattern: "^cfxk_(",
+						pattern: "^key_(",
 					},
 				]),
 			),
-		).toThrow('declares pattern "^cfxk_(", which is not a valid regular expression');
+		).toThrow('declares pattern "^key_(", which is not a valid regular expression');
 	});
 
-	test("takes the FiveM licence pattern, whose repeats are all bounded", () => {
+	it("takes an alternation whose repeats are all bounded", () => {
 		expect(() =>
 			validateDriver(
 				driverWithFields([
@@ -1543,14 +1564,14 @@ describe("field patterns", () => {
 						key: "LICENSE_KEY",
 						control: BridgeControl.Secret,
 						label: text("Licence key"),
-						pattern: "^(?:cfxk_\\w{1,60}_\\w{1,20}|\\w{32})$",
+						pattern: "^(?:key_\\w{1,60}_\\w{1,20}|\\w{32})$",
 					},
 				]),
 			),
 		).not.toThrow();
 	});
 
-	test("rejects a pattern that repeats a group which already repeats", () => {
+	it("rejects a pattern that repeats a group which already repeats", () => {
 		expect(() =>
 			validateDriver(
 				driverWithFields([
@@ -1565,7 +1586,7 @@ describe("field patterns", () => {
 		).toThrow('declares pattern "^(a+)+$", which repeats a group that already repeats');
 	});
 
-	test("rejects a starred group whose body repeats without a bound", () => {
+	it("rejects a starred group whose body repeats without a bound", () => {
 		expect(() =>
 			validateDriver(
 				driverWithFields([
@@ -1580,7 +1601,7 @@ describe("field patterns", () => {
 		).toThrow("repeats a group that already repeats");
 	});
 
-	test("rejects a pattern longer than the platform takes", () => {
+	it("rejects a pattern longer than the platform takes", () => {
 		expect(() =>
 			validateDriver(
 				driverWithFields([
@@ -1595,7 +1616,7 @@ describe("field patterns", () => {
 		).toThrow("pattern is 258 characters — the platform takes at most 256");
 	});
 
-	test("rejects a pattern hint with no pattern to explain", () => {
+	it("rejects a pattern hint with no pattern to explain", () => {
 		expect(() =>
 			validateDriver(
 				driverWithFields([
@@ -1603,14 +1624,14 @@ describe("field patterns", () => {
 						key: "LICENSE_KEY",
 						control: BridgeControl.Text,
 						label: text("Licence key"),
-						patternHint: text("A licence key starts with cfxk_"),
+						patternHint: text("A licence key starts with key_"),
 					},
 				]),
 			),
 		).toThrow("declares a pattern hint with no pattern to explain");
 	});
 
-	test("rejects a pattern on a control that never matches one", () => {
+	it("rejects a pattern on a control that never matches one", () => {
 		expect(() =>
 			validateDriver(
 				driverWithFields([
@@ -1625,7 +1646,7 @@ describe("field patterns", () => {
 		).toThrow('declares a pattern on a "boolean" control — only text and secret controls match one');
 	});
 
-	test("rejects a secret control that offers options", () => {
+	it("rejects a secret control that offers options", () => {
 		expect(() =>
 			validateDriver(
 				driverWithFields([
@@ -1710,7 +1731,7 @@ describe("section placement", () => {
 		return driver;
 	};
 
-	test("accepts a table placed on the players page", () => {
+	it("accepts a table placed on the players page", () => {
 		expect(() =>
 			validateDriver(
 				driverWithRoster({
@@ -1720,19 +1741,19 @@ describe("section placement", () => {
 		).not.toThrow();
 	});
 
-	test("rejects a form placed on the players page", () => {
+	it("rejects a form placed on the players page", () => {
 		expect(() => validateDriver(placedForm(BridgePlace.Players))).toThrow(
 			'tab "version" section "version" places a form section on players, which that page cannot render',
 		);
 	});
 
-	test("rejects a form placed on the overview page", () => {
+	it("rejects a form placed on the overview page", () => {
 		expect(() => validateDriver(placedForm(BridgePlace.Overview))).toThrow(
 			"places a form section on overview, which that page cannot render",
 		);
 	});
 
-	test("rejects a table placed on the overview page", () => {
+	it("rejects a table placed on the overview page", () => {
 		expect(() =>
 			validateDriver(
 				driverWithRoster({
@@ -1742,7 +1763,7 @@ describe("section placement", () => {
 		).toThrow('tab "roster" section "online" places a table section on overview, which that page cannot render');
 	});
 
-	test("accepts an offline row action on a table", () => {
+	it("accepts an offline row action on a table", () => {
 		expect(() =>
 			validateDriver(
 				driverWithRoster({
@@ -1759,7 +1780,7 @@ describe("section placement", () => {
 		).not.toThrow();
 	});
 
-	test("rejects an offline action on an actions section", () => {
+	it("rejects an offline action on an actions section", () => {
 		expect(() =>
 			validateDriver(
 				driverWithActions([
@@ -1775,7 +1796,7 @@ describe("section placement", () => {
 		);
 	});
 
-	test("rejects a form help that is missing a locale", () => {
+	it("rejects a form help that is missing a locale", () => {
 		const driver = driverWith(anyVisibleWhen);
 
 		formSectionOf(driver).help = {
@@ -1806,7 +1827,7 @@ describe("the lifecycle module", () => {
 		},
 	});
 
-	test("accepts an environment resolver", () => {
+	it("accepts an environment resolver", () => {
 		expect(() =>
 			validateDriver(
 				lifecycleDriver({
@@ -1818,11 +1839,11 @@ describe("the lifecycle module", () => {
 		).not.toThrow();
 	});
 
-	test("accepts a lifecycle that declares no environment", () => {
+	it("accepts a lifecycle that declares no environment", () => {
 		expect(() => validateDriver(lifecycleDriver({}))).not.toThrow();
 	});
 
-	test("accepts each stop signal the bridge sends", () => {
+	it("accepts each stop signal the bridge sends", () => {
 		for (const stopSignal of Object.values(BridgeStopSignal)) {
 			expect(() =>
 				validateDriver(
@@ -1834,7 +1855,7 @@ describe("the lifecycle module", () => {
 		}
 	});
 
-	test("refuses a stop signal the bridge does not send", () => {
+	it("refuses a stop signal the bridge does not send", () => {
 		expect(() =>
 			validateDriver(
 				lifecycleDriver({
@@ -1844,7 +1865,7 @@ describe("the lifecycle module", () => {
 		).toThrow('the lifecycle module declares stop signal "SIGHUP" — the bridge sends only SIGTERM or SIGINT');
 	});
 
-	test("accepts a ready timeout from one second to an hour", () => {
+	it("accepts a ready timeout from one second to an hour", () => {
 		for (const readyTimeoutSeconds of [
 			1,
 			900,
@@ -1860,7 +1881,7 @@ describe("the lifecycle module", () => {
 		}
 	});
 
-	test("refuses a ready timeout outside one second to an hour", () => {
+	it("refuses a ready timeout outside one second to an hour", () => {
 		for (const readyTimeoutSeconds of [
 			0,
 			3601,
@@ -1878,7 +1899,7 @@ describe("the lifecycle module", () => {
 		}
 	});
 
-	test("accepts each lifecycle subject the bridge names", () => {
+	it("accepts each lifecycle subject the bridge names", () => {
 		for (const subject of Object.values(BridgeLifecycleSubject)) {
 			expect(() =>
 				validateDriver(
@@ -1890,7 +1911,7 @@ describe("the lifecycle module", () => {
 		}
 	});
 
-	test("refuses a lifecycle subject the bridge does not name", () => {
+	it("refuses a lifecycle subject the bridge does not name", () => {
 		expect(() =>
 			validateDriver(
 				lifecycleDriver({
@@ -1900,7 +1921,7 @@ describe("the lifecycle module", () => {
 		).toThrow('the lifecycle module declares subject "server" — the bridge names only game or bot');
 	});
 
-	test("refuses an environment that is not a function", () => {
+	it("refuses an environment that is not a function", () => {
 		expect(() =>
 			validateDriver(
 				lifecycleDriver({