@serverkgg/bridge 0.1.1 → 0.1.3

package/src/cli/validate.test.ts

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, describe, expect, spyOn, test } from "bun:test";
+import { afterEach, beforeEach, describe, expect, it, spyOn } from "bun:test";
 import { mkdirSync, mkdtempSync, rmSync, writeFileSync } from "node:fs";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
@@ -16,7 +16,8 @@ import {
 	type BridgeTab,
 	type BridgeTableSection,
 } from "../panel";
-import { placementReportLines, runValidate, setupReportLines } from "./validate";
+import { RULES_FILE, readShippedRules, writeRules } from "./rules";
+import { availabilityReportLines, placementReportLines, runValidate, setupReportLines } from "./validate";
 
 const text = (value: string) => ({
 	ar: value,
@@ -54,7 +55,7 @@ describe("validate", () => {
 		});
 	});
 
-	test("refuses a file: bridge dependency", async () => {
+	it("refuses a file: bridge dependency", async () => {
 		writePackage({
 			"@serverkgg/bridge": "file:../serverk/modules/bridge",
 		});
@@ -64,7 +65,7 @@ describe("validate", () => {
 		);
 	});
 
-	test("refuses a link: bridge dependency", async () => {
+	it("refuses a link: bridge dependency", async () => {
 		writePackage({
 			"@serverkgg/bridge": "link:@serverkgg/bridge",
 		});
@@ -72,7 +73,7 @@ describe("validate", () => {
 		await expect(validate()).rejects.toThrow('depends on @serverkgg/bridge via "link:@serverkgg/bridge"');
 	});
 
-	test("refuses a file: bridge devDependency", async () => {
+	it("refuses a file: bridge devDependency", async () => {
 		writeFileSync(
 			join(root, "package.json"),
 			JSON.stringify({
@@ -86,7 +87,7 @@ describe("validate", () => {
 		await expect(validate()).rejects.toThrow("pin a published version");
 	});
 
-	test("accepts a published bridge dependency and moves on to the manifest", async () => {
+	it("accepts a published bridge dependency and moves on to the manifest", async () => {
 		writePackage({
 			"@serverkgg/bridge": "0.0.3",
 		});
@@ -94,7 +95,7 @@ describe("validate", () => {
 		await expect(validate()).rejects.toThrow("missing serverk.yml");
 	});
 
-	test("ignores a package without a bridge dependency", async () => {
+	it("ignores a package without a bridge dependency", async () => {
 		writePackage({
 			zod: "4.4.3",
 		});
@@ -102,19 +103,19 @@ describe("validate", () => {
 		await expect(validate()).rejects.toThrow("missing serverk.yml");
 	});
 
-	test("ignores a package.json that is absent", async () => {
+	it("ignores a package.json that is absent", async () => {
 		await expect(validate()).rejects.toThrow("missing serverk.yml");
 	});
 });
 
 describe("the setup report", () => {
-	test("says none for a driver that declares no setup", () => {
+	it("says none for a driver that declares no setup", () => {
 		expect(setupReportLines(undefined)).toEqual([
 			"  setup: none",
 		]);
 	});
 
-	test("names every step, its kind, whether it is required, and where it lands", () => {
+	it("names every step, its kind, whether it is required, and where it lands", () => {
 		const setup: BridgeSetupModule = {
 			kind: BridgeKind.Setup,
 			steps: [
@@ -138,8 +139,8 @@ describe("the setup report", () => {
 				},
 				{
 					kind: BridgeSetupStepKind.Open,
-					id: "txadmin",
-					title: text("txAdmin"),
+					id: "dashboard",
+					title: text("Dashboard"),
 					required: false,
 					target: {
 						tab: GuideOpenTab.Panel,
@@ -165,12 +166,12 @@ describe("the setup report", () => {
 			"  setup: 4 step(s)",
 			"    license — form, required → license/license [LICENSE_KEY]",
 			"    verify — driver, required → driver (needs the game running)",
-			'    txadmin — open, optional → panel tab "license"',
+			'    dashboard — open, optional → panel tab "license"',
 			"    invite — open, optional → access",
 		]);
 	});
 
-	test("leaves a driver step that needs nothing running with no target", () => {
+	it("leaves a driver step that needs nothing running with no target", () => {
 		expect(
 			setupReportLines({
 				kind: BridgeKind.Setup,
@@ -192,7 +193,7 @@ describe("the setup report", () => {
 		]);
 	});
 
-	test("reads an omitted required as required, the way the platform does", () => {
+	it("reads an omitted required as required, the way the platform does", () => {
 		expect(
 			setupReportLines({
 				kind: BridgeKind.Setup,
@@ -259,7 +260,7 @@ describe("the placement report", () => {
 		},
 	});
 
-	test("says nothing about a roster table that declares where it belongs", () => {
+	it("says nothing about a roster table that declares where it belongs", () => {
 		expect(
 			placementReportLines(
 				driverWithTabs([
@@ -277,7 +278,7 @@ describe("the placement report", () => {
 		]);
 	});
 
-	test("warns that the platform absorbs an unplaced players collection by convention", () => {
+	it("warns that the platform absorbs an unplaced players collection by convention", () => {
 		expect(
 			placementReportLines(
 				driverWithTabs([
@@ -295,7 +296,7 @@ describe("the placement report", () => {
 		]);
 	});
 
-	test("warns about a tab that takes the name of a platform sidebar entry", () => {
+	it("warns about a tab that takes the name of a platform sidebar entry", () => {
 		expect(
 			placementReportLines(
 				driverWithTabs([
@@ -313,7 +314,7 @@ describe("the placement report", () => {
 		]);
 	});
 
-	test("leaves a players tab alone while it is the roster tab", () => {
+	it("leaves a players tab alone while it is the roster tab", () => {
 		expect(
 			placementReportLines(
 				driverWithTabs([
@@ -331,7 +332,7 @@ describe("the placement report", () => {
 		]);
 	});
 
-	test("warns about a players tab that holds no roster", () => {
+	it("warns about a players tab that holds no roster", () => {
 		expect(
 			placementReportLines(
 				driverWithTabs([
@@ -349,7 +350,7 @@ describe("the placement report", () => {
 		]);
 	});
 
-	test("says nothing about a tab that declares no sections at all", () => {
+	it("says nothing about a tab that declares no sections at all", () => {
 		expect(
 			placementReportLines(
 				driverWithTabs([
@@ -363,7 +364,7 @@ describe("the placement report", () => {
 		).toEqual([]);
 	});
 
-	test("still warns about an empty players tab, which holds no roster", () => {
+	it("still warns about an empty players tab, which holds no roster", () => {
 		expect(
 			placementReportLines(
 				driverWithTabs([
@@ -379,7 +380,7 @@ describe("the placement report", () => {
 		]);
 	});
 
-	test("says nothing about a tab that keeps a leftover section", () => {
+	it("says nothing about a tab that keeps a leftover section", () => {
 		expect(
 			placementReportLines(
 				driverWithTabs([
@@ -493,6 +494,7 @@ describe("validating a bot package", () => {
 		writeFileSync(join(root, "assets/logo.webp"), webp(256, 256));
 		writeFileSync(join(root, "assets/banner.webp"), webp(1280, 720));
 		writeFileSync(join(root, "src/driver.ts"), BOT_DRIVER);
+		writeRules(root, readShippedRules());
 	};
 
 	const validate = () => {
@@ -521,7 +523,7 @@ describe("validating a bot package", () => {
 		});
 	});
 
-	test("validates a bot with no ports and says which kind it read", async () => {
+	it("validates a bot with no ports and says which kind it read", async () => {
 		writeBot(botManifest());
 
 		await validate();
@@ -535,7 +537,27 @@ describe("validating a bot package", () => {
 		expect(lines.some((line) => line.includes("warning"))).toBe(false);
 	});
 
-	test("says when a bot turns the environment vault on", async () => {
+	it("warns about a missing rule file and still validates", async () => {
+		writeBot(botManifest());
+		rmSync(join(root, RULES_FILE));
+
+		await validate();
+
+		expect(lines).toContain("bot-node is valid");
+		expect(lines).toContain(`  warning: ${RULES_FILE} is missing — run "serverk-bridge rules"`);
+	});
+
+	it("warns about a stale rule file and still validates", async () => {
+		writeBot(botManifest());
+		writeFileSync(join(root, RULES_FILE), "# edited by hand\n");
+
+		await validate();
+
+		expect(lines).toContain("bot-node is valid");
+		expect(lines).toContain(`  warning: ${RULES_FILE} is out of date — run "serverk-bridge rules"`);
+	});
+
+	it("says when a bot turns the environment vault on", async () => {
 		writeBot(
 			botManifest({
 				environment: {
@@ -549,7 +571,7 @@ describe("validating a bot package", () => {
 		expect(lines).toContain("  environment: vault on");
 	});
 
-	test("fails a bot whose detection table is missing", async () => {
+	it("fails a bot whose detection table is missing", async () => {
 		writeBot(botManifest(), null);
 
 		await expect(validate()).rejects.toThrow(
@@ -557,7 +579,7 @@ describe("validating a bot package", () => {
 		);
 	});
 
-	test("fails a bot whose detection table is stale", async () => {
+	it("fails a bot whose detection table is stale", async () => {
 		writeBot(botManifest(), "{}\n");
 
 		await expect(validate()).rejects.toThrow(
@@ -565,7 +587,7 @@ describe("validating a bot package", () => {
 		);
 	});
 
-	test("warns about a port on a bot but still validates it", async () => {
+	it("warns about a port on a bot but still validates it", async () => {
 		writeBot(
 			botManifest({
 				container: {
@@ -585,7 +607,7 @@ describe("validating a bot package", () => {
 		);
 	});
 
-	test("fails a bot that declares platform secrets", async () => {
+	it("fails a bot that declares platform secrets", async () => {
 		writeBot(
 			botManifest({
 				secrets: [
@@ -602,3 +624,75 @@ describe("validating a bot package", () => {
 		);
 	});
 });
+
+describe("availability report", () => {
+	const driverWithModules = (modules: BridgeDriver["modules"]): BridgeDriver => ({
+		install: {
+			kind: BridgeKind.Install,
+			run: async () => {
+				return;
+			},
+		},
+		lifecycle: {
+			kind: BridgeKind.Lifecycle,
+			command: async () => [
+				"run",
+			],
+		},
+		modules,
+	});
+
+	it("warns about a protected module that cannot say when its action is unavailable", () => {
+		expect(
+			availabilityReportLines(
+				driverWithModules({
+					worlds: {
+						kind: BridgeKind.Collection,
+						protectedActions: [
+							"delete",
+						],
+						list: async () => [],
+						actions: {
+							delete: async () => {
+								return;
+							},
+						},
+					},
+				}),
+			),
+		).toEqual([
+			'  warning: module "worlds" protects delete but declares no `unavailable` — the panel offers the action everywhere and the platform stops the game before the driver can refuse it',
+		]);
+	});
+
+	it("says nothing when the protected module declares the hook or protects nothing", () => {
+		expect(
+			availabilityReportLines(
+				driverWithModules({
+					worlds: {
+						kind: BridgeKind.Collection,
+						protectedActions: [
+							"delete",
+						],
+						list: async () => [],
+						actions: {
+							delete: async () => {
+								return;
+							},
+						},
+						unavailable: async () => ({}),
+					},
+					players: {
+						kind: BridgeKind.Collection,
+						list: async () => [],
+						actions: {
+							kick: async () => {
+								return;
+							},
+						},
+					},
+				}),
+			),
+		).toEqual([]);
+	});
+});

package/src/cli/validate.ts

@@ -5,7 +5,7 @@ import { parseArgs } from "node:util";
 import { z } from "zod";
 import { type ArticlePack, compileArticles } from "../blog";
 import { DETECT_TABLE_FILE, renderDetectTable } from "../bot/detect";
-import { PortProtocol } from "../core";
+import { BridgeKind, PortProtocol } from "../core";
 import { GuideOpenTab, type GuideOpenTarget, type GuidePack, readGuidePack, readGuidePackage } from "../guides";
 import {
 	declaredFiles,
@@ -31,6 +31,7 @@ import {
 } from "../protocol";
 import { validateDriver, validatePortActivation } from "../runtime/bridgeDriver";
 import { directorySource } from "./directorySource";
+import { rulesReportLines } from "./rules";
 
 const HELP = `serverk-bridge validate [options]
 
@@ -279,6 +280,39 @@ const reportPlacement = (driver: BridgeDriver) => {
 	}
 };
 
+export const availabilityReportLines = (driver: BridgeDriver): string[] => {
+	const lines: string[] = [];
+
+	for (const [id, module] of Object.entries(driver.modules ?? {})) {
+		if (
+			module.kind !== BridgeKind.Actions
+			&& module.kind !== BridgeKind.Catalog
+			&& module.kind !== BridgeKind.Collection
+			&& module.kind !== BridgeKind.Settings
+		) {
+			continue;
+		}
+
+		const protectedActions = module.protectedActions ?? [];
+
+		if (protectedActions.length === 0 || module.unavailable) {
+			continue;
+		}
+
+		lines.push(
+			`  warning: module "${id}" protects ${protectedActions.join(", ")} but declares no \`unavailable\` — the panel offers the action everywhere and the platform stops the game before the driver can refuse it`,
+		);
+	}
+
+	return lines;
+};
+
+const reportAvailability = (driver: BridgeDriver) => {
+	for (const line of availabilityReportLines(driver)) {
+		console.log(line);
+	}
+};
+
 const openTargetOf = (target: GuideOpenTarget) => {
 	if (target.tab === GuideOpenTab.Panel) {
 		return `panel tab "${target.tabId}"`;
@@ -413,6 +447,7 @@ const validateRunnablePackage = async (
 		...botPortLines(manifest),
 		...botEnvironmentLines(manifest),
 		...detectLines,
+		...rulesReportLines(dir),
 	]) {
 		console.log(line);
 	}
@@ -432,6 +467,7 @@ const validateRunnablePackage = async (
 	reportEvents(driver);
 	reportActionTargets(driver);
 	reportPlacement(driver);
+	reportAvailability(driver);
 };
 
 export const runValidate = async (args: string[]) => {

package/src/codecs/ini.test.ts

@@ -1,4 +1,4 @@
-import { describe, expect, test } from "bun:test";
+import { describe, expect, it } from "bun:test";
 import { mergeIni, parseIni } from "./ini";
 
 const SAMPLE = [
@@ -14,38 +14,38 @@ const SAMPLE = [
 ].join("\n");
 
 describe("parseIni", () => {
-	test("reads the keys of one section", () => {
+	it("reads the keys of one section", () => {
 		expect(parseIni(SAMPLE, "Server")).toEqual({
 			name: "serverk",
 			maxPlayers: "10",
 		});
 	});
 
-	test("reads another section without leaking keys", () => {
+	it("reads another section without leaking keys", () => {
 		expect(parseIni(SAMPLE, "Admin")).toEqual({
 			password: "secret",
 		});
 	});
 
-	test("reads the keys before the first header when no section is asked for", () => {
+	it("reads the keys before the first header when no section is asked for", () => {
 		expect(parseIni("welcome=hello\n[Server]\nname=x\n")).toEqual({
 			welcome: "hello",
 		});
 	});
 
-	test("keeps the last of a repeated key", () => {
+	it("keeps the last of a repeated key", () => {
 		expect(parseIni("[S]\nmod=a\nmod=b\n", "S")).toEqual({
 			mod: "b",
 		});
 	});
 
-	test("returns nothing for an unknown section", () => {
+	it("returns nothing for an unknown section", () => {
 		expect(parseIni(SAMPLE, "Nothing")).toEqual({});
 	});
 });
 
 describe("mergeIni", () => {
-	test("rewrites a value in place and keeps comments and order", () => {
+	it("rewrites a value in place and keeps comments and order", () => {
 		expect(
 			mergeIni(
 				SAMPLE,
@@ -69,7 +69,7 @@ describe("mergeIni", () => {
 		);
 	});
 
-	test("appends an unknown key at the end of its section", () => {
+	it("appends an unknown key at the end of its section", () => {
 		expect(
 			mergeIni(
 				SAMPLE,
@@ -93,7 +93,7 @@ describe("mergeIni", () => {
 		);
 	});
 
-	test("creates a section that does not exist yet", () => {
+	it("creates a section that does not exist yet", () => {
 		expect(
 			mergeIni(
 				"[Server]\nname=x\n",
@@ -105,7 +105,7 @@ describe("mergeIni", () => {
 		).toBe("[Server]\nname=x\n\n[Game]\nlevel=3\n");
 	});
 
-	test("drops repeated occurrences of a key it rewrites", () => {
+	it("drops repeated occurrences of a key it rewrites", () => {
 		expect(
 			mergeIni(
 				"[S]\nmod=a\nmod=b\nother=1\n",
@@ -117,7 +117,7 @@ describe("mergeIni", () => {
 		).toBe("[S]\nmod=c\nother=1\n");
 	});
 
-	test("writes into an empty file", () => {
+	it("writes into an empty file", () => {
 		expect(
 			mergeIni(
 				"",
@@ -129,7 +129,7 @@ describe("mergeIni", () => {
 		).toBe("[Server]\nname=x\n");
 	});
 
-	test("writes preamble keys when no section is asked for", () => {
+	it("writes preamble keys when no section is asked for", () => {
 		expect(
 			mergeIni("welcome=hello\n[Server]\nname=x\n", {
 				welcome: "salam",
@@ -138,7 +138,7 @@ describe("mergeIni", () => {
 		).toBe("welcome=salam\nextra=1\n[Server]\nname=x\n");
 	});
 
-	test("leaves other sections untouched", () => {
+	it("leaves other sections untouched", () => {
 		expect(
 			mergeIni(
 				"[A]\nkey=1\n[B]\nkey=2\n",

package/src/codecs/javaProperties.test.ts

@@ -1,33 +1,33 @@
-import { describe, expect, test } from "bun:test";
+import { describe, expect, it } from "bun:test";
 import { escapeJavaProperty, mergeJavaProperties, parseJavaProperties, unescapeJavaProperty } from "./javaProperties";
 
 const ARABIC = "سيرفرك";
 
 describe("parseJavaProperties", () => {
-	test("decodes the unicode escapes the java server writes", () => {
+	it("decodes the unicode escapes the java server writes", () => {
 		expect(parseJavaProperties("motd=\\u0633\\u064A\\u0631\\u0641\\u0631\\u0643\n")).toEqual({
 			motd: ARABIC,
 		});
 	});
 
-	test("decodes backslash escapes in values", () => {
+	it("decodes backslash escapes in values", () => {
 		expect(parseJavaProperties("resource-pack=https\\://example.com/pack.zip\nmotd=a\\tb\\\\c\n")).toEqual({
 			"resource-pack": "https://example.com/pack.zip",
 			motd: "a\tb\\c",
 		});
 	});
 
-	test("repairs a value the server stored as latin-1 bytes of utf-8", () => {
+	it("repairs a value the server stored as latin-1 bytes of utf-8", () => {
 		expect(parseJavaProperties("motd=ddss \\u00D9\\u008A\\u00D8\\u00AA\n")).toEqual({
 			motd: "ddss يت",
 		});
 	});
 
-	test("keeps plain latin-1 text that is not utf-8", () => {
+	it("keeps plain latin-1 text that is not utf-8", () => {
 		expect(unescapeJavaProperty("caf\\u00E9")).toBe("café");
 	});
 
-	test("keeps comments and plain values", () => {
+	it("keeps comments and plain values", () => {
 		expect(parseJavaProperties("#Minecraft server properties\nmax-players=20\n")).toEqual({
 			"max-players": "20",
 		});
@@ -35,7 +35,7 @@ describe("parseJavaProperties", () => {
 });
 
 describe("mergeJavaProperties", () => {
-	test("writes non ascii text as unicode escapes", () => {
+	it("writes non ascii text as unicode escapes", () => {
 		expect(
 			mergeJavaProperties("motd=A Minecraft Server\nmax-players=20\n", {
 				motd: ARABIC,
@@ -43,21 +43,21 @@ describe("mergeJavaProperties", () => {
 		).toBe("motd=\\u0633\\u064A\\u0631\\u0641\\u0631\\u0643\nmax-players=20\n");
 	});
 
-	test("escapes the characters java treats as separators", () => {
+	it("escapes the characters java treats as separators", () => {
 		expect(escapeJavaProperty("https://a.b#c=d!e")).toBe("https\\://a.b\\#c\\=d\\!e");
 	});
 
-	test("escapes a leading space and control characters", () => {
+	it("escapes a leading space and control characters", () => {
 		expect(escapeJavaProperty(" a\tb\\c")).toBe("\\ a\\tb\\\\c");
 	});
 
-	test("round trips astral characters", () => {
+	it("round trips astral characters", () => {
 		const value = "hi 🎮";
 
 		expect(unescapeJavaProperty(escapeJavaProperty(value))).toBe(value);
 	});
 
-	test("appends a key the file does not have yet", () => {
+	it("appends a key the file does not have yet", () => {
 		expect(
 			mergeJavaProperties("", {
 				motd: "hello",

package/src/codecs/sourceCfg.test.ts

@@ -1,4 +1,4 @@
-import { describe, expect, test } from "bun:test";
+import { describe, expect, it } from "bun:test";
 import {
 	decodeSourceCfgValue,
 	encodeSourceCfgValue,
@@ -8,21 +8,21 @@ import {
 } from "./sourceCfg";
 
 describe("readSourceCfgEntry", () => {
-	test("splits a key from its value", () => {
+	it("splits a key from its value", () => {
 		expect(readSourceCfgEntry('hostname "serverk"')).toEqual({
 			key: "hostname",
 			value: '"serverk"',
 		});
 	});
 
-	test("accepts a key with no value", () => {
+	it("accepts a key with no value", () => {
 		expect(readSourceCfgEntry("sv_lan")).toEqual({
 			key: "sv_lan",
 			value: "",
 		});
 	});
 
-	test("ignores comments and blank lines", () => {
+	it("ignores comments and blank lines", () => {
 		expect(readSourceCfgEntry("// a comment")).toBeNull();
 		expect(readSourceCfgEntry("# a comment")).toBeNull();
 		expect(readSourceCfgEntry("   ")).toBeNull();
@@ -31,32 +31,32 @@ describe("readSourceCfgEntry", () => {
 });
 
 describe("decodeSourceCfgValue", () => {
-	test("unquotes a string and unescapes it", () => {
+	it("unquotes a string and unescapes it", () => {
 		expect(decodeSourceCfgValue('"a \\"b\\" c"')).toBe('a "b" c');
 	});
 
-	test("reads booleans and numbers", () => {
+	it("reads booleans and numbers", () => {
 		expect(decodeSourceCfgValue("true")).toBe(true);
 		expect(decodeSourceCfgValue("false")).toBe(false);
 		expect(decodeSourceCfgValue("42")).toBe(42);
 		expect(decodeSourceCfgValue("-1.5")).toBe(-1.5);
 	});
 
-	test("leaves a bare word alone", () => {
+	it("leaves a bare word alone", () => {
 		expect(decodeSourceCfgValue("de_dust2")).toBe("de_dust2");
 	});
 });
 
 describe("encodeSourceCfgValue", () => {
-	test("quotes and escapes text", () => {
+	it("quotes and escapes text", () => {
 		expect(encodeSourceCfgValue('a "b"')).toBe('"a \\"b\\""');
 	});
 
-	test("flattens newlines", () => {
+	it("flattens newlines", () => {
 		expect(encodeSourceCfgValue("a\nb")).toBe('"a b"');
 	});
 
-	test("writes booleans and numbers bare", () => {
+	it("writes booleans and numbers bare", () => {
 		expect(encodeSourceCfgValue(true)).toBe("true");
 		expect(encodeSourceCfgValue(7)).toBe("7");
 		expect(encodeSourceCfgValue(null)).toBe('""');
@@ -64,7 +64,7 @@ describe("encodeSourceCfgValue", () => {
 });
 
 describe("parseSourceCfg", () => {
-	test("reads a source cfg file", () => {
+	it("reads a source cfg file", () => {
 		expect(parseSourceCfg('// header\nhostname "serverk"\nsv_maxplayers 24\nsv_cheats false\n')).toEqual({
 			hostname: "serverk",
 			sv_maxplayers: 24,
@@ -74,7 +74,7 @@ describe("parseSourceCfg", () => {
 });
 
 describe("mergeSourceCfg", () => {
-	test("rewrites a value and keeps the comments", () => {
+	it("rewrites a value and keeps the comments", () => {
 		expect(
 			mergeSourceCfg('// header\nhostname "old"\nsv_maxplayers 24\n', {
 				hostname: "new",
@@ -82,7 +82,7 @@ describe("mergeSourceCfg", () => {
 		).toBe('// header\nhostname "new"\nsv_maxplayers 24\n');
 	});
 
-	test("appends an unknown key", () => {
+	it("appends an unknown key", () => {
 		expect(
 			mergeSourceCfg('hostname "x"\n', {
 				rcon_password: "abc",
@@ -90,7 +90,7 @@ describe("mergeSourceCfg", () => {
 		).toBe('hostname "x"\nrcon_password "abc"\n');
 	});
 
-	test("collapses a repeated key it rewrites", () => {
+	it("collapses a repeated key it rewrites", () => {
 		expect(
 			mergeSourceCfg("map de_dust2\nmap de_nuke\n", {
 				map: "de_inferno",