@serverkgg/bridge 0.1.1 → 0.1.3

package/src/runtime/bridgeDriver.ts

@@ -3,6 +3,7 @@ import {
 	BRIDGE_MODULE_ID_PATTERN,
 	BridgeKind,
 	BridgeSetupStepKind,
+	isReadAction,
 	type LocalizedText,
 } from "../core";
 import { GuideOpenTab } from "../guides";
@@ -801,15 +802,17 @@ export const validateDriver = (driver: BridgeDriver) => {
 			countIn((module.dependsOn ?? []).length, BRIDGE_DEPENDS_ON_LIMIT, "dependencies", `module "${id}"`);
 		}
 
-		if (module.kind === BridgeKind.Actions) {
-			for (const action of Object.keys(module.actions)) {
-				validateModuleId(action, `module "${id}" action`);
-			}
-		}
-
-		if (module.kind === BridgeKind.Collection || module.kind === BridgeKind.Detail) {
+		if (
+			module.kind === BridgeKind.Actions
+			|| module.kind === BridgeKind.Collection
+			|| module.kind === BridgeKind.Detail
+		) {
 			for (const action of Object.keys(module.actions ?? {})) {
 				validateModuleId(action, `module "${id}" action`);
+
+				if (isReadAction(action)) {
+					fail(`module "${id}" declares action "${action}", which is reserved for reads — pick another id`);
+				}
 			}
 		}
 	}

package/src/runtime/bridgeEvents.test.ts

@@ -1,4 +1,4 @@
-import { describe, expect, test } from "bun:test";
+import { describe, expect, it } from "bun:test";
 import { mkdtemp, rm } from "node:fs/promises";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
@@ -162,19 +162,19 @@ const joined = {
 };
 
 describe("matching a game line against the driver patterns", () => {
-	test("emits on the plain line", async () => {
+	it("emits on the plain line", async () => {
 		expect(await emittedFor("[World|default] Player 'Meslzy' joined world default")).toEqual([
 			joined,
 		]);
 	});
 
-	test("emits on the line hytale really prints, wrapped in ansi resets at both ends", async () => {
+	it("emits on the line hytale really prints, wrapped in ansi resets at both ends", async () => {
 		expect(await emittedFor(`${RESET}[World|default] Player 'Meslzy' joined world default${RESET}`)).toEqual([
 			joined,
 		]);
 	});
 
-	test("holds a matched diagnostic line as evidence and spends it on the crash record", async () => {
+	it("holds a matched diagnostic line as evidence and spends it on the crash record", async () => {
 		const { held, events: drained } = await crashAfter(
 			"Exception in mod journeymap from journeymap-5.9.jar",
 			crashRecord({
@@ -205,7 +205,7 @@ describe("matching a game line against the driver patterns", () => {
 });
 
 describe("the crash record the ring builds", () => {
-	test("holds evidence out of the ring and lets every other event drain", () => {
+	it("holds evidence out of the ring and lets every other event drain", () => {
 		const ring = idleRing();
 
 		ring.emit(BridgeEventName.ModCrashed, {
@@ -221,7 +221,7 @@ describe("the crash record the ring builds", () => {
 		]);
 	});
 
-	test("prefers the oom kill over any evidence it holds", () => {
+	it("prefers the oom kill over any evidence it holds", () => {
 		const ring = idleRing();
 
 		ring.emit(BridgeEventName.ModCrashed, {
@@ -246,7 +246,7 @@ describe("the crash record the ring builds", () => {
 		});
 	});
 
-	test("follows the evidence precedence when more than one name is held", () => {
+	it("follows the evidence precedence when more than one name is held", () => {
 		const ring = idleRing();
 
 		ring.emit(BridgeEventName.ModCrashed, {
@@ -271,7 +271,7 @@ describe("the crash record the ring builds", () => {
 		});
 	});
 
-	test("lets the out of memory log pattern outrank the java version", () => {
+	it("lets the out of memory log pattern outrank the java version", () => {
 		const ring = idleRing();
 
 		ring.emit(BridgeEventName.WrongJavaVersion, {
@@ -290,7 +290,7 @@ describe("the crash record the ring builds", () => {
 		});
 	});
 
-	test("never lets the evidence overwrite the record keys", () => {
+	it("never lets the evidence overwrite the record keys", () => {
 		const ring = idleRing();
 
 		ring.emit(BridgeEventName.ModCrashed, {
@@ -310,7 +310,7 @@ describe("the crash record the ring builds", () => {
 		});
 	});
 
-	test("clears the evidence once it is spent", () => {
+	it("clears the evidence once it is spent", () => {
 		const ring = idleRing();
 
 		ring.emit(BridgeEventName.ModCrashed, {
@@ -332,7 +332,7 @@ describe("the crash record the ring builds", () => {
 		});
 	});
 
-	test("carries the last line only when nothing explains the crash", () => {
+	it("carries the last line only when nothing explains the crash", () => {
 		const ring = idleRing();
 
 		ring.emit(BridgeEventName.ModCrashed, {
@@ -360,7 +360,7 @@ describe("the crash record the ring builds", () => {
 		});
 	});
 
-	test("keeps listing the evidence names the driver declares", () => {
+	it("keeps listing the evidence names the driver declares", () => {
 		expect(idleRing().names()).toEqual([
 			BridgeEventName.PlayerJoined,
 			BridgeEventName.ModCrashed,
@@ -433,7 +433,7 @@ const crashedEvent = async (ring: BridgeEventRing) => {
 };
 
 describe("the evidence a run may spend", () => {
-	test("spends nothing printed before the run became stable on a running crash", async () => {
+	it("spends nothing printed before the run became stable on a running crash", async () => {
 		const wiring = await wired(EVIDENCE_BEFORE_READY, READY_LINE);
 
 		await wiring.supervisor.start();
@@ -452,7 +452,7 @@ describe("the evidence a run may spend", () => {
 		}
 	}, 20_000);
 
-	test("spends what the run printed after it became stable", async () => {
+	it("spends what the run printed after it became stable", async () => {
 		const wiring = await wired(EVIDENCE_AFTER_READY, READY_LINE);
 
 		await wiring.supervisor.start();
@@ -470,7 +470,7 @@ describe("the evidence a run may spend", () => {
 		}
 	}, 20_000);
 
-	test("drops what it holds when the stop was requested", async () => {
+	it("drops what it holds when the stop was requested", async () => {
 		const wiring = await wired(EVIDENCE_THEN_IDLE);
 
 		await wiring.supervisor.start();

package/src/runtime/bridgeFiles.test.ts

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, describe, expect, test } from "bun:test";
+import { afterEach, beforeEach, describe, expect, it } from "bun:test";
 import { mkdtemp, rm } from "node:fs/promises";
 import { tmpdir } from "node:os";
 import { crc32, deflateRawSync } from "node:zlib";
@@ -205,7 +205,7 @@ describe("extract", () => {
 		});
 	});
 
-	test("flattens paths, skips dotfiles and filters extensions by default", async () => {
+	it("flattens paths, skips dotfiles and filters extensions by default", async () => {
 		const path = await archive([
 			{
 				name: "overrides/mods/alpha.jar",
@@ -234,7 +234,7 @@ describe("extract", () => {
 		expect(await read("mods/alpha.jar")).toBe("alpha");
 	});
 
-	test("preflights duplicate paths, parent collisions and expansion before writing", async () => {
+	it("preflights duplicate paths, parent collisions and expansion before writing", async () => {
 		for (const entries of [
 			[
 				{
@@ -285,7 +285,7 @@ describe("extract", () => {
 		expect(await Bun.file(`${root}/staging/a.txt`).exists()).toBe(false);
 	});
 
-	test("allows a bounded per-entry runtime limit without bypassing archive limits", async () => {
+	it("allows a bounded per-entry runtime limit without bypassing archive limits", async () => {
 		const path = await archive([
 			{
 				name: "runtime",
@@ -323,7 +323,7 @@ describe("extract", () => {
 		expect(await read("staging/runtime")).toBe("12345678");
 	});
 
-	test("refuses unsafe unselected entries instead of accepting a partly safe archive", async () => {
+	it("refuses unsafe unselected entries instead of accepting a partly safe archive", async () => {
 		const path = await archive([
 			{
 				name: "safe/a.txt",
@@ -343,7 +343,7 @@ describe("extract", () => {
 		expect(await Bun.file(`${root}/staging/a.txt`).exists()).toBe(false);
 	});
 
-	test("flattens paths with no options at all", async () => {
+	it("flattens paths with no options at all", async () => {
 		const path = await archive([
 			{
 				name: "nested/deep/alpha.jar",
@@ -363,7 +363,7 @@ describe("extract", () => {
 		]);
 	});
 
-	test("keeps relative paths and creates intermediate directories in tree mode", async () => {
+	it("keeps relative paths and creates intermediate directories in tree mode", async () => {
 		const path = await archive([
 			{
 				name: "mods/alpha.jar",
@@ -388,7 +388,7 @@ describe("extract", () => {
 		expect(await read("staging/config/deep/beta.json")).toBe("beta");
 	});
 
-	test("keeps the dotfile skip in tree mode", async () => {
+	it("keeps the dotfile skip in tree mode", async () => {
 		const path = await archive([
 			{
 				name: "mods/alpha.jar",
@@ -413,7 +413,7 @@ describe("extract", () => {
 		]);
 	});
 
-	test("rejects absolute entries in tree mode", async () => {
+	it("rejects absolute entries in tree mode", async () => {
 		const path = await archive([
 			{
 				name: "/etc/passwd",
@@ -428,7 +428,7 @@ describe("extract", () => {
 		).rejects.toThrow("absolute path");
 	});
 
-	test("rejects escaping entries in tree mode", async () => {
+	it("rejects escaping entries in tree mode", async () => {
 		const path = await archive([
 			{
 				name: "overrides/../../escape.json",
@@ -443,7 +443,7 @@ describe("extract", () => {
 		).rejects.toThrow("is not allowed");
 	});
 
-	test("rejects symlink entries in tree mode", async () => {
+	it("rejects symlink entries in tree mode", async () => {
 		const path = await archive([
 			{
 				name: "mods/link.jar",
@@ -459,7 +459,7 @@ describe("extract", () => {
 		).rejects.toThrow("symlink");
 	});
 
-	test("extracts only the selected prefix and strips it", async () => {
+	it("extracts only the selected prefix and strips it", async () => {
 		const path = await archive([
 			{
 				name: "overrides/mods/alpha.jar",
@@ -486,7 +486,7 @@ describe("extract", () => {
 		expect(await read("alpha.jar")).toBe("alpha");
 	});
 
-	test("composes select with tree", async () => {
+	it("composes select with tree", async () => {
 		const path = await archive([
 			{
 				name: "overrides/mods/alpha.jar",
@@ -515,7 +515,7 @@ describe("extract", () => {
 		expect(await read("config/deep/beta.json")).toBe("beta");
 	});
 
-	test("fails when nothing matches", async () => {
+	it("fails when nothing matches", async () => {
 		const path = await archive([
 			{
 				name: "overrides/mods/alpha.jar",
@@ -531,7 +531,7 @@ describe("extract", () => {
 		).rejects.toThrow("contained no matching files");
 	});
 
-	test("takes every selected prefix in one pass and strips the one that matched", async () => {
+	it("takes every selected prefix in one pass and strips the one that matched", async () => {
 		const path = await archive([
 			{
 				name: "overrides/mods/alpha.jar",
@@ -564,7 +564,7 @@ describe("extract", () => {
 		expect(await read("config/beta.json")).toBe("beta");
 	});
 
-	test("keeps dotfiles in a tree when asked and never writes .git or .serverk", async () => {
+	it("keeps dotfiles in a tree when asked and never writes .git or .serverk", async () => {
 		const path = await archive([
 			{
 				name: "bot/.env.example",
@@ -605,7 +605,7 @@ describe("extract", () => {
 		expect(await files.exists(".serverk/deploy-requested")).toBe(false);
 	});
 
-	test("still skips dotfiles in a tree unless asked", async () => {
+	it("still skips dotfiles in a tree unless asked", async () => {
 		const path = await archive([
 			{
 				name: ".env.example",
@@ -626,7 +626,7 @@ describe("extract", () => {
 		]);
 	});
 
-	test("takes a single named entry when the select carries no trailing slash", async () => {
+	it("takes a single named entry when the select carries no trailing slash", async () => {
 		const path = await archive([
 			{
 				name: "modrinth.index.json",
@@ -652,7 +652,7 @@ describe("extract", () => {
 		expect(await read("staging/modrinth.index.json")).toBe("{}");
 	});
 
-	test("drops excluded paths after the prefix is stripped", async () => {
+	it("drops excluded paths after the prefix is stripped", async () => {
 		const path = await archive([
 			{
 				name: "overrides/mods/alpha.jar",
@@ -686,7 +686,7 @@ describe("extract", () => {
 		expect(await Bun.file(`${root}/shaderpacks/heavy.zip`).exists()).toBe(false);
 	});
 
-	test("answers with nothing when a select matches nothing", async () => {
+	it("answers with nothing when a select matches nothing", async () => {
 		const path = await archive([
 			{
 				name: "overrides/mods/alpha.jar",
@@ -704,7 +704,7 @@ describe("extract", () => {
 		expect(written).toEqual([]);
 	});
 
-	test("inflates a deflated entry", async () => {
+	it("inflates a deflated entry", async () => {
 		const path = await archive([
 			{
 				name: "mods/alpha.jar",
@@ -722,7 +722,7 @@ describe("extract", () => {
 		expect(await read("mods/alpha.jar")).toBe("alpha the deflated");
 	});
 
-	test("rejects stored data corruption before writing the destination", async () => {
+	it("rejects stored data corruption before writing the destination", async () => {
 		const data = buildZip([
 			{
 				name: "alpha.jar",
@@ -735,7 +735,7 @@ describe("extract", () => {
 		expect(await Bun.file(`${root}/mods/alpha.jar`).exists()).toBe(false);
 	});
 
-	test("rejects deflated data with a valid size but incorrect checksum", async () => {
+	it("rejects deflated data with a valid size but incorrect checksum", async () => {
 		const path = await archive([
 			{
 				name: "alpha.jar",
@@ -747,7 +747,7 @@ describe("extract", () => {
 		expect(await Bun.file(`${root}/mods/alpha.jar`).exists()).toBe(false);
 	});
 
-	test("rejects local and central compression method disagreement", async () => {
+	it("rejects local and central compression method disagreement", async () => {
 		const data = buildZip([
 			{
 				name: "alpha.jar",
@@ -759,7 +759,7 @@ describe("extract", () => {
 		await expect(files.extract("corrupt.zip", "mods")).rejects.toThrow("method");
 	});
 
-	test("rejects local names that differ from the central directory", async () => {
+	it("rejects local names that differ from the central directory", async () => {
 		const data = buildZip([
 			{
 				name: "alpha.jar",
@@ -771,7 +771,7 @@ describe("extract", () => {
 		await expect(files.extract("corrupt.zip", "mods")).rejects.toThrow("local name");
 	});
 
-	test("rejects a local variable header that crosses into the central directory", async () => {
+	it("rejects a local variable header that crosses into the central directory", async () => {
 		const data = buildZip([
 			{
 				name: "alpha.jar",
@@ -783,7 +783,7 @@ describe("extract", () => {
 		await expect(files.extract("corrupt.zip", "mods")).rejects.toThrow("bounds");
 	});
 
-	test("rejects compressed data ranges that cross the central directory", async () => {
+	it("rejects compressed data ranges that cross the central directory", async () => {
 		const data = buildZip([
 			{
 				name: "alpha.jar",
@@ -797,7 +797,7 @@ describe("extract", () => {
 		await expect(files.extract("corrupt.zip", "mods")).rejects.toThrow("bounds");
 	});
 
-	test("accepts descriptor entries with zero local checksum and validates the central checksum", async () => {
+	it("accepts descriptor entries with zero local checksum and validates the central checksum", async () => {
 		const path = await archive(
 			[
 				{
@@ -814,7 +814,7 @@ describe("extract", () => {
 		expect(await read("mods/alpha.jar")).toBe("alpha");
 	});
 
-	test("rejects local checksum disagreement", async () => {
+	it("rejects local checksum disagreement", async () => {
 		const data = buildZip([
 			{
 				name: "alpha.jar",
@@ -826,7 +826,7 @@ describe("extract", () => {
 		await expect(files.extract("corrupt.zip", "mods")).rejects.toThrow("local checksum");
 	});
 
-	test("rejects local compressed-size disagreement", async () => {
+	it("rejects local compressed-size disagreement", async () => {
 		const data = buildZip([
 			{
 				name: "alpha.jar",
@@ -838,7 +838,7 @@ describe("extract", () => {
 		await expect(files.extract("corrupt.zip", "mods")).rejects.toThrow("local sizes");
 	});
 
-	test("reads a zip64 archive through its locator and extended field", async () => {
+	it("reads a zip64 archive through its locator and extended field", async () => {
 		const path = await archive(
 			[
 				{
@@ -870,7 +870,7 @@ describe("extract", () => {
 		expect(await read("config/beta.json")).toBe("beta");
 	});
 
-	test("takes a suffix select from anywhere in the archive", async () => {
+	it("takes a suffix select from anywhere in the archive", async () => {
 		const path = await archive([
 			{
 				name: "BMC5 Server/mods/alpha.jar",
@@ -902,7 +902,7 @@ describe("extract", () => {
 		expect(await read("staging/beta.jar")).toBe("beta");
 	});
 
-	test("keeps the suffix path under the destination in tree mode", async () => {
+	it("keeps the suffix path under the destination in tree mode", async () => {
 		const path = await archive([
 			{
 				name: "BMC5 Server/mods/alpha.jar",
@@ -922,7 +922,7 @@ describe("extract", () => {
 		]);
 	});
 
-	test("never inflates an entry it did not select", async () => {
+	it("never inflates an entry it did not select", async () => {
 		const entries: ArchiveEntry[] = [
 			{
 				name: "BMC5 Server/mods/alpha.jar",
@@ -956,7 +956,7 @@ describe("extract", () => {
 		).rejects.toThrow();
 	});
 
-	test("answers with nothing when every match is excluded", async () => {
+	it("answers with nothing when every match is excluded", async () => {
 		const path = await archive([
 			{
 				name: "overrides/shaderpacks/heavy.zip",
@@ -997,7 +997,7 @@ describe("list", () => {
 			recursive: true,
 		});
 	});
-	test("filters before the result cap and scopes metadata beyond a large runtime tree", async () => {
+	it("filters before the result cap and scopes metadata beyond a large runtime tree", async () => {
 		for (let start = 0; start < 4097; start += 64) {
 			await Promise.all(
 				Array.from(
@@ -1027,7 +1027,7 @@ describe("list", () => {
 		]);
 		await expect(files.list("**/*")).rejects.toThrow("result limit");
 	});
-	test("nonrecursive globs ignore unrelated deep trees and respect wildcard directory boundaries", async () => {
+	it("nonrecursive globs ignore unrelated deep trees and respect wildcard directory boundaries", async () => {
 		await files.write("server.jar", "jar");
 		await files.write("server.properties", "settings");
 		await files.write("mods/custom/plugin.jar", "plugin");
@@ -1048,7 +1048,7 @@ describe("list", () => {
 			"server.jar",
 		]);
 	});
-	test("prunes subtrees past the depth limit instead of failing the whole listing", async () => {
+	it("prunes subtrees past the depth limit instead of failing the whole listing", async () => {
 		const nested = Array.from(
 			{
 				length: 14,
@@ -1078,7 +1078,7 @@ describe("list", () => {
 			"resource_packs/pack/pack_icon.png",
 		]);
 	});
-	test("warns about what a pruned listing left out", async () => {
+	it("warns about what a pruned listing left out", async () => {
 		const lines: string[] = [];
 		const noisy = createBridgeFiles({
 			root,
@@ -1109,7 +1109,7 @@ describe("list", () => {
 			"file listing skipped directories deeper than its depth limit",
 		]);
 	});
-	test("refuses unsafe listing roots", async () => {
+	it("refuses unsafe listing roots", async () => {
 		await expect(
 			files.list("**/*", {
 				directory: "../outside",
@@ -1142,7 +1142,7 @@ describe("digest", () => {
 		});
 	});
 
-	test("answers with the known vectors for abc", async () => {
+	it("answers with the known vectors for abc", async () => {
 		await Bun.write(`${root}/mods/alpha.jar`, "abc");
 
 		expect(await files.digest("mods/alpha.jar", "sha1")).toBe("a9993e364706816aba3e25717850c26c9cd0d89d");
@@ -1156,13 +1156,13 @@ describe("digest", () => {
 		);
 	});
 
-	test("answers with the known vector for an empty file", async () => {
+	it("answers with the known vector for an empty file", async () => {
 		await Bun.write(`${root}/empty.jar`, "");
 
 		expect(await files.digest("empty.jar", "sha1")).toBe("da39a3ee5e6b4b0d3255bfef95601890afd80709");
 	});
 
-	test("hashes a file larger than one read chunk", async () => {
+	it("hashes a file larger than one read chunk", async () => {
 		const content = "a".repeat(4 * 1024 * 1024 + 17);
 
 		await Bun.write(`${root}/big.bin`, content);
@@ -1170,14 +1170,14 @@ describe("digest", () => {
 		expect(await files.digest("big.bin", "sha256")).toBe(new Bun.CryptoHasher("sha256").update(content).digest("hex"));
 	});
 
-	test("refuses a path that is not a readable file", async () => {
+	it("refuses a path that is not a readable file", async () => {
 		await files.ensure("mods");
 
 		await expect(files.digest("mods", "sha1")).rejects.toThrow("is not a readable file");
 		await expect(files.digest("mods/missing.jar", "sha1")).rejects.toThrow("is not a readable file");
 	});
 
-	test("refuses a path outside the server directory", async () => {
+	it("refuses a path outside the server directory", async () => {
 		await expect(files.digest("../escape.jar", "sha1")).rejects.toThrow("is not allowed");
 	});
 });
@@ -1243,7 +1243,7 @@ describe("download", () => {
 		});
 	});
 
-	test("keeps a file the source undersized when the checksum passes", async () => {
+	it("keeps a file the source undersized when the checksum passes", async () => {
 		const content = "a mod jar the index measured wrong";
 
 		await filesServing(content).download("mods/alpha.jar", "https://cdn.example.com/alpha.jar", {
@@ -1257,7 +1257,7 @@ describe("download", () => {
 		]);
 	});
 
-	test("refuses a file whose checksum does not match, however right its size", async () => {
+	it("refuses a file whose checksum does not match, however right its size", async () => {
 		const content = "a mod jar somebody swapped";
 
 		await expect(
@@ -1270,7 +1270,7 @@ describe("download", () => {
 		expect(await Bun.file(`${root}/mods/alpha.jar`).exists()).toBe(false);
 	});
 
-	test("holds an unverified file to its declared size", async () => {
+	it("holds an unverified file to its declared size", async () => {
 		const content = "a mod jar with no checksum to trust";
 
 		await expect(
@@ -1282,7 +1282,7 @@ describe("download", () => {
 		expect(await Bun.file(`${root}/mods/alpha.jar`).exists()).toBe(false);
 	});
 
-	test("holds an unverified file that stops short of its declared size", async () => {
+	it("holds an unverified file that stops short of its declared size", async () => {
 		const content = "a mod jar with no checksum to trust";
 
 		await expect(
@@ -1292,7 +1292,7 @@ describe("download", () => {
 		).rejects.toThrow(`returned ${Buffer.byteLength(content)} bytes, expected ${Buffer.byteLength(content) + 2}`);
 	});
 
-	test("carries the upstream status when the egress channel refuses", async () => {
+	it("carries the upstream status when the egress channel refuses", async () => {
 		const refusing: BridgeEgress = {
 			fetch() {
 				throw new Error("the egress channel is not part of this test");
@@ -1323,7 +1323,7 @@ describe("download", () => {
 		expect(thrown instanceof BridgeDownloadError ? thrown.status : null).toBe(429);
 	});
 
-	test("narrates a file it downloads unless the caller asks otherwise", async () => {
+	it("narrates a file it downloads unless the caller asks otherwise", async () => {
 		const content = "a server jar worth watching";
 
 		await filesServing(content).download("server.jar", "https://cdn.example.com/server.jar", {
@@ -1337,7 +1337,7 @@ describe("download", () => {
 		]);
 	});
 
-	test("says nothing about a quiet file", async () => {
+	it("says nothing about a quiet file", async () => {
 		const content = "a mod jar in a batch of hundreds";
 
 		await filesServing(content).download("mods/alpha.jar", "https://cdn.example.com/alpha.jar", {
@@ -1350,7 +1350,7 @@ describe("download", () => {
 		expect(messages).toEqual([]);
 	});
 
-	test("still warns and still fails on a quiet file", async () => {
+	it("still warns and still fails on a quiet file", async () => {
 		const content = "a mod jar the index measured wrong";
 
 		await filesServing(content).download("mods/alpha.jar", "https://cdn.example.com/alpha.jar", {
@@ -1373,7 +1373,7 @@ describe("download", () => {
 		expect(messages).toEqual([]);
 	});
 
-	test("keeps no warning when the declared size is right", async () => {
+	it("keeps no warning when the declared size is right", async () => {
 		const content = "a mod jar the index measured right";
 
 		await filesServing(content).download("mods/alpha.jar", "https://cdn.example.com/alpha.jar", {