@serverkgg/bridge 0.1.2 → 0.1.4

package/src/rcon/rconWriter.test.ts

@@ -1,4 +1,4 @@
-import { describe, expect, test } from "bun:test";
+import { describe, expect, it } from "bun:test";
 import { createRconWriter } from "./rconWriter";
 
 const bytes = (...values: number[]) => new Uint8Array(values);
@@ -17,7 +17,7 @@ const recorder = (accept: (writes: number) => number | null) => {
 };
 
 describe("createRconWriter", () => {
-	test("reports a full write and keeps nothing pending", () => {
+	it("reports a full write and keeps nothing pending", () => {
 		const socket = recorder(() => null);
 		const writer = createRconWriter(socket.write);
 
@@ -35,7 +35,7 @@ describe("createRconWriter", () => {
 		expect(writer.dead).toBe(false);
 	});
 
-	test("buffers the remainder of a short write and finishes it on drain", () => {
+	it("buffers the remainder of a short write and finishes it on drain", () => {
 		const socket = recorder((writes) => (writes === 1 ? 2 : null));
 		const writer = createRconWriter(socket.write);
 
@@ -57,7 +57,7 @@ describe("createRconWriter", () => {
 		]);
 	});
 
-	test("appends a send made while a remainder is pending", () => {
+	it("appends a send made while a remainder is pending", () => {
 		const socket = recorder((writes) => (writes === 1 ? 1 : null));
 		const writer = createRconWriter(socket.write);
 
@@ -74,7 +74,7 @@ describe("createRconWriter", () => {
 		]);
 	});
 
-	test("reports zero accepted and dies when the socket answers minus one", () => {
+	it("reports zero accepted and dies when the socket answers minus one", () => {
 		const socket = recorder(() => -1);
 		const writer = createRconWriter(socket.write);
 
@@ -84,7 +84,7 @@ describe("createRconWriter", () => {
 		expect(socket.writes).toHaveLength(1);
 	});
 
-	test("does not write on a drain with an empty buffer", () => {
+	it("does not write on a drain with an empty buffer", () => {
 		const socket = recorder(() => null);
 		const writer = createRconWriter(socket.write);
 

package/src/runtime/bridgeActivity.test.ts

@@ -1,4 +1,4 @@
-import { describe, expect, test } from "bun:test";
+import { describe, expect, it } from "bun:test";
 import type { BridgeActivity, BridgeValues } from "../core";
 import {
 	BRIDGE_ACTIVITY_DETAILS_LIMIT,
@@ -69,14 +69,14 @@ const granted = (subject = "Steve"): BridgeActivity => ({
 });
 
 describe("normalizeActivity", () => {
-	test("keeps a well-formed entry as it is", () => {
+	it("keeps a well-formed entry as it is", () => {
 		const { options, warnings } = recorder();
 
 		expect(normalizeActivity(granted(), options)).toEqual(granted());
 		expect(warnings).toEqual([]);
 	});
 
-	test("drops a name the protocol does not know, with a warning", () => {
+	it("drops a name the protocol does not know, with a warning", () => {
 		const { options, warnings } = recorder();
 
 		expect(
@@ -99,7 +99,7 @@ describe("normalizeActivity", () => {
 		]);
 	});
 
-	test("drops an entry whose text is empty in one language", () => {
+	it("drops an entry whose text is empty in one language", () => {
 		const { options } = recorder();
 
 		expect(
@@ -116,7 +116,7 @@ describe("normalizeActivity", () => {
 		).toBeNull();
 	});
 
-	test("cuts the text and the subject to their limits", () => {
+	it("cuts the text and the subject to their limits", () => {
 		const { options, warnings } = recorder();
 
 		const normalized = normalizeActivity(
@@ -140,7 +140,7 @@ describe("normalizeActivity", () => {
 		]);
 	});
 
-	test("keeps at most sixteen details", () => {
+	it("keeps at most sixteen details", () => {
 		const { options } = recorder();
 
 		const details: BridgeValues = {};
@@ -160,7 +160,7 @@ describe("normalizeActivity", () => {
 		expect(Object.keys(normalized?.details ?? {})).toHaveLength(BRIDGE_ACTIVITY_DETAILS_LIMIT);
 	});
 
-	test("removes secret fields, reserved keys and values that are not scalars", () => {
+	it("removes secret fields, reserved keys and values that are not scalars", () => {
 		const { options } = recorder();
 
 		const normalized = normalizeActivity(
@@ -185,7 +185,7 @@ describe("normalizeActivity", () => {
 		});
 	});
 
-	test("blanks a secret value in the text, the subject and the details", () => {
+	it("blanks a secret value in the text, the subject and the details", () => {
 		const { options } = recorder();
 
 		const normalized = normalizeActivity(
@@ -211,7 +211,7 @@ describe("normalizeActivity", () => {
 });
 
 describe("createActivityReporter", () => {
-	test("emits the entry as an event carrying the text under the reserved keys", () => {
+	it("emits the entry as an event carrying the text under the reserved keys", () => {
 		const { reporter, emitted } = recorder();
 
 		reporter.report({
@@ -234,7 +234,7 @@ describe("createActivityReporter", () => {
 		]);
 	});
 
-	test("emits nothing for an entry normalisation drops", () => {
+	it("emits nothing for an entry normalisation drops", () => {
 		const { reporter, emitted } = recorder();
 
 		reporter.report({
@@ -247,7 +247,7 @@ describe("createActivityReporter", () => {
 });
 
 describe("createActivityScope", () => {
-	test("collects entries for the answer instead of emitting them", () => {
+	it("collects entries for the answer instead of emitting them", () => {
 		const { options, reporter, emitted } = recorder();
 		const scope = createActivityScope(reporter, options);
 
@@ -262,7 +262,7 @@ describe("createActivityScope", () => {
 		expect(emitted).toEqual([]);
 	});
 
-	test("drains what it collected to the ring when the handler throws", () => {
+	it("drains what it collected to the ring when the handler throws", () => {
 		const { options, reporter, emitted } = recorder();
 		const scope = createActivityScope(reporter, options);
 
@@ -275,7 +275,7 @@ describe("createActivityScope", () => {
 		expect(scope.close()).toEqual([]);
 	});
 
-	test("sends a report made after the answer to the ring", () => {
+	it("sends a report made after the answer to the ring", () => {
 		const { options, reporter, emitted } = recorder();
 		const scope = createActivityScope(reporter, options);
 
@@ -288,7 +288,7 @@ describe("createActivityScope", () => {
 		]);
 	});
 
-	test("keeps at most eight entries on one answer", () => {
+	it("keeps at most eight entries on one answer", () => {
 		const { options, reporter, warnings } = recorder();
 		const scope = createActivityScope(reporter, options);
 
@@ -300,7 +300,7 @@ describe("createActivityScope", () => {
 		expect(warnings.filter((warning) => warning.fields?.reason === "extra-entries")).toHaveLength(2);
 	});
 
-	test("never throws into the caller, whatever the entry holds", () => {
+	it("never throws into the caller, whatever the entry holds", () => {
 		const { options, reporter, warnings } = recorder();
 		const scope = createActivityScope(reporter, options);
 

package/src/runtime/bridgeBackupLease.test.ts

@@ -1,8 +1,8 @@
-import { describe, expect, test } from "bun:test";
+import { describe, expect, it } from "bun:test";
 import { createBackupLease } from "./bridgeBackupLease";
 
 describe("backup save ownership", () => {
-	test("keeps a long backup frozen while renewed and releases when its owner disappears", async () => {
+	it("keeps a long backup frozen while renewed and releases when its owner disappears", async () => {
 		let now = 0;
 		let releases = 0;
 		const lease = createBackupLease(
@@ -25,7 +25,7 @@ describe("backup save ownership", () => {
 		expect(releases).toBe(1);
 		expect(lease.active).toBe(false);
 	});
-	test("refuses overlapping holds and retains ownership after a failed resume", async () => {
+	it("refuses overlapping holds and retains ownership after a failed resume", async () => {
 		let fail = true;
 		const lease = createBackupLease(async () => {
 			if (fail) {
@@ -43,7 +43,7 @@ describe("backup save ownership", () => {
 	});
 });
 
-test("reserves ownership while the quiesce operation is still awaiting confirmation", async () => {
+it("reserves ownership while the quiesce operation is still awaiting confirmation", async () => {
 	const prepared = Promise.withResolvers<void>();
 	const lease = createBackupLease(async () => {});
 	const owner = lease.begin();
@@ -58,7 +58,7 @@ test("reserves ownership while the quiesce operation is still awaiting confirmat
 	await lease.finish(owner);
 });
 
-test("expiration waits for in-flight preparation before resuming saves", async () => {
+it("expiration waits for in-flight preparation before resuming saves", async () => {
 	let now = 0;
 	let releases = 0;
 	const prepared = Promise.withResolvers<void>();
@@ -83,7 +83,7 @@ test("expiration waits for in-flight preparation before resuming saves", async (
 	expect(lease.active).toBe(false);
 });
 
-test("late releases and tokenless callers cannot release a newer owner", async () => {
+it("late releases and tokenless callers cannot release a newer owner", async () => {
 	let releases = 0;
 	const lease = createBackupLease(async () => {
 		releases++;
@@ -99,7 +99,7 @@ test("late releases and tokenless callers cannot release a newer owner", async (
 	expect(releases).toBe(2);
 });
 
-test("legacy release is accepted only for a legacy hold", async () => {
+it("legacy release is accepted only for a legacy hold", async () => {
 	let releases = 0;
 	const lease = createBackupLease(async () => {
 		releases++;
@@ -111,7 +111,7 @@ test("legacy release is accepted only for a legacy hold", async () => {
 	expect(releases).toBe(1);
 });
 
-test("failed quiesce cleanup keeps the owner until resume succeeds", async () => {
+it("failed quiesce cleanup keeps the owner until resume succeeds", async () => {
 	let releases = 0;
 	const lease = createBackupLease(async () => {
 		releases++;
@@ -127,7 +127,7 @@ test("failed quiesce cleanup keeps the owner until resume succeeds", async () =>
 	expect(lease.active).toBe(false);
 });
 
-test("legacy owners retain the historical hold until explicit release without renewal", async () => {
+it("legacy owners retain the historical hold until explicit release without renewal", async () => {
 	let now = 0;
 	let releases = 0;
 	const lease = createBackupLease(

package/src/runtime/bridgeConfigReader.test.ts

@@ -1,4 +1,4 @@
-import { describe, expect, test } from "bun:test";
+import { describe, expect, it } from "bun:test";
 import { BridgeConfigError, parseConfig } from "./bridgeConfigReader";
 
 const written = (extra: Record<string, unknown> = {}) => {
@@ -22,7 +22,7 @@ const written = (extra: Record<string, unknown> = {}) => {
 };
 
 describe("parseConfig", () => {
-	test("reads a file an older agent wrote, with no public address and no port counts", () => {
+	it("reads a file an older agent wrote, with no public address and no port counts", () => {
 		const config = parseConfig(written());
 
 		expect(config.publicAddress).toBeUndefined();
@@ -33,7 +33,7 @@ describe("parseConfig", () => {
 		});
 	});
 
-	test("reads the public address and the port counts when the agent wrote them", () => {
+	it("reads the public address and the port counts when the agent wrote them", () => {
 		const config = parseConfig(
 			written({
 				publicAddress: "203.0.113.7",
@@ -51,7 +51,7 @@ describe("parseConfig", () => {
 		});
 	});
 
-	test("ignores a malformed public address and drops non-numeric counts", () => {
+	it("ignores a malformed public address and drops non-numeric counts", () => {
 		const config = parseConfig(
 			written({
 				publicAddress: 7,
@@ -68,7 +68,7 @@ describe("parseConfig", () => {
 		});
 	});
 
-	test("still refuses a game that would run as root", () => {
+	it("still refuses a game that would run as root", () => {
 		expect(() =>
 			parseConfig(
 				written({

package/src/runtime/bridgeConsoleHandler.test.ts

@@ -1,4 +1,4 @@
-import { describe, expect, test } from "bun:test";
+import { describe, expect, it } from "bun:test";
 import type { BridgeCommandOptions, BridgeContext, BridgeReply } from "../context";
 import { BridgeKind, BridgeUserError } from "../core";
 import type { BridgeDriver, BridgeTerminalModule } from "../modules";
@@ -60,7 +60,7 @@ const handlerFor = (driver: BridgeDriver) => {
 };
 
 describe("routing a console line to the driver before the game's stdin", () => {
-	test("answers with the driver's reply when the driver handles the line", async () => {
+	it("answers with the driver's reply when the driver handles the line", async () => {
 		const { handler, written } = handlerFor(
 			driverWith(
 				terminalWith(async (_context, input) => {
@@ -81,7 +81,7 @@ describe("routing a console line to the driver before the game's stdin", () => {
 		expect(written).toEqual([]);
 	});
 
-	test("hands the driver the same line the caller typed", async () => {
+	it("hands the driver the same line the caller typed", async () => {
 		const seen: string[] = [];
 
 		const { handler } = handlerFor(
@@ -105,7 +105,7 @@ describe("routing a console line to the driver before the game's stdin", () => {
 		]);
 	});
 
-	test("writes the line to stdin when the driver answers null", async () => {
+	it("writes the line to stdin when the driver answers null", async () => {
 		const { handler, written } = handlerFor(
 			driverWith(
 				terminalWith(async () => {
@@ -124,7 +124,7 @@ describe("routing a console line to the driver before the game's stdin", () => {
 		]);
 	});
 
-	test("keeps the expect and timeout of a line the driver leaves alone", async () => {
+	it("keeps the expect and timeout of a line the driver leaves alone", async () => {
 		const { handler } = handlerFor(
 			driverWith(
 				terminalWith(async () => {
@@ -143,7 +143,7 @@ describe("routing a console line to the driver before the game's stdin", () => {
 		).toBe("matched");
 	});
 
-	test("writes the line to stdin when the driver declares no handler", async () => {
+	it("writes the line to stdin when the driver declares no handler", async () => {
 		const { handler, written } = handlerFor(
 			driverWith({
 				kind: BridgeKind.Terminal,
@@ -158,7 +158,7 @@ describe("routing a console line to the driver before the game's stdin", () => {
 		]);
 	});
 
-	test("writes the line to stdin when the game declares no terminal at all", async () => {
+	it("writes the line to stdin when the game declares no terminal at all", async () => {
 		const { handler, written } = handlerFor(driverWith(undefined));
 
 		await handler("list", {});
@@ -168,7 +168,7 @@ describe("routing a console line to the driver before the game's stdin", () => {
 		]);
 	});
 
-	test("refuses the line with the driver's own text when the driver rejects it", async () => {
+	it("refuses the line with the driver's own text when the driver rejects it", async () => {
 		const refusal = {
 			ar: "هذا الأمر يشتغل داخل اللعبة بس.",
 			en: "That command only works inside the game.",

package/src/runtime/bridgeContext.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, writeFile } from "node:fs/promises";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
@@ -38,7 +38,7 @@ describe("the context's variable maps", () => {
 		});
 	});
 
-	test("hand a driver every variable and secret the agent wrote, and follow a rewrite", async () => {
+	it("hand a driver every variable and secret the agent wrote, and follow a rewrite", async () => {
 		const boot = configWith(
 			{
 				MY_VAR: "1",
@@ -185,7 +185,7 @@ describe("the context's ports and address", () => {
 		});
 	});
 
-	test("hand a driver the block the agent published and its public address", async () => {
+	it("hand a driver the block the agent published and its public address", async () => {
 		const boot = configWith({
 			portCounts: {
 				game: 1,
@@ -209,7 +209,7 @@ describe("the context's ports and address", () => {
 		expect(context.server.publicAddress).toBe("203.0.113.7");
 	});
 
-	test("treat every port as a single one and the address as unknown for a file an older agent wrote", async () => {
+	it("treat every port as a single one and the address as unknown for a file an older agent wrote", async () => {
 		const boot = configWith({});
 
 		await writeFile(path, JSON.stringify(boot));
@@ -224,7 +224,7 @@ describe("the context's ports and address", () => {
 		expect(() => context.portBlock("query")).toThrow('this game declares no port named "query"');
 	});
 
-	test("follow a rewrite of the public address", async () => {
+	it("follow a rewrite of the public address", async () => {
 		const boot = configWith({});
 
 		await writeFile(path, JSON.stringify(boot));