@serverkgg/bridge 0.1.2 → 0.1.3

package/src/runtime/bridgeSupervisor.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 { tmpdir } from "node:os";
 import { BridgeRuntime } from "../core";
 import { BridgeLifecycleSubject, BridgeStopSignal } from "../modules";
@@ -230,7 +230,7 @@ const wrapperArgv = () => {
 };
 
 describe("createSupervisor", () => {
-	test("runs the game outside the bridge's own process group", async () => {
+	it("runs the game outside the bridge's own process group", async () => {
 		const { supervisor, pid } = await startWrapper(WRAPPER, 1);
 
 		try {
@@ -247,7 +247,7 @@ describe("createSupervisor", () => {
 		expect(await waitGone(pid, 5000)).toBe(true);
 	}, 20_000);
 
-	test("takes the wrapper's child down with it when the stop times out", async () => {
+	it("takes the wrapper's child down with it when the stop times out", async () => {
 		const { supervisor, pid } = await startWrapper(WRAPPER, 1);
 
 		await supervisor.stop();
@@ -256,7 +256,7 @@ describe("createSupervisor", () => {
 		expect(await waitGone(pid, 5000), "the game was orphaned when the wrapper was signalled").toBe(true);
 	}, 20_000);
 
-	test("kills the wrapper's child through the escalation when the wrapper ignores SIGTERM", async () => {
+	it("kills the wrapper's child through the escalation when the wrapper ignores SIGTERM", async () => {
 		const { supervisor, pid } = await startWrapper(STUBBORN_WRAPPER, 1);
 
 		await supervisor.stop();
@@ -265,7 +265,7 @@ describe("createSupervisor", () => {
 		expect(await waitGone(pid, 5000), "the game outlived the SIGKILL escalation").toBe(true);
 	}, 30_000);
 
-	test("does not restart a game that exited before it ever reached its ready line", async () => {
+	it("does not restart a game that exited before it ever reached its ready line", async () => {
 		const { supervisor, records } = crashSupervisor(() => BOOT_CRASH, NEVER_READY);
 
 		await supervisor.start();
@@ -289,7 +289,7 @@ describe("createSupervisor", () => {
 		expect(supervisor.restarts).toBe(0);
 	}, 30_000);
 
-	test("restarts a game that crashed after it reached its ready line", async () => {
+	it("restarts a game that crashed after it reached its ready line", async () => {
 		const { supervisor, records } = crashSupervisor(() => RUNNING_CRASH, READY);
 
 		await supervisor.start();
@@ -307,7 +307,7 @@ describe("createSupervisor", () => {
 		await supervisor.stop();
 	}, 30_000);
 
-	test("flags a game killed with SIGKILL as out of memory", async () => {
+	it("flags a game killed with SIGKILL as out of memory", async () => {
 		const { supervisor, records } = crashSupervisor(() => OOM_CRASH, NEVER_READY);
 
 		await supervisor.start();
@@ -323,7 +323,7 @@ describe("createSupervisor", () => {
 		await supervisor.stop();
 	}, 30_000);
 
-	test("trips the breaker on the third running crash inside the window", async () => {
+	it("trips the breaker on the third running crash inside the window", async () => {
 		const { supervisor, records } = crashSupervisor(() => RUNNING_CRASH, READY);
 
 		await supervisor.start();
@@ -346,7 +346,7 @@ describe("createSupervisor", () => {
 		await supervisor.stop();
 	}, 40_000);
 
-	test("launches again when start is asked after a booting crash", async () => {
+	it("launches again when start is asked after a booting crash", async () => {
 		let script = BOOT_CRASH;
 
 		const { supervisor, records } = crashSupervisor(() => script, NEVER_READY);
@@ -367,7 +367,7 @@ describe("createSupervisor", () => {
 		expect(supervisor.runtime).toBe(BridgeRuntime.Stopped);
 	}, 30_000);
 
-	test("carries the last line the game printed into the crash record", async () => {
+	it("carries the last line the game printed into the crash record", async () => {
 		const { supervisor, records } = crashSupervisor(() => LAST_LINE_CRASH, NEVER_READY);
 
 		await supervisor.start();
@@ -376,7 +376,7 @@ describe("createSupervisor", () => {
 		expect(records.at(0)?.lastLine).toBe(LAST_LINE);
 	}, 30_000);
 
-	test("carries the last line even when the game exits with no pause after printing it", async () => {
+	it("carries the last line even when the game exits with no pause after printing it", async () => {
 		const { supervisor, records } = crashSupervisor(() => TIGHT_LAST_LINE_CRASH, NEVER_READY);
 
 		await supervisor.start();
@@ -386,7 +386,7 @@ describe("createSupervisor", () => {
 		expect(records.at(0)?.exitCode).toBe(3);
 	}, 30_000);
 
-	test("leaves no dead child behind when the game exits the instant it launches", async () => {
+	it("leaves no dead child behind when the game exits the instant it launches", async () => {
 		let adopted = 0;
 
 		for (let attempt = 0; attempt < 40 && adopted < 20; attempt += 1) {
@@ -421,7 +421,7 @@ describe("createSupervisor", () => {
 		expect(adopted, "the game never survived long enough to be adopted as the child").toBeGreaterThan(0);
 	}, 30_000);
 
-	test("treats a run that outlived the stable window as running even when the ready line never came", async () => {
+	it("treats a run that outlived the stable window as running even when the ready line never came", async () => {
 		const { supervisor, records, stables } = timedSupervisor(() => LATE_CRASH, NEVER_READY, 300);
 
 		await supervisor.start();
@@ -438,7 +438,7 @@ describe("createSupervisor", () => {
 		await supervisor.stop();
 	}, 30_000);
 
-	test("keeps a slow boot starting past the stable window when the driver declares a ready timeout", async () => {
+	it("keeps a slow boot starting past the stable window when the driver declares a ready timeout", async () => {
 		const { supervisor, records, stables } = timedSupervisor(() => LATE_CRASH, NEVER_READY, 300, 30);
 
 		await supervisor.start();
@@ -457,7 +457,7 @@ describe("createSupervisor", () => {
 		expect(stables()).toBe(0);
 	}, 30_000);
 
-	test("fails the boot when the declared ready timeout passes, never flipping it to running", async () => {
+	it("fails the boot when the declared ready timeout passes, never flipping it to running", async () => {
 		const { supervisor, records, stables } = timedSupervisor(() => "exec sleep 300", NEVER_READY, 60_000, 1);
 
 		await supervisor.start();
@@ -476,7 +476,7 @@ describe("createSupervisor", () => {
 		expect(stables()).toBe(0);
 	}, 30_000);
 
-	test("still treats a game with no declared ready timeout as running once the default wait passes", async () => {
+	it("still treats a game with no declared ready timeout as running once the default wait passes", async () => {
 		const { supervisor, stables } = timedSupervisor(() => "exec sleep 300", NEVER_READY, 60_000);
 
 		await supervisor.start();
@@ -488,7 +488,7 @@ describe("createSupervisor", () => {
 		await supervisor.stop();
 	}, 30_000);
 
-	test("opens a new episode when start is asked after the breaker tripped", async () => {
+	it("opens a new episode when start is asked after the breaker tripped", async () => {
 		let script = RUNNING_CRASH;
 
 		const { supervisor, records } = crashSupervisor(() => script, READY);
@@ -508,7 +508,7 @@ describe("createSupervisor", () => {
 		await supervisor.stop();
 	}, 40_000);
 
-	test("spawns one game when two starts race inside the launch window", async () => {
+	it("spawns one game when two starts race inside the launch window", async () => {
 		const { supervisor, pids, launches, overlapped } = racedSupervisor(200, wrapperArgv);
 
 		await Promise.all([
@@ -529,7 +529,7 @@ describe("createSupervisor", () => {
 		expect(await waitGone(pids.at(0) ?? 0, 5000)).toBe(true);
 	}, 30_000);
 
-	test("never spawns beside the game a stop is taking down", async () => {
+	it("never spawns beside the game a stop is taking down", async () => {
 		const { supervisor, pids, launches, overlapped } = racedSupervisor(0, wrapperArgv);
 
 		await supervisor.start();
@@ -555,7 +555,7 @@ describe("createSupervisor", () => {
 		expect(await waitGone(pids.at(1) ?? 0, 5000)).toBe(true);
 	}, 30_000);
 
-	test("starts again after a launch that threw before the game was spawned", async () => {
+	it("starts again after a launch that threw before the game was spawned", async () => {
 		let resolvable = false;
 
 		const { supervisor, pids } = racedSupervisor(0, () => {
@@ -581,7 +581,7 @@ describe("createSupervisor", () => {
 		expect(await waitGone(pids.at(0) ?? 0, 5000)).toBe(true);
 	}, 30_000);
 
-	test("stops a game that reports a failed boot instead of waiting for a ready line that never comes", async () => {
+	it("stops a game that reports a failed boot instead of waiting for a ready line that never comes", async () => {
 		const records: BridgeCrashRecord[] = [];
 		const pids: number[] = [];
 
@@ -628,7 +628,7 @@ describe("createSupervisor", () => {
 		expect(await waitGone(pids.at(0) ?? 0, 5000)).toBe(true);
 	}, 30_000);
 
-	test("ignores the failed pattern once the game has been running long enough to count as stable", async () => {
+	it("ignores the failed pattern once the game has been running long enough to count as stable", async () => {
 		const records: BridgeCrashRecord[] = [];
 
 		const supervisor = createSupervisor({
@@ -713,7 +713,7 @@ const restartSupervisor = (script: () => string, resolveEnvironment?: () => Prom
 };
 
 describe("the resolved environment", () => {
-	test("reaches the game beside what the bridge inherited", async () => {
+	it("reaches the game beside what the bridge inherited", async () => {
 		const { supervisor } = restartSupervisor(
 			() => 'echo "probe=$SERVERK_PROBE"; echo "path=$PATH"; echo ready; exec sleep 300',
 			async () => ({
@@ -731,7 +731,7 @@ describe("the resolved environment", () => {
 		await supervisor.stop();
 	}, 20_000);
 
-	test("leaves the inherited environment alone when the driver resolves none", async () => {
+	it("leaves the inherited environment alone when the driver resolves none", async () => {
 		const { supervisor } = restartSupervisor(() => 'echo "path=$PATH"; echo ready; exec sleep 300');
 		const path = supervisor.watch(/^path=(?<value>.*)$/, 10_000);
 
@@ -745,7 +745,7 @@ describe("the resolved environment", () => {
 		await supervisor.stop();
 	}, 20_000);
 
-	test("refuses a key that is not a variable name as a start failure", async () => {
+	it("refuses a key that is not a variable name as a start failure", async () => {
 		const { supervisor } = restartSupervisor(
 			() => READY_THEN_WAIT,
 			async () => ({
@@ -757,7 +757,7 @@ describe("the resolved environment", () => {
 		expect(supervisor.running).toBe(false);
 	});
 
-	test("refuses a value carrying a NUL byte without echoing it", async () => {
+	it("refuses a value carrying a NUL byte without echoing it", async () => {
 		const { supervisor } = restartSupervisor(
 			() => READY_THEN_WAIT,
 			async () => ({
@@ -776,7 +776,7 @@ describe("the resolved environment", () => {
 });
 
 describe("restart", () => {
-	test("replaces the process without ever reporting it stopped", async () => {
+	it("replaces the process without ever reporting it stopped", async () => {
 		const { supervisor, records, pids, stopped } = restartSupervisor(() => READY_THEN_WAIT);
 
 		await supervisor.start();
@@ -819,7 +819,7 @@ describe("restart", () => {
 		expect(stopped()).toBe(1);
 	}, 30_000);
 
-	test("counts a quick exit after a restart as a boot crash, not a running one", async () => {
+	it("counts a quick exit after a restart as a boot crash, not a running one", async () => {
 		let script = READY_THEN_WAIT;
 
 		const { supervisor, records, pids } = restartSupervisor(() => script);
@@ -848,7 +848,7 @@ describe("restart", () => {
 		expect(supervisor.restarts).toBe(0);
 	}, 30_000);
 
-	test("refuses to restart a game that is meant to be stopped", async () => {
+	it("refuses to restart a game that is meant to be stopped", async () => {
 		const { supervisor } = restartSupervisor(() => READY_THEN_WAIT);
 
 		await expect(supervisor.restart()).rejects.toThrow("the game is not meant to be running");
@@ -907,7 +907,7 @@ const runningWith = async (supervisor: BridgeSupervisor, pids: number[], count:
 };
 
 describe("the stop signal", () => {
-	test("stops a game that exits on SIGTERM well inside the timeout", async () => {
+	it("stops a game that exits on SIGTERM well inside the timeout", async () => {
 		const { supervisor, pids } = signalSupervisor(SIGNAL_EXITS, 10, BridgeStopSignal.Term);
 
 		await supervisor.start();
@@ -921,7 +921,7 @@ describe("the stop signal", () => {
 		expect(await waitGone(pids[0] ?? 0, 5000)).toBe(true);
 	}, 30_000);
 
-	test("stops a game that exits on SIGINT when the driver asks for it", async () => {
+	it("stops a game that exits on SIGINT when the driver asks for it", async () => {
 		const { supervisor, pids } = signalSupervisor(SIGNAL_EXITS, 10, BridgeStopSignal.Interrupt);
 
 		await supervisor.start();
@@ -934,7 +934,7 @@ describe("the stop signal", () => {
 		expect(await waitGone(pids[0] ?? 0, 5000)).toBe(true);
 	}, 30_000);
 
-	test("still kills a game that ignores the signal once the timeout passes", async () => {
+	it("still kills a game that ignores the signal once the timeout passes", async () => {
 		const { supervisor, pids } = signalSupervisor(SIGNAL_IGNORED, 1, BridgeStopSignal.Term);
 
 		await supervisor.start();
@@ -949,7 +949,7 @@ describe("the stop signal", () => {
 		expect(await waitGone(pids[0] ?? 0, 5000), "the ignoring child outlived the kill").toBe(true);
 	}, 30_000);
 
-	test("keeps waiting the timeout before SIGTERM when the driver declares no signal", async () => {
+	it("keeps waiting the timeout before SIGTERM when the driver declares no signal", async () => {
 		const { supervisor, pids } = signalSupervisor(SIGNAL_EXITS, 2);
 
 		await supervisor.start();
@@ -963,7 +963,7 @@ describe("the stop signal", () => {
 		expect(await waitGone(pids[0] ?? 0, 5000)).toBe(true);
 	}, 30_000);
 
-	test("restarts through the same signal-first path", async () => {
+	it("restarts through the same signal-first path", async () => {
 		const { supervisor, pids } = signalSupervisor(SIGNAL_EXITS, 10, BridgeStopSignal.Term);
 
 		await supervisor.start();
@@ -1056,7 +1056,7 @@ describe("secret redaction", () => {
 		restore();
 	});
 
-	test("hides a secret value from stdout, stderr, the stored lines and the crash last line", async () => {
+	it("hides a secret value from stdout, stderr, the stored lines and the crash last line", async () => {
 		const redact = createBridgeRedaction(
 			[
 				"BOT_TOKEN",
@@ -1081,7 +1081,7 @@ describe("secret redaction", () => {
 		expect(written.join("")).toContain(REDACTED);
 	}, 30_000);
 
-	test("hides a platform secret from the manifest that the game prints", async () => {
+	it("hides a platform secret from the manifest that the game prints", async () => {
 		const redact = createBridgeRedaction(
 			[],
 			variablesSource(
@@ -1105,7 +1105,7 @@ describe("secret redaction", () => {
 		expect(records.at(0)?.lastLine).toBe(`fatal: bad token ${REDACTED}`);
 	}, 30_000);
 
-	test("leaves a value shorter than eight characters in the lines", async () => {
+	it("leaves a value shorter than eight characters in the lines", async () => {
 		const redact = createBridgeRedaction(
 			[
 				"RCON_PASSWORD",
@@ -1122,7 +1122,7 @@ describe("secret redaction", () => {
 		expect(records.at(0)?.lastLine).toBe(`fatal: bad token ${LEAKED}`);
 	}, 30_000);
 
-	test("hides a value saved after the start from the lines that follow", async () => {
+	it("hides a value saved after the start from the lines that follow", async () => {
 		const { source, save } = variablesSource({});
 		const redact = createBridgeRedaction(
 			[
@@ -1152,7 +1152,7 @@ describe("secret redaction", () => {
 		await supervisor.stop();
 	}, 30_000);
 
-	test("runs a driver with no secret fields and no secrets without touching a line", async () => {
+	it("runs a driver with no secret fields and no secrets without touching a line", async () => {
 		const redact = createBridgeRedaction([], variablesSource({}).source);
 
 		const { supervisor, records } = redactingSupervisor(LEAK_AND_CRASH, redact);
@@ -1220,7 +1220,7 @@ const subjectSupervisor = (script: string, subject?: BridgeLifecycleSubject) =>
 };
 
 describe("the lifecycle subject", () => {
-	test("names the game in the lifecycle lines when the driver declares no subject", async () => {
+	it("names the game in the lifecycle lines when the driver declares no subject", async () => {
 		const { supervisor, messages } = subjectSupervisor(READY_THEN_WAIT);
 
 		await supervisor.start();
@@ -1236,7 +1236,7 @@ describe("the lifecycle subject", () => {
 		]);
 	}, 20_000);
 
-	test("names the bot in every lifecycle line of a bot, the crash loop included", async () => {
+	it("names the bot in every lifecycle line of a bot, the crash loop included", async () => {
 		const { supervisor, messages, records } = subjectSupervisor(RUNNING_CRASH, BridgeLifecycleSubject.Bot);
 
 		await supervisor.start();
@@ -1252,7 +1252,7 @@ describe("the lifecycle subject", () => {
 		expect(messages.filter((message) => message.includes("game"))).toEqual([]);
 	}, 40_000);
 
-	test("names the bot when it refuses an environment it cannot pass on", async () => {
+	it("names the bot when it refuses an environment it cannot pass on", async () => {
 		const { logger } = recordingLogger();
 
 		const supervisor = createSupervisor({

package/src/runtime/serve.ts

@@ -41,12 +41,12 @@ const DEFAULT_SAMPLE_REFRESH_SECONDS = 30;
 
 const INSTALL_FAILED_TEXT: Record<BridgeLifecycleSubject, LocalizedText> = {
 	[BridgeLifecycleSubject.Game]: {
-		ar: "ما قدرنا نثبّت اللعبة. جرّب مرة ثانية.",
-		en: "We could not install the game. Try again.",
+		ar: "تعذّر تثبيت اللعبة، جرب مرة ثانية.",
+		en: "We couldn't install the game. Try again.",
 	},
 	[BridgeLifecycleSubject.Bot]: {
-		ar: "ما قدرنا نثبّت كود بوتك. جرّب مرة ثانية.",
-		en: "We could not install your bot's code. Try again.",
+		ar: "تعذّر تثبيت كود بوتك، جرب مرة ثانية.",
+		en: "We couldn't install your bot's code. Try again.",
 	},
 };
 

package/src/steam/avatars.test.ts

@@ -1,4 +1,4 @@
-import { afterEach, describe, expect, setSystemTime, test } from "bun:test";
+import { afterEach, describe, expect, it, setSystemTime } from "bun:test";
 import type { BridgeContext } from "../context";
 import type { BridgeValues } from "../core";
 import { createSteamAvatars, STEAM_AVATAR_CACHE_TTL_MS, steamAvatarHashOf } from "./avatars";
@@ -79,22 +79,22 @@ afterEach(() => {
 });
 
 describe("reading the avatar hash out of a steam profile picture", () => {
-	test("takes the hash out of the full-size url steam answers with", () => {
+	it("takes the hash out of the full-size url steam answers with", () => {
 		expect(steamAvatarHashOf(`https://avatars.steamstatic.com/${ALICE_HASH}_full.jpg`)).toBe(ALICE_HASH);
 	});
 
-	test("takes the hash out of a url that carries no size suffix", () => {
+	it("takes the hash out of a url that carries no size suffix", () => {
 		expect(steamAvatarHashOf(`https://avatars.steamstatic.com/${BOB_HASH}.jpg`)).toBe(BOB_HASH);
 	});
 
-	test("reports no hash for a url that is not a steam avatar", () => {
+	it("reports no hash for a url that is not a steam avatar", () => {
 		expect(steamAvatarHashOf("https://example.com/avatar.png")).toBeNull();
 		expect(steamAvatarHashOf("")).toBeNull();
 	});
 });
 
 describe("keeping the steam avatars a roster shows", () => {
-	test("asks steam for a hash it has never seen, and answers with it", async () => {
+	it("asks steam for a hash it has never seen, and answers with it", async () => {
 		const steam = fakeSteam({
 			answer: () => [
 				summaryOf(ALICE, ALICE_HASH),
@@ -123,7 +123,7 @@ describe("keeping the steam avatars a roster shows", () => {
 		]);
 	});
 
-	test("answers the second time without asking steam again", async () => {
+	it("answers the second time without asking steam again", async () => {
 		const steam = fakeSteam({
 			answer: () => [
 				summaryOf(ALICE, ALICE_HASH),
@@ -147,7 +147,7 @@ describe("keeping the steam avatars a roster shows", () => {
 		expect(steam.calls.length).toBe(1);
 	});
 
-	test("asks only for the ids it is missing, and still answers for the cached one", async () => {
+	it("asks only for the ids it is missing, and still answers for the cached one", async () => {
 		const steam = fakeSteam({
 			answer: (steamIds) => {
 				return steamIds.map((steamId) => {
@@ -175,7 +175,7 @@ describe("keeping the steam avatars a roster shows", () => {
 		expect(hashes.get(BOB)).toBe(BOB_HASH);
 	});
 
-	test("asks steam again once the cached hash is six hours old", async () => {
+	it("asks steam again once the cached hash is six hours old", async () => {
 		setSystemTime(new Date(0));
 
 		const steam = fakeSteam({
@@ -211,7 +211,7 @@ describe("keeping the steam avatars a roster shows", () => {
 		expect(steam.calls.length).toBe(2);
 	});
 
-	test("never asks steam about a player the game does not map to a steam id", async () => {
+	it("never asks steam about a player the game does not map to a steam id", async () => {
 		const steam = fakeSteam({});
 		const avatars = createSteamAvatars({
 			steamIdOf: identity,
@@ -225,7 +225,7 @@ describe("keeping the steam avatars a roster shows", () => {
 		expect(steam.calls).toEqual([]);
 	});
 
-	test("answers with nothing while the platform has no steam key", async () => {
+	it("answers with nothing while the platform has no steam key", async () => {
 		const steam = fakeSteam({
 			key: null,
 		});
@@ -242,7 +242,7 @@ describe("keeping the steam avatars a roster shows", () => {
 		expect(steam.warnings).toEqual([]);
 	});
 
-	test("answers with nothing and warns once when steam refuses", async () => {
+	it("answers with nothing and warns once when steam refuses", async () => {
 		const steam = fakeSteam({
 			answer: () => {
 				throw new Error("429 too many requests");
@@ -266,7 +266,7 @@ describe("keeping the steam avatars a roster shows", () => {
 		expect(steam.warnings.length).toBe(1);
 	});
 
-	test("never writes the api key into the warning it logs", async () => {
+	it("never writes the api key into the warning it logs", async () => {
 		const steam = fakeSteam({
 			key: "top-secret-key",
 			answer: () => {
@@ -284,7 +284,7 @@ describe("keeping the steam avatars a roster shows", () => {
 		expect(JSON.stringify(steam.warnings)).not.toContain("top-secret-key");
 	});
 
-	test("keeps the roster moving when steam sends a profile with no usable picture", async () => {
+	it("keeps the roster moving when steam sends a profile with no usable picture", async () => {
 		const steam = fakeSteam({
 			answer: () => [
 				{

package/src/steam/steamApp.test.ts

@@ -1,4 +1,4 @@
-import { describe, expect, test } from "bun:test";
+import { describe, expect, it } from "bun:test";
 import type { BridgeContext } from "../context";
 import { appManifestPath, buildIdOf, installedBuildId, missingGameRoots } from "./steamApp";
 
@@ -24,32 +24,32 @@ const testContext = (files: Record<string, string>) => {
 };
 
 describe("appManifestPath", () => {
-	test("points at the steamapps manifest", () => {
+	it("points at the steamapps manifest", () => {
 		expect(appManifestPath("2394010")).toBe("steamapps/appmanifest_2394010.acf");
 	});
 
-	test("accepts another steamapps directory", () => {
+	it("accepts another steamapps directory", () => {
 		expect(appManifestPath("90", ".steam/steamapps")).toBe(".steam/steamapps/appmanifest_90.acf");
 	});
 });
 
 describe("buildIdOf", () => {
-	test("reads the build id out of an acf manifest", () => {
+	it("reads the build id out of an acf manifest", () => {
 		expect(buildIdOf(MANIFEST)).toBe("19582821");
 	});
 
-	test("returns null when there is no build id", () => {
+	it("returns null when there is no build id", () => {
 		expect(buildIdOf('"AppState"\n{\n}\n')).toBeNull();
 		expect(buildIdOf("")).toBeNull();
 	});
 });
 
 describe("installedBuildId", () => {
-	test("returns null when the app was never installed", async () => {
+	it("returns null when the app was never installed", async () => {
 		expect(await installedBuildId(testContext({}), "2394010")).toBeNull();
 	});
 
-	test("reads the installed build id", async () => {
+	it("reads the installed build id", async () => {
 		const context = testContext({
 			"steamapps/appmanifest_2394010.acf": MANIFEST,
 		});
@@ -59,7 +59,7 @@ describe("installedBuildId", () => {
 });
 
 describe("missingGameRoots", () => {
-	test("names only the paths that are absent", async () => {
+	it("names only the paths that are absent", async () => {
 		const context = testContext({
 			"PalServer.sh": "",
 			Engine: "",

package/src/steam/steamcmd.test.ts

@@ -1,4 +1,4 @@
-import { describe, expect, test } from "bun:test";
+import { describe, expect, it } from "bun:test";
 import { steamcmdArgv, steamcmdFailureText, steamcmdScript, steamcmdUpdateArgs } from "./steamcmd";
 import { SteamcmdLoginKind, steamcmdLoginArgs } from "./steamcmdLogin";
 import { SteamcmdPlatform } from "./steamcmdPlatform";
@@ -9,13 +9,13 @@ const OPTIONS = {
 };
 
 describe("steamcmdLoginArgs", () => {
-	test("logs in anonymously by default", () => {
+	it("logs in anonymously by default", () => {
 		expect(steamcmdLoginArgs()).toEqual([
 			"anonymous",
 		]);
 	});
 
-	test("passes the account credentials through", () => {
+	it("passes the account credentials through", () => {
 		expect(
 			steamcmdLoginArgs({
 				kind: SteamcmdLoginKind.Account,
@@ -30,14 +30,14 @@ describe("steamcmdLoginArgs", () => {
 });
 
 describe("steamcmdScript", () => {
-	test("lives inside the copied steamcmd directory", () => {
+	it("lives inside the copied steamcmd directory", () => {
 		expect(steamcmdScript()).toBe(".steamcmd/steamcmd.sh");
 		expect(steamcmdScript(".sc")).toBe(".sc/steamcmd.sh");
 	});
 });
 
 describe("steamcmdArgv", () => {
-	test("forces the install directory and logs in before the command", () => {
+	it("forces the install directory and logs in before the command", () => {
 		expect(
 			steamcmdArgv(OPTIONS, "/home/container", [
 				"+app_update",
@@ -55,7 +55,7 @@ describe("steamcmdArgv", () => {
 		]);
 	});
 
-	test("forces the windows platform before the login", () => {
+	it("forces the windows platform before the login", () => {
 		const argv = steamcmdArgv(
 			{
 				...OPTIONS,
@@ -70,7 +70,7 @@ describe("steamcmdArgv", () => {
 		expect(argv.indexOf("+@sSteamCmdForcePlatformType")).toBeLessThan(argv.indexOf("+login"));
 	});
 
-	test("leaves the platform alone on linux", () => {
+	it("leaves the platform alone on linux", () => {
 		expect(
 			steamcmdArgv(
 				{
@@ -85,7 +85,7 @@ describe("steamcmdArgv", () => {
 });
 
 describe("steamcmdUpdateArgs", () => {
-	test("updates without validating by default", () => {
+	it("updates without validating by default", () => {
 		expect(
 			steamcmdUpdateArgs("258550", {
 				validate: false,
@@ -96,7 +96,7 @@ describe("steamcmdUpdateArgs", () => {
 		]);
 	});
 
-	test("adds validate last of the update flags", () => {
+	it("adds validate last of the update flags", () => {
 		expect(
 			steamcmdUpdateArgs("258550", {
 				validate: true,
@@ -108,7 +108,7 @@ describe("steamcmdUpdateArgs", () => {
 		]);
 	});
 
-	test("passes a branch and its password", () => {
+	it("passes a branch and its password", () => {
 		expect(
 			steamcmdUpdateArgs("90", {
 				validate: true,
@@ -126,7 +126,7 @@ describe("steamcmdUpdateArgs", () => {
 		]);
 	});
 
-	test("ignores a branch password without a branch", () => {
+	it("ignores a branch password without a branch", () => {
 		expect(
 			steamcmdUpdateArgs("90", {
 				validate: false,
@@ -138,7 +138,7 @@ describe("steamcmdUpdateArgs", () => {
 		]);
 	});
 
-	test("appends the extra arguments", () => {
+	it("appends the extra arguments", () => {
 		expect(
 			steamcmdUpdateArgs("376030", {
 				validate: false,
@@ -157,7 +157,7 @@ describe("steamcmdUpdateArgs", () => {
 });
 
 describe("steamcmdFailureText", () => {
-	test("says the download account does not own the app when steam answers no subscription", () => {
+	it("says the download account does not own the app when steam answers no subscription", () => {
 		expect(
 			steamcmdFailureText(
 				{
@@ -167,10 +167,10 @@ describe("steamcmdFailureText", () => {
 				},
 				"assetto",
 			).en,
-		).toBe("Steam refused to download assetto because the download account does not own it. Contact support.");
+		).toBe("Steam refused to download assetto because the download account doesn't own it. Contact support.");
 	});
 
-	test("says the download failed for any other exit", () => {
+	it("says the download failed for any other exit", () => {
 		expect(
 			steamcmdFailureText(
 				{
@@ -181,8 +181,8 @@ describe("steamcmdFailureText", () => {
 				"palworld",
 			),
 		).toEqual({
-			ar: "ما قدرنا ننزّل palworld من Steam. جرّب مرة ثانية.",
-			en: "We could not download palworld from Steam. Try again.",
+			ar: "تعذّر تنزيل palworld من Steam، جرب مرة ثانية.",
+			en: "We couldn't download palworld from Steam. Try again.",
 		});
 	});
 });