From 4708909dc2d2953aa0b5b1cb65b94e13b8dbfda6 Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Fri, 8 May 2026 13:43:26 +0100 Subject: [PATCH] test: clarify resilience test wording --- extensions/matrix/src/matrix/monitor/events.test.ts | 2 +- extensions/twitch/src/twitch-client.test.ts | 4 ++-- src/gateway/server-http.hooks-request-timeout.test.ts | 2 +- src/oc-path/tests/scenarios/frontmatter-edges.test.ts | 4 ++-- 4 files changed, 6 insertions(+), 6 deletions(-) diff --git a/extensions/matrix/src/matrix/monitor/events.test.ts b/extensions/matrix/src/matrix/monitor/events.test.ts index 24e8250b9e8..51dc802b080 100644 --- a/extensions/matrix/src/matrix/monitor/events.test.ts +++ b/extensions/matrix/src/matrix/monitor/events.test.ts @@ -1809,7 +1809,7 @@ describe("registerMatrixMonitorEvents verification routing", () => { } }); - it("does not throw when getUserId fails during decrypt guidance lookup", async () => { + it("logs decrypt guidance when getUserId fails during lookup", async () => { const { logger, logVerboseMessage, failedDecryptListener } = createHarness({ accountId: "ops", selfUserIdError: new Error("lookup failed"), diff --git a/extensions/twitch/src/twitch-client.test.ts b/extensions/twitch/src/twitch-client.test.ts index cefc333b29a..0f279ba79d6 100644 --- a/extensions/twitch/src/twitch-client.test.ts +++ b/extensions/twitch/src/twitch-client.test.ts @@ -295,7 +295,7 @@ describe("TwitchClientManager", () => { }); it("should handle disconnecting non-existent client gracefully", async () => { - // disconnect doesn't throw, just does nothing + // Missing clients are ignored. await manager.disconnect(testAccount); expect(mockQuit).not.toHaveBeenCalled(); }); @@ -326,7 +326,7 @@ describe("TwitchClientManager", () => { }); it("should handle empty client list gracefully", async () => { - // disconnectAll doesn't throw, just does nothing + // Empty client sets are ignored. await manager.disconnectAll(); expect(mockQuit).not.toHaveBeenCalled(); }); diff --git a/src/gateway/server-http.hooks-request-timeout.test.ts b/src/gateway/server-http.hooks-request-timeout.test.ts index 60fa0aec4b2..eaf2f107f83 100644 --- a/src/gateway/server-http.hooks-request-timeout.test.ts +++ b/src/gateway/server-http.hooks-request-timeout.test.ts @@ -96,7 +96,7 @@ describe("createHooksRequestHandler timeout status mapping", () => { }); test.each(["0.0.0.0", "::"])( - "does not throw when bindHost=%s while parsing non-hook request URL", + "returns unhandled when bindHost=%s sees a non-hook request URL", async (bindHost) => { const handler = createHooksHandler({ bindHost }); const req = createHookRequest({ url: "/" }); diff --git a/src/oc-path/tests/scenarios/frontmatter-edges.test.ts b/src/oc-path/tests/scenarios/frontmatter-edges.test.ts index a75746067ea..8183bada2fc 100644 --- a/src/oc-path/tests/scenarios/frontmatter-edges.test.ts +++ b/src/oc-path/tests/scenarios/frontmatter-edges.test.ts @@ -2,8 +2,8 @@ * Wave 2 — frontmatter edges. * * Substrate guarantee: frontmatter is parsed as `key: value` entries - * with quote-stripping; malformed frontmatter doesn't crash the parser - * (soft-error policy: emit diagnostic, recover). + * with quote-stripping; malformed frontmatter follows the soft-error + * policy by emitting diagnostics and recovering. */ import { describe, expect, it } from "vitest"; import { parseMd } from "../../parse.js";